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

Posted to commits@maven.apache.org by st...@apache.org on 2011/11/22 13:04:53 UTC

svn commit: r1204939 - /maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java

Author: stephenc
Date: Tue Nov 22 12:04:53 2011
New Revision: 1204939

URL: http://svn.apache.org/viewvc?rev=1204939&view=rev
Log:
[SUREFIRE-772] Skip Maven Failsafe Report

o And this is why we write tests... to catch when you have a boolean the wrong way around.

o Todo: The tests are not ready to commit yet, but in progress

Modified:
    maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java

Modified: maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java?rev=1204939&r1=1204938&r2=1204939&view=diff
==============================================================================
--- maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java (original)
+++ maven/surefire/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java Tue Nov 22 12:04:53 2011
@@ -149,7 +149,7 @@ public abstract class AbstractSurefireRe
             return;
         }
 
-        if ( isGeneratedWhenNoResults() )
+        if ( !isGeneratedWhenNoResults() )
         {
             boolean atLeastOneDirectoryExists = false;
             for ( Iterator i = reportsDirectoryList.iterator(); i.hasNext() && !atLeastOneDirectoryExists; )
@@ -182,7 +182,7 @@ public abstract class AbstractSurefireRe
             return false;
         }
 
-        if ( isGeneratedWhenNoResults() )
+        if ( !isGeneratedWhenNoResults() )
         {
             boolean atLeastOneDirectoryExists = false;
             for ( Iterator i = reportsDirectoryList.iterator(); i.hasNext() && !atLeastOneDirectoryExists; )