cerc-io/lighthouse
16
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

fix progressive balance slashing tests

Browse Source
This commit is contained in:
realbigsean 2023-07-11 17:07:52 -04:00
parent 782a53ad9d
commit 57bb1d931e
No known key found for this signature in database
GPG Key ID: BE1B3DB104F6C788
1 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
consensus/fork_choice/tests/tests.rs
View File

@ -1353,6 +1353,14 @@ async fn progressive_balances_cache_attester_slashing() {
.apply_blocks_while(|_, state| state.finalized_checkpoint().epoch == 0) .apply_blocks_while(|_, state| state.finalized_checkpoint().epoch == 0)
.await .await
.unwrap() .unwrap()
// Note: This test may fail if the shuffling used changes, right now it re-runs with
// deterministic shuffling. A shuffling change my cause the slashed proposer to propose
// again in the next epoch, which results in a block processing failure
// (`HeaderInvalid::ProposerSlashed`). The harness should be re-worked to successfully skip
// the slot in this scenario rather than panic-ing. The same applies to
// `progressive_balances_cache_proposer_slashing`.
.apply_blocks(1)
.await
.add_previous_epoch_attester_slashing() .add_previous_epoch_attester_slashing()
.await .await
// expect fork choice to import blocks successfully after a previous epoch attester is // expect fork choice to import blocks successfully after a previous epoch attester is
@ -1376,6 +1384,14 @@ async fn progressive_balances_cache_proposer_slashing() {
.apply_blocks_while(|_, state| state.finalized_checkpoint().epoch == 0) .apply_blocks_while(|_, state| state.finalized_checkpoint().epoch == 0)
.await .await
.unwrap() .unwrap()
// Note: This test may fail if the shuffling used changes, right now it re-runs with
// deterministic shuffling. A shuffling change my cause the slashed proposer to propose
// again in the next epoch, which results in a block processing failure
// (`HeaderInvalid::ProposerSlashed`). The harness should be re-worked to successfully skip
// the slot in this scenario rather than panic-ing. The same applies to
// `progressive_balances_cache_attester_slashing`.
.apply_blocks(1)
.await
.add_previous_epoch_proposer_slashing(MainnetEthSpec::slots_per_epoch()) .add_previous_epoch_proposer_slashing(MainnetEthSpec::slots_per_epoch())
.await .await
// expect fork choice to import blocks successfully after a previous epoch proposer is // expect fork choice to import blocks successfully after a previous epoch proposer is