Skip to content

[Execute Infrastructure] Rename test_ai_* test files to test_coverage_* - #4993

Merged
swgu98 merged 1 commit into
PaddlePaddle:developfrom
liuhao2638:codex/rename-test-ai-to-test-coverage
Sep 10, 2026
Merged

[Execute Infrastructure] Rename test_ai_* test files to test_coverage_*#4993
swgu98 merged 1 commit into
PaddlePaddle:developfrom
liuhao2638:codex/rename-test-ai-to-test-coverage

Conversation

@liuhao2638

Copy link
Copy Markdown
Contributor

Summary

Rename all test_ai_* test files under tests/coverage_test to test_coverage_*, following up on #4966 which renamed the directory ai_edited_test to coverage_test.

Changes:

  • Rename 180 test_ai_*.py files under tests/coverage_test/ to test_coverage_*.py
  • Move the missed tests/ai_edited_test/quantization/ directory (3 files) into tests/coverage_test/quantization/, completing the ai_edited_test -> coverage_test rename
  • Update in-file references to the renamed test files

Test plan

  • No test logic changes: pure file renames with reference updates
  • grep -r "test_ai_" on the repo returns only the unrelated test_ai_studio method names in tests/transformers/

@CLAassistant

CLAassistant commented Sep 8, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Paddle-Bot

Paddle-Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Failed CI看板

流水线名称 问题标签 修复建议 日志片段
Require review-bot approval 审批门禁未通过 【与当前改动无关】为当前 head 2a95054f0217c5db51ab22c5946691d586b3c9e6 获取机器人或指定审批账号的 APPROVED 状态 Job
model-unittest-gpu-ci 任务被取消且日志获取失败 【无法判断】重新运行并获取完整 job 日志后再定位测试失败原因 Job
upload-allure 任务被取消且日志获取失败 【无法判断】在主测试任务完成后重新运行并确认是否存在产物依赖问题 Job
Integration test (A100) 任务被取消且日志获取失败 【无法判断】重新运行 A100 job 并获取完整日志,当前不能归因于 PR 改动 Job
Integration test (H20, multi-card) 任务被取消且日志获取失败 【无法判断】重新运行 H20 多卡 job 并获取完整日志,当前不能归因于 PR 改动 Job
日志分析报告

失败的测试 case:

Require review-bot approval
- 测试 case:无,执行的是审批状态检查脚本
- 关键步骤:读取当前 PR head 的 reviews,检查 risemeup1111、Paddle-Bot,以及 sneaxiy、From00 的最新决定性状态
- 关键错误:当前 head 没有满足条件的 APPROVED review
- 结果:exit code 1

model-unittest-gpu-ci
- 测试 case:无法确认,job 被 cancelled
- 关键错误:日志获取失败:gh: HTTP 404

upload-allure
- 测试 case:无法确认,job 被 cancelled
- 关键错误:日志获取失败:gh: HTTP 404

Integration test (A100)
- 测试 case:无法确认,job 被 cancelled
- 关键错误:日志获取失败:gh: HTTP 404

Integration test (H20, multi-card)
- 测试 case:无法确认,job 被 cancelled
- 关键错误:日志获取失败:gh: HTTP 404

根本原因分析:

PR diff 展示的改动几乎全部是测试文件重命名,文件内容保持 100% 相似;唯一实质内容变化是辅助文件 docstring 中的测试文件名更新。没有看到模型训练、GPU 配置、依赖、数据集或测试逻辑修改。

已确认的失败原因是审批门禁:脚本只接受当前 head 上的有效 APPROVED review,日志明确显示没有机器人或指定审批账号通过审批,因此主动执行 exit 1。这不是测试代码执行失败,也没有证据表明由本次重命名改动触发。

其余四个 job 仅显示 cancelled,且日志抓取本身返回 gh: HTTP 404。该 404 是日志不可用证据,不是 CI job 内部错误。快照无法确认这些任务是否因审批门禁、workflow 依赖、手动取消或基础设施原因被取消,也无法确认任何测试 case 已执行失败。因此不能将它们归因于 PR,也不能把 upload-allure 或其他取消任务认定为独立根因。

修复建议:

  1. 【与当前改动无关】Require review-bot approval:为当前 head 获取 risemeup1111Paddle-Bot 任一机器人,或 sneaxiyFrom00 任一审批账号的有效 APPROVED review,然后重新检查门禁。
  2. 【无法判断】model-unittest-gpu-ci:重新运行该 workflow 并保留完整 job 日志,确认是否实际开始执行测试以及首个真实错误。
  3. 【无法判断】upload-allure:重新运行关联任务;只有在主测试任务产生明确产物错误时,才进一步检查 Allure 上传逻辑。
  4. 【无法判断】Integration test (A100):重新运行 A100 job 并获取完整日志,当前没有可验证的测试失败证据。
  5. 【无法判断】Integration test (H20, multi-card):重新运行 H20 多卡 job 并获取完整日志,当前没有可验证的测试失败证据。

Powered by Nyanpasu with gpt-5.6-luna 默认推理级别, please check the suggestions carefully.

@liuhao2638
liuhao2638 force-pushed the codex/rename-test-ai-to-test-coverage branch from e484d89 to 2a95054 Compare September 9, 2026 03:26
@liuhao2638
liuhao2638 force-pushed the codex/rename-test-ai-to-test-coverage branch 2 times, most recently from 7e1b16d to a25769a Compare September 10, 2026 07:08
All 182 test files under tests/coverage_test are renamed from test_ai_* to
test_coverage_* to match the coverage_test directory naming. This also:
- Moves the missed tests/ai_edited_test/quantization directory into
  coverage_test, completing the ai_edited_test -> coverage_test rename
- Updates in-file references to the renamed files
@liuhao2638
liuhao2638 force-pushed the codex/rename-test-ai-to-test-coverage branch from a25769a to 42965c8 Compare September 10, 2026 07:18
@swgu98
swgu98 merged commit 33a4a88 into PaddlePaddle:develop Sep 10, 2026
24 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants