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() ); } }