| name | test-review |
| description | テストコードのレビューを行うとき、テストの品質チェックを行うときに使用 |
| allowed-tools | Read, Grep, Glob |
テストレビュー基準
テストコードレビュー時に以下の観点でチェックを行う。
テストカバレッジ
- 新規機能に対するテストの有無
- 既存機能への影響範囲のテスト
- エッジケースのカバー状況
- エラーケースのテスト
テストの種類と品質
- 単体テスト(Unit Test)の適切性
- 統合テスト(Integration Test)の必要性
- E2Eテストの必要性
- テストの可読性と保守性
テストケースの十分性
- 正常系のテスト
- 異常系のテスト(バリデーションエラー、ネットワークエラー等)
- 境界値テスト
- 並行処理のテスト(該当する場合)
セキュリティテスト
- 認証・認可のテスト
- 入力検証のテスト
- エラーハンドリングのテスト
モックとテストデータ
- 外部依存のモック化
- テストデータの適切性
- テスト環境の分離
パフォーマンステスト
過剰なテストの検出
- 実装の詳細に依存しすぎたテスト(脆弱なテスト)
- 重複したテストケース
- 価値の低いテスト(自明な処理のテスト)
- メンテナンスコストが高すぎるテスト
- 不要にモックが多すぎるテスト