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
Prepare spanner for double vote detection and fix a few bugs #4940
Prepare spanner for double vote detection and fix a few bugs #4940
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4940 +/- ##
========================================
Coverage ? 4.04%
========================================
Files ? 71
Lines ? 6130
Branches ? 0
========================================
Hits ? 248
Misses ? 5862
Partials ? 20 |
…slasher-change-to-target
…Prysm into slasher-change-to-target
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.
had one comment . would like to see a more complex test scenario otherwise looks good
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
…iclabs#4940) * Rename vars for clarity * Change spanner to take target epoch as key * Fix tests, add multiple val test * Fixes * Change the spanner to take in att on detect * Add back proto diagram tests * Remove unneeded comments
Part of #4836
This PR goes through the spanner and prepares it for the changes needed to catch double voting. Mainly test additions.
Detailed: https://hackmd.prylabs.network/EcYUo5koQhmWW0_pSlPHSQ