Remove the last references to JUnit 4 (#12442)

Motivation: This completes our migration to JUnit 5. Modification: Remove all JUnit 4 dependencies. Replace or remove the last references to JUnit 4 types in the code. Add a revapi exception because a public TestUtils method was exposing JUnit 4 types and had to be removed. Result: No more JUnit 4 in our project. Fixes #10757
netty · Jun 3, 2022 · 2afdf0b · 2afdf0b
1 parent 3bec8e0
commit 2afdf0b
Show file tree

Hide file tree

Showing 9 changed files with 5 additions and 85 deletions.
diff --git a/microbench/pom.xml b/microbench/pom.xml
@@ -170,16 +170,6 @@
       <artifactId>junit-jupiter-engine</artifactId>
       <scope>compile</scope>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-      <scope>compile</scope>
-    </dependency>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <scope>compile</scope>
-    </dependency>
     <dependency>
       <groupId>org.openjdk.jmh</groupId>
       <artifactId>jmh-core</artifactId>

diff --git a/pom.xml b/pom.xml
@@ -779,18 +779,6 @@
         <version>${junit.version}</version>
         <scope>test</scope>
       </dependency>
-      <dependency>
-        <groupId>org.junit.vintage</groupId>
-        <artifactId>junit-vintage-engine</artifactId>
-        <version>${junit.version}</version>
-        <scope>test</scope>
-      </dependency>
-      <dependency>
-        <groupId>junit</groupId>
-        <artifactId>junit</artifactId>
-        <version>4.13.1</version>
-        <scope>test</scope>
-      </dependency>
       <dependency>
         <groupId>io.netty</groupId>
         <artifactId>netty-build-common</artifactId>
@@ -909,16 +897,6 @@
       <artifactId>junit-jupiter-params</artifactId>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-      <scope>test</scope>
-    </dependency>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <scope>test</scope>
-    </dependency>
     <dependency>
       <groupId>io.netty</groupId>
       <artifactId>netty-build-common</artifactId>

diff --git a/testsuite-native/pom.xml b/testsuite-native/pom.xml
@@ -63,14 +63,6 @@
       <groupId>org.junit.jupiter</groupId>
       <artifactId>junit-jupiter-engine</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-    </dependency>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-    </dependency>
   </dependencies>
 
   <profiles>

diff --git a/testsuite-shading/pom.xml b/testsuite-shading/pom.xml
@@ -211,14 +211,6 @@
       <groupId>org.junit.jupiter</groupId>
       <artifactId>junit-jupiter-engine</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-    </dependency>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-    </dependency>
   </dependencies>
   <profiles>
     <profile>

diff --git a/testsuite/pom.xml b/testsuite/pom.xml
@@ -84,16 +84,6 @@
       <artifactId>junit-jupiter-params</artifactId>
       <scope>compile</scope>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-      <scope>compile</scope>
-    </dependency>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <scope>compile</scope>
-    </dependency>
     <dependency>
       <groupId>org.hamcrest</groupId>
       <artifactId>hamcrest-library</artifactId>

diff --git a/testsuite/src/main/java/io/netty5/testsuite/transport/socket/DatagramUnicastIPv6Test.java b/testsuite/src/main/java/io/netty5/testsuite/transport/socket/DatagramUnicastIPv6Test.java
@@ -16,8 +16,8 @@
 package io.netty5.testsuite.transport.socket;
 
 import io.netty5.channel.socket.InternetProtocolFamily;
-import org.junit.AssumptionViolatedException;
 import org.junit.jupiter.api.BeforeAll;
+import org.opentest4j.TestAbortedException;
 
 import java.io.IOException;
 import java.net.StandardProtocolFamily;
@@ -32,7 +32,7 @@ public static void assumeIpv6Supported() {
             Channel channel = SelectorProvider.provider().openDatagramChannel(StandardProtocolFamily.INET6);
             channel.close();
         } catch (UnsupportedOperationException e) {
-           throw new AssumptionViolatedException("IPv6 not supported", e);
+           throw new TestAbortedException("IPv6 not supported", e);
         } catch (IOException ignore) {
             // Ignore
         }

diff --git a/testsuite/src/main/java/io/netty5/testsuite/util/TestUtils.java b/testsuite/src/main/java/io/netty5/testsuite/util/TestUtils.java
@@ -19,7 +19,6 @@
 import io.netty5.util.internal.logging.InternalLogger;
 import io.netty5.util.internal.logging.InternalLoggerFactory;
 import org.junit.jupiter.api.TestInfo;
-import org.junit.rules.TestName;
 import org.tukaani.xz.LZMA2Options;
 import org.tukaani.xz.XZOutputStream;
 
@@ -78,17 +77,6 @@ public static String testMethodName(TestInfo testInfo) {
         return testMethodName;
     }
 
-    /**
-     * Returns the method name of the current test.
-     */
-    public static String testMethodName(TestName testName) {
-        String testMethodName = testName.getMethodName();
-        if (testMethodName.contains("[")) {
-            testMethodName = testMethodName.substring(0, testMethodName.indexOf('['));
-        }
-        return testMethodName;
-    }
-
     public static void dump(String filenamePrefix) throws IOException {
         requireNonNull(filenamePrefix, "filenamePrefix");
 

diff --git a/transport-native-unix-common-tests/pom.xml b/transport-native-unix-common-tests/pom.xml
@@ -54,15 +54,5 @@
       <artifactId>junit-jupiter-engine</artifactId>
       <scope>compile</scope>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-      <scope>compile</scope>
-    </dependency>
-    <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <scope>compile</scope>
-    </dependency>
   </dependencies>
 </project>
diff --git a/...sport-native-unix-common-tests/src/main/java/io/netty5/channel/unix/tests/SocketTest.java b/...sport-native-unix-common-tests/src/main/java/io/netty5/channel/unix/tests/SocketTest.java
@@ -17,10 +17,10 @@
 
 import io.netty5.channel.unix.Buffer;
 import io.netty5.channel.unix.Socket;
-import org.junit.AssumptionViolatedException;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.opentest4j.TestAbortedException;
 
 import java.io.IOException;
 import java.nio.ByteBuffer;
@@ -127,10 +127,10 @@ public void testRawOpt() throws IOException {
     }
 
     protected int level() {
-        throw new AssumptionViolatedException("Not supported");
+        throw new TestAbortedException("Not supported");
     }
 
     protected int optname() {
-        throw new AssumptionViolatedException("Not supported");
+        throw new TestAbortedException("Not supported");
     }
 }