Exclude files from running if they match with the ignoring pattern (when run via file mode) (#5341)

Author: mjesun

CHANGELOG.md

@@ -1,6 +1,14 @@
 ## master
 
-* `[jest-editor-support]` Add option to spawn command in shell ([#5340](https://github.com/facebook/jest/pull/5340))
+## jest 22.1.3
+
+### Fixes
+
+* `[jest-cli]` Check if the file belongs to the checked project before adding it
+  to the list, also checking that the file name is not explicitly blacklisted
+  ([#5341](https://github.com/facebook/jest/pull/5341))
+* `[jest-editor-support]` Add option to spawn command in shell
+  ([#5340](https://github.com/facebook/jest/pull/5340))
 
 ## jest 22.1.2
 

integration-tests/__tests__/execute-tests-once-in-mpr.js

@@ -25,8 +25,9 @@ test('Tests are executed only once even in an MPR', () => {
   // Make a global config that ignores all sub-projects.
   const config = {
     jest: {
-      projects: ['<rootDir>/foo/*/'],
+      projects: ['<rootDir>', '<rootDir>/foo/*/'],
       testPathIgnorePatterns: ['/foo/'],
+      testRegex: /my-test-.*\.js/.source,
     },
   };
 

packages/jest-cli/src/search_source.js

@@ -213,18 +213,25 @@ export default class SearchSource {
       const validTestPaths =
         paths &&
         paths.filter(name => {
+          const fullName = path.resolve(name);
+
           try {
-            if (!fs.lstatSync(name).isFile()) {
-              // It exists, but it is not a file; return false.
+            if (!fs.lstatSync(fullName).isFile()) {
+              // It exists, but it is not a file.
               return false;
             }
           } catch (e) {
-            // It does not exist; return false.
+            // It does not exist.
+            return false;
+          }
+
+          // The file exists, but it is explicitly blacklisted.
+          if (!this._testPathCases.testPathIgnorePatterns(fullName)) {
             return false;
           }
 
           // It exists and it is a file; return true if it's in the project.
-          return allFiles.has(path.resolve(name));
+          return allFiles.has(fullName);
         });
 
       if (validTestPaths && validTestPaths.length) {