chore: bump async-test-lib 1.7.0-RC1 -> 1.7.0-RC4 - #50
Merged
Conversation
RC1 shipped three detectors that were never wired into a real @AsyncTest run — LatchMisuseDetector, ExecutorDeadlockDetector and FutureBlockingDetector had full implementations, passing unit tests, and no DetectorType constant, registry field or accessor. Nothing constructed them, so their findings could not reach a report. RC4 wires all three and brings the detector count to 127. RC4 also fixes the third-party Detector SPI, which previously ran nowhere: a detector supplied via META-INF/services was discovered by nobody and its violations reached neither the reports nor the failOn gate. Neither of those is visible from here as a failure — a detector that never fires and a detector that finds nothing produce the same green run. That is the reason to move. ConcurrencySafetyTest passes unchanged on RC4: all eight @AsyncTest rounds green. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0153QiTQsRQEikh9pHa5zrpC
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
RC1 shipped three detectors that never ran.
LatchMisuseDetector,ExecutorDeadlockDetectorandFutureBlockingDetectorhad full implementations and passing unit tests, but noDetectorTypeconstant, no registry field and no accessor — nothing constructed them during a real@AsyncTest, so their findings could not reach a report. Upstream's own changelog puts it plainly: "the green test suite made them look shipped."RC4 wires all three and brings the verified detector count to 127.
RC4 also fixes the third-party
DetectorSPI, which previously ran nowhere: a detector supplied throughMETA-INF/serviceswas "discovered by nobody", and its violations reached neither the reports nor thefailOngate.The reason this matters here rather than being a routine bump: neither failure is visible from the consumer side. A detector that never fires and a detector that finds nothing produce the same green run.
ConcurrencySafetyTesthas been passing on RC1 while three of the detectors it nominally enables were dead code.Verification
./gradlew test— all eight@AsyncTestrounds pass unchanged on RC4:Same result, but now with the full detector set actually reaching the runner.
🤖 Generated with Claude Code
https://claude.ai/code/session_0153QiTQsRQEikh9pHa5zrpC