Open thernstig opened 4 years ago
Hi 👋
Can I work on this issue?
Yeah, that'd be lovely! Feel free to ask questions if you get stuck or want a pointer 🙂
I'd start in jest-circus
(jest-jasmine
is soft-deprecated, so unless you want to you can ignore it). Maybe the approach in #7201 might help you get started?
When beforeAll
fails, it also ignores it.only
and logs all tests as failing: https://repl.it/repls/MobileBaggyLearning
Wasn't sure if this bug will already be fixed by #10004 so please let me know if you'd like me to file a new issue!
Hi! Has there been any progress on this issue? :)
@vldmrkl Looking forward to you working on this issue 😄
Sorry, can't continue working on this issue. My latest progress could be found at #10004 I apologize for miscommunication.
This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.
I think this change still makes sense and the issue should remain open.
Hey, I'd like to work on this issue. :)
@itaizelther I'd say go for it and push a PR, continuing on the one already started.
@thernstig I have no write permissions to the existing PR, so I have created a new one: #13273 It is done, you may review it
I am not a maintainer here, just meant that I don't think anyone minds if you start a new PR. Great that you did!
@itaizelther thank you 🙏
This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.
Since there is a PR at https://github.com/jestjs/jest/pull/13273 I am bumping this
This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.
Since there is a PR at https://github.com/jestjs/jest/pull/13273 I am bumping this
🐛 Bug Report
When beforeAll fails, none of the other tests actually execute, but they all fail with the same reason, and they all print the same error.
Jest should avoid printing the individual tests if beforeAll fails. Right now the implementation just checks if there is an error and if yes, print it and fail the test without actually executing it.
Note that @SimenB asked me to create this https://github.com/facebook/jest/issues/6695#issuecomment-619405847.
To Reproduce
Run a beforeAll() that fails where a couple of tests are executed after. This is the failure received (with the supplied repro below):
Expected behavior
Jest should avoid printing the individual tests if beforeAll fails. It is highly confusing for users.
Link to repl or repo (highly encouraged)
envinfo