slawekjaranowski commented on a change in pull request #419:
URL: https://github.com/apache/maven-surefire/pull/419#discussion_r777111661
##########
File path:
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
##########
@@ -206,7 +207,7 @@ public void run()
// if tests failed, but if this does not happen then
printing warning to console is the only way to
// inform the users.
String msg = "ForkStarter IOException: " +
e.getLocalizedMessage() + ".";
- File reportsDir =
defaultReporterFactory.getReportsDirectory();
+ File reportsDir = reportMerger.getReportsDirectory();
Review comment:
`reportMerger` and `startupReportConfiguration` are final variable
`startupReportConfiguration` is pass to `reportMerger` constructor ...
`CloseableCloser` is inner class which use variable from ForkStarter
instance
is it possible that we have different instance of
`startupReportConfiguration`?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
