Correct summary message for flag --findRelatedTests. (#4309)

* fixes #4247 * added tests for findRelatedTests summary message
jestjs · Aug 21, 2017 · cb6bcba · cb6bcba
1 parent 90b5c05
commit cb6bcba
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/integration_tests/__tests__/find_related_files.test.js b/integration_tests/__tests__/find_related_files.test.js
@@ -39,4 +39,7 @@ test('runs tests related to filename', () => {
 
   const {stderr} = runJest(DIR, ['--findRelatedTests', 'a.js']);
   expect(stderr).toMatch('PASS  __tests__/test.test.js');
+
+  const summaryMsg = 'Ran all test suites related to files matching /a.js/i.';
+  expect(stderr).toMatch(summaryMsg);
 });
diff --git a/packages/jest-cli/src/reporters/summary_reporter.js b/packages/jest-cli/src/reporters/summary_reporter.js
@@ -240,12 +240,20 @@ class SummaryReporter extends BaseReporter {
   }
 
   _getTestSummary(contexts: Set<Context>, globalConfig: GlobalConfig) {
+    const getMatchingTestsInfo = () => {
+      const prefix = globalConfig.findRelatedTests
+        ? ' related to files matching '
+        : ' matching ';
+
+      return (
+        chalk.dim(prefix) +
+        testPathPatternToRegExp(globalConfig.testPathPattern).toString()
+      );
+    };
+
     const testInfo = globalConfig.onlyChanged
       ? chalk.dim(' related to changed files')
-      : globalConfig.testPathPattern
-        ? chalk.dim(' matching ') +
-          testPathPatternToRegExp(globalConfig.testPathPattern).toString()
-        : '';
+      : globalConfig.testPathPattern ? getMatchingTestsInfo() : '';
 
     const nameInfo = globalConfig.testNamePattern
       ? chalk.dim(' with tests matching ') + `"${globalConfig.testNamePattern}"`