Switch to JUint 5

maxmind · Nov 27, 2023 · 9b20755 · 9b20755
1 parent 0d64ce3
commit 9b20755
Show file tree

Hide file tree

Showing 6 changed files with 74 additions and 49 deletions.
diff --git a/pom.xml b/pom.xml
@@ -37,9 +37,15 @@
     </developers>
     <dependencies>
         <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <version>4.13.2</version>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter</artifactId>
+            <version>5.10.1</version>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.hamcrest</groupId>
+            <artifactId>java-hamcrest</artifactId>
+            <version>2.0.0.0</version>
             <scope>test</scope>
         </dependency>
     </dependencies>
@@ -121,6 +127,11 @@
                     <target>11</target>
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <version>3.2.2</version>
+            </plugin>
             <plugin>
                 <groupId>org.eluder.coveralls</groupId>
                 <artifactId>coveralls-maven-plugin</artifactId>

diff --git a/src/test/java/com/maxmind/db/DecoderTest.java b/src/test/java/com/maxmind/db/DecoderTest.java
@@ -2,9 +2,9 @@
 
 import static org.hamcrest.CoreMatchers.containsString;
 import static org.hamcrest.MatcherAssert.assertThat;
-import static org.junit.Assert.assertArrayEquals;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertArrayEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import java.io.File;
 import java.io.IOException;
