(commons-collections) branch master updated: Sort members

[ggregory]

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


The following commit(s) were added to refs/heads/master by this push:
     new e05e6dd24 Sort members
e05e6dd24 is described below

commit e05e6dd24846a608ce486b8b58a2ab1d0415e107
Author: Gary D. Gregory <garydgreg...@gmail.com>
AuthorDate: Tue May 13 16:25:08 2025 -0400

    Sort members
---
 .../commons/collections4/IteratorUtilsTest.java    | 24 +++++++++++-----------
 .../comparators/ComparatorChainTest.java           |  8 ++++----
 2 files changed, 16 insertions(+), 16 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/collections4/IteratorUtilsTest.java 
b/src/test/java/org/apache/commons/collections4/IteratorUtilsTest.java
index 8c7785278..9a211142b 100644
--- a/src/test/java/org/apache/commons/collections4/IteratorUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/IteratorUtilsTest.java
@@ -967,18 +967,6 @@ public class IteratorUtilsTest {
         assertThrows(IllegalArgumentException.class, () -> 
IteratorUtils.toList(list.iterator(), -1));
     }
 
-    @Test
-    public void testToSet() {
-        final Set<Object> set = new HashSet<>();
-        set.add(Integer.valueOf(1));
-        set.add("Two");
-        set.add(null);
-        final Set<Object> result = IteratorUtils.toSet(set.iterator());
-        assertEquals(set, result);
-        assertThrows(NullPointerException.class, () -> 
IteratorUtils.toSet(null, 10));
-        assertThrows(IllegalArgumentException.class, () -> 
IteratorUtils.toSet(set.iterator(), -1));
-    }
-
     @Test
     public void testToListIterator() {
         final List<Integer> list = new ArrayList<>();
@@ -1000,6 +988,18 @@ public class IteratorUtilsTest {
         assertThrows(NullPointerException.class, () -> 
IteratorUtils.toListIterator(null));
     }
 
+    @Test
+    public void testToSet() {
+        final Set<Object> set = new HashSet<>();
+        set.add(Integer.valueOf(1));
+        set.add("Two");
+        set.add(null);
+        final Set<Object> result = IteratorUtils.toSet(set.iterator());
+        assertEquals(set, result);
+        assertThrows(NullPointerException.class, () -> 
IteratorUtils.toSet(null, 10));
+        assertThrows(IllegalArgumentException.class, () -> 
IteratorUtils.toSet(set.iterator(), -1));
+    }
+
     @Test
     public void testTransformedIterator() {
         final ArrayList arrayList = new ArrayList();
diff --git 
a/src/test/java/org/apache/commons/collections4/comparators/ComparatorChainTest.java
 
b/src/test/java/org/apache/commons/collections4/comparators/ComparatorChainTest.java
index bd00fcd6f..e27ad1baf 100644
--- 
a/src/test/java/org/apache/commons/collections4/comparators/ComparatorChainTest.java
+++ 
b/src/test/java/org/apache/commons/collections4/comparators/ComparatorChainTest.java
@@ -83,15 +83,15 @@ public class ComparatorChainTest extends 
AbstractComparatorTest<ComparatorChainT
             return getColumn(0) == row.getColumn(0) && getColumn(1) == 
row.getColumn(1) && getColumn(2) == row.getColumn(2);
         }
 
+        public int getColumn(final int colIndex) {
+            return cols[colIndex];
+        }
+
         @Override
         public int hashCode() {
             return Arrays.hashCode(cols);
         }
 
-        public int getColumn(final int colIndex) {
-            return cols[colIndex];
-        }
-
         @Override
         public String toString() {
             return "[" + cols[0] + "," + cols[1] + "," + cols[2] + "]";