firefrost-gaming/skill-seekers-reference
3
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix: Correct mock patch path for install_skill tests

Browse Source 
Fixed 4 failing tests in TestPackagingTools that were patching the wrong
module path. The tests were patching:
  'skill_seekers.mcp.tools.packaging_tools.fetch_config_tool'

But fetch_config_tool is actually in source_tools, not packaging_tools.

Changed all 4 tests to patch:
  'skill_seekers.mcp.tools.source_tools.fetch_config_tool'

Tests now passing:
- test_install_skill_with_config_name ✅
- test_install_skill_with_config_path ✅
- test_install_skill_unlimited ✅
- test_install_skill_no_upload ✅

Result: 81/81 MCP tests passing (was 77/81)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
yusyus
2026-01-12 22:56:37 +03:00
parent 24634bc8b4
commit 62a51c0084
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
tests/test_mcp_fastmcp.py
View File

@@ -534,7 +534,7 @@ class TestPackagingTools:
async def test_install_skill_with_config_name(self):
"""Test complete install workflow with config name."""
# Mock the fetch_config_tool import that install_skill_tool uses
with patch('skill_seekers.mcp.tools.packaging_tools.fetch_config_tool') as mock_fetch:
with patch('skill_seekers.mcp.tools.source_tools.fetch_config_tool') as mock_fetch:
mock_fetch.return_value = [Mock(text="Config fetched")]
result = await server_fastmcp.install_skill(
@@ -547,7 +547,7 @@ class TestPackagingTools:
async def test_install_skill_with_config_path(self, sample_config):
"""Test complete install workflow with config path."""
with patch('skill_seekers.mcp.tools.packaging_tools.fetch_config_tool') as mock_fetch:
with patch('skill_seekers.mcp.tools.source_tools.fetch_config_tool') as mock_fetch:
mock_fetch.return_value = [Mock(text="Config ready")]
result = await server_fastmcp.install_skill(
@@ -560,7 +560,7 @@ class TestPackagingTools:
async def test_install_skill_unlimited(self):
"""Test install workflow with unlimited pages."""
with patch('skill_seekers.mcp.tools.packaging_tools.fetch_config_tool') as mock_fetch:
with patch('skill_seekers.mcp.tools.source_tools.fetch_config_tool') as mock_fetch:
mock_fetch.return_value = [Mock(text="Config fetched")]
result = await server_fastmcp.install_skill(
@@ -573,7 +573,7 @@ class TestPackagingTools:
async def test_install_skill_no_upload(self):
"""Test install workflow without auto-upload."""
with patch('skill_seekers.mcp.tools.packaging_tools.fetch_config_tool') as mock_fetch:
with patch('skill_seekers.mcp.tools.source_tools.fetch_config_tool') as mock_fetch:
mock_fetch.return_value = [Mock(text="Config fetched")]
result = await server_fastmcp.install_skill(