#121 - replace only the end occurrence of a forward slash along with …

…any possible flags
fastify · Apr 27, 2021 · 290123b · 290123b
1 parent c1f4f7d
commit 290123b
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 6 deletions.
diff --git a/src/StringSchema.js b/src/StringSchema.js
@@ -99,7 +99,7 @@ const StringSchema = (
       pattern = pattern
         .toString()
         .substr(1)
-        .replace(`/${flags}`, '')
+        .replace(new RegExp(`/${flags}$`), '')
     }
 
     return setAttribute({ schema, ...options }, ['pattern', pattern, 'string'])

diff --git a/src/StringSchema.test.js b/src/StringSchema.test.js
@@ -97,22 +97,34 @@ describe('StringSchema', () => {
       it('as a string', () => {
         expect(
           StringSchema()
-            .pattern('.*')
+            .pattern('\\/.*\\/')
             .valueOf()
         ).toEqual({
           type: 'string',
-          pattern: '.*',
+          pattern: '\\/.*\\/',
         })
       })
-      it('as a regex', () => {
+      it('as a regex without flags', () => {
         const prop = 'prop'
         expect(
           StringSchema()
-            .pattern(/.*/gi)
+            .pattern(/\/.*\//)
             .valueOf()
         ).toEqual({
           type: 'string',
-          pattern: '.*',
+          pattern: '\\/.*\\/',
+        })
+      })
+
+      it('as a regex with flags', () => {
+        const prop = 'prop'
+        expect(
+          StringSchema()
+            .pattern(/\/.*\//gi)
+            .valueOf()
+        ).toEqual({
+          type: 'string',
+          pattern: '\\/.*\\/',
         })
       })