Feat/issues 276 277 278 283

contract/contracts/hello-world/src/autoshare_logic.rs

@@ -265,7 +265,9 @@ pub fn initialize_admin(env: Env, admin: Address) {
 
         // Initialize empty supported tokens list in instance storage
         let empty_tokens: Vec<Address> = Vec::new(&env);
-        env.storage().instance().set(&INSTANCE_TOKENS, &empty_tokens);
+        env.storage()
+            .instance()
+            .set(&INSTANCE_TOKENS, &empty_tokens);
     }
 }
 
@@ -336,8 +338,6 @@ pub fn pause(env: Env, admin: Address) -> Result<(), Error> {
     }
 
     env.storage().instance().set(&INSTANCE_PAUSED, &true);
-    ContractPaused {}.publish(&env);
-    env.storage().persistent().set(&pause_key, &true);
     ContractPaused {
         category: NotificationCategory::Admin,
         priority: NotificationPriority::High,
@@ -361,8 +361,6 @@ pub fn unpause(env: Env, admin: Address) -> Result<(), Error> {
     }
 
     env.storage().instance().set(&INSTANCE_PAUSED, &false);
-    ContractUnpaused {}.publish(&env);
-    env.storage().persistent().set(&pause_key, &false);
     ContractUnpaused {
         category: NotificationCategory::Admin,
         priority: NotificationPriority::High,
@@ -468,10 +466,7 @@ pub fn set_usage_fee(env: Env, fee: u32, admin: Address) -> Result<(), Error> {
 }
 
 pub fn get_usage_fee(env: Env) -> u32 {
-    env.storage()
-        .instance()
-        .get(&INSTANCE_FEE)
-        .unwrap_or(10u32)
+    env.storage().instance().get(&INSTANCE_FEE).unwrap_or(10u32)
 }
 
 // ============================================================================
@@ -1166,4 +1161,3 @@ pub fn extend_notification_expiry(
 
     Ok(())
 }
-

contract/contracts/hello-world/src/base/events.rs

@@ -252,4 +252,3 @@ pub struct NotificationExtended {
     pub priority: NotificationPriority,
     pub new_expires_at: u64,
 }
-

contract/contracts/hello-world/src/lib.rs

@@ -312,7 +312,6 @@ impl AutoShareContract {
     ) -> bool {
         preferences_logic::is_category_enabled(env, recipient, category)
     }
-}
 
     // ============================================================================
     // Scheduled Notification Management
@@ -387,8 +386,13 @@ impl AutoShareContract {
         caller: Address,
         extension_seconds: u64,
     ) {
-        autoshare_logic::extend_notification_expiry(env, notification_id, caller, extension_seconds)
-            .unwrap();
+        autoshare_logic::extend_notification_expiry(
+            env,
+            notification_id,
+            caller,
+            extension_seconds,
+        )
+        .unwrap();
     }
 }
 
@@ -407,6 +411,8 @@ mod storage_optimization_test;
 #[cfg(test)]
 #[path = "tests/preferences_test.rs"]
 mod preferences_test;
