- 新增或修改的行为，关键路径是否有测试覆盖
- 测试真的能失败吗：断言是否具体，避免只断言不抛异常
- 修 bug 时先有能复现 bug 的测试（先红后绿）
- 用例名表达场景与期望，让失败信息可读
Google 评审指南把测试单列一档：正确且有用的测试是评审重点。
| 有效性验证 | 把断言改成相反期望，确认测试会红 |
|---|---|
| 回归保护 | 复现缺陷的用例纳入模块测试集持续运行 |
评审检查项：测试
测试检查：新增行为是否有对应测试、测试能否失败、断言是否具体，以及改动对既有行为的回归是否有测试保护。
易错点
- 改动行为却只改实现不改测试：旧断言被悄悄放宽
- 追求覆盖率数字补无意义测试：永远不会失败的测试等于没有测试
数据来源与参考
- Google Engineering Practices · What to look for in a code review（Tests）