测试运行器:位置参数真的只跑指定路径;文档改成实际能跑通的命令与数字
scripts/run_unit_tests.sh
用法注释里写的是「bash run_unit_tests.sh src/<pkg>/test/test_x.py -x」,但实现是
`pytest "${test_dirs[@]}" "$@"` —— 传进来的路径只是被**追加**到「全部测试目录」
后面,实际仍然跑整仓库。用法示例就是承诺,它不成立会让人以为「我只改了 x,
怎么蹦出别的包的失败」。改成:位置参数里**存在的路径**当作「只跑这些」,
其余(-k / -x / --tb=… 及其取值)原样透传。
docs/HANDBOOK.md §4.3
原文让人逐个包手敲 `python3 -m pytest src/xxx/test`。这条命令在本仓库**会失败**,
而且是两种最容易被误读的失败:
· 少挂一个 src 包 → 收集阶段 ModuleNotFoundError,现象是「0 个测试 / 1 个 error」,
像测试环境坏了;
· 更隐蔽:install/ 里那份**旧的**同名模块会先被找到。实测直接跑
src/arm_sim_gazebo/test 报 `cannot import name 'RESET_GIVE_UP'`,导的是
install/ 里的旧副本,源码里明明有这个符号 —— 测的根本不是你改的那份。
改为统一用 run_unit_tests.sh(它把 src/* 挂到 PYTHONPATH 最前面),并把这段
原因写进文档。顺带把过期的统计数字更新为实测值:608 → 1118 项,
分项 arm_application 233→547、arm_sim_gazebo 236→400、arm_interfaces 46→78。
实测:src/arm_sim_gazebo/test 直跑 = 2 个收集错误;经运行器 = 400 passed。
单文件 28 passed;-k real_preflight = 28 passed / 1 skipped;
无参数全量 = 1118 passed / 1 skipped。
docs/QUICKSTART.md
「第一次实机」以体检开场(它只读、不需要机械臂能动),并列出四项判据各抓什么,
免得现场跳过它直接起栈。
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
@@ -38,7 +38,23 @@ if [ "${#test_dirs[@]}" -eq 0 ]; then
|
||||
fi
|
||||
|
||||
cd "${ROOT}"
|
||||
if [ "$#" -gt 0 ]; then
|
||||
exec "${PY}" -m pytest "${test_dirs[@]}" -q "$@"
|
||||
# 位置参数里**存在的路径**当作"只跑这些",其余(-k / -x / --tb=… 及其取值)
|
||||
# 原样透传给 pytest。
|
||||
#
|
||||
# 为什么要分开:以前是 `pytest "${test_dirs[@]}" "$@"` —— 传进来的路径只是被
|
||||
# **追加**到"全部测试目录"后面,于是 `run_unit_tests.sh src/x/test` 实际跑的是
|
||||
# **整仓库**。脚本头部的用法示例看起来像"只跑这个文件",实际不是,很容易被读成
|
||||
# "我只改了 x,怎么蹦出别的包的失败"。用法示例就是承诺,得让它成立。
|
||||
paths=()
|
||||
extra=()
|
||||
for a in "$@"; do
|
||||
if [[ "${a}" != -* && -e "${a}" ]]; then
|
||||
paths+=("${a}")
|
||||
else
|
||||
extra+=("${a}")
|
||||
fi
|
||||
done
|
||||
if [ "${#paths[@]}" -gt 0 ]; then
|
||||
exec "${PY}" -m pytest "${paths[@]}" -q "${extra[@]}"
|
||||
fi
|
||||
exec "${PY}" -m pytest "${test_dirs[@]}" -q
|
||||
exec "${PY}" -m pytest "${test_dirs[@]}" -q "${extra[@]}"
|
||||
|
||||
Reference in New Issue
Block a user