Skip to content

test(desktop): survive early menu dismissals in the agent-voice spec - #6530

Open
Maxwellimus wants to merge 1 commit into
mainfrom
fix/huddle-voice-menu-open-race
Open

test(desktop): survive early menu dismissals in the agent-voice spec#6530
Maxwellimus wants to merge 1 commit into
mainfrom
fix/huddle-voice-menu-open-race

Conversation

@Maxwellimus

Copy link
Copy Markdown
Contributor

huddle-transcription.spec.tsassigns distinct agent voices… fails deterministically when run in a fresh worker (--repeat-each solo runs: 0/6 pass on pristine main): shortly after a cold load, background huddle-state settling re-renders the participant tiles and dismisses the just-opened Radix voice menu. Every CI retry runs in a fresh worker, so a first-attempt failure can never recover — this spec has been failing whole Smoke shards across unrelated PRs (e.g. #6455 twice today).

Fix: retry the open-and-read unit (open menu → title visible → TTS toggle checked) with expect().toPass() until the menu holds through it. After hardening: 10/10 sequential solo runs pass (0/6 before), full file 26/26.

Product bug worth a follow-up look (out of scope here): an open agent-voice popover can be dismissed by background state settling the user never initiated — trace shows the menu opening and closing with no user interaction. Repro: open the voice menu immediately after the huddle window cold-loads.

Shortly after a cold load, background huddle-state settling re-renders
the participant tiles and dismisses a just-opened Radix popover. In a
fresh worker a single click then fails deterministically — and every CI
retry runs in a fresh worker, so a first-attempt failure of this spec
could never recover. Retry the open-and-read unit (open, title, toggle)
until the menu holds through it.

The underlying dismissal is a product bug worth its own look: an open
agent-voice menu can be closed by background state settling the user
never initiated.

Signed-off-by: Max Lampert <maxwell@squareup.com>
@Maxwellimus
Maxwellimus marked this pull request as ready for review August 21, 2026 23:40
@Maxwellimus
Maxwellimus requested a review from a team as a code owner August 21, 2026 23:40
@Maxwellimus
Maxwellimus requested a review from wesbillman August 21, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant