Fix MCP test for auto-reconnect behavior on Python 3.11

Mock connect() in test_call_tool_without_connection so the
auto-reconnect path doesn't spawn a real subprocess that fails
differently across Python versions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Hmbown

tests/test_mcp.py

@@ -6,6 +6,7 @@
 from __future__ import annotations
 
 import json
+from unittest.mock import AsyncMock, patch
 
 import pytest
 
@@ -33,6 +34,8 @@ def test_request_id_increments(self):
     async def test_call_tool_without_connection(self):
         config = MCPServerConfig(name="test", command="echo", args=[])
         client = MCPClient(config)
-        result = await client.call_tool("test_tool", {"arg": "value"})
+        # Mock connect to simulate reconnect failure (no real server)
+        with patch.object(client, "connect", new_callable=AsyncMock, return_value=[]):
+            result = await client.call_tool("test_tool", {"arg": "value"})
         data = json.loads(result)
         assert "error" in data