+
+#[cfg(test)]
 mod tests {
     #[path = "../tests/autoshare_test.rs"]
     mod autoshare_test;

contract/contracts/hello-world/src/preferences_logic.rs

@@ -4,8 +4,8 @@
 /// Preferences are stored in persistent storage keyed by recipient address.
 use crate::base::errors::Error;
 use crate::base::preferences::{
-    CategoryPreference, ChannelPreference, DeliveryChannel, NotificationCategory,
-    RecipientPreferences, default_categories, default_channels, load_preferences, save_preferences,
+    default_categories, default_channels, load_preferences, save_preferences, CategoryPreference,
+    ChannelPreference, DeliveryChannel, NotificationCategory, RecipientPreferences,
 };
 use soroban_sdk::{Address, Env, Vec};
 
@@ -76,7 +76,9 @@ pub fn set_channel_preference(
     }
 
     if !found {
-        prefs.channels.push_back(ChannelPreference { channel, enabled });
+        prefs
+            .channels
+            .push_back(ChannelPreference { channel, enabled });
     }
 
     prefs.updated_at = env.ledger().timestamp();

contract/contracts/hello-world/src/tests/expiration_test.rs

@@ -477,4 +477,3 @@ fn test_cannot_extend_by_zero_seconds() {
     // Must panic
     client.extend_notification_expiry(&id, &creator, &0);
 }
-

contract/contracts/hello-world/src/tests/preferences_test.rs

@@ -6,7 +6,7 @@ mod preferences_tests {
     };
     use crate::test_utils::setup_test_env;
     use crate::{AutoShareContract, AutoShareContractClient};
-    use soroban_sdk::{testutils::Address as _, Address, Env, Vec};
+    use soroban_sdk::{testutils::Address as _, Address, Env, IntoVal, Vec};
 
     // ============================================================================
     // Default preferences
@@ -23,13 +23,21 @@ mod preferences_tests {
         // All three channels enabled by default
         assert_eq!(prefs.channels.len(), 3);
         for ch in prefs.channels.iter() {
-            assert!(ch.enabled, "Channel {:?} should be enabled by default", ch.channel);
+            assert!(
+                ch.enabled,
+                "Channel {:?} should be enabled by default",
+                ch.channel
+            );
         }
 
         // All five categories enabled by default
         assert_eq!(prefs.categories.len(), 5);
         for cat in prefs.categories.iter() {
-            assert!(cat.enabled, "Category {:?} should be enabled by default", cat.category);
+            assert!(
+                cat.enabled,
+                "Category {:?} should be enabled by default",
+                cat.category
+            );
         }
 
         assert_eq!(prefs.recipient, recipient);
@@ -113,8 +121,7 @@ mod preferences_tests {
         let recipient = test_env.users.get(0).unwrap();
 
         {
-            let client =
-                AutoShareContractClient::new(&test_env.env, &test_env.autoshare_contract);
+            let client = AutoShareContractClient::new(&test_env.env, &test_env.autoshare_contract);
             let mut channels = Vec::new(&test_env.env);
             channels.push_back(ChannelPreference {
                 channel: DeliveryChannel::Wallet,
@@ -155,8 +162,7 @@ mod preferences_tests {
         }
 
         // New client instance ("page refresh")
-        let client2 =
-            AutoShareContractClient::new(&test_env.env, &test_env.autoshare_contract);
+        let client2 = AutoShareContractClient::new(&test_env.env, &test_env.autoshare_contract);
         let prefs = client2.get_preferences(&recipient);
 
         // Wallet should be disabled
@@ -334,12 +340,7 @@ mod preferences_tests {
             invoke: &soroban_sdk::testutils::MockAuthInvoke {
                 contract: &contract_id,
                 fn_name: "set_channel_preference",
-                args: (
-                    victim.clone(),
-                    DeliveryChannel::Wallet,
-                    false,
-                )
-                    .into_val(&env),
+                args: (victim.clone(), DeliveryChannel::Wallet, false).into_val(&env),
                 sub_invokes: &[],
             },
         }]);

contract/contracts/hello-world/src/tests/revocation_test.rs

@@ -118,7 +118,8 @@ fn test_revoke_notification_emits_event() {
     set_now(&test_env.env, 2_000);
     client.revoke_notification(&id, &creator);
 
-    let topics = topics_of(&test_env.env, "notification_revoked").expect("revocation event must be emitted");
+    let topics =
+        topics_of(&test_env.env, "notification_revoked").expect("revocation event must be emitted");
     // [0] name, [1] notification_id, [2] revoked_by, [3] category, [4] priority.
     assert_eq!(topics.len(), 5);
 
@@ -327,12 +328,14 @@ fn test_revoke_event_has_high_priority() {
     set_now(&test_env.env, 2_000);
     client.revoke_notification(&id, &creator);
 
-    let topics = topics_of(&test_env.env, "notification_revoked").expect("revocation event must be emitted");
+    let topics =
+        topics_of(&test_env.env, "notification_revoked").expect("revocation event must be emitted");
     // Last topic is priority
     let priority_topic = topics.last().unwrap();
-    let priority = crate::base::events::NotificationPriority::try_from_val(&test_env.env, &priority_topic)
-        .expect("priority should be extractable");
-
+    let priority =
+        crate::base::events::NotificationPriority::try_from_val(&test_env.env, &priority_topic)
+            .expect("priority should be extractable");
+
     assert_eq!(priority, crate::base::events::NotificationPriority::High);
 }
 
@@ -349,12 +352,13 @@ fn test_revoke_event_has_notification_category() {
     set_now(&test_env.env, 2_000);
     client.revoke_notification(&id, &creator);
 
-    let topics = topics_of(&test_env.env, "notification_revoked").expect("revocation event must be emitted");
+    let topics =
+        topics_of(&test_env.env, "notification_revoked").expect("revocation event must be emitted");
     // Second to last topic is category
     let n = topics.len();
     let category_topic = topics.get(n - 2).unwrap();
     let category = NotificationCategory::try_from_val(&test_env.env, &category_topic)
         .expect("category should be extractable");
-    
+
     assert_eq!(category, NotificationCategory::Notification);
 }