Only match relative paths when specifying paths

CHANGELOG.md

@@ -19,6 +19,9 @@
 - `[jest-leak-detector]` Make leak-detector more aggressive when running GC ([#14526](https://github.com/jestjs/jest/pull/14526))
 - `[jest-util]` Make sure `isInteractive` works in a browser ([#14552](https://github.com/jestjs/jest/pull/14552))
 - `[pretty-format]` [**BREAKING**] Print `ArrayBuffer` and `DataView` correctly ([#14290](https://github.com/facebook/jest/pull/14290))
+- `[jest-cli]` When specifying paths on the command line, only match against the relative paths of the test files ([#12519](https://github.com/facebook/jest/pull/12519))
+  - [**BREAKING**] Changes `testPathPattern` configuration option to `testPathPatterns`, which now takes a list of patterns instead of the regex.
+  - [**BREAKING**] `--testPathPattern` is now `--testPathPatterns`
 
 ### Performance
 

docs/Configuration.md

@@ -787,7 +787,7 @@ While code transformation is applied to the linked setup-file, Jest will **not**
 
 ```js title="setup.js"
 module.exports = async function (globalConfig, projectConfig) {
-  console.log(globalConfig.testPathPattern);
+  console.log(globalConfig.testPathPatterns);
   console.log(projectConfig.cache);
 
   // Set reference to mongod in order to close the server during teardown.
@@ -797,7 +797,7 @@ module.exports = async function (globalConfig, projectConfig) {
 
 ```js title="teardown.js"
 module.exports = async function (globalConfig, projectConfig) {
-  console.log(globalConfig.testPathPattern);
+  console.log(globalConfig.testPathPatterns);
   console.log(projectConfig.cache);
 
   await globalThis.__MONGOD__.stop();

docs/WatchPlugins.md

@@ -172,7 +172,7 @@ For stability and safety reasons, only part of the global configuration keys can
 - [`onlyFailures`](configuration#onlyfailures-boolean)
 - [`reporters`](configuration#reporters-arraymodulename--modulename-options)
 - [`testNamePattern`](cli#--testnamepatternregex)
-- [`testPathPattern`](cli#--testpathpatternregex)
+- [`testPathPatterns`](cli#--testpathpatternregex)
 - [`updateSnapshot`](cli#--updatesnapshot)
 - [`verbose`](configuration#verbose-boolean)
 

e2e/__tests__/__snapshots__/showConfig.test.ts.snap

@@ -138,7 +138,7 @@ exports[`--showConfig outputs config info and exits 1`] = `
       "printBasicPrototype": false
     },
     "testFailureExitCode": 1,
-    "testPathPattern": "",
+    "testPathPatterns": [],
     "testSequencer": "<<REPLACED_JEST_PACKAGES_DIR>>/jest-test-sequencer/build/index.js",
     "updateSnapshot": "none",
     "useStderr": false,

e2e/__tests__/globalSetup.test.ts

@@ -56,7 +56,7 @@ test('globalSetup is triggered once before all test suites', () => {
   const setupPath = path.join(e2eDir, 'setup.js');
   const result = runWithJson(e2eDir, [
     `--globalSetup=${setupPath}`,
-    '--testPathPattern=__tests__',
+    '--testPathPatterns=__tests__',
   ]);
 
   expect(result.exitCode).toBe(0);
@@ -70,7 +70,7 @@ test('jest throws an error when globalSetup does not export a function', () => {
   const setupPath = path.resolve(__dirname, '../global-setup/invalidSetup.js');
   const {exitCode, stderr} = runJest(e2eDir, [
     `--globalSetup=${setupPath}`,
-    '--testPathPattern=__tests__',
+    '--testPathPatterns=__tests__',
   ]);
 
   expect(exitCode).toBe(1);
@@ -83,15 +83,13 @@ test('jest throws an error when globalSetup does not export a function', () => {
 test('globalSetup function gets global config object and project config as parameters', () => {
   const setupPath = path.resolve(e2eDir, 'setupWithConfig.js');
 
-  const testPathPattern = 'pass';
-
   const result = runJest(e2eDir, [
     `--globalSetup=${setupPath}`,
-    `--testPathPattern=${testPathPattern}`,
+    '--testPathPatterns=pass',
     '--cache=true',
   ]);
 
-  expect(result.stdout).toBe(`${testPathPattern}\ntrue`);
+  expect(result.stdout).toBe("[ 'pass' ]\ntrue");
 });
 
 test('should call globalSetup function of multiple projects', () => {
@@ -111,7 +109,7 @@ test('should not call a globalSetup of a project if there are no tests to run fr
 
   const result = runWithJson(e2eDir, [
     `--config=${configPath}`,
-    '--testPathPattern=project-1',
+    '--testPathPatterns=project-1',
   ]);
 
   expect(result.exitCode).toBe(0);
@@ -140,23 +138,21 @@ test('should not call any globalSetup if there are no tests to run', () => {
 test('globalSetup works with default export', () => {
   const setupPath = path.resolve(e2eDir, 'setupWithDefaultExport.js');
 
-  const testPathPattern = 'pass';
-
   const result = runJest(e2eDir, [
     `--globalSetup=${setupPath}`,
-    `--testPathPattern=${testPathPattern}`,
+    '--testPathPatterns=pass',
     '--cache=true',
   ]);
 
-  expect(result.stdout).toBe(`${testPathPattern}\ntrue`);
+  expect(result.stdout).toBe("[ 'pass' ]\ntrue");
 });
 
 test('globalSetup throws with named export', () => {
   const setupPath = path.resolve(e2eDir, 'invalidSetupWithNamedExport.js');
 
   const {exitCode, stderr} = runJest(e2eDir, [
     `--globalSetup=${setupPath}`,
-    '--testPathPattern=__tests__',
+    '--testPathPatterns=__tests__',
   ]);
 
   expect(exitCode).toBe(1);

e2e/__tests__/globalTeardown.test.ts

@@ -40,7 +40,7 @@ test('globalTeardown is triggered once after all test suites', () => {
   const teardownPath = path.resolve(e2eDir, 'teardown.js');
   const result = runWithJson('global-teardown', [
     `--globalTeardown=${teardownPath}`,
-    '--testPathPattern=__tests__',
+    '--testPathPatterns=__tests__',
   ]);
 
   expect(result.exitCode).toBe(0);
@@ -54,7 +54,7 @@ test('jest throws an error when globalTeardown does not export a function', () =
   const teardownPath = path.resolve(e2eDir, 'invalidTeardown.js');
   const {exitCode, stderr} = runJest(e2eDir, [
     `--globalTeardown=${teardownPath}`,
-    '--testPathPattern=__tests__',
+    '--testPathPatterns=__tests__',
   ]);
 
   expect(exitCode).toBe(1);
@@ -67,15 +67,13 @@ test('jest throws an error when globalTeardown does not export a function', () =
 test('globalSetup function gets global config object and project config as parameters', () => {
   const teardownPath = path.resolve(e2eDir, 'teardownWithConfig.js');
 
-  const testPathPattern = 'pass';
-
   const result = runJest(e2eDir, [
     `--globalTeardown=${teardownPath}`,
-    `--testPathPattern=${testPathPattern}`,
+    '--testPathPatterns=pass',
     '--cache=true',
   ]);
 
-  expect(result.stdout).toBe(`${testPathPattern}\ntrue`);
+  expect(result.stdout).toBe("[ 'pass' ]\ntrue");
 });
 
 test('should call globalTeardown function of multiple projects', () => {
@@ -95,7 +93,7 @@ test('should not call a globalTeardown of a project if there are no tests to run
 
   const result = runWithJson('global-teardown', [
     `--config=${configPath}`,
-    '--testPathPattern=project-1',
+    '--testPathPatterns=project-1',
   ]);
 
   expect(result.exitCode).toBe(0);
@@ -108,15 +106,13 @@ test('should not call a globalTeardown of a project if there are no tests to run
 test('globalTeardown works with default export', () => {
   const teardownPath = path.resolve(e2eDir, 'teardownWithDefaultExport.js');
 
-  const testPathPattern = 'pass';
-
   const result = runJest(e2eDir, [
     `--globalTeardown=${teardownPath}`,
-    `--testPathPattern=${testPathPattern}`,
+    '--testPathPatterns=pass',
     '--cache=true',
   ]);
 
-  expect(result.stdout).toBe(`${testPathPattern}\ntrue`);
+  expect(result.stdout).toBe("[ 'pass' ]\ntrue");
 });
 
 test('globalTeardown throws with named export', () => {
@@ -127,7 +123,7 @@ test('globalTeardown throws with named export', () => {
 
   const {exitCode, stderr} = runJest(e2eDir, [
     `--globalTeardown=${teardownPath}`,
-    '--testPathPattern=__tests__',
+    '--testPathPatterns=__tests__',
   ]);
 
   expect(exitCode).toBe(1);

e2e/__tests__/multiProjectRunner.test.ts

@@ -155,7 +155,7 @@ test('"No tests found" message for projects', () => {
     'project1',
     'project2',
   ]);
-  expect(verboseOutput).toContain('Pattern: xyz321 - 0 matches');
+  expect(verboseOutput).toContain('Pattern: /xyz321/i - 0 matches');
   const {stdout} = runJest(DIR, [
     '--no-watchman',
     'xyz321',

e2e/__tests__/noTestsFound.test.ts

@@ -13,7 +13,7 @@ const DIR = path.resolve(__dirname, '../no-tests-found-test');
 describe('No tests are found', () => {
   test('fails the test suite in standard situation', () => {
     const {exitCode, stdout} = runJest(DIR, [
-      '--testPathPattern',
+      '--testPathPatterns',
       '/non/existing/path/',
     ]);
 
@@ -26,7 +26,7 @@ describe('No tests are found', () => {
 
   test("doesn't fail the test suite if --passWithNoTests passed", () => {
     const {exitCode, stdout} = runJest(DIR, [
-      '--testPathPattern',
+      '--testPathPatterns',
       '/non/existing/path/',
       '--passWithNoTests',
     ]);

e2e/__tests__/onlyChanged.test.ts

@@ -246,7 +246,7 @@ test('collect test coverage when using onlyChanged', () => {
   expect(exitCode).toBe(0);
 });
 
-test('onlyChanged in config is overwritten by --all or testPathPattern', () => {
+test('onlyChanged in config is overwritten by --all or testPathPatterns', () => {
   writeFiles(DIR, {
     '.watchmanconfig': '',
     '__tests__/file1.test.js': "require('../file1'); test('file1', () => {});",

e2e/__tests__/testPathPatternReporterMessage.test.ts

@@ -30,9 +30,9 @@ test('prints a message with path pattern at the end', () => {
   ({stderr} = runJest(DIR, ['a', 'b']));
   expect(stderr).toMatch('Ran all test suites matching /a|b/i');
 
-  ({stderr} = runJest(DIR, ['--testPathPattern', 'a']));
+  ({stderr} = runJest(DIR, ['--testPathPatterns', 'a']));
   expect(stderr).toMatch('Ran all test suites matching /a/i');
 
-  ({stderr} = runJest(DIR, ['--testPathPattern', 'a|b']));
+  ({stderr} = runJest(DIR, ['--testPathPatterns', 'a|b']));
   expect(stderr).toMatch('Ran all test suites matching /a|b/i');
 });

e2e/global-setup/invalidSetupWithNamedExport.js

@@ -6,7 +6,7 @@
  */
 
 function invalidSetupWithNamedExport(jestConfig): void {
-  console.log(jestConfig.testPathPattern);
+  console.log(jestConfig.testPathPatterns);
 }
 
 export {invalidSetupWithNamedExport};

e2e/global-setup/setupWithConfig.js

@@ -6,6 +6,6 @@
  */
 
 module.exports = function (globalConfig, projectConfig) {
-  console.log(globalConfig.testPathPattern);
+  console.log(globalConfig.testPathPatterns);
   console.log(projectConfig.cache);
 };

e2e/global-setup/setupWithDefaultExport.js

@@ -6,6 +6,6 @@
  */
 
 export default function (globalConfig, projectConfig): void {
-  console.log(globalConfig.testPathPattern);
+  console.log(globalConfig.testPathPatterns);
   console.log(projectConfig.cache);
 }

e2e/global-teardown/invalidTeardownWithNamedExport.js

@@ -6,7 +6,7 @@
  */
 
 function invalidTeardownWithNamedExport(jestConfig): void {
-  console.log(jestConfig.testPathPattern);
+  console.log(jestConfig.testPathPatterns);
 }
 
 export {invalidTeardownWithNamedExport};

e2e/global-teardown/teardownWithConfig.js

@@ -6,6 +6,6 @@
  */
 
 module.exports = function (globalConfig, projectConfig) {
-  console.log(globalConfig.testPathPattern);
+  console.log(globalConfig.testPathPatterns);
   console.log(projectConfig.cache);
 };

e2e/global-teardown/teardownWithDefaultExport.js

@@ -6,6 +6,6 @@
  */
 
 export default function (globalConfig, projectConfig): void {
-  console.log(globalConfig.testPathPattern);
+  console.log(globalConfig.testPathPatterns);
   console.log(projectConfig.cache);
 }

packages/jest-cli/src/args.ts

@@ -97,7 +97,7 @@ export function check(argv: Config.Argv): true {
 }
 
 export const usage =
-  'Usage: $0 [--config=<pathToConfigFile>] [TestPathPattern]';
+  'Usage: $0 [--config=<pathToConfigFile>] [TestPathPatterns]';
 export const docs = 'Documentation: https://jestjs.io/';
 
 // The default values are all set in jest-config
@@ -431,7 +431,7 @@ export const options: {[key: string]: Options} = {
   },
   passWithNoTests: {
     description:
-      'Will not fail if no tests are found (for example while using `--testPathPattern`.)',
+      'Will not fail if no tests are found (for example while using `--testPathPatterns`.)',
     type: 'boolean',
   },
   preset: {
@@ -609,7 +609,7 @@ export const options: {[key: string]: Options} = {
     string: true,
     type: 'array',
   },
-  testPathPattern: {
+  testPathPatterns: {
     description:
       'A regexp pattern string that is matched against all tests ' +
       'paths before executing the test.',

packages/jest-config/src/__tests__/__snapshots__/normalize.test.ts.snap

@@ -482,9 +482,9 @@ exports[`testMatch throws if testRegex and testMatch are both specified 1`] = `
 <red></color>"
 `;
 
-exports[`testPathPattern <regexForTestFiles> ignores invalid regular expressions and logs a warning 1`] = `"<red>  Invalid testPattern a( supplied. Running all tests instead.</color>"`;
+exports[`testPathPatterns <regexForTestFiles> ignores invalid regular expressions and logs a warning 1`] = `"<red>  Invalid testPattern /a(/i supplied. Running all tests instead.</color>"`;
 
-exports[`testPathPattern --testPathPattern ignores invalid regular expressions and logs a warning 1`] = `"<red>  Invalid testPattern a( supplied. Running all tests instead.</color>"`;
+exports[`testPathPatterns --testPathPatterns ignores invalid regular expressions and logs a warning 1`] = `"<red>  Invalid testPattern /a(/i supplied. Running all tests instead.</color>"`;
 
 exports[`testTimeout should throw an error if timeout is a negative number 1`] = `
 "<red><bold><bold>● </intensity><bold>Validation Error</intensity>:</color>