chore: 重複した checksum 改ざんテストを統合（#43）

[yk-lab]

概要

tests/checksum-tamper.test.ts が tests/checksum.test.ts 内の tamper detection describe と完全に同一だったため、重複を解消する（#43）。

変更内容

• tests/checksum-tamper.test.ts を削除
  • 同ファイルの 3 ケース（tampered 拒否 / genuine 受理 / asset 不在）は checksum.test.ts の fetchChecksum + verifyChecksum: tamper detection describe と ASSET・payload・stub helper まで一致
  • parse / sha256File / verifyChecksum の単体テスト + tamper 統合テストは checksum.test.ts に集約済み（要求仕様書 §10.3）
• TODO.md
  • [chore] checksum 改ざんテストの重複ファイルを統合 #43 を解決済みへ移動
  • §10.3 のテスト参照を glob（checksum*.test.ts）から単一ファイル（checksum.test.ts）へ

検証

• pnpm run all ✅（typecheck / lint / 47 tests / build）
• 削除ファイルを参照する CI / config / docs は無し（vitest デフォルト discovery、self-test.yml のコメントは残す側 checksum.test.ts を参照）

参照

• Closes [chore] checksum 改ざんテストの重複ファイルを統合 #43
• 要求仕様書 §10.3

🤖 Generated with Claude Code

---

Summary by cubic

Removed the duplicate checksum tamper tests and consolidated all coverage into tests/checksum.test.ts to simplify maintenance and keep §10.3 tests in one place. Closes #43.

• Refactors
  • Deleted tests/checksum-tamper.test.ts; its three scenarios are already covered in tests/checksum.test.ts.
  • Updated TODO.md to point §10.3 to tests/checksum.test.ts and moved [chore] checksum 改ざんテストの重複ファイルを統合 #43 to resolved.

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

[cubic-dev-ai]

No issues found across 2 files

_{Re-trigger cubic}

[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: ffb1f983-f00d-43c6-809c-d6a3987141a6

📥 Commits

Reviewing files that changed from the base of the PR and between c333892 and fd364dc.

📒 Files selected for processing (2)

• TODO.md
• tests/checksum-tamper.test.ts

💤 Files with no reviewable changes (1)

• tests/checksum-tamper.test.ts

---

📝 Walkthrough

Summary by CodeRabbit

• Chores
  • タスク一覧と完了状況を更新しました。
  • checksum 関連の重複していた検証内容を整理し、管理情報を最新化しました。

Walkthrough

TODO.md の v1.0.0 DoD で参照する checksum テスト名が更新され、Backlog から Issue #43 の項目が削除されました。解決済み Issue の備考には Issue #43 と PR #52 が追記されています。

Changes

TODO.md 更新

Layer / File(s)	Summary
checksum テスト参照と Issue 記録の更新 TODO.md	v1.0.0 DoD の参照先を tests/checksum.test.ts に更新し、Backlog から Issue #43 を削除、解決済み一覧に Issue #43 と #52 を追記した。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• yk-lab/setup-task#6: tests/checksum-tamper.test.ts の削除に関する変更で、Issue #43 の整理と直接つながる。

Poem

ぴょん、TODO の葉っぱが整列した
checksum の足あとも一列に
ぼくの耳はふるえ、Issue はおやすみ
#52 の月がぴかり
にんじん色の明日へ、ぴょん！

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	変更内容の中心である checksum 重複テストの統合を簡潔に示しており、内容と一致しています。
Description check	✅ Passed	概要・関連 issue・検証が含まれており、テンプレートの主要項目は概ね満たしています。
Linked Issues check	✅ Passed	Issue #43 の重複テスト統合、残存テストの集約、TODO 更新が要件どおりです。
Out of Scope Changes check	✅ Passed	削除と TODO 更新以外の無関係な変更は見当たりません。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/dedup-checksum-tests

---

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.}