diff --git a/pom.xml b/pom.xml index 848156e40..71091bf4d 100644 --- a/pom.xml +++ b/pom.xml @@ -99,16 +99,17 @@ under the License. - - - org.ow2.asm - asm - 8.0.1 - - + + + org.ow2.asm + asm + 8.0.1 + + + org.apache.maven @@ -116,8 +117,6 @@ under the License. ${mavenVersion} test - - org.apache.maven maven-artifact @@ -190,11 +189,6 @@ under the License. doxia-core ${doxiaVersion} - - org.apache.maven.doxia - doxia-core - ${doxiaVersion} - org.apache.maven.doxia doxia-site-renderer @@ -212,11 +206,6 @@ under the License. plexus-utils 3.3.0 - - org.apache.maven.shared - file-management - 3.0.0 - org.codehaus.plexus plexus-io diff --git a/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java b/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java index 8b1c97327..e802f9d0b 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java +++ b/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java @@ -22,9 +22,9 @@ import java.io.File; import java.io.IOException; +import org.apache.commons.io.FileUtils; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugins.dependency.testUtils.DependencyArtifactStubFactory; -import org.apache.maven.plugins.dependency.testUtils.DependencyTestUtils; import org.apache.maven.plugin.testing.AbstractMojoTestCase; public abstract class AbstractDependencyMojoTestCase @@ -48,7 +48,7 @@ protected void setUp( String testDirStr, boolean createFiles, boolean flattenedP super.setUp(); testDir = new File( getBasedir(), "target" + File.separatorChar + "unit-tests" + File.separatorChar + testDirStr + File.separatorChar ); - DependencyTestUtils.removeDirectory( testDir ); + FileUtils.deleteDirectory( testDir ); assertFalse( testDir.exists() ); stubFactory = new DependencyArtifactStubFactory( this.testDir, createFiles, flattenedPath ); @@ -60,7 +60,7 @@ protected void tearDown() { try { - DependencyTestUtils.removeDirectory( testDir ); + FileUtils.deleteDirectory( testDir ); } catch ( IOException e ) { diff --git a/src/test/java/org/apache/maven/plugins/dependency/testUtils/DependencyTestUtils.java b/src/test/java/org/apache/maven/plugins/dependency/testUtils/DependencyTestUtils.java deleted file mode 100644 index 43790f3ae..000000000 --- a/src/test/java/org/apache/maven/plugins/dependency/testUtils/DependencyTestUtils.java +++ /dev/null @@ -1,110 +0,0 @@ -package org.apache.maven.plugins.dependency.testUtils; - -/* - * Licensed to the Apache Software Foundation (ASF) under one - * or more contributor license agreements. See the NOTICE file - * distributed with this work for additional information - * regarding copyright ownership. The ASF licenses this file - * to you under the Apache License, Version 2.0 (the - * "License"); you may not use this file except in compliance - * with the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, - * software distributed under the License is distributed on an - * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY - * KIND, either express or implied. See the License for the - * specific language governing permissions and limitations - * under the License. - */ - -import static org.junit.Assert.assertTrue; - -import java.io.File; -import java.io.IOException; -import java.lang.reflect.Field; -import java.util.HashMap; - -import org.apache.maven.artifact.factory.ArtifactFactory; -import org.apache.maven.artifact.factory.DefaultArtifactFactory; -import org.apache.maven.artifact.handler.manager.ArtifactHandlerManager; -import org.apache.maven.artifact.handler.manager.DefaultArtifactHandlerManager; -import org.apache.maven.plugin.logging.Log; -import org.apache.maven.plugin.testing.SilentLog; -import org.apache.maven.shared.model.fileset.FileSet; -import org.apache.maven.shared.model.fileset.util.FileSetManager; -import org.codehaus.plexus.util.ReflectionUtils; - -public class DependencyTestUtils -{ - - /** - * Deletes a directory and its contents. - * - * @param dir {@link File} The base directory of the included and excluded files. - * @throws IOException in case of an error. When a directory failed to get deleted. - */ - public static void removeDirectory( File dir ) - throws IOException - { - if ( dir != null ) - { - Log log = new SilentLog(); - FileSetManager fileSetManager = new FileSetManager( log, false ); - - FileSet fs = new FileSet(); - fs.setDirectory( dir.getPath() ); - fs.addInclude( "**/**" ); - fileSetManager.delete( fs ); - - } - } - - public static ArtifactFactory getArtifactFactory() - throws IllegalAccessException - { - ArtifactFactory artifactFactory; - ArtifactHandlerManager manager = new DefaultArtifactHandlerManager(); - setVariableValueToObject( manager, "artifactHandlers", new HashMap<>() ); - - artifactFactory = new DefaultArtifactFactory(); - setVariableValueToObject( artifactFactory, "artifactHandlerManager", manager ); - - return artifactFactory; - } - - /** - * convenience method to set values to variables in objects that don't have setters - * - * @param object {@link Object} - * @param variable the field name. - * @param value The value to be set. - * @throws IllegalAccessException in case of an error. - */ - public static void setVariableValueToObject( Object object, String variable, Object value ) - throws IllegalAccessException - { - Field field = ReflectionUtils.getFieldByNameIncludingSuperclasses( variable, object.getClass() ); - - field.setAccessible( true ); - - field.set( object, value ); - } - - public static void setFileModifiedTime( File file ) - throws InterruptedException - { - Thread.sleep( 100 ); - // round down to the last second - long time = System.currentTimeMillis(); - time = time - ( time % 1000 ); - assertTrue( "Updating last modification time of marker file " + file.getAbsolutePath() - + " failed unexpectedly.", file.setLastModified( time ) ); - - // wait at least a second for filesystems that only record to the - // nearest second. - Thread.sleep( 1000 ); - } - -} diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java index 76655b02a..86dca0c48 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java +++ b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java @@ -26,10 +26,10 @@ import junit.framework.TestCase; +import org.apache.commons.io.FileUtils; import org.apache.maven.artifact.Artifact; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugins.dependency.testUtils.DependencyArtifactStubFactory; -import org.apache.maven.plugins.dependency.testUtils.DependencyTestUtils; import org.apache.maven.plugins.dependency.utils.DependencyUtil; import org.apache.maven.plugin.logging.Log; import org.apache.maven.plugin.testing.SilentLog; @@ -55,7 +55,7 @@ protected void setUp() super.setUp(); outputFolder = new File( "target/markers/" ); - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); assertFalse( outputFolder.exists() ); this.fact = new DependencyArtifactStubFactory( outputFolder, false ); @@ -65,7 +65,7 @@ protected void setUp() protected void tearDown() throws IOException { - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); } public File createFile( Artifact artifact ) diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestMarkerFileFilter.java b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestMarkerFileFilter.java index cb3168a1e..8069c9d50 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestMarkerFileFilter.java +++ b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestMarkerFileFilter.java @@ -26,10 +26,10 @@ import junit.framework.TestCase; +import org.apache.commons.io.FileUtils; import org.apache.maven.artifact.Artifact; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugins.dependency.testUtils.DependencyArtifactStubFactory; -import org.apache.maven.plugins.dependency.testUtils.DependencyTestUtils; import org.apache.maven.plugins.dependency.utils.markers.DefaultFileMarkerHandler; import org.apache.maven.plugin.logging.Log; import org.apache.maven.plugin.testing.SilentLog; @@ -55,7 +55,7 @@ protected void setUp() super.setUp(); outputFolder = new File( "target/markers/" ); - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); assertFalse( outputFolder.exists() ); this.fact = new DependencyArtifactStubFactory( outputFolder, false ); @@ -65,7 +65,7 @@ protected void setUp() protected void tearDown() throws IOException { - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); } public void testMarkerFile() @@ -99,7 +99,7 @@ public void testMarkerSnapshots() result = filter.filter( artifacts ); assertEquals( 2, result.size() ); assertTrue( handler.clearMarker() ); - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); assertFalse( outputFolder.exists() ); } @@ -119,7 +119,7 @@ public void testMarkerRelease() assertEquals( 2, result.size() ); assertTrue( handler.clearMarker() ); - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); assertFalse( outputFolder.exists() ); } @@ -157,7 +157,7 @@ public void testMarkerTimestamp() assertFalse( handler.isMarkerSet() ); snap.getFile().delete(); release.getFile().delete(); - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); assertFalse( outputFolder.exists() ); } diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestResolveMarkerFileFilter.java b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestResolveMarkerFileFilter.java index 76f60981d..4c36fe438 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestResolveMarkerFileFilter.java +++ b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestResolveMarkerFileFilter.java @@ -29,10 +29,10 @@ import junit.framework.TestCase; +import org.apache.commons.io.FileUtils; import org.apache.maven.artifact.Artifact; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugins.dependency.testUtils.DependencyArtifactStubFactory; -import org.apache.maven.plugins.dependency.testUtils.DependencyTestUtils; import org.apache.maven.plugins.dependency.utils.markers.SourcesFileMarkerHandler; import org.apache.maven.plugin.logging.Log; import org.apache.maven.plugin.testing.SilentLog; @@ -58,7 +58,7 @@ protected void setUp() super.setUp(); outputFolder = new File( "target/markers/" ); - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); assertFalse( outputFolder.exists() ); this.fact = new DependencyArtifactStubFactory( outputFolder, false ); @@ -68,7 +68,7 @@ protected void setUp() protected void tearDown() throws IOException { - DependencyTestUtils.removeDirectory( outputFolder ); + FileUtils.deleteDirectory( outputFolder ); } public void testResolveFile() diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestDefaultMarkerFileHandler.java b/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestDefaultMarkerFileHandler.java index 4704511b1..4d328cba9 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestDefaultMarkerFileHandler.java +++ b/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestDefaultMarkerFileHandler.java @@ -26,13 +26,13 @@ import junit.framework.TestCase; +import org.apache.commons.io.FileUtils; import org.apache.maven.artifact.Artifact; import org.apache.maven.artifact.DefaultArtifact; import org.apache.maven.artifact.handler.ArtifactHandler; import org.apache.maven.artifact.handler.DefaultArtifactHandler; import org.apache.maven.artifact.versioning.VersionRange; import org.apache.maven.plugin.MojoExecutionException; -import org.apache.maven.plugins.dependency.testUtils.DependencyTestUtils; import org.apache.maven.plugins.dependency.testUtils.stubs.StubDefaultFileMarkerHandler; import org.apache.maven.plugin.logging.Log; import org.apache.maven.plugin.testing.SilentLog; @@ -66,14 +66,14 @@ protected void setUp() artifacts.add( artifact ); outputFolder = new File( "target/markers/" ); - DependencyTestUtils.removeDirectory( this.outputFolder ); + FileUtils.deleteDirectory( this.outputFolder ); assertFalse( outputFolder.exists() ); } protected void tearDown() throws IOException { - DependencyTestUtils.removeDirectory( this.outputFolder ); + FileUtils.deleteDirectory( this.outputFolder ); } public void testSetMarker() diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestUnpackMarkerFileHandler.java b/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestUnpackMarkerFileHandler.java index b9036667c..8501b217f 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestUnpackMarkerFileHandler.java +++ b/src/test/java/org/apache/maven/plugins/dependency/utils/markers/TestUnpackMarkerFileHandler.java @@ -24,11 +24,11 @@ import java.util.ArrayList; import java.util.List; +import org.apache.commons.io.FileUtils; import org.apache.maven.artifact.Artifact; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugins.dependency.fromConfiguration.ArtifactItem; import org.apache.maven.plugins.dependency.testUtils.DependencyArtifactStubFactory; -import org.apache.maven.plugins.dependency.testUtils.DependencyTestUtils; import org.apache.maven.plugins.dependency.testUtils.stubs.StubUnpackFileMarkerHandler; import org.apache.maven.plugin.logging.Log; import org.apache.maven.plugin.testing.AbstractMojoTestCase; @@ -54,7 +54,7 @@ protected void setUp() testDir = new File( getBasedir(), "target" + File.separatorChar + "unit-tests" + File.separatorChar + "unpack-markers" + File.separatorChar ); - DependencyTestUtils.removeDirectory( testDir ); + FileUtils.deleteDirectory( testDir ); assertFalse( testDir.exists() ); stubFactory = new DependencyArtifactStubFactory( this.testDir, false ); @@ -77,14 +77,14 @@ protected void setUp() artifactItems.add( artifactItem ); outputFolder = new File( "target/markers/" ); - DependencyTestUtils.removeDirectory( this.outputFolder ); + FileUtils.deleteDirectory( this.outputFolder ); assertFalse( outputFolder.exists() ); } protected void tearDown() throws IOException { - DependencyTestUtils.removeDirectory( this.outputFolder ); + FileUtils.deleteDirectory( this.outputFolder ); } /**