From 3ba3e2beb990880fe7953d2cb0f1b9618b501355 Mon Sep 17 00:00:00 2001 From: "Molecule AI Dev Engineer A (Kimi)" Date: Sat, 13 Jun 2026 22:39:48 +0000 Subject: [PATCH] test(terms): remove stale skipped TermsGate test (#2761) The skipped test at TermsGate.test.tsx:184 ("disables the button while submitting") was kept pending a fake-timer refactor, but the same behavior is now covered by the active deferred-POST tests in the "I agree button accessibility" describe block: - shows ellipsis on the I agree button while POST is in flight - has aria-disabled while submitting Removing the stale skip reduces skipped-test noise and makes future skipped-test audits more discriminating. Fixes #2761 Co-Authored-By: Claude --- canvas/src/components/__tests__/TermsGate.test.tsx | 7 ------- 1 file changed, 7 deletions(-) diff --git a/canvas/src/components/__tests__/TermsGate.test.tsx b/canvas/src/components/__tests__/TermsGate.test.tsx index 1e1c28ec1..d23fa94e1 100644 --- a/canvas/src/components/__tests__/TermsGate.test.tsx +++ b/canvas/src/components/__tests__/TermsGate.test.tsx @@ -180,13 +180,6 @@ describe("TermsGate — accept flow", () => { // Dialog is still open expect(screen.getByRole("dialog")).toBeTruthy(); }); - - it.skip("disables the button while submitting (requires fake-timers around fireEvent.click)", async () => { - // This test requires vi.useFakeTimers() + act(() => { fireEvent.click(btn); vi.runAllTimers(); }) - // to synchronously advance through the async boundary between click and fetch initiation. - // The current test structure fires the fetch before click, so this is skipped pending - // a refactor of the component to not initiate fetch synchronously on user gesture. - }); }); describe("TermsGate — I agree button accessibility", () => { -- 2.52.0