Skip to content

Add -XX:+EnableDynamicAgentLoading to test argLine (all reactor modules)#157

Merged
bernardladenthin merged 2 commits into
mainfrom
claude/jacoco-argline-sync-bfam2k
Jul 21, 2026
Merged

Add -XX:+EnableDynamicAgentLoading to test argLine (all reactor modules)#157
bernardladenthin merged 2 commits into
mainfrom
claude/jacoco-argline-sync-bfam2k

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

claude added 2 commits July 20, 2026 22:44
Silences the JDK 21 'A Java agent has been loaded dynamically' warning
emitted when a test-time agent self-attaches at runtime with no static
-javaagent configured — in the core srcmorph module the Lincheck
instrumentation agent (AiGenerationKindLincheckTest runs in the default
surefire run), plus the jcstress/vmlens agents under their opt-in
profiles. That warning is written to the native stderr, outside
Surefire's wrapped System.err, and can intermittently corrupt the encoded
fork channel — surfacing as 'Corrupted channel by directly writing to
native stream' followed by a bogus 'There was a timeout in the fork' even
though every test passed.

Applied identically to all three module poms to keep the reactor's
surefire argLine uniform; mirrors the same guard added to the sibling
BitcoinAddressFinder repo (there triggered by Mockito's inline mock maker
self-attaching byte-buddy).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y84TKMNfhbFGi7EoCEvP99
… pointer

Replace the inline rationale in all three reactor module poms with a one-line
comment pointing to the canonical workspace policy (ci-test-diagnostics.md
section 2.1) plus the root-cause upstream bug (byte-buddy#1639), keeping the
explanation in one synchronized place.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y84TKMNfhbFGi7EoCEvP99
@bernardladenthin
bernardladenthin merged commit b7b29a9 into main Jul 21, 2026
6 of 12 checks passed
@bernardladenthin
bernardladenthin deleted the claude/jacoco-argline-sync-bfam2k branch July 21, 2026 09:17
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