skill-seekers-reference

firefrost-gaming/skill-seekers-reference

Fork 0

Commit Graph

Author	SHA1	Message	Date
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	962b5b9340	Add comprehensive bash script tests and fix old mcp/ path references - Created tests/test_setup_scripts.py with 19 tests covering: * setup_mcp.sh validation (11 tests) * General bash script quality (4 tests) * MCP path consistency across codebase (4 tests) - Fixed old 'mcp/' references in documentation: * docs/B1_COMPLETE_SUMMARY.md (3 refs) * docs/PDF_MCP_TOOL.md (2 refs) * docs/MCP_SETUP.md (18 refs) * docs/TEST_MCP_IN_CLAUDE_CODE.md (4 refs) These tests would have caught Issue #157 before it reached users. Tests verify: - Bash syntax validity - No hardcoded paths - Correct skill_seeker_mcp/ directory references - Files referenced in scripts actually exist - No deprecated backticks - Proper error handling (set -e) All 19 tests passing ✅	2025-10-26 17:33:39 +03:00

Author

SHA1

Message

Date

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

962b5b9340

Add comprehensive bash script tests and fix old mcp/ path references

- Created tests/test_setup_scripts.py with 19 tests covering:
  * setup_mcp.sh validation (11 tests)
  * General bash script quality (4 tests)
  * MCP path consistency across codebase (4 tests)

- Fixed old 'mcp/' references in documentation:
  * docs/B1_COMPLETE_SUMMARY.md (3 refs)
  * docs/PDF_MCP_TOOL.md (2 refs)
  * docs/MCP_SETUP.md (18 refs)
  * docs/TEST_MCP_IN_CLAUDE_CODE.md (4 refs)

These tests would have caught Issue #157 before it reached users.

Tests verify:
- Bash syntax validity
- No hardcoded paths
- Correct skill_seeker_mcp/ directory references
- Files referenced in scripts actually exist
- No deprecated backticks
- Proper error handling (set -e)

All 19 tests passing ✅

2025-10-26 17:33:39 +03:00

2 Commits