Fix handling regex symbols in tests path on Windows

jestjs · Apr 7, 2018 · d331af8 · d331af8
1 parent 6979b8e
commit d331af8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/integration-tests/__tests__/regex_(char_in_path.test.js b/integration-tests/__tests__/regex_(char_in_path.test.js
@@ -8,12 +8,9 @@
  */
 'use strict';
 
-const SkipOnWindows = require('../../scripts/SkipOnWindows');
 const runJest = require('../runJest');
 
 describe('Regex Char In Path', () => {
-  SkipOnWindows.suite();
-
   it('parses paths containing regex chars correctly', () => {
     const {json} = runJest.json('regex-(char-in-path', []);
 

diff --git a/packages/jest-regex-util/src/__tests__/index.test.js b/packages/jest-regex-util/src/__tests__/index.test.js
@@ -39,5 +39,9 @@ describe('replacePathSepForRegex()', () => {
         'a\\\\\\\\\\.dotfile',
       );
     });
+
+    it('should not escape an escaped regexp symbol', () => {
+      expect(replacePathSepForRegex('b\\(86')).toBe('b\\(86');
+    });
   });
 });
diff --git a/packages/jest-regex-util/src/index.js b/packages/jest-regex-util/src/index.js
@@ -23,7 +23,7 @@ export const escapeStrForRegex = (string: string) =>
 
 export const replacePathSepForRegex = (string: string) => {
   if (path.sep === '\\') {
-    return string.replace(/(\/|\\(?!\.))/g, '\\\\');
+    return string.replace(/(\/|\\(?![[\]{}()*+?.^$|]))/g, '\\\\');
   }
   return string;
 };