docs: テスト規約を stub-fetch unit test の実態に合わせる（#55）

[yk-lab]

概要

CLAUDE.md の Conventions「Tests cover pure logic only … Network/IO modules (github, download, install, main) are validated end-to-end by self-test.yml」は実態と乖離していた。実際は HTTP/IO モジュールも境界を mock した決定的な unit testを持つ（github/checksum → ./fetch、download → @actions/tool-cache + ../src/github、proxy → undici）。次に触る人が迷わないよう doc を実態へ合わせる（#42/#54 のレビューで指摘された乖離）。

変更内容

• CLAUDE.md: Tests 項を Test layering に更新
  • 「unit は決定的・実ネットワークに触れない」「pure logic は直接 / HTTP・IO は境界 mock」「real IO は self-test.yml の E2E（matrix + skip-checksum + behind-a-proxy）で担保」を明記
• TODO.md: [documentation] テスト規約（CLAUDE.md）を stub-fetch unit test の実態に合わせる #55 を解決済みへ移動

検証

• ドキュメントのみの変更（コード変更なし）。typos ✅
• paths-filter で markdown-only と判定され、重い self-test/CI はスキップ（gate は green）

参照

• Closes [documentation] テスト規約（CLAUDE.md）を stub-fetch unit test の実態に合わせる #55
• [security] ダウンロード先ホスト/リダイレクト先を検証（NFR-1） #42 / [enhancement] HTTP(S) proxy 環境で全 fetch を proxy 経由にする #54（乖離が判明したレビュー）

🤖 Generated with Claude Code

---

Summary by cubic

CLAUDE.md のテスト規約を「Test layering」に更新し、HTTP/IO はネットワークに触れない決定的な unit test（境界を mock）、実 IO は .github/workflows/self-test.yml の E2E で担保する方針を明記。合わせて TODO.md で #55 を解決済みに移動。

^{Written for commit e3a9c70. Summary will update on new commits.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3668925f-dba4-4018-9177-218561433cf6

📥 Commits

Reviewing files that changed from the base of the PR and between 09231e1 and e3a9c70.

📒 Files selected for processing (2)

• CLAUDE.md
• TODO.md

---

📝 Walkthrough

Summary by CodeRabbit

• Documentation
  • テスト方針の説明を更新し、ユニットテストは決定的で実ネットワークに依存しないことを明確化しました。
  • HTTP/IO 関連は境界のモックで検証し、実際の I/O やダウンロード確認はエンドツーエンドで行う方針を追記しました。
  • バックログ整理を行い、完了済みの項目として関連 Issue を移動しました。

Walkthrough

CLAUDE.md のテスト方針説明を1行拡張し、HTTP/IO モジュールの境界モックの具体例（github/checksum が ./fetch を、download が @actions/tool-cache と ../src/github を、proxy が undici をモック）を明記。TODO.md のバックログから Issue #55 を削除し、解決済みセクションへ PR #58 として追記。

Changes

ドキュメント更新

Layer / File(s)	Summary
CLAUDE.md テスト方針の更新と TODO 整理 CLAUDE.md, TODO.md	CLAUDE.md のテスト規約を実態に合わせて拡張（モック境界の具体例を追記）。TODO.md のバックログから Issue #55 を削除し、解決済みセクションへ移動。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• yk-lab/setup-task#48: 同様に TODO.md のバックログ Issue を「既に解決した Issue」セクションへ移動する変更を含む。

Poem

🐰 ルールを書き換え、実態に合わせ
モックの境界、はっきり示す
バックログから #55 が消えて
解決済みの列に仲間入り
ドキュメントも、すっきりぴょん！ 🎉

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	変更内容の中心であるテスト規約の実態合わせを簡潔に表しています。
Description check	✅ Passed	概要、関連 issue、検証内容が揃っており、テンプレート要件をほぼ満たしています。
Linked Issues check	✅ Passed	#55 の要件どおり、stub fetch の unit test と self-test.yml の E2E の役割分担と download.test.ts との整合が明記されています。
Out of Scope Changes check	✅ Passed	変更は CLAUDE.md と TODO.md の文書整理に限定され、要件外のコード変更はありません。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch docs/test-convention

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[cubic-dev-ai]

No issues found across 2 files

_{Re-trigger cubic}