Skip to content

Commit 6b149a3

Browse files
niechenclaude
andcommitted
fix: add missing mock for client edit test failing in CI
- Add GlobalConfigManager mock to test_client_edit_command_client_not_installed - Ensures test has servers available to avoid early exit with code 1 - Fixes CI test failure where no servers were available 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 38d6f36 commit 6b149a3

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

tests/test_client.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -294,6 +294,11 @@ def test_client_edit_command_client_not_installed(monkeypatch):
294294
monkeypatch.setattr(ClientRegistry, "get_client_manager", Mock(return_value=mock_client_manager))
295295
monkeypatch.setattr(ClientRegistry, "get_client_info", Mock(return_value={"name": "Windsurf"}))
296296

297+
# Mock GlobalConfigManager - need servers to avoid early exit
298+
mock_global_config = Mock()
299+
mock_global_config.list_servers.return_value = {"test-server": Mock(description="Test server")}
300+
monkeypatch.setattr("mcpm.commands.client.global_config_manager", mock_global_config)
301+
297302
# Run the command
298303
runner = CliRunner()
299304
result = runner.invoke(edit_client, ["windsurf"])

0 commit comments

Comments
 (0)