This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-dbutils.git


The following commit(s) were added to refs/heads/master by this push:
     new 19ac26d  Use better JUnit API
19ac26d is described below

commit 19ac26d9e77f8f6682d889dbcabe0491af9ab60f
Author: Gary Gregory <[email protected]>
AuthorDate: Fri Mar 22 16:33:27 2024 -0400

    Use better JUnit API
---
 .../commons/dbutils/BasicRowProcessorTest.java     |  4 ++--
 .../wrappers/SqlNullCheckedResultSetTest.java      | 26 +++++++++++-----------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/dbutils/BasicRowProcessorTest.java 
b/src/test/java/org/apache/commons/dbutils/BasicRowProcessorTest.java
index f62b9c8..997c1d2 100644
--- a/src/test/java/org/apache/commons/dbutils/BasicRowProcessorTest.java
+++ b/src/test/java/org/apache/commons/dbutils/BasicRowProcessorTest.java
@@ -93,7 +93,7 @@ public class BasicRowProcessorTest extends BaseTestCase {
         assertEquals("not set", row.getDoNotSet());
         assertEquals(3, row.getIntTest());
         assertEquals(Integer.valueOf(4), row.getIntegerTest());
-        assertEquals(null, row.getNullObjectTest());
+        assertNull(row.getNullObjectTest());
         assertEquals(0, row.getNullPrimitiveTest());
         // test date -> string handling
         assertNotNull(row.getNotDate());
@@ -124,7 +124,7 @@ public class BasicRowProcessorTest extends BaseTestCase {
         assertEquals("not set", b.getDoNotSet());
         assertEquals(3, b.getIntTest());
         assertEquals(Integer.valueOf(4), b.getIntegerTest());
-        assertEquals(null, b.getNullObjectTest());
+        assertNull(b.getNullObjectTest());
         assertEquals(0, b.getNullPrimitiveTest());
         // test date -> string handling
         assertNotNull(b.getNotDate());
diff --git 
a/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
 
b/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
index e19db92..be9ea0d 100644
--- 
a/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
+++ 
b/src/test/java/org/apache/commons/dbutils/wrappers/SqlNullCheckedResultSetTest.java
@@ -568,9 +568,9 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      * Tests the getString implementation.
      */
     public void testGetString() throws SQLException {
-        assertEquals(null, rs.getString(1));
+        assertNull(rs.getString(1));
         assertTrue(rs.wasNull());
-        assertEquals(null, rs.getString("column"));
+        assertNull(rs.getString("column"));
         assertTrue(rs.wasNull());
         // Set what gets returned to something other than the default
         final String s = "hello, world";
@@ -728,11 +728,11 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      */
     public void testSetNullBytes() throws SQLException {
         // test the default, unset value
-        assertEquals(null, rs2.getNullBytes());
+        assertNull(rs2.getNullBytes());
 
         // test that setting null is safe
         rs2.setNullBytes(null);
-        assertEquals(null, rs2.getNullBytes());
+        assertNull(rs2.getNullBytes());
 
         // Set what gets returned to something other than the default
         final byte[] b = new byte[5];
@@ -784,11 +784,11 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      */
     public void testSetNullDate() throws SQLException {
         // test the default, unset value
-        assertEquals(null, rs2.getNullDate());
+        assertNull(rs2.getNullDate());
 
         // test that setting null is safe
         rs2.setNullDate(null);
-        assertEquals(null, rs2.getNullDate());
+        assertNull(rs2.getNullDate());
 
         // Set what gets returned to something other than the default
         final java.sql.Date date = new java.sql.Date(new 
java.util.Date().getTime());
@@ -906,7 +906,7 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      * Tests the setNullString implementation.
      */
     public void testSetNullString() throws SQLException {
-        assertEquals(null, rs2.getNullString());
+        assertNull(rs2.getNullString());
         // Set what gets returned to something other than the default
         final String s = "hello, world";
         rs2.setNullString(s);
@@ -919,11 +919,11 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      */
     public void testSetNullTime() throws SQLException {
         // test the default, unset value
-        assertEquals(null, rs2.getNullTime());
+        assertNull(rs2.getNullTime());
 
         // test that setting null is safe
         rs2.setNullTime(null);
-        assertEquals(null, rs2.getNullTime());
+        assertNull(rs2.getNullTime());
 
         // Set what gets returned to something other than the default
         final Time time = new Time(new java.util.Date().getTime());
@@ -943,11 +943,11 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      */
     public void testSetNullTimestamp() throws SQLException {
         // test the default, unset value
-        assertEquals(null, rs2.getNullTimestamp());
+        assertNull(rs2.getNullTimestamp());
 
         // test that setting null is safe
         rs2.setNullTimestamp(null);
-        assertEquals(null, rs2.getNullTimestamp());
+        assertNull(rs2.getNullTimestamp());
 
         // Set what gets returned to something other than the default
         final Timestamp ts = new Timestamp(new java.util.Date().getTime());
@@ -966,9 +966,9 @@ public class SqlNullCheckedResultSetTest extends 
BaseTestCase {
      * Tests the getURL and setNullURL implementations.
      */
     public void testURL() throws SQLException, MalformedURLException, 
IllegalAccessException, IllegalArgumentException, InvocationTargetException {
-        assertEquals(null, rs.getURL(1));
+        assertNull(rs.getURL(1));
         assertTrue(rs.wasNull());
-        assertEquals(null, rs.getURL("column"));
+        assertNull(rs.getURL("column"));
         assertTrue(rs.wasNull());
         // Set what gets returned to something other than the default
         final URL u = new URL("http://www.apache.org";);

Reply via email to