scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-09 03:18:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 109

fix(tests): add spec_path=None to MCP server mocks to fix Pydantic validation

Browse source 
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 <noreply@anthropic.com>
This commit is contained in:
Julio Quinteros Pro 2026-02-20 13:13:48 -03:00
parent 6af3d073f0
commit 375f79de03
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
tests/mcp_tests/test_mcp_server.py
View file

@ -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