Skip to content

test(dashboard): 避免两级会话发现被默认等待预算误判 - #142

Merged
Disdjj merged 2 commits into
mainfrom
Disdjj/device-session-test-stability
Sep 15, 2026
Merged

Disdjj merged 2 commits into
mainfrom
Disdjj/device-session-test-stability

Conversation

@Disdjj

@Disdjj Disdjj commented Sep 15, 2026

Copy link
Copy Markdown
Member

问题与变化

进程会话发现需要两次异步 Help 查询和 React 渲染。本地高负载时,Testing Library 默认一秒等待会把调度延迟误判为发现失败。将这一个可见性断言的等待上限明确设为五秒,保留真实 SDK 解析、精确请求路径及全部行为断言。

仅修改测试和对应 llmdoc 指纹,不改变发布产物,无需追加版本。

验证

  • Dashboard 全部 32 文件、222 测试通过(本地单 worker)。
  • 提交 hook:lint-staged 和全仓 typecheck 通过。
  • 本地完整 verify 复跑:其余包通过,server 旧设备回收测试在 30 秒超时(159/160 测试通过),日志无断言差异;不将该轮标记为通过。
  • 本 PR 以 Node 22 / Ubuntu 的完整 CI 门禁作为合入依据;产品提交此前 PR feat(device): 可观察长命令会话与设备上下文 #141 和 main CI 已全部通过。

整包运行的 CPU 争用会使两次 Help 查询与 React 渲染超过 Testing Library 默认一秒;单独用例和 Node22 CI 均通过。明确五秒等待上限,仍断言相同面板和精确请求路径,避免把调度延迟误判成发现失败。仅测试变更,不影响已发布产物。
@Disdjj
Disdjj merged commit e67959c into main Sep 15, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant