[ 
https://issues.apache.org/jira/browse/SUREFIRE-1934?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17800112#comment-17800112
 ] 

ASF GitHub Bot commented on SUREFIRE-1934:
------------------------------------------

michael-o commented on PR #670:
URL: https://github.com/apache/maven-surefire/pull/670#issuecomment-1868376997

   I have now re-read the code and your change. I fail to see how this change 
only applies to successful tests. It applies to *all* type of results: 
`SUCCESS` and non-`SUCCESS` otherwise you wouldn't modify `#getTestProblems()`, 
 no?
   This is no not necessarily wrong, but inconsistent with your description. 
See the original change: 
https://github.com/apache/maven-surefire/commit/b80325616daa323eb71a04a61d7485a1d303230c#diff-c634b539151b4b76bde50d93d5a6cd2b0f51aa411281398cc295cc721bdfa1d5




> Ability to disable system-out/system-err for successfuly passed tests
> ---------------------------------------------------------------------
>
>                 Key: SUREFIRE-1934
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-1934
>             Project: Maven Surefire
>          Issue Type: Improvement
>          Components: Maven Surefire Plugin
>    Affects Versions: 3.0.0-M5
>            Reporter: Andrey Turbanov
>            Priority: Major
>
> After SUREFIRE-1744 surefire-plugin always reports system-out/system-err even 
> for successfully passed test. A lot of old projects with big amount of tests 
> now have to deal with huge output.
> I think there should be option to disable new behavior.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to