Sanity fixes

Author: orsissimo

testnet/stacks-node/src/tests/signer/commands/bitcoin_mining.rs

@@ -1,8 +1,7 @@
 use std::sync::Arc;
 
 use madhouse::{Command, CommandWrapper};
-use proptest::prop_oneof;
-use proptest::strategy::{Just, Strategy};
+use proptest::strategy::Strategy;
 use tracing::info;
 
 use super::context::{SignerTestContext, SignerTestState};
@@ -120,7 +119,10 @@ impl Command<SignerTestState, SignerTestContext> for ChainGenerateBitcoinBlocks
     ) -> impl Strategy<Value = CommandWrapper<SignerTestState, SignerTestContext>> {
         (1u64..=5u64).prop_map({
             move |num_blocks| {
-                CommandWrapper::new(ChainGenerateBitcoinBlocks::multiple(ctx.clone(), num_blocks))
+                CommandWrapper::new(ChainGenerateBitcoinBlocks::multiple(
+                    ctx.clone(),
+                    num_blocks,
+                ))
             }
         })
     }

testnet/stacks-node/src/tests/signer/commands/block_commit.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use madhouse::{Command, CommandWrapper};
-use proptest::prelude::{Just, Strategy};
+use proptest::prelude::Strategy;
 
 use super::context::{SignerTestContext, SignerTestState};
 
@@ -62,12 +62,11 @@ impl Command<SignerTestState, SignerTestContext> for MinerSubmitBlockCommit {
             ),
         }
 
-        assert!(
-            self.ctx
-                .get_counters_for_miner(self.miner_index)
-                .naka_skip_commit_op
-                .get()
-        );
+        assert!(self
+            .ctx
+            .get_counters_for_miner(self.miner_index)
+            .naka_skip_commit_op
+            .get());
     }
 
     fn label(&self) -> String {
@@ -79,6 +78,6 @@ impl Command<SignerTestState, SignerTestContext> for MinerSubmitBlockCommit {
     ) -> impl Strategy<Value = CommandWrapper<SignerTestState, SignerTestContext>> {
         (1usize..=2usize).prop_map(move |miner_index| {
             CommandWrapper::new(MinerSubmitBlockCommit::new(ctx.clone(), miner_index))
-       })
+        })
     }
 }

testnet/stacks-node/src/tests/signer/commands/block_verify.rs

@@ -120,16 +120,12 @@ impl Command<SignerTestState, SignerTestContext> for ChainVerifyMinerBlockCount
     fn build(
         ctx: Arc<SignerTestContext>,
     ) -> impl Strategy<Value = CommandWrapper<SignerTestState, SignerTestContext>> {
-        (1usize..=2usize, 1usize..=5usize).prop_map(
-            move |(miner_index, expected_block_count)| {
-                CommandWrapper::new(
-                    ChainVerifyMinerBlockCount::after_boot_to_epoch3(
-                        ctx.clone(),
-                        miner_index,
-                        expected_block_count,
-                    ),
-                )
-            },
-        )
+        (1usize..=2usize, 1usize..=5usize).prop_map(move |(miner_index, expected_block_count)| {
+            CommandWrapper::new(ChainVerifyMinerBlockCount::after_boot_to_epoch3(
+                ctx.clone(),
+                miner_index,
+                expected_block_count,
+            ))
+        })
     }
 }

testnet/stacks-node/src/tests/signer/commands/context.rs

@@ -1,5 +1,4 @@
 use std::fmt::Debug;
-use std::sync::atomic::Ordering;
 use std::sync::{Arc, Mutex};
 use std::time::Duration;