chore: migrate junit3/4 to junit 5

pom.xml

@@ -206,12 +206,15 @@ under the License.
       <artifactId>commons-lang3</artifactId>
       <version>3.19.0</version>
     </dependency>
-
     <!-- test -->
     <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>4.13.2</version>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-api</artifactId>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.junit.vintage</groupId>
+      <artifactId>junit-vintage-engine</artifactId>
       <scope>test</scope>
     </dependency>
     <dependency>

src/it/MCHECKSTYLE-129/pom.xml

@@ -25,14 +25,6 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>checkstyle-new</name>
   <url>http://maven.apache.org</url>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
-  </dependencies>
   <build>
     <plugins>
       <plugin>

src/it/MCHECKSTYLE-129/src/test/java/com/example/AppTest.java

@@ -19,39 +19,8 @@
  * under the License.
  */
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
 
-/**
- * Unit test for simple App.
- */
+
 public class AppTest 
-    extends TestCase
 {
-    /**
-     * Create the test case
-     *
-     * @param testName name of the test case
-     */
-    public AppTest( String testName )
-    {
-        super( testName );
-    }
-
-    /**
-     * @return the suite of tests being tested
-     */
-    public static Test suite()
-    {
-        return new TestSuite( AppTest.class );
-    }
-
-    /**
-     * Rigourous Test :-)
-     */
-    public void testApp()
-    {
-        assertTrue( true );
-    }
 }

src/it/MCHECKSTYLE-131/pom.xml

@@ -26,14 +26,6 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>Example</name>
   <url>http://maven.apache.org</url>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
-  </dependencies>
   <modules>
     <module>module</module>
   </modules>

src/it/MCHECKSTYLE-99-custom-xref-test-location/pom.xml

@@ -25,14 +25,6 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>checkstyle-new</name>
   <url>http://maven.apache.org</url>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
-  </dependencies>
   <build>
     <pluginManagement>
       <plugins>

src/it/MCHECKSTYLE-99-custom-xref-test-location/src/test/java/com/example/AppTest.java

@@ -19,39 +19,6 @@
  * under the License.
  */
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
-
-/**
- * Unit test for simple App.
- */
-public class AppTest 
-    extends TestCase
+public class AppTest
 {
-    /**
-     * Create the test case
-     *
-     * @param testName name of the test case
-     */
-    public AppTest( String testName )
-    {
-        super( testName );
-    }
-
-    /**
-     * @return the suite of tests being tested
-     */
-    public static Test suite()
-    {
-        return new TestSuite( AppTest.class );
-    }
-
-    /**
-     * Rigourous Test :-)
-     */
-    public void testApp()
-    {
-        assertTrue( true );
-    }
-}
+}

src/it/MCHECKSTYLE-99-custom-xref-test-location/verify.groovy

@@ -21,6 +21,6 @@
 report = new File( basedir, 'target/reports/checkstyle.html' )
 assert report.exists();
 assert report.text.contains( '<a href="./xref/com/example/App.html#L26">' )
-assert report.text.contains( '<a href="./custom-xref-test-location/com/example/AppTest.html#L32">' )
+assert report.text.contains( '<a href="./custom-xref-test-location/com/example/AppTest.html#L23">' )
 
 return true;

src/it/MCHECKSTYLE-99/pom.xml

@@ -25,14 +25,6 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>checkstyle-new</name>
   <url>http://maven.apache.org</url>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
-  </dependencies>
   <build>
     <pluginManagement>
       <plugins>

src/it/MCHECKSTYLE-99/src/test/java/com/example/AppTest.java

@@ -19,39 +19,6 @@
  * under the License.
  */
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
-
-/**
- * Unit test for simple App.
- */
-public class AppTest 
-    extends TestCase
+public class AppTest
 {
-    /**
-     * Create the test case
-     *
-     * @param testName name of the test case
-     */
-    public AppTest( String testName )
-    {
-        super( testName );
-    }
-
-    /**
-     * @return the suite of tests being tested
-     */
-    public static Test suite()
-    {
-        return new TestSuite( AppTest.class );
-    }
-
-    /**
-     * Rigourous Test :-)
-     */
-    public void testApp()
-    {
-        assertTrue( true );
-    }
 }

