From aa6623731e16b77a133df55f5504ffe0df482eb2 Mon Sep 17 00:00:00 2001 From: openhands Date: Thu, 19 Mar 2026 17:58:51 +0000 Subject: [PATCH] fix: update tests for information-request and intermediate pages - Fix use-is-on-intermediate-page test to check correct paths (/onboarding/new-account-form and /onboarding/information-request) - Fix information-request test to use correct i18n key (ENTERPRISE$SELF_HOSTED_CARD_DESCRIPTION) - Mock useTracking hook to avoid QueryClient dependency Co-authored-by: openhands --- .../hooks/use-is-on-intermediate-page.test.ts | 10 ++++++++-- frontend/__tests__/routes/information-request.test.tsx | 9 ++++++++- 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/frontend/__tests__/hooks/use-is-on-intermediate-page.test.ts b/frontend/__tests__/hooks/use-is-on-intermediate-page.test.ts index 2879815745..2c67f04b0a 100644 --- a/frontend/__tests__/hooks/use-is-on-intermediate-page.test.ts +++ b/frontend/__tests__/hooks/use-is-on-intermediate-page.test.ts @@ -27,8 +27,14 @@ describe("useIsOnIntermediatePage", () => { expect(result.current).toBe(true); }); - it("should return true when on /onboarding page", () => { - useLocationMock.mockReturnValue({ pathname: "/onboarding" }); + it("should return true when on /onboarding/new-account-form page", () => { + useLocationMock.mockReturnValue({ pathname: "/onboarding/new-account-form" }); + const { result } = renderHook(() => useIsOnIntermediatePage()); + expect(result.current).toBe(true); + }); + + it("should return true when on /onboarding/information-request page", () => { + useLocationMock.mockReturnValue({ pathname: "/onboarding/information-request" }); const { result } = renderHook(() => useIsOnIntermediatePage()); expect(result.current).toBe(true); }); diff --git a/frontend/__tests__/routes/information-request.test.tsx b/frontend/__tests__/routes/information-request.test.tsx index 47938fea14..934b6c3e2f 100644 --- a/frontend/__tests__/routes/information-request.test.tsx +++ b/frontend/__tests__/routes/information-request.test.tsx @@ -14,6 +14,13 @@ vi.mock("react-router", async () => { }; }); +// Mock useTracking to avoid QueryClient dependency +vi.mock("#/hooks/use-tracking", () => ({ + useTracking: () => ({ + trackEnterpriseLeadFormSubmitted: vi.fn(), + }), +})); + describe("InformationRequest", () => { beforeEach(() => { vi.clearAllMocks(); @@ -64,7 +71,7 @@ describe("InformationRequest", () => { renderWithRouter(); expect(screen.getByText("ENTERPRISE$SELF_HOSTED_TITLE")).toBeInTheDocument(); - expect(screen.getByText("ENTERPRISE$SELF_HOSTED_DESCRIPTION")).toBeInTheDocument(); + expect(screen.getByText("ENTERPRISE$SELF_HOSTED_CARD_DESCRIPTION")).toBeInTheDocument(); }); it("should render SaaS features", () => {