slachiewicz opened a new pull request, #1794:
URL: https://github.com/apache/maven-mvnd/pull/1794

   ## Problem
   
   `RawStreamsTest` prints the entire raw client/daemon protocol stream to 
stdout on every run:
   
   ```java
   o.getMessages().forEach(m -> System.out.println(m.toString()));
   ```
   
   With the daemon idling behind an mrm-proxied build this is ~30 `KeepAlive` 
lines plus `BuildStarted`/`MojoStarted`/... noise at INFO level in the surefire 
log, burying the actual test output. (Already removed on `master`; this brings 
the same cleanup to `mvnd-1.x`.)
   
   ## Change
   
   Drop the unconditional print and include the captured messages in the 
assertion failure message instead, so they are only produced when the test 
fails — keeping the diagnosability without the per-run noise.
   
   ## Verification
   
   - `RawStreamsTest.java` compiles against the test classpath; change is 
test-only, no production code touched.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to