--- jsr166/src/test/tck/AtomicReferenceArrayTest.java	2011/06/10 20:01:21	1.24
+++ jsr166/src/test/tck/AtomicReferenceArrayTest.java	2021/01/27 01:57:24	1.40
@@ -6,13 +6,15 @@
  * Pat Fisher, Mike Judd.
  */
 
-import junit.framework.*;
 import java.util.Arrays;
 import java.util.concurrent.atomic.AtomicReferenceArray;
 
+import junit.framework.Test;
+import junit.framework.TestSuite;
+
 public class AtomicReferenceArrayTest extends JSR166TestCase {
     public static void main(String[] args) {
-        junit.textui.TestRunner.run(suite());
+        main(suite(), args);
     }
     public static Test suite() {
         return new TestSuite(AtomicReferenceArrayTest.class);
@@ -22,9 +24,9 @@ public class AtomicReferenceArrayTest ex
      * constructor creates array of given size with all elements null
      */
     public void testConstructor() {
-        AtomicReferenceArray<Integer> ai = new AtomicReferenceArray<Integer>(SIZE);
-        for (int i = 0; i < SIZE; ++i) {
-            assertNull(ai.get(i));
+        AtomicReferenceArray<Integer> aa = new AtomicReferenceArray<>(SIZE);
+        for (int i = 0; i < SIZE; i++) {
+            assertNull(aa.get(i));
         }
     }
 
@@ -34,7 +36,7 @@ public class AtomicReferenceArrayTest ex
     public void testConstructor2NPE() {
         try {
             Integer[] a = null;
-            AtomicReferenceArray<Integer> ai = new AtomicReferenceArray<Integer>(a);
+            new AtomicReferenceArray<Integer>(a);
             shouldThrow();
         } catch (NullPointerException success) {}
     }
@@ -43,21 +45,21 @@ public class AtomicReferenceArrayTest ex
      * constructor with array is of same size and has all elements
      */
     public void testConstructor2() {
-        Integer[] a = { two, one, three, four, seven };
-        AtomicReferenceArray<Integer> ai = new AtomicReferenceArray<Integer>(a);
-        assertEquals(a.length, ai.length());
-        for (int i = 0; i < a.length; ++i)
-            assertEquals(a[i], ai.get(i));
+        Item[] a = { two, one, three, four, seven };
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(a);
+        assertEquals(a.length, aa.length());
+        for (int i = 0; i < a.length; i++)
+            assertEquals(a[i], aa.get(i));
     }
 
     /**
-     * Initialize AtomicReference<Class> with SubClass[]
+     * Initialize AtomicReferenceArray<Class> with SubClass[]
      */
     public void testConstructorSubClassArray() {
-        Integer[] a = { two, one, three, four, seven };
-        AtomicReferenceArray<Number> aa = new AtomicReferenceArray<Number>(a);
+        Item[] a = { two, one, three, four, seven };
+        AtomicReferenceArray<Number> aa = new AtomicReferenceArray<>(a);
         assertEquals(a.length, aa.length());
-        for (int i = 0; i < a.length; ++i) {
+        for (int i = 0; i < a.length; i++) {
             assertSame(a[i], aa.get(i));
             Long x = Long.valueOf(i);
             aa.set(i, x);
@@ -68,27 +70,30 @@ public class AtomicReferenceArrayTest ex
     /**
      * get and set for out of bound indices throw IndexOutOfBoundsException
      */
+    @SuppressWarnings("deprecation")
     public void testIndexing() {
-        AtomicReferenceArray<Integer> ai = new AtomicReferenceArray<Integer>(SIZE);
-        try {
-            ai.get(SIZE);
-            shouldThrow();
-        } catch (IndexOutOfBoundsException success) {
-        }
-        try {
-            ai.get(-1);
-            shouldThrow();
-        } catch (IndexOutOfBoundsException success) {
-        }
-        try {
-            ai.set(SIZE, null);
-            shouldThrow();
-        } catch (IndexOutOfBoundsException success) {
-        }
-        try {
-            ai.set(-1, null);
-            shouldThrow();
-        } catch (IndexOutOfBoundsException success) {
+        AtomicReferenceArray<Integer> aa = new AtomicReferenceArray<>(SIZE);
+        for (int index : new int[] { -1, SIZE }) {
+            try {
+                aa.get(index);
+                shouldThrow();
+            } catch (IndexOutOfBoundsException success) {}
+            try {
+                aa.set(index, null);
+                shouldThrow();
+            } catch (IndexOutOfBoundsException success) {}
+            try {
+                aa.lazySet(index, null);
+                shouldThrow();
+            } catch (IndexOutOfBoundsException success) {}
+            try {
+                aa.compareAndSet(index, null, null);
+                shouldThrow();
+            } catch (IndexOutOfBoundsException success) {}
+            try {
+                aa.weakCompareAndSet(index, null, null);
+                shouldThrow();
+            } catch (IndexOutOfBoundsException success) {}
         }
     }
 
@@ -96,14 +101,14 @@ public class AtomicReferenceArrayTest ex
      * get returns the last value set at index
      */
     public void testGetSet() {
-        AtomicReferenceArray ai = new AtomicReferenceArray(SIZE);
-        for (int i = 0; i < SIZE; ++i) {
-            ai.set(i, one);
-            assertSame(one, ai.get(i));
-            ai.set(i, two);
-            assertSame(two, ai.get(i));
-            ai.set(i, m3);
-            assertSame(m3, ai.get(i));
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(SIZE);
+        for (int i = 0; i < SIZE; i++) {
+            aa.set(i, one);
+            assertSame(one, aa.get(i));
+            aa.set(i, two);
+            assertSame(two, aa.get(i));
+            aa.set(i, minusThree);
+            assertSame(minusThree, aa.get(i));
         }
     }
 
@@ -111,14 +116,14 @@ public class AtomicReferenceArrayTest ex
      * get returns the last value lazySet at index by same thread
      */
     public void testGetLazySet() {
-        AtomicReferenceArray ai = new AtomicReferenceArray(SIZE);
-        for (int i = 0; i < SIZE; ++i) {
-            ai.lazySet(i, one);
-            assertSame(one, ai.get(i));
-            ai.lazySet(i, two);
-            assertSame(two, ai.get(i));
-            ai.lazySet(i, m3);
-            assertSame(m3, ai.get(i));
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(SIZE);
+        for (int i = 0; i < SIZE; i++) {
+            aa.lazySet(i, one);
+            assertSame(one, aa.get(i));
+            aa.lazySet(i, two);
+            assertSame(two, aa.get(i));
+            aa.lazySet(i, minusThree);
+            assertSame(minusThree, aa.get(i));
         }
     }
 
@@ -126,16 +131,16 @@ public class AtomicReferenceArrayTest ex
      * compareAndSet succeeds in changing value if equal to expected else fails
      */
     public void testCompareAndSet() {
-        AtomicReferenceArray ai = new AtomicReferenceArray(SIZE);
-        for (int i = 0; i < SIZE; ++i) {
-            ai.set(i, one);
-            assertTrue(ai.compareAndSet(i, one, two));
-            assertTrue(ai.compareAndSet(i, two, m4));
-            assertSame(m4, ai.get(i));
-            assertFalse(ai.compareAndSet(i, m5, seven));
-            assertSame(m4, ai.get(i));
-            assertTrue(ai.compareAndSet(i, m4, seven));
-            assertSame(seven, ai.get(i));
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(SIZE);
+        for (int i = 0; i < SIZE; i++) {
+            aa.set(i, one);
+            assertTrue(aa.compareAndSet(i, one, two));
+            assertTrue(aa.compareAndSet(i, two, minusFour));
+            assertSame(minusFour, aa.get(i));
+            assertFalse(aa.compareAndSet(i, minusFive, seven));
+            assertSame(minusFour, aa.get(i));
+            assertTrue(aa.compareAndSet(i, minusFour, seven));
+            assertSame(seven, aa.get(i));
         }
     }
 
@@ -144,7 +149,7 @@ public class AtomicReferenceArrayTest ex
      * to succeed
      */
     public void testCompareAndSetInMultipleThreads() throws InterruptedException {
-        final AtomicReferenceArray a = new AtomicReferenceArray(1);
+        final AtomicReferenceArray<Item> a = new AtomicReferenceArray<>(1);
         a.set(0, one);
         Thread t = new Thread(new CheckedRunnable() {
             public void realRun() {
@@ -156,22 +161,23 @@ public class AtomicReferenceArrayTest ex
         assertTrue(a.compareAndSet(0, one, two));
         t.join(LONG_DELAY_MS);
         assertFalse(t.isAlive());
-        assertSame(a.get(0), three);
+        assertSame(three, a.get(0));
     }
 
     /**
      * repeated weakCompareAndSet succeeds in changing value when equal
      * to expected
      */
+    @SuppressWarnings("deprecation")
     public void testWeakCompareAndSet() {
-        AtomicReferenceArray ai = new AtomicReferenceArray(SIZE);
-        for (int i = 0; i < SIZE; ++i) {
-            ai.set(i, one);
-            while (!ai.weakCompareAndSet(i, one, two));
-            while (!ai.weakCompareAndSet(i, two, m4));
-            assertSame(m4, ai.get(i));
-            while (!ai.weakCompareAndSet(i, m4, seven));
-            assertSame(seven, ai.get(i));
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(SIZE);
+        for (int i = 0; i < SIZE; i++) {
+            aa.set(i, one);
+            do {} while (!aa.weakCompareAndSet(i, one, two));
+            do {} while (!aa.weakCompareAndSet(i, two, minusFour));
+            assertSame(minusFour, aa.get(i));
+            do {} while (!aa.weakCompareAndSet(i, minusFour, seven));
+            assertSame(seven, aa.get(i));
         }
     }
 
@@ -179,25 +185,25 @@ public class AtomicReferenceArrayTest ex
      * getAndSet returns previous value and sets to given value at given index
      */
     public void testGetAndSet() {
-        AtomicReferenceArray ai = new AtomicReferenceArray(SIZE);
-        for (int i = 0; i < SIZE; ++i) {
-            ai.set(i, one);
-            assertSame(one, ai.getAndSet(i, zero));
-            assertSame(zero, ai.getAndSet(i, m10));
-            assertSame(m10, ai.getAndSet(i, one));
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(SIZE);
+        for (int i = 0; i < SIZE; i++) {
+            aa.set(i, one);
+            assertSame(one, aa.getAndSet(i, zero));
+            assertSame(zero, aa.getAndSet(i, minusTen));
+            assertSame(minusTen, aa.getAndSet(i, one));
         }
     }
 
     /**
-     * a deserialized serialized array holds same values
+     * a deserialized/reserialized array holds same values in same order
      */
     public void testSerialization() throws Exception {
-        AtomicReferenceArray x = new AtomicReferenceArray(SIZE);
+        AtomicReferenceArray<Item> x = new AtomicReferenceArray<>(SIZE);
         for (int i = 0; i < SIZE; i++) {
-            x.set(i, new Integer(-i));
+            x.set(i, minusOne);
         }
-        AtomicReferenceArray y = serialClone(x);
-        assertTrue(x != y);
+        AtomicReferenceArray<Item> y = serialClone(x);
+        assertNotSame(x, y);
         assertEquals(x.length(), y.length());
         for (int i = 0; i < SIZE; i++) {
             assertEquals(x.get(i), y.get(i));
@@ -208,8 +214,9 @@ public class AtomicReferenceArrayTest ex
      * toString returns current value.
      */
     public void testToString() {
-        Integer[] a = { two, one, three, four, seven };
-        AtomicReferenceArray<Integer> ai = new AtomicReferenceArray<Integer>(a);
-        assertEquals(Arrays.toString(a), ai.toString());
+        Item[] a = { two, one, three, four, seven };
+        AtomicReferenceArray<Item> aa = new AtomicReferenceArray<>(a);
+        assertEquals(Arrays.toString(a), aa.toString());
     }
+
 }