This is an automated email from the ASF dual-hosted git repository. elharo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/maven-dependency-plugin.git
The following commit(s) were added to refs/heads/master by this push: new d77623db Remove obsolete commented out code d77623db is described below commit d77623dbe1a8b5e1e5f7579164b6b82a00f1bf7f Author: Elliotte Rusty Harold <elh...@ibiblio.org> AuthorDate: Sun Nov 10 12:15:31 2024 -0500 Remove obsolete commented out code --- .../dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java | 1 - .../plugins/dependency/fromDependencies/TestBuildClasspathMojo.java | 1 - .../plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java | 3 --- .../plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java | 1 - .../fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java | 1 - .../dependency/fromDependencies/TestUnpackDependenciesMojo.java | 3 --- .../dependency/fromDependencies/TestUnpackDependenciesMojo2.java | 1 - 7 files changed, 11 deletions(-) diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java index 5778f0e2..02c3644c 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java @@ -56,7 +56,6 @@ public class TestIncludeExcludeUnpackMojo extends AbstractDependencyMojoTestCase File testPom = new File(getBasedir(), "target/test-classes/unit/unpack-test/plugin-config.xml"); mojo = (UnpackMojo) lookupMojo("unpack", testPom); mojo.setOutputDirectory(new File(this.testDir, "outputDirectory")); - // mojo.silent = true; // i'm using one file repeatedly to archive so I can test the name // programmatically. diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java index 74d80b5b..f310ec0d 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java @@ -56,7 +56,6 @@ public class TestBuildClasspathMojo extends AbstractDependencyMojoTestCase { public void testEnvironment() throws Exception { MavenProject project = mojo.getProject(); - // mojo.silent = true; Set<Artifact> artifacts = this.stubFactory.getScopedArtifacts(); Set<Artifact> directArtifacts = this.stubFactory.getReleaseAndSnapshotArtifacts(); artifacts.addAll(directArtifacts); diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java index 724d30fb..91ebbd46 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java @@ -61,7 +61,6 @@ public class TestCopyDependenciesMojo extends AbstractDependencyMojoTestCase { File testPom = new File(getBasedir(), "target/test-classes/unit/copy-dependencies-test/plugin-config.xml"); mojo = (CopyDependenciesMojo) lookupMojo("copy-dependencies", testPom); mojo.outputDirectory = new File(this.testDir, "outputDirectory"); - // mojo.silent = true; assertNotNull(mojo); assertNotNull(mojo.getProject()); @@ -572,7 +571,6 @@ public class TestCopyDependenciesMojo extends AbstractDependencyMojoTestCase { mojo.getProject().setArtifacts(stubFactory.getScopedArtifacts()); mojo.getProject().setDependencyArtifacts(new HashSet<>()); mojo.excludeScope = "provided"; - // mojo.silent = false; mojo.execute(); @@ -590,7 +588,6 @@ public class TestCopyDependenciesMojo extends AbstractDependencyMojoTestCase { mojo.getProject().setArtifacts(stubFactory.getScopedArtifacts()); mojo.getProject().setDependencyArtifacts(new HashSet<>()); mojo.excludeScope = "system"; - // mojo.silent = false; mojo.execute(); diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java index 5a299837..f3a1f8b4 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java @@ -69,7 +69,6 @@ public class TestCopyDependenciesMojo2 extends AbstractDependencyMojoTestCase { File testPom = new File(getBasedir(), "target/test-classes/unit/copy-dependencies-test/plugin-config.xml"); mojo = (CopyDependenciesMojo) lookupMojo("copy-dependencies", testPom); mojo.outputDirectory = new File(this.testDir, "outputDirectory"); - // mojo.silent = true; assertNotNull(mojo); assertNotNull(mojo.getProject()); diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java index 8195ae16..0617b875 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java @@ -51,7 +51,6 @@ public class TestIncludeExcludeUnpackDependenciesMojo extends AbstractDependency File testPom = new File(getBasedir(), "target/test-classes/unit/unpack-dependencies-test/plugin-config.xml"); mojo = (UnpackDependenciesMojo) lookupMojo("unpack-dependencies", testPom); mojo.outputDirectory = new File(this.testDir, "outputDirectory"); - // mojo.silent = true; // it needs to get the archivermanager // stubFactory.setUnpackableFile( mojo.getArchiverManager() ); diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java index 463f8f9b..ef795a2e 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java @@ -67,7 +67,6 @@ public class TestUnpackDependenciesMojo extends AbstractDependencyMojoTestCase { File testPom = new File(getBasedir(), "target/test-classes/unit/unpack-dependencies-test/plugin-config.xml"); mojo = (UnpackDependenciesMojo) lookupMojo("unpack-dependencies", testPom); mojo.outputDirectory = new File(this.testDir, "outputDirectory"); - // mojo.silent = true; // it needs to get the archivermanager stubFactory.setUnpackableFile(lookup(ArchiverManager.class)); @@ -161,7 +160,6 @@ public class TestUnpackDependenciesMojo extends AbstractDependencyMojoTestCase { mojo.getProject().setArtifacts(stubFactory.getScopedArtifacts()); mojo.getProject().setDependencyArtifacts(new HashSet<>()); mojo.excludeScope = "provided"; - // mojo.silent = false; mojo.execute(); @@ -174,7 +172,6 @@ public class TestUnpackDependenciesMojo extends AbstractDependencyMojoTestCase { mojo.getProject().setArtifacts(stubFactory.getScopedArtifacts()); mojo.getProject().setDependencyArtifacts(new HashSet<>()); mojo.excludeScope = "system"; - // mojo.silent = false; mojo.execute(); diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java index 70941f9c..cba5392f 100644 --- a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java +++ b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java @@ -55,7 +55,6 @@ public class TestUnpackDependenciesMojo2 extends AbstractDependencyMojoTestCase File testPom = new File(getBasedir(), "target/test-classes/unit/unpack-dependencies-test/plugin-config.xml"); mojo = (UnpackDependenciesMojo) lookupMojo("unpack-dependencies", testPom); mojo.outputDirectory = new File(this.testDir, "outputDirectory"); - // mojo.silent = true; // it needs to get the archivermanager stubFactory.setUnpackableFile(lookup(ArchiverManager.class));