You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2022/02/14 03:00:18 UTC

[GitHub] [maven-surefire] Tibor17 edited a comment on pull request #469: [SUREFIRE-2011] Updated abstractions which helps associating standard out/err with a test

Tibor17 edited a comment on pull request #469:
URL: https://github.com/apache/maven-surefire/pull/469#issuecomment-1038574801


   @slawekjaranowski 
   I have made one more commit because both listeners (ForkingRunListener and TestSetRunListener) are a mirror of the same interface in fork/plugin JVM and so they should implement one and the same interface as mandatory. This way we have avoided ugly casting to type. Any combinations of two interfaces out of three (RunListener, TestOutputReceiver, ConsoleLogger) are avoided and the only TestReportListener is used. Removed ConsoleStream interface and used ConsoleLogger instead. Simplified code around logger in JUnitCoreProvider. Renamed method in ReporterFactory.


-- 
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-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org