-
-
Notifications
You must be signed in to change notification settings - Fork 317
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: consolidate sim-merge tests #6344
Conversation
@g11tech Please review, if there is any assertion left in |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6344 +/- ##
=========================================
Coverage 61.66% 61.66%
=========================================
Files 554 554
Lines 58078 58078
Branches 1837 1837
=========================================
Hits 35811 35811
Misses 22228 22228
Partials 39 39 |
Performance Report✔️ no performance regression detected Full benchmark results
|
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.
lets observe sim tests reliability, and then review this if things on sim tests are stable (blocking it till then)
123ba02
to
2f0e55d
Compare
2f0e55d
to
02ae312
Compare
Tried locally and on CI many consecutive runs. With recent changes made to improve sim tests, it's stability seem good now. @wemeetagain @g11tech I feel it's good to review/merge this PR. |
Sim merge tests seem to be stuck, running for almost 3 hours already |
🎉 This PR is included in v1.17.0 🎉 |
Motivation
Consolidate the sim tests, to minimize the CI run time and reduce maintenance effort.
Description
merge-interop.test.ts
tomulti_fork.test.ts
merge-interop.test.ts
Steps to test or reproduce