@@ -20,7 +20,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.UUID;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 @SuppressWarnings({"boxing", "static-method"})
 public class DecoderTest {
@@ -434,42 +434,42 @@ private static <T> void testTypeDecoding(Type type, Map<T, byte[]> tests)
 
                 // XXX - this could be streamlined
                 if (type.equals(Type.BYTES)) {
-                    assertArrayEquals(desc, (byte[]) expect, decoder.decode(0, byte[].class));
+                    assertArrayEquals((byte[]) expect, decoder.decode(0, byte[].class), desc);
                 } else if (type.equals(Type.ARRAY)) {
-                    assertEquals(desc, expect, decoder.decode(0, List.class));
+                    assertEquals(expect, decoder.decode(0, List.class), desc);
                 } else if (type.equals(Type.UINT16)
                     || type.equals(Type.INT32)) {
-                    assertEquals(desc, expect, decoder.decode(0, Integer.class));
+                    assertEquals(expect, decoder.decode(0, Integer.class), desc);
                 } else if (type.equals(Type.UINT32)
                     || type.equals(Type.POINTER)) {
-                    assertEquals(desc, expect, decoder.decode(0, Long.class));
+                    assertEquals(expect, decoder.decode(0, Long.class), desc);
                 } else if (type.equals(Type.UINT64)
                     || type.equals(Type.UINT128)) {
-                    assertEquals(desc, expect, decoder.decode(0, BigInteger.class));
+                    assertEquals(expect, decoder.decode(0, BigInteger.class), desc);
                 } else if (type.equals(Type.DOUBLE)) {
-                    assertEquals(desc, expect, decoder.decode(0, Double.class));
+                    assertEquals(expect, decoder.decode(0, Double.class), desc);
                 } else if (type.equals(Type.FLOAT)) {
-                    assertEquals(desc, expect, decoder.decode(0, Float.class));
+                    assertEquals(expect, decoder.decode(0, Float.class), desc);
                 } else if (type.equals(Type.UTF8_STRING)) {
-                    assertEquals(desc, expect, decoder.decode(0, String.class));
+                    assertEquals(expect, decoder.decode(0, String.class), desc);
                 } else if (type.equals(Type.BOOLEAN)) {
-                    assertEquals(desc, expect, decoder.decode(0, Boolean.class));
+                    assertEquals(expect, decoder.decode(0, Boolean.class), desc);
                 } else {
                     // We hit this for Type.MAP.
 
                     Map got = decoder.decode(0, Map.class);
                     Map expectMap = (Map) expect;
 
-                    assertEquals(desc, expectMap.size(), got.size());
+                    assertEquals(expectMap.size(), got.size(), desc);
 
                     for (Object keyObject : expectMap.keySet()) {
                         String key = (String) keyObject;
                         Object value = expectMap.get(key);
 
                         if (value instanceof Object[]) {
-                            assertArrayEquals(desc, (Object[]) value, (Object[]) got.get(key));
+                            assertArrayEquals((Object[]) value, (Object[]) got.get(key), desc);
                         } else {
-                            assertEquals(desc, value, got.get(key));
+                            assertEquals(value, got.get(key), desc);
                         }
                     }
                 }

diff --git a/src/test/java/com/maxmind/db/MultiThreadedTest.java b/src/test/java/com/maxmind/db/MultiThreadedTest.java
@@ -1,6 +1,6 @@
 package com.maxmind.db;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.io.IOException;
 import java.net.InetAddress;
@@ -12,7 +12,7 @@
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 public class MultiThreadedTest {
 

diff --git a/src/test/java/com/maxmind/db/NetworkTest.java b/src/test/java/com/maxmind/db/NetworkTest.java
@@ -1,10 +1,10 @@
 package com.maxmind.db;
 
-import static junit.framework.TestCase.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.net.InetAddress;
 import java.net.UnknownHostException;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 public class NetworkTest {
     @Test

diff --git a/src/test/java/com/maxmind/db/PointerTest.java b/src/test/java/com/maxmind/db/PointerTest.java
@@ -1,13 +1,13 @@
 package com.maxmind.db;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import com.maxmind.db.Reader.FileMode;
 import java.io.File;
 import java.io.IOException;
 import java.util.HashMap;
 import java.util.Map;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 public class PointerTest {
     @SuppressWarnings("static-method")

diff --git a/src/test/java/com/maxmind/db/ReaderTest.java b/src/test/java/com/maxmind/db/ReaderTest.java
@@ -3,14 +3,14 @@
 import static org.hamcrest.CoreMatchers.containsString;
 import static org.hamcrest.CoreMatchers.equalTo;
 import static org.hamcrest.MatcherAssert.assertThat;
-import static org.junit.Assert.assertArrayEquals;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertThrows;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertArrayEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.io.File;
 import java.io.IOException;
@@ -27,19 +27,19 @@
 import java.util.Map;
 import java.util.Vector;
 import java.util.concurrent.ConcurrentHashMap;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 public class ReaderTest {
     private Reader testReader;
 
-    @Before
+    @BeforeEach
     public void setupReader() {
         this.testReader = null;
     }
 
-    @After
+    @AfterEach
     public void teardownReader() throws IOException {
         if (this.testReader != null) {
             this.testReader.close();
@@ -93,9 +93,11 @@ public void testNetworks() throws IOException, InvalidDatabaseException, Invalid
 
                     InetAddress actualIPInData = InetAddress.getByName(data.get("ip"));
 
-                    assertEquals("expected ip address", 
+                    assertEquals(
                         iteration.getNetwork().getNetworkAddress(), 
-                        actualIPInData);
+                        actualIPInData,
+                        "expected ip address"
+                    );
                 }
 
                 reader.close();
@@ -1208,7 +1210,7 @@ private void testMetadata(Reader reader, int ipVersion, long recordSize) {
 
         Metadata metadata = reader.getMetadata();
 
-        assertEquals("major version", 2, metadata.getBinaryFormatMajorVersion());
+        assertEquals(2, metadata.getBinaryFormatMajorVersion(), "major version");
         assertEquals(0, metadata.getBinaryFormatMinorVersion());
         assertEquals(ipVersion, metadata.getIpVersion());
         assertEquals("Test", metadata.getDatabaseType());
@@ -1237,8 +1239,11 @@ private void testIpV4(Reader reader, File file) throws IOException {
             Map<String, String> data = new HashMap<>();
             data.put("ip", address);
 
-            assertEquals("found expected data record for " + address + " in "
-                + file, data, reader.get(InetAddress.getByName(address), Map.class));
+            assertEquals(
+                data,
+                reader.get(InetAddress.getByName(address), Map.class),
+                "found expected data record for " + address + " in " + file
+            );
         }
 
         Map<String, String> pairs = new HashMap<>();
@@ -1253,8 +1258,11 @@ private void testIpV4(Reader reader, File file) throws IOException {
             Map<String, String> data = new HashMap<>();
             data.put("ip", pairs.get(address));
 
-            assertEquals("found expected data record for " + address + " in "
-                + file, data, reader.get(InetAddress.getByName(address), Map.class));
+            assertEquals(
+                data,
+                reader.get(InetAddress.getByName(address), Map.class),
+                "found expected data record for " + address + " in " + file
+            );
         }
 
         for (String ip : new String[] {"1.1.1.33", "255.254.253.123"}) {
@@ -1271,8 +1279,11 @@ private void testIpV6(Reader reader, File file) throws IOException {
             Map<String, String> data = new HashMap<>();
             data.put("ip", address);
 
-            assertEquals("found expected data record for " + address + " in "
-                + file, data, reader.get(InetAddress.getByName(address), Map.class));
+            assertEquals(
+                data,
+                reader.get(InetAddress.getByName(address), Map.class),
+                "found expected data record for " + address + " in " + file
+            );
         }
 
         Map<String, String> pairs = new HashMap<>();
@@ -1289,8 +1300,11 @@ private void testIpV6(Reader reader, File file) throws IOException {
             Map<String, String> data = new HashMap<>();
             data.put("ip", pairs.get(address));
 
-            assertEquals("found expected data record for " + address + " in "
-                + file, data, reader.get(InetAddress.getByName(address), Map.class));
+            assertEquals(
+                data,
+                reader.get(InetAddress.getByName(address), Map.class),
+                "found expected data record for " + address + " in " + file
+            );
         }
 
         for (String ip : new String[] {"1.1.1.33", "255.254.253.123", "89fa::"}) {