Python test fix

Author: SteveSandersonMS

python/copilot/client.py

@@ -152,7 +152,10 @@ def __init__(self, options: Optional[CopilotClientOptions] = None):
             self._actual_port = None
 
         # Determine CLI path: explicit option > bundled binary
-        if opts.get("cli_path"):
+        # Not needed when connecting to external server via cli_url
+        if opts.get("cli_url"):
+            default_cli_path = ""  # Not used for external server
+        elif opts.get("cli_path"):
             default_cli_path = opts["cli_path"]
         else:
             bundled_path = _get_bundled_cli_path()

python/test_client.py

@@ -96,25 +96,36 @@ def test_is_external_server_true(self):
 
 class TestAuthOptions:
     def test_accepts_github_token(self):
-        client = CopilotClient({"github_token": "gho_test_token", "log_level": "error"})
+        client = CopilotClient(
+            {"cli_path": CLI_PATH, "github_token": "gho_test_token", "log_level": "error"}
+        )
         assert client.options.get("github_token") == "gho_test_token"
 
     def test_default_use_logged_in_user_true_without_token(self):
-        client = CopilotClient({"log_level": "error"})
+        client = CopilotClient({"cli_path": CLI_PATH, "log_level": "error"})
         assert client.options.get("use_logged_in_user") is True
 
     def test_default_use_logged_in_user_false_with_token(self):
-        client = CopilotClient({"github_token": "gho_test_token", "log_level": "error"})
+        client = CopilotClient(
+            {"cli_path": CLI_PATH, "github_token": "gho_test_token", "log_level": "error"}
+        )
         assert client.options.get("use_logged_in_user") is False
 
     def test_explicit_use_logged_in_user_true_with_token(self):
         client = CopilotClient(
-            {"github_token": "gho_test_token", "use_logged_in_user": True, "log_level": "error"}
+            {
+                "cli_path": CLI_PATH,
+                "github_token": "gho_test_token",
+                "use_logged_in_user": True,
+                "log_level": "error",
+            }
         )
         assert client.options.get("use_logged_in_user") is True
 
     def test_explicit_use_logged_in_user_false_without_token(self):
-        client = CopilotClient({"use_logged_in_user": False, "log_level": "error"})
+        client = CopilotClient(
+            {"cli_path": CLI_PATH, "use_logged_in_user": False, "log_level": "error"}
+        )
         assert client.options.get("use_logged_in_user") is False
 
     def test_github_token_with_cli_url_raises(self):