| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189 |
- #!/usr/bin/env python3
- """
- Test suite for modern CLI command patterns
- Tests that all CLI scripts use correct unified CLI commands in usage messages and print statements
- """
- import sys
- import os
- import unittest
- import subprocess
- from pathlib import Path
- # Add parent directory to path
- sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
- class TestModernCLICommands(unittest.TestCase):
- """Test that all CLI scripts use modern unified CLI commands"""
- def test_doc_scraper_uses_modern_commands(self):
- """Test doc_scraper.py uses skill-seekers commands"""
- script_path = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli' / 'doc_scraper.py'
- with open(script_path, 'r') as f:
- content = f.read()
- # Should use modern commands
- self.assertIn('skill-seekers scrape', content)
- # Should NOT use old python3 cli/ pattern
- self.assertNotIn('python3 cli/doc_scraper.py', content)
- def test_enhance_skill_local_uses_modern_commands(self):
- """Test enhance_skill_local.py uses skill-seekers commands"""
- script_path = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli' / 'enhance_skill_local.py'
- with open(script_path, 'r') as f:
- content = f.read()
- # Should use modern commands
- self.assertIn('skill-seekers', content)
- # Should NOT use old python3 cli/ pattern
- self.assertNotIn('python3 cli/enhance_skill_local.py', content)
- def test_estimate_pages_uses_modern_commands(self):
- """Test estimate_pages.py uses skill-seekers commands"""
- script_path = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli' / 'estimate_pages.py'
- with open(script_path, 'r') as f:
- content = f.read()
- # Should use modern commands
- self.assertIn('skill-seekers estimate', content)
- # Should NOT use old python3 cli/ pattern
- self.assertNotIn('python3 cli/estimate_pages.py', content)
- def test_package_skill_uses_modern_commands(self):
- """Test package_skill.py uses skill-seekers commands"""
- script_path = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli' / 'package_skill.py'
- with open(script_path, 'r') as f:
- content = f.read()
- # Should use modern commands
- self.assertIn('skill-seekers package', content)
- # Should NOT use old python3 cli/ pattern
- self.assertNotIn('python3 cli/package_skill.py', content)
- def test_github_scraper_uses_modern_commands(self):
- """Test github_scraper.py uses skill-seekers commands"""
- script_path = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli' / 'github_scraper.py'
- with open(script_path, 'r') as f:
- content = f.read()
- # Should use modern commands
- self.assertIn('skill-seekers', content)
- # Should NOT use old python3 cli/ pattern
- self.assertNotIn('python3 cli/github_scraper.py', content)
- class TestUnifiedCLIEntryPoints(unittest.TestCase):
- """Test that unified CLI entry points work correctly"""
- def test_main_cli_help_output(self):
- """Test skill-seekers --help works"""
- try:
- result = subprocess.run(
- ['skill-seekers', '--help'],
- capture_output=True,
- text=True,
- timeout=5
- )
- # Should return successfully
- self.assertIn(result.returncode, [0, 2],
- f"skill-seekers --help failed with code {result.returncode}")
- # Should show subcommands
- output = result.stdout + result.stderr
- self.assertIn('scrape', output)
- self.assertIn('github', output)
- self.assertIn('package', output)
- except FileNotFoundError:
- # If skill-seekers is not installed, skip this test
- self.skipTest("skill-seekers command not found - install package first")
- def test_main_cli_version_output(self):
- """Test skill-seekers --version works"""
- try:
- result = subprocess.run(
- ['skill-seekers', '--version'],
- capture_output=True,
- text=True,
- timeout=5
- )
- # Should return successfully
- self.assertEqual(result.returncode, 0,
- f"skill-seekers --version failed: {result.stderr}")
- # Should show version
- output = result.stdout + result.stderr
- self.assertIn('2.1.1', output)
- except FileNotFoundError:
- # If skill-seekers is not installed, skip this test
- self.skipTest("skill-seekers command not found - install package first")
- class TestNoHardcodedPaths(unittest.TestCase):
- """Test that no scripts have hardcoded absolute paths"""
- def test_no_hardcoded_paths_in_cli_scripts(self):
- """Test that CLI scripts don't have hardcoded paths"""
- cli_dir = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli'
- hardcoded_paths = [
- '/mnt/skills/examples/skill-creator/scripts/',
- '/home/',
- '/Users/',
- ]
- for script_path in cli_dir.glob('*.py'):
- with open(script_path, 'r') as f:
- content = f.read()
- for hardcoded_path in hardcoded_paths:
- self.assertNotIn(hardcoded_path, content,
- f"{script_path.name} contains hardcoded path: {hardcoded_path}")
- class TestPackageStructure(unittest.TestCase):
- """Test that package structure is correct"""
- def test_src_layout_exists(self):
- """Test that src/ layout directory exists"""
- src_dir = Path(__file__).parent.parent / 'src' / 'skill_seekers'
- self.assertTrue(src_dir.exists(), "src/skill_seekers/ directory should exist")
- def test_cli_package_exists(self):
- """Test that CLI package exists in src/"""
- cli_dir = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli'
- self.assertTrue(cli_dir.exists(), "src/skill_seekers/cli/ directory should exist")
- init_file = cli_dir / '__init__.py'
- self.assertTrue(init_file.exists(), "src/skill_seekers/cli/__init__.py should exist")
- def test_mcp_package_exists(self):
- """Test that MCP package exists in src/"""
- mcp_dir = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'mcp'
- self.assertTrue(mcp_dir.exists(), "src/skill_seekers/mcp/ directory should exist")
- init_file = mcp_dir / '__init__.py'
- self.assertTrue(init_file.exists(), "src/skill_seekers/mcp/__init__.py should exist")
- def test_main_cli_file_exists(self):
- """Test that main.py unified CLI exists"""
- main_file = Path(__file__).parent.parent / 'src' / 'skill_seekers' / 'cli' / 'main.py'
- self.assertTrue(main_file.exists(), "src/skill_seekers/cli/main.py should exist")
- if __name__ == '__main__':
- unittest.main()
|