Freegle / Iznik / 5575

72%

master: 70%

Build Type

Pull #232

circleci

Committed by edwh

Commit Message

fix(test): add Promise.race hydration wait before logout visibility check

Under parallel CI load the navbar may not have hydrated by the time
logoutIfLoggedIn runs its synchronous isVisible() checks.  When both
return false the function falls through to gotoAndVerify('/') which can
stall for the full 202.5s timeout; the catch block repeats it, burning
most of the 10-minute test budget — symptom: test-reply-flow-existing-user
and test-reply-flow-social timing out at 20.0m on CI builds 5523/5535.

Fix: race a short waitFor({ state: 'visible', timeout: 10000 }) on either
logout selector before the visibility checks.  Matches the same fix
already on master (commit 850fda671, merged via PR #226).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #232: test(coverage): unit tests for utils package — Flex types, Blur, TidyName, Haversine

Coverage Stats

8880 of 10261 branches covered (86.54%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

318 existing lines in 5 files now uncovered.

84588 of 127350 relevant lines covered (66.42%)

19.0 hits per line