22
33from unittest .mock import MagicMock , patch
44
5- import pytest
6-
75from src .tools import ToolManager
86
97
@@ -40,14 +38,13 @@ def test_get_mcp_instructions_strips_whitespace(self):
4038 @patch ("src.tools.os.environ.get" )
4139 @patch ("src.tools.os.path.exists" )
4240 @patch ("builtins.open" )
43- def test_load_mcp_tools_collects_instructions (
44- self , mock_open , mock_exists , mock_env_get , mock_create
45- ):
41+ def test_load_mcp_tools_collects_instructions (self , mock_open , mock_exists , mock_env_get , mock_create ):
4642 """Verify load_mcp_tools collects server_instructions from MCPClients."""
4743 mock_env_get .return_value = "/tmp/mcp.json"
4844 mock_exists .return_value = True
4945
5046 import json
47+
5148 mcp_config = {"mcpServers" : {"test-server" : {"command" : "echo" , "args" : []}}}
5249 mock_open .return_value .__enter__ = lambda s : s
5350 mock_open .return_value .__exit__ = MagicMock (return_value = False )
@@ -68,14 +65,13 @@ def test_load_mcp_tools_collects_instructions(
6865 @patch ("src.tools.os.environ.get" )
6966 @patch ("src.tools.os.path.exists" )
7067 @patch ("builtins.open" )
71- def test_load_mcp_tools_skips_none_instructions (
72- self , mock_open , mock_exists , mock_env_get , mock_create
73- ):
68+ def test_load_mcp_tools_skips_none_instructions (self , mock_open , mock_exists , mock_env_get , mock_create ):
7469 """Verify load_mcp_tools skips servers without instructions."""
7570 mock_env_get .return_value = "/tmp/mcp.json"
7671 mock_exists .return_value = True
7772
7873 import json
74+
7975 mcp_config = {"mcpServers" : {"no-inst" : {"command" : "echo" , "args" : []}}}
8076 mock_open .return_value .__enter__ = lambda s : s
8177 mock_open .return_value .__exit__ = MagicMock (return_value = False )
0 commit comments