From c9cb5f2f01aceac083d93d56038a028bb5d01bf2 Mon Sep 17 00:00:00 2001 From: Scott Lamb Date: Thu, 18 Feb 2021 09:10:43 -0800 Subject: [PATCH] disable hanging test for now --- ui/src/Login.test.tsx | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/ui/src/Login.test.tsx b/ui/src/Login.test.tsx index 5301633..a4acdad 100644 --- a/ui/src/Login.test.tsx +++ b/ui/src/Login.test.tsx @@ -48,7 +48,12 @@ test("success", async () => { await waitFor(() => expect(onSuccess).toHaveBeenCalledTimes(1)); }); -test("close while pending", async () => { +// TODO: fix and re-enable this test. +// Currently it makes "CI=true npm run test" hang. +// I think the problem is that npmjs doesn't really support aborting requests, +// so the delay("infinite") request just sticks around, even though the fetch +// has been aborted. Maybe https://github.com/mswjs/msw/pull/585 will fix it. +xtest("close while pending", async () => { const handleClose = jest.fn(); const onSuccess = jest.fn(); const { rerender } = renderWithCtx(