fix: Update test imports for new package structure
Updated 8 test files to use new skill_seekers.* imports: - test_async_scraping.py - test_estimate_pages.py - test_package_skill.py - test_parallel_scraping.py - test_unified.py - test_unified_mcp_integration.py - test_upload_skill.py - test_utilities.py Changed: - from cli.* → from skill_seekers.cli.* - from skill_seeker_mcp.* → from skill_seekers.mcp.* - Removed obsolete sys.path.insert() calls Result: - 364/389 tests passing (93.5% pass rate) - Remaining 25 failures are path-related tests that need updating for new unified CLI commands (will fix next) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
@@ -13,10 +13,7 @@ from pathlib import Path
|
||||
from unittest.mock import Mock, patch, AsyncMock, MagicMock
|
||||
from collections import deque
|
||||
|
||||
# Add cli directory to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from doc_scraper import DocToSkillConverter
|
||||
from skill_seekers.cli.doc_scraper import DocToSkillConverter
|
||||
|
||||
|
||||
class TestAsyncConfiguration(unittest.TestCase):
|
||||
|
||||
@@ -9,10 +9,7 @@ import json
|
||||
from pathlib import Path
|
||||
import sys
|
||||
|
||||
# Add cli directory to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from estimate_pages import estimate_pages
|
||||
from skill_seekers.cli.estimate_pages import estimate_pages
|
||||
|
||||
|
||||
class TestEstimatePages(unittest.TestCase):
|
||||
|
||||
@@ -9,10 +9,7 @@ import zipfile
|
||||
from pathlib import Path
|
||||
import sys
|
||||
|
||||
# Add cli directory to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from package_skill import package_skill
|
||||
from skill_seekers.cli.package_skill import package_skill
|
||||
|
||||
|
||||
class TestPackageSkill(unittest.TestCase):
|
||||
|
||||
@@ -13,10 +13,7 @@ from pathlib import Path
|
||||
from unittest.mock import Mock, patch, MagicMock
|
||||
from collections import deque
|
||||
|
||||
# Add cli directory to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from doc_scraper import DocToSkillConverter
|
||||
from skill_seekers.cli.doc_scraper import DocToSkillConverter
|
||||
|
||||
|
||||
class TestParallelScrapingConfiguration(unittest.TestCase):
|
||||
|
||||
@@ -16,13 +16,10 @@ import pytest
|
||||
import tempfile
|
||||
from pathlib import Path
|
||||
|
||||
# Add CLI to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from config_validator import ConfigValidator, validate_config
|
||||
from conflict_detector import ConflictDetector, Conflict
|
||||
from merge_sources import RuleBasedMerger
|
||||
from unified_skill_builder import UnifiedSkillBuilder
|
||||
from skill_seekers.cli.config_validator import ConfigValidator, validate_config
|
||||
from skill_seekers.cli.conflict_detector import ConflictDetector, Conflict
|
||||
from skill_seekers.cli.merge_sources import RuleBasedMerger
|
||||
from skill_seekers.cli.unified_skill_builder import UnifiedSkillBuilder
|
||||
|
||||
|
||||
# ===========================
|
||||
|
||||
@@ -15,10 +15,7 @@ from pathlib import Path
|
||||
# Configure pytest to only use asyncio backend (not trio)
|
||||
pytestmark = pytest.mark.anyio
|
||||
|
||||
# Add skill_seeker_mcp to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'skill_seeker_mcp'))
|
||||
|
||||
from server import validate_config_tool, scrape_docs_tool
|
||||
from skill_seekers.mcp.server import validate_config_tool, scrape_docs_tool
|
||||
|
||||
|
||||
async def test_mcp_validate_unified_config():
|
||||
|
||||
@@ -10,10 +10,7 @@ import os
|
||||
from pathlib import Path
|
||||
import sys
|
||||
|
||||
# Add cli directory to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from upload_skill import upload_skill_api
|
||||
from skill_seekers.cli.upload_skill import upload_skill_api
|
||||
|
||||
|
||||
class TestUploadSkillAPI(unittest.TestCase):
|
||||
|
||||
@@ -10,10 +10,7 @@ import zipfile
|
||||
from pathlib import Path
|
||||
import sys
|
||||
|
||||
# Add cli directory to path
|
||||
sys.path.insert(0, str(Path(__file__).parent.parent / 'cli'))
|
||||
|
||||
from utils import (
|
||||
from skill_seekers.cli.utils import (
|
||||
has_api_key,
|
||||
get_api_key,
|
||||
get_upload_url,
|
||||
|
||||
Reference in New Issue
Block a user