chore: move p2p::conv tests back to stackslib

Author: kantai

.github/workflows/bitcoin-tests.yml

@@ -121,19 +121,6 @@ jobs:
           - tests::nakamoto_integrations::follower_bootup_across_multiple_cycles
           - tests::nakamoto_integrations::utxo_check_on_startup_panic
           - tests::nakamoto_integrations::utxo_check_on_startup_recover
-          - tests::p2p::convergence::test_walk_ring_allow_15
-          - tests::p2p::convergence::test_walk_ring_15_plain
-          - tests::p2p::convergence::test_walk_ring_15_pingback
-          - tests::p2p::convergence::test_walk_ring_15_org_biased
-          - tests::p2p::convergence::test_walk_line_allowed_15
-          - tests::p2p::convergence::test_walk_line_15_plain
-          - tests::p2p::convergence::test_walk_line_15_org_biased
-          - tests::p2p::convergence::test_walk_line_15_pingback
-          - tests::p2p::convergence::test_walk_star_allowed_15
-          - tests::p2p::convergence::test_walk_star_15_plain
-          - tests::p2p::convergence::test_walk_star_15_pingback
-          - tests::p2p::convergence::test_walk_star_15_org_biased
-          - tests::p2p::convergence::test_walk_inbound_line_15
           # Do not run this one until we figure out why it fails in CI
           # - tests::neon_integrations::bitcoin_reorg_flap
           # - tests::neon_integrations::bitcoin_reorg_flap_with_follower

Cargo.lock

@@ -59,10 +59,6 @@ siphasher = "0.3.7"
 wsts = { workspace = true }
 hashbrown = { workspace = true }
 rusqlite = { workspace = true }
-rstest = { version = "0.17.0", optional = true }
-rstest_reuse = { version = "0.5.0", optional = true }
-stdext = { version = "0.3.1", optional = true }
-stx-genesis = { path = "../stx-genesis", optional = true }
 
 [target.'cfg(not(any(target_os = "macos",target_os="windows", target_arch = "arm" )))'.dependencies]
 tikv-jemallocator = {workspace = true}
@@ -99,12 +95,13 @@ features = ["std"]
 assert-json-diff = "1.0.0"
 criterion = "0.3.5"
 stdext = "0.3.1"
-stx-genesis = { path = "../stx-genesis" }
+stx-genesis = { path = "../stx-genesis"}
 clarity = { features = ["default", "testing"], path = "../clarity" }
 stacks-common = { features = ["default", "testing"], path = "../stacks-common" }
 rstest = "0.17.0"
 rstest_reuse = "0.5.0"
 mutants = "0.0.3"
+rlimit = "0.10.2"
 
 [features]
 default = []
@@ -113,7 +110,7 @@ disable-costs = []
 developer-mode = ["clarity/developer-mode"]
 monitoring_prom = ["prometheus"]
 slog_json = ["slog-json", "stacks-common/slog_json", "clarity/slog_json", "pox-locking/slog_json"]
-testing = ["stdext", "rstest", "rstest_reuse", "stx-genesis"]
+testing = []
 
 [target.'cfg(all(any(target_arch = "x86_64", target_arch = "x86", target_arch = "aarch64"), not(any(target_os="windows"))))'.dependencies]
 sha2 = { version = "0.10", features = ["asm"] }

stackslib/Cargo.toml

