Skip to content

Commit bc9e884

Browse files
fix: add missing MCPManager() instantiation in test
The test_sync_from_config_registers_new_servers test was setting up mocks but never actually creating an MCPManager instance, so the sync_from_config() method was never called and register() was never invoked. Added MCPManager() call inside the with block to trigger the initialization flow that calls sync_from_config().
1 parent b4f7ce6 commit bc9e884

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

tests/mcp/test_manager_extended.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -744,6 +744,9 @@ def test_sync_from_config_registers_new_servers(self):
744744
),
745745
patch("code_puppy.mcp_.manager.ServerStatusTracker"),
746746
):
747+
# Actually instantiate the manager to trigger sync_from_config!
748+
MCPManager()
749+
747750
# Verify register was called
748751
assert mock_registry_instance.register.called
749752

0 commit comments

Comments
 (0)