You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sj...@apache.org on 2022/08/30 20:41:06 UTC

[maven-dependency-plugin] 03/04: [MDEP-782] Remove todo comments

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

sjaranowski pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-dependency-plugin.git

commit 00cfb8810d0d611c7095bc49e2e0f990c6489a5f
Author: Jasper Kamerling <ja...@alliander.com>
AuthorDate: Tue Aug 30 10:24:01 2022 +0200

    [MDEP-782] Remove todo comments
---
 .../plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java  | 2 --
 .../org/apache/maven/plugins/dependency/utils/TestDependencyUtil.java   | 1 -
 2 files changed, 3 deletions(-)

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 d88415ce..61b282d4 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
@@ -391,7 +391,6 @@ public class TestCopyDependenciesMojo2
         Set<Artifact> artifacts = mojo.getProject().getArtifacts();
         for ( Artifact artifact : artifacts )
         {
-            // TODO fix test
             String fileName = DependencyUtil.getFormattedFileName( artifact, false );
             File folder = DependencyUtil.getFormattedOutputDirectory( false, false, true, false, false, true,
                 mojo.outputDirectory, artifact );
@@ -414,7 +413,6 @@ public class TestCopyDependenciesMojo2
         Set<Artifact> artifacts = mojo.getProject().getArtifacts();
         for ( Artifact artifact : artifacts )
         {
-            // TODO fix test
             String fileName = DependencyUtil.getFormattedFileName( artifact, false );
             File folder = DependencyUtil.getFormattedOutputDirectory( false, true, true, false, false, true,
                                                                       mojo.outputDirectory, artifact );
diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyUtil.java b/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyUtil.java
index 86f71790..c9cb4c19 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyUtil.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyUtil.java
@@ -163,7 +163,6 @@ public class TestDependencyUtil
         assertEquals( expectedResult, name.getAbsolutePath() );
     }
 
-    // TODO fix test
     public void testDirectoryName2()
     {
         File folder = new File( "target/a" );