From d16d4056e9e2cbcfc31b1d3aa5ab49be6c3c2c72 Mon Sep 17 00:00:00 2001 From: Adam Szkoda Date: Tue, 11 Aug 2020 12:17:42 +0200 Subject: [PATCH] Reproduce the bug A bug in a test case hid a bug in the code: prunes_abandoned_fork_between_two_finalized_checkpoints: contrary to its name, the abandoned fork isn't actually wholly contained between two finalized checkpoints in terms of slot values. --- beacon_node/beacon_chain/tests/store_tests.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/beacon_node/beacon_chain/tests/store_tests.rs b/beacon_node/beacon_chain/tests/store_tests.rs index 6ab2908bcec..bafef37882f 100644 --- a/beacon_node/beacon_chain/tests/store_tests.rs +++ b/beacon_node/beacon_chain/tests/store_tests.rs @@ -743,7 +743,7 @@ fn prunes_abandoned_fork_between_two_finalized_checkpoints() { let (stray_blocks, stray_states, _, stray_head, _) = harness.add_stray_blocks( harness.get_head_state(), slot, - slots_per_epoch - 1, + slots_per_epoch - 3, &faulty_validators, );