- Improved code
Make test code more Java 5 like to reduce the number of warnings.
git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1622517 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/pom.xml b/pom.xml
index 6496743..4f7f1fe 100644
--- a/pom.xml
+++ b/pom.xml
@@ -116,6 +116,7 @@
<artifactId>maven-plugin-annotations</artifactId>
<scope>provided</scope>
</dependency>
+ <!-- FIXME: Check for Maven 2.2.1 / Maven 3 compatibility. -->
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-io</artifactId>
@@ -147,6 +148,7 @@
<version>1.19</version>
</dependency>
+ <!-- FIXME: This can be removed with upgrade to Maven 2.2.1 minimum version -->
<dependency>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-container-default</artifactId>
diff --git a/src/test/java/org/apache/maven/plugin/war/overlay/OverlayManagerTest.java b/src/test/java/org/apache/maven/plugin/war/overlay/OverlayManagerTest.java
index e389599..87a39f5 100644
--- a/src/test/java/org/apache/maven/plugin/war/overlay/OverlayManagerTest.java
+++ b/src/test/java/org/apache/maven/plugin/war/overlay/OverlayManagerTest.java
@@ -45,7 +45,7 @@
throws Exception
{
final MavenProjectArtifactsStub project = new MavenProjectArtifactsStub();
- final List overlays = new ArrayList();
+ final List<Overlay> overlays = new ArrayList<Overlay>();
try
{
final Overlay currentProjectOVerlay = Overlay.createInstance();
@@ -70,7 +70,7 @@
final ArtifactStub first = newWarArtifact( "test", "test-webapp" );
project.addArtifact( first );
- final List overlays = new ArrayList();
+ final List<Overlay> overlays = new ArrayList<Overlay>();
try
{
@@ -97,7 +97,7 @@
final ArtifactStub first = newWarArtifact( "test", "test-webapp" );
project.addArtifact( first );
- final List overlays = new ArrayList();
+ final List<Overlay> overlays = new ArrayList<Overlay>();
overlays.add( new DefaultOverlay( first ) );
try
@@ -125,7 +125,7 @@
final ArtifactStub first = newWarArtifact( "test", "test-webapp" );
project.addArtifact( first );
- final List overlays = new ArrayList();
+ final List<Overlay> overlays = new ArrayList<Overlay>();
overlays.add( new Overlay( "test", "test-webapp-2" ) );
try
@@ -150,7 +150,7 @@
project.addArtifact( first );
project.addArtifact( second );
- final List overlays = new ArrayList();
+ final List<Overlay> overlays = new ArrayList<Overlay>();
overlays.add( new DefaultOverlay( first ) );
final Overlay currentProjectOverlay = Overlay.createInstance();
overlays.add( currentProjectOverlay );
@@ -184,7 +184,7 @@
project.addArtifact( first );
project.addArtifact( second );
- final List overlays = new ArrayList();
+ final List<Overlay> overlays = new ArrayList<Overlay>();
overlays.add( new DefaultOverlay( first ) );
overlays.add( new DefaultOverlay( second ) );
diff --git a/src/test/java/org/apache/maven/plugin/war/stub/MavenProject4CopyConstructor.java b/src/test/java/org/apache/maven/plugin/war/stub/MavenProject4CopyConstructor.java
index c371891..bede8f3 100644
--- a/src/test/java/org/apache/maven/plugin/war/stub/MavenProject4CopyConstructor.java
+++ b/src/test/java/org/apache/maven/plugin/war/stub/MavenProject4CopyConstructor.java
@@ -24,6 +24,8 @@
import java.util.LinkedList;
import java.util.List;
+import org.apache.maven.artifact.Artifact;
+
public class MavenProject4CopyConstructor
extends MavenProjectBasicStub
{
@@ -35,9 +37,9 @@
initializeParentFields();
}
- public List getAttachedArtifacts()
+ public List<Artifact> getAttachedArtifacts()
{
- return new LinkedList();
+ return new LinkedList<Artifact>();
}
// to prevent the MavenProject copy constructor from blowing up
@@ -45,15 +47,15 @@
{
// the pom should be located in the isolated dummy root
super.setFile( new File( getBasedir(), "pom.xml" ) );
- super.setDependencyArtifacts( new HashSet() );
- super.setArtifacts( new HashSet() );
- super.setPluginArtifacts( new HashSet() );
- super.setReportArtifacts( new HashSet() );
- super.setExtensionArtifacts( new HashSet() );
- super.setRemoteArtifactRepositories( new LinkedList() );
- super.setPluginArtifactRepositories( new LinkedList() );
- super.setCollectedProjects( new LinkedList() );
- super.setActiveProfiles( new LinkedList() );
+ super.setDependencyArtifacts( new HashSet<Artifact>() );
+ super.setArtifacts( new HashSet<Artifact>() );
+ super.setPluginArtifacts( new HashSet<Artifact>() );
+ super.setReportArtifacts( new HashSet<Artifact>() );
+ super.setExtensionArtifacts( new HashSet<Artifact>() );
+ super.setRemoteArtifactRepositories( new LinkedList<Artifact>() );
+ super.setPluginArtifactRepositories( new LinkedList<Artifact>() );
+ super.setCollectedProjects( new LinkedList<Artifact>() );
+ super.setActiveProfiles( new LinkedList<Artifact>() );
super.setOriginalModel( null );
super.setExecutionProject( this );
super.setBuild( getBuild() );
diff --git a/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectArtifactsStub.java b/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectArtifactsStub.java
index 3e29807..1b2dacf 100644
--- a/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectArtifactsStub.java
+++ b/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectArtifactsStub.java
@@ -31,12 +31,12 @@
public class MavenProjectArtifactsStub
extends MavenProjectBasicStub
{
- TreeSet artifacts;
+ TreeSet<Artifact> artifacts;
public MavenProjectArtifactsStub()
throws Exception
{
- artifacts = new TreeSet();
+ artifacts = new TreeSet<Artifact>();
}
public void addArtifact( ArtifactStub stub )
@@ -44,18 +44,18 @@
artifacts.add( stub );
}
- public Set getArtifacts()
+ public Set<Artifact> getArtifacts()
{
return artifacts;
}
- public List getDependencies()
+ public List<Dependency> getDependencies()
{
if ( getArtifacts() == null )
{
- return new ArrayList();
+ return new ArrayList<Dependency>();
}
- final List dependencies = new ArrayList();
+ final List<Dependency> dependencies = new ArrayList<Dependency>();
for (Object o : getArtifacts()) {
Artifact a = (Artifact) o;
Dependency dependency = new Dependency();
diff --git a/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectBasicStub.java b/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectBasicStub.java
index 5f72f16..2d55ddb 100644
--- a/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectBasicStub.java
+++ b/src/test/java/org/apache/maven/plugin/war/stub/MavenProjectBasicStub.java
@@ -19,15 +19,16 @@
* under the License.
*/
-import org.apache.maven.model.Build;
-import org.apache.maven.model.Organization;
-import org.apache.maven.project.MavenProject;
-
import java.io.File;
import java.util.HashSet;
import java.util.Properties;
import java.util.Set;
+import org.apache.maven.artifact.Artifact;
+import org.apache.maven.model.Build;
+import org.apache.maven.model.Organization;
+import org.apache.maven.project.MavenProject;
+
/**
* Stub
*/
@@ -45,9 +46,9 @@
properties = new Properties();
}
- public Set getArtifacts()
+ public Set<Artifact> getArtifacts()
{
- return new HashSet();
+ return new HashSet<Artifact>();
}
public String getName()
diff --git a/src/test/java/org/apache/maven/plugin/war/stub/MavenZipProject.java b/src/test/java/org/apache/maven/plugin/war/stub/MavenZipProject.java
index ec199a8..faea95e 100644
--- a/src/test/java/org/apache/maven/plugin/war/stub/MavenZipProject.java
+++ b/src/test/java/org/apache/maven/plugin/war/stub/MavenZipProject.java
@@ -32,21 +32,21 @@
public class MavenZipProject
extends MavenProject4CopyConstructor
{
- private Set artifacts;
+ private Set<Artifact> artifacts;
public MavenZipProject()
throws Exception
{
super();
- this.artifacts = new HashSet();
+ this.artifacts = new HashSet<Artifact>();
}
- public Set getArtifacts()
+ public Set<Artifact> getArtifacts()
{
return this.artifacts;
}
- public Set getDependencyArtifacts()
+ public Set<Artifact> getDependencyArtifacts()
{
return this.artifacts;
}
