All 5 thin-skill adapter repos now have: - tests/README.md explaining limited coverage rationale (prose-only skills) - tests/test_smoke.py covering plugin.yaml, SKILL.md frontmatter, adapter re-export Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| README.md | ||
| test_smoke.py | ||
Test Rationale — molecule-skill-code-review
What this plugin does
molecule-skill-code-review is a prose-only skill: its logic lives entirely in
skills/code-review/SKILL.md. The adapter (adapters/claude_code.py) is a thin
re-export of AgentskillsAdaptor from plugins_registry.builtins — no business
logic, no network calls, no side effects.
What is tested
plugin.yamlis valid YAML with required fields (name, version, runtimes, skills)skills/code-review/SKILL.mdhas valid YAML frontmatter and a body with headingsadapters/claude_code.pyexists and re-exportsAgentskillsAdaptorvalidate-plugin.py(.molecule-ci/scripts/) exits zero
What is NOT unit-tested (and why)
The multi-criteria review logic (best-practices checks, modularity scoring, etc.) is prose guidance inside SKILL.md — there is no Python function to call. Testing it would require a full workspace runtime with a real Claude session. Smoke tests cover the artifact structure; full evaluation requires integration tests.
Running tests
python -m pytest tests/ -v