Fix SimpleRegex#19, add test case

AuxControl · May 17, 2021 · a313f8e · a313f8e
1 parent 79a8fd1
commit a313f8e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/lib/Builder.js b/lib/Builder.js
@@ -704,6 +704,8 @@ class Builder {
  clone._modifiers = this._modifiers
  clone._lastMethodType = this._lastMethodType
  clone._group = this._group
+ clone._captureNames = Array.from(this._captureNames)
+ clone._implodeString = this.implodeString
 
  return clone
  }

diff --git a/test/builder-test.js b/test/builder-test.js
@@ -178,4 +178,18 @@ describe('Builder isMatching', () => {
 
  assert.deepEqual(regex, /(?:foo)/)
  })
+
+ it('Stores named captures', () => {
+ const regex_new = new SRL("capture (anything once or more) as first");
+ const testcase = 'hello world';
+
+ let matches_new = regex_new.getMatch(testcase)
+ assert.equal(matches_new["first"], 'hello world')
+
+ const regex_cached = new SRL("capture (anything once or more) as first");
+
+ let matches_cached = regex_cached.getMatch(testcase)
+ assert.equal(matches_cached["first"], 'hello world')
+
+ })
 })