Skip to content

4.x: Unit test lambdaification 20 of N#8162

Merged
akarnokd merged 2 commits into
ReactiveX:4.xfrom
akarnokd:4xUnitTestLambdas20
Jun 22, 2026
Merged

4.x: Unit test lambdaification 20 of N#8162
akarnokd merged 2 commits into
ReactiveX:4.xfrom
akarnokd:4xUnitTestLambdas20

Conversation

@akarnokd

Copy link
Copy Markdown
Member

Will go over the unit tests in ASCII order of the classpath and classes.

IntelliJ: Inspect -> RedundantCast, Inspect -> Anonymous type can be replaced by lambda
Search regexp: new\s+\w+(?:\s*<(?:[\s\w<>(\[\])?,.?]|\s*<[\s\w<>(\[\])?,.?]*>)*>)?\s*\([^)]*\)\s*\{

/* NFI */ = Not Functional Interface so break the regexp pattern

If you complain instead of proposing a PR about "why not convert to method references" you'll be banned.

Related: #8080

@akarnokd akarnokd added this to the 4.0 milestone Jun 22, 2026
@github-actions

Copy link
Copy Markdown

🐷 TruffleHog + Entropy Beauty Scan

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

Changed files entropy:

src/jmh/java/io/reactivex/rxjava4/core/MemoryPerf.java: 4.972
src/jmh/java/io/reactivex/rxjava4/core/OperatorFlatMapPerf.java: 4.926
src/jmh/java/io/reactivex/rxjava4/core/OperatorMergePerf.java: 4.747
src/jmh/java/io/reactivex/rxjava4/core/RxVsStreamPerf.java: 5.025
src/jmh/java/io/reactivex/rxjava4/core/XMapYPerf.java: 5.001
src/jmh/java/io/reactivex/rxjava4/parallel/ParallelPerf.java: 5.028
src/test/java/io/reactivex/rxjava4/internal/schedulers/ParallelSchedulerTest.java: 4.692
src/test/java/io/reactivex/rxjava4/internal/schedulers/ScheduledDirectPeriodicTaskTest.java: 4.795
src/test/java/io/reactivex/rxjava4/internal/schedulers/ScheduledRunnableTest.java: 4.760
src/test/java/io/reactivex/rxjava4/internal/schedulers/SchedulerMultiWorkerSupportTest.java: 4.554
src/test/java/io/reactivex/rxjava4/internal/schedulers/SchedulerPoolFactoryTest.java: 4.917
src/test/java/io/reactivex/rxjava4/internal/schedulers/SharedSchedulerTest.java: 4.683
src/test/java/io/reactivex/rxjava4/internal/schedulers/TrampolineSchedulerInternalTest.java: 4.680
src/test/java/io/reactivex/rxjava4/internal/subscribers/QueueDrainSubscriberTest.java: 4.564
src/test/java/io/reactivex/rxjava4/internal/subscribers/SubscriberResourceWrapperTest.java: 4.809
src/test/java/io/reactivex/rxjava4/internal/subscriptions/SubscriptionHelperTest.java: 4.708
src/test/java/io/reactivex/rxjava4/internal/util/MiscUtilTest.java: 4.759
src/test/java/io/reactivex/rxjava4/internal/util/QueueDrainHelperTest.java: 4.175
src/test/java/io/reactivex/rxjava4/observers/SerializedObserverTest.java: 4.433
src/test/java/io/reactivex/rxjava4/observers/TestObserverTest.java: 4.576

✅ No secrets or suspicious high-entropy strings found.

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

@akarnokd akarnokd merged commit 4034581 into ReactiveX:4.x Jun 22, 2026
4 checks passed
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.75%. Comparing base (835d093) to head (e4050c6).

Additional details and impacted files
@@             Coverage Diff              @@
##                4.x    #8162      +/-   ##
============================================
- Coverage     98.75%   98.75%   -0.01%     
  Complexity     6941     6941              
============================================
  Files           773      773              
  Lines         48994    48994              
  Branches       6586     6586              
============================================
- Hits          48386    48383       -3     
- Misses          442      447       +5     
+ Partials        166      164       -2     

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

@akarnokd akarnokd deleted the 4xUnitTestLambdas20 branch June 22, 2026 20:32
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