Function clickToReveal

e2e/src/surfaces/browser.ts:143–159

(
  trigger: Locator,
  revealed: Locator,
  { attempts = 5, revealTimeoutMs = 4_000 }: { attempts?: number; revealTimeoutMs?: number } = {},
)

 * surfaces as the ordinary locator error.
 */
export const clickToReveal = async (
  trigger: Locator,
  revealed: Locator,
  { attempts = 5, revealTimeoutMs = 4_000 }: { attempts?: number; revealTimeoutMs?: number } = {},
): Promise<void> => {
  for (let attempt = 1; attempt < attempts; attempt++) {
    await trigger.click();
    const shown = await revealed
      .waitFor({ timeout: revealTimeoutMs })
      .then(() => true)
      // oxlint-disable-next-line executor/no-promise-catch -- retry boundary: a missed reveal is the signal to click again, not a failure
      .catch(() => false);
    if (shown) return;
  }
  await trigger.click();
  await revealed.waitFor({ timeout: revealTimeoutMs });
};

// acquireUseRelease so a vitest timeout (fiber interruption) still closes the
// browser and flushes video + trace — a bare promise would leak Chromium.