[MDEP-923] Extract copyFile method from AbstractDependencyMojo#389
[MDEP-923] Extract copyFile method from AbstractDependencyMojo#389slawekjaranowski merged 4 commits intomasterfrom
Conversation
src/main/java/org/apache/maven/plugins/dependency/utils/CopyUtil.java
Outdated
Show resolved
Hide resolved
src/main/java/org/apache/maven/plugins/dependency/utils/CopyUtil.java
Outdated
Show resolved
Hide resolved
slawekjaranowski
force-pushed
the
MDEP-923-copyFile
branch
from
7b3f845 to
8fcd200
Compare
| } | ||
|
|
||
| public void testCopyFile() throws MojoExecutionException, IOException { | ||
| public void testCopyFile() throws Exception { |
There was a problem hiding this comment.
did this change to throw a raw exception? Otherwise it's better to declare the actual exceptions instead of the common superclass.
There was a problem hiding this comment.
It is a Unit Test ... all other methods simply throws Exception
There was a problem hiding this comment.
I like to know when an exception is added or removed from an API, and I'd make the same comment about the other methods in this class. :-)
There was a problem hiding this comment.
ok, one exception is new in 3.7.0 but one is old from copied class - so should I discovered versions for original class or add since now
src/main/java/org/apache/maven/plugins/dependency/utils/CopyUtil.java
Outdated
Show resolved
Hide resolved
this method is only needed in CopyMojo and CopyDependenciesMojo so it is not needed in AbstractDependencyMojo
slawekjaranowski
force-pushed
the
MDEP-923-copyFile
branch
from
8fcd200 to
bf0081e
Compare
| } | ||
|
|
||
| public void testCopyFile() throws MojoExecutionException, IOException { | ||
| public void testCopyFile() throws Exception { |
There was a problem hiding this comment.
I like to know when an exception is added or removed from an API, and I'd make the same comment about the other methods in this class. :-)
|
Resolve #1420 |
3 participants
this method is only needed in CopyMojo and CopyDependenciesMojo so it is not needed in AbstractDependencyMojo