Updated test files to work with new src/ layout and unified CLI: Fixed Tests (17 tests): - test_cli_paths.py: Complete rewrite for modern CLI * Check for skill-seekers commands instead of python3 cli/ * Test unified CLI entry points * Verify src/ package structure - test_estimate_pages.py: Update CLI tests for entry points - test_package_skill.py: Update CLI tests for entry points - test_upload_skill.py: Update CLI tests for entry points - test_setup_scripts.py: Update paths for src/skill_seekers/mcp/ Changes: - Old: Check for python3 cli/*.py commands - New: Check for skill-seekers subcommands - Old: Look in cli/ and skill_seeker_mcp/ directories - New: Look in src/skill_seekers/cli/ and src/skill_seekers/mcp/ - Added FileNotFoundError handling to skip tests if not installed - Accept exit code 0 or 2 from argparse --help Results: - ✅ 381 tests passing (up from 364) - ✅ 17 tests fixed - ⚠️ 2 tests flaky (pass individually, fail in full suite) - ⏭️ 28 tests skipped (MCP server tests - require MCP install) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
161 lines
5.1 KiB
Python
161 lines
5.1 KiB
Python
#!/usr/bin/env python3
|
|
"""
|
|
Tests for cli/estimate_pages.py functionality
|
|
"""
|
|
|
|
import unittest
|
|
import tempfile
|
|
import json
|
|
from pathlib import Path
|
|
import sys
|
|
|
|
from skill_seekers.cli.estimate_pages import estimate_pages
|
|
|
|
|
|
class TestEstimatePages(unittest.TestCase):
|
|
"""Test estimate_pages function"""
|
|
|
|
def test_estimate_pages_with_minimal_config(self):
|
|
"""Test estimation with minimal configuration"""
|
|
config = {
|
|
'name': 'test',
|
|
'base_url': 'https://example.com/',
|
|
'rate_limit': 0.1
|
|
}
|
|
|
|
# This will make real HTTP request to example.com
|
|
# We use low max_discovery to keep test fast
|
|
result = estimate_pages(config, max_discovery=2, timeout=5)
|
|
|
|
# Check result structure
|
|
self.assertIsInstance(result, dict)
|
|
self.assertIn('discovered', result)
|
|
self.assertIn('estimated_total', result)
|
|
# Actual key is elapsed_seconds, not time_elapsed
|
|
self.assertIn('elapsed_seconds', result)
|
|
|
|
def test_estimate_pages_returns_discovered_count(self):
|
|
"""Test that result contains discovered page count"""
|
|
config = {
|
|
'name': 'test',
|
|
'base_url': 'https://example.com/',
|
|
'rate_limit': 0.1
|
|
}
|
|
|
|
result = estimate_pages(config, max_discovery=1, timeout=5)
|
|
|
|
self.assertGreaterEqual(result['discovered'], 0)
|
|
self.assertIsInstance(result['discovered'], int)
|
|
|
|
def test_estimate_pages_respects_max_discovery(self):
|
|
"""Test that estimation respects max_discovery limit"""
|
|
config = {
|
|
'name': 'test',
|
|
'base_url': 'https://example.com/',
|
|
'rate_limit': 0.1
|
|
}
|
|
|
|
result = estimate_pages(config, max_discovery=3, timeout=5)
|
|
|
|
# Should not discover more than max_discovery
|
|
self.assertLessEqual(result['discovered'], 3)
|
|
|
|
def test_estimate_pages_with_start_urls(self):
|
|
"""Test estimation with custom start_urls"""
|
|
config = {
|
|
'name': 'test',
|
|
'base_url': 'https://example.com/',
|
|
'start_urls': ['https://example.com/'],
|
|
'rate_limit': 0.1
|
|
}
|
|
|
|
result = estimate_pages(config, max_discovery=2, timeout=5)
|
|
|
|
self.assertIsInstance(result, dict)
|
|
self.assertIn('discovered', result)
|
|
|
|
|
|
class TestEstimatePagesCLI(unittest.TestCase):
|
|
"""Test estimate_pages command-line interface (via entry point)"""
|
|
|
|
def test_cli_help_output(self):
|
|
"""Test that skill-seekers estimate --help works"""
|
|
import subprocess
|
|
|
|
try:
|
|
result = subprocess.run(
|
|
['skill-seekers', 'estimate', '--help'],
|
|
capture_output=True,
|
|
text=True,
|
|
timeout=5
|
|
)
|
|
|
|
# Should return successfully (0 or 2 for argparse)
|
|
self.assertIn(result.returncode, [0, 2])
|
|
output = result.stdout + result.stderr
|
|
self.assertTrue('usage:' in output.lower() or 'estimate' in output.lower())
|
|
except FileNotFoundError:
|
|
self.skipTest("skill-seekers command not installed")
|
|
|
|
def test_cli_executes_with_help_flag(self):
|
|
"""Test that skill-seekers-estimate entry point works"""
|
|
import subprocess
|
|
|
|
try:
|
|
result = subprocess.run(
|
|
['skill-seekers-estimate', '--help'],
|
|
capture_output=True,
|
|
text=True,
|
|
timeout=5
|
|
)
|
|
|
|
# Should return successfully
|
|
self.assertIn(result.returncode, [0, 2])
|
|
except FileNotFoundError:
|
|
self.skipTest("skill-seekers-estimate command not installed")
|
|
|
|
def test_cli_requires_config_argument(self):
|
|
"""Test that CLI requires config file argument"""
|
|
import subprocess
|
|
|
|
try:
|
|
# Run without config argument
|
|
result = subprocess.run(
|
|
['skill-seekers', 'estimate'],
|
|
capture_output=True,
|
|
text=True,
|
|
timeout=5
|
|
)
|
|
|
|
# Should fail (non-zero exit code) or show usage
|
|
self.assertTrue(
|
|
result.returncode != 0 or 'usage' in result.stderr.lower() or 'usage' in result.stdout.lower()
|
|
)
|
|
except FileNotFoundError:
|
|
self.skipTest("skill-seekers command not installed")
|
|
|
|
|
|
class TestEstimatePagesWithRealConfig(unittest.TestCase):
|
|
"""Test estimation with real config files (if available)"""
|
|
|
|
def test_estimate_with_real_config_file(self):
|
|
"""Test estimation using a real config file (if exists)"""
|
|
config_path = Path('configs/react.json')
|
|
|
|
if not config_path.exists():
|
|
self.skipTest("configs/react.json not found")
|
|
|
|
with open(config_path, 'r') as f:
|
|
config = json.load(f)
|
|
|
|
# Use very low max_discovery to keep test fast
|
|
result = estimate_pages(config, max_discovery=3, timeout=5)
|
|
|
|
self.assertIsInstance(result, dict)
|
|
self.assertIn('discovered', result)
|
|
self.assertGreater(result['discovered'], 0)
|
|
|
|
|
|
if __name__ == '__main__':
|
|
unittest.main()
|