执行摘要
此PR修复了Python unittest中setUpClass失败时tearDownClass被跳过导致的资源泄漏问题，通过修改CustomTestCase添加自动清理机制，并更新文档推广防御性写法。变更影响所有测试类，提升CI稳定性，降低端口冲突风险。
功能与动机
动机：这是一个预防性修复，旨在解决CI中潜在的资源泄漏问题。当测试的setUpClass中途失败（如服务器启动超时或内存不足）时，Python的unittest框架会跳过tearDownClass，导致已分配的资源（如进程、端口）无法释放，可能引发后续测试的端口冲突。PR body中强调：“这可以防止级联端口冲突”。
实现拆解
实现分为三个关键部分：
| 模块 |
文件 |
关键变更 |
说明 |
| 测试工具 |
python/sglang/test/test_utils.py |
在CustomTestCase类中添加__init_subclass__方法 |
包装setUpClass，在异常时自动调用tearDownClass，并处理多级继承的标记检查。代码片段： |
def safe_setUpClass(klass, _orig=setup):
try:
_orig.__func__(klass)
except Exception:
try:
klass.tearDownClass()
except Exception:
pass
raise
| 文档 | .claude/skills/write-sglang-test/SKILL.md | 更新测试编写指南 | 添加规则：始终使用CustomTestCase，tearDownClass必须防御性（使用hasattr检查）。 |
| 示例测试 | test/registered/hicache/test_hicache_storage_file_backend.py | 修改tearDownClass方法 | 使用hasattr检查cls.process和cls.temp_dir，确保资源存在再清理。 |
评论区精华
PR中没有review评论，表明变更可能被快速接受。但从提交历史可见，初始实现后通过提交“[CI] Fix sentinel check for multi-level inheritance in CustomTestCase”修复了标记检查问题，确保包装逻辑在子类继承链中正确工作。这揭示了设计时需考虑的边缘情况。
风险与影响
风险：
- 异常处理逻辑可能引入新bug，如tearDownClass异常被抑制，但PR中已处理以优先传播原始异常。
- 防御性清理可能掩盖setUpClass中的初始化错误，但这是确保资源释放的必要权衡。
- 变更影响217+测试类，需验证现有CI通过，PR测试计划已涵盖happy path验证。
影响：
- 对系统：提高CI作业稳定性，减少因资源泄漏导致的失败，间接提升开发效率。
- 对团队：开发人员需遵循新文档编写防御性tearDownClass，增加代码健壮性。
- 对用户：无直接功能影响，但更可靠的CI有助于更快交付高质量代码。
关联脉络
从近期历史PR看，本PR是CI稳定性改进系列的一部分。例如：
- PR 21331重写
killall_sglang优化资源清理。
- PR 21345改进CI请求效率。
- PR 21371修复HiCache测试失败。
这些PR共同显示团队对测试基础设施的持续投入，本PR通过框架级修复预防资源泄漏，补充了现有CI优化工作。
参与讨论