Merge pull request #44 from apache/i598
[MARCHETYPE-598] update JUnit
diff --git a/archetype-common/src/main/java/org/apache/maven/archetype/common/MavenJDOMWriter.java b/archetype-common/src/main/java/org/apache/maven/archetype/common/MavenJDOMWriter.java
index e541c89..eac9149 100644
--- a/archetype-common/src/main/java/org/apache/maven/archetype/common/MavenJDOMWriter.java
+++ b/archetype-common/src/main/java/org/apache/maven/archetype/common/MavenJDOMWriter.java
@@ -252,7 +252,7 @@
* @param list
* @param parent
*/
- protected Element findAndReplaceSimpleLists( Counter counter, Element parent, java.util.Collection list,
+ protected Element findAndReplaceSimpleLists( Counter counter, Element parent, Collection list,
String parentName, String childName )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
@@ -297,8 +297,7 @@
}
}
return element;
- } // -- Element findAndReplaceSimpleLists(Counter, Element, java.util.Collection, String,
- // String)
+ } // -- Element findAndReplaceSimpleLists(Counter, Element, Collection, String, String)
/**
* Method findAndReplaceXpp3DOM.
@@ -380,8 +379,8 @@
* @param list
* @param parent
*/
- protected void iterateContributor( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ protected void iterateContributor( Counter counter, Element parent, Collection list,
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -424,8 +423,8 @@
}
}
}
- } // -- void iterateContributor(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateContributor(Counter, Element, Collection, String,
+ // String)
/**
* Method iterateDependency.
@@ -436,8 +435,8 @@
* @param list
* @param parent
*/
- protected void iterateDependency( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ protected void iterateDependency( Counter counter, Element parent, Collection list,
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -480,8 +479,8 @@
}
}
}
- } // -- void iterateDependency(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateDependency(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateDeveloper.
@@ -492,8 +491,8 @@
* @param list
* @param parent
*/
- protected void iterateDeveloper( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ protected void iterateDeveloper( Counter counter, Element parent, Collection list,
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -536,8 +535,8 @@
}
}
}
- } // -- void iterateDeveloper(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateDeveloper(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateExclusion.
@@ -548,8 +547,8 @@
* @param list
* @param parent
*/
- protected void iterateExclusion( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ protected void iterateExclusion( Counter counter, Element parent, Collection list,
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -592,8 +591,7 @@
}
}
}
- } // -- void iterateExclusion(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateExclusion(Counter, Element, Collection, String, String)
/**
* Method iterateExtension.
@@ -604,8 +602,8 @@
* @param list
* @param parent
*/
- protected void iterateExtension( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ protected void iterateExtension( Counter counter, Element parent, Collection list,
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -648,8 +646,7 @@
}
}
}
- } // -- void iterateExtension(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateExtension(Counter, Element, Collection, String, String)
/**
* Method iterateLicense.
@@ -660,8 +657,8 @@
* @param list
* @param parent
*/
- protected void iterateLicense( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ protected void iterateLicense( Counter counter, Element parent, Collection list,
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -704,8 +701,8 @@
}
}
}
- } // -- void iterateLicense(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateLicense(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateMailingList.
@@ -717,7 +714,7 @@
* @param parent
*/
protected void iterateMailingList( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -760,8 +757,8 @@
}
}
}
- } // -- void iterateMailingList(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateMailingList(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateNotifier.
@@ -773,7 +770,7 @@
* @param parent
*/
protected void iterateNotifier( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -816,8 +813,8 @@
}
}
}
- } // -- void iterateNotifier(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateNotifier(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iteratePlugin.
@@ -829,7 +826,7 @@
* @param parent
*/
protected void iteratePlugin( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -872,8 +869,8 @@
}
}
}
- } // -- void iteratePlugin(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iteratePlugin(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iteratePluginExecution.
@@ -885,7 +882,7 @@
* @param parent
*/
protected void iteratePluginExecution( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -928,8 +925,8 @@
}
}
}
- } // -- void iteratePluginExecution(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iteratePluginExecution(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateProfile.
@@ -941,7 +938,7 @@
* @param parent
*/
protected void iterateProfile( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -984,8 +981,8 @@
}
}
}
- } // -- void iterateProfile(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateProfile(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateReportPlugin.
@@ -997,7 +994,7 @@
* @param parent
*/
protected void iterateReportPlugin( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -1040,8 +1037,8 @@
}
}
}
- } // -- void iterateReportPlugin(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateReportPlugin(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateReportSet.
@@ -1053,7 +1050,7 @@
* @param parent
*/
protected void iterateReportSet( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -1096,8 +1093,8 @@
}
}
}
- } // -- void iterateReportSet(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateReportSet(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateRepository.
@@ -1109,7 +1106,7 @@
* @param parent
*/
protected void iterateRepository( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -1152,8 +1149,8 @@
}
}
}
- } // -- void iterateRepository(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateRepository(Counter, Element, java.util.Collection, String,
+ // String)
/**
* Method iterateResource.
@@ -1165,7 +1162,7 @@
* @param parent
*/
protected void iterateResource( Counter counter, Element parent, java.util.Collection list,
- java.lang.String parentTag, java.lang.String childTag )
+ String parentTag, String childTag )
{
boolean shouldExist = ( list != null ) && ( list.size() > 0 );
Element element = updateElement( counter, parent, parentTag, shouldExist );
@@ -1208,8 +1205,7 @@
}
}
}
- } // -- void iterateResource(Counter, Element, java.util.Collection, java.lang.String,
- // java.lang.String)
+ } // -- void iterateResource(Counter, Element, Collection, String, String)
/**
* Method replaceXpp3DOM.
@@ -1229,7 +1225,6 @@
domChilds.add( childs[i] );
}
- int domIndex = 0;
ListIterator it = parent.getChildren().listIterator();
while ( it.hasNext() )
{
diff --git a/archetype-common/src/main/java/org/apache/maven/archetype/old/DefaultOldArchetype.java b/archetype-common/src/main/java/org/apache/maven/archetype/old/DefaultOldArchetype.java
index d235cf4..68d6317 100644
--- a/archetype-common/src/main/java/org/apache/maven/archetype/old/DefaultOldArchetype.java
+++ b/archetype-common/src/main/java/org/apache/maven/archetype/old/DefaultOldArchetype.java
@@ -175,7 +175,7 @@
URL[] urls;
try
{
- urls = new URL[] {archetypeFile.toURL() };
+ urls = new URL[] {archetypeFile.toURI().toURL() };
}
catch ( MalformedURLException e )
{
diff --git a/maven-archetype-plugin/src/main/java/org/apache/maven/archetype/mojos/IntegrationTestMojo.java b/maven-archetype-plugin/src/main/java/org/apache/maven/archetype/mojos/IntegrationTestMojo.java
index 1889d7c..feb21ef 100644
--- a/maven-archetype-plugin/src/main/java/org/apache/maven/archetype/mojos/IntegrationTestMojo.java
+++ b/maven-archetype-plugin/src/main/java/org/apache/maven/archetype/mojos/IntegrationTestMojo.java
@@ -34,6 +34,7 @@
import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;
+import java.util.Objects;
import java.util.Properties;
import java.util.Set;
@@ -440,7 +441,7 @@
{
refLine = referenceFileReader.readLine();
actualLine = actualFileReader.readLine();
- if ( !StringUtils.equals( refLine, actualLine ) )
+ if ( !Objects.equals( refLine, actualLine ) )
{
return false;
}
diff --git a/pom.xml b/pom.xml
index bb2d1ff..4f0d7f1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -213,7 +213,7 @@
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
- <version>4.11</version>
+ <version>4.13</version>
</dependency>
<dependency>
<groupId>org.apache.maven.plugin-testing</groupId>