Skip to content

Conversation

@varunbharadwaj
Copy link
Contributor

@varunbharadwaj varunbharadwaj commented Nov 22, 2025

Description

Fix flaky DefaultStreamPollerTests.testConsumerReinitializationWithNoInitialMessages test. Observed the following error: Unfinished stubbing detected here
Initialize the mocks before starting the poller to avoid race conditions.

Related Issues

Resolves #20069

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.

Signed-off-by: Varun Bharadwaj <[email protected]>
@github-actions github-actions bot added >test-failure Test failure from CI, local build, etc. autocut flaky-test Random test failure that succeeds on second run labels Nov 22, 2025
@varunbharadwaj varunbharadwaj marked this pull request as ready for review November 22, 2025 04:15
@varunbharadwaj varunbharadwaj requested a review from a team as a code owner November 22, 2025 04:15
@varunbharadwaj varunbharadwaj changed the title Fix flaky DefaultStreamPollerTests test Fix flaky DefaultStreamPoller test Nov 22, 2025
@github-actions
Copy link
Contributor

❌ Gradle check result for a4510ec: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for a4510ec: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for a4510ec: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

✅ Gradle check result for a4510ec: SUCCESS

@codecov
Copy link

codecov bot commented Nov 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.24%. Comparing base (3ceeffa) to head (a4510ec).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #20075      +/-   ##
============================================
- Coverage     73.29%   73.24%   -0.05%     
+ Complexity    71656    71650       -6     
============================================
  Files          5790     5790              
  Lines        327549   327549              
  Branches      47181    47181              
============================================
- Hits         240061   239928     -133     
- Misses        68261    68419     +158     
+ Partials      19227    19202      -25     

☔ 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.

@karenyrx karenyrx merged commit 97d3864 into opensearch-project:main Nov 22, 2025
44 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autocut flaky-test Random test failure that succeeds on second run skip-changelog >test-failure Test failure from CI, local build, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AUTOCUT] Gradle Check Flaky Test Report for DefaultStreamPollerTests

2 participants