Github user britter commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/127#discussion_r82631034
--- Diff:
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/NullStatisticsReporter.java
---
@@ -35,7 +36,7 @@
public NullStatisticsReporter()
{
- super( FileUtils.getTempDirectory() );
+ super( FileUtils.getFile( FileUtils.getTempDirectory(),
RandomStringUtils.randomAlphabetic( 24 ) ) );
--- End diff --
@Tibor17 makes sense. If we change it like this, we don't need the test to
make sure the null objects can be created. I'll modify the PR to incorporate
protected default constructors and remove the empty tests.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
