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. Added Javadoc explaining the arcitecture, see the interface `TestReportListener`. -- 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
