-
Notifications
You must be signed in to change notification settings - Fork 2.2k
chore(test): add hints after test failed #11995
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
base: master
Are you sure you want to change the base?
Conversation
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.
i think it's ok to highlight e.g. forge test --rerun
but I don't think we want a 5 bullet point list when tests fail
Thanks! only new version ⬇️
|
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.
please also address the test failures:)
Co-authored-by: onbjerg <[email protected]>
Hi @onbjerg, I've updated all the test snapshots to include the tip message. Could you please help re-trigger the CI? Latest run was 1635/1636 - flaky tests passed on retry, the one failure seems to be an unrelated cast issue, but let me know if I missed something. |
During optimized #11993, I just realized there are much more magic function to help the test, but for the lazy people who don't like to read the whole docs like me, its really hard to know this, we just wait for all the test pass 😓. These hints may give alot help.