utest: keep testcase failures across units#11601
Conversation
📌 Code Review Assignment🏷️ Tag: componentsReviewers: @Maihuanyi Changed Files (Click to expand)
🏷️ Tag: kernelReviewers: @GorrayLi @ReviewSun @hamburger-os @lianux-mm @wdfk-prog @xu18838022837 Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-07-14 11:34 CST)
📝 Review Instructions
|
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
|
7f14941 to
cce6687
Compare
拉取/合并请求描述(PR description)
[
为什么提交这份PR (why to submit this PR)
修复 utest testcase 汇总误判问题:当一个 testcase 内部包含多个
UTEST_UNIT_RUN时,某个 unit 失败后,如果后续 unit 通过,会因为utest_unit_run()重置local_utest.failed_num,导致 testcase 和最终汇总仍显示通过。你的解决方案是什么 (what is your solution)
在 testcase 开始执行前清零统计;每个 unit 仍保持独立统计,unit 执行结束后将本 unit 的 pass/fail 数累加回 testcase 总计。这样可以保留 unit 级隔离,同时保证 testcase 级失败不会被后续 unit 覆盖。
请提供验证的bsp和config (provide the config and bsp)
BSP: bsp/qemu-vexpress-a9
.config: CONFIG_RT_USING_UTEST=y,CONFIG_RT_USING_UTESTCASES=y
action: 本地环境缺少可用交叉编译工具链,未触发 action;已执行
git diff --check]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I have considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up