@layerzerolabs/protocol-stellar-v2 0.2.121 → 0.2.123

package/{contracts/workers → workers}/dvn/src/tests/auth.rs

@@ -6,7 +6,7 @@ use crate::{
 use ed25519_dalek::{Signer, SigningKey};
 use rand::thread_rng;
 use soroban_sdk::{
-    auth::{ContractContext, Context},
+    auth::{Context, ContractContext},
     testutils::Address as _,
     vec, Address, BytesN, Env, IntoVal, Symbol, Val, Vec,
 };
@@ -35,10 +35,7 @@ impl Ed25519KeyPair {
 }
 
 /// Creates a single self-call auth context and matching Call for testing.
-fn make_self_call_context(
-    env: &Env,
-    contract_id: &soroban_sdk::Address,
-) -> (Vec<Context>, Vec<Call>) {
+fn make_self_call_context(env: &Env, contract_id: &soroban_sdk::Address) -> (Vec<Context>, Vec<Call>) {
     let fn_name = Symbol::new(env, "set_paused");
     let args: Vec<Val> = vec![env, true.into_val(env)];
 
@@ -67,12 +64,7 @@ fn make_upgrade_contexts(
     let upgrader_ctx = Context::Contract(ContractContext {
         contract: upgrader_id.clone(),
         fn_name: Symbol::new(env, "upgrade_and_migrate"),
-        args: vec![
-            env,
-            contract_id.clone().into_val(env),
-            wasm_hash_val.clone(),
-            migration_data_val.clone(),
-        ],
+        args: vec![env, contract_id.clone().into_val(env), wasm_hash_val, migration_data_val],
     });
     // [1]: upgrade self-call
     let upgrade_ctx = Context::Contract(ContractContext {
@@ -374,12 +366,7 @@ fn test_check_auth_sender_none_fails_when_admin_required() {
     let sig = setup.key_pairs[0].sign_bytes(&env, &hash);
 
     // Use Sender::None which should fail when admin is required
-    let tx_auth = TransactionAuthData {
-        vid: VID,
-        expiration,
-        signatures: vec![&env, sig],
-        sender: Sender::None,
-    };
+    let tx_auth = TransactionAuthData { vid: VID, expiration, signatures: vec![&env, sig], sender: Sender::None };
 
     let res = env.try_invoke_contract_check_auth::<DvnError>(
         &setup.contract_id,
@@ -410,26 +397,15 @@ fn test_check_auth_set_admin_bypasses_admin_verification() {
 
     let payload = BytesN::from_array(&env, &[0u8; 32]);
 
-    let calls: Vec<Call> = vec![
-        &env,
-        Call {
-            to: setup.contract_id.clone(),
-            func: Symbol::new(&env, "set_admin"),
-            args: Vec::new(&env),
-        },
-    ];
+    let calls: Vec<Call> =
+        vec![&env, Call { to: setup.contract_id.clone(), func: Symbol::new(&env, "set_admin"), args: Vec::new(&env) }];
 
     let dvn_client = LzDVNClient::new(&env, &setup.contract_id);
     let hash = dvn_client.hash_call_data(&VID, &expiration, &calls);
     let sig = setup.key_pairs[0].sign_bytes(&env, &hash);
 
     // Use Sender::None - should succeed for set_admin since it bypasses admin verification
-    let tx_auth = TransactionAuthData {
-        vid: VID,
-        expiration,
-        signatures: vec![&env, sig],
-        sender: Sender::None,
-    };
+    let tx_auth = TransactionAuthData { vid: VID, expiration, signatures: vec![&env, sig], sender: Sender::None };
 
     let res = env.try_invoke_contract_check_auth::<DvnError>(
         &setup.contract_id,
@@ -599,23 +575,14 @@ fn test_set_upgrader_requires_admin() {
 
     let calls: Vec<Call> = vec![
         &env,
-        Call {
-            to: setup.contract_id.clone(),
-            func: Symbol::new(&env, "set_upgrader"),
-            args: Vec::new(&env),
-        },
+        Call { to: setup.contract_id.clone(), func: Symbol::new(&env, "set_upgrader"), args: Vec::new(&env) },
     ];
 
     let dvn_client = LzDVNClient::new(&env, &setup.contract_id);
     let hash = dvn_client.hash_call_data(&VID, &expiration, &calls);
     let sig = setup.key_pairs[0].sign_bytes(&env, &hash);
 
-    let tx_auth = TransactionAuthData {
-        vid: VID,
-        expiration,
-        signatures: vec![&env, sig],
-        sender: Sender::None,
-    };
+    let tx_auth = TransactionAuthData { vid: VID, expiration, signatures: vec![&env, sig], sender: Sender::None };
 
     let res = env.try_invoke_contract_check_auth::<DvnError>(
         &setup.contract_id,
@@ -692,12 +659,7 @@ fn test_check_auth_upgrade_no_upgrader_set() {
     let hash = dvn_client.hash_call_data(&VID, &expiration, &calls);
     let sig = setup.key_pairs[0].sign_bytes(&env, &hash);
 
-    let tx_auth = TransactionAuthData {
-        vid: VID,
-        expiration,
-        signatures: vec![&env, sig],
-        sender: Sender::None,
-    };
+    let tx_auth = TransactionAuthData { vid: VID, expiration, signatures: vec![&env, sig], sender: Sender::None };
 
     let res = env.try_invoke_contract_check_auth::<DvnError>(
         &setup.contract_id,
@@ -733,12 +695,7 @@ fn test_check_auth_upgrade_wrong_upgrader() {
     let hash = dvn_client.hash_call_data(&VID, &expiration, &calls);
     let sig = setup.key_pairs[0].sign_bytes(&env, &hash);
 
-    let tx_auth = TransactionAuthData {
-        vid: VID,
-        expiration,
-        signatures: vec![&env, sig],
-        sender: Sender::None,
-    };
+    let tx_auth = TransactionAuthData { vid: VID, expiration, signatures: vec![&env, sig], sender: Sender::None };
 
     let res = env.try_invoke_contract_check_auth::<DvnError>(
         &setup.contract_id,
@@ -784,12 +741,7 @@ fn test_check_auth_upgrade_missing_migrate() {
     let hash = dvn_client.hash_call_data(&VID, &expiration, &dummy_calls);
     let sig = setup.key_pairs[0].sign_bytes(&env, &hash);
 
-    let tx_auth = TransactionAuthData {
-        vid: VID,
-        expiration,
-        signatures: vec![&env, sig],
-        sender: Sender::None,
-    };
+    let tx_auth = TransactionAuthData { vid: VID, expiration, signatures: vec![&env, sig], sender: Sender::None };
 
     let res = env.try_invoke_contract_check_auth::<DvnError>(
         &setup.contract_id,

package/{contracts/workers → workers}/dvn-fee-lib/Cargo.toml

@@ -2,12 +2,11 @@
 name = "dvn-fee-lib"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [features]
 library = []
-testutils = []
+testutils = ["soroban-sdk/testutils"]
 
 [lib]
 crate-type = ["cdylib", "rlib"]

package/{contracts/workers → workers}/executor/Cargo.toml

@@ -2,7 +2,6 @@
 name = "executor"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [lib]
@@ -11,7 +10,7 @@ doctest = false
 
 [features]
 library = []
-testutils = []
+testutils = ["soroban-sdk/testutils"]
 
 [dependencies]
 cfg-if = { workspace = true }
@@ -25,6 +24,8 @@ message-lib-common = { workspace = true }
 endpoint-v2 = { workspace = true, features = ["library"] }
 
 [dev-dependencies]
+# self-dep to enable this crate's `testutils` for its own test build
+executor = { path = ".", features = ["testutils"] }
 soroban-sdk = { workspace = true, features = ["testutils"] }
 rand = { workspace = true }
 ed25519-dalek = { workspace = true }

package/{contracts/workers → workers}/executor/src/auth.rs

@@ -111,13 +111,10 @@ impl LzExecutor {
         }
 
         // Helper path: first context must be on the registered executor helper
-        let helper_config =
-            ExecutorStorage::executor_helper(env).ok_or(ExecutorError::UnauthorizedContext)?;
+        let helper_config = ExecutorStorage::executor_helper(env).ok_or(ExecutorError::UnauthorizedContext)?;
 
         // Validate first context: must be the helper address with an allowed function name
-        if first_ctx.contract != helper_config.address
-            || !helper_config.allowed_functions.contains(first_fn_name)
-        {
+        if first_ctx.contract != helper_config.address || !helper_config.allowed_functions.contains(first_fn_name) {
             return Err(ExecutorError::UnauthorizedContext);
         }
 
@@ -128,9 +125,7 @@ impl LzExecutor {
         };
 
         let second_fn_name = &second_ctx.fn_name;
-        if *second_fn_name != Symbol::new(env, "lz_receive")
-            && *second_fn_name != Symbol::new(env, "lz_compose")
-        {
+        if *second_fn_name != Symbol::new(env, "lz_receive") && *second_fn_name != Symbol::new(env, "lz_compose") {
             return Err(ExecutorError::UnauthorizedContext);
         }
 

package/{contracts/workers → workers}/executor/src/storage.rs

@@ -1,5 +1,5 @@
 use common_macros::storage;
-use soroban_sdk::{contracttype, Address, Vec, Symbol};
+use soroban_sdk::{contracttype, Address, Symbol, Vec};
 
 use crate::DstConfig;
 

package/{contracts/workers → workers}/executor/src/tests/auth.rs

@@ -252,7 +252,12 @@ fn test_check_auth_rejects_invalid_helper_fn_name() {
     let bad_addr: Vec<Context> = vec![
         &setup.env,
         contract_ctx(&setup.env, wrong_helper, "execute", Vec::new(&setup.env)),
-        contract_ctx(&setup.env, Address::generate(&setup.env), "lz_receive", vec![&setup.env, 0i128.into_val(&setup.env)]),
+        contract_ctx(
+            &setup.env,
+            Address::generate(&setup.env),
+            "lz_receive",
+            vec![&setup.env, 0i128.into_val(&setup.env)],
+        ),
     ];
     assert_eq!(check_auth(&setup, &payload, sig, &bad_addr), Err(Ok(ExecutorError::UnauthorizedContext)));
 }

package/{contracts/workers → workers}/executor/src/tests/executor.rs

@@ -16,8 +16,8 @@ fn test_constructor_sets_endpoint_and_worker_config() {
 
     assert_eq!(setup.client.owner(), Some(setup.owner.clone()));
     assert_eq!(setup.client.admins(), setup.admins);
-    assert_eq!(setup.client.is_admin(&setup.admins.get(0).unwrap()), true);
-    assert_eq!(setup.client.is_supported_message_lib(&setup.send_lib), true);
+    assert!(setup.client.is_admin(&setup.admins.get(0).unwrap()));
+    assert!(setup.client.is_supported_message_lib(&setup.send_lib));
     assert_eq!(setup.client.message_libs(), vec![&setup.env, setup.send_lib.clone()]);
 
     assert_eq!(setup.client.endpoint(), setup.endpoint);
@@ -25,7 +25,7 @@ fn test_constructor_sets_endpoint_and_worker_config() {
     assert_eq!(setup.client.price_feed(), Some(setup.price_feed.clone()));
     assert_eq!(setup.client.worker_fee_lib(), Some(setup.worker_fee_lib.clone()));
     assert_eq!(setup.client.default_multiplier_bps(), setup.default_multiplier_bps);
-    assert_eq!(setup.client.paused(), false);
+    assert!(!setup.client.paused());
 }
 
 // =============================================================================
@@ -71,12 +71,12 @@ fn test_set_admin_add_and_remove() {
     // Add new admin
     setup.mock_owner_auth("set_admin", (&new_admin, true));
     setup.client.set_admin(&new_admin, &true);
-    assert_eq!(setup.client.is_admin(&new_admin), true);
+    assert!(setup.client.is_admin(&new_admin));
 
     // Remove that admin
     setup.mock_owner_auth("set_admin", (&new_admin, false));
     setup.client.set_admin(&new_admin, &false);
-    assert_eq!(setup.client.is_admin(&new_admin), false);
+    assert!(!setup.client.is_admin(&new_admin));
 }
 
 #[test]

package/{contracts/workers → workers}/executor/src/tests/setup.rs

@@ -167,8 +167,7 @@ impl<'a> TestSetup<'a> {
 
         // Register executor helper config directly in storage
         let executor_helper = Address::generate(&env);
-        let allowed_functions: Vec<Symbol> =
-            vec![&env, Symbol::new(&env, "execute"), Symbol::new(&env, "compose")];
+        let allowed_functions: Vec<Symbol> = vec![&env, Symbol::new(&env, "execute"), Symbol::new(&env, "compose")];
         env.as_contract(&contract_id, || {
             crate::storage::ExecutorStorage::set_executor_helper(
                 &env,

package/{contracts/workers → workers}/executor-fee-lib/Cargo.toml

@@ -2,7 +2,6 @@
 name = "executor-fee-lib"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [lib]
@@ -11,7 +10,7 @@ doctest = false
 
 [features]
 library = []
-testutils = []
+testutils = ["soroban-sdk/testutils"]
 
 [dependencies]
 cfg-if = { workspace = true }

package/{contracts/workers → workers}/executor-fee-lib/src/tests/executor_fee_lib.rs

@@ -546,13 +546,13 @@ fn test_is_v1_eid() {
     let env = Env::default();
 
     // V1: eid < 30000
-    assert_eq!(ExecutorFeeLib::is_v1_eid_for_test(&env, 0), true);
-    assert_eq!(ExecutorFeeLib::is_v1_eid_for_test(&env, 1), true);
-    assert_eq!(ExecutorFeeLib::is_v1_eid_for_test(&env, 29_999), true);
+    assert!(ExecutorFeeLib::is_v1_eid_for_test(&env, 0));
+    assert!(ExecutorFeeLib::is_v1_eid_for_test(&env, 1));
+    assert!(ExecutorFeeLib::is_v1_eid_for_test(&env, 29_999));
 
     // V2: eid >= 30000
-    assert_eq!(ExecutorFeeLib::is_v1_eid_for_test(&env, 30_000), false);
-    assert_eq!(ExecutorFeeLib::is_v1_eid_for_test(&env, 30_001), false);
+    assert!(!ExecutorFeeLib::is_v1_eid_for_test(&env, 30_000));
+    assert!(!ExecutorFeeLib::is_v1_eid_for_test(&env, 30_001));
 }
 
 #[test]

package/{contracts/workers → workers}/executor-fee-lib/src/tests/executor_option.rs

@@ -23,7 +23,7 @@ fn test_parse_executor_options_aggregates_values() {
     assert_eq!(agg.total_gas, 17); // 10 + 7
     assert_eq!(agg.total_value, 10); // 2 + 5 + 3
     assert_eq!(agg.num_lz_compose, 1);
-    assert_eq!(agg.ordered, true);
+    assert!(agg.ordered);
 }
 
 #[test]
@@ -53,7 +53,7 @@ fn test_parse_executor_options_accumulates_multiple_lz_receive_gas_and_value() {
     assert_eq!(agg.total_gas, 30);
     assert_eq!(agg.total_value, 5);
     assert_eq!(agg.num_lz_compose, 0);
-    assert_eq!(agg.ordered, false);
+    assert!(!agg.ordered);
 }
 
 #[test]

package/{contracts/workers → workers}/executor-helper/Cargo.toml

@@ -2,7 +2,6 @@
 name = "executor-helper"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [lib]

package/{contracts/workers → workers}/executor-helper/src/executor_helper.rs

@@ -120,7 +120,6 @@ impl ExecutorHelper {
     ) {
         ExecutorClient::new(env, executor).native_drop(admin, origin, &dst_eid, oapp, params);
     }
-
 }
 
 #[inline]

package/{contracts/workers → workers}/executor-helper/src/tests/executor_helper.rs

@@ -338,4 +338,3 @@ fn test_lz_compose_alert_with_empty_reason() {
     assert!(record.is_some());
     assert_eq!(record.unwrap().reason.len(), 0);
 }
-

package/{contracts/workers → workers}/fee-lib-interfaces/Cargo.toml

@@ -2,7 +2,6 @@
 name = "fee-lib-interfaces"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [lib]

package/{contracts/workers → workers}/price-feed/Cargo.toml

@@ -2,7 +2,6 @@
 name = "price-feed"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [lib]
@@ -11,7 +10,7 @@ doctest = false
 
 [features]
 library = []
-testutils = []
+testutils = ["soroban-sdk/testutils"]
 
 [dependencies]
 cfg-if = { workspace = true }

package/{contracts/workers → workers}/price-feed/src/tests/price_feed.rs

@@ -13,7 +13,7 @@ fn test_constructor_sets_owner_and_price_updater() {
     let setup = TestSetup::new();
 
     assert_eq!(setup.client.owner(), Some(setup.owner.clone()));
-    assert_eq!(setup.client.is_price_updater(&setup.price_updater), true);
+    assert!(setup.client.is_price_updater(&setup.price_updater));
 }
 
 #[test]
@@ -91,7 +91,7 @@ fn test_estimate_fee_by_eid_default_model() {
     //     = (16_500_000_000_000 * 1e20) / 1e20
     //     = 16_500_000_000_000
 
-    assert_eq!(estimate.total_gas_fee, 16_500_000_000_000 as i128);
+    assert_eq!(estimate.total_gas_fee, 16_500_000_000_000_i128);
     assert_eq!(estimate.price_ratio, price.price_ratio);
     assert_eq!(estimate.price_ratio_denominator, 10u128.pow(20));
     assert_eq!(estimate.native_price_usd, 0);
@@ -214,7 +214,7 @@ fn test_estimate_fee_by_eid_arbitrum_model_hardcoded_eid() {
     //     = 34_306 * 10_000_000
     //     = 343_060_000_000
 
-    assert_eq!(estimate.total_gas_fee, 343_060_000_000 as i128);
+    assert_eq!(estimate.total_gas_fee, 343_060_000_000_i128);
 
     assert_eq!(estimate.price_ratio, arb_price.price_ratio);
 }
@@ -237,7 +237,7 @@ fn test_estimate_fee_by_eid_arbitrum_model_hardcoded_eid_10143() {
     setup.mock_auth(&fee_lib, "estimate_fee_by_eid", (&fee_lib, 10143u32, 1000u32, 500u128));
     let estimate = setup.client.estimate_fee_by_eid(&fee_lib, &10143, &1000, &500);
 
-    assert_eq!(estimate.total_gas_fee, 343_060_000_000 as i128);
+    assert_eq!(estimate.total_gas_fee, 343_060_000_000_i128);
     assert_eq!(estimate.price_ratio, arb_price.price_ratio);
 }
 
@@ -299,7 +299,7 @@ fn test_estimate_fee_by_eid_arbitrum_model_configured_eid() {
     let estimate = setup.client.estimate_fee_by_eid(&fee_lib, &200, &1000, &500);
 
     // Should use Arbitrum model (same expected fee as the hardcoded Arbitrum test)
-    assert_eq!(estimate.total_gas_fee, 343_060_000_000 as i128);
+    assert_eq!(estimate.total_gas_fee, 343_060_000_000_i128);
     assert_eq!(estimate.price_ratio, arb_price.price_ratio);
 }
 
@@ -342,7 +342,7 @@ fn test_estimate_fee_by_eid_optimism_model_hardcoded_eid() {
     //          = 7_235_492_071_308 + 36_813_447_000
     //          = 7_272_305_518_308
 
-    assert_eq!(estimate.total_gas_fee, 7_272_305_518_308 as i128);
+    assert_eq!(estimate.total_gas_fee, 7_272_305_518_308_i128);
     assert_eq!(estimate.price_ratio, op_price.price_ratio);
 }
 
@@ -368,7 +368,7 @@ fn test_estimate_fee_by_eid_optimism_model_goerli() {
 
     // Deterministic expected fee (Optimism model):
     // L1 uses Ethereum Goerli (10121) with +3188 overhead, L2 uses Optimism Goerli (10132).
-    assert_eq!(estimate.total_gas_fee, 7_272_305_518_308 as i128);
+    assert_eq!(estimate.total_gas_fee, 7_272_305_518_308_i128);
     assert_eq!(estimate.price_ratio, op_price.price_ratio);
 }
 #[test]
@@ -402,7 +402,7 @@ fn test_estimate_fee_by_eid_optimism_model_configured_eid() {
     assert_eq!(estimate_300.total_gas_fee, estimate_111.total_gas_fee);
     assert_eq!(estimate_300.price_ratio, estimate_111.price_ratio);
     assert_eq!(estimate_300.price_ratio, op_price.price_ratio);
-    assert_eq!(estimate_300.total_gas_fee, 7_272_305_518_308 as i128);
+    assert_eq!(estimate_300.total_gas_fee, 7_272_305_518_308_i128);
 }
 
 #[test]
@@ -511,12 +511,12 @@ fn test_set_price_updater_add_and_remove() {
     // Add new price updater
     setup.mock_owner_auth("set_price_updater", (&new_updater, true));
     setup.client.set_price_updater(&new_updater, &true);
-    assert_eq!(setup.client.is_price_updater(&new_updater), true);
+    assert!(setup.client.is_price_updater(&new_updater));
 
     // Remove price updater
     setup.mock_owner_auth("set_price_updater", (&new_updater, false));
     setup.client.set_price_updater(&new_updater, &false);
-    assert_eq!(setup.client.is_price_updater(&new_updater), false);
+    assert!(!setup.client.is_price_updater(&new_updater));
 }
 
 #[test]
@@ -806,7 +806,7 @@ fn test_is_price_updater_returns_false_for_unknown_address() {
     let setup = TestSetup::new();
     let unknown = Address::generate(&setup.env);
 
-    assert_eq!(setup.client.is_price_updater(&unknown), false);
+    assert!(!setup.client.is_price_updater(&unknown));
 }
 
 // =============================================================================

package/{contracts/workers → workers}/worker/Cargo.toml

@@ -2,7 +2,6 @@
 name = "worker"
 version.workspace = true
 edition.workspace = true
-license.workspace = true
 publish = false
 
 [lib]

package/{contracts/workers → workers}/worker/src/tests/worker.rs

@@ -27,12 +27,12 @@ fn test_set_paused_toggles_state() {
     setup.mock_owner_auth("set_paused", (true,));
     setup.client.set_paused(&true);
     assert_eq_event(&setup.env, &setup.contract_id, Paused { pauser: setup.owner.clone() });
-    assert_eq!(setup.client.paused(), true);
+    assert!(setup.client.paused());
 
     setup.mock_owner_auth("set_paused", (false,));
     setup.client.set_paused(&false);
     assert_eq_event(&setup.env, &setup.contract_id, Unpaused { unpauser: setup.owner.clone() });
-    assert_eq!(setup.client.paused(), false);
+    assert!(!setup.client.paused());
 }
 
 // allowlist
@@ -68,22 +68,22 @@ fn test_allowlist_remove_decrements_size_and_restores_default_acl() {
     let oapp_b = Address::generate(&setup.env);
 
     // Add allowlist entry => allowlist becomes non-empty; non-allowlisted is denied
-    assert_eq!(setup.client.is_on_allowlist(&oapp_a), false);
+    assert!(!setup.client.is_on_allowlist(&oapp_a));
     setup.mock_owner_auth("set_allowlist", (&oapp_a, true));
     setup.client.set_allowlist(&oapp_a, &true);
     assert_eq_event(&setup.env, &setup.contract_id, SetAllowlist { oapp: oapp_a.clone(), allowed: true });
-    assert_eq!(setup.client.is_on_allowlist(&oapp_a), true);
+    assert!(setup.client.is_on_allowlist(&oapp_a));
     assert_eq!(setup.client.allowlist_size(), 1);
-    assert_eq!(setup.client.has_acl(&oapp_a), true);
-    assert_eq!(setup.client.has_acl(&oapp_b), false);
+    assert!(setup.client.has_acl(&oapp_a));
+    assert!(!setup.client.has_acl(&oapp_b));
 
     // Remove last allowlist entry => allowlist empty; default allows all (unless denylisted)
     setup.mock_owner_auth("set_allowlist", (&oapp_a, false));
     setup.client.set_allowlist(&oapp_a, &false);
     assert_eq_event(&setup.env, &setup.contract_id, SetAllowlist { oapp: oapp_a.clone(), allowed: false });
-    assert_eq!(setup.client.is_on_allowlist(&oapp_a), false);
+    assert!(!setup.client.is_on_allowlist(&oapp_a));
     assert_eq!(setup.client.allowlist_size(), 0);
-    assert_eq!(setup.client.has_acl(&oapp_b), true);
+    assert!(setup.client.has_acl(&oapp_b));
 }
 
 // denylist
@@ -129,7 +129,7 @@ fn test_message_lib_add_remove_and_errors() {
         &setup.contract_id,
         SetSupportedMessageLib { message_lib: new_lib.clone(), supported: true },
     );
-    assert_eq!(setup.client.is_supported_message_lib(&new_lib), true);
+    assert!(setup.client.is_supported_message_lib(&new_lib));
 
     // Remove it
     setup.mock_owner_auth("set_supported_message_lib", (&new_lib, false));
@@ -139,10 +139,10 @@ fn test_message_lib_add_remove_and_errors() {
         &setup.contract_id,
         SetSupportedMessageLib { message_lib: new_lib.clone(), supported: false },
     );
-    assert_eq!(setup.client.is_supported_message_lib(&new_lib), false);
+    assert!(!setup.client.is_supported_message_lib(&new_lib));
 
     // Existing is still supported
-    assert_eq!(setup.client.is_supported_message_lib(&existing), true);
+    assert!(setup.client.is_supported_message_lib(&existing));
 }
 
 #[test]
@@ -245,41 +245,41 @@ fn test_acl_allowlist_denylist_precedence() {
     let oapp_b = Address::generate(&setup.env);
 
     // Empty allowlist => allowed unless denylisted
-    assert_eq!(setup.client.has_acl(&oapp_a), true);
+    assert!(setup.client.has_acl(&oapp_a));
 
     // Denylist overrides everything
-    assert_eq!(setup.client.is_on_denylist(&oapp_a), false);
+    assert!(!setup.client.is_on_denylist(&oapp_a));
     setup.mock_owner_auth("set_denylist", (&oapp_a, true));
     setup.client.set_denylist(&oapp_a, &true);
     assert_eq_event(&setup.env, &setup.contract_id, SetDenylist { oapp: oapp_a.clone(), denied: true });
-    assert_eq!(setup.client.is_on_denylist(&oapp_a), true);
-    assert_eq!(setup.client.has_acl(&oapp_a), false);
+    assert!(setup.client.is_on_denylist(&oapp_a));
+    assert!(!setup.client.has_acl(&oapp_a));
 
     // Remove from denylist => allowed again (since allowlist is empty)
     setup.mock_owner_auth("set_denylist", (&oapp_a, false));
     setup.client.set_denylist(&oapp_a, &false);
     assert_eq_event(&setup.env, &setup.contract_id, SetDenylist { oapp: oapp_a.clone(), denied: false });
-    assert_eq!(setup.client.is_on_denylist(&oapp_a), false);
-    assert_eq!(setup.client.has_acl(&oapp_a), true);
+    assert!(!setup.client.is_on_denylist(&oapp_a));
+    assert!(setup.client.has_acl(&oapp_a));
 
     // Add allowlist entry => allowlist becomes non-empty; non-allowlisted is denied
     setup.mock_owner_auth("set_allowlist", (&oapp_a, true));
     setup.client.set_allowlist(&oapp_a, &true);
     assert_eq!(setup.client.allowlist_size(), 1);
-    assert_eq!(setup.client.has_acl(&oapp_a), true);
-    assert_eq!(setup.client.has_acl(&oapp_b), false);
+    assert!(setup.client.has_acl(&oapp_a));
+    assert!(!setup.client.has_acl(&oapp_b));
 
     // Denylist must override allowlist even for the same OApp.
     setup.mock_owner_auth("set_denylist", (&oapp_a, true));
     setup.client.set_denylist(&oapp_a, &true);
     assert_eq_event(&setup.env, &setup.contract_id, SetDenylist { oapp: oapp_a.clone(), denied: true });
-    assert_eq!(setup.client.has_acl(&oapp_a), false);
+    assert!(!setup.client.has_acl(&oapp_a));
 
     // Removing from denylist should restore allowlist effect.
     setup.mock_owner_auth("set_denylist", (&oapp_a, false));
     setup.client.set_denylist(&oapp_a, &false);
     assert_eq_event(&setup.env, &setup.contract_id, SetDenylist { oapp: oapp_a.clone(), denied: false });
-    assert_eq!(setup.client.has_acl(&oapp_a), true);
+    assert!(setup.client.has_acl(&oapp_a));
 }
 
 // uninitialized getters (BareWorker)
@@ -369,7 +369,7 @@ fn test_init_worker_can_reinitialize() {
 fn test_init_worker_sets_config_and_defaults() {
     let setup = TestSetup::new();
 
-    assert_eq!(setup.client.paused(), false);
+    assert!(!setup.client.paused());
     assert_eq!(setup.client.admins(), setup.admins);
     assert_eq!(setup.client.message_libs(), setup.message_libs);
     assert_eq!(setup.client.price_feed(), Some(setup.price_feed));
@@ -379,7 +379,7 @@ fn test_init_worker_sets_config_and_defaults() {
     assert_eq!(setup.client.allowlist_size(), 0);
 
     let lib = setup.message_libs.get(0).unwrap();
-    assert_eq!(setup.client.is_supported_message_lib(&lib), true);
+    assert!(setup.client.is_supported_message_lib(&lib));
 }
 
 // admin management (admins list)
@@ -392,12 +392,12 @@ fn test_admin_management_by_owner_adds_and_removes_admin() {
     setup.mock_owner_auth("set_admin", (&new_admin, true));
     setup.client.set_admin(&new_admin, &true);
     assert_eq_event(&setup.env, &setup.contract_id, SetAdmin { admin: new_admin.clone(), active: true });
-    assert_eq!(setup.client.is_admin(&new_admin), true);
+    assert!(setup.client.is_admin(&new_admin));
 
     setup.mock_owner_auth("set_admin", (&new_admin, false));
     setup.client.set_admin(&new_admin, &false);
     assert_eq_event(&setup.env, &setup.contract_id, SetAdmin { admin: new_admin.clone(), active: false });
-    assert_eq!(setup.client.is_admin(&new_admin), false);
+    assert!(!setup.client.is_admin(&new_admin));
 }
 
 #[test]
@@ -454,7 +454,7 @@ fn test_admin_management_can_remove_last_admin() {
 
     // Removing the last admin is now allowed
     client.set_admin(&only_admin, &false);
-    assert_eq!(client.is_admin(&only_admin), false);
+    assert!(!client.is_admin(&only_admin));
     assert_eq!(client.admins().len(), 0);
 }
 
@@ -467,7 +467,7 @@ fn test_admin_management_by_admin_can_add_admin() {
     setup.mock_auth(&caller, "set_admin_by_admin_for_test", (&caller, &new_admin, true));
     setup.client.set_admin_by_admin_for_test(&caller, &new_admin, &true);
     assert_eq_event(&setup.env, &setup.contract_id, SetAdmin { admin: new_admin.clone(), active: true });
-    assert_eq!(setup.client.is_admin(&new_admin), true);
+    assert!(setup.client.is_admin(&new_admin));
 }
 
 #[test]