diff --git a/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/LegacyPathParser.java b/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/LegacyPathParser.java index e1d5eaba3..e62d1516d 100644 --- a/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/LegacyPathParser.java +++ b/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/LegacyPathParser.java @@ -204,13 +204,6 @@ public ArtifactReference toArtifactReference( String path ) + "] and layout specified type [" + expectedType + "] (which maps to extension: [" + expectedExtension + "]) on path [" + path + "]" ); } - -// if ( classifier != null && !filename.endsWith( "-" + classifier + "." + extension ) ) -// { -// throw new LayoutException( INVALID_ARTIFACT_PATH + "mismatch on filename [" + filename -// + "] and layout specified type [" + expectedType + "] (which maps to classifier: [" -// + classifier + "]) on path [" + path + "]" ); -// } } return artifact; diff --git a/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/LegacyPathParserTest.java b/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/LegacyPathParserTest.java index 0267e2aba..c86d66ff1 100644 --- a/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/LegacyPathParserTest.java +++ b/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/LegacyPathParserTest.java @@ -239,17 +239,9 @@ public void testGoodFooLibSources() public void testBadClassifierFooLibSources() throws LayoutException { - String path = "com.foo.lib/java-sources/foo-lib-2.1-alpha-1.jar"; - - try - { - parser.toArtifactReference( path ); - fail( "Expected an exception" ); - } - catch ( LayoutException e ) - { - assertTrue( true ); - } + assertBadPath( "com.foo.lib/java-sources/foo-lib-2.1-alpha-1.jar", "missing required classifier" ); + assertBadPath( "com.foo.lib/java-sources/foo-lib-2.1-alpha-1-javadoc.jar", "incorrect classifier" ); + assertBadPath( "com.foo.lib/java-sources/foo-lib-2.1-alpha-1-other.jar", "incorrect classifier" ); } /**