Skip to content

fix(functional-tests): harden tests against sign-out race condition#20058

Merged
vbudhram merged 1 commit intomainfrom
fix-signout-test-race-condition-v2
Feb 13, 2026
Merged

fix(functional-tests): harden tests against sign-out race condition#20058
vbudhram merged 1 commit intomainfrom
fix-signout-test-race-condition-v2

Conversation

@vbudhram
Copy link
Contributor

Because

  • There is a race condition on the sign out, while we investigate the fix, we can hard the tests a bit

This pull request

  • Uses waitForEvent to make sure the page is settle before checking for headers
  • Uses playwright expect that is a little more robust since it keeps checking on page.goto

Issue that this pull request solves

Closes: (issue number)

Checklist

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Screenshot 2026-02-13 at 9 18 24 AM

The last failure is because I'm rate limited from Fastly.

… destruction

- signOut(): wait for actual 'load' event from window.location.assign
  instead of URL assertions fooled by intermediate SPA redirects
- fillOutEmailFirstForm(): wait for email textbox visibility after
  hard navigation
- checkWebChannelMessage(): use expect().toPass() retry pattern to
  survive execution context destruction from client-side redirects
@vbudhram vbudhram marked this pull request as ready for review February 13, 2026 14:19
@vbudhram vbudhram requested a review from a team as a code owner February 13, 2026 14:19
@vbudhram vbudhram merged commit b4b12b0 into main Feb 13, 2026
21 checks passed
@vbudhram vbudhram deleted the fix-signout-test-race-condition-v2 branch February 13, 2026 15:04
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.

2 participants