fix(test): 主机单测/覆盖率产物按工作树分家 + 每次跑独占目录（task-10）
现象（2026-10-25 三条道并行开发实测，看板 task-10）：全量 ci_local 的 L0 里 test-cppcheck / test-coverage 偶发 "files were modified by this hook"，单独重跑 即绿、工作树零改动。根因在 unit_test_harness：obj/exe 直接落在**全机共用**的 %TEMP%，名字由套件名派生（%TEMP%\net_socket_tests.exe、%TEMP%\net_socket_obj）， 多棵工作树并行跑同名套件必然互踩。 真正要命的不是编不过，而是**结论张冠李戴**：A 树编译完，B 树把同名 exe 又盖一 遍，A 树随后 subprocess 执行的是 B 树编出来的二进制——两棵树的 summary_marker 一模一样，harness 分辨不出来，于是拿别人的运行结果给自己判 PASS。这种失败不会 变红，只会静默给出错误结论，比红难查一个量级。 修法（不是加锁）： - 产物根按**工作树**分家：%TEMP%\dtu_host\<仓库根路径 sha1 前 10 位>\。树身份取 harness 文件自身位置（test/host/common 往上三级），不依赖 cwd/环境变量；工作树 之间只联接 .venv 与模拟器 SDK，test/host 每棵树各有一份，所以这个身份可靠。 - **每次运行再独占一个子目录** runs\<pid>-<纳秒>\，obj 与 exe 都放进去，跑完 finally 删（含 Ctrl-C 路径）。这一步顺带把"同树并发同名套件"也消灭了：两个 进程既拿不到同一个 exe，也拿不到同一个 .obj（实测共享根时并发会因 obj 撞名 直接编不过，返回 rc=2）。没有牺牲缓存：每个测试单元就一个 .c，cl 每次整体重编， "共享 obj 省编译"本来就是错觉。**不加锁**：这里没有需要互斥的独占资源，只是 名字撞了；加锁等于把 L0.5 变成 build/bench 那样的全局串行。 - 独占目录里留 <exe>.owner.json（suite / repo_root / pid / started）：需要证明"这 份产物属于哪棵树哪次跑"时有的可查，不必靠时间戳猜。 - 编译返回 0 但产物不存在 → 明确报环境错误，绝不跑来路不明的二进制。 - 两个开关：DTU_HOST_BUILD 改产物根（CI/容器固定路径），DTU_HOST_BUILD_KEEP=1 保留本次目录（要单步调试 exe 时用）。判据单一事实源 = host_build_root()。 - scripts/check_test_coverage.py 同源改造：work_root 由固定共享名 dtu_test_cov 改为 复用 harness 的 host_build_root()/coverage，并同步 --work-dir 帮助文案与 docstring （docs 里"并行期每人只跑 --module 自己模块"那条规避就此作废，它是为绕开旧共享名 而写的）。顺带删掉因此不再使用的 import tempfile。 附带修一处同族不诚实（做本任务时从自己的输出里发现）：ci_local 的收尾那行以前 无条件打印"L0 + L0b + L0c + L0.5 全部通过"，--l05-only 时等于把没跑过的段宣布 为通过——正是 A-5 那个"SKIP 冒充 PASS"的镜像版。改为只列本次真正跑过的段。 自证（都是实跑，不是推演）： - 两棵树身份并发跑同一套件：把修好的 harness 复制成 A/B 两个树身份（repo_root 不 同 → 两个不同产物根），套件 BUILD 从真仓库加载，各跑 4 轮并发 → 8/8 rc=0， 两侧 roots 分别是 …\55deb98e3b 与 …\902100bc41，互不可见。 - 敌意场景：把两边 DTU_HOST_BUILD 强行指到同一目录（旧写法必撞 exe/obj）→ 8/8 rc=0，共享根下只剩一个空的 runs/。同树并发同理被这一条覆盖。 - 单套：test/host/sha256 rc=0（13/13），产物确实落在 …\4b9166f602\runs\<pid>-…； 覆盖率 --module sha256 rc=0（line=111/114 func=7/7），落在 …\coverage\sha256。 - 全量：ci_local --l05-only → 33 runner(s)，2276/2276 assertions passed，fail=0。 - 临时目录与 scratch 已清理（脚本在仓库外，不入库；ci-artifacts/ 之外无残留）。 文档：test/host/common/README.md 新增"产物目录归属"一节（落点结构、为什么按树、 为什么每次独占、两个开关、一行命令查当前树的产物根）；harness 顶部注释是判据正文。 docs 侧两处按分工归 lead 在 done 时统一改，合并请求里已给可直接粘贴的替换文本： docs/测试说明/主机单测覆盖率口径.md 的产物目录条与"坑 12"、以及并行任务工作树 方案.md 的产物归属条。AGENTS.md 与 docs/** 本次一个字节没动。 RAM/Flash 增量：0 B（纯测试工具链，不进固件）。gate：见提交钩子输出，未用 --no-verify。 遗留：tools/font_gen/make_font.py 同样用固定共享名（%TEMP%\simsun_regular.ttf、 font_extra_<cp>.c）——同一类跨树竞态，但属道B 字体域，我没动，建议另开一张卡。
This commit is contained in:
@@ -46,6 +46,12 @@ r"""
|
||||
2 = 环境错误（找不到 cl.exe / 找不到 mscoverage / 编译失败 / XML 缺失或解析失败）
|
||||
3 = 用法错误
|
||||
|
||||
中间产物目录（2026-10-25 起与主机单测同家）：
|
||||
%TEMP%\dtu_host\<工作树哈希>\coverage\<模块>\ —— exe / PDB / .coverage /
|
||||
cobertura XML 都在这里，**不进仓库**。按工作树分家，是因为旧的固定共享名
|
||||
%TEMP%\dtu_test_cov 在多棵工作树并行跑覆盖率时会互相覆盖（判据与来龙去脉见
|
||||
test/host/common/unit_test_harness.py 顶部"产物目录归属"一节）。改根：--work-dir
|
||||
或环境变量 DTU_HOST_BUILD。
|
||||
用法（仓库根目录）：
|
||||
& '.venv\Scripts\python.exe' scripts/check_test_coverage.py # 全量跑 + 比基线
|
||||
& '.venv\Scripts\python.exe' scripts/check_test_coverage.py --module sha256 # 只跑一套（调试）
|
||||
@@ -79,7 +85,6 @@ import re
|
||||
import shutil
|
||||
import subprocess
|
||||
import sys
|
||||
import tempfile
|
||||
import xml.etree.ElementTree as ET
|
||||
from pathlib import Path
|
||||
|
||||
@@ -682,7 +687,17 @@ def run(args):
|
||||
prod = production_sources()
|
||||
canon = {rel.lower(): rel for rel in prod}
|
||||
|
||||
work_root = args.work_dir or os.path.join(tempfile.gettempdir(), "dtu_test_cov")
|
||||
# 中间产物根：**跟着工作树走**，与主机单测同一个家（task-10）。旧写法是
|
||||
# tempfile.gettempdir()/dtu_test_cov —— 全机一个名字，多棵工作树并行跑覆盖率就
|
||||
# 互相覆盖 exe/PDB/cov 文件（docs/测试说明/主机单测覆盖率口径.md 里"并行期每人
|
||||
# 只跑 --module 自己模块"那条规避，就是为了绕开它；分家之后不再需要）。
|
||||
# 复用 harness 的同一套判据（含 DTU_HOST_BUILD 覆盖），避免两处口径漂移。
|
||||
_host_build_root = getattr(harness, "host_build_root", None)
|
||||
if _host_build_root is None:
|
||||
print("[ENV] unit_test_harness 缺 host_build_root()——覆盖率产物无法与主机"
|
||||
"单测同家（harness 版本过旧？），拒绝跑。", file=sys.stderr)
|
||||
return 2
|
||||
work_root = args.work_dir or os.path.join(_host_build_root(), "coverage")
|
||||
if args.clean:
|
||||
shutil.rmtree(work_root, ignore_errors=True)
|
||||
os.makedirs(work_root, exist_ok=True)
|
||||
@@ -820,7 +835,7 @@ def main():
|
||||
ap.add_argument("--md", metavar="PATH", help="额外输出 Markdown 报告")
|
||||
ap.add_argument("--json", metavar="PATH", help="额外输出 JSON 明细")
|
||||
ap.add_argument("--work-dir", metavar="PATH",
|
||||
help="中间产物目录（默认临时目录下 dtu_test_cov）")
|
||||
help="中间产物目录（默认与主机单测同家：TEMP 下 dtu_host/<工作树哈希>/coverage）")
|
||||
ap.add_argument("--clean", action="store_true", help="跑前清空中间目录")
|
||||
ap.add_argument("--timeout", type=int, default=600, metavar="SEC",
|
||||
help="单模块插桩运行超时（默认 600 秒）")
|
||||
|
||||
+10
-1
@@ -441,17 +441,24 @@ def main(argv=None):
|
||||
return 2
|
||||
|
||||
rc = 0
|
||||
# 实际跑过哪些段——收尾那行只报这些。理由与 A-5 同源且是同一类错误：
|
||||
# "--l05-only 却宣布 L0/L0b/L0c 全部通过"就是把没跑的说成跑过了（SKIP 冒充 PASS
|
||||
# 的镜像版）。2026-10-25 做 task-10 时从 --l05-only 的输出里发现，当场改掉。
|
||||
ran = []
|
||||
try:
|
||||
if args.size_only:
|
||||
return run_size_gate()
|
||||
if not args.l05_only:
|
||||
rc = run_l0()
|
||||
ran.append("L0")
|
||||
# 提交信息门禁独立于 --all-files（stage 不同），故单独跑一段；
|
||||
# 即使 L0 已失败也照跑，一次看全所有违规项。
|
||||
rc_msg = run_gitlint(args.commits)
|
||||
ran.append("L0b")
|
||||
# L0c 体积门禁同样**无条件照跑**：它回答的是"这份代码测过体积没有"，
|
||||
# 与 L0/L0b 的结论正交，且只要 0.3 s——少跑一次就是 A-5 那个盲区。
|
||||
rc_size = run_size_gate()
|
||||
ran.append("L0c")
|
||||
if rc != 0 or rc_msg != 0 or rc_size != 0:
|
||||
if not args.keep_going:
|
||||
print("")
|
||||
@@ -461,9 +468,11 @@ def main(argv=None):
|
||||
rc = rc or rc_msg or rc_size
|
||||
if not args.l0_only:
|
||||
rc05 = run_l05(args)
|
||||
ran.append("L0.5")
|
||||
rc = rc or rc05
|
||||
if args.coverage:
|
||||
rc = rc or run_coverage()
|
||||
ran.append("L0.5c")
|
||||
except FileNotFoundError as e:
|
||||
print("[ERROR] 环境错误：{}".format(e))
|
||||
return 2
|
||||
@@ -471,7 +480,7 @@ def main(argv=None):
|
||||
print("")
|
||||
print("=" * 64)
|
||||
if rc == 0:
|
||||
print(" [OK] 本地 CI 全部通过（L0 + L0b + L0c + L0.5）")
|
||||
print(" [OK] 本地 CI 全部通过（{}）".format(" + ".join(ran)))
|
||||
else:
|
||||
print(" [FAIL] 本地 CI 有未通过项（见上方输出）")
|
||||
print("=" * 64)
|
||||
|
||||
Reference in New Issue
Block a user