fix instances of findRelatedTests

findRelatedTests used to be a boolean field. It has been updated to be Array<string>. We will check the length of the array to determine if findRelatedTests is falsy or truthy since 0 will evaluate to false and aything greater than 0 will evaluate to true.
jestjs · Apr 24, 2019 · 813fab3 · SimenB · Apr 24, 2019 · 813fab3
1 parent 9d7aed1
commit 813fab3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/jest-core/src/runJest.ts b/packages/jest-core/src/runJest.ts
@@ -212,7 +212,7 @@ export default (async function runJest({
 
     if (
       globalConfig.passWithNoTests ||
-      globalConfig.findRelatedTests ||
+      globalConfig.findRelatedTests.length ||
       globalConfig.lastCommit ||
       globalConfig.onlyChanged
     ) {

diff --git a/packages/jest-reporters/src/summary_reporter.ts b/packages/jest-reporters/src/summary_reporter.ts
@@ -189,7 +189,7 @@ export default class SummaryReporter extends BaseReporter {
     globalConfig: Config.GlobalConfig,
   ) {
     const getMatchingTestsInfo = () => {
-      const prefix = globalConfig.findRelatedTests
+      const prefix = globalConfig.findRelatedTests.length
         ? ' related to files matching '
         : ' matching ';