This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch java8
in repository https://gitbox.apache.org/repos/asf/maven-resolver.git

commit 82cbaec27b8ff61022df65714f6691cb9475830a
Author: Sylwester Lachiewicz <slachiew...@apache.org>
AuthorDate: Wed Jun 17 20:30:52 2020 +0200

    Cleanup warnings
---
 .../eclipse/aether/impl/DefaultServiceLocator.java |  6 +++--
 .../internal/impl/AbstractChecksumPolicy.java      |  1 -
 .../aether/internal/impl/collect/NodeStack.java    |  2 +-
 .../aether/impl/StubArtifactDescriptorReader.java  |  1 -
 .../eclipse/aether/impl/StubVersionResolver.java   |  1 -
 .../impl/StubRepositoryConnectorProvider.java      |  1 -
 .../transport/classpath/ClasspathTransporter.java  |  1 -
 .../aether/transport/wagon/MemStreamWagon.java     | 13 ++++-------
 .../eclipse/aether/transport/wagon/MemWagon.java   | 26 ++++++----------------
 .../aether/transport/wagon/MemWagonUtils.java      |  2 +-
 .../util/graph/transformer/ConflictMarker.java     |  2 --
 .../transformer/JavaDependencyContextRefiner.java  |  2 --
 .../util/graph/transformer/JavaScopeDeriver.java   |  2 --
 .../util/graph/transformer/JavaScopeSelector.java  |  2 --
 .../NoopDependencyGraphTransformer.java            |  2 --
 .../transformer/SimpleOptionalitySelector.java     |  2 --
 .../aether/util/version/GenericVersionScheme.java  |  1 -
 .../util/filter/ExclusionDependencyFilterTest.java | 11 +++++----
 .../graph/transformer/SimpleConflictMarker.java    |  1 -
 .../aether/util/version/UnionVersionRangeTest.java |  9 +-------
 20 files changed, 23 insertions(+), 65 deletions(-)

diff --git 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/impl/DefaultServiceLocator.java
 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/impl/DefaultServiceLocator.java
index 71b4704..406b19c 100644
--- 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/impl/DefaultServiceLocator.java
+++ 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/impl/DefaultServiceLocator.java
@@ -96,7 +96,8 @@ public final class DefaultServiceLocator
             providers = new LinkedHashSet<>( 8 );
         }
 
