Upgrade to JUnit5 (#2810)

Author: pivovarit

.github/workflows/build-version.yml

@@ -12,15 +12,15 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        java: [ '8', '11', '17']
+        java: [ '8', '11', '17' ]
         architecture: [ 'x64' ]
     name: Build with JDK ${{ matrix.java }} on ${{ matrix.architecture }}
     steps:
       - uses: actions/checkout@v4
       - name: Setup JDK
         uses: actions/setup-java@v4
         with:
-          distribution: 'adopt'
+          distribution: 'temurin'
           java-version: ${{ matrix.java }}
           architecture: ${{ matrix.architecture }}
           cache: 'maven'

pom.xml

@@ -69,7 +69,7 @@ We use these goals frequently to keep the dependencies and plugins up-to-date:
         <eclipse.lifecycle.mapping.version>1.0.0</eclipse.lifecycle.mapping.version>
         <java.version>1.8</java.version>
         <jmh.version>1.21</jmh.version>
-        <junit.version>4.13-beta-1</junit.version>
+        <junit.version>5.11.0</junit.version>
         <maven.enforcer.version>3.0.0-M2</maven.enforcer.version>
         <maven.build-helper.version>3.0.0</maven.build-helper.version>
         <maven.bundle.version>5.1.1</maven.bundle.version>
@@ -93,9 +93,10 @@ We use these goals frequently to keep the dependencies and plugins up-to-date:
     <dependencyManagement>
         <dependencies>
             <dependency>
-                <groupId>junit</groupId>
-                <artifactId>junit</artifactId>
+                <groupId>org.junit.jupiter</groupId>
+                <artifactId>junit-jupiter</artifactId>
                 <version>${junit.version}</version>
+                <scope>test</scope>
             </dependency>
             <dependency>
                 <groupId>org.assertj</groupId>

vavr-benchmark/pom.xml

@@ -65,8 +65,8 @@
             <version>15.0</version>
         </dependency>
         <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>

vavr-benchmark/src/test/java/io/vavr/collection/ArrayBenchmark.java

@@ -20,7 +20,7 @@
 package io.vavr.collection;
 
 import io.vavr.JmhRunner;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.openjdk.jmh.annotations.*;
 
 import java.util.ArrayList;

vavr-benchmark/src/test/java/io/vavr/collection/BitSetBenchmark.java

@@ -20,7 +20,7 @@
 package io.vavr.collection;
 
 import io.vavr.JmhRunner;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.openjdk.jmh.annotations.*;
 
 import static io.vavr.JmhRunner.create;

vavr-benchmark/src/test/java/io/vavr/collection/CharSeqBenchmark.java

@@ -20,7 +20,7 @@
 package io.vavr.collection;
 
 import io.vavr.JmhRunner;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.openjdk.jmh.annotations.*;
 
 import java.util.Objects;

vavr-benchmark/src/test/java/io/vavr/collection/HashSetBenchmark.java

@@ -20,7 +20,7 @@
 package io.vavr.collection;
 
 import io.vavr.JmhRunner;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.openjdk.jmh.annotations.*;
 
 import static io.vavr.JmhRunner.create;

vavr-benchmark/src/test/java/io/vavr/collection/IteratorBenchmark.java

@@ -20,26 +20,34 @@
 package io.vavr.collection;
 
 import io.vavr.JmhRunner;
-import org.junit.Assert;
-import org.junit.Test;
-import org.openjdk.jmh.annotations.*;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+import org.openjdk.jmh.annotations.Benchmark;
+import org.openjdk.jmh.annotations.Param;
+import org.openjdk.jmh.annotations.Scope;
+import org.openjdk.jmh.annotations.Setup;
+import org.openjdk.jmh.annotations.State;
 import org.openjdk.jmh.infra.Blackhole;
 
 import java.util.Random;
 
-import static io.vavr.JmhRunner.Includes.*;
+import static io.vavr.JmhRunner.Includes.JAVA;
+import static io.vavr.JmhRunner.Includes.SCALA;
+import static io.vavr.JmhRunner.Includes.VAVR;
 import static io.vavr.JmhRunner.getRandomValues;
 
-@SuppressWarnings({ "ALL", "unchecked", "rawtypes" })
+@SuppressWarnings({"ALL", "unchecked", "rawtypes"})
 public class IteratorBenchmark {
 
     static final Array<Class<?>> CLASSES = Array.of(
-            Sliding.class,
-            Concat.class
+      Sliding.class,
+      Concat.class
     );
 
     @Test
-    public void testAsserts() { JmhRunner.runDebugWithAsserts(CLASSES); }
+    public void testAsserts() {
+        JmhRunner.runDebugWithAsserts(CLASSES);
+    }
 
     public static void main(String... args) {
         JmhRunner.runDebugWithAsserts(CLASSES);
@@ -60,7 +68,8 @@ public static class Base {
         public void setup() {
             final Random random = new Random(0);
             ELEMENTS = getRandomValues(CONTAINER_SIZE, false, random);
-            scalaIterator = (scala.collection.Iterator<Integer>) (Object) scala.collection.mutable.WrappedArray$.MODULE$.make(ELEMENTS).iterator();
+            scalaIterator = (scala.collection.Iterator<Integer>) (Object) scala.collection.mutable.WrappedArray$.MODULE$.make(ELEMENTS)
+              .iterator();
             vavrIterator = Iterator.of(ELEMENTS);
         }
     }
@@ -87,32 +96,32 @@ public void vavr_persistent(Blackhole bh) {
     @State(Scope.Benchmark)
     public static class Concat {
 
-        @Param({ "10", "20" , "100", "1000" })
+        @Param({"10", "20", "100", "1000"})
         private int size;
 
         @Benchmark
         public void vavr_persistent(Blackhole bh) {
             Iterator<Integer> iterator = Iterator.range(0, size)
-                    .foldLeft(Iterator.empty(), (result, __) -> result.concat(Iterator.of(1)));
+              .foldLeft(Iterator.empty(), (result, __) -> result.concat(Iterator.of(1)));
 
             long sum = 0;
             while (iterator.hasNext()) {
                 sum += iterator.next();
             }
-            Assert.assertEquals(size, sum);
+            Assertions.assertEquals(size, sum);
         }
 
         @Benchmark
         public void scala_persistent(Blackhole bh) {
             final scala.collection.Iterator<Integer> iterator = scala.collection.Iterator.range(0, size)
-                    .foldLeft((scala.collection.Iterator<Integer>) (Object) scala.collection.Iterator.empty(),
-                    (result, i) -> result.$plus$plus(() -> scala.collection.Iterator.single(1)));
-            
+              .foldLeft((scala.collection.Iterator<Integer>) (Object) scala.collection.Iterator.empty(),
+                (result, i) -> result.$plus$plus(() -> scala.collection.Iterator.single(1)));
+
             long sum = 0;
             while (iterator.hasNext()) {
                 sum += iterator.next();
             }
-            Assert.assertEquals(size, sum);
+            Assertions.assertEquals(size, sum);
         }
     }
 }

vavr-benchmark/src/test/java/io/vavr/collection/ListBenchmark.java

@@ -20,7 +20,7 @@
 package io.vavr.collection;
 
 import io.vavr.JmhRunner;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.openjdk.jmh.annotations.*;
 
 import java.util.ArrayList;

vavr-benchmark/src/test/java/io/vavr/collection/MapBenchmark.java

@@ -21,7 +21,7 @@
 
 import io.vavr.Function1;
 import io.vavr.JmhRunner;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.openjdk.jmh.annotations.*;
 import org.openjdk.jmh.infra.Blackhole;