src/it/MCHECKSTYLE-99/verify.groovy

@@ -21,6 +21,6 @@
 report = new File( basedir, 'target/reports/checkstyle.html' )
 assert report.exists();
 assert report.text.contains( '<a href="./xref/com/example/App.html#L26">' )
-assert report.text.contains( '<a href="./xref-test/com/example/AppTest.html#L32">' )
+assert report.text.contains( '<a href="./xref-test/com/example/AppTest.html#L23">' )
 
 return true;

src/it/multi-modules-aggregate/child-b/pom.xml

@@ -29,12 +29,13 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>multi-modules-child-nosource</name>
   <url>http://maven.apache.org</url>
+
   <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
+      <dependency>
+          <groupId>org.junit.jupiter</groupId>
+          <artifactId>junit-jupiter-api</artifactId>
+          <version>5.13.4</version>
+          <scope>test</scope>
+      </dependency>
   </dependencies>
 </project>

src/it/multi-modules-aggregate/child-b/src/test/java/org/apache/maven/plugins/checkstyle/its/AppTest.java

@@ -1,36 +1,10 @@
 package org.apache.maven.plugins.checkstyle.its;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
+import org.junit.jupiter.api.Test;
 
-/**
- * Unit test for simple App.
- */
-public class AppTest 
-    extends TestCase
+public class AppTest
 {
-    /**
-     * Create the test case
-     *
-     * @param testName name of the test case
-     */
-    public AppTest( String testName )
-    {
-        super( testName );
-    }
-
-    /* missing javadoc */
-    public static Test suite()
-    {
-        return new TestSuite( AppTest.class );
-    }
-
-    /**
-     * Rigourous Test :-)
-     */
+    @Test
     public void testApp()
-    {
-        assertTrue( true );
-    }
+    {}
 }

src/it/multi-modules/multi-modules-child-nosource/pom.xml

@@ -29,12 +29,4 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>multi-modules-child-nosource</name>
   <url>http://maven.apache.org</url>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
-  </dependencies>
 </project>

src/it/multi-modules/multi-modules-child-nosource/src/test/java/org/apache/maven/plugins/checkstyle/its/AppTest.java

@@ -19,34 +19,6 @@
  * under the License.
  */
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
-
-/**
- * Unit test for simple App.
- */
-public class AppTest extends TestCase {
-    /**
-     * Create the test case
-     *
-     * @param testName name of the test case
-     */
-    public AppTest(String testName) {
-        super(testName);
-    }
-
-    /**
-     * @return the suite of tests being tested
-     */
-    public static Test suite() {
-        return new TestSuite(AppTest.class);
-    }
-
-    /**
-     * Rigourous Test :-)
-     */
-    public void testApp() {
-        assertTrue(true);
-    }
-}
+public class AppTest
+{
+}

src/it/multi-modules/multi-modules-child/pom.xml

@@ -29,14 +29,6 @@ under the License.
   <version>1.0-SNAPSHOT</version>
   <name>multi-modules-child</name>
   <url>http://maven.apache.org</url>
-  <dependencies>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>3.8.1</version>
-      <scope>test</scope>
-    </dependency>
-  </dependencies>
   <build>
     <pluginManagement>
       <plugins>

src/it/multi-modules/multi-modules-child/src/test/java/org/apache/maven/plugins/checkstyle/its/AppTest.java

@@ -19,34 +19,6 @@
  * under the License.
  */
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-import junit.framework.TestSuite;
-
-/**
- * Unit test for simple App.
- */
-public class AppTest extends TestCase {
-    /**
-     * Create the test case
-     *
-     * @param testName name of the test case
-     */
-    public AppTest(String testName) {
-        super(testName);
-    }
-
-    /**
-     * @return the suite of tests being tested
-     */
-    public static Test suite() {
-        return new TestSuite(AppTest.class);
-    }
-
-    /**
-     * Rigourous Test :-)
-     */
-    public void testApp() {
-        assertTrue(true);
-    }
-}
+public class AppTest
+{
+}