fix(iota-sdk-types): fix feature flags for tests

Author: Thoralf-M

crates/iota-sdk-types/Cargo.toml

@@ -36,7 +36,7 @@ serde = [
 schemars = ["serde", "dep:schemars", "dep:serde_json"]
 rand = ["dep:rand_core"]
 hash = ["dep:blake2"]
-proptest = ["dep:proptest", "dep:test-strategy", "serde"]
+proptest = ["dep:proptest", "dep:test-strategy", "serde", "schemars"]
 
 [dependencies]
 base64ct = { workspace = true, features = ["alloc"] }

crates/iota-sdk-types/src/address.rs

@@ -284,7 +284,20 @@ impl schemars::JsonSchema for Address {
 
 #[cfg(test)]
 mod tests {
-    use test_strategy::proptest;
+    #[cfg(feature = "proptest")]
+    mod proptests {
+        use test_strategy::proptest;
+
+        use super::*;
+
+        #[proptest]
+        fn roundtrip_display_fromstr(address: Address) {
+            let s = address.to_string();
+            let a = s.parse::<Address>().unwrap();
+            assert_eq!(address, a);
+        }
+    }
+
     #[cfg(target_arch = "wasm32")]
     use wasm_bindgen_test::wasm_bindgen_test as test;
 
@@ -308,11 +321,4 @@ mod tests {
         let a: Address = serde_json::from_str("\"0x2\"").unwrap();
         println!("{a}");
     }
-
-    #[proptest]
-    fn roundtrip_display_fromstr(address: Address) {
-        let s = address.to_string();
-        let a = s.parse::<Address>().unwrap();
-        assert_eq!(address, a);
-    }
 }

crates/iota-sdk-types/src/crypto/mod.rs

@@ -129,7 +129,7 @@ macro_rules! impl_base64_helper {
             }
         }
 
-        #[cfg(test)]
+        #[cfg(all(test, feature = "proptest"))]
         mod $test_module {
             use test_strategy::proptest;
 

crates/iota-sdk-types/src/crypto/passkey.rs

@@ -407,6 +407,7 @@ impl proptest::arbitrary::Arbitrary for PasskeyAuthenticator {
     }
 }
 
+#[cfg(feature = "serde")]
 #[cfg(test)]
 mod tests {
     use crate::UserSignature;

crates/iota-sdk-types/src/crypto/signature.rs

@@ -822,13 +822,15 @@ mod serialization {
     #[cfg(test)]
     mod tests {
         use base64ct::{Base64, Encoding};
+        #[cfg(feature = "proptest")]
         use test_strategy::proptest;
         #[cfg(target_arch = "wasm32")]
         use wasm_bindgen_test::wasm_bindgen_test as test;
 
         use super::*;
 
         #[proptest]
+        #[cfg(feature = "proptest")]
         fn roundtrip_signature_scheme(scheme: SignatureScheme) {
             assert_eq!(Ok(scheme), SignatureScheme::from_byte(scheme.to_u8()));
         }

crates/iota-sdk-types/src/crypto/zklogin.rs

@@ -652,11 +652,41 @@ impl std::str::FromStr for Bn254FieldElement {
 
 #[cfg(test)]
 mod tests {
-    use std::str::FromStr;
+    #[cfg(feature = "proptest")]
+    mod proptests {
+        use std::str::FromStr;
+
+        use num_bigint::BigUint;
+        use proptest::prelude::*;
+        use test_strategy::proptest;
+
+        use super::*;
+
+        #[proptest]
+        fn dont_crash_on_large_inputs(
+            #[strategy(proptest::collection::vec(any::<u8>(), 33..1024))] bytes: Vec<u8>,
+        ) {
+            let big_int = BigUint::from_bytes_be(&bytes);
+            let radix10 = big_int.to_str_radix(10);
+
+            // doesn't crash
+            let _ = Bn254FieldElement::from_str(&radix10);
+        }
+
+        #[proptest]
+        fn valid_address_seeds(
+            #[strategy(proptest::collection::vec(any::<u8>(), 1..=32))] bytes: Vec<u8>,
+        ) {
+            let big_int = BigUint::from_bytes_be(&bytes);
+            let radix10 = big_int.to_str_radix(10);
+
+            let seed = Bn254FieldElement::from_str(&radix10).unwrap();
+            assert_eq!(radix10, seed.to_string());
+            // Ensure unpadded doesn't crash
+            seed.unpadded();
+        }
+    }
 
-    use num_bigint::BigUint;
-    use proptest::prelude::*;
-    use test_strategy::proptest;
     #[cfg(target_arch = "wasm32")]
     use wasm_bindgen_test::wasm_bindgen_test as test;
 
@@ -672,30 +702,6 @@ mod tests {
         seed.0[0] = 0;
         assert_eq!(seed.unpadded(), [1; 31].as_slice());
     }
-
-    #[proptest]
-    fn dont_crash_on_large_inputs(
-        #[strategy(proptest::collection::vec(any::<u8>(), 33..1024))] bytes: Vec<u8>,
-    ) {
-        let big_int = BigUint::from_bytes_be(&bytes);
-        let radix10 = big_int.to_str_radix(10);
-
-        // doesn't crash
-        let _ = Bn254FieldElement::from_str(&radix10);
-    }
-
-    #[proptest]
-    fn valid_address_seeds(
-        #[strategy(proptest::collection::vec(any::<u8>(), 1..=32))] bytes: Vec<u8>,
-    ) {
-        let big_int = BigUint::from_bytes_be(&bytes);
-        let radix10 = big_int.to_str_radix(10);
-
-        let seed = Bn254FieldElement::from_str(&radix10).unwrap();
-        assert_eq!(radix10, seed.to_string());
-        // Ensure unpadded doesn't crash
-        seed.unpadded();
-    }
 }
 
 #[cfg(feature = "serde")]

crates/iota-sdk-types/src/digest.rs

@@ -215,7 +215,7 @@ impl std::error::Error for DigestParseError {}
 // serialized
 pub type SigningDigest = [u8; Digest::LENGTH];
 
-#[cfg(test)]
+#[cfg(all(test, feature = "proptest"))]
 mod tests {
     use test_strategy::proptest;
 

crates/iota-sdk-types/src/gas.rs

@@ -123,15 +123,14 @@ impl std::fmt::Display for GasCostSummary {
     }
 }
 
-#[cfg(test)]
+#[cfg(all(test, feature = "serde"))]
 mod tests {
     #[cfg(target_arch = "wasm32")]
     use wasm_bindgen_test::wasm_bindgen_test as test;
 
     use super::*;
 
     #[test]
-    #[cfg(feature = "serde")]
     fn formats() {
         let actual = GasCostSummary {
             computation_cost: 42,

crates/iota-sdk-types/src/hash.rs

@@ -522,7 +522,7 @@ impl crate::ObjectId {
     }
 }
 
-#[cfg(test)]
+#[cfg(all(test, feature = "proptest"))]
 mod tests {
     use test_strategy::proptest;
 

crates/iota-sdk-types/src/move_package.rs

@@ -149,7 +149,7 @@ mod serialization {
     }
 }
 
-#[cfg(test)]
+#[cfg(all(test, feature = "serde"))]
 mod tests {
     use super::*;
 
@@ -161,7 +161,8 @@ mod tests {
         let new_json = serde_json::to_string(&package).unwrap();
         assert_eq!(new_json, PACKAGE);
     }
-
+    
+    #[cfg(feature = "hash")]
     #[test]
     fn test_digest() {
         let json_package: MovePackageData = serde_json::from_str(PACKAGE).unwrap();