Add ArrayHashMap; Use 'supportNullValue' optimizing ArrayHashSet and ArrayHashMap; Unify ctor for both impl.

Add/Enhance unit tests for both.

1 files changed, 99 insertions, 24 deletions

diff --git a/src/junit/com/jogamp/common/util/TestArrayHashSet01.java b/src/junit/com/jogamp/common/util/TestArrayHashSet01.java
index 51db2c7..3e8fbc0 100644
--- a/src/junit/com/jogamp/common/util/TestArrayHashSet01.java
+++ b/src/junit/com/jogamp/common/util/TestArrayHashSet01.java
@@ -74,47 +74,99 @@ public class TestArrayHashSet01 extends SingletonJunitCase {
         }
     }
 
-    public void populate(final List<Dummy> l, final int start, final int len, final int i2, final int i3, final int expectedPlusSize) {
+    void populate(final List<Dummy> l, final int start, final int len,
+                  final int i2, final int i3, final int expectedPlusSize) {
         final int oldSize = l.size();
-        int pos = start+len-1;
-        while(pos>=start) {
-            l.add(new Dummy(pos--, i2, i3));
+        for(int pos = start+len-1; pos>=start; pos--) {
+            l.add(new Dummy(pos, i2, i3));
         }
         Assert.assertEquals(expectedPlusSize, l.size() - oldSize);
     }
+    boolean checkOrder(final List<Dummy> l, final int startIdx, final int start, final int len) {
+        for(int i=0; i<len; i++) {
+            final Dummy d = l.get(startIdx+i);
+            final int i1 = start+len-1-i;
+            if( d.i1 != i1 ) {
+                return false;
+            }
+        }
+        return true;
+    }
 
     @Test
-    public void test01ArrayHashSet() {
-        final ArrayHashSet<Dummy> l = new ArrayHashSet<Dummy>();
-        populate(l, 10, 100, 22, 34, 100); // [10 .. 109]
-        populate(l, 10, 100, 22, 34,   0); // [10 .. 109]
-        populate(l,  6,   5, 22, 34,   4); // [ 6 .. 9], 10 already exists
-
-        final Dummy p6_22_34 = new Dummy(6, 22, 34);
+    public void test01ArrayHashSetWithNullValue() {
+        testArrayHashSetImpl(true);
+    }
+    @Test
+    public void test02ArrayHashSetWithoutNullValue() {
+        testArrayHashSetImpl(false);
+    }
+    void testArrayHashSetImpl(final boolean supportNullValue) {
+        final ArrayHashSet<Dummy> l =
+                new ArrayHashSet<Dummy>(supportNullValue,
+                                        ArrayHashSet.DEFAULT_INITIAL_CAPACITY,
+                                        ArrayHashSet.DEFAULT_LOAD_FACTOR);
+        Assert.assertEquals(supportNullValue, l.supportsNullValue());
+        final int p7_22_34_idx;
+        final Dummy p7_22_34_orig;
+        final int p6_22_34_idx;
+        final Dummy p6_22_34_orig;
+        {
+            populate(l, 10, 100, 22, 34, 100); // [109 .. 10]
+            Assert.assertTrue(checkOrder(l, 0, 10, 100));
+            populate(l, 10, 100, 22, 34,   0); // [109 .. 10]
+            Assert.assertTrue(checkOrder(l, 0, 10, 100));
+            populate(l,  6,   5, 22, 34,   4); // [  9 ..  6], 10 already exists
+            Assert.assertTrue(checkOrder(l, 100, 6, 4));
+            p7_22_34_idx = l.size() - 2;
+            p7_22_34_orig = l.get(p7_22_34_idx);
+            p6_22_34_idx = l.size() - 1;
+            p6_22_34_orig = l.get(p6_22_34_idx);
+        }
+        Assert.assertNotNull(p7_22_34_orig);
+        Assert.assertEquals(7, p7_22_34_orig.i1);
+        Assert.assertEquals(l.getData().get(p7_22_34_idx), p7_22_34_orig);
+        Assert.assertNotNull(p6_22_34_orig);
+        Assert.assertEquals(6, p6_22_34_orig.i1);
+        Assert.assertEquals(l.getData().get(p6_22_34_idx), p6_22_34_orig);
+
+        final Dummy p7_22_34_other = new Dummy(7, 22, 34);
+        Assert.assertEquals(p7_22_34_other, p7_22_34_orig);
+        Assert.assertTrue(p7_22_34_other.hashCode() == p7_22_34_orig.hashCode());
+        Assert.assertTrue(p7_22_34_other != p7_22_34_orig); // diff reference
+        final Dummy p6_22_34_other = new Dummy(6, 22, 34);
+        Assert.assertEquals(p6_22_34_other, p6_22_34_orig);
+        Assert.assertTrue(p6_22_34_other.hashCode() == p6_22_34_orig.hashCode());
+        Assert.assertTrue(p6_22_34_other != p6_22_34_orig); // diff reference
 
         // slow get on position ..
-        final int i = l.indexOf(p6_22_34);
+        final int i = l.indexOf(p6_22_34_other);
         Dummy q = l.get(i);
         Assert.assertNotNull(q);
-        Assert.assertEquals(p6_22_34, q);
-        Assert.assertTrue(p6_22_34.hashCode() == q.hashCode());
-        Assert.assertTrue(p6_22_34 != q); // diff reference
+        Assert.assertEquals(p6_22_34_other, q);
+        Assert.assertTrue(p6_22_34_other.hashCode() == q.hashCode());
+        Assert.assertTrue(p6_22_34_other != q); // diff reference
+        Assert.assertTrue(p6_22_34_orig == q); // same reference
 
         // fast identity ..
-        q = l.get(p6_22_34);
+        q = l.get(p6_22_34_other);
         Assert.assertNotNull(q);
-        Assert.assertEquals(p6_22_34, q);
-        Assert.assertTrue(p6_22_34.hashCode() == q.hashCode());
-        Assert.assertTrue(p6_22_34 != q); // diff reference
+        Assert.assertEquals(p6_22_34_other, q);
+        Assert.assertTrue(p6_22_34_other.hashCode() == q.hashCode());
+        Assert.assertTrue(p6_22_34_other != q); // diff reference
+        Assert.assertTrue(p6_22_34_orig == q); // same reference
 
         Assert.assertTrue(!l.add(q)); // add same
-        Assert.assertTrue(!l.add(p6_22_34)); // add equivalent
+        Assert.assertTrue(!l.add(p6_22_34_other)); // add equivalent
 
-        q = l.getOrAdd(p6_22_34); // not added test
+        q = l.getOrAdd(p6_22_34_other); // not added test w/ diff hash-obj
         Assert.assertNotNull(q);
-        Assert.assertEquals(p6_22_34, q);
-        Assert.assertTrue(p6_22_34.hashCode() == q.hashCode());
-        Assert.assertTrue(p6_22_34 != q); // diff reference
+        Assert.assertEquals(p6_22_34_other, q);
+        Assert.assertTrue(p6_22_34_other.hashCode() == q.hashCode());
+        Assert.assertTrue(p6_22_34_other != q); // diff reference
+        Assert.assertTrue(p6_22_34_orig == q); // same reference
+        Assert.assertTrue(checkOrder(l, 0, 10, 100));
+        Assert.assertTrue(checkOrder(l, 100, 6, 4));
 
         final Dummy p1_2_3 = new Dummy(1, 2, 3); // a new one ..
         q = l.getOrAdd(p1_2_3); // added test
@@ -122,6 +174,29 @@ public class TestArrayHashSet01 extends SingletonJunitCase {
         Assert.assertEquals(p1_2_3, q);
         Assert.assertTrue(p1_2_3.hashCode() == q.hashCode());
         Assert.assertTrue(p1_2_3 == q); // _same_ reference, since getOrAdd added it
+        Assert.assertTrue(checkOrder(l, 0, 10, 100));
+        Assert.assertTrue(checkOrder(l, 100, 6, 4));
+
+        final Dummy pNull = null;
+        NullPointerException npe = null;
+        try {
+            q = l.getOrAdd(pNull);
+            Assert.assertNull(q);
+        } catch (final NullPointerException _npe) { npe = _npe; }
+        if( l.supportsNullValue() ) {
+            Assert.assertNull(npe);
+        } else {
+            Assert.assertNotNull(npe);
+        }
+
+        try {
+            Assert.assertTrue(l.remove(pNull));
+        } catch (final NullPointerException _npe) { npe = _npe; }
+        if( l.supportsNullValue() ) {
+            Assert.assertNull(npe);
+        } else {
+            Assert.assertNotNull(npe);
+        }
     }
 
     public static void main(final String args[]) throws IOException {