Prhub

#30608 [misc] Add unit test admission criteria to agent rules

原始 PR 作者 hnyls2002 合并时间 2026-07-09 14:41 文件变更 1 提交数 1 评论 1 代码增减 +43 / -0

执行摘要

新增单元测试准入规则至 agent 配置

PR body 指出需要定义清晰的单元测试准入标准,引导开发者只编写能对实际 diff 产生守卫价值的测试,避免低质量或冗余的测试浪费维护精力。

值得精读该规则文档,并建议在团队内推广使用,尤其适合作为新成员编写测试的指引。可与 write-sglang-test 技能文档配合使用。

讨论亮点

PR 无 review 评论,故无讨论亮点。

实现拆解

  1. .claude/rules/unit-test-admission.md 中定义路径范围:test/**/*.py,仅当修改测试文件时注入该规则。
  2. 明确三类可准入测试:bug 回归(要求能在修复前代码上复现)、推导属性(保护边界/协议语义等推导结论)、关键路径记账(如注册表完整性、序列化兼容性)。
  3. 明确拒绝三类测试:快乐路径赘述、镜像测试(实现逻辑重述)、概率性压力测试(无法确定性能复现故障)。
  4. 强调强测试优于多测试,每个用例应守卫独立的故障模式。
文件 模块 状态 重要度
.claude/rules/unit-test-admission.md 开发规范 added 4.03

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

该 PR 仅添加文档规则,无代码变更,无技术风险。但规则的有效性依赖于团队执行,需后续通过 code review 和 CI 来确保遵循。

影响范围限于 SRT 团队内部代码审查流程:开发者在编写测试时会参考该规则,从而提升测试质量,减少低价值测试。对系统运行无直接影响。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论