This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch simplify
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git

commit 315d3e8f7fecc5d19e572933dc144dd0b88abce4
Author: Michael Osipov <[email protected]>
AuthorDate: Mon May 29 22:39:42 2023 +0200

    Simplify 
o.a.m.plugins.surefire.report.AbstractSurefireReportMojo.canGenerateReport()
    
    The call to super method is redundant because it is always true.
    
    This closes #651
---
 .../maven/plugins/surefire/report/AbstractSurefireReportMojo.java     | 4 ----
 1 file changed, 4 deletions(-)

diff --git 
a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java
 
b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java
index 72867bfbb..d2246b447 100644
--- 
a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java
+++ 
b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java
@@ -163,10 +163,6 @@ public abstract class AbstractSurefireReportMojo extends 
AbstractMavenReport {
 
     @Override
     public boolean canGenerateReport() {
-        return hasReportDirectories() && super.canGenerateReport();
-    }
-
-    private boolean hasReportDirectories() {
         if (isSkipped()) {
             return false;
         }

Reply via email to