Code improvements
authorKarl Heinz Marbaise <khmarbaise@apache.org>
Sat, 6 Jan 2018 18:18:38 +0000 (19:18 +0100)
committerKarl Heinz Marbaise <khmarbaise@apache.org>
Tue, 19 Jun 2018 17:05:45 +0000 (19:05 +0200)
 o Removed unused class.
 o Some minore improvements.

enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/ReactorModuleConvergence.java
enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
enforcer-rules/src/test/java/org/apache/maven/plugins/enforcer/MockPathTranslator.java [deleted file]

index 1f5e6a1..55fb74a 100644 (file)
@@ -44,6 +44,8 @@ import org.codehaus.plexus.util.StringUtils;
 public class ReactorModuleConvergence\r
     extends AbstractNonCacheableEnforcerRule\r
 {\r
+    private static final String MODULE_TEXT = " module: ";\r
+\r
     private boolean ignoreModuleDependencies = false;\r
 \r
     private Log logger;\r
@@ -90,7 +92,7 @@ public class ReactorModuleConvergence
             addMessageIfExist( sb );\r
             for ( MavenProject mavenProject : parentsWhichAreNotPartOfTheReactor )\r
             {\r
-                sb.append( " module: " );\r
+                sb.append( MODULE_TEXT );\r
                 sb.append( mavenProject.getId() );\r
                 sb.append( SystemUtils.LINE_SEPARATOR );\r
             }\r
@@ -115,7 +117,7 @@ public class ReactorModuleConvergence
             addMessageIfExist( sb );\r
             for ( MavenProject mavenProject : modulesWithoutParentsInReactor )\r
             {\r
-                sb.append( " module: " );\r
+                sb.append( MODULE_TEXT );\r
                 sb.append( mavenProject.getId() );\r
                 sb.append( SystemUtils.LINE_SEPARATOR );\r
             }\r
@@ -138,7 +140,7 @@ public class ReactorModuleConvergence
             // CHECKSTYLE_OFF: LineLength\r
             for ( Entry<MavenProject, List<Dependency>> item : areThereDependenciesWhichAreNotPartOfTheReactor.entrySet() )\r
             {\r
-                sb.append( " module: " );\r
+                sb.append( MODULE_TEXT );\r
                 sb.append( item.getKey().getId() );\r
                 sb.append( SystemUtils.LINE_SEPARATOR );\r
                 for ( Dependency dependency : item.getValue() )\r
index 9766ce7..7c7689b 100644 (file)
@@ -48,7 +48,7 @@ public class RequireMavenVersion
         }
         catch ( ExpressionEvaluationException e )
         {
-            e.printStackTrace();
+            throw new EnforcerRuleException( "Unable to retrieve the session.", e );
         }
 
     }
diff --git a/enforcer-rules/src/test/java/org/apache/maven/plugins/enforcer/MockPathTranslator.java b/enforcer-rules/src/test/java/org/apache/maven/plugins/enforcer/MockPathTranslator.java
deleted file mode 100644 (file)
index 73a74db..0000000
+++ /dev/null
@@ -1,76 +0,0 @@
-package org.apache.maven.plugins.enforcer;
-
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
- *
- *  http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-import java.io.File;
-
-import org.apache.maven.model.Model;
-import org.apache.maven.project.path.PathTranslator;
-
-/**
- * The Class MockPathTranslator.
- *
- * @author <a href="mailto:brianf@apache.org">Brian Fox</a>
- */
-public class MockPathTranslator
-    implements PathTranslator
-{
-
-    /*
-     * (non-Javadoc)
-     *
-     * @see org.apache.maven.project.path.PathTranslator#alignToBaseDirectory(org.apache.maven.model.Model,
-     *      java.io.File)
-     */
-    public void alignToBaseDirectory( Model theModel, File theBasedir )
-    {
-    }
-
-    /*
-     * (non-Javadoc)
-     *
-     * @see org.apache.maven.project.path.PathTranslator#alignToBaseDirectory(java.lang.String, java.io.File)
-     */
-    public String alignToBaseDirectory( String thePath, File theBasedir )
-    {
-        return theBasedir.getAbsolutePath();
-    }
-
-    /*
-     * (non-Javadoc)
-     *
-     * @see org.apache.maven.project.path.PathTranslator#unalignFromBaseDirectory(org.apache.maven.model.Model,
-     *      java.io.File)
-     */
-    public void unalignFromBaseDirectory( Model theModel, File theBasedir )
-    {
-    }
-
-    /*
-     * (non-Javadoc)
-     *
-     * @see org.apache.maven.project.path.PathTranslator#unalignFromBaseDirectory(java.lang.String, java.io.File)
-     */
-    public String unalignFromBaseDirectory( String theDirectory, File theBasedir )
-    {
-        return theBasedir.getAbsolutePath();
-    }
-
-}