From 7f47f8f8d2668fcdeb82bc62c399540bd4624246 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Iv=C3=A1n=20G=C3=B3mez=20Pinta?= <44321109+GomezIvann@users.noreply.github.com> Date: Thu, 21 Mar 2024 16:59:09 +0100 Subject: [PATCH] Breadcrumbs updates based on feedback --- lib/src/breadcrumbs/Breadcrumbs.stories.tsx | 6 ++---- lib/src/breadcrumbs/Breadcrumbs.test.tsx | 8 ++++---- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/lib/src/breadcrumbs/Breadcrumbs.stories.tsx b/lib/src/breadcrumbs/Breadcrumbs.stories.tsx index 32f74a5a0..b4d467361 100644 --- a/lib/src/breadcrumbs/Breadcrumbs.stories.tsx +++ b/lib/src/breadcrumbs/Breadcrumbs.stories.tsx @@ -178,10 +178,8 @@ const Breadcrumbs = () => ( - <ExampleContainer> - <DxcContainer height="200px"> - <DxcBreadcrumbs items={items} /> - </DxcContainer> + <ExampleContainer expanded> + <DxcBreadcrumbs items={items} /> </ExampleContainer> </> ); diff --git a/lib/src/breadcrumbs/Breadcrumbs.test.tsx b/lib/src/breadcrumbs/Breadcrumbs.test.tsx index 6474c931a..199a62069 100644 --- a/lib/src/breadcrumbs/Breadcrumbs.test.tsx +++ b/lib/src/breadcrumbs/Breadcrumbs.test.tsx @@ -69,13 +69,13 @@ describe("Breadcrumbs component tests", () => { <DxcBreadcrumbs onItemClick={onItemClick} items={[ - { label: "Home", href: "/" }, - { label: "Preferences", href: "/" }, + { label: "Home", href: "/home" }, + { label: "Preferences", href: "/preferences" }, ]} /> ); userEvent.click(getByText("Home")); - expect(onItemClick).toHaveBeenCalled(); + expect(onItemClick).toHaveBeenCalledWith("/home"); }); test("The onClick prop from an item is properly called (collapsed)", async () => { const onItemClick = jest.fn(); @@ -92,6 +92,6 @@ describe("Breadcrumbs component tests", () => { ); await userEvent.click(getByRole("button")); await userEvent.click(getByText("Preferences")); - expect(onItemClick).toHaveBeenCalled(); + expect(onItemClick).toHaveBeenCalledWith("/"); }); });