Pablo Estevez
|
c33c6f9073
|
change max lenght
|
2026-01-17 17:48:15 +00:00 |
|
Pablo Estevez
|
5ed767ff9a
|
run ruff
|
2026-01-17 17:29:21 +00:00 |
|
yusyus
|
998be0d2dd
|
fix: Update setup_mcp.sh for v2.0.0 src/ layout + test fixes (#201)
Merges setup_mcp.sh fix for v2.0.0 src/ layout + test updates.
Original fix by @501981732 in PR #197.
Test updates to make CI pass.
Closes #192
|
2025-11-29 21:34:51 +03:00 |
|
yusyus
|
ccbf67bb80
|
test: Fix tests for modern Python packaging structure
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>
|
2025-11-10 21:35:44 +03:00 |
|
yusyus
|
9931066741
|
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>
|
2025-11-07 01:21:29 +03:00 |
|
yusyus
|
13fcce1f4e
|
Add comprehensive test coverage for CLI utilities
Expand test suite from 118 to 166 tests (+48 new tests) with focus on
untested CLI tools and utility functions. Overall coverage increased
from 14% to 25%.
New test files:
- tests/test_utilities.py (42 tests) - API keys, file validation, formatting
- tests/test_package_skill.py (11 tests) - Skill packaging workflow
- tests/test_estimate_pages.py (8 tests) - Page estimation functionality
- tests/test_upload_skill.py (7 tests) - Skill upload validation
Coverage improvements by module:
- cli/utils.py: 0% → 72% (+72%)
- cli/upload_skill.py: 0% → 53% (+53%)
- cli/estimate_pages.py: 0% → 47% (+47%)
- cli/package_skill.py: 0% → 43% (+43%)
All 166 tests passing. Added pytest-cov for coverage reporting.
Updated requirements.txt with all dependencies including MCP packages.
Test execution: 9.6s for complete suite
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
|
2025-10-22 22:08:02 +03:00 |
|