Prhub

#41877 [CI] Add tests/parser to CI coverage

原始 PR 作者 sfeng33 合并时间 2026-05-12 03:08 文件变更 2 提交数 2 评论 0 代码增减 +4 / -0

执行摘要

将 tests/parser 加入 CI 覆盖率

PR body 说明:"Add tests/parser coverage in CI in the CPU only test group." 目的是将 parser 相关测试纳入 CI 持续覆盖范围,防止回归。

该 PR 变更简单明了,无需精读。值得关注的是 CI 配置的维护模式:新增测试目录后需同步更新两个配置文件。

讨论亮点

无 review 讨论。机器人自动评论无实质反馈,chaunceyjiang 直接批准。

实现拆解

  1. .buildkite/test-amd.yamlAsync Engine, Inputs, Utils, Worker, Config (CPU) 步骤中,在 source_file_dependencies 列表添加 - tests/parser,并在对应的 commands 列表添加 - pytest -v -s parser
  2. .buildkite/test_areas/misc.yaml 的 CPU small 设备步骤中,同样在 source_file_dependencies 列表添加 - tests/parser,并在 commands 列表添加 - pytest -v -s parser
  3. 两个配置保持了相同的变更模式,确保在 AMD (MI250) 和通用 CPU small 环境中都运行 parser 测试。
文件 模块 状态 重要度
.buildkite/test-amd.yaml 构建脚本 modified 2.5
.buildkite/test_areas/misc.yaml 构建脚本 modified 2.5

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

评论区精华

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

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

风险与影响

风险极低。变更仅为 CI 配置添加,不涉及任何运行时逻辑。需确保 tests/parser 目录下的测试用例在无 GPU 环境下可正常通过,否则可能导致 CI 步骤失败。但鉴于已有类似测试目录(如 tests/tool_parserstests/reasoning)配置方式相同,且测试目录通常设计为 CPU 可运行,风险可控。

影响范围:仅限于 CI 流程。变更后,tests/parser 下的所有测试将在两个 CI 步骤中自动运行,增强 parser 模块的回归防护。对用户无直接影响。影响程度低。

低风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论