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
fix: align json output with jest's #1034
Conversation
✅ Deploy Preview for vitest-dev ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Thanks for the PR @zxch3n! Things like status being @cexbrayat would you help us review this PR and check that it won't be a breaking change for your use case? |
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.
LGTM, thanks @zxch3n 👍
Hey all. Just like to add wanted to weight in from a hmm regular user's perspective ;) |
😬 |
Hmm I might give this branch a try later today... Maybe I should just
update my addon - I was so proud of it, so not gonna ditch it so fast lol
pon., 28 mar 2022, 13:45 użytkownik patak ***@***.***>
napisał:
… ***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#1034 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC3M4PECKEXENNDUMGPPSVLVCGLWNANCNFSM5RYGN4AA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
In the current version(v0.7.11), the JSON reporter's output format deviates from jest's. See Stackblitz example.
In PR #489 json reporter is added, it implements the subset of jest's AggregatedResult type. However, jest --json output is using FormattedTestResults.
And each object in
testResults
should represents a file rather than a test case.