fix: jest --onlyFailures --listTests now correctly lists only failed tests (#15700)

Author: notoriousmango

CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Fixes
 
 - `[jest-config]` Fix ESM TS config loading in a CJS project ([#15694](https://github.com/jestjs/jest/pull/15694))
+- `[jest-core]` jest --onlyFailures --listTests now correctly lists only failed tests([#15700](https://github.com/jestjs/jest/pull/15700))
 
 ### Features
 

e2e/__tests__/listTests.test.ts

@@ -8,6 +8,8 @@
 import * as path from 'path';
 import * as fs from 'graceful-fs';
 import runJest from '../runJest';
+import {tmpdir} from 'os';
+import {cleanup, createEmptyPackage, writeFiles} from '../Utils';
 
 const testRootDir = path.resolve(__dirname, '..', '..');
 
@@ -85,4 +87,54 @@ describe('--listTests flag', () => {
       ).toMatchSnapshot();
     });
   });
+
+  describe('--onlyFailures flag', () => {
+    const tempDir = path.resolve(tmpdir(), 'temp-only-failures');
+
+    beforeEach(() => {
+      // Clean up any existing temp directory and create a new one
+      cleanup(tempDir);
+      createEmptyPackage(tempDir, {jest: {testEnvironment: 'node'}});
+
+      // Create test files
+      writeFiles(tempDir, {
+        'failing.test.js':
+          'test("failing test", () => { expect(true).toBe(false); });',
+        'passing.test.js':
+          'test("passing test", () => { expect(true).toBe(true); });',
+      });
+    });
+
+    afterEach(() => {
+      cleanup(tempDir);
+    });
+
+    it('lists only failed tests after a test run', () => {
+      runJest(tempDir);
+
+      const {exitCode, stdout} = runJest(tempDir, [
+        '--listTests',
+        '--onlyFailures',
+      ]);
+
+      expect(exitCode).toBe(0);
+      const listedTests = stdout
+        .trim()
+        .split('\n')
+        .filter(line => line.length > 0);
+      expect(listedTests).toHaveLength(1);
+      expect(listedTests[0]).toMatch(/failing\.test\.js$/);
+      expect(stdout).not.toMatch(/passing\.test\.js$/);
+    });
+
+    it('lists no tests when no failed tests exist', () => {
+      const {exitCode, stdout} = runJest('list-tests', [
+        '--listTests',
+        '--onlyFailures',
+      ]);
+
+      expect(exitCode).toBe(0);
+      expect(stdout.trim()).toBe('');
+    });
+  });
 });

packages/jest-core/src/runJest.ts

@@ -216,6 +216,14 @@ export default async function runJest({
 
   allTests = await sequencer.sort(allTests);
 
+  if (globalConfig.onlyFailures) {
+    if (failedTestsCache) {
+      allTests = failedTestsCache.filterTests(allTests);
+    } else {
+      allTests = await sequencer.allFailedTests(allTests);
+    }
+  }
+
   if (globalConfig.listTests) {
     const testsPaths = [...new Set(allTests.map(test => test.path))];
     let testsListOutput;
@@ -238,14 +246,6 @@ export default async function runJest({
     return;
   }
 
-  if (globalConfig.onlyFailures) {
-    if (failedTestsCache) {
-      allTests = failedTestsCache.filterTests(allTests);
-    } else {
-      allTests = await sequencer.allFailedTests(allTests);
-    }
-  }
-
   const hasTests = allTests.length > 0;
 
   if (!hasTests) {