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
test: fix jest wrapper #27636
test: fix jest wrapper #27636
Conversation
@viceice merge queue failure |
do you know the test? |
🎉 This PR is included in version 37.220.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
I imagine that this PR is why I see the following test failures in my custom tests in my fork:
I guess it's not a big issue since it's just for the snapshots. But it is definitely weird and unexpected. Anyone seeing those snapshots later will go "Well that's an invalid windows path" |
yes. feel free to open a PR with a better fix which works cross platform. |
Changes
set proper exitcode from jest wrapper
Context
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: