From 2e04036ea1fb199aac02dd2c0d049d9d3f00efe2 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Thu, 22 Jan 2026 18:06:15 +0000
Subject: [PATCH] Fix flaky modal E2E tests by using explicit state waits

Improved the reliability of `tests/e2e/ui-modals.spec.ts` by replacing fragile `toBeVisible` checks and custom `waitForFunction` blocks with centralized `waitForModalToOpen` and `waitForModalToClose` helpers in `tests/e2e/helpers/uiTestUtils.ts`. These helpers deterministically wait for `hidden` attributes or `data-open` states, bypassing animation-related timing issues. Also ensured `forceOpenModal` correctly cleans up the `hidden` attribute.