test(coverage): batch 19 — service/restart, memory/store/factories (tinyhumansai#530)

Add 7 new tests:
- service/restart: default source/reason, whitespace trimming, empty-string
  defaults, RestartStatus serde, startup delay noop
- memory/store/factories: effective_memory_backend_name, migration-disabled error

All 4223 tests pass.

Author: CodeGhost21

Cargo.lock

@@ -93,3 +93,29 @@ pub fn create_memory_for_migration(
 ) -> anyhow::Result<Box<dyn Memory>> {
     anyhow::bail!("memory migration is disabled for the unified namespace memory core")
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn effective_memory_backend_name_always_returns_namespace() {
+        assert_eq!(effective_memory_backend_name("sqlite", None), "namespace");
+        assert_eq!(effective_memory_backend_name("anything", None), "namespace");
+        assert_eq!(effective_memory_backend_name("", None), "namespace");
+    }
+
+    #[test]
+    fn create_memory_for_migration_always_errors() {
+        let tmp = tempfile::tempdir().unwrap();
+        // Box<dyn Memory> doesn't impl Debug, so we can't use .unwrap_err().
+        // Use match instead.
+        match create_memory_for_migration("any", tmp.path()) {
+            Ok(_) => panic!("expected error"),
+            Err(e) => assert!(
+                e.to_string().contains("migration is disabled"),
+                "unexpected error: {e}"
+            ),
+        }
+    }
+}

app/src-tauri/Cargo.lock

@@ -205,4 +205,55 @@ mod tests {
 
         handle.cancel();
     }
+
+    #[tokio::test]
+    async fn service_restart_defaults_source_and_reason() {
+        event_bus::init_global(event_bus::DEFAULT_CAPACITY);
+        let outcome = service_restart(None, None)
+            .await
+            .expect("restart should succeed");
+        assert!(outcome.value.accepted);
+        assert_eq!(outcome.value.source, "jsonrpc");
+        assert_eq!(outcome.value.reason, "service.restart");
+    }
+
+    #[tokio::test]
+    async fn service_restart_trims_whitespace() {
+        event_bus::init_global(event_bus::DEFAULT_CAPACITY);
+        let outcome = service_restart(Some("  ui  ".into()), Some("  user request  ".into()))
+            .await
+            .expect("restart should succeed");
+        assert_eq!(outcome.value.source, "ui");
+        assert_eq!(outcome.value.reason, "user request");
+    }
+
+    #[tokio::test]
+    async fn service_restart_empty_strings_use_defaults() {
+        event_bus::init_global(event_bus::DEFAULT_CAPACITY);
+        let outcome = service_restart(Some("".into()), Some("  ".into()))
+            .await
+            .expect("restart should succeed");
+        assert_eq!(outcome.value.source, "jsonrpc");
+        assert_eq!(outcome.value.reason, "service.restart");
+    }
+
+    #[test]
+    fn restart_status_serializes() {
+        let status = RestartStatus {
+            accepted: true,
+            source: "test".into(),
+            reason: "testing".into(),
+        };
+        let json = serde_json::to_string(&status).unwrap();
+        assert!(json.contains("\"accepted\":true"));
+        assert!(json.contains("\"source\":\"test\""));
+    }
+
+    #[test]
+    fn apply_startup_restart_delay_from_env_noop_when_unset() {
+        // Ensure the env var is not set, then call — should not block
+        let _prev = std::env::var(RESTART_DELAY_ENV).ok();
+        std::env::remove_var(RESTART_DELAY_ENV);
+        apply_startup_restart_delay_from_env(); // should return immediately
+    }
 }