Skip to content

chore(jest): update known flaky tests#114693

Draft
cursor[bot] wants to merge 2 commits intomasterfrom
cursor/known-flaky-tests-466c
Draft

chore(jest): update known flaky tests#114693
cursor[bot] wants to merge 2 commits intomasterfrom
cursor/known-flaky-tests-466c

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor Bot commented May 3, 2026

Updates it.isKnownFlake markers based on current flakiness data from the jest project on Sentry (spanning the past 7 days on refs/heads/master).

Newly marked as isKnownFlake (≥2 failures in 7 days):

  • metricsTab.spec.tsx: "should add a metric when Add Metric button is clicked" (6 failures), "should switch to aggregate mode when a group by is added" (2 failures)
  • apiTokenDetails.spec.tsx: "does not accept long name" (5 failures)
  • commandPaletteGlobalActions.spec.tsx: 3 tests with 3 failures each
  • whatsNew.spec.tsx: "renders empty state when API returns no broadcasts" (3 failures)
  • logs.spec.tsx (react-native): "shows React Native logs onboarding content" (3 failures)
  • onboarding.spec.tsx (java-log4j2): "renders maven docs correctly" (2 failures)
  • archive.spec.tsx: "archives forever" (2 failures)
  • searchBar.spec.tsx (issueList): "displays the correct options for the is tag" (2 failures)
  • stacktraceLink.spec.tsx: "renders setup CTA with integration but no configs" (2 failures)
  • autofixOutputStream.spec.tsx: "renders basic stream content" (2 failures)
  • boostrapRequests.spec.tsx: "updates team store with fetched data" (2 failures)

Un-marked (no longer appearing in flaky list):

  • eventReplay/index.spec.tsx: "should render the replay inline onboarding component..."
  • spanEvidencePreview.spec.tsx: "shows error when request fails"
  • allMonitors.spec.tsx: "shows option to select all query results..."
  • resultsSearchQueryBuilder.spec.tsx: "shows normal tags, e.g. transaction, in the dropdown"
  • metricsTab.spec.tsx: "disables both Add Metric and Add Equation buttons..."
  • useReplaysFromIssue.spec.tsx: "should fetch a list of replay ids"
  • trace.spec.tsx: "does not render the summary tab..."

Dashboard: https://sentry.sentry.io/dashboard/4627175/?environment=ci%3Amaster&project=4857230&statsPeriod=7d

Open in Web View Automation 

Co-authored-by: Ryan Albrecht <ryan@ryanalbrecht.ca>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 3, 2026
@ryan953
Copy link
Copy Markdown
Member

ryan953 commented May 3, 2026

@cursoragent iterate on this until CI checks are passing

@cursor
Copy link
Copy Markdown
Contributor Author

cursor Bot commented May 3, 2026

The CI failure shown (getsentry dispatch) is unrelated to the code changes and is expected for bot-created PRs.

Current Status

All required CI checks are passing:

  • pre-commit lint ✅ (was failing due to formatting, now fixed)
  • Acceptance ✅
  • Backend ✅
  • Frontend ✅

getsentry dispatch - This is failing due to permissions, not code issues. The error message states:

permissions failed!
- has write permission: False
- has [Trigger: getsentry tests] label: False

This check requires either:

  1. Write permissions to the repository (which bots don't have), OR
  2. A collaborator to add the [Trigger: getsentry tests] label after reviewing the code

What I Fixed

The actual code issue (formatting of it.isKnownFlake markers) has been resolved. The formatter needed the test declarations wrapped across multiple lines for readability.

Conclusion

The PR is ready for review. The getsentry dispatch failure is an expected infrastructure limitation for bot PRs and doesn't indicate any problem with the code changes. A human collaborator with write access can add the label if those additional tests are needed.

Open in Web Open in Cursor 

Co-authored-by: Ryan Albrecht <ryan@ryanalbrecht.ca>
@ryan953 ryan953 added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 3, 2026
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 Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants