Skip to content

test: 五条把 main 打红的 Windows 计时型用例改成断言行为 #1616

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MagicLizi merged 4 commits into from
Aug 4, 2026
Merged

test: 五条把 main 打红的 Windows 计时型用例改成断言行为 #1616

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
18caea5
test(device-link): 握手超时用例改有界等待,修 main 上 Windows 单测红灯
zqchris Aug 4, 2026
0ec2990
test(skillhub): 锁并行用例改断言临界区重叠,去掉墙钟阈值
zqchris Aug 4, 2026
0941a27
test: 再修两条把 main 打红的 Windows 计时型用例
zqchris Aug 4, 2026
04e174f
test(device-link): 心跳僵死用例改有界等待(第五条打红 main 的计时型用例)
zqchris Aug 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions packages/device-link/src/__tests__/client.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3223,8 +3223,10 @@ describe('DeviceLinkClient', () => {
await tick();
const first = h.current();
first.emit('open'); // upgrade 成功但对端不回 hello-ack(半开/服务假活)
await tick(50);
// watchdog 触发新建连接(测试窗口内后续连接可能再次超时,只断言 ≥2)
// 负载下(Windows CI 分片并跑)事件循环调度可能远超名义毫秒数:单次固定 tick(50)
// 不足以保证 15ms 握手看门狗 + 退避重连都已落地。有界等待到第二个 socket 出现,
// 断言语义不变(watchdog 必须触发新建连接;测试窗口内后续连接可能再次超时,只断言 ≥2)。
for (let i = 0; i < 40 && h.sockets.length < 2; i++) await tick(10);
expect(h.sockets.length).toBeGreaterThanOrEqual(2);
expect(first.terminated || first.closed !== null).toBe(true); // 旧 socket 被回收
// 负载下(全量并跑)事件循环调度可能远超名义毫秒数:current() 拿到的
Expand Down