[maven-dependency-plugin] branch ver created (now 8b0c75b)

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[maven-dependency-plugin] branch ver created (now 8b0c75b)

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

elharo pushed a change to branch ver
in repository https://gitbox.apache.org/repos/asf/maven-dependency-plugin.git.


      at 8b0c75b  remove todos and other IDE inserted boilerplate

This branch includes the following new commits:

     new 8b0c75b  remove todos and other IDE inserted boilerplate

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Reply | Threaded
Open this post in threaded view
|

[maven-dependency-plugin] 01/01: remove todos and other IDE inserted boilerplate

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

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

commit 8b0c75b10114b248180d72cef6387461140fea15
Author: Elliotte Rusty Harold <[hidden email]>
AuthorDate: Tue Jun 30 12:39:21 2020 -0400

    remove todos and other IDE inserted boilerplate
---
 .../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 )