Make testFailureExitCode compatible with bail option

CHANGELOG.md

@@ -18,6 +18,7 @@
 
 ### Fixes
 
+- `[jest-cli]` use testFailureExitCode when bailing from a failed test
 - `[babel-plugin-jest-hoist]` Add `__dirname` and `__filename` to whitelisted globals ([#10903](https://github.com/facebook/jest/pull/10903))
 - `[expect]` [**BREAKING**] Revise `expect.not.objectContaining()` to be the inverse of `expect.objectContaining()`, as documented. ([#10708](https://github.com/facebook/jest/pull/10708))
 - `[expect]` [**BREAKING**] Make `toContain` more strict with the received type ([#10119](https://github.com/facebook/jest/pull/10119) & [#10929](https://github.com/facebook/jest/pull/10929))

e2e/__tests__/jestChangedFiles.test.ts

@@ -21,7 +21,7 @@ const HG = 'hg --config ui.username=jest_test';
 
 const gitVersionSupportsInitialBranch = (() => {
   const {stdout} = run(`${GIT} --version`);
-  const gitVersion = stdout.split(' ').slice(-1)[0];
+  const gitVersion = stdout;
 
   const match = gitVersion.match(/(?<version>\d+\.\d+\.\d+)/);
 

e2e/__tests__/onlyChanged.test.ts

@@ -17,7 +17,7 @@ const HG = 'hg --config ui.username=jest_test';
 
 const gitVersionSupportsInitialBranch = (() => {
   const {stdout} = run(`${GIT} --version`);
-  const gitVersion = stdout.split(' ').slice(-1)[0];
+  const gitVersion = stdout;
 
   const match = gitVersion.match(/(?<version>\d+\.\d+\.\d+)/);
 

e2e/__tests__/testFailureExitCode.test.ts

@@ -38,3 +38,29 @@ test('exits with a specified code when test fail', () => {
   ({exitCode} = runJest(DIR));
   expect(exitCode).toBe(1);
 });
+
+test('exits with a specified code when bailing from a failed test', () => {
+  writeFiles(DIR, {
+    '__tests__/test.test.js': `test('test', () => { expect(1).toBe(2); });`,
+    '__tests__/test2.test.js': `test('test2', () => { expect(1).toBe(2); });`,
+    'package.json': JSON.stringify({
+      jest: {testEnvironment: 'node', testFailureExitCode: 99},
+    }),
+  });
+
+  let {exitCode} = runJest(DIR, ['--bail']);
+  expect(exitCode).toBe(99);
+
+  ({exitCode} = runJest(DIR, ['--bail', '--testFailureExitCode', '77']));
+  expect(exitCode).toBe(77);
+
+  writeFiles(DIR, {
+    '__tests__/test.test.js': `test('test', () => { expect(1).toBe(2); });`,
+    '__tests__/test2.test.js': `test('test2', () => { expect(1).toBe(2); });`,
+    'package.json': JSON.stringify({
+      jest: {testEnvironment: 'node'},
+    }),
+  });
+  ({exitCode} = runJest(DIR));
+  expect(exitCode).toBe(1);
+});

packages/jest-core/src/TestScheduler.ts

@@ -436,7 +436,8 @@ export default class TestScheduler {
       try {
         await this._dispatcher.onRunComplete(contexts, aggregatedResults);
       } finally {
-        exit(1);
+        const exitCode = this._globalConfig.testFailureExitCode;
+        exit(exitCode);
       }
     }
   }

packages/pretty-format/src/__tests__/ReactElement.test.ts

@@ -15,7 +15,7 @@ setPrettyPrint([ReactElement]);
 
 describe('ReactElement Plugin', () => {
   let forwardRefComponent: {
-    (_props: unknown, _ref: unknown): unknown;
+    (_props: unknown, _ref: unknown): React.ReactElement | null;
     displayName?: string;
   };
 

packages/pretty-format/src/__tests__/react.test.tsx

@@ -706,7 +706,7 @@ test('ReactTestComponent plugin highlights syntax with color from theme option',
 });
 
 test('supports forwardRef with a child', () => {
-  function Cat(props: any) {
+  function Cat(props: any, _ref: any) {
     return React.createElement('div', props, props.children);
   }