- 行为变化无新测试：要求补，且确认测试能失败
- bug 修复无复现用例：先加能复现的测试再合并
- 关键分支（错误路径、边界）无覆盖：要求补关键路径
- 可豁免：纯注释、纯格式、无行为变化的机械重构
要求补测试时给出范围：只提'加测试'会让作者不知从哪下手。
| 要求写法 | 具体指出哪个行为、哪条路径缺测试 |
|---|---|
| 豁免记录 | 理由留在评论里，避免下次重复讨论 |
评审中要求补测试的时机
遇到行为变化无测试、修复无复现用例、关键分支无覆盖三种情况应要求补测试；纯注释、纯格式与无行为变化的机械重构可豁免。
易错点
- 笼统要求测试再全点：作者补了无意义测试交差
- 对机械性重构强行要求测试：浪费精力，现有测试已能验证行为不变
数据来源与参考
- Google Engineering Practices · What to look for in a code review（Tests）