Skip to content

Commit 1ebdaaa

Browse files
committed
Fix test_simple_signer to not attempt to match the received time value
Signed-off-by: Jacinta Ferrant <[email protected]>
1 parent 85fdb5a commit 1ebdaaa

File tree

2 files changed

+23
-1
lines changed
  • libsigner/src/tests
  • testnet/stacks-node/src/tests/signer

2 files changed

+23
-1
lines changed

libsigner/src/tests/mod.rs

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,28 @@ fn test_simple_signer() {
200200
})
201201
.collect();
202202

203-
assert_eq!(sent_events, accepted_events);
203+
for (sent_event, accepted_event) in sent_events.iter().zip(accepted_events.iter()) {
204+
let SignerEvent::SignerMessages {
205+
signer_set,
206+
messages,
207+
received_time,
208+
} = sent_event
209+
else {
210+
panic!("BUG: should not have sent anything but a signer message");
211+
};
212+
let SignerEvent::SignerMessages {
213+
signer_set: accepted_signer_set,
214+
messages: accepted_messages,
215+
received_time: accepted_time,
216+
} = accepted_event
217+
else {
218+
panic!("BUG: should not have accepted anything but a signer message");
219+
};
220+
221+
assert_eq!(signer_set, accepted_signer_set);
222+
assert_eq!(messages, accepted_messages);
223+
assert_ne!(received_time, accepted_time);
224+
}
204225
mock_stacks_node.join().unwrap();
205226
}
206227

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4009,6 +4009,7 @@ fn idle_tenure_extend_active_mining() {
40094009
}
40104010
TEST_MINE_STALL.set(false);
40114011
});
4012+
40124013
let latest_response = signer_test.get_latest_block_response(slot_id);
40134014
let naka_blocks = test_observer::get_mined_nakamoto_blocks();
40144015
info!(

0 commit comments

Comments
 (0)