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.49 by jsr166, Tue Apr 3 05:49:43 2018 UTC vs.
Revision 1.50 by dl, Tue Jan 26 13:33:05 2021 UTC

# Line 29 | Line 29 | public class CopyOnWriteArrayListTest ex
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 i; }
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();
# Line 49 | Line 50 | public class CopyOnWriteArrayListTest ex
50                  CollectionTest.testSuite(new SubListImplementation()));
51      }
52  
53 <    static CopyOnWriteArrayList<Integer> populatedList(int n) {
54 <        CopyOnWriteArrayList<Integer> list = new CopyOnWriteArrayList<>();
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 <            list.add(i);
58 <        assertEquals(n <= 0, list.isEmpty());
59 <        assertEquals(n, list.size());
57 >            mustAdd(list, i);
58 >        mustEqual(n <= 0, list.isEmpty());
59 >        mustEqual(n, list.size());
60          return list;
61      }
62  
63 <    static CopyOnWriteArrayList<Integer> populatedList(Integer[] elements) {
64 <        CopyOnWriteArrayList<Integer> list = new CopyOnWriteArrayList<>();
63 >    static CopyOnWriteArrayList<Item> populatedList(Item[] elements) {
64 >        CopyOnWriteArrayList<Item> list = new CopyOnWriteArrayList<>();
65          assertTrue(list.isEmpty());
66 <        for (Integer element : elements)
66 >        for (Item element : elements)
67              list.add(element);
68          assertFalse(list.isEmpty());
69 <        assertEquals(elements.length, list.size());
69 >        mustEqual(elements.length, list.size());
70          return list;
71      }
72  
# Line 73 | Line 74 | public class CopyOnWriteArrayListTest ex
74       * a new list is empty
75       */
76      public void testConstructor() {
77 <        List list = new CopyOnWriteArrayList();
77 >        List<Item> list = new CopyOnWriteArrayList<Item>();
78          assertTrue(list.isEmpty());
79      }
80  
# Line 81 | Line 82 | public class CopyOnWriteArrayListTest ex
82       * new list contains all elements of initializing array
83       */
84      public void testConstructor2() {
85 <        Integer[] elts = new Integer[SIZE];
86 <        for (int i = 0; i < SIZE - 1; ++i)
86 <            elts[i] = i;
87 <        List list = new CopyOnWriteArrayList(elts);
85 >        Item[] elts = defaultItems;
86 >        List<Item> list = new CopyOnWriteArrayList<Item>(elts);
87          for (int i = 0; i < SIZE; ++i)
88 <            assertEquals(elts[i], list.get(i));
88 >            mustEqual(elts[i], list.get(i));
89      }
90  
91      /**
92       * new list contains all elements of initializing collection
93       */
94      public void testConstructor3() {
95 <        Integer[] elts = new Integer[SIZE];
96 <        for (int i = 0; i < SIZE - 1; ++i)
98 <            elts[i] = i;
99 <        List list = new CopyOnWriteArrayList(Arrays.asList(elts));
95 >        Item[] elts = defaultItems;
96 >        List<Item> list = new CopyOnWriteArrayList<Item>(Arrays.asList(elts));
97          for (int i = 0; i < SIZE; ++i)
98 <            assertEquals(elts[i], list.get(i));
98 >            mustEqual(elts[i], list.get(i));
99      }
100  
101      /**
102       * addAll adds each element from the given collection, including duplicates
103       */
104      public void testAddAll() {
105 <        List list = populatedList(3);
105 >        List<Item> list = populatedList(3);
106          assertTrue(list.addAll(Arrays.asList(three, four, five)));
107 <        assertEquals(6, list.size());
107 >        mustEqual(6, list.size());
108          assertTrue(list.addAll(Arrays.asList(three, four, five)));
109 <        assertEquals(9, list.size());
109 >        mustEqual(9, list.size());
110      }
111  
112      /**
# Line 117 | Line 114 | public class CopyOnWriteArrayListTest ex
114       * already exist in the List
115       */
116      public void testAddAllAbsent() {
117 <        CopyOnWriteArrayList list = populatedList(3);
117 >        CopyOnWriteArrayList<Item> list = populatedList(3);
118          // "one" is duplicate and will not be added
119 <        assertEquals(2, list.addAllAbsent(Arrays.asList(three, four, one)));
120 <        assertEquals(5, list.size());
121 <        assertEquals(0, list.addAllAbsent(Arrays.asList(three, four, one)));
122 <        assertEquals(5, list.size());
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
127       */
128      public void testAddIfAbsent() {
129 <        CopyOnWriteArrayList list = populatedList(SIZE);
129 >        CopyOnWriteArrayList<Item> list = populatedList(SIZE);
130          list.addIfAbsent(one);
131 <        assertEquals(SIZE, list.size());
131 >        mustEqual(SIZE, list.size());
132      }
133  
134      /**
135       * addIfAbsent adds the element when it does not exist in the list
136       */
137      public void testAddIfAbsent2() {
138 <        CopyOnWriteArrayList list = populatedList(SIZE);
138 >        CopyOnWriteArrayList<Item> list = populatedList(SIZE);
139          list.addIfAbsent(three);
140 <        assertTrue(list.contains(three));
140 >        mustContain(list, three);
141      }
142  
143      /**
144       * clear removes all elements from the list
145       */
146      public void testClear() {
147 <        List list = populatedList(SIZE);
147 >        List<Item> list = populatedList(SIZE);
148          list.clear();
149 <        assertEquals(0, list.size());
149 >        mustEqual(0, list.size());
150      }
151  
152      /**
153       * Cloned list is equal
154       */
155      public void testClone() {
156 <        CopyOnWriteArrayList l1 = populatedList(SIZE);
157 <        CopyOnWriteArrayList l2 = (CopyOnWriteArrayList)(l1.clone());
158 <        assertEquals(l1, l2);
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      }
# Line 167 | Line 165 | public class CopyOnWriteArrayListTest ex
165       * contains is true for added elements
166       */
167      public void testContains() {
168 <        List list = populatedList(3);
169 <        assertTrue(list.contains(one));
170 <        assertFalse(list.contains(five));
168 >        List<Item> list = populatedList(3);
169 >        mustContain(list, one);
170 >        mustNotContain(list, five);
171      }
172  
173      /**
174       * adding at an index places it in the indicated index
175       */
176      public void testAddIndex() {
177 <        List list = populatedList(3);
178 <        list.add(0, m1);
179 <        assertEquals(4, list.size());
180 <        assertEquals(m1, list.get(0));
181 <        assertEquals(zero, list.get(1));
182 <
183 <        list.add(2, m2);
184 <        assertEquals(5, list.size());
185 <        assertEquals(m2, list.get(2));
186 <        assertEquals(two, list.get(4));
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 >        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       * lists with same elements are equal and have same hashCode
191       */
192      public void testEquals() {
193 <        List a = populatedList(3);
194 <        List b = populatedList(3);
193 >        List<Item> a = populatedList(3);
194 >        List<Item> b = populatedList(3);
195          assertTrue(a.equals(b));
196          assertTrue(b.equals(a));
197          assertTrue(a.containsAll(b));
198          assertTrue(b.containsAll(a));
199 <        assertEquals(a.hashCode(), b.hashCode());
200 <        a.add(m1);
199 >        mustEqual(a.hashCode(), b.hashCode());
200 >        a.add(minusOne);
201          assertFalse(a.equals(b));
202          assertFalse(b.equals(a));
203          assertTrue(a.containsAll(b));
204          assertFalse(b.containsAll(a));
205 <        b.add(m1);
205 >        b.add(minusOne);
206          assertTrue(a.equals(b));
207          assertTrue(b.equals(a));
208          assertTrue(a.containsAll(b));
209          assertTrue(b.containsAll(a));
210 <        assertEquals(a.hashCode(), b.hashCode());
210 >        mustEqual(a.hashCode(), b.hashCode());
211  
212          assertFalse(a.equals(null));
213      }
# Line 218 | Line 216 | public class CopyOnWriteArrayListTest ex
216       * containsAll returns true for collections with subset of elements
217       */
218      public void testContainsAll() {
219 <        List list = populatedList(3);
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)));
# Line 235 | Line 233 | public class CopyOnWriteArrayListTest ex
233       * get returns the value at the given index
234       */
235      public void testGet() {
236 <        List list = populatedList(3);
237 <        assertEquals(0, list.get(0));
236 >        List<Item> list = populatedList(3);
237 >        mustEqual(0, list.get(0));
238      }
239  
240      /**
# Line 245 | Line 243 | public class CopyOnWriteArrayListTest ex
243       * contain the element
244       */
245      public void testIndexOf() {
246 <        List list = populatedList(3);
247 <        assertEquals(-1, list.indexOf(-42));
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 <            assertEquals(i, list.indexOf(i));
251 <            assertEquals(i, list.subList(0, size).indexOf(i));
252 <            assertEquals(i, list.subList(0, i + 1).indexOf(i));
253 <            assertEquals(-1, list.subList(0, i).indexOf(i));
254 <            assertEquals(0, list.subList(i, size).indexOf(i));
255 <            assertEquals(-1, list.subList(i + 1, size).indexOf(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(1);
260 <        assertEquals(1, list.indexOf(1));
261 <        assertEquals(1, list.subList(0, size + 1).indexOf(1));
262 <        assertEquals(0, list.subList(1, size + 1).indexOf(1));
263 <        assertEquals(size - 2, list.subList(2, size + 1).indexOf(1));
264 <        assertEquals(0, list.subList(size, size + 1).indexOf(1));
265 <        assertEquals(-1, list.subList(size + 1, size + 1).indexOf(1));
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      /**
# Line 272 | Line 271 | public class CopyOnWriteArrayListTest ex
271       * or returns -1 if the element is not found
272       */
273      public void testIndexOf2() {
274 <        CopyOnWriteArrayList list = populatedList(3);
274 >        CopyOnWriteArrayList<Item> list = populatedList(3);
275          int size = list.size();
276 <        assertEquals(-1, list.indexOf(-42, 0));
276 >        mustEqual(-1, list.indexOf(minusTen, 0));
277  
278          // we might expect IOOBE, but spec says otherwise
279 <        assertEquals(-1, list.indexOf(0, size));
280 <        assertEquals(-1, list.indexOf(0, Integer.MAX_VALUE));
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(0, -1),
285 <            () -> list.indexOf(0, Integer.MIN_VALUE));
284 >            () -> list.indexOf(zero, -1),
285 >            () -> list.indexOf(zero, Integer.MIN_VALUE));
286  
287          for (int i = 0; i < size; i++) {
288 <            assertEquals(i, list.indexOf(i, 0));
289 <            assertEquals(i, list.indexOf(i, i));
290 <            assertEquals(-1, list.indexOf(i, i + 1));
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(1);
295 <        assertEquals(1, list.indexOf(1, 0));
296 <        assertEquals(1, list.indexOf(1, 1));
297 <        assertEquals(size, list.indexOf(1, 2));
298 <        assertEquals(size, list.indexOf(1, size));
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 true when empty, else false
303       */
304      public void testIsEmpty() {
305 <        List empty = new CopyOnWriteArrayList();
305 >        List<Item> empty = new CopyOnWriteArrayList<Item>();
306          assertTrue(empty.isEmpty());
307          assertTrue(empty.subList(0, 0).isEmpty());
308  
309 <        List full = populatedList(SIZE);
309 >        List<Item> full = populatedList(SIZE);
310          assertFalse(full.isEmpty());
311          assertTrue(full.subList(0, 0).isEmpty());
312          assertTrue(full.subList(SIZE, SIZE).isEmpty());
# Line 317 | Line 317 | public class CopyOnWriteArrayListTest ex
317       * list in insertion order
318       */
319      public void testIterator() {
320 <        Collection empty = new CopyOnWriteArrayList();
320 >        Collection<Item> empty = new CopyOnWriteArrayList<Item>();
321          assertFalse(empty.iterator().hasNext());
322          try {
323              empty.iterator().next();
324              shouldThrow();
325          } catch (NoSuchElementException success) {}
326  
327 <        Integer[] elements = new Integer[SIZE];
328 <        for (int i = 0; i < SIZE; i++)
329 <            elements[i] = i;
327 >        Item[] elements = seqItems(SIZE);
328          shuffle(elements);
329 <        Collection<Integer> full = populatedList(elements);
329 >        Collection<Item> full = populatedList(elements);
330  
331 <        Iterator it = full.iterator();
331 >        Iterator<? extends Item> it = full.iterator();
332          for (int j = 0; j < SIZE; j++) {
333              assertTrue(it.hasNext());
334 <            assertEquals(elements[j], it.next());
334 >            mustEqual(elements[j], it.next());
335          }
336          assertIteratorExhausted(it);
337      }
# Line 342 | Line 340 | public class CopyOnWriteArrayListTest ex
340       * iterator of empty collection has no elements
341       */
342      public void testEmptyIterator() {
343 <        Collection c = new CopyOnWriteArrayList();
343 >        Collection<Item> c = new CopyOnWriteArrayList<Item>();
344          assertIteratorExhausted(c.iterator());
345      }
346  
# Line 350 | Line 348 | public class CopyOnWriteArrayListTest ex
348       * iterator.remove throws UnsupportedOperationException
349       */
350      public void testIteratorRemove() {
351 <        CopyOnWriteArrayList list = populatedList(SIZE);
352 <        Iterator it = list.iterator();
351 >        CopyOnWriteArrayList<Item> list = populatedList(SIZE);
352 >        Iterator<? extends Item> it = list.iterator();
353          it.next();
354          try {
355              it.remove();
# Line 363 | Line 361 | public class CopyOnWriteArrayListTest ex
361       * toString contains toString of elements
362       */
363      public void testToString() {
364 <        assertEquals("[]", new CopyOnWriteArrayList().toString());
365 <        List list = populatedList(3);
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 <        assertEquals(new ArrayList(list).toString(),
369 >        mustEqual(new ArrayList<Item>(list).toString(),
370                       list.toString());
371      }
372  
# Line 378 | Line 376 | public class CopyOnWriteArrayListTest ex
376       * contain the element
377       */
378      public void testLastIndexOf1() {
379 <        List list = populatedList(3);
380 <        assertEquals(-1, list.lastIndexOf(-42));
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 <            assertEquals(i, list.lastIndexOf(i));
384 <            assertEquals(i, list.subList(0, size).lastIndexOf(i));
385 <            assertEquals(i, list.subList(0, i + 1).lastIndexOf(i));
386 <            assertEquals(-1, list.subList(0, i).lastIndexOf(i));
387 <            assertEquals(0, list.subList(i, size).lastIndexOf(i));
388 <            assertEquals(-1, list.subList(i + 1, size).lastIndexOf(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 <        list.add(1);
393 <        assertEquals(size, list.lastIndexOf(1));
394 <        assertEquals(size, list.subList(0, size + 1).lastIndexOf(1));
395 <        assertEquals(1, list.subList(0, size).lastIndexOf(1));
396 <        assertEquals(0, list.subList(1, 2).lastIndexOf(1));
397 <        assertEquals(-1, list.subList(0, 1).indexOf(1));
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      /**
# Line 404 | Line 403 | public class CopyOnWriteArrayListTest ex
403       * returns -1 if the element is not found
404       */
405      public void testLastIndexOf2() {
406 <        CopyOnWriteArrayList list = populatedList(3);
406 >        CopyOnWriteArrayList<Item> list = populatedList(3);
407  
408          // we might expect IOOBE, but spec says otherwise
409 <        assertEquals(-1, list.lastIndexOf(0, -1));
409 >        mustEqual(-1, list.lastIndexOf(zero, -1));
410  
411          int size = list.size();
412          assertThrows(
413              IndexOutOfBoundsException.class,
414 <            () -> list.lastIndexOf(0, size),
415 <            () -> list.lastIndexOf(0, Integer.MAX_VALUE));
414 >            () -> list.lastIndexOf(zero, size),
415 >            () -> list.lastIndexOf(zero, Integer.MAX_VALUE));
416  
417          for (int i = 0; i < size; i++) {
418 <            assertEquals(i, list.lastIndexOf(i, i));
419 <            assertEquals(list.indexOf(i), list.lastIndexOf(i, 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 <                assertEquals(-1, list.lastIndexOf(i, i - 1));
422 >                mustEqual(-1, list.lastIndexOf(I, i - 1));
423          }
424          list.add(one);
425          list.add(three);
426 <        assertEquals(1, list.lastIndexOf(one, 1));
427 <        assertEquals(1, list.lastIndexOf(one, 2));
428 <        assertEquals(3, list.lastIndexOf(one, 3));
429 <        assertEquals(3, list.lastIndexOf(one, 4));
430 <        assertEquals(-1, list.lastIndexOf(three, 3));
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       * listIterator traverses all elements
435       */
436      public void testListIterator1() {
437 <        List list = populatedList(SIZE);
438 <        ListIterator i = list.listIterator();
437 >        List<Item> list = populatedList(SIZE);
438 >        ListIterator<? extends Item> i = list.listIterator();
439          int j;
440          for (j = 0; i.hasNext(); j++)
441 <            assertEquals(j, i.next());
442 <        assertEquals(SIZE, j);
441 >            mustEqual(j, i.next());
442 >        mustEqual(SIZE, j);
443      }
444  
445      /**
446       * listIterator only returns those elements after the given index
447       */
448      public void testListIterator2() {
449 <        List list = populatedList(3);
450 <        ListIterator i = list.listIterator(1);
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 <            assertEquals(j + 1, i.next());
454 <        assertEquals(2, j);
453 >            mustEqual(j + 1, i.next());
454 >        mustEqual(2, j);
455      }
456  
457      /**
# Line 460 | Line 460 | public class CopyOnWriteArrayListTest ex
460      public void testRemove_int() {
461          int SIZE = 3;
462          for (int i = 0; i < SIZE; i++) {
463 <            List list = populatedList(SIZE);
464 <            assertEquals(i, list.remove(i));
465 <            assertEquals(SIZE - 1, list.size());
466 <            assertFalse(list.contains(new Integer(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  
# Line 473 | Line 473 | public class CopyOnWriteArrayListTest ex
473      public void testRemove_Object() {
474          int SIZE = 3;
475          for (int i = 0; i < SIZE; i++) {
476 <            List list = populatedList(SIZE);
477 <            assertFalse(list.remove(new Integer(-42)));
478 <            assertTrue(list.remove(new Integer(i)));
479 <            assertEquals(SIZE - 1, list.size());
480 <            assertFalse(list.contains(new Integer(i)));
481 <        }
482 <        CopyOnWriteArrayList x = new CopyOnWriteArrayList(Arrays.asList(4, 5, 6));
483 <        assertTrue(x.remove(new Integer(6)));
484 <        assertEquals(x, Arrays.asList(4, 5));
485 <        assertTrue(x.remove(new Integer(4)));
486 <        assertEquals(x, Arrays.asList(5));
487 <        assertTrue(x.remove(new Integer(5)));
488 <        assertEquals(x, Arrays.asList());
489 <        assertFalse(x.remove(new Integer(5)));
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       * removeAll removes all elements from the given collection
494       */
495      public void testRemoveAll() {
496 <        List list = populatedList(3);
496 >        List<Item> list = populatedList(3);
497          assertTrue(list.removeAll(Arrays.asList(one, two)));
498 <        assertEquals(1, list.size());
498 >        mustEqual(1, list.size());
499          assertFalse(list.removeAll(Arrays.asList(one, two)));
500 <        assertEquals(1, list.size());
500 >        mustEqual(1, list.size());
501      }
502  
503      /**
504       * set changes the element at the given index
505       */
506      public void testSet() {
507 <        List list = populatedList(3);
508 <        assertEquals(2, list.set(2, four));
509 <        assertEquals(4, list.get(2));
507 >        List<Item> list = populatedList(3);
508 >        mustEqual(2, list.set(2, four));
509 >        mustEqual(4, list.get(2));
510      }
511  
512      /**
513       * size returns the number of elements
514       */
515      public void testSize() {
516 <        List empty = new CopyOnWriteArrayList();
517 <        assertEquals(0, empty.size());
518 <        assertEquals(0, empty.subList(0, 0).size());
519 <
520 <        List full = populatedList(SIZE);
521 <        assertEquals(SIZE, full.size());
522 <        assertEquals(0, full.subList(0, 0).size());
523 <        assertEquals(0, full.subList(SIZE, SIZE).size());
516 >        List<Item> empty = new CopyOnWriteArrayList<Item>();
517 >        mustEqual(0, empty.size());
518 >        mustEqual(0, empty.subList(0, 0).size());
519 >
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      /**
# Line 528 | Line 528 | public class CopyOnWriteArrayListTest ex
528       * the list in insertion order
529       */
530      public void testToArray() {
531 <        Object[] a = new CopyOnWriteArrayList().toArray();
531 >        Object[] a = new CopyOnWriteArrayList<Item>().toArray();
532          assertTrue(Arrays.equals(new Object[0], a));
533          assertSame(Object[].class, a.getClass());
534  
535 <        Integer[] elements = new Integer[SIZE];
536 <        for (int i = 0; i < SIZE; i++)
537 <            elements[i] = i;
535 >        Item[] elements = seqItems(SIZE);
536          shuffle(elements);
537 <        Collection<Integer> full = populatedList(elements);
537 >        Collection<Item> full = populatedList(elements);
538  
539          assertTrue(Arrays.equals(elements, full.toArray()));
540          assertSame(Object[].class, full.toArray().getClass());
541      }
542  
543      /**
544 <     * toArray(Integer array) returns an Integer array containing all
544 >     * toArray(Item array) returns an Item array containing all
545       * elements from the list in insertion order
546       */
547      public void testToArray2() {
548 <        Collection empty = new CopyOnWriteArrayList();
549 <        Integer[] a;
548 >        Collection<Item> empty = new CopyOnWriteArrayList<Item>();
549 >        Item[] a;
550  
551 <        a = new Integer[0];
551 >        a = new Item[0];
552          assertSame(a, empty.toArray(a));
553  
554 <        a = new Integer[SIZE / 2];
555 <        Arrays.fill(a, 42);
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 <            assertEquals(42, (int) a[i]);
559 >            mustEqual(42, a[i]);
560  
561 <        Integer[] elements = new Integer[SIZE];
564 <        for (int i = 0; i < SIZE; i++)
565 <            elements[i] = i;
561 >        Item[] elements = seqItems(SIZE);
562          shuffle(elements);
563 <        Collection<Integer> full = populatedList(elements);
563 >        Collection<Item> full = populatedList(elements);
564  
565 <        Arrays.fill(a, 42);
565 >        Arrays.fill(a, fortytwo);
566          assertTrue(Arrays.equals(elements, full.toArray(a)));
567          for (int i = 0; i < a.length; i++)
568 <            assertEquals(42, (int) a[i]);
569 <        assertSame(Integer[].class, full.toArray(a).getClass());
568 >            mustEqual(42, a[i]);
569 >        assertSame(Item[].class, full.toArray(a).getClass());
570  
571 <        a = new Integer[SIZE];
572 <        Arrays.fill(a, 42);
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 Integer[2 * SIZE];
577 <        Arrays.fill(a, 42);
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 <            assertEquals(42, (int) a[i]);
582 >            mustEqual(42, a[i]);
583      }
584  
585      /**
586       * sublists contains elements at indexes offset from their base
587       */
588      public void testSubList() {
589 <        List a = populatedList(10);
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 b = a.subList(j,i);
593 >                List<Item> b = a.subList(j,i);
594                  for (int k = j; k < i; ++k) {
595 <                    assertEquals(new Integer(k), b.get(k-j));
595 >                    mustEqual(itemFor(k), b.get(k-j));
596                  }
597              }
598          }
599  
600 <        List s = a.subList(2, 5);
601 <        assertEquals(3, s.size());
602 <        s.set(2, m1);
603 <        assertEquals(a.get(4), m1);
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 <        assertEquals(7, a.size());
605 >        mustEqual(7, a.size());
606  
607          assertThrows(
608              IndexOutOfBoundsException.class,
609              () -> s.get(0),
610 <            () -> s.set(0, 42));
610 >            () -> s.set(0, fortytwo));
611      }
612  
613      // Exception tests
# Line 621 | Line 617 | public class CopyOnWriteArrayListTest ex
617       * can not store the objects inside the list
618       */
619      public void testToArray_ArrayStoreException() {
620 <        List list = new CopyOnWriteArrayList();
621 <        // Integers are not auto-converted to Longs
622 <        list.add(86);
623 <        list.add(99);
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(
# Line 670 | Line 667 | public class CopyOnWriteArrayListTest ex
667       */
668      public void testIndexOutOfBoundsException() {
669          ThreadLocalRandom rnd = ThreadLocalRandom.current();
670 <        List x = populatedList(rnd.nextInt(5));
670 >        List<Item> x = populatedList(rnd.nextInt(5));
671          testIndexOutOfBoundsException(x);
672  
673          int start = rnd.nextInt(x.size() + 1);
# Line 678 | Line 675 | public class CopyOnWriteArrayListTest ex
675          assertThrows(
676              IndexOutOfBoundsException.class,
677              () -> x.subList(start, start - 1));
678 <        List subList = x.subList(start, end);
678 >        List<Item> subList = x.subList(start, end);
679          testIndexOutOfBoundsException(x);
680      }
681  
# Line 686 | Line 683 | public class CopyOnWriteArrayListTest ex
683       * a deserialized/reserialized list equals original
684       */
685      public void testSerialization() throws Exception {
686 <        List x = populatedList(SIZE);
687 <        List y = serialClone(x);
686 >        List<Item> x = populatedList(SIZE);
687 >        List<Item> y = serialClone(x);
688  
689          assertNotSame(x, y);
690 <        assertEquals(x.size(), y.size());
691 <        assertEquals(x.toString(), y.toString());
690 >        mustEqual(x.size(), y.size());
691 >        mustEqual(x.toString(), y.toString());
692          assertTrue(Arrays.equals(x.toArray(), y.toArray()));
693 <        assertEquals(x, y);
694 <        assertEquals(y, x);
693 >        mustEqual(x, y);
694 >        mustEqual(y, x);
695          while (!x.isEmpty()) {
696              assertFalse(y.isEmpty());
697 <            assertEquals(x.remove(0), y.remove(0));
697 >            mustEqual(x.remove(0), y.remove(0));
698          }
699          assertTrue(y.isEmpty());
700      }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines