report rejection step in miner log, ensure signe_test shutdown in int…

…egration test
stacks-network · Jan 31, 2025 · fe429f4 · fe429f4
1 parent eb883ee
commit fe429f4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/testnet/stacks-node/src/nakamoto_node/signer_coordinator.rs b/testnet/stacks-node/src/nakamoto_node/signer_coordinator.rs
@@ -390,7 +390,7 @@ impl SignerCoordinator {
 
             if rejections != block_status.total_reject_weight as u64 {
                 rejections = block_status.total_reject_weight as u64;
-                let rejections_timeout_tuple = self
+                let (rejections_step, new_rejections_timeout) = self
                     .block_rejection_timeout_steps
                     .range((Included(0), Included(rejections)))
                     .last()
@@ -399,10 +399,11 @@ impl SignerCoordinator {
                             "Invalid rejection timeout step function definition".into(),
                         )
                     })?;
-                rejections_timeout = rejections_timeout_tuple.1;
+                rejections_timeout = new_rejections_timeout;
                 info!("Number of received rejections updated, resetting timeout";
                                     "rejections" => rejections,
                                     "rejections_timeout" => rejections_timeout.as_secs(),
+                                    "rejections_step" => rejections_step,
                                     "rejections_threshold" => self.total_weight.saturating_sub(self.weight_threshold));
 
                 Counters::set(

diff --git a/testnet/stacks-node/src/tests/signer/v0.rs b/testnet/stacks-node/src/tests/signer/v0.rs
@@ -8111,6 +8111,9 @@ fn block_validation_check_rejection_timeout_heuristic() {
     // reset reject/ignore
     TEST_REJECT_ALL_BLOCK_PROPOSAL.set(vec![]);
     TEST_IGNORE_ALL_BLOCK_PROPOSALS.set(vec![]);
+
+    info!("------------------------- Shutdown -------------------------");
+    signer_test.shutdown();
 }
 
 /// Test scenario: