Merge pull request #21681 from BerriAI/fix/mcp-server-test-spec-path-mock

fix(tests): add spec_path=None to MCP server mocks to fix Pydantic validation
This commit is contained in:
jquinter 2026-02-20 13:20:45 -03:00 committed by GitHub
commit 33783aed10
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

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