From e20c1148111fe5eaa5c0e73aafbf88da4896e427 Mon Sep 17 00:00:00 2001 From: Ishaan Jaffer Date: Tue, 14 Apr 2026 11:05:22 -0700 Subject: [PATCH] fix(mcp): set instructions=None in SigV4BuildFromTable test mocks New MCPServer.instructions field requires a str; MagicMock attributes not explicitly set return a MagicMock object, which fails Pydantic validation. --- .../proxy/_experimental/mcp_server/test_mcp_sigv4_auth.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_sigv4_auth.py b/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_sigv4_auth.py index 7c142e3a771..eb9f4dde55f 100644 --- a/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_sigv4_auth.py +++ b/tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_sigv4_auth.py @@ -838,6 +838,7 @@ class TestSigV4BuildFromTable: table_record.tool_name_to_description = None table_record.byok_api_key_help_url = None table_record.oauth2_flow = None + table_record.instructions = None manager = MCPServerManager() @@ -895,6 +896,7 @@ class TestSigV4BuildFromTable: table_record.tool_name_to_description = None table_record.byok_api_key_help_url = None table_record.oauth2_flow = None + table_record.instructions = None manager = MCPServerManager()