[maven-dependency-plugin] branch master updated: remove todos and other IDE inserted boilerplate (#61)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[maven-dependency-plugin] branch master updated: remove todos and other IDE inserted boilerplate (#61)

elharo
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 05ad8b8  remove todos and other IDE inserted boilerplate (#61)
05ad8b8 is described below

commit 05ad8b8f2fc9c06fe76794ce03a428c395371564
Author: Elliotte Rusty Harold <[hidden email]>
AuthorDate: Tue Jun 30 17:55:13 2020 -0400

    remove todos and other IDE inserted boilerplate (#61)
---
 .../plugins/dependency/AbstractDependencyMojoTestCase.java     | 10 ----------
 .../testUtils/stubs/StubDefaultFileMarkerHandler.java          |  1 -
 .../plugins/dependency/testUtils/stubs/StubMarkerFile.java     |  7 -------
 .../testUtils/stubs/StubSourcesFileMarkerHandler.java          |  1 -
 4 files changed, 19 deletions(-)

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 c86f6aa..8b1c973 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java
@@ -35,11 +35,6 @@ public abstract class AbstractDependencyMojoTestCase
 
     protected DependencyArtifactStubFactory stubFactory;
 
-    public AbstractDependencyMojoTestCase()
-    {
-        super();
-    }
-
     protected void setUp( String testDirStr, boolean createFiles )
         throws Exception
     {
@@ -69,16 +64,11 @@ public abstract class AbstractDependencyMojoTestCase
             }
             catch ( IOException e )
             {
-                // TODO Auto-generated catch block
                 e.printStackTrace();
                 fail( "Trying to remove directory:" + testDir + System.lineSeparator() + e.toString() );
             }
             assertFalse( testDir.exists() );
-
-            testDir = null;
         }
-
-        stubFactory = null;
     }
 
     protected void copyFile( AbstractDependencyMojo mojo, File artifact, File destFile )
diff --git a/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubDefaultFileMarkerHandler.java b/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubDefaultFileMarkerHandler.java
index e038894..7ef4fb2 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubDefaultFileMarkerHandler.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubDefaultFileMarkerHandler.java
@@ -31,7 +31,6 @@ public class StubDefaultFileMarkerHandler
     public StubDefaultFileMarkerHandler( Artifact artifact, File markerFilesDirectory )
     {
         super( artifact, markerFilesDirectory );
-        // TODO Auto-generated constructor stub
     }
 
     protected File getMarkerFile()
diff --git a/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubMarkerFile.java b/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubMarkerFile.java
index e7e9066..02efee9 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubMarkerFile.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubMarkerFile.java
@@ -27,33 +27,26 @@ public class StubMarkerFile
     extends File
 {
 
-    /**
-     *
-     */
     private static final long serialVersionUID = 1L;
 
     public StubMarkerFile( String pathname )
     {
         super( pathname );
-        // TODO Auto-generated constructor stub
     }
 
     public StubMarkerFile( URI uri )
     {
         super( uri );
-        // TODO Auto-generated constructor stub
     }
 
     public StubMarkerFile( File parent, String child )
     {
         super( parent, child );
-        // TODO Auto-generated constructor stub
     }
 
     public StubMarkerFile( String parent, String child )
     {
         super( parent, child );
-        // TODO Auto-generated constructor stub
     }
 
     public boolean createNewFile()
diff --git a/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubSourcesFileMarkerHandler.java b/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubSourcesFileMarkerHandler.java
index 801fc48..7d836cf 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubSourcesFileMarkerHandler.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/testUtils/stubs/StubSourcesFileMarkerHandler.java
@@ -31,7 +31,6 @@ public class StubSourcesFileMarkerHandler
     public StubSourcesFileMarkerHandler( Artifact artifact, File markerFilesDirectory, boolean resolved )
     {
         super( artifact, markerFilesDirectory, resolved );
-        // TODO Auto-generated constructor stub
     }
 
     protected File getMarkerFile( boolean res )