Sort members

Author: garydgregory

src/test/java/org/apache/commons/lang3/compare/ObjectToStringComparatorTest.java

@@ -47,36 +47,36 @@ public String toString() {
     }
 
     @Test
-    void testNulls() {
+    void testNullLeft() {
         final Thing thing = new Thing("y");
-        final List<Thing> things = Arrays.asList(null, thing, null);
+        final List<Thing> things = Arrays.asList(null, thing);
         things.sort(ObjectToStringComparator.INSTANCE);
         assertEquals("y", things.get(0).string);
-        assertEquals(3, things.size());
+        assertEquals(2, things.size());
         assertSame(thing, things.get(0));
         assertNull(things.get(1));
-        assertNull(things.get(2));
     }
 
     @Test
-    void testNullLeft() {
+    void testNullRight() {
         final Thing thing = new Thing("y");
-        final List<Thing> things = Arrays.asList(null, thing);
+        final List<Thing> things = Arrays.asList(thing, null);
         things.sort(ObjectToStringComparator.INSTANCE);
-        assertEquals("y", things.get(0).string);
         assertEquals(2, things.size());
         assertSame(thing, things.get(0));
         assertNull(things.get(1));
     }
 
     @Test
-    void testNullRight() {
+    void testNulls() {
         final Thing thing = new Thing("y");
-        final List<Thing> things = Arrays.asList(thing, null);
+        final List<Thing> things = Arrays.asList(null, thing, null);
         things.sort(ObjectToStringComparator.INSTANCE);
-        assertEquals(2, things.size());
+        assertEquals("y", things.get(0).string);
+        assertEquals(3, things.size());
         assertSame(thing, things.get(0));
         assertNull(things.get(1));
+        assertNull(things.get(2));
     }
 
     @Test