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