Using JUnit 4 for TestCopyMojo to enable Assumptions

Author: mkarg

src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestCopyMojo.java

@@ -36,13 +36,19 @@
 import org.apache.maven.plugins.dependency.AbstractDependencyMojoTestCase;
 import org.apache.maven.plugins.dependency.utils.DependencyUtil;
 import org.apache.maven.project.MavenProject;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.BlockJUnit4ClassRunner;
 
+@RunWith(BlockJUnit4ClassRunner.class)
 public class TestCopyMojo
     extends AbstractDependencyMojoTestCase
 {
     private CopyMojo mojo;
 
-    protected void setUp()
+    @Before
+    public void setUp()
         throws Exception
     {
         super.setUp( "copy", false, false );
@@ -84,6 +90,7 @@ public void testSetArtifactWithoutPackaging()
         assertNull( item.getClassifier() );
     }
 
+    @Test
     public void testSetArtifactWithoutClassifier()
         throws Exception
     {
@@ -96,6 +103,7 @@ public void testSetArtifactWithoutClassifier()
         assertNull( item.getClassifier() );
     }
 
+    @Test
     public void testSetArtifact()
         throws Exception
     {
@@ -108,6 +116,7 @@ public void testSetArtifact()
         assertEquals( "e", item.getClassifier() );
     }
 
+    @Test
     public void testGetArtifactItems()
         throws Exception
     {
@@ -160,6 +169,7 @@ public void assertFileIsLink( ArtifactItem item, boolean isLink )
         assertEquals( isLink, Files.isSymbolicLink( path ) );
     }
 
+    @Test
     public void testMojoDefaults()
     {
         CopyMojo themojo = new CopyMojo();
@@ -169,6 +179,7 @@ public void testMojoDefaults()
         assertFalse( themojo.isStripClassifier() );
     }
 
+    @Test
     public void testCopyFile()
         throws Exception
     {
@@ -181,6 +192,7 @@ public void testCopyFile()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testCopyFileWithBaseVersion()
         throws Exception
     {
@@ -200,6 +212,7 @@ public void testCopyFileWithBaseVersion()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testSkip()
         throws Exception
     {
@@ -218,6 +231,7 @@ public void testSkip()
 
     }
 
+    @Test
     public void testCopyFileNoOverwrite()
         throws Exception
     {
@@ -235,6 +249,7 @@ public void testCopyFileNoOverwrite()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testCopyToLocation()
         throws Exception
     {
@@ -249,6 +264,7 @@ public void testCopyToLocation()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testLink()
         throws Exception
     {
@@ -263,6 +279,7 @@ public void testLink()
         assertFilesAreLinks( list, true );
     }
 
+    @Test
     public void testCopyStripVersionSetInMojo()
         throws Exception
     {
@@ -280,6 +297,7 @@ public void testCopyStripVersionSetInMojo()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testCopyStripClassifierSetInMojo()
         throws Exception
     {
@@ -298,6 +316,7 @@ public void testCopyStripClassifierSetInMojo()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testNonClassifierStrip()
         throws Exception
     {
@@ -310,6 +329,7 @@ public void testNonClassifierStrip()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testNonClassifierNoStrip()
         throws Exception
     {
@@ -322,6 +342,7 @@ public void testNonClassifierNoStrip()
         assertFilesExist( list, true );
     }
 
+    @Test
     public void testMissingVersionNotFound()
         throws Exception
     {
@@ -370,6 +391,7 @@ public List<Dependency> getDependencyList( ArtifactItem item )
         return list;
     }
 
+    @Test
     public void testMissingVersionFromDependencies()
         throws Exception
     {
@@ -392,6 +414,7 @@ public void testMissingVersionFromDependencies()
         assertEquals( "2.0-SNAPSHOT", item.getVersion() );
     }
 
+    @Test
     public void testMissingVersionFromDependenciesLooseMatch()
         throws Exception
     {
@@ -423,6 +446,7 @@ public void testMissingVersionFromDependenciesLooseMatch()
         assertEquals( "2.1", item.getVersion() );
     }
 
+    @Test
     public void testMissingVersionFromDependenciesWithClassifier()
         throws Exception
     {
@@ -468,6 +492,7 @@ public List<Dependency> getDependencyMgtList( ArtifactItem item )
         return list;
     }
 
+    @Test
     public void testMissingVersionFromDependencyMgt()
         throws Exception
     {
@@ -501,6 +526,7 @@ public void testMissingVersionFromDependencyMgt()
         assertEquals( "3.0-SNAPSHOT", item.getVersion() );
     }
 
+    @Test
     public void testMissingVersionFromDependencyMgtLooseMatch()
         throws Exception
     {
@@ -541,6 +567,7 @@ public void testMissingVersionFromDependencyMgtLooseMatch()
         assertEquals( "3.1", item.getVersion() );
     }
 
+    @Test
     public void testMissingVersionFromDependencyMgtWithClassifier()
         throws Exception
     {
@@ -574,12 +601,14 @@ public void testMissingVersionFromDependencyMgtWithClassifier()
         assertEquals( "3.1", item.getVersion() );
     }
 
+    @Test
     public void testArtifactNotFound()
         throws Exception
     {
         dotestArtifactExceptions( false, true );
     }
 
+    @Test
     public void testArtifactResolutionException()
         throws Exception
     {
@@ -612,6 +641,7 @@ public void dotestArtifactExceptions( boolean are, boolean anfe )
         }
     }
 
+    @Test
     public void testNoArtifactItems()
     {
         try
@@ -626,6 +656,7 @@ public void testNoArtifactItems()
 
     }
 
+    @Test
     public void testCopyDontOverWriteReleases()
         throws Exception
     {
@@ -657,6 +688,7 @@ public void testCopyDontOverWriteReleases()
         assertEquals( time, copiedFile.lastModified() );
     }
 
+    @Test
     public void testCopyDontOverWriteSnapshots()
         throws Exception
     {
@@ -688,6 +720,7 @@ public void testCopyDontOverWriteSnapshots()
         assertEquals( time, copiedFile.lastModified() );
     }
 
+    @Test
     public void testCopyOverWriteReleases()
         throws Exception
     {
@@ -717,6 +750,7 @@ public void testCopyOverWriteReleases()
         assertEquals( 1000L, timeCopyNow );
     }
 
+    @Test
     public void testCopyOverWriteSnapshot()
         throws Exception
     {
@@ -747,6 +781,7 @@ public void testCopyOverWriteSnapshot()
         assertEquals( 1000L, timeCopyNow );
     }
 
+    @Test
     public void testCopyOverWriteIfNewer()
         throws Exception
     {
@@ -776,6 +811,7 @@ public void testCopyOverWriteIfNewer()
         assertTrue( time < copiedFile.lastModified() );
     }
 
+    @Test
     public void testCopyFileWithOverideLocalRepo()
         throws Exception
     {