add test

Author: evanliu048

crates/chat-cli/src/cli/chat/mcp.rs

@@ -159,7 +159,7 @@ pub async fn import_mcp_server(ctx: &Context, output: &mut SharedWriter, args: M
     for (name, cfg) in src_cfg.mcp_servers {
         if dst_cfg.mcp_servers.contains_key(&name) && !args.force {
             bail!(
-                "MCP server '{}' already exists in {} (scope {}). Use --force to overwrite.\n",
+                "\nMCP server '{}' already exists in {} (scope {}). Use --force to overwrite.\n",
                 name,
                 config_path.display(),
                 scope

crates/chat-cli/src/database/settings.rs

@@ -193,10 +193,12 @@ mod test {
         assert_eq!(settings.get(Setting::TelemetryEnabled), None);
         assert_eq!(settings.get(Setting::OldClientId), None);
         assert_eq!(settings.get(Setting::ShareCodeWhispererContent), None);
+        assert_eq!(settings.get(Setting::McpLoadedBefore), None);
 
         settings.set(Setting::TelemetryEnabled, true).await.unwrap();
         settings.set(Setting::OldClientId, "test").await.unwrap();
         settings.set(Setting::ShareCodeWhispererContent, false).await.unwrap();
+        settings.set(Setting::McpLoadedBefore, true).await.unwrap();
 
         assert_eq!(settings.get(Setting::TelemetryEnabled), Some(&Value::Bool(true)));
         assert_eq!(
@@ -207,13 +209,16 @@ mod test {
             settings.get(Setting::ShareCodeWhispererContent),
             Some(&Value::Bool(false))
         );
+        assert_eq!(settings.get(Setting::McpLoadedBefore),Some(&Value::Bool(true)));
 
         settings.remove(Setting::TelemetryEnabled).await.unwrap();
         settings.remove(Setting::OldClientId).await.unwrap();
         settings.remove(Setting::ShareCodeWhispererContent).await.unwrap();
+        settings.remove(Setting::McpLoadedBefore).await.unwrap();
 
         assert_eq!(settings.get(Setting::TelemetryEnabled), None);
         assert_eq!(settings.get(Setting::OldClientId), None);
         assert_eq!(settings.get(Setting::ShareCodeWhispererContent), None);
+        assert_eq!(settings.get(Setting::McpLoadedBefore), None);
     }
 }