Adds unit tests.

Switches from cursor() to iterator() for hashCode() and equals().
Adds toString().

Author: brianburton

src/main/java/org/javimmutable/collections/array/JImmutableTrieArray.java

@@ -44,7 +44,7 @@
 import org.javimmutable.collections.SplitableIterator;
 import org.javimmutable.collections.common.AbstractJImmutableArray;
 import org.javimmutable.collections.common.MutableDelta;
-import org.javimmutable.collections.cursors.Cursors;
+import org.javimmutable.collections.iterators.IteratorHelper;
 import org.javimmutable.collections.iterators.TransformIterator;
 import org.javimmutable.collections.serialization.JImmutableArrayProxy;
 
@@ -193,19 +193,19 @@ public void checkInvariants()
     @Override
     public boolean equals(Object o)
     {
-        return (o == this) || ((o instanceof JImmutableArray) && Cursors.areEqual(cursor(), ((JImmutableArray)o).cursor()));
+        return (o == this) || ((o instanceof JImmutableArray) && IteratorHelper.iteratorEquals(iterator(), ((JImmutableArray)o).iterator()));
     }
 
     @Override
     public int hashCode()
     {
-        return Cursors.computeHashCode(cursor());
+        return IteratorHelper.iteratorHashCode(iterator());
     }
 
     @Override
     public String toString()
     {
-        return Cursors.makeString(cursor());
+        return IteratorHelper.iteratorToString(iterator());
     }
 
     private Object writeReplace()

src/main/java/org/javimmutable/collections/array/LeafTrieNode.java

@@ -39,6 +39,7 @@
 import org.javimmutable.collections.Holder;
 import org.javimmutable.collections.Holders;
 import org.javimmutable.collections.JImmutableMap;
+import org.javimmutable.collections.MapEntry;
 import org.javimmutable.collections.SplitableIterator;
 import org.javimmutable.collections.common.MutableDelta;
 import org.javimmutable.collections.cursors.SingleValueCursor;
@@ -220,6 +221,12 @@ public int hashCode()
         return result;
     }
 
+    @Override
+    public String toString()
+    {
+        return MapEntry.makeToString(this);
+    }
+
     private TrieNode<T> withValue(T newValue)
     {
         return new LeafTrieNode<>(index, newValue, shift);

src/test/java/org/javimmutable/collections/array/JImmutableTrieArrayTest.java

@@ -73,11 +73,15 @@ public void testTrimming()
         array = array.assign(33, 33);
         array = array.assign(65, 65);
         array = array.assign(97, 97);
+        assertEquals("[0=0,33=33,65=65,97=97]", array.toString());
+        assertEquals(33559365, array.hashCode());
         array = array.delete(-1);
         array = array.delete(33);
         array = array.delete(97);
         array = array.delete(65);
         array = array.delete(0);
+        assertEquals("[]", array.toString());
+        assertEquals(0, array.hashCode());
     }
 
     public void testRandom()
@@ -111,6 +115,21 @@ public void testRandom()
         array.checkInvariants();
     }
 
+    public void testCollector()
+    {
+        Random r = new Random(20L);
+        JImmutableArray<Integer> array = JImmutableTrieArray.of();
+        int size = 0;
+        while (size < 50_000) {
+            size += r.nextInt(250);
+            while (array.size() < size) {
+                final int value = r.nextInt();
+                array = array.assign(array.size(), value);
+            }
+            assertEquals(array, array.values().parallelStream().collect(JImmutableTrieArray.collector()));
+        }
+    }
+
     public void testSequential()
     {
         Map<Integer, Integer> expected = new TreeMap<>();