Skip to content

Conversation

@andrross
Copy link
Member

This reverts commit 4b9aa27 from #18550

This newly introduced test is very flaky. See #19354

Check List

  • Functionality includes testing.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@andrross andrross requested a review from a team as a code owner September 25, 2025 19:36
@github-actions
Copy link
Contributor

✅ Gradle check result for e834f66: SUCCESS

@codecov
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.90%. Comparing base (7abef8c) to head (e834f66).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #19423      +/-   ##
============================================
- Coverage     72.93%   72.90%   -0.03%     
+ Complexity    69947    69939       -8     
============================================
  Files          5676     5676              
  Lines        321121   321121              
  Branches      46427    46427              
============================================
- Hits         234195   234119      -76     
- Misses        68032    68118      +86     
+ Partials      18894    18884      -10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@jainankitk jainankitk left a comment

Choose a reason for hiding this comment

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

Can we merge #19425 instead to skip the tests? In general, is there guidance around when to revert test vs when to skip test?

@jainankitk
Copy link
Contributor

Closing in favor of #19425!

@jainankitk jainankitk closed this Sep 26, 2025
@andrross andrross deleted the revert-wlm-integ branch November 4, 2025 02:16
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