From f21ac590400120415f6753c2bc5033ff7d5ba37e Mon Sep 17 00:00:00 2001 From: Groom Book CTO Date: Thu, 19 Mar 2026 03:16:46 +0000 Subject: [PATCH] Fix test selectors for branded nav text - Use regex /Groom\s*Book/ to match split-element brand text - Use getByRole("link") for Book CTA to avoid matching brand Co-Authored-By: Paperclip --- apps/web/src/__tests__/App.test.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/apps/web/src/__tests__/App.test.tsx b/apps/web/src/__tests__/App.test.tsx index cfd422d..47b9656 100644 --- a/apps/web/src/__tests__/App.test.tsx +++ b/apps/web/src/__tests__/App.test.tsx @@ -23,12 +23,12 @@ function renderApp(route = "/admin") { describe("App navigation", () => { it("renders the Groom Book brand", () => { const nav = renderApp(); - expect(within(nav).getByText("Groom Book")).toBeInTheDocument(); + expect(within(nav).getByText(/Groom\s*Book/)).toBeInTheDocument(); }); it("renders the Book CTA button", () => { const nav = renderApp(); - expect(within(nav).getByText("Book")).toBeInTheDocument(); + expect(within(nav).getByRole("link", { name: "Book" })).toBeInTheDocument(); }); it("renders all primary nav links", () => { @@ -61,6 +61,6 @@ describe("App navigation", () => { ); // Customer portal should render at root - no admin nav present - expect(screen.queryByText("Groom Book")).not.toBeInTheDocument(); + expect(screen.queryByText(/Groom\s*Book/)).not.toBeInTheDocument(); }); });