test: fix test

Author: JoJoJoJoJoJoJo

src/mcpm/cli.py

@@ -98,7 +98,7 @@ def main(ctx, help_flag, version):
     # Check if a command is being executed (and it's not help, no command, or the client command)
     if (
         ctx.invoked_subcommand
-        and ctx.invoked_subcommand not in ["target", "client", "profile", "router"]
+        and ctx.invoked_subcommand not in ["target", "client", "profile", "router", "share"]
         and not help_flag
     ):
         # Check if active client is set

src/mcpm/commands/__init__.py

@@ -15,6 +15,7 @@
     "transfer",
     "router",
     "custom",
+    "target",
 ]
 
 # All command modules

src/mcpm/commands/router.py

@@ -440,7 +440,7 @@ def share(address, profile, http):
     # print share link
     console.print(f"[bold green]Router is sharing at {share_url}[/]")
     console.print(
-        f"[green]Your profile can be accessed with the url {share_url}?profile={profile}{f'&s={api_key}' if api_key else ''}[/]\n"
+        f"[green]Your profile can be accessed with the url {share_url}?{f's={api_key}&' if api_key else ''}profile={profile}[/]\n"
     )
     console.print(
         "[bold yellow]Be careful about the share link, it will be exposed to the public. Make sure to share to trusted users only.[/]"

src/mcpm/profile/profile_config.py

@@ -1,4 +1,5 @@
 import json
+import logging
 import os
 from typing import Dict, Optional
 
@@ -8,6 +9,8 @@
 
 DEFAULT_PROFILE_PATH = os.path.expanduser("~/.config/mcpm/profiles.json")
 
+logger = logging.getLogger(__name__)
+
 
 class ProfileConfigManager:
     def __init__(self, profile_path: str = DEFAULT_PROFILE_PATH):
@@ -17,8 +20,12 @@ def __init__(self, profile_path: str = DEFAULT_PROFILE_PATH):
     def _load_profiles(self) -> Dict[str, list[ServerConfig]]:
         if not os.path.exists(self.profile_path):
             return {}
-        with open(self.profile_path, "r", encoding="utf-8") as f:
-            profiles = json.load(f) or {}
+        try:
+            with open(self.profile_path, "r", encoding="utf-8") as f:
+                profiles = json.load(f) or {}
+        except json.JSONDecodeError as e:
+            logger.error(f"Error loading profiles from {self.profile_path}: {e}")
+            return {}
         return {
             name: [TypeAdapter(ServerConfig).validate_python(config) for config in configs]
             for name, configs in profiles.items()

tests/test_add.py

@@ -213,13 +213,13 @@ def test_add_profile_to_client(windsurf_manager, monkeypatch):
     profile_name = "work"
     client_name = "windsurf"
 
+    monkeypatch.setattr(ClientRegistry, "get_active_target", Mock(return_value="@" + client_name))
     monkeypatch.setattr(ClientRegistry, "get_client_manager", Mock(return_value=windsurf_manager))
-    monkeypatch.setattr(ClientRegistry, "get_active_client", Mock(return_value=client_name))
     monkeypatch.setattr(ClientRegistry, "get_active_client_manager", Mock(return_value=windsurf_manager))
     monkeypatch.setattr(ConfigManager, "get_router_config", Mock(return_value={"host": "localhost", "port": 8080}))
 
     # test cli
     runner = CliRunner()
     result = runner.invoke(add, ["%" + profile_name, "--force", "--alias", "work"])
     assert result.exit_code == 0
-    assert "Successfully added profile: work" in result.output
+    assert "Successfully added profile %work to windsurf!" in result.output

tests/test_client.py

@@ -151,7 +151,7 @@ def test_client_edit_command_client_not_supported(monkeypatch):
 
         # Check the result
         assert result.exit_code == 0
-        mock_print_error.assert_called_once_with("Unsupported")
+        mock_print_error.assert_called_once_with()
 
 
 def test_client_edit_command_client_not_installed(monkeypatch):