chore: move generators module (paradigmxyz#8429)

mw2000 · Jun 5, 2024 · 080c01e · 080c01e
1 parent e830887
commit 080c01e
Show file tree

Hide file tree

Showing 8 changed files with 19 additions and 14 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/crates/interfaces/Cargo.toml b/crates/interfaces/Cargo.toml
@@ -18,9 +18,11 @@ reth-fs-util.workspace = true
 reth-network-p2p.workspace = true
 reth-storage-errors.workspace = true
 
+reth-testing-utils = { workspace = true, optional = true }
+
 # misc
 thiserror.workspace = true
 
 [features]
-test-utils = ["reth-consensus/test-utils", "reth-network-p2p/test-utils"]
+test-utils = ["reth-consensus/test-utils", "reth-network-p2p/test-utils", "reth-testing-utils"]
 clap = ["reth-storage-errors/clap"]
diff --git a/crates/interfaces/src/lib.rs b/crates/interfaces/src/lib.rs
@@ -36,4 +36,7 @@ pub use reth_blockchain_tree_api as blockchain_tree;
 
 /// Common test helpers for mocking out Consensus, Downloaders and Header Clients.
 #[cfg(feature = "test-utils")]
-pub use reth_network_p2p::test_utils;
+pub mod test_utils {
+    pub use reth_network_p2p::test_utils::*;
+    pub use reth_testing_utils::generators;
+}
diff --git a/crates/net/p2p/Cargo.toml b/crates/net/p2p/Cargo.toml
@@ -28,13 +28,7 @@ auto_impl.workspace = true
 thiserror.workspace = true
 tracing.workspace = true
 
-secp256k1 = { workspace = true, default-features = false, features = [
-    "alloc",
-    "recovery",
-    "rand",
-], optional = true }
 parking_lot = { workspace = true, optional = true }
-rand = { workspace = true, optional = true }
 
 [dev-dependencies]
 reth-consensus = { workspace = true, features = ["test-utils"] }
@@ -45,4 +39,4 @@ tokio = { workspace = true, features = ["full"] }
 secp256k1 = { workspace = true, features = ["alloc", "recovery", "rand"] }
 
 [features]
-test-utils = ["reth-consensus/test-utils", "secp256k1", "rand", "parking_lot"]
+test-utils = ["reth-consensus/test-utils", "parking_lot"]
diff --git a/crates/net/p2p/src/test_utils/mod.rs b/crates/net/p2p/src/test_utils/mod.rs
@@ -2,9 +2,6 @@ mod bodies;
 mod full_block;
 mod headers;
 
-/// Generators for different data structures like block headers, block bodies and ranges of those.
-pub mod generators;
-
 pub use bodies::*;
 pub use full_block::*;
 pub use headers::*;
diff --git a/testing/testing-utils/Cargo.toml b/testing/testing-utils/Cargo.toml
@@ -12,6 +12,9 @@ repository.workspace = true
 workspace = true
 
 [dependencies]
-secp256k1.workspace = true
-alloy-genesis.workspace = true
 reth-primitives.workspace = true
+
+alloy-genesis.workspace = true
+
+secp256k1.workspace = true
+rand.workspace = true
diff --git a/crates/net/p2p/src/test_utils/generators.rs → testing/testing-utils/src/generators.rs b/crates/net/p2p/src/test_utils/generators.rs → testing/testing-utils/src/generators.rs
@@ -1,3 +1,5 @@
+//! Generators for different data structures like block headers, block bodies and ranges of those.
+
 pub use rand::Rng;
 use rand::{
     distributions::uniform::SampleRange, rngs::StdRng, seq::SliceRandom, thread_rng, SeedableRng,

diff --git a/testing/testing-utils/src/lib.rs b/testing/testing-utils/src/lib.rs
@@ -11,3 +11,5 @@
 pub mod genesis_allocator;
 
 pub use genesis_allocator::GenesisAllocator;
+
+pub mod generators;
Original file line number	Diff line number	Diff line change
Expand Up		@@ -11,3 +11,5 @@
		pub mod genesis_allocator;

		pub use genesis_allocator::GenesisAllocator;

		pub mod generators;