Skip to content

fix(jest-test-sequencer): make --onlyFailures flag working for tests failed due to compilation errors#15851

Merged
SimenB merged 3 commits intojestjs:mainfrom
hainenber:fix/cache-failed-compiled-test-result
Oct 10, 2025
Merged

fix(jest-test-sequencer): make --onlyFailures flag working for tests failed due to compilation errors#15851
SimenB merged 3 commits intojestjs:mainfrom
hainenber:fix/cache-failed-compiled-test-result

Conversation

@hainenber
Copy link
Copy Markdown
Contributor

Summary

Fixes #15382

Failed test execution due to trans-pilation/transform issues should be counted towards failed result for caching. The later caching would later help filtering out previously failed tests for re-execution when used with --onlyFailures CLI option.

Test plan

I've added new E2E test and updated existing unit tests for cacheResult function so acceptance criteria should be Green CI

…s failed due to compilation errors

Signed-off-by: hainenber <dotronghai96@gmail.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Oct 4, 2025

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 76d2d28
🔍 Latest deploy log https://app.netlify.com/projects/jestjs/deploys/68e141a77b971500085f49d2
😎 Deploy Preview https://deploy-preview-15851--jestjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: hainenber <dotronghai96@gmail.com>
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Oct 4, 2025

Open in StackBlitz

babel-jest

npm i https://pkg.pr.new/babel-jest@15851

babel-plugin-jest-hoist

npm i https://pkg.pr.new/babel-plugin-jest-hoist@15851

babel-preset-jest

npm i https://pkg.pr.new/babel-preset-jest@15851

create-jest

npm i https://pkg.pr.new/create-jest@15851

@jest/diff-sequences

npm i https://pkg.pr.new/@jest/diff-sequences@15851

expect

npm i https://pkg.pr.new/expect@15851

@jest/expect-utils

npm i https://pkg.pr.new/@jest/expect-utils@15851

jest

npm i https://pkg.pr.new/jest@15851

jest-changed-files

npm i https://pkg.pr.new/jest-changed-files@15851

jest-circus

npm i https://pkg.pr.new/jest-circus@15851

jest-cli

npm i https://pkg.pr.new/jest-cli@15851

jest-config

npm i https://pkg.pr.new/jest-config@15851

@jest/console

npm i https://pkg.pr.new/@jest/console@15851

@jest/core

npm i https://pkg.pr.new/@jest/core@15851

@jest/create-cache-key-function

npm i https://pkg.pr.new/@jest/create-cache-key-function@15851

jest-diff

npm i https://pkg.pr.new/jest-diff@15851

jest-docblock

npm i https://pkg.pr.new/jest-docblock@15851

jest-each

npm i https://pkg.pr.new/jest-each@15851

@jest/environment

npm i https://pkg.pr.new/@jest/environment@15851

jest-environment-jsdom

npm i https://pkg.pr.new/jest-environment-jsdom@15851

@jest/environment-jsdom-abstract

npm i https://pkg.pr.new/@jest/environment-jsdom-abstract@15851

jest-environment-node

npm i https://pkg.pr.new/jest-environment-node@15851

@jest/expect

npm i https://pkg.pr.new/@jest/expect@15851

@jest/fake-timers

npm i https://pkg.pr.new/@jest/fake-timers@15851

@jest/get-type

npm i https://pkg.pr.new/@jest/get-type@15851

@jest/globals

npm i https://pkg.pr.new/@jest/globals@15851

jest-haste-map

npm i https://pkg.pr.new/jest-haste-map@15851

jest-jasmine2

npm i https://pkg.pr.new/jest-jasmine2@15851

jest-leak-detector

npm i https://pkg.pr.new/jest-leak-detector@15851

jest-matcher-utils

npm i https://pkg.pr.new/jest-matcher-utils@15851

jest-message-util

npm i https://pkg.pr.new/jest-message-util@15851

jest-mock

npm i https://pkg.pr.new/jest-mock@15851

@jest/pattern

npm i https://pkg.pr.new/@jest/pattern@15851

jest-phabricator

npm i https://pkg.pr.new/jest-phabricator@15851

jest-regex-util

npm i https://pkg.pr.new/jest-regex-util@15851

@jest/reporters

npm i https://pkg.pr.new/@jest/reporters@15851

jest-resolve

npm i https://pkg.pr.new/jest-resolve@15851

jest-resolve-dependencies

npm i https://pkg.pr.new/jest-resolve-dependencies@15851

jest-runner

npm i https://pkg.pr.new/jest-runner@15851

jest-runtime

npm i https://pkg.pr.new/jest-runtime@15851

@jest/schemas

npm i https://pkg.pr.new/@jest/schemas@15851

jest-snapshot

npm i https://pkg.pr.new/jest-snapshot@15851

@jest/snapshot-utils

npm i https://pkg.pr.new/@jest/snapshot-utils@15851

@jest/source-map

npm i https://pkg.pr.new/@jest/source-map@15851

@jest/test-result

npm i https://pkg.pr.new/@jest/test-result@15851

@jest/test-sequencer

npm i https://pkg.pr.new/@jest/test-sequencer@15851

@jest/transform

npm i https://pkg.pr.new/@jest/transform@15851

@jest/types

npm i https://pkg.pr.new/@jest/types@15851

jest-util

npm i https://pkg.pr.new/jest-util@15851

jest-validate

npm i https://pkg.pr.new/jest-validate@15851

jest-watcher

npm i https://pkg.pr.new/jest-watcher@15851

jest-worker

npm i https://pkg.pr.new/jest-worker@15851

pretty-format

npm i https://pkg.pr.new/pretty-format@15851

commit: 76d2d28

Signed-off-by: hainenber <dotronghai96@gmail.com>
Copy link
Copy Markdown
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thank you! elegant fix and good tests 👍

@SimenB SimenB merged commit 33d1166 into jestjs:main Oct 10, 2025
74 checks passed
@hainenber hainenber deleted the fix/cache-failed-compiled-test-result branch October 11, 2025 05:58
@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Scripts with syntax errors are ignored by --onlyFailures

2 participants