-
Notifications
You must be signed in to change notification settings - Fork 493
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
Bug Fix: Adds validity round check for failing tests #4085
Conversation
Resolves #2088 Adds validity round checks
The issue linked does not look to be a correct one. I suggest to put a better description into the PR |
Codecov Report
@@ Coverage Diff @@
## master #4085 +/- ##
==========================================
- Coverage 54.55% 54.49% -0.06%
==========================================
Files 391 391
Lines 48662 48668 +6
==========================================
- Hits 26546 26524 -22
- Misses 19890 19921 +31
+ Partials 2226 2223 -3
Continue to review full report at Codecov.
|
Updated. |
Run at 02dfe12
|
with the recent fix the second test passed as well
|
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.
@winder please take a look as well
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 @AlgoStephenAkiki
Resolves #2088
Currently these tests are failing:
TestParticipationAccountsExpirationFuture
TestParticipationAccountsExpirationNonFuture
Add validity round checks so that rounds are appropriately set when generating the registration transaction.