(playwright): fix flaky online users & Teams spec#28899
Conversation
❌ PR checklist incompleteThis PR cannot be merged until the following are addressed on its linked issue:
The fields live on the linked issue in the Shipping project (open the issue → right sidebar → Projects). After you set them, re-run this check (or push a commit) — issue/project changes do not re-trigger it automatically. Maintainers can bypass this check by adding the |
|
🟡 Playwright Results — all passed (13 flaky)✅ 4271 passed · ❌ 0 failed · 🟡 13 flaky · ⏭️ 88 skipped
🟡 13 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
Code Review ✅ ApprovedIncreases timeout for the online users spec and implements strict API URL matching for the teams spec to resolve intermittent test flakiness. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|
Changes have been cherry-picked to the 1.13 branch. |
|
Failed to cherry-pick changes to the 1.12.11 branch. |
* (playwright): fix flaky online users spec * fix flaky teams spec (cherry picked from commit 1bf2c9a)



Describe your changes:
Type of change:
High-level design:
N/A — small change.
Tests:
Use cases covered
Unit tests
Backend integration tests
Ingestion integration tests
Playwright (UI) tests
Manual testing performed
UI screen recording / screenshots:
Not applicable.
Checklist:
Fixes <issue-number>: <short explanation>Fixes #<issue-number>above.Summary by Gitar
test.slow()to theOnlineUsers.spec.tstest to accommodate multiple logins and navigation steps.Teams.spec.tsto use explicitPATCHandGETrequest interceptors instead of a generic wildcard listener to ensure reliable synchronization.This will update automatically on new commits.