fix test: use never-resolving promise and fake timers for withTimeout

The previous mock used mockRejectedValue which immediately rejects,
so Promise.race resolved before withTimeout's setTimeout fired.
Now we use new Promise(() => {}) to simulate a hanging request
and advance timers to properly exercise the 2s timeout logic.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
This commit is contained in:
privilegedescalation-engineer
2026-03-25 07:41:47 +00:00
parent 66932958b1
commit 3aa9c15e80
+11 -13
View File
@@ -155,25 +155,23 @@ describe('IntelGpuDataProvider', () => {
it('treats a hanging CRD request as unavailable after 2s timeout', async () => {
vi.useFakeTimers();
const nodeWrapper = { jsonData: {} };
vi.mocked(K8s.ResourceClasses.Node.useList).mockReturnValue([
[nodeWrapper],
null,
] as any);
vi.mocked(K8s.ResourceClasses.Pod.useList).mockReturnValue([
[nodeWrapper],
null,
] as any);
vi.mocked(ApiProxy.request).mockRejectedValue(
new Error('Request timed out after 2000ms')
);
vi.mocked(K8s.ResourceClasses.Node.useList).mockReturnValue([[nodeWrapper], null] as any);
vi.mocked(K8s.ResourceClasses.Pod.useList).mockReturnValue([[nodeWrapper], null] as any);
vi.mocked(ApiProxy.request).mockReturnValue(new Promise(() => {}));
const { result } = renderHook(() => useIntelGpuContext(), { wrapper: Wrapper });
await act(async () => {
await vi.advanceTimersByTimeAsync(100);
await vi.advanceTimersByTimeAsync(1999);
});
expect(result.current.loading).toBe(false);
expect(result.current.loading).toBe(true);
await act(async () => {
await vi.advanceTimersByTimeAsync(200);
});
await waitFor(() => expect(result.current.loading).toBe(false));
expect(result.current.crdAvailable).toBe(false);
vi.useRealTimers();
});
});