feat(jest-config): openHandlesTimeout config option

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ### Features
 
+- `[jest-config]` Add `openHandlesTimeout` option to configure possible open handles warning. ([#13875](https://github.com/facebook/jest/pull/13875))
 - `[jest-message-util]` Add support for [error causes](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error/cause)
 
 ### Fixes

docs/CLI.md

@@ -306,6 +306,10 @@ Activates notifications for test results. Good for when you don't want your cons
 
 Alias: `-o`. Attempts to identify which tests to run based on which files have changed in the current repository. Only works if you're running tests in a git/hg repository at the moment and requires a static dependency graph (ie. no dynamic requires).
 
+### `--openHandlesTimeout=<milliseconds>`
+
+When `--detectOpenHandles` and `--forceExit` are *disabled*, Jest will print a warning if the process has not exited cleanly after this number of milliseconds. A value of `0` disables the warning. Defaults to `1000`.
+
 ### `--outputFile=<filename>`
 
 Write test results to a file when the `--json` option is also specified. The returned JSON structure is documented in [testResultsProcessor](Configuration.md#testresultsprocessor-string).

e2e/__tests__/__snapshots__/detectOpenHandles.ts.snap

@@ -7,7 +7,13 @@ exports[`prints message about flag on forceExit 1`] = `"Force exiting Jest: Have
 exports[`prints message about flag on slow tests 1`] = `
 "Jest did not exit one second after the test run has completed.
 
-This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with \`--detectOpenHandles\` to troubleshoot this issue."
+'This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with \`--detectOpenHandles\` to troubleshoot this issue."
+`;
+
+exports[`prints message about flag on slow tests with a custom timeout 1`] = `
+"Jest did not exit 0.5 seconds after the test run has completed.
+
+'This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with \`--detectOpenHandles\` to troubleshoot this issue."
 `;
 
 exports[`prints out info about open handlers 1`] = `

e2e/__tests__/detectOpenHandles.ts

@@ -29,6 +29,17 @@ it('prints message about flag on slow tests', async () => {
   expect(textAfterTest).toMatchSnapshot();
 });
 
+it('prints message about flag on slow tests with a custom timeout', async () => {
+  const run = runContinuous('detect-open-handles', ['outside', '--openHandlesTimeout=500']);
+  await run.waitUntil(({stderr}) =>
+    stderr.includes('Jest did not exit 0.5 seconds'),
+  );
+  const {stderr} = await run.end();
+  const textAfterTest = getTextAfterTest(stderr);
+
+  expect(textAfterTest).toMatchSnapshot();
+});
+
 it('prints message about flag on forceExit', async () => {
   const run = runContinuous('detect-open-handles', ['outside', '--forceExit']);
   await run.waitUntil(({stderr}) => stderr.includes('Force exiting Jest'));

packages/jest-cli/src/run.ts

@@ -132,18 +132,24 @@ const readResultsAndExit = (
     }
 
     exit(code);
-  } else if (!globalConfig.detectOpenHandles) {
+  } else if (
+    !globalConfig.detectOpenHandles &&
+    globalConfig.openHandlesTimeout !== 0
+  ) {
+    const timeout = globalConfig.openHandlesTimeout;
     setTimeout(() => {
       console.warn(
         chalk.yellow.bold(
-          'Jest did not exit one second after the test run has completed.\n\n',
+          `Jest did not exit ${
+            timeout === 1000 ? 'one second' : `${timeout / 1000} seconds`
+          } after the test run has completed.\n\n'`,
         ) +
           chalk.yellow(
             'This usually means that there are asynchronous operations that ' +
               "weren't stopped in your tests. Consider running Jest with " +
               '`--detectOpenHandles` to troubleshoot this issue.',
           ),
       );
-    }, 1000).unref();
+    }, timeout).unref();
   }
 };

packages/jest-config/src/Defaults.ts

