Prhub

#29308 Fix CI broken by #28450

原始 PR 作者 b8zhong 合并时间 2026-06-26 01:35 文件变更 1 提交数 1 评论 2 代码增减 +3 / -3

执行摘要

修复 #28450 引入的 CI 测试断言错误

28450 合并后 CI 持续失败,PR body 指出该失败并非 flake,而是 100% 复现。需要修复测试使 CI 通过。

该 PR 是常规的 CI 修复,无需精读。值得注意的点:原始 PR #28450 在合并前应确保 CI 通过,此修复提醒团队加强合并前检查。

讨论亮点

无 reviewer 评论。仅自动化 bot 总结变更内容,以及另一 reviewer 直接批准。

实现拆解

  1. 修改 test/registered/moe/test_topk_padded_region.pytest_invalid_pad_count_tensor_raises 测试方法:将 assertRaises(AssertionError) 替换为 assertRaises(TypeError)(针对 Python int 和 float tensor 参数)和 assertRaises(ValueError)(针对多维 tensor 参数),以匹配 _fill_padded_rows 实际抛出的异常类型。
  2. 仅修改了该测试文件中的断言类型,未涉及任何源码逻辑变更。
文件 模块 状态 重要度
test/registered/moe/test_topk_padded_region.py MoE 测试 modified 3.55

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

评论区精华

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

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

风险与影响

风险极低。仅修改测试断言以匹配现有实现行为,不涉及任何生产代码或运行时逻辑。

直接影响:修复 stage-b CUDA CI 测试,确保后续 PR 不会被此失败的测试阻塞。间接影响:提高了测试的精确性,使异常类型检查更加严格。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论