diff --git a/src/main/java/org/apache/maven/shared/transfer/artifact/deploy/internal/DefaultArtifactDeployer.java b/src/main/java/org/apache/maven/shared/transfer/artifact/deploy/internal/DefaultArtifactDeployer.java index ff2b0975..11a268c2 100644 --- a/src/main/java/org/apache/maven/shared/transfer/artifact/deploy/internal/DefaultArtifactDeployer.java +++ b/src/main/java/org/apache/maven/shared/transfer/artifact/deploy/internal/DefaultArtifactDeployer.java @@ -94,16 +94,11 @@ private void validateParameters( ProjectBuildingRequest request, Collection(); + this.dependencies = new ArrayList<>(); } this.dependencies.add( dependency ); } @@ -243,7 +243,7 @@ public CollectRequest addManagedDependency( Dependency managedDependency ) { if ( this.managedDependencies.isEmpty() ) { - this.managedDependencies = new ArrayList(); + this.managedDependencies = new ArrayList<>(); } this.managedDependencies.add( managedDependency ); } @@ -291,7 +291,7 @@ public CollectRequest addRepository( ArtifactRepository repository ) { if ( this.repositories.isEmpty() ) { - this.repositories = new ArrayList(); + this.repositories = new ArrayList<>(); } this.repositories.add( repository ); } diff --git a/src/main/java/org/apache/maven/shared/transfer/collection/internal/DefaultDependencyCollector.java b/src/main/java/org/apache/maven/shared/transfer/collection/internal/DefaultDependencyCollector.java index 962fd0a4..e6ce50ee 100644 --- a/src/main/java/org/apache/maven/shared/transfer/collection/internal/DefaultDependencyCollector.java +++ b/src/main/java/org/apache/maven/shared/transfer/collection/internal/DefaultDependencyCollector.java @@ -133,23 +133,18 @@ private void validateRoot( Object root ) */ private boolean isMaven31() { - return canFindCoreClass( "org.eclipse.aether.artifact.Artifact" ); // Maven 3.1 specific + try + { + // Maven 3.1 specific + Thread.currentThread().getContextClassLoader().loadClass( "org.eclipse.aether.artifact.Artifact" ); + return true; + } + catch ( ClassNotFoundException e ) + { + return false; + } } - private boolean canFindCoreClass( String className ) - { - try - { - Thread.currentThread().getContextClassLoader().loadClass( className ); - - return true; - } - catch ( ClassNotFoundException e ) - { - return false; - } - } - /** * Injects the Plexus content. * diff --git a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Invoker.java b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Invoker.java index c78b5afb..642c4b62 100644 --- a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Invoker.java +++ b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Invoker.java @@ -22,7 +22,6 @@ import java.lang.reflect.InvocationTargetException; import org.apache.maven.shared.transfer.collection.DependencyCollectionException; -import org.apache.maven.shared.transfer.dependencies.collect.DependencyCollectorException; /** * Invokes method on objects using reflection. @@ -72,7 +71,7 @@ public static T invoke( Class objectClazz, String staticMethod, Class * @param argClasses the classes of the argument, used to select the right static method * @param args the actual arguments to be passed * @return the result of the method invocation - * @throws DependencyCollectorException if any checked exception occurs + * @throws DependencyCollectionException if any checked exception occurs */ public static T invoke( Class objectClazz, String staticMethod, Class[] argClasses, Object[] args ) throws DependencyCollectionException diff --git a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyCollector.java b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyCollector.java index 5d9032d7..108db214 100644 --- a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyCollector.java +++ b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyCollector.java @@ -116,7 +116,7 @@ public CollectResult collectDependencies( Model root ) .invoke( RepositoryUtils.class, "newArtifactTypeRegistry", ArtifactHandlerManager.class, artifactHandlerManager ); - List aetherDependencies = new ArrayList( root.getDependencies().size() ); + List aetherDependencies = new ArrayList<>( root.getDependencies().size() ); for ( org.apache.maven.model.Dependency mavenDependency : root.getDependencies() ) { aetherDependencies.add( toDependency( mavenDependency, typeRegistry ) ); @@ -125,8 +125,8 @@ public CollectResult collectDependencies( Model root ) if ( root.getDependencyManagement() != null ) { - List aetherManagerDependencies = - new ArrayList( root.getDependencyManagement().getDependencies().size() ); + List aetherManagerDependencies = new ArrayList<>( + root.getDependencyManagement().getDependencies().size() ); for ( org.apache.maven.model.Dependency mavenDependency : root.getDependencyManagement().getDependencies() ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyNodeAdapter.java b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyNodeAdapter.java index b8e6ea9f..fa96b351 100644 --- a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyNodeAdapter.java +++ b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven30DependencyNodeAdapter.java @@ -71,7 +71,7 @@ public List getChildren() public List getRemoteRepositories() { List aetherRepositories = dependencyNode.getRepositories(); - List mavenRepositories = new ArrayList( aetherRepositories.size() ); + List mavenRepositories = new ArrayList<>( aetherRepositories.size() ); for ( RemoteRepository aetherRepository : aetherRepositories ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyCollector.java b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyCollector.java index ef5057b6..43846abf 100644 --- a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyCollector.java +++ b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyCollector.java @@ -116,7 +116,7 @@ public CollectResult collectDependencies( Model root ) .invoke( RepositoryUtils.class, "newArtifactTypeRegistry", ArtifactHandlerManager.class, artifactHandlerManager ); - List aetherDependencies = new ArrayList( root.getDependencies().size() ); + List aetherDependencies = new ArrayList<>( root.getDependencies().size() ); for ( org.apache.maven.model.Dependency mavenDependency : root.getDependencies() ) { aetherDependencies.add( toDependency( mavenDependency, typeRegistry ) ); @@ -125,8 +125,8 @@ public CollectResult collectDependencies( Model root ) if ( root.getDependencyManagement() != null ) { - List aetherManagerDependencies = - new ArrayList( root.getDependencyManagement().getDependencies().size() ); + List aetherManagerDependencies = new ArrayList<>( + root.getDependencyManagement().getDependencies().size() ); for ( org.apache.maven.model.Dependency mavenDependency : root.getDependencyManagement().getDependencies() ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyNodeAdapter.java b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyNodeAdapter.java index 5987f7fd..eae15421 100644 --- a/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyNodeAdapter.java +++ b/src/main/java/org/apache/maven/shared/transfer/collection/internal/Maven31DependencyNodeAdapter.java @@ -71,7 +71,7 @@ public List getChildren() public List getRemoteRepositories() { List aetherRepositories = dependencyNode.getRepositories(); - List mavenRepositories = new ArrayList( aetherRepositories.size() ); + List mavenRepositories = new ArrayList<>( aetherRepositories.size() ); for ( RemoteRepository aetherRepository : aetherRepositories ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollector.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollector.java index ca416ec3..fb1e498e 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollector.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollector.java @@ -134,16 +134,11 @@ private void validateRoot( Object root ) * @return true if the current Maven version is Maven 3.1. */ private boolean isMaven31() - { - return canFindCoreClass( "org.eclipse.aether.artifact.Artifact" ); // Maven 3.1 specific - } - - private boolean canFindCoreClass( String className ) { try { - Thread.currentThread().getContextClassLoader().loadClass( className ); - + // Maven 3.1 specific + Thread.currentThread().getContextClassLoader().loadClass( "org.eclipse.aether.artifact.Artifact" ); return true; } catch ( ClassNotFoundException e ) diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Invoker.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Invoker.java index 17c74322..45872e97 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Invoker.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Invoker.java @@ -19,10 +19,10 @@ * under the License. */ -import java.lang.reflect.InvocationTargetException; - import org.apache.maven.shared.transfer.dependencies.collect.DependencyCollectorException; +import java.lang.reflect.InvocationTargetException; + /** * Invokes method on objects using reflection. */ diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30CollectorResult.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30CollectorResult.java index f07c10f1..081dd663 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30CollectorResult.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30CollectorResult.java @@ -52,7 +52,7 @@ class Maven30CollectorResult implements CollectorResult @Override public List getRemoteRepositories() { - final Set aetherRepositories = new HashSet(); + final Set aetherRepositories = new HashSet<>(); DependencyVisitor visitor = new DependencyVisitor() { @@ -72,7 +72,7 @@ public boolean visitLeave( DependencyNode node ) collectResult.getRoot().accept( visitor ); - List mavenRepositories = new ArrayList( aetherRepositories.size() ); + List mavenRepositories = new ArrayList<>( aetherRepositories.size() ); for ( RemoteRepository aetherRepository : aetherRepositories ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyCollector.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyCollector.java index 1a33fbc5..8a850035 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyCollector.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyCollector.java @@ -72,7 +72,7 @@ private static Dependency toDependency( org.apache.maven.model.Dependency mavenD Object[] args = new Object[] {mavenDependency, typeRegistry}; - return (Dependency) Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args ); + return Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args ); } @Override diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyNodeAdapter.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyNodeAdapter.java index 339c3ea7..87a10f8e 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyNodeAdapter.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven30DependencyNodeAdapter.java @@ -71,7 +71,7 @@ public List getChildren() public List getRemoteRepositories() { List aetherRepositories = dependencyNode.getRepositories(); - List mavenRepositories = new ArrayList( aetherRepositories.size() ); + List mavenRepositories = new ArrayList<>( aetherRepositories.size() ); for ( RemoteRepository aetherRepository : aetherRepositories ) { @@ -152,7 +152,7 @@ private Artifact getArtifact( org.sonatype.aether.artifact.Artifact aetherArtifa { try { - return (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact", + return Invoker.invoke( RepositoryUtils.class, "toArtifact", org.sonatype.aether.artifact.Artifact.class, aetherArtifact ); } catch ( DependencyCollectorException e ) diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31CollectorResult.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31CollectorResult.java index 8d463909..dbf8717a 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31CollectorResult.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31CollectorResult.java @@ -52,7 +52,7 @@ class Maven31CollectorResult implements CollectorResult @Override public List getRemoteRepositories() { - final Set aetherRepositories = new HashSet(); + final Set aetherRepositories = new HashSet<>(); DependencyVisitor visitor = new DependencyVisitor() { @@ -72,7 +72,7 @@ public boolean visitLeave( DependencyNode node ) collectResult.getRoot().accept( visitor ); - List mavenRepositories = new ArrayList( aetherRepositories.size() ); + List mavenRepositories = new ArrayList<>( aetherRepositories.size() ); for ( RemoteRepository aetherRepository : aetherRepositories ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyCollector.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyCollector.java index de982108..50f37004 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyCollector.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyCollector.java @@ -72,7 +72,7 @@ private static Dependency toDependency( org.apache.maven.model.Dependency mavenD Object[] args = new Object[] {mavenDependency, typeRegistry}; - return (Dependency) Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args ); + return Invoker.invoke( RepositoryUtils.class, "toDependency", argClasses, args ); } @Override diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyNodeAdapter.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyNodeAdapter.java index 0a5ec8eb..f0cccc41 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyNodeAdapter.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/collect/internal/Maven31DependencyNodeAdapter.java @@ -71,7 +71,7 @@ public List getChildren() public List getRemoteRepositories() { List aetherRepositories = dependencyNode.getRepositories(); - List mavenRepositories = new ArrayList( aetherRepositories.size() ); + List mavenRepositories = new ArrayList<>( aetherRepositories.size() ); for ( RemoteRepository aetherRepository : aetherRepositories ) { @@ -152,7 +152,7 @@ private Artifact getArtifact( org.eclipse.aether.artifact.Artifact aetherArtifac { try { - return (Artifact) Invoker.invoke( RepositoryUtils.class, "toArtifact", + return Invoker.invoke( RepositoryUtils.class, "toArtifact", org.eclipse.aether.artifact.Artifact.class, aetherArtifact ); } catch ( DependencyCollectorException e ) diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/DefaultDependencyResolver.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/DefaultDependencyResolver.java index 05a63c83..7718c19e 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/DefaultDependencyResolver.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/DefaultDependencyResolver.java @@ -100,16 +100,11 @@ public Iterable resolveDependencies( ProjectBuildingRequest buil * @return true if the current Maven version is Maven 3.1. */ private boolean isMaven31() - { - return canFindCoreClass( "org.eclipse.aether.artifact.Artifact" ); // Maven 3.1 specific - } - - private boolean canFindCoreClass( String className ) { try { - Thread.currentThread().getContextClassLoader().loadClass( className ); - + // Maven 3.1 specific + Thread.currentThread().getContextClassLoader().loadClass( "org.eclipse.aether.artifact.Artifact" ); return true; } catch ( ClassNotFoundException e ) diff --git a/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/Maven31DependencyResolver.java b/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/Maven31DependencyResolver.java index 94d7bb41..ea35bc50 100644 --- a/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/Maven31DependencyResolver.java +++ b/src/main/java/org/apache/maven/shared/transfer/dependencies/resolve/internal/Maven31DependencyResolver.java @@ -222,8 +222,7 @@ private Iterable iterator() { // CHECKSTYLE_OFF: LineLength - Collection artResults = - new ArrayList( + Collection artResults = new ArrayList<>( dependencyResults.getArtifactResults().size() ); // CHECKSTYLE_ON: LineLength diff --git a/src/main/java/org/apache/maven/shared/transfer/project/deploy/internal/DefaultProjectDeployer.java b/src/main/java/org/apache/maven/shared/transfer/project/deploy/internal/DefaultProjectDeployer.java index 166ca246..16d1d1b9 100644 --- a/src/main/java/org/apache/maven/shared/transfer/project/deploy/internal/DefaultProjectDeployer.java +++ b/src/main/java/org/apache/maven/shared/transfer/project/deploy/internal/DefaultProjectDeployer.java @@ -91,7 +91,7 @@ public void deploy( ProjectBuildingRequest buildingRequest, ProjectDeployerReque int retryFailedDeploymentCount = projectDeployerRequest.getRetryFailedDeploymentCount(); - List deployableArtifacts = new ArrayList(); + List deployableArtifacts = new ArrayList<>(); if ( isPomArtifact ) { deployableArtifacts.add( artifact ); @@ -118,10 +118,7 @@ else if ( !attachedArtifacts.isEmpty() ) } } - for ( Artifact attached : attachedArtifacts ) - { - deployableArtifacts.add( attached ); - } + deployableArtifacts.addAll( attachedArtifacts ); deploy( buildingRequest, deployableArtifacts, artifactRepository, retryFailedDeploymentCount ); } diff --git a/src/main/java/org/apache/maven/shared/transfer/project/install/internal/DefaultProjectInstaller.java b/src/main/java/org/apache/maven/shared/transfer/project/install/internal/DefaultProjectInstaller.java index 94701ea0..675b55c5 100644 --- a/src/main/java/org/apache/maven/shared/transfer/project/install/internal/DefaultProjectInstaller.java +++ b/src/main/java/org/apache/maven/shared/transfer/project/install/internal/DefaultProjectInstaller.java @@ -83,7 +83,7 @@ public void install( ProjectBuildingRequest buildingRequest, ProjectInstallerReq ProjectArtifactMetadata metadata; - Collection metadataFiles = new LinkedHashSet(); + Collection metadataFiles = new LinkedHashSet<>(); if ( isPomArtifact ) { @@ -108,7 +108,7 @@ public void install( ProjectBuildingRequest buildingRequest, ProjectInstallerReq // but not package). We are designing in a proper solution for Maven 2.1 if ( file != null && file.isFile() ) { - installer.install( buildingRequest, Collections.singletonList( artifact ) ); + installer.install( buildingRequest, Collections.singletonList( artifact ) ); addMetaDataFilesForArtifact( buildingRequest, artifact, metadataFiles ); } else if ( !attachedArtifacts.isEmpty() ) diff --git a/src/main/java/org/apache/maven/shared/transfer/repository/internal/DefaultRepositoryManager.java b/src/main/java/org/apache/maven/shared/transfer/repository/internal/DefaultRepositoryManager.java index a38dce64..c29cc089 100644 --- a/src/main/java/org/apache/maven/shared/transfer/repository/internal/DefaultRepositoryManager.java +++ b/src/main/java/org/apache/maven/shared/transfer/repository/internal/DefaultRepositoryManager.java @@ -159,8 +159,8 @@ private MavenRepositoryManager getMavenRepositoryManager( ProjectBuildingRequest org.eclipse.aether.RepositorySystem m31RepositorySystem = container.lookup( org.eclipse.aether.RepositorySystem.class ); - org.eclipse.aether.RepositorySystemSession session = - (org.eclipse.aether.RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" ); + org.eclipse.aether.RepositorySystemSession session = Invoker.invoke( buildingRequest, + "getRepositorySession" ); return new Maven31RepositoryManager( m31RepositorySystem, session ); } @@ -169,8 +169,8 @@ private MavenRepositoryManager getMavenRepositoryManager( ProjectBuildingRequest org.sonatype.aether.RepositorySystem m30RepositorySystem = container.lookup( org.sonatype.aether.RepositorySystem.class ); - org.sonatype.aether.RepositorySystemSession session = - (org.sonatype.aether.RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" ); + org.sonatype.aether.RepositorySystemSession session = Invoker.invoke( buildingRequest, + "getRepositorySession" ); if ( isMaven302() ) { diff --git a/src/main/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManager.java b/src/main/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManager.java index 936ea12f..b7b92143 100644 --- a/src/main/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManager.java +++ b/src/main/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManager.java @@ -104,7 +104,7 @@ public ProjectBuildingRequest setLocalRepositoryBasedir( ProjectBuildingRequest RepositorySystemSession session; try { - session = (RepositorySystemSession) Invoker.invoke( buildingRequest, "getRepositorySession" ); + session = Invoker.invoke( buildingRequest, "getRepositorySession" ); } catch ( RepositoryManagerException e ) { diff --git a/src/test/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolverTest.java b/src/test/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolverTest.java index d36340b1..9f12a7ad 100644 --- a/src/test/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolverTest.java +++ b/src/test/java/org/apache/maven/shared/transfer/artifact/resolve/internal/DefaultArtifactResolverTest.java @@ -24,10 +24,8 @@ import org.apache.maven.artifact.Artifact; import org.apache.maven.project.ProjectBuildingRequest; import org.apache.maven.shared.transfer.artifact.ArtifactCoordinate; -import org.apache.maven.shared.transfer.artifact.deploy.ArtifactDeployerException; import org.apache.maven.shared.transfer.artifact.resolve.ArtifactResolver; import org.apache.maven.shared.transfer.artifact.resolve.ArtifactResolverException; -import org.apache.maven.shared.transfer.artifact.resolve.internal.DefaultArtifactResolver; import org.junit.Before; import org.junit.Rule; import org.junit.Test; @@ -54,7 +52,7 @@ public void setUp() @Test public void resolveArtifactWithArtifactShouldFaileWithIAEWhenParameterBuildingRequestIsNull() - throws ArtifactDeployerException, ArtifactResolverException + throws ArtifactResolverException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter buildingRequest is not allowed to be null." ); @@ -64,7 +62,7 @@ public void resolveArtifactWithArtifactShouldFaileWithIAEWhenParameterBuildingRe @Test public void resolveArtifactWithArtifactShouldFaileWithIAEWhenArtifactIsNull() - throws ArtifactDeployerException, ArtifactResolverException + throws ArtifactResolverException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter mavenArtifact is not allowed to be null." ); @@ -76,7 +74,7 @@ public void resolveArtifactWithArtifactShouldFaileWithIAEWhenArtifactIsNull() @Test public void resolveArtifactWithCoordinateShouldFaileWithIAEWhenParameterBuildingRequestIsNull() - throws ArtifactDeployerException, ArtifactResolverException + throws ArtifactResolverException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter buildingRequest is not allowed to be null." ); @@ -86,7 +84,7 @@ public void resolveArtifactWithCoordinateShouldFaileWithIAEWhenParameterBuilding @Test public void resolveArtifactWithCoordinateShouldFaileWithIAEWhenArtifactIsNull() - throws ArtifactDeployerException, ArtifactResolverException + throws ArtifactResolverException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter coordinate is not allowed to be null." ); diff --git a/src/test/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollectorTest.java b/src/test/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollectorTest.java index a54af314..213435bf 100644 --- a/src/test/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollectorTest.java +++ b/src/test/java/org/apache/maven/shared/transfer/dependencies/collect/internal/DefaultDependencyCollectorTest.java @@ -50,7 +50,7 @@ public void setUp() @Test public void collectDependenciesWithDependableCoordinatShouldFailWithIAEWhenParameterBuildingRequestIsNull() - throws ArtifactDeployerException, ArtifactResolverException, DependencyCollectorException + throws DependencyCollectorException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter buildingRequest is not allowed to be null." ); @@ -60,7 +60,7 @@ public void collectDependenciesWithDependableCoordinatShouldFailWithIAEWhenParam @Test public void collectDependenciesWithDependableCoordinatShouldFailWithIAEWhenParameterRootIsNull() - throws ArtifactDeployerException, ArtifactResolverException, DependencyCollectorException + throws DependencyCollectorException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter root is not allowed to be null." ); @@ -71,7 +71,7 @@ public void collectDependenciesWithDependableCoordinatShouldFailWithIAEWhenParam @Test public void collectDependenciesWithDependencyShouldFailWithIAEWhenParameterBuildingRequestIsNull() - throws ArtifactDeployerException, ArtifactResolverException, DependencyCollectorException + throws DependencyCollectorException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter buildingRequest is not allowed to be null." ); @@ -81,7 +81,7 @@ public void collectDependenciesWithDependencyShouldFailWithIAEWhenParameterBuild @Test public void collectDependenciesWithDependencyShouldFailWithIAEWhenParameterRootIsNull() - throws ArtifactDeployerException, ArtifactResolverException, DependencyCollectorException + throws DependencyCollectorException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter root is not allowed to be null." ); @@ -92,7 +92,7 @@ public void collectDependenciesWithDependencyShouldFailWithIAEWhenParameterRootI @Test public void collectDependenciesWithModelShouldFailWithIAEWhenParameterBuildingRequestIsNull() - throws ArtifactDeployerException, ArtifactResolverException, DependencyCollectorException + throws DependencyCollectorException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter buildingRequest is not allowed to be null." ); @@ -102,7 +102,7 @@ public void collectDependenciesWithModelShouldFailWithIAEWhenParameterBuildingRe @Test public void collectDependenciesWithModelShouldFailWithIAEWhenParameterRootIsNull() - throws ArtifactDeployerException, ArtifactResolverException, DependencyCollectorException + throws DependencyCollectorException { thrown.expect( IllegalArgumentException.class ); thrown.expectMessage( "The parameter root is not allowed to be null." ); diff --git a/src/test/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManagerTest.java b/src/test/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManagerTest.java index b4263162..6e20cb46 100644 --- a/src/test/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManagerTest.java +++ b/src/test/java/org/apache/maven/shared/transfer/repository/internal/Maven30RepositoryManagerTest.java @@ -44,7 +44,7 @@ public void setUp() throws Exception repositorySystem = lookup( RepositorySystem.class ); } - public void testSetLocalRepositoryBasedirSimple() throws Exception + public void testSetLocalRepositoryBasedirSimple() { DefaultProjectBuildingRequest buildingRequest = new DefaultProjectBuildingRequest(); MavenRepositorySystemSession repositorySession = new MavenRepositorySystemSession(); @@ -62,7 +62,7 @@ public void testSetLocalRepositoryBasedirSimple() throws Exception } - public void testSetLocalRepositoryBasedirEnhanced() throws Exception + public void testSetLocalRepositoryBasedirEnhanced() { DefaultProjectBuildingRequest buildingRequest = new DefaultProjectBuildingRequest(); MavenRepositorySystemSession repositorySession = new MavenRepositorySystemSession();