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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(e2e): attempt to log better errors when running commands #13881

Merged
merged 2 commits into from
Feb 9, 2023

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Feb 9, 2023

Summary

We currently have a CI error which is hard to understand, e.g.

image

It doesn't say much of anything. Logging the entire result before rethrowing seems better.

Test plan

馃し

Comment on lines -28 to -29
// For compat with cross-spawn
result.status = result.exitCode;
Copy link
Member Author

@SimenB SimenB Feb 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we throw if it's non zero, so no reason to decorate it

@SimenB
Copy link
Member Author

SimenB commented Feb 9, 2023

Hah, of course that job passed now 馃槄

EDIT: another did, tho. seems to be a permission error moving zip archives. so no biggie, but at least it's printed now

@SimenB SimenB changed the title chore: attempt to log better errors when running commands chore(e2e): attempt to log better errors when running commands Feb 9, 2023
@SimenB SimenB merged commit 94b73a2 into jestjs:main Feb 9, 2023
@SimenB SimenB deleted the better-error-e2e branch February 9, 2023 08:29
@github-actions
Copy link

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.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants