feat: Add comprehensive test suite for Soroban smart contracts

contracts/assetsup/src/detokenization.rs

@@ -73,7 +73,8 @@ pub fn execute_detokenization(env: &Env, asset_id: u64, proposal_id: u64) -> Res
 
     // Get list of all token holders before clearing
     let holders_list_key = TokenDataKey::TokenHoldersList(asset_id);
-    let holders = store.get::<_, soroban_sdk::Vec<Address>>(&holders_list_key)
+    let holders = store
+        .get::<_, soroban_sdk::Vec<Address>>(&holders_list_key)
         .ok_or(Error::AssetNotTokenized)?;
 
     // Remove all token holder records

contracts/assetsup/src/lib.rs

@@ -822,7 +822,10 @@ impl AssetUpContract {
     }
 
     /// Get a specific policy
-    pub fn get_insurance_policy(env: Env, policy_id: BytesN<32>) -> Option<insurance::InsurancePolicy> {
+    pub fn get_insurance_policy(
+        env: Env,
+        policy_id: BytesN<32>,
+    ) -> Option<insurance::InsurancePolicy> {
         insurance::get_policy(env, policy_id)
     }
 

contracts/assetsup/src/tests/admin.rs

@@ -0,0 +1,105 @@
+use crate::tests::helpers::*;
+use soroban_sdk::{testutils::Address as _, Address};
+
+#[test]
+fn test_update_admin_success() {
+    let env = create_env();
+    let (admin, new_admin, _, _) = create_mock_addresses(&env);
+    let client = initialize_contract(&env, &admin);
+
+    env.mock_all_auths();
+    client.update_admin(&new_admin);
+
+    // Verify admin was updated
+    assert_eq!(client.get_admin(), new_admin);
+
+    // Verify new admin is authorized registrar
+    assert!(client.is_authorized_registrar(&new_admin));
+
+    // Verify old admin is no longer authorized registrar
+    assert!(!client.is_authorized_registrar(&admin));
+}
+
+#[test]
+#[should_panic(expected = "Error(Contract, #39)")]
+fn test_update_admin_zero_address() {
+    let env = create_env();
+    let admin = Address::generate(&env);
+    let client = initialize_contract(&env, &admin);
+
+    let zero_address = Address::from_string(&soroban_sdk::String::from_str(
+        &env,
+        "GAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAWHF",
+    ));
+
+    env.mock_all_auths();
+
+    // Should panic with InvalidOwnerAddress error
+    client.update_admin(&zero_address);
+}
+
+#[test]
+fn test_pause_unpause_contract() {
+    let env = create_env();
+    let admin = Address::generate(&env);
+    let client = initialize_contract(&env, &admin);
+
+    env.mock_all_auths();
+
+    // Initially not paused
+    assert!(!client.is_paused());
+
+    // Pause contract
+    client.pause_contract();
+    assert!(client.is_paused());
+
+    // Unpause contract
+    client.unpause_contract();
+    assert!(!client.is_paused());
+}
+
+#[test]
+fn test_add_authorized_registrar() {
+    let env = create_env();
+    let (admin, user1, _, _) = create_mock_addresses(&env);
+    let client = initialize_contract(&env, &admin);
+
+    env.mock_all_auths();
+
+    // Initially user1 is not authorized
+    assert!(!client.is_authorized_registrar(&user1));
+
+    // Add user1 as authorized registrar
+    client.add_authorized_registrar(&user1);
+    assert!(client.is_authorized_registrar(&user1));
+}
+
+#[test]
+fn test_remove_authorized_registrar() {
+    let env = create_env();
+    let (admin, user1, _, _) = create_mock_addresses(&env);
+    let client = initialize_contract(&env, &admin);
+
+    env.mock_all_auths();
+
+    // Add user1 as authorized registrar
+    client.add_authorized_registrar(&user1);
+    assert!(client.is_authorized_registrar(&user1));
+
+    // Remove user1 from authorized registrars
+    client.remove_authorized_registrar(&user1);
+    assert!(!client.is_authorized_registrar(&user1));
+}
+
+#[test]
+#[should_panic(expected = "Error(Contract, #8)")]
+fn test_remove_admin_from_registrars() {
+    let env = create_env();
+    let admin = Address::generate(&env);
+    let client = initialize_contract(&env, &admin);
+
+    env.mock_all_auths();
+
+    // Should panic with Unauthorized error - cannot remove admin
+    client.remove_authorized_registrar(&admin);
+}