From 9267677d383f23e593be305bc3057cf4152decb8 Mon Sep 17 00:00:00 2001 From: tanzhenxin Date: Tue, 16 Dec 2025 20:08:43 +0800 Subject: [PATCH] fix failed test --- .../cli/src/ui/hooks/useResumeCommand.test.ts | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/packages/cli/src/ui/hooks/useResumeCommand.test.ts b/packages/cli/src/ui/hooks/useResumeCommand.test.ts index a0441cd5..daaedfcc 100644 --- a/packages/cli/src/ui/hooks/useResumeCommand.test.ts +++ b/packages/cli/src/ui/hooks/useResumeCommand.test.ts @@ -158,21 +158,23 @@ describe('useResumeCommand', () => { }); expect(result.current.isResumeDialogOpen).toBe(true); - const resumePromise = act(async () => { - // Intentionally do not resolve loadSession yet. - await result.current.handleResume('session-2'); + let resumePromise: Promise | undefined; + act(() => { + // Start resume but do not await it yet — we want to assert the dialog + // closes immediately before the async session load completes. + resumePromise = result.current.handleResume('session-2') as unknown as + | Promise + | undefined; }); - - // After the first flush, the dialog should already be closed even though - // the session load is still pending. - await act(async () => {}); expect(result.current.isResumeDialogOpen).toBe(false); // Now finish the async load and let the handler complete. resumeMocks.resolvePendingLoadSession({ conversation: [{ role: 'user', parts: [{ text: 'hello' }] }], }); - await resumePromise; + await act(async () => { + await resumePromise; + }); expect(config.startNewSession).toHaveBeenCalledWith( 'session-2',