# PR #21130 完整报告

- 仓库：`sgl-project/sglang`
- 标题：Add ut guide to test skills
- 合并时间：2026-03-22 20:55
- 原文链接：http://prhub.com.cn/sgl-project/sglang/pull/21130

---

# 执行摘要
本次 PR 更新了 SGLang 项目的测试编写指南，新增单元测试专用章节和模板，旨在提高测试组织效率和一致性。变更仅涉及文档文件，对系统运行时无影响，但需注意 review 中提到的文档对齐问题。

# 功能与动机
PR 的主要动机是改进测试编写文档，为单元测试提供明确指导。根据 review 评论，gemini-code-assist[bot] 指出“This pull request updates the SGLang test writing guide to include specific instructions for unit tests.”，目标是解决现有指南中的不明确之处，帮助开发者更有效地编写不依赖服务器的测试。

# 实现拆解
实现集中在两个文档文件的修改：
- `.claude/skills/write-sglang-test/SKILL.md`：添加了“Unit Tests”章节，更新了模型选择表格以包括 `stage-a-cpu-only` 套件，并提供了单元测试模板。关键变更如将“Prefer mock over real server”描述更精确，并强调单元测试应放置在 `test/registered/unit/`。
- `test/registered/unit/README.md`：调整示例代码使用 `CustomTestCase`，并更新规则部分以与主指南对齐。

# 评论区精华
review 中，gemini-code-assist[bot] 提出了两个关键不一致点：
> “This line refers to `test/registered/unit/README.md`, which contains guidance that conflicts with this document... To avoid confusion, it would be good to align these two documents.”
> “I noticed a related inconsistency in the checklist... the checklist should be updated to match the template.”

这些评论强调了文档对齐的重要性，但 PR 已合并，未显示解决状态。

# 风险与影响
**风险**：主要风险是文档不一致可能导致开发者遵循错误指南，例如在单元测试中使用不正确的测试基类或 CI 注册方式。具体在 SKILL.md 和 README.md 之间的冲突，可能引发 confusion，但无代码执行风险。

**影响**：对开发者有积极影响，提供更清晰的测试编写指南，有助于提高测试质量和开发效率。对团队，可能减少因测试配置错误导致的 CI 失败，优化资源使用。

# 关联脉络
从近期历史 PR 看，无直接相关的文档变更 PR。但测试相关的 PR（如 #20625 修复 abort 测试）可能受益于本次指南更新，促进更规范的测试编写。整体上，本次 PR 是测试基础设施文档的迭代改进。