Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: randomize jest test order #14851

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

starsirius
Copy link
Member

The type of this PR is: Chore

Description

Noticed that Jest v29.5 now supports randomizing the test order within a test file, which is the default behavior or rspec, and encourage setting up and writing test cases in an order independent way.

I'm not sure if we want to pursue this in the Jest land, considering Force has accumulated ~3500 tests, but opening the PR up to see the current status.

@starsirius starsirius self-assigned this Nov 15, 2024
@artsy-peril
Copy link
Contributor

artsy-peril bot commented Nov 15, 2024

Warnings
⚠️ Please assign someone to merge this PR, and optionally include people who should review.

Generated by 🚫 dangerJS against e03d43d

@starsirius
Copy link
Member Author

And yeah, this build shows 46 failures in the test job and 7 failures in the test-legacy job.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant