Skip to content

Commit fe733af

Browse files
chore: fix flake on useAgentLogs (#17919)
We need to wait for the result since the result is depending on effects. Fix coder/internal#644
1 parent 433f0be commit fe733af

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

site/src/modules/resources/useAgentLogs.test.ts

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { renderHook } from "@testing-library/react";
1+
import { renderHook, waitFor } from "@testing-library/react";
22
import type { WorkspaceAgentLog } from "api/typesGenerated";
33
import WS from "jest-websocket-mock";
44
import { MockWorkspaceAgent } from "testHelpers/entities";
@@ -29,17 +29,23 @@ describe("useAgentLogs", () => {
2929

3030
// Send 3 logs
3131
server.send(JSON.stringify(generateLogs(3)));
32-
expect(result.current).toHaveLength(3);
32+
await waitFor(() => {
33+
expect(result.current).toHaveLength(3);
34+
});
3335

3436
// Disable the hook
3537
rerender({ enabled: false });
36-
expect(result.current).toHaveLength(0);
38+
await waitFor(() => {
39+
expect(result.current).toHaveLength(0);
40+
});
3741

3842
// Enable the hook again
3943
rerender({ enabled: true });
4044
await server.connected;
4145
server.send(JSON.stringify(generateLogs(3)));
42-
expect(result.current).toHaveLength(3);
46+
await waitFor(() => {
47+
expect(result.current).toHaveLength(3);
48+
});
4349
});
4450
});
4551

0 commit comments

Comments
 (0)