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-validator.git

commit fb7667a10503e8c97db7dfd7411c7c2efa298c12
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Mar 22 16:23:55 2024 -0400

    Use better JUnit API
---
 .../commons/validator/routines/CodeValidatorTest.java       |  7 ++++---
 .../commons/validator/routines/RegexValidatorTest.java      | 13 +++++++------
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/validator/routines/CodeValidatorTest.java 
b/src/test/java/org/apache/commons/validator/routines/CodeValidatorTest.java
index bef56df2..b905cd2e 100644
--- a/src/test/java/org/apache/commons/validator/routines/CodeValidatorTest.java
+++ b/src/test/java/org/apache/commons/validator/routines/CodeValidatorTest.java
@@ -19,6 +19,7 @@ package org.apache.commons.validator.routines;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import org.apache.commons.validator.routines.checkdigit.CheckDigit;
 import org.apache.commons.validator.routines.checkdigit.EAN13CheckDigit;
@@ -42,8 +43,8 @@ public class CodeValidatorTest {
         assertNull(validator.getCheckDigit(), "No CheckDigit");
         assertEquals(invalidEAN, validator.validate(invalidEAN), "No 
CheckDigit invalid");
         assertEquals(validEAN, validator.validate(validEAN), "No CheckDigit 
valid");
-        assertEquals(true, validator.isValid(invalidEAN), "No CheckDigit (is) 
invalid");
-        assertEquals(true, validator.isValid(validEAN), "No CheckDigit (is) 
valid");
+        assertTrue(validator.isValid(invalidEAN), "No CheckDigit (is) 
invalid");
+        assertTrue(validator.isValid(validEAN), "No CheckDigit (is) valid");
 
         // Use the EAN-13 check digit routine
         validator = new CodeValidator((String) null, -1, 
EAN13CheckDigit.EAN13_CHECK_DIGIT);
@@ -52,7 +53,7 @@ public class CodeValidatorTest {
         assertEquals(null, validator.validate(invalidEAN), "EAN CheckDigit 
invalid");
         assertEquals(validEAN, validator.validate(validEAN), "EAN CheckDigit 
valid");
         assertEquals(false, validator.isValid(invalidEAN), "EAN CheckDigit 
(is) invalid");
-        assertEquals(true, validator.isValid(validEAN), "EAN CheckDigit (is) 
valid");
+        assertTrue(validator.isValid(validEAN), "EAN CheckDigit (is) valid");
         assertEquals(null, validator.validate("978193011099X"), "EAN 
CheckDigit ex");
     }
 
diff --git 
a/src/test/java/org/apache/commons/validator/routines/RegexValidatorTest.java 
b/src/test/java/org/apache/commons/validator/routines/RegexValidatorTest.java
index 43f3d33e..5d58f5d4 100644
--- 
a/src/test/java/org/apache/commons/validator/routines/RegexValidatorTest.java
+++ 
b/src/test/java/org/apache/commons/validator/routines/RegexValidatorTest.java
@@ -18,6 +18,7 @@ package org.apache.commons.validator.routines;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotSame;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.fail;
 
 import java.util.regex.Pattern;
@@ -169,9 +170,9 @@ public class RegexValidatorTest {
         final String[] array = { "AAC", "FDE", "321" };
 
         // isValid()
-        assertEquals(true, multiple.isValid(value), "isValid() Multiple");
+        assertTrue(multiple.isValid(value), "isValid() Multiple");
         assertEquals(false, single1.isValid(value), "isValid() 1st");
-        assertEquals(true, single2.isValid(value), "isValid() 2nd");
+        assertTrue(single2.isValid(value), "isValid() 2nd");
         assertEquals(false, single3.isValid(value), "isValid() 3rd");
 
         // validate()
@@ -211,9 +212,9 @@ public class RegexValidatorTest {
         final String[] array = { "aac", "FDE", "321" };
 
         // isValid()
-        assertEquals(true, multiple.isValid(value), "Sensitive isValid() 
Multiple");
+        assertTrue( multiple.isValid(value), "Sensitive isValid() Multiple");
         assertEquals(false, single1.isValid(value), "Sensitive isValid() 1st");
-        assertEquals(true, single2.isValid(value), "Sensitive isValid() 2nd");
+        assertTrue( single2.isValid(value), "Sensitive isValid() 2nd");
         assertEquals(false, single3.isValid(value), "Sensitive isValid() 3rd");
 
         // validate()
@@ -256,9 +257,9 @@ public class RegexValidatorTest {
         final RegexValidator insensitive = new RegexValidator(REGEX, false);
 
         // isValid()
-        assertEquals(true, sensitive.isValid("ac-DE-1"), "Sensitive isValid() 
valid");
+        assertTrue( sensitive.isValid("ac-DE-1"), "Sensitive isValid() valid");
         assertEquals(false, sensitive.isValid("AB-de-1"), "Sensitive isValid() 
invalid");
-        assertEquals(true, insensitive.isValid("AB-de-1"), "Insensitive 
isValid() valid");
+        assertTrue( insensitive.isValid("AB-de-1"), "Insensitive isValid() 
valid");
         assertEquals(false, insensitive.isValid("ABd-de-1"), "Insensitive 
isValid() invalid");
 
         // validate()

Reply via email to