Skip to content

Commit b17874c

Browse files
committed
fix: assertion of prometheus response
1 parent 8be83c8 commit b17874c

File tree

1 file changed

+10
-3
lines changed
  • testnet/stacks-node/src/tests/signer

1 file changed

+10
-3
lines changed

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

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -558,16 +558,23 @@ fn miner_gather_signatures() {
558558
// Test prometheus metrics response
559559
#[cfg(feature = "monitoring_prom")]
560560
{
561+
let naka_blocks_confirmed = signer_test
562+
.running_nodes
563+
.nakamoto_blocks_mined
564+
.load(Ordering::SeqCst);
565+
info!("Nakamoto blocks confirmed: {}", naka_blocks_confirmed);
561566
let metrics_response = signer_test.get_signer_metrics();
562567

568+
let expected_blocks = (naka_blocks_confirmed as usize) * num_signers;
569+
563570
// Because 5 signers are running in the same process, the prometheus metrics
564571
// are incremented once for every signer. This is why we expect the metric to be
565-
// `5`, even though there is only one block proposed.
566-
let expected_result = format!("stacks_signer_block_proposals_received {}", num_signers);
572+
// `5` * `naka_blocks_confirmed`.
573+
let expected_result = format!("stacks_signer_block_proposals_received {}", expected_blocks);
567574
assert!(metrics_response.contains(&expected_result));
568575
let expected_result = format!(
569576
"stacks_signer_block_responses_sent{{response_type=\"accepted\"}} {}",
570-
num_signers
577+
expected_blocks
571578
);
572579
assert!(metrics_response.contains(&expected_result));
573580
}

0 commit comments

Comments
 (0)