From 926c4e239be49e36ba177e9f544170841eb7a6ef Mon Sep 17 00:00:00 2001
From: GitHub Actions <actions@github.com>
Date: Sun, 14 Dec 2025 03:37:37 +0000
Subject: [PATCH] fix: wrap mockOnClose in act() to fix flaky LiveLogViewer
 test

Fixes race condition where WebSocket disconnect event wasn't being
processed within React's rendering cycle, causing intermittent CI
failures. Wrapping mockOnClose() in act() ensures React state updates
are flushed before assertions run.

Resolves #237
---
 frontend/src/components/__tests__/LiveLogViewer.test.tsx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/frontend/src/components/__tests__/LiveLogViewer.test.tsx b/frontend/src/components/__tests__/LiveLogViewer.test.tsx
index f6750e56..e30fc0eb 100644
--- a/frontend/src/components/__tests__/LiveLogViewer.test.tsx
+++ b/frontend/src/components/__tests__/LiveLogViewer.test.tsx
@@ -321,7 +321,9 @@ describe('LiveLogViewer', () => {
 
     await waitFor(() => expect(screen.getByText('Connected')).toBeTruthy());
 
-    mockOnClose?.();
+    act(() => {
+      mockOnClose?.();
+    });
 
     await waitFor(() => expect(screen.getByText('Disconnected')).toBeTruthy());
   });