Skip to content

Commit 4dbbbc2

Browse files
committed
Fix reorging_signers_capitulate_to_nonreorging_signers_during_tenure_fork
Signed-off-by: Jacinta Ferrant <jacinta.ferrant@gmail.com>
1 parent b4a296d commit 4dbbbc2

File tree

1 file changed

+13
-50
lines changed
  • testnet/stacks-node/src/tests/signer

1 file changed

+13
-50
lines changed

testnet/stacks-node/src/tests/signer/v0.rs

Lines changed: 13 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -14866,56 +14866,19 @@ fn reorging_signers_capitulate_to_nonreorging_signers_during_tenure_fork() {
1486614866
assert_ne!(tip_c.burn_header_hash, tip_a.burn_header_hash);
1486714867
assert_eq!(tip_c.block_height, burn_height_before + 1);
1486814868

14869-
wait_for(30, || {
14870-
let mut nmb_matches = 0;
14871-
let stackerdb_events = test_observer::get_stackerdb_chunks();
14872-
for chunk in stackerdb_events
14873-
.into_iter()
14874-
.flat_map(|chunk| chunk.modified_slots)
14875-
{
14876-
let message = SignerMessage::consensus_deserialize(&mut chunk.data.as_slice())
14877-
.expect("Failed to deserialize SignerMessage");
14878-
let SignerMessage::StateMachineUpdate(update) = message else {
14879-
continue;
14880-
};
14881-
let (burn_block, burn_block_height, current_miner_pkh) =
14882-
match (SUPPORTED_SIGNER_PROTOCOL_VERSION, update.content) {
14883-
(
14884-
0,
14885-
StateMachineUpdateContent::V0 {
14886-
burn_block,
14887-
burn_block_height,
14888-
current_miner:
14889-
StateMachineUpdateMinerState::ActiveMiner {
14890-
current_miner_pkh, ..
14891-
},
14892-
..
14893-
},
14894-
)
14895-
| (
14896-
1,
14897-
StateMachineUpdateContent::V1 {
14898-
burn_block,
14899-
burn_block_height,
14900-
current_miner:
14901-
StateMachineUpdateMinerState::ActiveMiner {
14902-
current_miner_pkh, ..
14903-
},
14904-
..
14905-
},
14906-
) => (burn_block, burn_block_height, current_miner_pkh),
14907-
_ => continue,
14908-
};
14909-
if burn_block == tenure_c_block_proposal.header.consensus_hash
14910-
&& burn_block_height == burn_height_before + 1
14911-
&& current_miner_pkh == miner_pkh_1
14912-
{
14913-
nmb_matches += 1;
14914-
}
14915-
}
14916-
Ok(nmb_matches == 5)
14917-
})
14918-
.unwrap();
14869+
let all_signers = miners.signer_test.signer_test_pks();
14870+
14871+
info!("--------------- Waiting for Signers to Capitulate to Miner {miner_pkh_1} with Expected Stacks Height {} ----------------", tip_a.stacks_block_height;
14872+
);
14873+
wait_for_state_machine_update(
14874+
30,
14875+
&tenure_c_block_proposal.header.consensus_hash,
14876+
burn_height_before + 1,
14877+
Some((miner_pkh_1, tip_a.stacks_block_height)),
14878+
&all_signers,
14879+
SUPPORTED_SIGNER_PROTOCOL_VERSION,
14880+
)
14881+
.expect("Timed out waiting for state machine updates");
1491914882

1492014883
info!("--------------- Miner 1 Extends Tenure B over Tenure C ---------------");
1492114884
TEST_BROADCAST_PROPOSAL_STALL.set(vec![]);

0 commit comments

Comments
 (0)