Add an option to fail if no tests are found (#3672)

* add tests for `--failWithNoTests`

* invert the logic

Author: EnoahNetzach

integration_tests/__tests__/config.test.js

@@ -20,7 +20,7 @@ test('config as JSON', () => {
   ]);
   const stdout = result.stdout.toString();
 
-  expect(result.status).toBe(0);
+  expect(result.status).toBe(1);
   expect(stdout).toMatch('No tests found');
 });
 

integration_tests/__tests__/pass_with_no_tests-test.js

@@ -0,0 +1,40 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the BSD-style license found in the
+ * LICENSE file in the root directory of this source tree. An additional grant
+ * of patent rights can be found in the PATENTS file in the same directory.
+ *
+ * @flow
+ */
+
+'use strict';
+
+const path = require('path');
+const runJest = require('../runJest');
+
+const DIR = path.resolve(__dirname, '../pass_with_no_tests-test');
+
+describe('jest --passWithNoTests', () => {
+  test('fails the test suite if no files are found', () => {
+    const result = runJest(DIR, ['--testPathPattern', '/non/existing/path/']);
+    const status = result.status;
+    const stdout = result.stdout.toString();
+
+    expect(stdout).toMatch('No tests found');
+    expect(status).toBe(1);
+  });
+
+  test("doesn't fail the test suite if no files are found", () => {
+    const result = runJest(DIR, [
+      '--testPathPattern',
+      '/non/existing/path/',
+      '--passWithNoTests',
+    ]);
+    const status = result.status;
+    const stdout = result.stdout.toString();
+
+    expect(stdout).toMatch('No tests found');
+    expect(status).toBe(0);
+  });
+});

integration_tests/pass_with_no_tests-test/package.json

@@ -0,0 +1,5 @@
+{
+  "jest": {
+    "testEnvironment": "node"
+  }
+}

packages/jest-cli/src/cli/args.js

@@ -340,6 +340,12 @@ export const options = {
       'also specified.',
     type: 'string',
   },
+  passWithNoTests: {
+    default: false,
+    description:
+      'Will not fail if no tests are found (for example while using `--testPathPattern`.)',
+    type: 'boolean',
+  },
   preset: {
     description: "A preset that is used as a base for Jest's configuration.",
     type: 'string',

packages/jest-cli/src/run_jest.js

@@ -124,9 +124,18 @@ export default async function runJest({
   }
 
   if (!allTests.length) {
-    new Console(outputStream, outputStream).log(
-      getNoTestsFoundMessage(testRunData, globalConfig),
+    const noTestsFoundMessage = getNoTestsFoundMessage(
+      testRunData,
+      globalConfig,
     );
+
+    if (globalConfig.passWithNoTests) {
+      new Console(outputStream, outputStream).log(noTestsFoundMessage);
+    } else {
+      new Console(outputStream, outputStream).error(noTestsFoundMessage);
+
+      process.exit(1);
+    }
   } else if (
     allTests.length === 1 &&
     globalConfig.silent !== true &&

packages/jest-config/src/index.js

@@ -96,6 +96,7 @@ const getConfigs = (
       notify: options.notify,
       onlyChanged: options.onlyChanged,
       outputFile: options.outputFile,
+      passWithNoTests: options.passWithNoTests,
       projects: options.projects,
       replname: options.replname,
       reporters: options.reporters,

test_utils.js

@@ -35,6 +35,7 @@ const DEFAULT_GLOBAL_CONFIG: GlobalConfig = {
   notify: false,
   onlyChanged: false,
   outputFile: null,
+  passWithNoTests: false,
   projects: [],
   replname: null,
   reporters: [],

types/Config.js

@@ -165,6 +165,7 @@ export type GlobalConfig = {|
   notify: boolean,
   outputFile: ?Path,
   onlyChanged: boolean,
+  passWithNoTests: boolean,
   projects: Array<Glob>,
   replname: ?string,
   reporters: Array<ReporterConfig>,