chore(batcher): add testing feature, use automock only as test

commit-id:d05770a1

crates/starknet_batcher_types/Cargo.toml

@@ -5,16 +5,24 @@ edition.workspace = true
 license.workspace = true
 repository.workspace = true
 
+[features]
+testing = ["mockall"]
+
 [lints]
 workspace = true
 
 [dependencies]
 async-trait.workspace = true
 chrono = { workspace = true, features = ["serde"] }
 derive_more.workspace = true
-mockall.workspace = true
+mockall = { workspace = true, optional = true }
 papyrus_proc_macros.workspace = true
 serde = { workspace = true, features = ["derive"] }
 starknet_api.workspace = true
 starknet_sequencer_infra.workspace = true
 thiserror.workspace = true
+
+
+[dev-dependencies]
+# Enable self with "testing" feature in tests.
+starknet_batcher_types = { workspace = true, features = ["testing"] }

crates/starknet_batcher_types/src/communication.rs

@@ -1,8 +1,8 @@
 use std::sync::Arc;
 
 use async_trait::async_trait;
-use mockall::predicate::*;
-use mockall::*;
+#[cfg(any(feature = "testing", test))]
+use mockall::automock;
 use papyrus_proc_macros::handle_response_variants;
 use serde::{Deserialize, Serialize};
 use starknet_sequencer_infra::component_client::{
@@ -38,7 +38,7 @@ pub type SharedBatcherClient = Arc<dyn BatcherClient>;
 
 /// Serves as the batcher's shared interface. Requires `Send + Sync` to allow transferring and
 /// sharing resources (inputs, futures) across threads.
-#[automock]
+#[cfg_attr(any(test, feature = "testing"), automock)]
 #[async_trait]
 pub trait BatcherClient: Send + Sync {
     /// Starts the process of building a proposal.