@@ -60,6 +60,7 @@ const defaultOptions: Config.DefaultOptions = {
   noStackTrace: false,
   notify: false,
   notifyMode: 'failure-change',
+  openHandlesTimeout: 1000,
   passWithNoTests: false,
   prettierPath: 'prettier',
   resetMocks: false,

packages/jest-config/src/ValidConfig.ts

@@ -114,6 +114,7 @@ export const initialOptions: Config.InitialOptions = {
   noStackTrace: false,
   notify: false,
   notifyMode: 'failure-change',
+  openHandlesTimeout: 1000,
   onlyChanged: false,
   onlyFailures: false,
   passWithNoTests: false,
@@ -261,6 +262,7 @@ export const initialProjectOptions: Config.InitialProjectOptions = {
   },
   modulePathIgnorePatterns: ['<rootDir>/build/'],
   modulePaths: ['/shared/vendor/modules'],
+  openHandlesTimeout: 1000,
   preset: 'react-native',
   prettierPath: '<rootDir>/node_modules/prettier',
   resetMocks: false,

packages/jest-config/src/index.ts

@@ -113,6 +113,7 @@ const groupOptions = (
     notifyMode: options.notifyMode,
     onlyChanged: options.onlyChanged,
     onlyFailures: options.onlyFailures,
+    openHandlesTimeout: options.openHandlesTimeout,
     outputFile: options.outputFile,
     passWithNoTests: options.passWithNoTests,
     projects: options.projects,
@@ -168,6 +169,7 @@ const groupOptions = (
     moduleNameMapper: options.moduleNameMapper,
     modulePathIgnorePatterns: options.modulePathIgnorePatterns,
     modulePaths: options.modulePaths,
+    openHandlesTimeout: options.openHandlesTimeout,
     prettierPath: options.prettierPath,
     resetMocks: options.resetMocks,
     resetModules: options.resetModules,

packages/jest-config/src/normalize.ts

@@ -912,6 +912,7 @@ export default async function normalize(
       case 'notifyMode':
       case 'onlyChanged':
       case 'onlyFailures':
+      case 'openHandlesTimeout':
       case 'outputFile':
       case 'passWithNoTests':
       case 'replname':

packages/jest-types/src/Config.ts

@@ -178,6 +178,7 @@ export type DefaultOptions = {
   noStackTrace: boolean;
   notify: boolean;
   notifyMode: NotifyMode;
+  openHandlesTimeout: number;
   passWithNoTests: boolean;
   prettierPath: string;
   resetMocks: boolean;
@@ -273,6 +274,7 @@ export type InitialOptions = Partial<{
   notifyMode: string;
   onlyChanged: boolean;
   onlyFailures: boolean;
+  openHandlesTimeout: number;
   outputFile: string;
   passWithNoTests: boolean;
   preset: string | null | undefined;
@@ -389,6 +391,7 @@ export type GlobalConfig = {
   outputFile?: string;
   onlyChanged: boolean;
   onlyFailures: boolean;
+  openHandlesTimeout: number;
   passWithNoTests: boolean;
   projects: Array<string>;
   replname?: string;
@@ -448,6 +451,7 @@ export type ProjectConfig = {
   moduleNameMapper: Array<[string, string]>;
   modulePathIgnorePatterns: Array<string>;
   modulePaths?: Array<string>;
+  openHandlesTimeout: number;
   preset?: string;
   prettierPath: string;
   resetMocks: boolean;

packages/test-utils/src/config.ts

@@ -40,6 +40,7 @@ const DEFAULT_GLOBAL_CONFIG: Config.GlobalConfig = {
   notifyMode: 'failure-change',
   onlyChanged: false,
   onlyFailures: false,
+  openHandlesTimeout: 1000,
   outputFile: undefined,
   passWithNoTests: false,
   projects: [],
@@ -92,6 +93,7 @@ const DEFAULT_PROJECT_CONFIG: Config.ProjectConfig = {
   moduleNameMapper: [],
   modulePathIgnorePatterns: [],
   modulePaths: [],
+  openHandlesTimeout: 1000,
   prettierPath: 'prettier',
   resetMocks: false,
   resetModules: false,