Prhub

#28934 [MUSA][24/N] CI:Fix LLM server smoke test

原始 PR 作者 Stella-17 合并时间 2026-06-24 13:58 文件变更 1 提交数 1 评论 2 代码增减 +1 / -1

执行摘要

修复 MUSA CI 烟感测试断言

MUSA LLM server smoke test 中 chat completion 响应断言过于严格,模型返回有效输出时可能因前导空格(如 " Beijing")导致误判失败,影响 CI 可靠性。PR body 明确说明:"Avoid false failures when the model returns valid output with leading whitespace."

该 PR 是极小范围的测试健壮性修复,无需精读。可关注其变更模式:测试断言中先 strip 再 lower,是一种通用的字符串匹配防御写法。

讨论亮点

无 review 讨论。Review 由 yeahdongcn 直接批准。

实现拆解

  1. test/registered/musa/test_llm_server_smoke_musa.pytest_send_receive_chat_message_contains_beijing 方法中,将第 85 行的断言语句从 self.assertIn("Beijing", content.lower()) 改为 self.assertIn("beijing", content.strip().lower())
  2. 主要改动:对 content 字符串先调用 .strip() 去除两端空白,再转为小写进行子串匹配。预期子串从 "Beijing" 改为全小写 "beijing",保持大小写一致性。
  3. 该文件是 MUSA 硬件后端的 LLM server 烟感测试,属于端到端测试,不涉及核心逻辑变更,仅修正测试断言健壮性。
文件 模块 状态 重要度
test/registered/musa/test_llm_server_smoke_musa.py CI 测试 modified 3.25

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

评论区精华

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

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

风险与影响

风险极低,仅修改一行测试断言,且由原大写匹配改为全小写匹配并加 strip,误报概率降低。不会影响生产逻辑或其他后端测试。

仅影响 MUSA 后端的 LLM server 烟感测试(CI nightly),确保模型返回合法输出时测试通过,提高 CI 稳定性。对其他后端、用户功能无影响。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论