tdd
HelmLog-specific test patterns and the pre-existing-error allowlist for ruff/mypy. The Red-Green-Refactor cycle and lint commands are already mandated in CLAUDE.md — this skill encodes only the patterns and known-pre-existing-errors list that aren't recoverable from existing tests at a glance. TRIGGER when writing or modifying Python source code in src/helmlog/. DO NOT trigger for documentation, config, templates, CSS/JS, skill definitions, or changes that don't affect runtime behavior.
ソース情報
- リポジトリ
- weaties/helmlog
- ソースの最終更新活動
- 2026年6月4日 19:29
- 検出された SKILL.md の言語
- 英語
- スター
- 2
- フォーク
- 2
インストール方法
デフォルトでは、最初にソースを確認する Prompt が選択されています。直接コマンドに切り替えるか、ローカルコピーをダウンロードすることもできます。
ソースファイルを確認
インストールを決める前に、SKILL.md と SkillsMP に表示されている付属ファイルをお読みください。
ファイルエクスプローラー
2 ファイルSKILL.md を表示中
SKILL.md
ソースの指示 · 読み取り専用プレビュー- name
- tdd
- description
- HelmLog-specific test patterns and the pre-existing-error allowlist for ruff/mypy. The Red-Green-Refactor cycle and lint commands are already mandated in CLAUDE.md — this skill encodes only the patterns and known-pre-existing-errors list that aren't recoverable from existing tests at a glance. TRIGGER when writing or modifying Python source code in src/helmlog/. DO NOT trigger for documentation, config, templates, CSS/JS, skill definitions, or changes that don't affect runtime behavior.
# TDD — HelmLog patterns
CLAUDE.md already mandates: failing test → implement → green → lint. This
skill encodes only the project-specific bits that aren't obvious from
existing tests:
## Test patterns
**Storage tests** — use the shared `storage` fixture from `conftest.py`
(in-memory SQLite, fully migrated; never construct `Storage` by hand in a
test):
```python
@pytest.mark.asyncio
async def test_my_feature(storage: Storage) -> None:
# storage is ready with all migrations applied
...
```
**Web route tests** — use `httpx.AsyncClient` with `ASGITransport`, not the
sync `TestClient`:
```python
@pytest.mark.asyncio
async def test_my_endpoint(storage: Storage) -> None:
from helmlog.web import create_app
app = create_app(storage)
async with httpx.AsyncClient(
transport=ASGITransport(app=app), base_url="http://test"
) as client:
resp = await client.get("/api/my-endpoint")
assert resp.status_code == 200
```
**Hardware mocking** — patch hardware modules at the import site so the
test never touches real devices:
```python
with patch("helmlog.cameras.httpx.AsyncClient") as mock_client:
...
```
Hardware modules to mock: `audio.py`, `can_reader.py`, `sk_reader.py`,
`cameras.py`.
## Don't rationalize skipping the cycle
The cycle only helps if you don't talk yourself out of it. Common excuses
and their rebuttals:
| Rationalization | Rebuttal |
|---|---|
| "This change is too small to need a test." | Size predicts neither breakage nor regression. A failing test first is what proves the change does what you think — write it. |
| "I'll write the test after I see it work." | Test-after rationalizes whatever the code already does, bugs included. Red-Green-Refactor is mandated in CLAUDE.md for a reason. |
| "It's just a renderer/route tweak — I'll eyeball it." | Web routes are the easiest place to ship a silent 500. Use the `AsyncClient`/`ASGITransport` pattern above. |
| "Tests pass, so it's correct." | Passing tests are evidence, not proof. Confirm the test exercises the new behavior and would actually fail without your change. |
| "ruff/mypy is noisy here, I'll skip the lint gate." | Only the pre-existing-error allowlist below is exempt. Everything else is green-before-PR. |
## Pre-existing errors to ignore
Do not fix these unless explicitly asked — they are tracked separately:
- `web.py`: `Item "None" of "datetime | None" has no attribute "isoformat"`
- `web.py`: `Item "None" of "AudioRecorder | None" has no attribute "stop"` (×2)
- `main.py`: `Unused "type: ignore" comment`
- `storage.py`: 2 pre-existing E501 line-length violations
GitHubで見る