fix: Fix mock dependency (#1507)

## What changes are included in this PR?

`mockall` dependency is introduced in
#1484, which is used for unit
test on retry.
But it's only used in unit tests, but not production code; so it should
be placed under dev section, and completely disabled in production code
path.
Current implementation unnecessarily increases crate size for production
usage.

## Are these changes tested?

N/A, a no-op change, a passed compilation should be enough

Author: dentiny

crates/iceberg/Cargo.toml

@@ -67,7 +67,6 @@ expect-test = { workspace = true }
 fnv = { workspace = true }
 futures = { workspace = true }
 itertools = { workspace = true }
-mockall = { workspace = true }
 moka = { version = "0.12.10", features = ["future"] }
 murmur3 = { workspace = true }
 num-bigint = { workspace = true }
@@ -97,6 +96,7 @@ zstd = { workspace = true }
 ctor = { workspace = true }
 expect-test = { workspace = true }
 iceberg_test_utils = { path = "../test_utils", features = ["tests"] }
+mockall = { workspace = true }
 pretty_assertions = { workspace = true }
 rand = { workspace = true }
 regex = { workspace = true }

crates/iceberg/src/catalog/mod.rs

@@ -29,6 +29,7 @@ use std::sync::Arc;
 use _serde::deserialize_snapshot;
 use async_trait::async_trait;
 pub use memory::MemoryCatalog;
+#[cfg(test)]
 use mockall::automock;
 use serde_derive::{Deserialize, Serialize};
 use typed_builder::TypedBuilder;
@@ -44,7 +45,7 @@ use crate::{Error, ErrorKind, Result};
 
 /// The catalog API for Iceberg Rust.
 #[async_trait]
-#[automock]
+#[cfg_attr(test, automock)]
 pub trait Catalog: Debug + Sync + Send {
     /// List namespaces inside the catalog.
     async fn list_namespaces(&self, parent: Option<&NamespaceIdent>)