Skip to content

fix(metrics): Resolve flaky metrics tab tests#116280

Merged
nsdeschenes merged 2 commits into
masterfrom
nd/chore-metrics-resolve-flaky-metrics-tab-test
May 27, 2026
Merged

fix(metrics): Resolve flaky metrics tab tests#116280
nsdeschenes merged 2 commits into
masterfrom
nd/chore-metrics-resolve-flaky-metrics-tab-test

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

Replace synchronous DOM queries with async findBy* queries to properly wait for React state updates after user interactions.

Replace synchronous DOM queries with async findBy queries to properly
wait for React state updates after user interactions. Remove the
isKnownFlake marker now that the race conditions are fixed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.58%

Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5a98d48. Configure here.

Comment thread static/app/views/explore/metrics/metricsTab.spec.tsx Outdated
@nsdeschenes nsdeschenes marked this pull request as ready for review May 27, 2026 11:02
@nsdeschenes nsdeschenes requested a review from a team as a code owner May 27, 2026 11:02
Co-authored-by: Nick <nsdeschenes@users.noreply.github.com>
@nsdeschenes nsdeschenes merged commit 31cb4e0 into master May 27, 2026
75 checks passed
@nsdeschenes nsdeschenes deleted the nd/chore-metrics-resolve-flaky-metrics-tab-test branch May 27, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants