vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-07-10 01:29:17 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 12

fix(loop): update Session token-limit test for the disable() breaker

Browse source 
CI regression from the disable() refactor (7f488e6c6): the token-limit
breaker now calls scheduler.disable() instead of stop(), but the
cron-fired token-limit test still asserted scheduler.stop and its mock
scheduler had no disable() — so the call hit an undefined method and the
stop spy saw 0 calls. Missed because the prior change ran core tests +
cli tsc but not the cli test suite (Session.test.ts is cli).

Add disable() to the mock and assert it's called once. The breaker
disables (permanent for the session, so a later LoopWakeup is rejected),
which internally stops too.

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
This commit is contained in:
qqqys 2026-06-17 16:26:00 +08:00
parent 37ee6fe362
commit 240e8892bc
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
packages/cli/src/acp-integration/session/Session.test.ts
View file

@ -3259,6 +3259,7 @@ describe('Session', () => {
callback({ prompt: 'scheduled prompt' });
}),
stop: vi.fn(),
disable: vi.fn(),
getExitSummary: vi.fn().mockReturnValue(undefined),
};
mockConfig.isCronEnabled = vi.fn().mockReturnValue(true);
@ -3308,7 +3309,9 @@ describe('Session', () => {
},
},
});
expect(scheduler.stop).toHaveBeenCalledTimes(1);
// Token limit disables the scheduler (permanent for the session, so
// a later LoopWakeup is rejected), not just stops it.
expect(scheduler.disable).toHaveBeenCalledTimes(1);
await vi.waitFor(() => {
expect(mockClient.sessionUpdate).toHaveBeenCalledWith({
sessionId: 'test-session-id',