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

bail on jest tests #2378

Open
bdemann opened this issue Dec 20, 2024 · 1 comment
Open

bail on jest tests #2378

bdemann opened this issue Dec 20, 2024 · 1 comment
Assignees

Comments

@bdemann
Copy link
Member

bdemann commented Dec 20, 2024

No description provided.

@bdemann bdemann self-assigned this Dec 20, 2024
@bdemann bdemann linked a pull request Dec 20, 2024 that will close this issue
6 tasks
@bdemann
Copy link
Member Author

bdemann commented Dec 20, 2024

Based on our experience, it seems that Jest's bail option only takes effect when multiple test files are running. In our case, all of our tests are contained within a single test file. This setup is intentional because:

  1. All tests share the same canisters on the same replica, and splitting them into multiple files could introduce race conditions.
  2. Many of our tests depend on state created or modified by earlier tests, making it logical to group them in one file.

Unfortunately, this also means there's no way to abort the entire test suite early if one of the initial tests fails, which limits the usefulness of bail in our current setup.

We plan to conduct further research to confirm this behavior and explore potential solutions, but now isn’t the right time to address this comprehensively. For now, this is an important consideration for any plans involving bail in our Jest configuration.

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 a pull request may close this issue.

2 participants
@bdemann and others