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

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

kriegaex commented on PR #670:
URL: https://github.com/apache/maven-surefire/pull/670#issuecomment-1863878024

   > what behaviour do you think is required as part of the plugin: the ability 
to disable output for all tests (as in this PR, my cases) or for successful 
tests only (as initially asked in issue)?
   
   I am not Michael, but IMO it makes sense to have both. The simplest way to 
do that would be two boolean options, one for failed tests and tests in error 
and one for passing tests. An alternative would be some kind of list with a set 
of fixed values, but then you also need to add all possible combinations to the 
list and name them accordingly. With two options, this is not much, but what if 
in the future we want to support more test statuses, e.g. differentiate failed 
tests from tests in error, or what if skipped tests in some tools can print the 
reason for skipping to the console? You get the picture.
   
   I am sure, @michael-o can guide you towards a good design decision. 




> 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