test(health): 循环测试不再断言墙钟速率

[modusensus]

main 现在有一条腿是红的：Python 3.11 (macos-latest)，test_run_loop_stops_cleanly 的 assert len(seen) >= 3 拿到 2。这个 PR 修掉它，以及同文件里同类的第二条。

不是产品代码的问题，是断言考错了对象

run_loop 每轮是 check() + stop_event.wait(interval)，而测试的写法是：

stop = hc.run_loop(interval=0.05, callback=...)
time.sleep(0.3)          # 0.3 / 0.05 = 6 次的名目值
assert len(seen) >= 3    # 只留了 2 倍余量

CI 记录里两次检查的时间戳相隔 0.12 秒——带 coverage 插桩的 macOS runner 每轮比标称慢 2.4 倍，2 倍余量就没了。也就是说：这个断言考的是"这台机器够快"，不是"循环会反复检查"。同一文件末尾那句"停止后计数不再增长"也是固定 0.15 秒窗口，方向相反地同样脆。

改法

• 次数：改为带截止时间地等（新增 _wait_for，原有的 _wait_for_values 改为复用它），不再睡固定秒数。describe 传函数，失败信息在等过之后才取，否则记的是等待前那一刻的状态。
• "停下来"：改用 join 证明循环线程真的结束，而不是睡一段时间再看计数没变——这才是测试名字里的 stops cleanly。顺带把实现的承诺写进断言：设置停止事件后至多一次在途回调（循环只在 wait 之前查一次事件）。
• isinstance 检查挪到 join 之后：线程已死、计数不会再变，遍历才不是"边遍历边被改"。
• 同文件的 test_run_loop_callback_error_swallowed（睡 0.15 秒后断言回调异常已记录）用了同一个 _wait_for。

验证

用一个插件把每轮 check() 拖慢来模拟慢 runner：

条件	旧版本	新版本
0.1 秒/轮	test_run_loop_stops_cleanly 失败（与 CI 同一条 AssertionError: 2）	全过
0.2 秒/轮	两条测试都失败	全过
0.5 秒/轮	—	全过

本地 pytest 413 passed、覆盖率 86.71%、ruff/mypy（本机与 --platform linux）/_smoke_test.py 全过；test_health.py 连跑 10 次输出一致。

我看到的、但这次没动的一处同类

tests/test_core.py:419：tm.connect() 之后 time.sleep(0.05)，断言 stderr 排出线程记的日志已经落地。形状一样（拿固定睡眠当一个异步副作用的上限），但我没有复现出它的失败——所以这里只是指出，不是断言它现在会挂。要动的话得先想好那段等待的谓词是什么，以及跨文件是否该共用一个等待助手。

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: cabb2de7-6b70-412c-b209-5ea0c3c87d5e

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}