test(e2e): match the coach conversations handoff query string

The drawer's Conversations button now hands off to
/coach?view=conversations so the full page opens the conversation list on
arrival; widen the navigation assertion to accept the query string.
This commit is contained in:
Marc-André Bombeck committed 2026-06-21 14:35:59 +02:00
1 parent 41dab24985
commit 67ead71e24
1 file changed
+5 -4
+5 -4
View File
@@ -280,8 +280,8 @@ test.describe("Coach launch surfaces on insights sub-pages", () => {
page,
}, testInfo) => {
// The in-panel history tray is gone from the drawer; the button
// navigates to `/coach` where the conversation list
// renders inline on lg+.
// navigates to `/coach?view=conversations`, where the full page opens
// the conversation list on arrival instead of a blank pane.
test.skip(
testInfo.project.name !== "chromium-desktop",
"exercises the desktop drawer layout",
@@ -312,8 +312,9 @@ test.describe("Coach launch surfaces on insights sub-pages", () => {
await expect(historyButton).toBeVisible({ timeout: 10_000 });
await historyButton.click();
// No in-panel tray opens — the click navigates to the full view.
await page.waitForURL("**/coach", { timeout: 10_000 });
// No in-panel tray opens — the click navigates to the full view, which
// carries the `view=conversations` handoff so the list opens on arrival.
await page.waitForURL(/\/coach(\?|$)/, { timeout: 10_000 });
await expect(page.locator('[data-slot="coach-page"]')).toBeVisible({
timeout: 10_000,
});