You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sj...@apache.org on 2022/07/31 12:25:47 UTC

[maven-enforcer] branch master updated: [MENFORCER-398] show rules processed

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

sjaranowski pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-enforcer.git


The following commit(s) were added to refs/heads/master by this push:
     new b61b6b8  [MENFORCER-398] show rules processed
b61b6b8 is described below

commit b61b6b85805ad0579888b695cd722d2b057f5114
Author: Björn Raupach <ra...@me.com>
AuthorDate: Mon Jul 25 18:51:45 2022 +0200

    [MENFORCER-398] show rules processed
---
 .../src/main/java/org/apache/maven/plugins/enforcer/EnforceMojo.java  | 2 +-
 .../test/java/org/apache/maven/plugins/enforcer/TestEnforceMojo.java  | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/maven-enforcer-plugin/src/main/java/org/apache/maven/plugins/enforcer/EnforceMojo.java b/maven-enforcer-plugin/src/main/java/org/apache/maven/plugins/enforcer/EnforceMojo.java
index 4c624e4..ca5e221 100644
--- a/maven-enforcer-plugin/src/main/java/org/apache/maven/plugins/enforcer/EnforceMojo.java
+++ b/maven-enforcer-plugin/src/main/java/org/apache/maven/plugins/enforcer/EnforceMojo.java
@@ -187,7 +187,6 @@ public class EnforceMojo
                 // store the current rule for
                 // logging purposes
                 currentRule = rule.getClass().getName();
-                log.debug( "Executing rule: " + currentRule );
                 try
                 {
                     if ( ignoreCache || shouldExecute( rule ) )
@@ -196,6 +195,7 @@ public class EnforceMojo
                         // noinspection SynchronizationOnLocalVariableOrMethodParameter
                         synchronized ( rule )
                         {
+                            log.info( "Executing rule: " + currentRule );
                             rule.execute( helper );
                         }
                     }
diff --git a/maven-enforcer-plugin/src/test/java/org/apache/maven/plugins/enforcer/TestEnforceMojo.java b/maven-enforcer-plugin/src/test/java/org/apache/maven/plugins/enforcer/TestEnforceMojo.java
index e2c834a..5dcbb91 100644
--- a/maven-enforcer-plugin/src/test/java/org/apache/maven/plugins/enforcer/TestEnforceMojo.java
+++ b/maven-enforcer-plugin/src/test/java/org/apache/maven/plugins/enforcer/TestEnforceMojo.java
@@ -53,6 +53,8 @@ public class TestEnforceMojo
     {
         setupBasics( false );
 
+        Log logSpy = setupLogSpy();
+
         try
         {
             mojo.execute();
@@ -70,6 +72,8 @@ public class TestEnforceMojo
 
         mojo.execute();
 
+        Mockito.verify( logSpy, Mockito.times(2) ).info( Mockito.contains("Executing rule: " + MockEnforcerRule.class.getName()) );
+
         try
         {
             mojo.setFailFast( false );