From 08e6f045ae3f8a34b68150db568cb3fa23e24b4b Mon Sep 17 00:00:00 2001 From: le king fu Date: Sun, 18 Oct 2026 21:07:44 -0400 Subject: [PATCH] test: read the banner's dots once in the presence test (#446) Self-review tidy-up: the presence test looked the dots up twice; it now keeps the one list it asserts on. No assertion changed. Resolves #446 Co-Authored-By: Claude Opus 5.5 --- src/components/onboarding/OnboardingBanner.render.test.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/components/onboarding/OnboardingBanner.render.test.tsx b/src/components/onboarding/OnboardingBanner.render.test.tsx index a369166..b4ad18f 100644 --- a/src/components/onboarding/OnboardingBanner.render.test.tsx +++ b/src/components/onboarding/OnboardingBanner.render.test.tsx @@ -145,9 +145,9 @@ describe("OnboardingBanner — banner mode (#397)", () => { it("once the total is known, a decorative dot separates the counter from the next step (#446)", () => { const props = bannerProps(); - const [dot] = dots(props); - expect(dots(props)).toHaveLength(1); - expect(dot.props["aria-hidden"]).toBe("true"); + const found = dots(props); + expect(found).toHaveLength(1); + expect(found[0].props["aria-hidden"]).toBe("true"); const html = markup(props); const counter = html.indexOf("t(onboarding.banner.progress"); const dotAt = html.search(/