ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/jsr166/jsr166/src/test/tck/CopyOnWriteArrayListTest.java
(Generate patch)

Comparing jsr166/src/test/tck/CopyOnWriteArrayListTest.java (file contents):
Revision 1.4 by dl, Sat Sep 20 18:20:07 2003 UTC vs.
Revision 1.50 by dl, Tue Jan 26 13:33:05 2021 UTC

# Line 1 | Line 1
1   /*
2 < * Written by members of JCP JSR-166 Expert Group and released to the
3 < * public domain. Use, modify, and redistribute this code in any way
4 < * without acknowledgement. Other contributors include Andrew Wright,
5 < * Jeffrey Hayes, Pat Fischer, Mike Judd.
2 > * Written by Doug Lea with assistance from members of JCP JSR-166
3 > * Expert Group and released to the public domain, as explained at
4 > * http://creativecommons.org/publicdomain/zero/1.0/
5 > * Other contributors include Andrew Wright, Jeffrey Hayes,
6 > * Pat Fisher, Mike Judd.
7   */
8  
9 < import junit.framework.*;
10 < import java.util.*;
11 < import java.util.concurrent.*;
12 < import java.io.*;
9 > import java.util.ArrayList;
10 > import java.util.Arrays;
11 > import java.util.Collection;
12 > import java.util.Collections;
13 > import java.util.Iterator;
14 > import java.util.List;
15 > import java.util.ListIterator;
16 > import java.util.NoSuchElementException;
17 > import java.util.concurrent.CopyOnWriteArrayList;
18 > import java.util.concurrent.ThreadLocalRandom;
19 >
20 > import junit.framework.Test;
21 >
22 > public class CopyOnWriteArrayListTest extends JSR166TestCase {
23  
13 public class CopyOnWriteArrayListTest extends JSR166TestCase{
14    
24      public static void main(String[] args) {
25 <        junit.textui.TestRunner.run (suite());  
25 >        main(suite(), args);
26      }
27  
28      public static Test suite() {
29 <        return new TestSuite(CopyOnWriteArrayListTest.class);
30 <    }
31 <
32 <    static CopyOnWriteArrayList populatedArray(int n){
33 <        CopyOnWriteArrayList a = new CopyOnWriteArrayList();
34 <        assertTrue(a.isEmpty());
35 <        for (int i = 0; i < n; ++i)
36 <            a.add(new Integer(i));
37 <        assertFalse(a.isEmpty());
38 <        assertEquals(n, a.size());
39 <        return a;
29 >        class Implementation implements CollectionImplementation {
30 >            public Class<?> klazz() { return CopyOnWriteArrayList.class; }
31 >            public List emptyCollection() { return new CopyOnWriteArrayList(); }
32 >            public Object makeElement(int i) { return JSR166TestCase.itemFor(i); }
33 >            public boolean isConcurrent() { return true; }
34 >            public boolean permitsNulls() { return true; }
35 >        }
36 >        class SubListImplementation extends Implementation {
37 >            @SuppressWarnings("unchecked")
38 >            public List emptyCollection() {
39 >                List list = super.emptyCollection();
40 >                ThreadLocalRandom rnd = ThreadLocalRandom.current();
41 >                if (rnd.nextBoolean())
42 >                    list.add(makeElement(rnd.nextInt()));
43 >                int i = rnd.nextInt(list.size() + 1);
44 >                return list.subList(i, i);
45 >            }
46 >        }
47 >        return newTestSuite(
48 >                CopyOnWriteArrayListTest.class,
49 >                CollectionTest.testSuite(new Implementation()),
50 >                CollectionTest.testSuite(new SubListImplementation()));
51 >    }
52 >
53 >    static CopyOnWriteArrayList<Item> populatedList(int n) {
54 >        CopyOnWriteArrayList<Item> list = new CopyOnWriteArrayList<>();
55 >        assertTrue(list.isEmpty());
56 >        for (int i = 0; i < n; i++)
57 >            mustAdd(list, i);
58 >        mustEqual(n <= 0, list.isEmpty());
59 >        mustEqual(n, list.size());
60 >        return list;
61 >    }
62 >
63 >    static CopyOnWriteArrayList<Item> populatedList(Item[] elements) {
64 >        CopyOnWriteArrayList<Item> list = new CopyOnWriteArrayList<>();
65 >        assertTrue(list.isEmpty());
66 >        for (Item element : elements)
67 >            list.add(element);
68 >        assertFalse(list.isEmpty());
69 >        mustEqual(elements.length, list.size());
70 >        return list;
71      }
72  
33
73      /**
74 <     *
74 >     * a new list is empty
75       */
76      public void testConstructor() {
77 <        CopyOnWriteArrayList a = new CopyOnWriteArrayList();
78 <        assertTrue(a.isEmpty());
77 >        List<Item> list = new CopyOnWriteArrayList<Item>();
78 >        assertTrue(list.isEmpty());
79      }
80  
81      /**
82 <     *
82 >     * new list contains all elements of initializing array
83       */
84      public void testConstructor2() {
85 <        Integer[] ints = new Integer[SIZE];
86 <        for (int i = 0; i < SIZE-1; ++i)
87 <            ints[i] = new Integer(i);
88 <        CopyOnWriteArrayList a = new CopyOnWriteArrayList(ints);
50 <        for (int i = 0; i < SIZE; ++i)
51 <            assertEquals(ints[i], a.get(i));
85 >        Item[] elts = defaultItems;
86 >        List<Item> list = new CopyOnWriteArrayList<Item>(elts);
87 >        for (int i = 0; i < SIZE; ++i)
88 >            mustEqual(elts[i], list.get(i));
89      }
90  
91      /**
92 <     *
92 >     * new list contains all elements of initializing collection
93       */
94      public void testConstructor3() {
95 <        Integer[] ints = new Integer[SIZE];
96 <        for (int i = 0; i < SIZE-1; ++i)
97 <            ints[i] = new Integer(i);
98 <        CopyOnWriteArrayList a = new CopyOnWriteArrayList(Arrays.asList(ints));
62 <        for (int i = 0; i < SIZE; ++i)
63 <            assertEquals(ints[i], a.get(i));
95 >        Item[] elts = defaultItems;
96 >        List<Item> list = new CopyOnWriteArrayList<Item>(Arrays.asList(elts));
97 >        for (int i = 0; i < SIZE; ++i)
98 >            mustEqual(elts[i], list.get(i));
99      }
65        
100  
101      /**
102 <     *   addAll correctly adds each element from the given collection
102 >     * addAll adds each element from the given collection, including duplicates
103       */
104      public void testAddAll() {
105 <        CopyOnWriteArrayList full = populatedArray(3);
106 <        Vector v = new Vector();
107 <        v.add(three);
108 <        v.add(four);
109 <        v.add(five);
76 <        full.addAll(v);
77 <        assertEquals(6, full.size());
105 >        List<Item> list = populatedList(3);
106 >        assertTrue(list.addAll(Arrays.asList(three, four, five)));
107 >        mustEqual(6, list.size());
108 >        assertTrue(list.addAll(Arrays.asList(three, four, five)));
109 >        mustEqual(9, list.size());
110      }
111  
112      /**
113 <     *   addAllAbsent adds each element from the given collection that did not
114 <     *  already exist in the List
113 >     * addAllAbsent adds each element from the given collection that did not
114 >     * already exist in the List
115       */
116      public void testAddAllAbsent() {
117 <        CopyOnWriteArrayList full = populatedArray(3);
118 <        Vector v = new Vector();
119 <        v.add(three);
120 <        v.add(four);
121 <        v.add(one); // will not add this element
122 <        full.addAllAbsent(v);
91 <        assertEquals(5, full.size());
117 >        CopyOnWriteArrayList<Item> list = populatedList(3);
118 >        // "one" is duplicate and will not be added
119 >        mustEqual(2, list.addAllAbsent(Arrays.asList(three, four, one)));
120 >        mustEqual(5, list.size());
121 >        mustEqual(0, list.addAllAbsent(Arrays.asList(three, four, one)));
122 >        mustEqual(5, list.size());
123      }
124  
125      /**
126 <     *   addIfAbsent will not add the element if it already exists in the list
126 >     * addIfAbsent will not add the element if it already exists in the list
127       */
128      public void testAddIfAbsent() {
129 <        CopyOnWriteArrayList full = populatedArray(SIZE);
130 <        full.addIfAbsent(one);
131 <        assertEquals(SIZE, full.size());
129 >        CopyOnWriteArrayList<Item> list = populatedList(SIZE);
130 >        list.addIfAbsent(one);
131 >        mustEqual(SIZE, list.size());
132      }
133  
134      /**
135 <     *   addIfAbsent correctly adds the element when it does not exist in the list
135 >     * addIfAbsent adds the element when it does not exist in the list
136       */
137      public void testAddIfAbsent2() {
138 <        CopyOnWriteArrayList full = populatedArray(SIZE);
139 <        full.addIfAbsent(three);
140 <        assertTrue(full.contains(three));
138 >        CopyOnWriteArrayList<Item> list = populatedList(SIZE);
139 >        list.addIfAbsent(three);
140 >        mustContain(list, three);
141      }
142  
143      /**
144 <     *   clear correctly removes all elements from the list
144 >     * clear removes all elements from the list
145       */
146      public void testClear() {
147 <        CopyOnWriteArrayList full = populatedArray(SIZE);
148 <        full.clear();
149 <        assertEquals(0, full.size());
147 >        List<Item> list = populatedList(SIZE);
148 >        list.clear();
149 >        mustEqual(0, list.size());
150      }
151  
121
152      /**
153 <     *  Cloned list is equal
153 >     * Cloned list is equal
154       */
155      public void testClone() {
156 <        CopyOnWriteArrayList l1 = populatedArray(SIZE);
157 <        CopyOnWriteArrayList l2 = (CopyOnWriteArrayList)(l1.clone());
158 <        assertEquals(l1, l2);
159 <        l1.clear();
156 >        CopyOnWriteArrayList<Item> l1 = populatedList(SIZE);
157 >        @SuppressWarnings("unchecked")
158 >        CopyOnWriteArrayList<Item> l2 = (CopyOnWriteArrayList<Item>)(l1.clone());
159 >        mustEqual(l1, l2);
160 >        l1.clear();
161          assertFalse(l1.equals(l2));
162      }
163  
164      /**
165 <     *   contains returns the correct values
165 >     * contains is true for added elements
166       */
167      public void testContains() {
168 <        CopyOnWriteArrayList full = populatedArray(3);
169 <        assertTrue(full.contains(one));
170 <        assertFalse(full.contains(five));
168 >        List<Item> list = populatedList(3);
169 >        mustContain(list, one);
170 >        mustNotContain(list, five);
171      }
172  
173      /**
174 <     *
174 >     * adding at an index places it in the indicated index
175       */
176      public void testAddIndex() {
177 <        CopyOnWriteArrayList full = populatedArray(3);
178 <        full.add(0, m1);
179 <        assertEquals(4, full.size());
180 <        assertEquals(m1, full.get(0));
181 <        assertEquals(zero, full.get(1));
177 >        List<Item> list = populatedList(3);
178 >        list.add(0, minusOne);
179 >        mustEqual(4, list.size());
180 >        mustEqual(minusOne, list.get(0));
181 >        mustEqual(zero, list.get(1));
182  
183 <        full.add(2, m2);
184 <        assertEquals(5, full.size());
185 <        assertEquals(m2, full.get(2));
186 <        assertEquals(two, full.get(4));
183 >        list.add(2, minusTwo);
184 >        mustEqual(5, list.size());
185 >        mustEqual(minusTwo, list.get(2));
186 >        mustEqual(two, list.get(4));
187      }
188  
189      /**
190 <     *
190 >     * lists with same elements are equal and have same hashCode
191       */
192      public void testEquals() {
193 <        CopyOnWriteArrayList a = populatedArray(3);
194 <        CopyOnWriteArrayList b = populatedArray(3);
193 >        List<Item> a = populatedList(3);
194 >        List<Item> b = populatedList(3);
195          assertTrue(a.equals(b));
196          assertTrue(b.equals(a));
197 <        assertEquals(a.hashCode(), b.hashCode());
198 <        a.add(m1);
197 >        assertTrue(a.containsAll(b));
198 >        assertTrue(b.containsAll(a));
199 >        mustEqual(a.hashCode(), b.hashCode());
200 >        a.add(minusOne);
201          assertFalse(a.equals(b));
202          assertFalse(b.equals(a));
203 <        b.add(m1);
203 >        assertTrue(a.containsAll(b));
204 >        assertFalse(b.containsAll(a));
205 >        b.add(minusOne);
206          assertTrue(a.equals(b));
207          assertTrue(b.equals(a));
208 <        assertEquals(a.hashCode(), b.hashCode());
208 >        assertTrue(a.containsAll(b));
209 >        assertTrue(b.containsAll(a));
210 >        mustEqual(a.hashCode(), b.hashCode());
211 >
212 >        assertFalse(a.equals(null));
213      }
214  
176    
215      /**
216 <     *   containsAll returns the correct values
216 >     * containsAll returns true for collections with subset of elements
217       */
218      public void testContainsAll() {
219 <        CopyOnWriteArrayList full = populatedArray(3);
220 <        Vector v = new Vector();
221 <        v.add(one);
222 <        v.add(two);
223 <        assertTrue(full.containsAll(v));
224 <        v.add(six);
225 <        assertFalse(full.containsAll(v));
219 >        List<Item> list = populatedList(3);
220 >        assertTrue(list.containsAll(Arrays.asList()));
221 >        assertTrue(list.containsAll(Arrays.asList(one)));
222 >        assertTrue(list.containsAll(Arrays.asList(one, two)));
223 >        assertFalse(list.containsAll(Arrays.asList(one, two, six)));
224 >        assertFalse(list.containsAll(Arrays.asList(six)));
225 >
226 >        try {
227 >            list.containsAll(null);
228 >            shouldThrow();
229 >        } catch (NullPointerException success) {}
230      }
231  
232      /**
233 <     *   get returns the correct value for the given index
233 >     * get returns the value at the given index
234       */
235      public void testGet() {
236 <        CopyOnWriteArrayList full = populatedArray(3);
237 <        assertEquals(0, ((Integer)full.get(0)).intValue());
236 >        List<Item> list = populatedList(3);
237 >        mustEqual(0, list.get(0));
238      }
239  
240      /**
241 <     *   indexOf gives the correct index for the given object
241 >     * indexOf(Object) returns the index of the first occurrence of the
242 >     * specified element in this list, or -1 if this list does not
243 >     * contain the element
244       */
245      public void testIndexOf() {
246 <        CopyOnWriteArrayList full = populatedArray(3);
247 <        assertEquals(1, full.indexOf(one));
248 <        assertEquals(-1, full.indexOf("puppies"));
246 >        List<Item> list = populatedList(3);
247 >        mustEqual(-1, list.indexOf(minusTen));
248 >        int size = list.size();
249 >        for (int i = 0; i < size; i++) {
250 >            Item I = itemFor(i);
251 >            mustEqual(i, list.indexOf(I));
252 >            mustEqual(i, list.subList(0, size).indexOf(I));
253 >            mustEqual(i, list.subList(0, i + 1).indexOf(I));
254 >            mustEqual(-1, list.subList(0, i).indexOf(I));
255 >            mustEqual(0, list.subList(i, size).indexOf(I));
256 >            mustEqual(-1, list.subList(i + 1, size).indexOf(I));
257 >        }
258 >
259 >        list.add(one);
260 >        mustEqual(1, list.indexOf(one));
261 >        mustEqual(1, list.subList(0, size + 1).indexOf(one));
262 >        mustEqual(0, list.subList(1, size + 1).indexOf(one));
263 >        mustEqual(size - 2, list.subList(2, size + 1).indexOf(one));
264 >        mustEqual(0, list.subList(size, size + 1).indexOf(one));
265 >        mustEqual(-1, list.subList(size + 1, size + 1).indexOf(one));
266      }
267  
268      /**
269 <     *   indexOf gives the correct index based on the given index
270 <     *  at which to start searching
269 >     * indexOf(E, int) returns the index of the first occurrence of the
270 >     * specified element in this list, searching forwards from index,
271 >     * or returns -1 if the element is not found
272       */
273      public void testIndexOf2() {
274 <        CopyOnWriteArrayList full = populatedArray(3);
275 <        assertEquals(1, full.indexOf(one, 0));
276 <        assertEquals(-1, full.indexOf(one, 2));
274 >        CopyOnWriteArrayList<Item> list = populatedList(3);
275 >        int size = list.size();
276 >        mustEqual(-1, list.indexOf(minusTen, 0));
277 >
278 >        // we might expect IOOBE, but spec says otherwise
279 >        mustEqual(-1, list.indexOf(zero, size));
280 >        mustEqual(-1, list.indexOf(zero, Integer.MAX_VALUE));
281 >
282 >        assertThrows(
283 >            IndexOutOfBoundsException.class,
284 >            () -> list.indexOf(zero, -1),
285 >            () -> list.indexOf(zero, Integer.MIN_VALUE));
286 >
287 >        for (int i = 0; i < size; i++) {
288 >            Item I = itemFor(i);
289 >            mustEqual(i, list.indexOf(I, 0));
290 >            mustEqual(i, list.indexOf(I, i));
291 >            mustEqual(-1, list.indexOf(I, i + 1));
292 >        }
293 >
294 >        list.add(one);
295 >        mustEqual(1, list.indexOf(one, 0));
296 >        mustEqual(1, list.indexOf(one, 1));
297 >        mustEqual(size, list.indexOf(one, 2));
298 >        mustEqual(size, list.indexOf(one, size));
299      }
300  
301      /**
302 <     *   isEmpty returns the correct values
302 >     * isEmpty returns true when empty, else false
303       */
304      public void testIsEmpty() {
305 <        CopyOnWriteArrayList empty = new CopyOnWriteArrayList();
306 <        CopyOnWriteArrayList full = populatedArray(SIZE);
307 <        assertTrue(empty.isEmpty());
224 <        assertFalse(full.isEmpty());
225 <    }
305 >        List<Item> empty = new CopyOnWriteArrayList<Item>();
306 >        assertTrue(empty.isEmpty());
307 >        assertTrue(empty.subList(0, 0).isEmpty());
308  
309 <    /**
310 <     *   iterator() returns an iterator containing the elements of the list
311 <     */
312 <    public void testIterator() {
231 <        CopyOnWriteArrayList full = populatedArray(SIZE);
232 <        Iterator i = full.iterator();
233 <        int j;
234 <        for(j = 0; i.hasNext(); j++)
235 <            assertEquals(j, ((Integer)i.next()).intValue());
236 <        assertEquals(SIZE, j);
309 >        List<Item> full = populatedList(SIZE);
310 >        assertFalse(full.isEmpty());
311 >        assertTrue(full.subList(0, 0).isEmpty());
312 >        assertTrue(full.subList(SIZE, SIZE).isEmpty());
313      }
314  
315      /**
316 <     *
316 >     * iterator() returns an iterator containing the elements of the
317 >     * list in insertion order
318       */
319 <    public void testIteratorRemove () {
320 <        CopyOnWriteArrayList full = populatedArray(SIZE);
321 <        Iterator it = full.iterator();
245 <        it.next();
319 >    public void testIterator() {
320 >        Collection<Item> empty = new CopyOnWriteArrayList<Item>();
321 >        assertFalse(empty.iterator().hasNext());
322          try {
323 <            it.remove();
323 >            empty.iterator().next();
324              shouldThrow();
325 <        }
250 <        catch (UnsupportedOperationException success) {}
251 <    }
325 >        } catch (NoSuchElementException success) {}
326  
327 <    /**
328 <     *
329 <     */
256 <    public void testToString() {
257 <        CopyOnWriteArrayList full = populatedArray(3);
258 <        String s = full.toString();
259 <        for (int i = 0; i < 3; ++i) {
260 <            assertTrue(s.indexOf(String.valueOf(i)) >= 0);
261 <        }
262 <    }        
263 <
264 <    /**
265 <     *   lastIndexOf returns the correct index for the given object
266 <     */
267 <    public void testLastIndexOf1() {
268 <        CopyOnWriteArrayList full = populatedArray(3);
269 <        full.add(one);
270 <        full.add(three);
271 <        assertEquals(3, full.lastIndexOf(one));
272 <        assertEquals(-1, full.lastIndexOf(six));
273 <    }
327 >        Item[] elements = seqItems(SIZE);
328 >        shuffle(elements);
329 >        Collection<Item> full = populatedList(elements);
330  
331 <    /**
332 <     *   lastIndexOf returns the correct index from the given starting point
333 <     */
334 <    public void testlastIndexOf2() {
335 <        CopyOnWriteArrayList full = populatedArray(3);
336 <        full.add(one);
281 <        full.add(three);
282 <        assertEquals(3, full.lastIndexOf(one, 4));
283 <        assertEquals(-1, full.lastIndexOf(three, 3));
331 >        Iterator<? extends Item> it = full.iterator();
332 >        for (int j = 0; j < SIZE; j++) {
333 >            assertTrue(it.hasNext());
334 >            mustEqual(elements[j], it.next());
335 >        }
336 >        assertIteratorExhausted(it);
337      }
338  
339      /**
340 <     *  Identical to testIterator, except ListInterator has more functionality
340 >     * iterator of empty collection has no elements
341       */
342 <    public void testListIterator1() {
343 <        CopyOnWriteArrayList full = populatedArray(SIZE);
344 <        ListIterator i = full.listIterator();
292 <        int j;
293 <        for(j = 0; i.hasNext(); j++)
294 <            assertEquals(j, ((Integer)i.next()).intValue());
295 <        assertEquals(SIZE, j);
342 >    public void testEmptyIterator() {
343 >        Collection<Item> c = new CopyOnWriteArrayList<Item>();
344 >        assertIteratorExhausted(c.iterator());
345      }
346  
347      /**
348 <     *  listIterator only returns those elements after the given index
348 >     * iterator.remove throws UnsupportedOperationException
349       */
350 <    public void testListIterator2() {
351 <        CopyOnWriteArrayList full = populatedArray(3);
352 <        ListIterator i = full.listIterator(1);
353 <        int j;
354 <        for(j = 0; i.hasNext(); j++)
355 <            assertEquals(j+1, ((Integer)i.next()).intValue());
356 <        assertEquals(2, j);
350 >    public void testIteratorRemove() {
351 >        CopyOnWriteArrayList<Item> list = populatedList(SIZE);
352 >        Iterator<? extends Item> it = list.iterator();
353 >        it.next();
354 >        try {
355 >            it.remove();
356 >            shouldThrow();
357 >        } catch (UnsupportedOperationException success) {}
358      }
359  
360      /**
361 <     *   remove correctly removes and returns the object at the given index
361 >     * toString contains toString of elements
362       */
363 <    public void testRemove() {
364 <        CopyOnWriteArrayList full = populatedArray(3);
365 <        assertEquals(two, full.remove(2));
366 <        assertEquals(2, full.size());
363 >    public void testToString() {
364 >        mustEqual("[]", new CopyOnWriteArrayList<Item>().toString());
365 >        List<Item> list = populatedList(3);
366 >        String s = list.toString();
367 >        for (int i = 0; i < 3; ++i)
368 >            assertTrue(s.contains(String.valueOf(i)));
369 >        mustEqual(new ArrayList<Item>(list).toString(),
370 >                     list.toString());
371      }
372  
373      /**
374 <     *   removeAll correctly removes all elements from the given collection
374 >     * lastIndexOf(Object) returns the index of the last occurrence of
375 >     * the specified element in this list, or -1 if this list does not
376 >     * contain the element
377       */
378 <    public void testRemoveAll() {
379 <        CopyOnWriteArrayList full = populatedArray(3);
380 <        Vector v = new Vector();
381 <        v.add(one);
382 <        v.add(two);
383 <        full.removeAll(v);
384 <        assertEquals(1, full.size());
385 <    }
378 >    public void testLastIndexOf1() {
379 >        List<Item> list = populatedList(3);
380 >        mustEqual(-1, list.lastIndexOf(itemFor(-42)));
381 >        int size = list.size();
382 >        for (int i = 0; i < size; i++) {
383 >            Item I = itemFor(i);
384 >            mustEqual(i, list.lastIndexOf(I));
385 >            mustEqual(i, list.subList(0, size).lastIndexOf(I));
386 >            mustEqual(i, list.subList(0, i + 1).lastIndexOf(I));
387 >            mustEqual(-1, list.subList(0, i).lastIndexOf(I));
388 >            mustEqual(0, list.subList(i, size).lastIndexOf(I));
389 >            mustEqual(-1, list.subList(i + 1, size).lastIndexOf(I));
390 >        }
391  
392 <    /**
393 <     *   set correctly changes the element at the given index
394 <     */
395 <    public void testSet() {
396 <        CopyOnWriteArrayList full = populatedArray(3);
397 <        assertEquals(two, full.set(2, four));
337 <        assertEquals(4, ((Integer)full.get(2)).intValue());
392 >        list.add(one);
393 >        mustEqual(size, list.lastIndexOf(one));
394 >        mustEqual(size, list.subList(0, size + 1).lastIndexOf(one));
395 >        mustEqual(1, list.subList(0, size).lastIndexOf(one));
396 >        mustEqual(0, list.subList(1, 2).lastIndexOf(one));
397 >        mustEqual(-1, list.subList(0, 1).indexOf(one));
398      }
399  
400      /**
401 <     *   size returns the correct values
402 <     */
403 <    public void testSize() {
404 <        CopyOnWriteArrayList empty = new CopyOnWriteArrayList();
405 <        CopyOnWriteArrayList full = populatedArray(SIZE);
406 <        assertEquals(SIZE, full.size());
407 <        assertEquals(0, empty.size());
401 >     * lastIndexOf(E, int) returns the index of the last occurrence of the
402 >     * specified element in this list, searching backwards from index, or
403 >     * returns -1 if the element is not found
404 >     */
405 >    public void testLastIndexOf2() {
406 >        CopyOnWriteArrayList<Item> list = populatedList(3);
407 >
408 >        // we might expect IOOBE, but spec says otherwise
409 >        mustEqual(-1, list.lastIndexOf(zero, -1));
410 >
411 >        int size = list.size();
412 >        assertThrows(
413 >            IndexOutOfBoundsException.class,
414 >            () -> list.lastIndexOf(zero, size),
415 >            () -> list.lastIndexOf(zero, Integer.MAX_VALUE));
416 >
417 >        for (int i = 0; i < size; i++) {
418 >            Item I = itemFor(i);
419 >            mustEqual(i, list.lastIndexOf(I, i));
420 >            mustEqual(list.indexOf(I), list.lastIndexOf(I, i));
421 >            if (i > 0)
422 >                mustEqual(-1, list.lastIndexOf(I, i - 1));
423 >        }
424 >        list.add(one);
425 >        list.add(three);
426 >        mustEqual(1, list.lastIndexOf(one, 1));
427 >        mustEqual(1, list.lastIndexOf(one, 2));
428 >        mustEqual(3, list.lastIndexOf(one, 3));
429 >        mustEqual(3, list.lastIndexOf(one, 4));
430 >        mustEqual(-1, list.lastIndexOf(three, 3));
431      }
432  
433      /**
434 <     *   toArray returns an Object array containing all elements from the list
434 >     * listIterator traverses all elements
435       */
436 <    public void testToArray() {
437 <        CopyOnWriteArrayList full = populatedArray(3);
438 <        Object[] o = full.toArray();
439 <        assertEquals(3, o.length);
440 <        assertEquals(0, ((Integer)o[0]).intValue());
441 <        assertEquals(1, ((Integer)o[1]).intValue());
442 <        assertEquals(2, ((Integer)o[2]).intValue());
436 >    public void testListIterator1() {
437 >        List<Item> list = populatedList(SIZE);
438 >        ListIterator<? extends Item> i = list.listIterator();
439 >        int j;
440 >        for (j = 0; i.hasNext(); j++)
441 >            mustEqual(j, i.next());
442 >        mustEqual(SIZE, j);
443      }
444  
445      /**
446 <     *   toArray returns an Integer array containing all elements from
364 <     *   the list
446 >     * listIterator only returns those elements after the given index
447       */
448 <    public void testToArray2() {
449 <        CopyOnWriteArrayList full = populatedArray(3);
450 <        Integer[] i = new Integer[3];
451 <        i = (Integer[])full.toArray(i);
452 <        assertEquals(3, i.length);
453 <        assertEquals(0, i[0].intValue());
454 <        assertEquals(1, i[1].intValue());
373 <        assertEquals(2, i[2].intValue());
448 >    public void testListIterator2() {
449 >        List<Item> list = populatedList(3);
450 >        ListIterator<? extends Item> i = list.listIterator(1);
451 >        int j;
452 >        for (j = 0; i.hasNext(); j++)
453 >            mustEqual(j + 1, i.next());
454 >        mustEqual(2, j);
455      }
456  
376
457      /**
458 <     *
459 <     */
460 <    public void testSubList() {
461 <        CopyOnWriteArrayList a = populatedArray(10);
462 <        assertTrue(a.subList(1,1).isEmpty());
463 <        for(int j = 0; j < 9; ++j) {
464 <            for(int i = j ; i < 10; ++i) {
465 <                List b = a.subList(j,i);
466 <                for(int k = j; k < i; ++k) {
467 <                    assertEquals(new Integer(k), b.get(k-j));
388 <                }
389 <            }
390 <        }
391 <
392 <        List s = a.subList(2, 5);
393 <        assertEquals(s.size(), 3);
394 <        s.set(2, m1);
395 <        assertEquals(a.get(4), m1);
396 <        s.clear();
397 <        assertEquals(a.size(), 7);
458 >     * remove(int) removes and returns the object at the given index
459 >     */
460 >    public void testRemove_int() {
461 >        int SIZE = 3;
462 >        for (int i = 0; i < SIZE; i++) {
463 >            List<Item> list = populatedList(SIZE);
464 >            mustEqual(i, list.remove(i));
465 >            mustEqual(SIZE - 1, list.size());
466 >            mustNotContain(list, i);
467 >        }
468      }
469  
400    // Exception tests
401
470      /**
471 <     *   toArray throws an ArrayStoreException when the given array
404 <     *  can not store the objects inside the list
471 >     * remove(Object) removes the object if found and returns true
472       */
473 <    public void testToArray_ArrayStoreException() {
474 <        try {
475 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
476 <            c.add("zfasdfsdf");
477 <            c.add("asdadasd");
478 <            c.toArray(new Long[5]);
479 <            shouldThrow();
480 <        } catch(ArrayStoreException e){}
473 >    public void testRemove_Object() {
474 >        int SIZE = 3;
475 >        for (int i = 0; i < SIZE; i++) {
476 >            List<Item> list = populatedList(SIZE);
477 >            mustNotRemove(list, fortytwo);
478 >            mustRemove(list, i);
479 >            mustEqual(SIZE - 1, list.size());
480 >            mustNotContain(list, i);
481 >        }
482 >        CopyOnWriteArrayList<Item> x = new CopyOnWriteArrayList<Item>(Arrays.asList(four, five, six));
483 >        mustRemove(x, six);
484 >        mustEqual(x, Arrays.asList(four, five));
485 >        mustRemove(x, four);
486 >        mustEqual(x, Arrays.asList(five));
487 >        mustRemove(x, five);
488 >        mustEqual(x, Arrays.asList());
489 >        mustNotRemove(x, five);
490      }
491  
492      /**
493 <     *   get throws an IndexOutOfBoundsException on a negative index
493 >     * removeAll removes all elements from the given collection
494       */
495 <    public void testGet1_IndexOutOfBoundsException() {
496 <        try {
497 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
498 <            c.get(-1);
499 <            shouldThrow();
500 <        } catch(IndexOutOfBoundsException e){}
425 <    }
426 <    
427 <    /**
428 <     *   get throws an IndexOutOfBoundsException on a too high index
429 <     */
430 <    public void testGet2_IndexOutOfBoundsException() {
431 <        try {
432 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
433 <            c.add("asdasd");
434 <            c.add("asdad");
435 <            c.get(100);
436 <            shouldThrow();
437 <        } catch(IndexOutOfBoundsException e){}
495 >    public void testRemoveAll() {
496 >        List<Item> list = populatedList(3);
497 >        assertTrue(list.removeAll(Arrays.asList(one, two)));
498 >        mustEqual(1, list.size());
499 >        assertFalse(list.removeAll(Arrays.asList(one, two)));
500 >        mustEqual(1, list.size());
501      }
502  
503      /**
504 <     *   set throws an IndexOutOfBoundsException on a negative index
442 <     */
443 <    public void testSet1_IndexOutOfBoundsException() {
444 <        try {
445 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
446 <            c.set(-1,"qwerty");
447 <            shouldThrow();
448 <        } catch(IndexOutOfBoundsException e){}
449 <    }
450 <    
451 <    /**
452 <     *   set throws an IndexOutOfBoundsException on a too high index
504 >     * set changes the element at the given index
505       */
506 <    public void testSet2() {
507 <        try {
508 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
509 <            c.add("asdasd");
458 <            c.add("asdad");
459 <            c.set(100, "qwerty");
460 <            shouldThrow();
461 <        } catch(IndexOutOfBoundsException e){}
506 >    public void testSet() {
507 >        List<Item> list = populatedList(3);
508 >        mustEqual(2, list.set(2, four));
509 >        mustEqual(4, list.get(2));
510      }
511  
512      /**
513 <     *   add throws an IndexOutOfBoundsException on a negative index
513 >     * size returns the number of elements
514       */
515 <    public void testAdd1_IndexOutOfBoundsException() {
516 <        try {
517 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
518 <            c.add(-1,"qwerty");
471 <            shouldThrow();
472 <        } catch(IndexOutOfBoundsException e){}
473 <    }
474 <    
475 <    /**
476 <     *   add throws an IndexOutOfBoundsException on a too high index
477 <     */
478 <    public void testAdd2_IndexOutOfBoundsException() {
479 <        try {
480 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
481 <            c.add("asdasd");
482 <            c.add("asdasdasd");
483 <            c.add(100, "qwerty");
484 <            shouldThrow();
485 <        } catch(IndexOutOfBoundsException e){}
486 <    }
515 >    public void testSize() {
516 >        List<Item> empty = new CopyOnWriteArrayList<Item>();
517 >        mustEqual(0, empty.size());
518 >        mustEqual(0, empty.subList(0, 0).size());
519  
520 <    /**
521 <     *   remove throws an IndexOutOfBoundsException on a negative index
522 <     */
523 <    public void testRemove1_IndexOutOfBounds() {
492 <        try {
493 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
494 <            c.remove(-1);
495 <            shouldThrow();
496 <        } catch(IndexOutOfBoundsException e){}
520 >        List<Item> full = populatedList(SIZE);
521 >        mustEqual(SIZE, full.size());
522 >        mustEqual(0, full.subList(0, 0).size());
523 >        mustEqual(0, full.subList(SIZE, SIZE).size());
524      }
525  
526      /**
527 <     *   remove throws an IndexOutOfBoundsException on a too high index
527 >     * toArray() returns an Object array containing all elements from
528 >     * the list in insertion order
529       */
530 <    public void testRemove2_IndexOutOfBounds() {
531 <        try {
532 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
533 <            c.add("asdasd");
506 <            c.add("adasdasd");
507 <            c.remove(100);
508 <            shouldThrow();
509 <        } catch(IndexOutOfBoundsException e){}
510 <    }
511 <    
512 <    /**
513 <     *   addAll throws an IndexOutOfBoundsException on a negative index
514 <     */
515 <    public void testAddAll1_IndexOutOfBoundsException() {
516 <        try {
517 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
518 <            c.addAll(-1,new LinkedList());
519 <            shouldThrow();
520 <        } catch(IndexOutOfBoundsException e){}
521 <    }
522 <    
523 <    /**
524 <     *   addAll throws an IndexOutOfBoundsException on a too high index
525 <     */
526 <    public void testAddAll2_IndexOutOfBoundsException() {
527 <        try {
528 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
529 <            c.add("asdasd");
530 <            c.add("asdasdasd");
531 <            c.addAll(100, new LinkedList());
532 <            shouldThrow();
533 <        } catch(IndexOutOfBoundsException e){}
534 <    }
530 >    public void testToArray() {
531 >        Object[] a = new CopyOnWriteArrayList<Item>().toArray();
532 >        assertTrue(Arrays.equals(new Object[0], a));
533 >        assertSame(Object[].class, a.getClass());
534  
535 <    /**
536 <     *   listIterator throws an IndexOutOfBoundsException on a negative index
537 <     */
539 <    public void testListIterator1_IndexOutOfBoundsException() {
540 <        try {
541 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
542 <            c.listIterator(-1);
543 <            shouldThrow();
544 <        } catch(IndexOutOfBoundsException e){}
545 <    }
535 >        Item[] elements = seqItems(SIZE);
536 >        shuffle(elements);
537 >        Collection<Item> full = populatedList(elements);
538  
539 <    /**
540 <     *   listIterator throws an IndexOutOfBoundsException on a too high index
549 <     */
550 <    public void testListIterator2_IndexOutOfBoundsException() {
551 <        try {
552 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
553 <            c.add("adasd");
554 <            c.add("asdasdas");
555 <            c.listIterator(100);
556 <            shouldThrow();
557 <        } catch(IndexOutOfBoundsException e){}
539 >        assertTrue(Arrays.equals(elements, full.toArray()));
540 >        assertSame(Object[].class, full.toArray().getClass());
541      }
542  
543      /**
544 <     *   subList throws an IndexOutOfBoundsException on a negative index
544 >     * toArray(Item array) returns an Item array containing all
545 >     * elements from the list in insertion order
546       */
547 <    public void testSubList1_IndexOutOfBoundsException() {
548 <        try {
549 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
566 <            c.subList(-1,100);
547 >    public void testToArray2() {
548 >        Collection<Item> empty = new CopyOnWriteArrayList<Item>();
549 >        Item[] a;
550  
551 <            shouldThrow();
552 <        } catch(IndexOutOfBoundsException e){}
570 <    }
551 >        a = new Item[0];
552 >        assertSame(a, empty.toArray(a));
553  
554 <    /**
555 <     *   subList throws an IndexOutOfBoundsException on a too high index
556 <     */
557 <    public void testSubList2_IndexOutOfBoundsException() {
558 <        try {
559 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
560 <            c.add("asdasd");
561 <            c.subList(1,100);
562 <            shouldThrow();
563 <        } catch(IndexOutOfBoundsException e){}
554 >        a = new Item[SIZE / 2];
555 >        Arrays.fill(a, fortytwo);
556 >        assertSame(a, empty.toArray(a));
557 >        assertNull(a[0]);
558 >        for (int i = 1; i < a.length; i++)
559 >            mustEqual(42, a[i]);
560 >
561 >        Item[] elements = seqItems(SIZE);
562 >        shuffle(elements);
563 >        Collection<Item> full = populatedList(elements);
564 >
565 >        Arrays.fill(a, fortytwo);
566 >        assertTrue(Arrays.equals(elements, full.toArray(a)));
567 >        for (int i = 0; i < a.length; i++)
568 >            mustEqual(42, a[i]);
569 >        assertSame(Item[].class, full.toArray(a).getClass());
570 >
571 >        a = new Item[SIZE];
572 >        Arrays.fill(a, fortytwo);
573 >        assertSame(a, full.toArray(a));
574 >        assertTrue(Arrays.equals(elements, a));
575 >
576 >        a = new Item[2 * SIZE];
577 >        Arrays.fill(a, fortytwo);
578 >        assertSame(a, full.toArray(a));
579 >        assertTrue(Arrays.equals(elements, Arrays.copyOf(a, SIZE)));
580 >        assertNull(a[SIZE]);
581 >        for (int i = SIZE + 1; i < a.length; i++)
582 >            mustEqual(42, a[i]);
583      }
584  
585      /**
586 <     *   subList throws IndexOutOfBoundsException when the second index
586 <     *  is lower then the first
586 >     * sublists contains elements at indexes offset from their base
587       */
588 <    public void testSubList3_IndexOutOfBoundsException() {
589 <        try {
590 <            CopyOnWriteArrayList c = new CopyOnWriteArrayList();
591 <            c.subList(3,1);
588 >    public void testSubList() {
589 >        List<Item> a = populatedList(10);
590 >        assertTrue(a.subList(1,1).isEmpty());
591 >        for (int j = 0; j < 9; ++j) {
592 >            for (int i = j ; i < 10; ++i) {
593 >                List<Item> b = a.subList(j,i);
594 >                for (int k = j; k < i; ++k) {
595 >                    mustEqual(itemFor(k), b.get(k-j));
596 >                }
597 >            }
598 >        }
599  
600 <            shouldThrow();
601 <        } catch(IndexOutOfBoundsException e){}
600 >        List<Item> s = a.subList(2, 5);
601 >        mustEqual(3, s.size());
602 >        s.set(2, minusOne);
603 >        mustEqual(a.get(4), minusOne);
604 >        s.clear();
605 >        mustEqual(7, a.size());
606 >
607 >        assertThrows(
608 >            IndexOutOfBoundsException.class,
609 >            () -> s.get(0),
610 >            () -> s.set(0, fortytwo));
611      }
612  
613 +    // Exception tests
614 +
615      /**
616 <     *
616 >     * toArray throws an ArrayStoreException when the given array
617 >     * can not store the objects inside the list
618       */
619 <    public void testSerialization() {
620 <        CopyOnWriteArrayList q = populatedArray(SIZE);
621 <
622 <        try {
623 <            ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
624 <            ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
625 <            out.writeObject(q);
626 <            out.close();
627 <
628 <            ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
629 <            ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
630 <            CopyOnWriteArrayList r = (CopyOnWriteArrayList)in.readObject();
631 <            assertEquals(q.size(), r.size());
632 <            assertTrue(q.equals(r));
633 <            assertTrue(r.equals(q));
634 <        } catch(Exception e){
635 <            unexpectedException();
619 >    public void testToArray_ArrayStoreException() {
620 >        List<Item> list = new CopyOnWriteArrayList<Item>();
621 >        // Items are not auto-converted to Longs
622 >        list.add(eightysix);
623 >        list.add(ninetynine);
624 >        assertThrows(
625 >            ArrayStoreException.class,
626 >            () -> list.toArray(new Long[0]),
627 >            () -> list.toArray(new Long[5]));
628 >    }
629 >
630 >    @SuppressWarnings("unchecked")
631 >    void testIndexOutOfBoundsException(List list) {
632 >        int size = list.size();
633 >        assertThrows(
634 >            IndexOutOfBoundsException.class,
635 >            () -> list.get(-1),
636 >            () -> list.get(size),
637 >            () -> list.set(-1, "qwerty"),
638 >            () -> list.set(size, "qwerty"),
639 >            () -> list.add(-1, "qwerty"),
640 >            () -> list.add(size + 1, "qwerty"),
641 >            () -> list.remove(-1),
642 >            () -> list.remove(size),
643 >            () -> list.addAll(-1, Collections.emptyList()),
644 >            () -> list.addAll(size + 1, Collections.emptyList()),
645 >            () -> list.listIterator(-1),
646 >            () -> list.listIterator(size + 1),
647 >            () -> list.subList(-1, size),
648 >            () -> list.subList(0, size + 1));
649 >
650 >        // Conversely, operations that must not throw
651 >        list.addAll(0, Collections.emptyList());
652 >        list.addAll(size, Collections.emptyList());
653 >        list.add(0, "qwerty");
654 >        list.add(list.size(), "qwerty");
655 >        list.get(0);
656 >        list.get(list.size() - 1);
657 >        list.set(0, "azerty");
658 >        list.set(list.size() - 1, "azerty");
659 >        list.listIterator(0);
660 >        list.listIterator(list.size());
661 >        list.subList(0, list.size());
662 >        list.remove(list.size() - 1);
663 >    }
664 >
665 >    /**
666 >     * IndexOutOfBoundsException is thrown when specified
667 >     */
668 >    public void testIndexOutOfBoundsException() {
669 >        ThreadLocalRandom rnd = ThreadLocalRandom.current();
670 >        List<Item> x = populatedList(rnd.nextInt(5));
671 >        testIndexOutOfBoundsException(x);
672 >
673 >        int start = rnd.nextInt(x.size() + 1);
674 >        int end = rnd.nextInt(start, x.size() + 1);
675 >        assertThrows(
676 >            IndexOutOfBoundsException.class,
677 >            () -> x.subList(start, start - 1));
678 >        List<Item> subList = x.subList(start, end);
679 >        testIndexOutOfBoundsException(x);
680 >    }
681 >
682 >    /**
683 >     * a deserialized/reserialized list equals original
684 >     */
685 >    public void testSerialization() throws Exception {
686 >        List<Item> x = populatedList(SIZE);
687 >        List<Item> y = serialClone(x);
688 >
689 >        assertNotSame(x, y);
690 >        mustEqual(x.size(), y.size());
691 >        mustEqual(x.toString(), y.toString());
692 >        assertTrue(Arrays.equals(x.toArray(), y.toArray()));
693 >        mustEqual(x, y);
694 >        mustEqual(y, x);
695 >        while (!x.isEmpty()) {
696 >            assertFalse(y.isEmpty());
697 >            mustEqual(x.remove(0), y.remove(0));
698          }
699 +        assertTrue(y.isEmpty());
700      }
701 <    
701 >
702   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines