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

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


The following commit(s) were added to refs/heads/master by this push:
     new be300233b Better use of JUnit API.
be300233b is described below

commit be300233b2afcb8251d0e702163de49b118de819
Author: Gary Gregory <[email protected]>
AuthorDate: Mon Jul 27 15:24:07 2026 +0000

    Better use of JUnit API.
---
 .../collections4/comparators/FixedOrderComparatorTest.java   |  6 +++---
 .../commons/collections4/map/ReferenceIdentityMapTest.java   |  4 ++--
 .../commons/collections4/multiset/AbstractMultiSetTest.java  | 12 ++++++------
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/collections4/comparators/FixedOrderComparatorTest.java
 
b/src/test/java/org/apache/commons/collections4/comparators/FixedOrderComparatorTest.java
index 2a8a92c70..d39e45c97 100644
--- 
a/src/test/java/org/apache/commons/collections4/comparators/FixedOrderComparatorTest.java
+++ 
b/src/test/java/org/apache/commons/collections4/comparators/FixedOrderComparatorTest.java
@@ -89,20 +89,20 @@ class FixedOrderComparatorTest extends 
AbstractComparatorTest<String> {
             final FixedOrderComparator<Integer> comparator2 = new 
FixedOrderComparator<>(1, 2, 3);
             comparator1.compare(1, 2);
             comparator2.compare(1, 2);
-            assertTrue(comparator1.equals(comparator2));
+            assertEquals(comparator1, comparator2);
         }
 
         @Test
         void expectTrueWhenBothComparatorsWithoutAnyItems() {
             final FixedOrderComparator<Integer> comparator1 = new 
FixedOrderComparator<>();
             final FixedOrderComparator<Integer> comparator2 = new 
FixedOrderComparator<>();
-            assertTrue(comparator1.equals(comparator2));
+            assertEquals(comparator1, comparator2);
         }
 
         @Test
         void expectTrueWhenBothObjectsAreSame() {
             final FixedOrderComparator<Integer> comparator = new 
FixedOrderComparator<>();
-            assertTrue(comparator.equals(comparator));
+            assertEquals(comparator, comparator);
         }
     }
 
diff --git 
a/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java
 
b/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java
index 287c5ba56..8c2874c8f 100644
--- 
a/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java
+++ 
b/src/test/java/org/apache/commons/collections4/map/ReferenceIdentityMapTest.java
@@ -298,8 +298,8 @@ public class ReferenceIdentityMapTest<K, V> extends 
AbstractIterableMapTest<K, V
         final Map.Entry<K, V> entry2 = it.next();
         final Map.Entry<K, V> entry3 = it.next();
 
-        assertTrue(entry1.equals(entry2));
-        assertTrue(entry2.equals(entry1));
+        assertEquals(entry1, entry2);
+        assertEquals(entry2, entry1);
         assertNotEquals(entry1, entry3);
     }
 
diff --git 
a/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
 
b/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
index 170f0c4c0..b324d7e72 100644
--- 
a/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
+++ 
b/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
@@ -372,11 +372,11 @@ public abstract class AbstractMultiSetTest<T> extends 
AbstractCollectionTest<T>
 
         final MultiSet<T> multiset = makeObject();
         final MultiSet<T> multiset2 = makeObject();
-        assertTrue(multiset.equals(multiset2));
+        assertEquals(multiset, multiset2);
         multiset.add((T) "A");
         assertNotEquals(multiset, multiset2);
         multiset2.add((T) "A");
-        assertTrue(multiset.equals(multiset2));
+        assertEquals(multiset, multiset2);
         multiset.add((T) "A");
         multiset.add((T) "B");
         multiset.add((T) "B");
@@ -385,7 +385,7 @@ public abstract class AbstractMultiSetTest<T> extends 
AbstractCollectionTest<T>
         multiset2.add((T) "B");
         multiset2.add((T) "B");
         multiset2.add((T) "C");
-        assertTrue(multiset.equals(multiset2));
+        assertEquals(multiset, multiset2);
     }
 
     @Test
@@ -397,11 +397,11 @@ public abstract class AbstractMultiSetTest<T> extends 
AbstractCollectionTest<T>
 
         final MultiSet<T> multiset = makeObject();
         final MultiSet<T> multiset2 = new HashMultiSet<>();
-        assertTrue(multiset.equals(multiset2));
+        assertEquals(multiset, multiset2);
         multiset.add((T) "A");
         assertNotEquals(multiset, multiset2);
         multiset2.add((T) "A");
-        assertTrue(multiset.equals(multiset2));
+        assertEquals(multiset, multiset2);
         multiset.add((T) "A");
         multiset.add((T) "B");
         multiset.add((T) "B");
@@ -410,7 +410,7 @@ public abstract class AbstractMultiSetTest<T> extends 
AbstractCollectionTest<T>
         multiset2.add((T) "B");
         multiset2.add((T) "B");
         multiset2.add((T) "C");
-        assertTrue(multiset.equals(multiset2));
+        assertEquals(multiset, multiset2);
     }
 
     @Test

Reply via email to