Fix mistake as test files when run coverage issue.

CHANGELOG.md

@@ -91,6 +91,7 @@
 - `[jest-cli]` Fix to set prettierPath via config file ([#7412](https://github.com/facebook/jest/pull/7412))
 - `[jest-cli]` Support dashed args ([#7497](https://github.com/facebook/jest/pull/7497))
 - `[jest-cli]` Fix to run in band tests if watch mode enable when runInBand arg used ([#7518](https://github.com/facebook/jest/pull/7518))
+- `[jest-runtime]` Fix mistake as test files when run coverage issue. ([#7506](https://github.com/facebook/jest/pull/7506))
 
 ### Chore & Maintenance
 

packages/jest-runtime/src/__tests__/should_instrument.test.js

@@ -27,7 +27,7 @@ describe('should_instrument', () => {
       expect(result).toBe(true);
     };
 
-    it('when testRegex provided and file is not a test file', () => {
+    it('when testRegex is provided and file is not a test file', () => {
       testShouldInstrument('source_file.js', defaultOptions, {
         testRegex: ['.*\\.(test)\\.(js)$'],
       });
@@ -45,6 +45,32 @@ describe('should_instrument', () => {
       });
     });
 
+    it('when testPathIgnorePatterns is provided and file is not a test file', () => {
+      testShouldInstrument('src/test.js', defaultOptions, {
+        testPathIgnorePatterns: ['src/'],
+      });
+    });
+
+    it('when more than one testPathIgnorePatterns is provided and filename is not a test file', () => {
+      testShouldInstrument('src/test.js', defaultOptions, {
+        testPathIgnorePatterns: ['test/', 'src/'],
+      });
+    });
+
+    it('when testRegex and testPathIgnorePatterns are provided and file is not a test file', () => {
+      testShouldInstrument('src/source_file.js', defaultOptions, {
+        testPathIgnorePatterns: ['test/'],
+        testRegex: ['.*\\.(test)\\.(js)$'],
+      });
+    });
+
+    it('when testMatch and testPathIgnorePatterns are provided and file is not a test file', () => {
+      testShouldInstrument('src/source_file.js', defaultOptions, {
+        testMatch: ['**/?(*.)(test).js', '!**/dont/**/*.js'],
+        testPathIgnorePatterns: ['test/'],
+      });
+    });
+
     it('should return true when file is in collectCoverageOnlyFrom when provided', () => {
       testShouldInstrument(
         'collect/only/from/here.js',
@@ -119,7 +145,7 @@ describe('should_instrument', () => {
       );
     });
 
-    it('when testRegex provided and filename is a test file', () => {
+    it('when testRegex is provided and filename is a test file', () => {
       testShouldInstrument(defaultFilename, defaultOptions, {
         testRegex: ['.*\\.(test)\\.(js)$'],
       });
@@ -137,6 +163,20 @@ describe('should_instrument', () => {
       });
     });
 
+    it('when testRegex and testPathIgnorePatterns are provided and filename is a test file', () => {
+      testShouldInstrument('test/' + defaultFilename, defaultOptions, {
+        testPathIgnorePatterns: ['src/'],
+        testRegex: ['.*\\.(test)\\.(js)$'],
+      });
+    });
+
+    it('when testMatch and testPathIgnorePatterns are provided and file is a test file', () => {
+      testShouldInstrument('test/' + defaultFilename, defaultOptions, {
+        testMatch: ['**/?(*.)(test).js'],
+        testPathIgnorePatterns: ['src/'],
+      });
+    });
+
     it('when file is not in collectCoverageOnlyFrom when provided', () => {
       testShouldInstrument(
         'source_file.js',

packages/jest-runtime/src/should_instrument.js

@@ -36,18 +36,23 @@ export default function shouldInstrument(
   }
 
   if (
-    config.testRegex &&
-    config.testRegex.some(regex => new RegExp(regex).test(filename))
+    !config.testPathIgnorePatterns ||
+    !config.testPathIgnorePatterns.some(pattern => filename.match(pattern))
   ) {
-    return false;
-  }
+    if (
+      config.testRegex &&
+      config.testRegex.some(regex => new RegExp(regex).test(filename))
+    ) {
+      return false;
+    }
 
-  if (
-    config.testMatch &&
-    config.testMatch.length &&
-    micromatch([filename], config.testMatch).length
-  ) {
-    return false;
+    if (
+      config.testMatch &&
+      config.testMatch.length &&
+      micromatch([filename], config.testMatch).length
+    ) {
+      return false;
+    }
   }
 
   if (