Skip to content

4.x: Remove Pioneer-RetryTest as it is itself flaky?!#8203

Merged
akarnokd merged 1 commit into
ReactiveX:4.xfrom
akarnokd:4xPioneerRetryIsShit
Jun 30, 2026
Merged

4.x: Remove Pioneer-RetryTest as it is itself flaky?!#8203
akarnokd merged 1 commit into
ReactiveX:4.xfrom
akarnokd:4xPioneerRetryIsShit

Conversation

@akarnokd

Copy link
Copy Markdown
Member

LMAO the retry infrastructure is flaky itself.

2026-06-30T09:06:58.7103707Z XFlatMapTest > singlePublisher() FAILED
2026-06-30T09:06:58.7105028Z org.junit.platform.commons.PreconditionViolationException: Illegal state: required test method is not present in the current ExtensionContext
2026-06-30T09:06:58.7106662Z at app//org.junitpioneer.jupiter.RetryingTestExtension.retrierFor(RetryingTestExtension.java:68)
2026-06-30T09:06:58.7108579Z at app//org.junitpioneer.jupiter.RetryingTestExtension.handleTestExecutionException(RetryingTestExtension.java:64)
2026-06-30T09:06:58.7109967Z at java.base@27-beta/java.util.ArrayList.forEach(ArrayList.java:1612)
2026-06-30T09:06:58.7110913Z at java.base@27-beta/java.util.ArrayList.forEach(ArrayList.java:1612)
2026-06-30T09:06:58.7111515Z

https://github.com/ReactiveX/RxJava/actions/runs/28432952293/job/84251845327

@github-actions

Copy link
Copy Markdown

🐷 TruffleHog + Entropy Beauty Scan

Average entropy of changed code: 4.678 bits/char
Verdict: ✅ Mid-4 beauty detected (thoughtful human code!)

Changed files entropy:

src/test/java/io/reactivex/rxjava4/internal/schedulers/SharedSchedulerTest.java: 4.678

✅ No secrets or suspicious high-entropy strings found.

Mid-4 beauty heuristic in action — powered by our entropy chats! 😊

@akarnokd akarnokd merged commit c357f75 into ReactiveX:4.x Jun 30, 2026
4 checks passed
@akarnokd akarnokd deleted the 4xPioneerRetryIsShit branch June 30, 2026 10:01
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.34%. Comparing base (cba142c) to head (b1942e1).
⚠️ Report is 2 commits behind head on 4.x.

Additional details and impacted files
@@            Coverage Diff            @@
##                4.x    #8203   +/-   ##
=========================================
  Coverage     98.34%   98.34%           
- Complexity     6783     6784    +1     
=========================================
  Files           777      777           
  Lines         48506    48506           
  Branches       6570     6570           
=========================================
+ Hits          47701    47705    +4     
- Misses          605      607    +2     
+ Partials        200      194    -6     

☔ View full report in Codecov by Harness.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant