From eac20f5eec4df54fa25f38446d787b2c65b30428 Mon Sep 17 00:00:00 2001 From: sheehanmunim Date: Wed, 19 Aug 2026 16:43:58 -0400 Subject: [PATCH] test(web): assert the app-icon sync in the AppRoot tree `AppRoot` gained an `` beside the voice provider when the app icon chooser landed, but the structural test still expected a single child of the atom registry. Model: Claude Opus 5, harness: Claude Code --- apps/web/src/AppRoot.test.tsx | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/apps/web/src/AppRoot.test.tsx b/apps/web/src/AppRoot.test.tsx index 10c5364458b6..6ae9de08cca0 100644 --- a/apps/web/src/AppRoot.test.tsx +++ b/apps/web/src/AppRoot.test.tsx @@ -19,14 +19,16 @@ describe("AppRoot", () => { const providerChildren = Children.toArray( (root as ReactElement<{ readonly children: ReactNode }>).props.children, ); - expect(providerChildren).toHaveLength(1); - expect(isValidElement(providerChildren[0]) && providerChildren[0].type).toBe( + // The icon sync sits beside the voice provider: it needs the registry (the + // choice is a server setting) but owns no subtree of its own. + expect(providerChildren).toHaveLength(2); + expect(isValidElement(providerChildren[1]) && providerChildren[1].type).toBe( VoiceSessionProvider, ); const children = Children.toArray( ( - providerChildren[0] as ReactElement<{ + providerChildren[1] as ReactElement<{ readonly children: ReactNode; }> ).props.children,