Code improvements
o Removed unused class.
o Some minore improvements.
diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/ReactorModuleConvergence.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/ReactorModuleConvergence.java
index 1f5e6a1..55fb74a 100644
--- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/ReactorModuleConvergence.java
+++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/ReactorModuleConvergence.java
@@ -44,6 +44,8 @@
public class ReactorModuleConvergence
extends AbstractNonCacheableEnforcerRule
{
+ private static final String MODULE_TEXT = " module: ";
+
private boolean ignoreModuleDependencies = false;
private Log logger;
@@ -90,7 +92,7 @@
addMessageIfExist( sb );
for ( MavenProject mavenProject : parentsWhichAreNotPartOfTheReactor )
{
- sb.append( " module: " );
+ sb.append( MODULE_TEXT );
sb.append( mavenProject.getId() );
sb.append( SystemUtils.LINE_SEPARATOR );
}
@@ -115,7 +117,7 @@
addMessageIfExist( sb );
for ( MavenProject mavenProject : modulesWithoutParentsInReactor )
{
- sb.append( " module: " );
+ sb.append( MODULE_TEXT );
sb.append( mavenProject.getId() );
sb.append( SystemUtils.LINE_SEPARATOR );
}
@@ -138,7 +140,7 @@
// CHECKSTYLE_OFF: LineLength
for ( Entry<MavenProject, List<Dependency>> item : areThereDependenciesWhichAreNotPartOfTheReactor.entrySet() )
{
- sb.append( " module: " );
+ sb.append( MODULE_TEXT );
sb.append( item.getKey().getId() );
sb.append( SystemUtils.LINE_SEPARATOR );
for ( Dependency dependency : item.getValue() )
diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
index 9766ce7..7c7689b 100644
--- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
+++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
@@ -48,7 +48,7 @@
}
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
index 73a74db..0000000
--- a/enforcer-rules/src/test/java/org/apache/maven/plugins/enforcer/MockPathTranslator.java
+++ /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();
- }
-
-}