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. -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
