removed useless test attributes

Author: rdeioris

testnet/stacks-node/src/nakamoto_node/signer_coordinator.rs

@@ -410,14 +410,11 @@ impl SignerCoordinator {
                                     "rejections_threshold" => self.total_weight.saturating_sub(self.weight_threshold));
                 rejections_timer = Instant::now();
 
-                #[cfg(test)]
-                {
-                    Counters::set(
-                        &counters.naka_miner_current_rejections_timeout_secs,
-                        rejections_timeout.as_secs(),
-                    );
-                    Counters::set(&counters.naka_miner_current_rejections, rejections);
-                }
+                Counters::set(
+                    &counters.naka_miner_current_rejections_timeout_secs,
+                    rejections_timeout.as_secs(),
+                );
+                Counters::set(&counters.naka_miner_current_rejections, rejections);
             }
 
             if block_status

testnet/stacks-node/src/run_loop/neon.rs

@@ -117,9 +117,7 @@ pub struct Counters {
     pub naka_signer_pushed_blocks: RunLoopCounter,
     pub naka_miner_directives: RunLoopCounter,
 
-    #[cfg(test)]
     pub naka_miner_current_rejections: RunLoopCounter,
-    #[cfg(test)]
     pub naka_miner_current_rejections_timeout_secs: RunLoopCounter,
 
     #[cfg(test)]
@@ -145,7 +143,7 @@ impl Counters {
     }
 
     #[cfg(not(test))]
-    fn set(_ctr: &RunLoopCounter, _value: u64) {}
+    pub fn set(_ctr: &RunLoopCounter, _value: u64) {}
 
     pub fn bump_blocks_processed(&self) {
         Counters::inc(&self.blocks_processed);