@@ -167,7 +167,7 @@ impl BitcoinIndexerConfig {
         }
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn test_default(spv_headers_path: String) -> BitcoinIndexerConfig {
         BitcoinIndexerConfig {
             peer_host: "127.0.0.1".to_string(),
@@ -203,7 +203,7 @@ impl BitcoinIndexerRuntime {
 }
 
 impl BitcoinIndexer {
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn new(
         config: BitcoinIndexerConfig,
         runtime: BitcoinIndexerRuntime,
@@ -216,7 +216,7 @@ impl BitcoinIndexer {
         }
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn new_unit_test(working_dir: &str) -> BitcoinIndexer {
         let mut working_dir_path = PathBuf::from(working_dir);
         if fs::metadata(&working_dir_path).is_err() {
@@ -861,7 +861,7 @@ impl BitcoinIndexer {
         Ok(new_tip)
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn raw_store_header(&mut self, header: BurnchainBlockHeader) -> Result<(), btc_error> {
         let mut spv_client = SpvClient::new(
             &self.config.spv_headers_path,
@@ -887,7 +887,7 @@ impl BitcoinIndexer {
         Ok(())
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn mock_bitcoin_header(
         parent_block_hash: &BurnchainHeaderHash,
         timestamp: u32,

stackslib/src/burnchains/bitcoin/indexer.rs

@@ -182,7 +182,7 @@ impl SpvClient {
         Ok(client)
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn new_without_migration(
         headers_path: &str,
         start_block: u64,
@@ -211,7 +211,7 @@ impl SpvClient {
         Ok(client)
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn disable_check_txcount(&mut self) {
         self.check_txcount = false;
     }
@@ -220,7 +220,7 @@ impl SpvClient {
         &self.headers_db
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn conn_mut(&mut self) -> &mut DBConn {
         &mut self.headers_db
     }
@@ -277,7 +277,7 @@ impl SpvClient {
             .and_then(|_| Ok(()))
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn test_db_migrate(conn: &mut DBConn) -> Result<(), btc_error> {
         SpvClient::db_migrate(conn)
     }
@@ -925,7 +925,7 @@ impl SpvClient {
         Ok(())
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn test_write_block_headers(
         &mut self,
         height: u64,

stackslib/src/burnchains/bitcoin/spv.rs

@@ -623,7 +623,7 @@ impl Burnchain {
         ret
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn default_unittest(
         first_block_height: u64,
         first_block_hash: &BurnchainHeaderHash,

stackslib/src/burnchains/burnchain.rs

@@ -57,7 +57,7 @@ pub mod burnchain;
 pub mod db;
 pub mod indexer;
 
-#[cfg(any(test, feature = "testing"))]
+#[cfg(test)]
 pub mod tests;
 
 pub struct Txid(pub [u8; 32]);
@@ -351,7 +351,7 @@ impl PoxConstants {
             _shadow: PhantomData,
         }
     }
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn test_default() -> PoxConstants {
         // 20 reward slots; 10 prepare-phase slots
         PoxConstants::new(
@@ -369,7 +369,7 @@ impl PoxConstants {
         )
     }
 
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     /// Create a PoX constants used in tests with 5-block cycles,
     ///  3-block prepare phases, a threshold of 3, rejection fraction of 25%,
     ///  a participation threshold of 5% and no sunset or transition to pox-2 or beyond.
@@ -821,7 +821,7 @@ impl From<btc_error> for Error {
 }
 
 impl BurnchainView {
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn make_test_data(&mut self) {
         let oldest_height = if self.burn_stable_block_height < MAX_NEIGHBOR_BLOCK_DELAY {
             0

stackslib/src/burnchains/mod.rs

@@ -49,7 +49,7 @@ impl BurnchainDB {
 
     /// Get back all of the parsed burnchain operations for a given block.
     /// Used in testing to replay burnchain data.
-    #[cfg(any(test, feature = "testing"))]
+    #[cfg(test)]
     pub fn get_burnchain_block_ops(
         &self,
         block_hash: &BurnchainHeaderHash,

stackslib/src/burnchains/tests/db.rs

@@ -13,7 +13,6 @@
 //
 // You should have received a copy of the GNU General Public License
 // along with this program.  If not, see <http://www.gnu.org/licenses/>.
-#![allow(unused)]
 
 pub mod affirmation;
 pub mod burnchain;

stackslib/src/burnchains/tests/mod.rs

@@ -5482,7 +5482,7 @@ impl<'a> SortitionHandleTx<'a> {
             sn.canonical_stacks_tip_height,
         )?;
 
-        #[cfg(any(test, feature = "testing"))]
+        #[cfg(test)]
         {
             let (block_consensus_hash, block_bhh) =
                 SortitionDB::get_canonical_stacks_chain_tip_hash(self).unwrap();
@@ -6566,7 +6566,7 @@ impl ChainstateDB for SortitionDB {
     }
 }
 
-#[cfg(any(test, feature = "testing"))]
+#[cfg(test)]
 pub mod tests {
     use std::sync::mpsc::sync_channel;
     use std::thread;