From 375f79de032b26657ba2b288890b66509ad674b5 Mon Sep 17 00:00:00 2001 From: Julio Quinteros Pro Date: Fri, 20 Feb 2026 13:13:48 -0300 Subject: [PATCH] fix(tests): add spec_path=None to MCP server mocks to fix Pydantic validation spec_path was added to LiteLLM_MCPServerTable but the three test_add_update_server_* mocks weren't updated. MagicMock auto-creates a MagicMock for unset attributes, which fails the Optional[str] Pydantic validation. Fixes test_add_update_server_with_alias, test_add_update_server_without_alias and test_add_update_server_fallback_to_server_id. Co-Authored-By: Claude Sonnet 4.6 --- tests/mcp_tests/test_mcp_server.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/mcp_tests/test_mcp_server.py b/tests/mcp_tests/test_mcp_server.py index 98cf3c40c81..930b0a03042 100644 --- a/tests/mcp_tests/test_mcp_server.py +++ b/tests/mcp_tests/test_mcp_server.py @@ -1430,6 +1430,7 @@ async def test_add_update_server_with_alias(): mock_mcp_server.command = None mock_mcp_server.args = [] mock_mcp_server.env = None + mock_mcp_server.spec_path = None # OAuth fields - set explicitly to None to avoid MagicMock objects mock_mcp_server.client_id = None mock_mcp_server.client_secret = None @@ -1470,6 +1471,7 @@ async def test_add_update_server_without_alias(): mock_mcp_server.command = None mock_mcp_server.args = [] mock_mcp_server.env = None + mock_mcp_server.spec_path = None # OAuth fields - set explicitly to None to avoid MagicMock objects mock_mcp_server.client_id = None mock_mcp_server.client_secret = None @@ -1510,6 +1512,7 @@ async def test_add_update_server_fallback_to_server_id(): mock_mcp_server.command = None mock_mcp_server.args = [] mock_mcp_server.env = None + mock_mcp_server.spec_path = None # OAuth fields - set explicitly to None to avoid MagicMock objects mock_mcp_server.client_id = None mock_mcp_server.client_secret = None