-        public synchronized void setServices( T... services )
+        @SafeVarargs
+        public final synchronized void setServices( T... services )
         {
             providers.clear();
             if ( services != null )
@@ -267,7 +268,8 @@ public final class DefaultServiceLocator
      * @param services The instances of the service, may be {@code null} but 
must not contain {@code null} elements.
      * @return This locator for chaining, never {@code null}.
      */
-    public <T> DefaultServiceLocator setServices( Class<T> type, T... services 
)
+    @SafeVarargs
+    public final <T> DefaultServiceLocator setServices( Class<T> type, T... 
services )
     {
         getEntry( type, true ).setServices( services );
         return this;
diff --git 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/AbstractChecksumPolicy.java
 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/AbstractChecksumPolicy.java
index 7fe0822..341e012 100644
--- 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/AbstractChecksumPolicy.java
+++ 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/AbstractChecksumPolicy.java
@@ -53,7 +53,6 @@ abstract class AbstractChecksumPolicy
     }
 
     public void onChecksumError( String algorithm, int kind, 
ChecksumFailureException exception )
-        throws ChecksumFailureException
     {
         logger.debug( "Could not validate {} checksum for {}", algorithm, 
resource.getResourceName(), exception );
     }
diff --git 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/collect/NodeStack.java
 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/collect/NodeStack.java
index 668dbc4..1eee3c0 100644
--- 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/collect/NodeStack.java
+++ 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/collect/NodeStack.java
@@ -30,7 +30,7 @@ import org.eclipse.aether.graph.DependencyNode;
 final class NodeStack
 {
 
-    @SuppressWarnings( {"unchecked", "checkstyle:magicnumber" } )
+    @SuppressWarnings( {"checkstyle:magicnumber" } )
     // CHECKSTYLE_OFF: MagicNumber
     private DependencyNode[] nodes = new DependencyNode[96];
     // CHECKSTYLE_ON: MagicNumber
diff --git 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubArtifactDescriptorReader.java
 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubArtifactDescriptorReader.java
index 2882dc2..767c5fe 100644
--- 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubArtifactDescriptorReader.java
+++ 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubArtifactDescriptorReader.java
@@ -30,7 +30,6 @@ public class StubArtifactDescriptorReader
 
     public ArtifactDescriptorResult readArtifactDescriptor( 
RepositorySystemSession session,
                                                             
ArtifactDescriptorRequest request )
-        throws ArtifactDescriptorException
     {
         return new ArtifactDescriptorResult( request );
     }
diff --git 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubVersionResolver.java
 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubVersionResolver.java
index 4baa461..8b5f113 100644
--- 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubVersionResolver.java
+++ 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/impl/StubVersionResolver.java
@@ -29,7 +29,6 @@ public class StubVersionResolver
 {
 
     public VersionResult resolveVersion( RepositorySystemSession session, 
VersionRequest request )
-        throws VersionResolutionException
     {
         return new VersionResult( request );
     }
diff --git 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/StubRepositoryConnectorProvider.java
 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/StubRepositoryConnectorProvider.java
index 3cb5e38..8dc05c1 100644
--- 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/StubRepositoryConnectorProvider.java
+++ 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/StubRepositoryConnectorProvider.java
@@ -46,7 +46,6 @@ class StubRepositoryConnectorProvider
     }
 
     public RepositoryConnector newRepositoryConnector( RepositorySystemSession 
session, RemoteRepository repository )
-        throws NoRepositoryConnectorException
     {
         return connector;
     }
diff --git 
a/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporter.java
 
b/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporter.java
index 968c231..ebe2dcc 100644
--- 
a/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporter.java
+++ 
b/maven-resolver-transport-classpath/src/main/java/org/eclipse/aether/transport/classpath/ClasspathTransporter.java
@@ -135,7 +135,6 @@ final class ClasspathTransporter
 
     @Override
     protected void implPut( PutTask task )
-        throws Exception
     {
         throw new UnsupportedOperationException( "Uploading to a classpath: 
repository is not supported" );
     }
diff --git 
a/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemStreamWagon.java
 
b/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemStreamWagon.java
index 6b4be8f..997cfb5 100644
--- 
a/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemStreamWagon.java
+++ 
b/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemStreamWagon.java
@@ -68,16 +68,14 @@ public class MemStreamWagon
 
     @Override
     protected void openConnectionInternal()
-        throws ConnectionException, AuthenticationException
+        throws ConnectionException
     {
-        fs =
-            MemWagonUtils.openConnection( this, getAuthenticationInfo(),
-                                          getProxyInfo( "mem", 
getRepository().getHost() ), headers );
+        fs = MemWagonUtils.openConnection( this, getAuthenticationInfo(),
+                getProxyInfo( "mem", getRepository().getHost() ), headers );
     }
 
     @Override
     public void closeConnection()
-        throws ConnectionException
     {
         fs = null;
     }
@@ -89,7 +87,6 @@ public class MemStreamWagon
 
     @Override
     public boolean resourceExists( String resourceName )
-        throws TransferFailedException, AuthorizationException
     {
         String data = getData( resourceName );
         return data != null;
@@ -97,7 +94,7 @@ public class MemStreamWagon
 
     @Override
     public void fillInputData( InputData inputData )
-        throws TransferFailedException, ResourceDoesNotExistException, 
AuthorizationException
+        throws ResourceDoesNotExistException
     {
         String data = getData( inputData.getResource().getName() );
         if ( data == null )
@@ -111,14 +108,12 @@ public class MemStreamWagon
 
     @Override
     public void fillOutputData( OutputData outputData )
-        throws TransferFailedException
     {
         outputData.setOutputStream( new ByteArrayOutputStream() );
     }
 
     @Override
     protected void finishPutTransfer( Resource resource, InputStream input, 
OutputStream output )
-        throws TransferFailedException, AuthorizationException, 
ResourceDoesNotExistException
     {
         String data = new String( ( (ByteArrayOutputStream) output 
).toByteArray(), StandardCharsets.UTF_8 );
         fs.put( URI.create( resource.getName() ).getSchemeSpecificPart(), data 
);
diff --git 
a/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagon.java
 
b/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagon.java
index bc62d37..29d4703 100644
--- 
a/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagon.java
+++ 
b/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagon.java
@@ -35,7 +35,6 @@ import org.apache.maven.wagon.InputData;
 import org.apache.maven.wagon.OutputData;
 import org.apache.maven.wagon.ResourceDoesNotExistException;
 import org.apache.maven.wagon.TransferFailedException;
-import org.apache.maven.wagon.authentication.AuthenticationException;
 import org.apache.maven.wagon.authorization.AuthorizationException;
 import org.apache.maven.wagon.events.TransferEvent;
 import org.apache.maven.wagon.resource.Resource;
@@ -70,16 +69,14 @@ public class MemWagon
 
     @Override
     protected void openConnectionInternal()
-        throws ConnectionException, AuthenticationException
+        throws ConnectionException
     {
-        fs =
-            MemWagonUtils.openConnection( this, getAuthenticationInfo(),
-                                          getProxyInfo( "mem", 
getRepository().getHost() ), headers );
+        fs = MemWagonUtils.openConnection( this, getAuthenticationInfo(),
+                getProxyInfo( "mem", getRepository().getHost() ), headers );
     }
 
     @Override
     protected void closeConnection()
-        throws ConnectionException
     {
         fs = null;
     }
@@ -91,7 +88,6 @@ public class MemWagon
 
     @Override
     public boolean resourceExists( String resourceName )
-        throws TransferFailedException, AuthorizationException
     {
         String data = getData( resourceName );
         return data != null;
@@ -104,7 +100,7 @@ public class MemWagon
     }
 
     public boolean getIfNewer( String resourceName, File destination, long 
timestamp )
-        throws TransferFailedException, ResourceDoesNotExistException, 
AuthorizationException
+        throws TransferFailedException, ResourceDoesNotExistException
     {
         Resource resource = new Resource( resourceName );
         fireGetInitiated( resource, destination );
@@ -114,7 +110,7 @@ public class MemWagon
     }
 
     protected InputStream getInputStream( Resource resource )
-        throws TransferFailedException, ResourceDoesNotExistException, 
AuthorizationException
+        throws ResourceDoesNotExistException
     {
         InputData inputData = new InputData();
         inputData.setResource( resource );
@@ -122,7 +118,7 @@ public class MemWagon
         {
             fillInputData( inputData );
         }
-        catch ( TransferFailedException | AuthorizationException | 
ResourceDoesNotExistException e )
+        catch ( ResourceDoesNotExistException e )
         {
             fireTransferError( resource, e, TransferEvent.REQUEST_GET );
             cleanupGetTransfer( resource );
@@ -139,7 +135,7 @@ public class MemWagon
     }
 
     protected void fillInputData( InputData inputData )
-        throws TransferFailedException, ResourceDoesNotExistException, 
AuthorizationException
+        throws ResourceDoesNotExistException
     {
         String data = getData( inputData.getResource().getName() );
         if ( data == null )
@@ -163,7 +159,6 @@ public class MemWagon
     }
 
     protected OutputStream getOutputStream( Resource resource )
-        throws TransferFailedException
     {
         OutputData outputData = new OutputData();
         outputData.setResource( resource );
@@ -171,11 +166,6 @@ public class MemWagon
         {
             fillOutputData( outputData );
         }
-        catch ( TransferFailedException e )
-        {
-            fireTransferError( resource, e, TransferEvent.REQUEST_PUT );
-            throw e;
-        }
         finally
         {
             if ( outputData.getOutputStream() == null )
@@ -188,14 +178,12 @@ public class MemWagon
     }
 
     protected void fillOutputData( OutputData outputData )
-        throws TransferFailedException
     {
         outputData.setOutputStream( new ByteArrayOutputStream() );
     }
 
     @Override
     protected void finishPutTransfer( Resource resource, InputStream input, 
OutputStream output )
-        throws TransferFailedException, AuthorizationException, 
ResourceDoesNotExistException
     {
         String data = new String( ( (ByteArrayOutputStream) output 
).toByteArray(), StandardCharsets.UTF_8 );
         fs.put( URI.create( resource.getName() ).getSchemeSpecificPart(), data 
);
diff --git 
a/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagonUtils.java
 
b/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagonUtils.java
index 993a26a..7add2d0 100644
--- 
a/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagonUtils.java
+++ 
b/maven-resolver-transport-wagon/src/test/java/org/eclipse/aether/transport/wagon/MemWagonUtils.java
@@ -55,7 +55,7 @@ class MemWagonUtils
 
     public static Map<String, String> openConnection( Wagon wagon, 
AuthenticationInfo auth, ProxyInfo proxy,
                                                       Properties headers )
-        throws ConnectionException, AuthenticationException
+        throws ConnectionException
     {
         URI uri = URI.create( wagon.getRepository().getUrl() );
 
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/ConflictMarker.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/ConflictMarker.java
index 85df74a..3c46148 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/ConflictMarker.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/ConflictMarker.java
@@ -27,7 +27,6 @@ import java.util.IdentityHashMap;
 import java.util.Map;
 import java.util.Set;
 
-import org.eclipse.aether.RepositoryException;
 import org.eclipse.aether.artifact.Artifact;
 import org.eclipse.aether.collection.DependencyGraphTransformationContext;
 import org.eclipse.aether.collection.DependencyGraphTransformer;
@@ -51,7 +50,6 @@ public final class ConflictMarker
      * @see DependencyGraphTransformer#transformGraph(DependencyNode, 
DependencyGraphTransformationContext)
      */
     public DependencyNode transformGraph( DependencyNode node, 
DependencyGraphTransformationContext context )
-        throws RepositoryException
     {
         @SuppressWarnings( "unchecked" )
         Map<String, Object> stats = (Map<String, Object>) context.get( 
TransformationContextKeys.STATS );
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaDependencyContextRefiner.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaDependencyContextRefiner.java
index d96e04e..53f9667 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaDependencyContextRefiner.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaDependencyContextRefiner.java
@@ -19,7 +19,6 @@ package org.eclipse.aether.util.graph.transformer;
  * under the License.
  */
 
-import org.eclipse.aether.RepositoryException;
 import org.eclipse.aether.collection.DependencyGraphTransformationContext;
 import org.eclipse.aether.collection.DependencyGraphTransformer;
 import org.eclipse.aether.graph.Dependency;
@@ -38,7 +37,6 @@ public final class JavaDependencyContextRefiner
 {
 
     public DependencyNode transformGraph( DependencyNode node, 
DependencyGraphTransformationContext context )
-        throws RepositoryException
     {
         String ctx = node.getRequestContext();
 
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeDeriver.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeDeriver.java
index 4c5fd3e..d0e3fc8 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeDeriver.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeDeriver.java
@@ -19,7 +19,6 @@ package org.eclipse.aether.util.graph.transformer;
  * under the License.
  */
 
-import org.eclipse.aether.RepositoryException;
 import org.eclipse.aether.util.artifact.JavaScopes;
 import org.eclipse.aether.util.graph.transformer.ConflictResolver.ScopeContext;
 import org.eclipse.aether.util.graph.transformer.ConflictResolver.ScopeDeriver;
@@ -40,7 +39,6 @@ public final class JavaScopeDeriver
 
     @Override
     public void deriveScope( ScopeContext context )
-        throws RepositoryException
     {
         context.setDerivedScope( getDerivedScope( context.getParentScope(), 
context.getChildScope() ) );
     }
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeSelector.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeSelector.java
index 0f923a7..e085793 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeSelector.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/JavaScopeSelector.java
@@ -23,7 +23,6 @@ import java.util.Collection;
 import java.util.HashSet;
 import java.util.Set;
 
-import org.eclipse.aether.RepositoryException;
 import org.eclipse.aether.util.artifact.JavaScopes;
 import 
org.eclipse.aether.util.graph.transformer.ConflictResolver.ConflictContext;
 import org.eclipse.aether.util.graph.transformer.ConflictResolver.ConflictItem;
@@ -47,7 +46,6 @@ public final class JavaScopeSelector
 
     @Override
     public void selectScope( ConflictContext context )
-        throws RepositoryException
     {
         String scope = context.getWinner().getDependency().getScope();
         if ( !JavaScopes.SYSTEM.equals( scope ) )
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/NoopDependencyGraphTransformer.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/NoopDependencyGraphTransformer.java
index 55b6175..d5c060c 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/NoopDependencyGraphTransformer.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/NoopDependencyGraphTransformer.java
@@ -19,7 +19,6 @@ package org.eclipse.aether.util.graph.transformer;
  * under the License.
  */
 
-import org.eclipse.aether.RepositoryException;
 import org.eclipse.aether.collection.DependencyGraphTransformationContext;
 import org.eclipse.aether.collection.DependencyGraphTransformer;
 import org.eclipse.aether.graph.DependencyNode;
@@ -45,7 +44,6 @@ public final class NoopDependencyGraphTransformer
     }
 
     public DependencyNode transformGraph( DependencyNode node, 
DependencyGraphTransformationContext context )
-        throws RepositoryException
     {
         return node;
     }
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/SimpleOptionalitySelector.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/SimpleOptionalitySelector.java
index 0cc7a73..add6c8b 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/SimpleOptionalitySelector.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/transformer/SimpleOptionalitySelector.java
@@ -21,7 +21,6 @@ package org.eclipse.aether.util.graph.transformer;
 
 import java.util.Collection;
 
-import org.eclipse.aether.RepositoryException;
 import 
org.eclipse.aether.util.graph.transformer.ConflictResolver.ConflictContext;
 import org.eclipse.aether.util.graph.transformer.ConflictResolver.ConflictItem;
 import 
org.eclipse.aether.util.graph.transformer.ConflictResolver.OptionalitySelector;
@@ -44,7 +43,6 @@ public final class SimpleOptionalitySelector
 
     @Override
     public void selectOptionality( ConflictContext context )
-        throws RepositoryException
     {
         boolean optional = chooseEffectiveOptionality( context.getItems() );
         context.setOptional( optional );
diff --git 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/version/GenericVersionScheme.java
 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/version/GenericVersionScheme.java
index c191a90..8f4a71d 100644
--- 
a/maven-resolver-util/src/main/java/org/eclipse/aether/util/version/GenericVersionScheme.java
+++ 
b/maven-resolver-util/src/main/java/org/eclipse/aether/util/version/GenericVersionScheme.java
@@ -65,7 +65,6 @@ public final class GenericVersionScheme
     }
 
     public Version parseVersion( final String version )
-        throws InvalidVersionSpecificationException
     {
         return new GenericVersion( version );
     }
diff --git 
a/maven-resolver-util/src/test/java/org/eclipse/aether/util/filter/ExclusionDependencyFilterTest.java
 
b/maven-resolver-util/src/test/java/org/eclipse/aether/util/filter/ExclusionDependencyFilterTest.java
index 1da95bb..249cc20 100644
--- 
a/maven-resolver-util/src/test/java/org/eclipse/aether/util/filter/ExclusionDependencyFilterTest.java
+++ 
b/maven-resolver-util/src/test/java/org/eclipse/aether/util/filter/ExclusionDependencyFilterTest.java
@@ -19,17 +19,16 @@ package org.eclipse.aether.util.filter;
  * under the License.
  */
 
-import static org.junit.Assert.*;
+import org.eclipse.aether.graph.DependencyNode;
+import org.eclipse.aether.internal.test.util.NodeBuilder;
+import org.junit.Test;
 
 import java.util.Arrays;
-import java.util.Collection;
 import java.util.LinkedList;
 import java.util.List;
 
-import org.eclipse.aether.graph.DependencyNode;
-import org.eclipse.aether.internal.test.util.NodeBuilder;
-import org.eclipse.aether.util.filter.ExclusionsDependencyFilter;
-import org.junit.Test;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
 
 public class ExclusionDependencyFilterTest
 {
diff --git 
a/maven-resolver-util/src/test/java/org/eclipse/aether/util/graph/transformer/SimpleConflictMarker.java
 
b/maven-resolver-util/src/test/java/org/eclipse/aether/util/graph/transformer/SimpleConflictMarker.java
index 2c596e0..024feef 100644
--- 
a/maven-resolver-util/src/test/java/org/eclipse/aether/util/graph/transformer/SimpleConflictMarker.java
+++ 
b/maven-resolver-util/src/test/java/org/eclipse/aether/util/graph/transformer/SimpleConflictMarker.java
@@ -37,7 +37,6 @@ class SimpleConflictMarker
 {
 
     public DependencyNode transformGraph( DependencyNode node, 
DependencyGraphTransformationContext context )
-        throws RepositoryException
     {
         @SuppressWarnings( "unchecked" )
         Map<DependencyNode, Object> conflictIds =
diff --git 
a/maven-resolver-util/src/test/java/org/eclipse/aether/util/version/UnionVersionRangeTest.java
 
b/maven-resolver-util/src/test/java/org/eclipse/aether/util/version/UnionVersionRangeTest.java
index 570b6b7..4193725 100644
--- 
a/maven-resolver-util/src/test/java/org/eclipse/aether/util/version/UnionVersionRangeTest.java
+++ 
b/maven-resolver-util/src/test/java/org/eclipse/aether/util/version/UnionVersionRangeTest.java
@@ -53,14 +53,7 @@ public class UnionVersionRangeTest
             assertNotNull( bound );
             assertNotNull( bound.getVersion() );
             assertEquals( inclusive, bound.isInclusive() );
-            try
-            {
-                assertEquals( new GenericVersionScheme().parseVersion( version 
), bound.getVersion() );
-            }
-            catch ( InvalidVersionSpecificationException e )
-            {
-                throw new IllegalArgumentException( e );
-            }
+            assertEquals( new GenericVersionScheme().parseVersion( version ), 
bound.getVersion() );
         }
     }
 

Reply via email to