-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Part 2] Individual Test Case Report - Report progress of individual test cases #6
Conversation
@SimenB, @jevakallio: Review is requested and suggestions on changes to be made if any. There's just 1 e2e test failing but rest of the things are working as expected. Running |
241581f
to
6aee98d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really shaping up, great work!
packages/jest-circus/src/legacy-code-todo-rewrite/jestAdapter.ts
Outdated
Show resolved
Hide resolved
packages/jest-circus/src/legacy-code-todo-rewrite/jestAdapterInit.ts
Outdated
Show resolved
Hide resolved
Reference: jestjs#10227 |
0be2e81
to
69a6aa7
Compare
9296d86
to
d08c35d
Compare
Merged into master and rebased the fork 🎉 |
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. |
Summary
Part 2 of 2
Aim: Report progress of individual test cases (this is the final milestone PR)
Reference: jestjs#9001
Fixes: jestjs#6616 #1
Breakdowns:
Test plan
onCustomMessage
functions to pass tests (caused by updating its Type)Update so far:
Running
JEST_CIRCUS=1 yarn jest
should increment atomic tests and not batch update them as it did before.TODO
To Reproduce:
$ JEST_CIRCUS=1 yarn test-leak