RemoveModifier Test

Author: teabyii

Diff for: lib/Builder.js

@@ -639,7 +639,8 @@ class Builder {
      * @return {Builder}
      */
     removeModifier(flag) {
-        this._modifiers.replace(flag, '')
+        this._modifiers = this._modifiers.replace(flag, '')
+        this._result = null
 
         return this
     }

Diff for: test/builder-test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const assert = require('assert')
-const SRL = require('../lib/Builder')
+const SRL = require('../')
 
 describe('Builder isMatching', () => {
     it('Simple Phone Number Format', () => {
@@ -169,4 +169,13 @@ describe('Builder isMatching', () => {
         assert.ok(!regex.isMatching('fooa'))
         assert.ok(!regex.isMatching('foobar'))
     })
+
+    it('Remove modifier', () => {
+        const regex = new SRL()
+            .literally('foo')
+            .removeModifier('g')
+            .get()
+
+        assert.deepEqual(regex, /(?:foo)/)
+    })
 })