[MPMD-322] Display when PMD/CPD is skipped

Author: adangel

src/main/java/org/apache/maven/plugins/pmd/CpdReport.java

@@ -170,6 +170,7 @@ public boolean canGenerateReport()
     {
         if ( skip )
         {
+            getLog().info( "Skipping CPD execution" );
             return false;
         }
 

src/main/java/org/apache/maven/plugins/pmd/CpdViolationCheckMojo.java

@@ -75,10 +75,13 @@ public CpdViolationCheckMojo()
     public void execute()
         throws MojoExecutionException, MojoFailureException
     {
-        if ( !skip )
+        if ( skip )
         {
-            executeCheck( "cpd.xml", "duplication", "CPD duplication", 10 );
+            getLog().info( "Skipping CPD execution" );
+            return;
         }
+
+        executeCheck( "cpd.xml", "duplication", "CPD duplication", 10 );
     }
 
     /**

src/main/java/org/apache/maven/plugins/pmd/PmdReport.java

@@ -315,6 +315,7 @@ public boolean canGenerateReport()
     {
         if ( skip )
         {
+            getLog().info( "Skipping PMD execution" );
             return false;
         }
 

src/main/java/org/apache/maven/plugins/pmd/PmdViolationCheckMojo.java

@@ -84,10 +84,13 @@ public PmdViolationCheckMojo()
     public void execute()
         throws MojoExecutionException, MojoFailureException
     {
-        if ( !skip )
+        if ( skip )
         {
-            executeCheck( "pmd.xml", "violation", "PMD violation", failurePriority );
+            getLog().info( "Skipping PMD execution" );
+            return;
         }
+
+        executeCheck( "pmd.xml", "violation", "PMD violation", failurePriority );
     }
 
     /**

src/test/java/org/apache/maven/plugins/pmd/PmdReportTest.java

@@ -365,6 +365,10 @@ public void testSkipConfiguration()
 
         generatedFile = new File( getBasedir(), "target/test/unit/custom-configuration/target/site/pmd.html" );
         assertFalse( FileUtils.fileExists( generatedFile.getAbsolutePath() ) );
+
+        // the fact, the PMD execution has been skipped, should be logged
+        String output = CapturingPrintStream.getOutput();
+        assertTrue ( output.contains( "Skipping PMD execution" ) );
     }
 
     public void testSkipEmptyReportConfiguration()