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

Comparing jsr166/src/test/tck/ArrayBlockingQueueTest.java (file contents):
Revision 1.15 by jsr166, Sat Nov 21 02:07:26 2009 UTC vs.
Revision 1.74 by jsr166, Sun Oct 30 21:07:27 2016 UTC

# Line 1 | Line 1
1   /*
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/licenses/publicdomain
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 static java.util.concurrent.TimeUnit.MILLISECONDS;
10  
11 < import junit.framework.*;
12 < import java.util.*;
13 < import java.util.concurrent.*;
14 < import java.io.*;
11 > import java.util.ArrayList;
12 > import java.util.Arrays;
13 > import java.util.Collection;
14 > import java.util.Iterator;
15 > import java.util.NoSuchElementException;
16 > import java.util.Queue;
17 > import java.util.concurrent.ArrayBlockingQueue;
18 > import java.util.concurrent.BlockingQueue;
19 > import java.util.concurrent.CountDownLatch;
20 > import java.util.concurrent.Executors;
21 > import java.util.concurrent.ExecutorService;
22 > import java.util.concurrent.ThreadLocalRandom;
23 >
24 > import junit.framework.Test;
25  
26   public class ArrayBlockingQueueTest extends JSR166TestCase {
27 +
28 +    public static class Fair extends BlockingQueueTest {
29 +        protected BlockingQueue emptyCollection() {
30 +            return new ArrayBlockingQueue(SIZE, true);
31 +        }
32 +    }
33 +
34 +    public static class NonFair extends BlockingQueueTest {
35 +        protected BlockingQueue emptyCollection() {
36 +            return new ArrayBlockingQueue(SIZE, false);
37 +        }
38 +    }
39 +
40      public static void main(String[] args) {
41 <        junit.textui.TestRunner.run (suite());
41 >        main(suite(), args);
42      }
43 +
44      public static Test suite() {
45 <        return new TestSuite(ArrayBlockingQueueTest.class);
45 >        class Implementation implements CollectionImplementation {
46 >            public Class<?> klazz() { return ArrayBlockingQueue.class; }
47 >            public Collection emptyCollection() { return new ArrayBlockingQueue(SIZE, false); }
48 >            public Object makeElement(int i) { return i; }
49 >            public boolean isConcurrent() { return true; }
50 >            public boolean permitsNulls() { return false; }
51 >        }
52 >        return newTestSuite(ArrayBlockingQueueTest.class,
53 >                            new Fair().testSuite(),
54 >                            new NonFair().testSuite(),
55 >                            CollectionTest.testSuite(new Implementation()));
56      }
57  
58      /**
59 <     * Create a queue of given size containing consecutive
60 <     * Integers 0 ... n.
59 >     * Returns a new queue of given size containing consecutive
60 >     * Integers 0 ... n - 1.
61       */
62 <    private ArrayBlockingQueue populatedQueue(int n) {
63 <        ArrayBlockingQueue q = new ArrayBlockingQueue(n);
62 >    private ArrayBlockingQueue<Integer> populatedQueue(int n) {
63 >        ArrayBlockingQueue<Integer> q = new ArrayBlockingQueue<Integer>(n);
64          assertTrue(q.isEmpty());
65          for (int i = 0; i < n; i++)
66              assertTrue(q.offer(new Integer(i)));
67          assertFalse(q.isEmpty());
68          assertEquals(0, q.remainingCapacity());
69          assertEquals(n, q.size());
70 +        assertEquals((Integer) 0, q.peek());
71          return q;
72      }
73  
# Line 43 | Line 79 | public class ArrayBlockingQueueTest exte
79      }
80  
81      /**
82 <     * Constructor throws IAE if  capacity argument nonpositive
82 >     * Constructor throws IAE if capacity argument nonpositive
83       */
84      public void testConstructor2() {
85          try {
86 <            ArrayBlockingQueue q = new ArrayBlockingQueue(0);
86 >            new ArrayBlockingQueue(0);
87              shouldThrow();
88 <        }
53 <        catch (IllegalArgumentException success) {}
88 >        } catch (IllegalArgumentException success) {}
89      }
90  
91      /**
# Line 58 | Line 93 | public class ArrayBlockingQueueTest exte
93       */
94      public void testConstructor3() {
95          try {
96 <            ArrayBlockingQueue q = new ArrayBlockingQueue(1, true, null);
96 >            new ArrayBlockingQueue(1, true, null);
97              shouldThrow();
98 <        }
64 <        catch (NullPointerException success) {}
98 >        } catch (NullPointerException success) {}
99      }
100  
101      /**
102       * Initializing from Collection of null elements throws NPE
103       */
104      public void testConstructor4() {
105 +        Collection<Integer> elements = Arrays.asList(new Integer[SIZE]);
106          try {
107 <            Integer[] ints = new Integer[SIZE];
73 <            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE, false, Arrays.asList(ints));
107 >            new ArrayBlockingQueue(SIZE, false, elements);
108              shouldThrow();
109 <        }
76 <        catch (NullPointerException success) {}
109 >        } catch (NullPointerException success) {}
110      }
111  
112      /**
113       * Initializing from Collection with some null elements throws NPE
114       */
115      public void testConstructor5() {
116 +        Integer[] ints = new Integer[SIZE];
117 +        for (int i = 0; i < SIZE - 1; ++i)
118 +            ints[i] = i;
119 +        Collection<Integer> elements = Arrays.asList(ints);
120          try {
121 <            Integer[] ints = new Integer[SIZE];
85 <            for (int i = 0; i < SIZE-1; ++i)
86 <                ints[i] = new Integer(i);
87 <            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE, false, Arrays.asList(ints));
121 >            new ArrayBlockingQueue(SIZE, false, elements);
122              shouldThrow();
123 <        }
90 <        catch (NullPointerException success) {}
123 >        } catch (NullPointerException success) {}
124      }
125  
126      /**
127       * Initializing from too large collection throws IAE
128       */
129      public void testConstructor6() {
130 +        Integer[] ints = new Integer[SIZE];
131 +        for (int i = 0; i < SIZE; ++i)
132 +            ints[i] = i;
133 +        Collection<Integer> elements = Arrays.asList(ints);
134          try {
135 <            Integer[] ints = new Integer[SIZE];
99 <            for (int i = 0; i < SIZE; ++i)
100 <                ints[i] = new Integer(i);
101 <            ArrayBlockingQueue q = new ArrayBlockingQueue(1, false, Arrays.asList(ints));
135 >            new ArrayBlockingQueue(SIZE - 1, false, elements);
136              shouldThrow();
137 <        }
104 <        catch (IllegalArgumentException success) {}
137 >        } catch (IllegalArgumentException success) {}
138      }
139  
140      /**
141       * Queue contains all elements of collection used to initialize
142       */
143      public void testConstructor7() {
144 <        try {
145 <            Integer[] ints = new Integer[SIZE];
146 <            for (int i = 0; i < SIZE; ++i)
147 <                ints[i] = new Integer(i);
148 <            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE, true, Arrays.asList(ints));
149 <            for (int i = 0; i < SIZE; ++i)
150 <                assertEquals(ints[i], q.poll());
118 <        }
119 <        finally {}
144 >        Integer[] ints = new Integer[SIZE];
145 >        for (int i = 0; i < SIZE; ++i)
146 >            ints[i] = i;
147 >        Collection<Integer> elements = Arrays.asList(ints);
148 >        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE, true, elements);
149 >        for (int i = 0; i < SIZE; ++i)
150 >            assertEquals(ints[i], q.poll());
151      }
152  
153      /**
# Line 138 | Line 169 | public class ArrayBlockingQueueTest exte
169       * remainingCapacity decreases on add, increases on remove
170       */
171      public void testRemainingCapacity() {
172 <        ArrayBlockingQueue q = populatedQueue(SIZE);
172 >        BlockingQueue q = populatedQueue(SIZE);
173          for (int i = 0; i < SIZE; ++i) {
174              assertEquals(i, q.remainingCapacity());
175 <            assertEquals(SIZE-i, q.size());
176 <            q.remove();
175 >            assertEquals(SIZE, q.size() + q.remainingCapacity());
176 >            assertEquals(i, q.remove());
177          }
178          for (int i = 0; i < SIZE; ++i) {
179 <            assertEquals(SIZE-i, q.remainingCapacity());
180 <            assertEquals(i, q.size());
181 <            q.add(new Integer(i));
179 >            assertEquals(SIZE - i, q.remainingCapacity());
180 >            assertEquals(SIZE, q.size() + q.remainingCapacity());
181 >            assertTrue(q.add(i));
182          }
183      }
184  
185      /**
155     *  offer(null) throws NPE
156     */
157    public void testOfferNull() {
158        try {
159            ArrayBlockingQueue q = new ArrayBlockingQueue(1);
160            q.offer(null);
161            shouldThrow();
162        } catch (NullPointerException success) { }
163    }
164
165    /**
166     *  add(null) throws NPE
167     */
168    public void testAddNull() {
169        try {
170            ArrayBlockingQueue q = new ArrayBlockingQueue(1);
171            q.add(null);
172            shouldThrow();
173        } catch (NullPointerException success) { }
174    }
175
176    /**
186       * Offer succeeds if not full; fails if full
187       */
188      public void testOffer() {
# Line 186 | Line 195 | public class ArrayBlockingQueueTest exte
195       * add succeeds if not full; throws ISE if full
196       */
197      public void testAdd() {
198 <        try {
199 <            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
200 <            for (int i = 0; i < SIZE; ++i) {
192 <                assertTrue(q.add(new Integer(i)));
193 <            }
194 <            assertEquals(0, q.remainingCapacity());
195 <            q.add(new Integer(SIZE));
196 <        } catch (IllegalStateException success) {
198 >        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
199 >        for (int i = 0; i < SIZE; ++i) {
200 >            assertTrue(q.add(new Integer(i)));
201          }
202 <    }
199 <
200 <    /**
201 <     *  addAll(null) throws NPE
202 <     */
203 <    public void testAddAll1() {
202 >        assertEquals(0, q.remainingCapacity());
203          try {
204 <            ArrayBlockingQueue q = new ArrayBlockingQueue(1);
206 <            q.addAll(null);
204 >            q.add(new Integer(SIZE));
205              shouldThrow();
206 <        }
209 <        catch (NullPointerException success) {}
206 >        } catch (IllegalStateException success) {}
207      }
208  
209      /**
210       * addAll(this) throws IAE
211       */
212      public void testAddAllSelf() {
213 +        ArrayBlockingQueue q = populatedQueue(SIZE);
214          try {
217            ArrayBlockingQueue q = populatedQueue(SIZE);
215              q.addAll(q);
216              shouldThrow();
217 <        }
221 <        catch (IllegalArgumentException success) {}
217 >        } catch (IllegalArgumentException success) {}
218      }
219  
224
225    /**
226     *  addAll of a collection with null elements throws NPE
227     */
228    public void testAddAll2() {
229        try {
230            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
231            Integer[] ints = new Integer[SIZE];
232            q.addAll(Arrays.asList(ints));
233            shouldThrow();
234        }
235        catch (NullPointerException success) {}
236    }
220      /**
221       * addAll of a collection with any null elements throws NPE after
222       * possibly adding some elements
223       */
224      public void testAddAll3() {
225 +        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
226 +        Integer[] ints = new Integer[SIZE];
227 +        for (int i = 0; i < SIZE - 1; ++i)
228 +            ints[i] = new Integer(i);
229          try {
243            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
244            Integer[] ints = new Integer[SIZE];
245            for (int i = 0; i < SIZE-1; ++i)
246                ints[i] = new Integer(i);
230              q.addAll(Arrays.asList(ints));
231              shouldThrow();
232 <        }
250 <        catch (NullPointerException success) {}
232 >        } catch (NullPointerException success) {}
233      }
234 +
235      /**
236       * addAll throws ISE if not enough room
237       */
238      public void testAddAll4() {
239 +        ArrayBlockingQueue q = new ArrayBlockingQueue(1);
240 +        Integer[] ints = new Integer[SIZE];
241 +        for (int i = 0; i < SIZE; ++i)
242 +            ints[i] = new Integer(i);
243          try {
257            ArrayBlockingQueue q = new ArrayBlockingQueue(1);
258            Integer[] ints = new Integer[SIZE];
259            for (int i = 0; i < SIZE; ++i)
260                ints[i] = new Integer(i);
244              q.addAll(Arrays.asList(ints));
245              shouldThrow();
246 <        }
264 <        catch (IllegalStateException success) {}
246 >        } catch (IllegalStateException success) {}
247      }
248 +
249      /**
250       * Queue contains all elements, in traversal order, of successful addAll
251       */
252      public void testAddAll5() {
253 <        try {
254 <            Integer[] empty = new Integer[0];
255 <            Integer[] ints = new Integer[SIZE];
256 <            for (int i = 0; i < SIZE; ++i)
257 <                ints[i] = new Integer(i);
258 <            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
259 <            assertFalse(q.addAll(Arrays.asList(empty)));
260 <            assertTrue(q.addAll(Arrays.asList(ints)));
261 <            for (int i = 0; i < SIZE; ++i)
279 <                assertEquals(ints[i], q.poll());
280 <        }
281 <        finally {}
253 >        Integer[] empty = new Integer[0];
254 >        Integer[] ints = new Integer[SIZE];
255 >        for (int i = 0; i < SIZE; ++i)
256 >            ints[i] = new Integer(i);
257 >        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
258 >        assertFalse(q.addAll(Arrays.asList(empty)));
259 >        assertTrue(q.addAll(Arrays.asList(ints)));
260 >        for (int i = 0; i < SIZE; ++i)
261 >            assertEquals(ints[i], q.poll());
262      }
263  
264      /**
285     *  put(null) throws NPE
286     */
287     public void testPutNull() {
288        try {
289            ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
290            q.put(null);
291            shouldThrow();
292        }
293        catch (NullPointerException success) {
294        }
295        catch (InterruptedException ie) {
296            unexpectedException();
297        }
298     }
299
300    /**
265       * all elements successfully put are contained
266       */
267 <     public void testPut() {
268 <         try {
269 <             ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
270 <             for (int i = 0; i < SIZE; ++i) {
271 <                 Integer I = new Integer(i);
272 <                 q.put(I);
309 <                 assertTrue(q.contains(I));
310 <             }
311 <             assertEquals(0, q.remainingCapacity());
312 <         }
313 <        catch (InterruptedException ie) {
314 <            unexpectedException();
267 >    public void testPut() throws InterruptedException {
268 >        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
269 >        for (int i = 0; i < SIZE; ++i) {
270 >            Integer x = new Integer(i);
271 >            q.put(x);
272 >            assertTrue(q.contains(x));
273          }
274 +        assertEquals(0, q.remainingCapacity());
275      }
276  
277      /**
278       * put blocks interruptibly if full
279       */
280 <    public void testBlockingPut() {
280 >    public void testBlockingPut() throws InterruptedException {
281          final ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE);
282 <        Thread t = new Thread(new Runnable() {
283 <                public void run() {
284 <                    int added = 0;
285 <                    try {
286 <                        for (int i = 0; i < SIZE; ++i) {
287 <                            q.put(new Integer(i));
288 <                            ++added;
289 <                        }
290 <                        q.put(new Integer(SIZE));
291 <                        threadShouldThrow();
292 <                    } catch (InterruptedException ie) {
293 <                        threadAssertEquals(added, SIZE);
294 <                    }
295 <                }});
296 <        try {
297 <            t.start();
298 <           Thread.sleep(MEDIUM_DELAY_MS);
299 <           t.interrupt();
300 <           t.join();
301 <        }
302 <        catch (InterruptedException ie) {
303 <            unexpectedException();
304 <        }
282 >        final CountDownLatch pleaseInterrupt = new CountDownLatch(1);
283 >        Thread t = newStartedThread(new CheckedRunnable() {
284 >            public void realRun() throws InterruptedException {
285 >                for (int i = 0; i < SIZE; ++i)
286 >                    q.put(i);
287 >                assertEquals(SIZE, q.size());
288 >                assertEquals(0, q.remainingCapacity());
289 >
290 >                Thread.currentThread().interrupt();
291 >                try {
292 >                    q.put(99);
293 >                    shouldThrow();
294 >                } catch (InterruptedException success) {}
295 >                assertFalse(Thread.interrupted());
296 >
297 >                pleaseInterrupt.countDown();
298 >                try {
299 >                    q.put(99);
300 >                    shouldThrow();
301 >                } catch (InterruptedException success) {}
302 >                assertFalse(Thread.interrupted());
303 >            }});
304 >
305 >        await(pleaseInterrupt);
306 >        assertThreadStaysAlive(t);
307 >        t.interrupt();
308 >        awaitTermination(t);
309 >        assertEquals(SIZE, q.size());
310 >        assertEquals(0, q.remainingCapacity());
311      }
312  
313      /**
314 <     * put blocks waiting for take when full
314 >     * put blocks interruptibly waiting for take when full
315       */
316 <    public void testPutWithTake() {
317 <        final ArrayBlockingQueue q = new ArrayBlockingQueue(2);
318 <        Thread t = new Thread(new Runnable() {
319 <                public void run() {
320 <                    int added = 0;
321 <                    try {
322 <                        q.put(new Object());
323 <                        ++added;
324 <                        q.put(new Object());
325 <                        ++added;
326 <                        q.put(new Object());
327 <                        ++added;
328 <                        q.put(new Object());
329 <                        ++added;
330 <                        threadShouldThrow();
331 <                    } catch (InterruptedException e) {
332 <                        threadAssertTrue(added >= 2);
333 <                    }
334 <                }
335 <            });
336 <        try {
337 <            t.start();
338 <            Thread.sleep(SHORT_DELAY_MS);
339 <            q.take();
340 <            t.interrupt();
341 <            t.join();
342 <        } catch (Exception e) {
343 <            unexpectedException();
344 <        }
316 >    public void testPutWithTake() throws InterruptedException {
317 >        final int capacity = 2;
318 >        final ArrayBlockingQueue q = new ArrayBlockingQueue(capacity);
319 >        final CountDownLatch pleaseTake = new CountDownLatch(1);
320 >        final CountDownLatch pleaseInterrupt = new CountDownLatch(1);
321 >        Thread t = newStartedThread(new CheckedRunnable() {
322 >            public void realRun() throws InterruptedException {
323 >                for (int i = 0; i < capacity; i++)
324 >                    q.put(i);
325 >                pleaseTake.countDown();
326 >                q.put(86);
327 >
328 >                pleaseInterrupt.countDown();
329 >                try {
330 >                    q.put(99);
331 >                    shouldThrow();
332 >                } catch (InterruptedException success) {}
333 >                assertFalse(Thread.interrupted());
334 >            }});
335 >
336 >        await(pleaseTake);
337 >        assertEquals(0, q.remainingCapacity());
338 >        assertEquals(0, q.take());
339 >
340 >        await(pleaseInterrupt);
341 >        assertThreadStaysAlive(t);
342 >        t.interrupt();
343 >        awaitTermination(t);
344 >        assertEquals(0, q.remainingCapacity());
345      }
346  
347      /**
348       * timed offer times out if full and elements not taken
349       */
350 <    public void testTimedOffer() {
350 >    public void testTimedOffer() throws InterruptedException {
351          final ArrayBlockingQueue q = new ArrayBlockingQueue(2);
352 <        Thread t = new Thread(new Runnable() {
353 <                public void run() {
354 <                    try {
355 <                        q.put(new Object());
356 <                        q.put(new Object());
357 <                        threadAssertFalse(q.offer(new Object(), SHORT_DELAY_MS/2, TimeUnit.MILLISECONDS));
358 <                        q.offer(new Object(), LONG_DELAY_MS, TimeUnit.MILLISECONDS);
359 <                        threadShouldThrow();
360 <                    } catch (InterruptedException success) {}
361 <                }
362 <            });
363 <
364 <        try {
365 <            t.start();
366 <            Thread.sleep(SHORT_DELAY_MS);
367 <            t.interrupt();
368 <            t.join();
369 <        } catch (Exception e) {
370 <            unexpectedException();
406 <        }
352 >        final CountDownLatch pleaseInterrupt = new CountDownLatch(1);
353 >        Thread t = newStartedThread(new CheckedRunnable() {
354 >            public void realRun() throws InterruptedException {
355 >                q.put(new Object());
356 >                q.put(new Object());
357 >                long startTime = System.nanoTime();
358 >                assertFalse(q.offer(new Object(), timeoutMillis(), MILLISECONDS));
359 >                assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
360 >                pleaseInterrupt.countDown();
361 >                try {
362 >                    q.offer(new Object(), 2 * LONG_DELAY_MS, MILLISECONDS);
363 >                    shouldThrow();
364 >                } catch (InterruptedException success) {}
365 >            }});
366 >
367 >        await(pleaseInterrupt);
368 >        assertThreadStaysAlive(t);
369 >        t.interrupt();
370 >        awaitTermination(t);
371      }
372  
373      /**
374       * take retrieves elements in FIFO order
375       */
376 <    public void testTake() {
377 <        try {
378 <            ArrayBlockingQueue q = populatedQueue(SIZE);
379 <            for (int i = 0; i < SIZE; ++i) {
416 <                assertEquals(i, ((Integer)q.take()).intValue());
417 <            }
418 <        } catch (InterruptedException e) {
419 <            unexpectedException();
376 >    public void testTake() throws InterruptedException {
377 >        ArrayBlockingQueue q = populatedQueue(SIZE);
378 >        for (int i = 0; i < SIZE; ++i) {
379 >            assertEquals(i, q.take());
380          }
381      }
382  
383      /**
384 <     * take blocks interruptibly when empty
384 >     * Take removes existing elements until empty, then blocks interruptibly
385       */
386 <    public void testTakeFromEmpty() {
387 <        final ArrayBlockingQueue q = new ArrayBlockingQueue(2);
388 <        Thread t = new Thread(new Runnable() {
389 <                public void run() {
390 <                    try {
391 <                        q.take();
392 <                        threadShouldThrow();
433 <                    } catch (InterruptedException success) { }
386 >    public void testBlockingTake() throws InterruptedException {
387 >        final ArrayBlockingQueue q = populatedQueue(SIZE);
388 >        final CountDownLatch pleaseInterrupt = new CountDownLatch(1);
389 >        Thread t = newStartedThread(new CheckedRunnable() {
390 >            public void realRun() throws InterruptedException {
391 >                for (int i = 0; i < SIZE; ++i) {
392 >                    assertEquals(i, q.take());
393                  }
435            });
436        try {
437            t.start();
438            Thread.sleep(SHORT_DELAY_MS);
439            t.interrupt();
440            t.join();
441        } catch (Exception e) {
442            unexpectedException();
443        }
444    }
394  
395 <    /**
396 <     * Take removes existing elements until empty, then blocks interruptibly
397 <     */
398 <    public void testBlockingTake() {
399 <        Thread t = new Thread(new Runnable() {
400 <                public void run() {
452 <                    try {
453 <                        ArrayBlockingQueue q = populatedQueue(SIZE);
454 <                        for (int i = 0; i < SIZE; ++i) {
455 <                            threadAssertEquals(i, ((Integer)q.take()).intValue());
456 <                        }
457 <                        q.take();
458 <                        threadShouldThrow();
459 <                    } catch (InterruptedException success) {
460 <                    }
461 <                }});
462 <        try {
463 <            t.start();
464 <            Thread.sleep(SHORT_DELAY_MS);
465 <            t.interrupt();
466 <            t.join();
467 <        }
468 <        catch (InterruptedException ie) {
469 <            unexpectedException();
470 <        }
471 <    }
395 >                Thread.currentThread().interrupt();
396 >                try {
397 >                    q.take();
398 >                    shouldThrow();
399 >                } catch (InterruptedException success) {}
400 >                assertFalse(Thread.interrupted());
401  
402 +                pleaseInterrupt.countDown();
403 +                try {
404 +                    q.take();
405 +                    shouldThrow();
406 +                } catch (InterruptedException success) {}
407 +                assertFalse(Thread.interrupted());
408 +            }});
409 +
410 +        await(pleaseInterrupt);
411 +        assertThreadStaysAlive(t);
412 +        t.interrupt();
413 +        awaitTermination(t);
414 +    }
415  
416      /**
417       * poll succeeds unless empty
# Line 477 | Line 419 | public class ArrayBlockingQueueTest exte
419      public void testPoll() {
420          ArrayBlockingQueue q = populatedQueue(SIZE);
421          for (int i = 0; i < SIZE; ++i) {
422 <            assertEquals(i, ((Integer)q.poll()).intValue());
422 >            assertEquals(i, q.poll());
423          }
424          assertNull(q.poll());
425      }
426  
427      /**
428 <     * timed pool with zero timeout succeeds when non-empty, else times out
428 >     * timed poll with zero timeout succeeds when non-empty, else times out
429       */
430 <    public void testTimedPoll0() {
431 <        try {
432 <            ArrayBlockingQueue q = populatedQueue(SIZE);
433 <            for (int i = 0; i < SIZE; ++i) {
492 <                assertEquals(i, ((Integer)q.poll(0, TimeUnit.MILLISECONDS)).intValue());
493 <            }
494 <            assertNull(q.poll(0, TimeUnit.MILLISECONDS));
495 <        } catch (InterruptedException e) {
496 <            unexpectedException();
430 >    public void testTimedPoll0() throws InterruptedException {
431 >        ArrayBlockingQueue q = populatedQueue(SIZE);
432 >        for (int i = 0; i < SIZE; ++i) {
433 >            assertEquals(i, q.poll(0, MILLISECONDS));
434          }
435 +        assertNull(q.poll(0, MILLISECONDS));
436 +        checkEmpty(q);
437      }
438  
439      /**
440 <     * timed pool with nonzero timeout succeeds when non-empty, else times out
440 >     * timed poll with nonzero timeout succeeds when non-empty, else times out
441       */
442 <    public void testTimedPoll() {
443 <        try {
444 <            ArrayBlockingQueue q = populatedQueue(SIZE);
445 <            for (int i = 0; i < SIZE; ++i) {
446 <                assertEquals(i, ((Integer)q.poll(SHORT_DELAY_MS, TimeUnit.MILLISECONDS)).intValue());
447 <            }
448 <            assertNull(q.poll(SHORT_DELAY_MS, TimeUnit.MILLISECONDS));
449 <        } catch (InterruptedException e) {
450 <            unexpectedException();
451 <        }
442 >    public void testTimedPoll() throws InterruptedException {
443 >        ArrayBlockingQueue q = populatedQueue(SIZE);
444 >        for (int i = 0; i < SIZE; ++i) {
445 >            long startTime = System.nanoTime();
446 >            assertEquals(i, q.poll(LONG_DELAY_MS, MILLISECONDS));
447 >            assertTrue(millisElapsedSince(startTime) < LONG_DELAY_MS);
448 >        }
449 >        long startTime = System.nanoTime();
450 >        assertNull(q.poll(timeoutMillis(), MILLISECONDS));
451 >        assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
452 >        checkEmpty(q);
453      }
454  
455      /**
456       * Interrupted timed poll throws InterruptedException instead of
457       * returning timeout status
458       */
459 <    public void testInterruptedTimedPoll() {
460 <        Thread t = new Thread(new Runnable() {
461 <                public void run() {
462 <                    try {
463 <                        ArrayBlockingQueue q = populatedQueue(SIZE);
464 <                        for (int i = 0; i < SIZE; ++i) {
465 <                            threadAssertEquals(i, ((Integer)q.poll(SHORT_DELAY_MS, TimeUnit.MILLISECONDS)).intValue());
466 <                        }
527 <                        threadAssertNull(q.poll(SHORT_DELAY_MS, TimeUnit.MILLISECONDS));
528 <                    } catch (InterruptedException success) {
529 <                    }
530 <                }});
531 <        try {
532 <            t.start();
533 <            Thread.sleep(SHORT_DELAY_MS);
534 <            t.interrupt();
535 <            t.join();
536 <        }
537 <        catch (InterruptedException ie) {
538 <            unexpectedException();
539 <        }
540 <    }
541 <
542 <    /**
543 <     *  timed poll before a delayed offer fails; after offer succeeds;
544 <     *  on interruption throws
545 <     */
546 <    public void testTimedPollWithOffer() {
547 <        final ArrayBlockingQueue q = new ArrayBlockingQueue(2);
548 <        Thread t = new Thread(new Runnable() {
549 <                public void run() {
550 <                    try {
551 <                        threadAssertNull(q.poll(SHORT_DELAY_MS, TimeUnit.MILLISECONDS));
552 <                        q.poll(LONG_DELAY_MS, TimeUnit.MILLISECONDS);
553 <                        q.poll(LONG_DELAY_MS, TimeUnit.MILLISECONDS);
554 <                        threadShouldThrow();
555 <                    } catch (InterruptedException success) { }
459 >    public void testInterruptedTimedPoll() throws InterruptedException {
460 >        final BlockingQueue<Integer> q = populatedQueue(SIZE);
461 >        final CountDownLatch aboutToWait = new CountDownLatch(1);
462 >        Thread t = newStartedThread(new CheckedRunnable() {
463 >            public void realRun() throws InterruptedException {
464 >                long startTime = System.nanoTime();
465 >                for (int i = 0; i < SIZE; ++i) {
466 >                    assertEquals(i, (int) q.poll(LONG_DELAY_MS, MILLISECONDS));
467                  }
468 <            });
469 <        try {
470 <            t.start();
471 <            Thread.sleep(SMALL_DELAY_MS);
472 <            assertTrue(q.offer(zero, SHORT_DELAY_MS, TimeUnit.MILLISECONDS));
473 <            t.interrupt();
474 <            t.join();
475 <        } catch (Exception e) {
476 <            unexpectedException();
477 <        }
468 >                aboutToWait.countDown();
469 >                try {
470 >                    q.poll(LONG_DELAY_MS, MILLISECONDS);
471 >                    shouldThrow();
472 >                } catch (InterruptedException success) {
473 >                    assertTrue(millisElapsedSince(startTime) < LONG_DELAY_MS);
474 >                }
475 >            }});
476 >
477 >        await(aboutToWait);
478 >        waitForThreadToEnterWaitState(t);
479 >        t.interrupt();
480 >        awaitTermination(t);
481 >        checkEmpty(q);
482      }
483  
569
484      /**
485       * peek returns next element, or null if empty
486       */
487      public void testPeek() {
488          ArrayBlockingQueue q = populatedQueue(SIZE);
489          for (int i = 0; i < SIZE; ++i) {
490 <            assertEquals(i, ((Integer)q.peek()).intValue());
491 <            q.poll();
490 >            assertEquals(i, q.peek());
491 >            assertEquals(i, q.poll());
492              assertTrue(q.peek() == null ||
493 <                       i != ((Integer)q.peek()).intValue());
493 >                       !q.peek().equals(i));
494          }
495          assertNull(q.peek());
496      }
# Line 587 | Line 501 | public class ArrayBlockingQueueTest exte
501      public void testElement() {
502          ArrayBlockingQueue q = populatedQueue(SIZE);
503          for (int i = 0; i < SIZE; ++i) {
504 <            assertEquals(i, ((Integer)q.element()).intValue());
505 <            q.poll();
504 >            assertEquals(i, q.element());
505 >            assertEquals(i, q.poll());
506          }
507          try {
508              q.element();
509              shouldThrow();
510 <        }
597 <        catch (NoSuchElementException success) {}
510 >        } catch (NoSuchElementException success) {}
511      }
512  
513      /**
# Line 603 | Line 516 | public class ArrayBlockingQueueTest exte
516      public void testRemove() {
517          ArrayBlockingQueue q = populatedQueue(SIZE);
518          for (int i = 0; i < SIZE; ++i) {
519 <            assertEquals(i, ((Integer)q.remove()).intValue());
519 >            assertEquals(i, q.remove());
520          }
521          try {
522              q.remove();
523              shouldThrow();
524 <        } catch (NoSuchElementException success) {
612 <        }
613 <    }
614 <
615 <    /**
616 <     * remove(x) removes x and returns true if present
617 <     */
618 <    public void testRemoveElement() {
619 <        ArrayBlockingQueue q = populatedQueue(SIZE);
620 <        for (int i = 1; i < SIZE; i+=2) {
621 <            assertTrue(q.remove(new Integer(i)));
622 <        }
623 <        for (int i = 0; i < SIZE; i+=2) {
624 <            assertTrue(q.remove(new Integer(i)));
625 <            assertFalse(q.remove(new Integer(i+1)));
626 <        }
627 <        assertTrue(q.isEmpty());
524 >        } catch (NoSuchElementException success) {}
525      }
526  
527      /**
# Line 634 | Line 531 | public class ArrayBlockingQueueTest exte
531          ArrayBlockingQueue q = populatedQueue(SIZE);
532          for (int i = 0; i < SIZE; ++i) {
533              assertTrue(q.contains(new Integer(i)));
534 <            q.poll();
534 >            assertEquals(i, q.poll());
535              assertFalse(q.contains(new Integer(i)));
536          }
537      }
# Line 683 | Line 580 | public class ArrayBlockingQueueTest exte
580                  assertTrue(changed);
581  
582              assertTrue(q.containsAll(p));
583 <            assertEquals(SIZE-i, q.size());
583 >            assertEquals(SIZE - i, q.size());
584              p.remove();
585          }
586      }
# Line 696 | Line 593 | public class ArrayBlockingQueueTest exte
593              ArrayBlockingQueue q = populatedQueue(SIZE);
594              ArrayBlockingQueue p = populatedQueue(i);
595              assertTrue(q.removeAll(p));
596 <            assertEquals(SIZE-i, q.size());
596 >            assertEquals(SIZE - i, q.size());
597              for (int j = 0; j < i; ++j) {
598 <                Integer I = (Integer)(p.remove());
599 <                assertFalse(q.contains(I));
598 >                Integer x = (Integer)(p.remove());
599 >                assertFalse(q.contains(x));
600              }
601          }
602      }
603  
604 <    /**
605 <     *  toArray contains all elements
606 <     */
607 <    public void testToArray() {
608 <        ArrayBlockingQueue q = populatedQueue(SIZE);
609 <        Object[] o = q.toArray();
610 <        try {
611 <        for (int i = 0; i < o.length; i++)
612 <            assertEquals(o[i], q.take());
613 <        } catch (InterruptedException e) {
614 <            unexpectedException();
604 >    void checkToArray(ArrayBlockingQueue<Integer> q) {
605 >        int size = q.size();
606 >        Object[] a1 = q.toArray();
607 >        assertEquals(size, a1.length);
608 >        Integer[] a2 = q.toArray(new Integer[0]);
609 >        assertEquals(size, a2.length);
610 >        Integer[] a3 = q.toArray(new Integer[Math.max(0, size - 1)]);
611 >        assertEquals(size, a3.length);
612 >        Integer[] a4 = new Integer[size];
613 >        assertSame(a4, q.toArray(a4));
614 >        Integer[] a5 = new Integer[size + 1];
615 >        Arrays.fill(a5, 42);
616 >        assertSame(a5, q.toArray(a5));
617 >        Integer[] a6 = new Integer[size + 2];
618 >        Arrays.fill(a6, 42);
619 >        assertSame(a6, q.toArray(a6));
620 >        Object[][] as = { a1, a2, a3, a4, a5, a6 };
621 >        for (Object[] a : as) {
622 >            if (a.length > size) assertNull(a[size]);
623 >            if (a.length > size + 1) assertEquals(42, a[size + 1]);
624 >        }
625 >        Iterator it = q.iterator();
626 >        Integer s = q.peek();
627 >        for (int i = 0; i < size; i++) {
628 >            Integer x = (Integer) it.next();
629 >            assertEquals(s + i, (int) x);
630 >            for (Object[] a : as)
631 >                assertSame(a1[i], x);
632          }
633      }
634  
635      /**
636 <     * toArray(a) contains all elements
636 >     * toArray() and toArray(a) contain all elements in FIFO order
637       */
638 <    public void testToArray2() {
639 <        ArrayBlockingQueue q = populatedQueue(SIZE);
640 <        Integer[] ints = new Integer[SIZE];
641 <        ints = (Integer[])q.toArray(ints);
642 <        try {
643 <            for (int i = 0; i < ints.length; i++)
644 <                assertEquals(ints[i], q.take());
645 <        } catch (InterruptedException e) {
646 <            unexpectedException();
638 >    public void testToArray() {
639 >        final ThreadLocalRandom rnd = ThreadLocalRandom.current();
640 >        final int size = rnd.nextInt(6);
641 >        final int capacity = Math.max(1, size + rnd.nextInt(size + 1));
642 >        ArrayBlockingQueue<Integer> q = new ArrayBlockingQueue<>(capacity);
643 >        for (int i = 0; i < size; i++) {
644 >            checkToArray(q);
645 >            q.add(i);
646 >        }
647 >        // Provoke wraparound
648 >        int added = size * 2;
649 >        for (int i = 0; i < added; i++) {
650 >            checkToArray(q);
651 >            assertEquals((Integer) i, q.poll());
652 >            q.add(size + i);
653 >        }
654 >        for (int i = 0; i < size; i++) {
655 >            checkToArray(q);
656 >            assertEquals((Integer) (added + i), q.poll());
657          }
658      }
659  
660      /**
661 <     * toArray(null) throws NPE
661 >     * toArray(incompatible array type) throws ArrayStoreException
662       */
663 <    public void testToArray_BadArg() {
663 >    public void testToArray_incompatibleArrayType() {
664 >        ArrayBlockingQueue q = populatedQueue(SIZE);
665          try {
666 <            ArrayBlockingQueue q = populatedQueue(SIZE);
742 <            Object o[] = q.toArray(null);
666 >            q.toArray(new String[10]);
667              shouldThrow();
668 <        } catch (NullPointerException success) {}
745 <    }
746 <
747 <    /**
748 <     * toArray with incompatible array type throws CCE
749 <     */
750 <    public void testToArray1_BadArg() {
668 >        } catch (ArrayStoreException success) {}
669          try {
670 <            ArrayBlockingQueue q = populatedQueue(SIZE);
753 <            Object o[] = q.toArray(new String[10] );
670 >            q.toArray(new String[0]);
671              shouldThrow();
672 <        } catch (ArrayStoreException  success) {}
672 >        } catch (ArrayStoreException success) {}
673      }
674  
758
675      /**
676       * iterator iterates through all elements
677       */
678 <    public void testIterator() {
678 >    public void testIterator() throws InterruptedException {
679          ArrayBlockingQueue q = populatedQueue(SIZE);
680          Iterator it = q.iterator();
681 <        try {
682 <            while (it.hasNext()) {
683 <                assertEquals(it.next(), q.take());
684 <            }
685 <        } catch (InterruptedException e) {
686 <            unexpectedException();
687 <        }
681 >        int i;
682 >        for (i = 0; it.hasNext(); i++)
683 >            assertTrue(q.contains(it.next()));
684 >        assertEquals(i, SIZE);
685 >        assertIteratorExhausted(it);
686 >
687 >        it = q.iterator();
688 >        for (i = 0; it.hasNext(); i++)
689 >            assertEquals(it.next(), q.take());
690 >        assertEquals(i, SIZE);
691 >        assertIteratorExhausted(it);
692 >    }
693 >
694 >    /**
695 >     * iterator of empty collection has no elements
696 >     */
697 >    public void testEmptyIterator() {
698 >        assertIteratorExhausted(new ArrayBlockingQueue(SIZE).iterator());
699      }
700  
701      /**
702       * iterator.remove removes current element
703       */
704 <    public void testIteratorRemove () {
704 >    public void testIteratorRemove() {
705          final ArrayBlockingQueue q = new ArrayBlockingQueue(3);
706          q.add(two);
707          q.add(one);
# Line 785 | Line 712 | public class ArrayBlockingQueueTest exte
712          it.remove();
713  
714          it = q.iterator();
715 <        assertEquals(it.next(), one);
716 <        assertEquals(it.next(), three);
715 >        assertSame(it.next(), one);
716 >        assertSame(it.next(), three);
717          assertFalse(it.hasNext());
718      }
719  
# Line 803 | Line 730 | public class ArrayBlockingQueueTest exte
730  
731          int k = 0;
732          for (Iterator it = q.iterator(); it.hasNext();) {
733 <            int i = ((Integer)(it.next())).intValue();
807 <            assertEquals(++k, i);
733 >            assertEquals(++k, it.next());
734          }
735          assertEquals(3, k);
736      }
# Line 812 | Line 738 | public class ArrayBlockingQueueTest exte
738      /**
739       * Modifications do not cause iterators to fail
740       */
741 <    public void testWeaklyConsistentIteration () {
741 >    public void testWeaklyConsistentIteration() {
742          final ArrayBlockingQueue q = new ArrayBlockingQueue(3);
743          q.add(one);
744          q.add(two);
745          q.add(three);
746 <        try {
747 <            for (Iterator it = q.iterator(); it.hasNext();) {
748 <                q.remove();
823 <                it.next();
824 <            }
825 <        }
826 <        catch (ConcurrentModificationException e) {
827 <            unexpectedException();
746 >        for (Iterator it = q.iterator(); it.hasNext();) {
747 >            q.remove();
748 >            it.next();
749          }
750          assertEquals(0, q.size());
751      }
752  
832
753      /**
754       * toString contains toStrings of elements
755       */
# Line 837 | Line 757 | public class ArrayBlockingQueueTest exte
757          ArrayBlockingQueue q = populatedQueue(SIZE);
758          String s = q.toString();
759          for (int i = 0; i < SIZE; ++i) {
760 <            assertTrue(s.indexOf(String.valueOf(i)) >= 0);
760 >            assertTrue(s.contains(String.valueOf(i)));
761          }
762      }
763  
844
764      /**
765       * offer transfers elements across Executor tasks
766       */
# Line 849 | Line 768 | public class ArrayBlockingQueueTest exte
768          final ArrayBlockingQueue q = new ArrayBlockingQueue(2);
769          q.add(one);
770          q.add(two);
771 <        ExecutorService executor = Executors.newFixedThreadPool(2);
772 <        executor.execute(new Runnable() {
773 <            public void run() {
774 <                threadAssertFalse(q.offer(three));
775 <                try {
776 <                    threadAssertTrue(q.offer(three, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS));
777 <                    threadAssertEquals(0, q.remainingCapacity());
778 <                }
779 <                catch (InterruptedException e) {
780 <                    threadUnexpectedException();
862 <                }
863 <            }
864 <        });
865 <
866 <        executor.execute(new Runnable() {
867 <            public void run() {
868 <                try {
869 <                    Thread.sleep(SMALL_DELAY_MS);
870 <                    threadAssertEquals(one, q.take());
871 <                }
872 <                catch (InterruptedException e) {
873 <                    threadUnexpectedException();
874 <                }
875 <            }
876 <        });
877 <
878 <        joinPool(executor);
771 >        final CheckedBarrier threadsStarted = new CheckedBarrier(2);
772 >        final ExecutorService executor = Executors.newFixedThreadPool(2);
773 >        try (PoolCleaner cleaner = cleaner(executor)) {
774 >            executor.execute(new CheckedRunnable() {
775 >                public void realRun() throws InterruptedException {
776 >                    assertFalse(q.offer(three));
777 >                    threadsStarted.await();
778 >                    assertTrue(q.offer(three, LONG_DELAY_MS, MILLISECONDS));
779 >                    assertEquals(0, q.remainingCapacity());
780 >                }});
781  
782 +            executor.execute(new CheckedRunnable() {
783 +                public void realRun() throws InterruptedException {
784 +                    threadsStarted.await();
785 +                    assertEquals(0, q.remainingCapacity());
786 +                    assertSame(one, q.take());
787 +                }});
788 +        }
789      }
790  
791      /**
792 <     * poll retrieves elements across Executor threads
792 >     * timed poll retrieves elements across Executor threads
793       */
794      public void testPollInExecutor() {
795          final ArrayBlockingQueue q = new ArrayBlockingQueue(2);
796 <        ExecutorService executor = Executors.newFixedThreadPool(2);
797 <        executor.execute(new Runnable() {
798 <            public void run() {
799 <                threadAssertNull(q.poll());
800 <                try {
801 <                    threadAssertTrue(null != q.poll(MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS));
802 <                    threadAssertTrue(q.isEmpty());
803 <                }
804 <                catch (InterruptedException e) {
805 <                    threadUnexpectedException();
897 <                }
898 <            }
899 <        });
796 >        final CheckedBarrier threadsStarted = new CheckedBarrier(2);
797 >        final ExecutorService executor = Executors.newFixedThreadPool(2);
798 >        try (PoolCleaner cleaner = cleaner(executor)) {
799 >            executor.execute(new CheckedRunnable() {
800 >                public void realRun() throws InterruptedException {
801 >                    assertNull(q.poll());
802 >                    threadsStarted.await();
803 >                    assertSame(one, q.poll(LONG_DELAY_MS, MILLISECONDS));
804 >                    checkEmpty(q);
805 >                }});
806  
807 <        executor.execute(new Runnable() {
808 <            public void run() {
809 <                try {
904 <                    Thread.sleep(SMALL_DELAY_MS);
807 >            executor.execute(new CheckedRunnable() {
808 >                public void realRun() throws InterruptedException {
809 >                    threadsStarted.await();
810                      q.put(one);
811 <                }
907 <                catch (InterruptedException e) {
908 <                    threadUnexpectedException();
909 <                }
910 <            }
911 <        });
912 <
913 <        joinPool(executor);
914 <    }
915 <
916 <    /**
917 <     * A deserialized serialized queue has same elements in same order
918 <     */
919 <    public void testSerialization() {
920 <        ArrayBlockingQueue q = populatedQueue(SIZE);
921 <
922 <        try {
923 <            ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
924 <            ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
925 <            out.writeObject(q);
926 <            out.close();
927 <
928 <            ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
929 <            ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
930 <            ArrayBlockingQueue r = (ArrayBlockingQueue)in.readObject();
931 <            assertEquals(q.size(), r.size());
932 <            while (!q.isEmpty())
933 <                assertEquals(q.remove(), r.remove());
934 <        } catch (Exception e) {
935 <            unexpectedException();
936 <        }
937 <    }
938 <
939 <    /**
940 <     * drainTo(null) throws NPE
941 <     */
942 <    public void testDrainToNull() {
943 <        ArrayBlockingQueue q = populatedQueue(SIZE);
944 <        try {
945 <            q.drainTo(null);
946 <            shouldThrow();
947 <        } catch (NullPointerException success) {
811 >                }});
812          }
813      }
814  
815      /**
816 <     * drainTo(this) throws IAE
816 >     * A deserialized serialized queue has same elements in same order
817       */
818 <    public void testDrainToSelf() {
819 <        ArrayBlockingQueue q = populatedQueue(SIZE);
820 <        try {
821 <            q.drainTo(q);
822 <            shouldThrow();
823 <        } catch (IllegalArgumentException success) {
818 >    public void testSerialization() throws Exception {
819 >        Queue x = populatedQueue(SIZE);
820 >        Queue y = serialClone(x);
821 >
822 >        assertNotSame(x, y);
823 >        assertEquals(x.size(), y.size());
824 >        assertEquals(x.toString(), y.toString());
825 >        assertTrue(Arrays.equals(x.toArray(), y.toArray()));
826 >        while (!x.isEmpty()) {
827 >            assertFalse(y.isEmpty());
828 >            assertEquals(x.remove(), y.remove());
829          }
830 +        assertTrue(y.isEmpty());
831      }
832  
833      /**
# Line 967 | Line 837 | public class ArrayBlockingQueueTest exte
837          ArrayBlockingQueue q = populatedQueue(SIZE);
838          ArrayList l = new ArrayList();
839          q.drainTo(l);
840 <        assertEquals(q.size(), 0);
841 <        assertEquals(l.size(), SIZE);
840 >        assertEquals(0, q.size());
841 >        assertEquals(SIZE, l.size());
842          for (int i = 0; i < SIZE; ++i)
843              assertEquals(l.get(i), new Integer(i));
844          q.add(zero);
# Line 978 | Line 848 | public class ArrayBlockingQueueTest exte
848          assertTrue(q.contains(one));
849          l.clear();
850          q.drainTo(l);
851 <        assertEquals(q.size(), 0);
852 <        assertEquals(l.size(), 2);
851 >        assertEquals(0, q.size());
852 >        assertEquals(2, l.size());
853          for (int i = 0; i < 2; ++i)
854              assertEquals(l.get(i), new Integer(i));
855      }
# Line 987 | Line 857 | public class ArrayBlockingQueueTest exte
857      /**
858       * drainTo empties full queue, unblocking a waiting put.
859       */
860 <    public void testDrainToWithActivePut() {
860 >    public void testDrainToWithActivePut() throws InterruptedException {
861          final ArrayBlockingQueue q = populatedQueue(SIZE);
862 <        Thread t = new Thread(new Runnable() {
863 <                public void run() {
864 <                    try {
865 <                        q.put(new Integer(SIZE+1));
996 <                    } catch (InterruptedException ie) {
997 <                        threadUnexpectedException();
998 <                    }
999 <                }
1000 <            });
1001 <        try {
1002 <            t.start();
1003 <            ArrayList l = new ArrayList();
1004 <            q.drainTo(l);
1005 <            assertTrue(l.size() >= SIZE);
1006 <            for (int i = 0; i < SIZE; ++i)
1007 <                assertEquals(l.get(i), new Integer(i));
1008 <            t.join();
1009 <            assertTrue(q.size() + l.size() >= SIZE);
1010 <        } catch (Exception e) {
1011 <            unexpectedException();
1012 <        }
1013 <    }
1014 <
1015 <    /**
1016 <     * drainTo(null, n) throws NPE
1017 <     */
1018 <    public void testDrainToNullN() {
1019 <        ArrayBlockingQueue q = populatedQueue(SIZE);
1020 <        try {
1021 <            q.drainTo(null, 0);
1022 <            shouldThrow();
1023 <        } catch (NullPointerException success) {
1024 <        }
1025 <    }
862 >        Thread t = new Thread(new CheckedRunnable() {
863 >            public void realRun() throws InterruptedException {
864 >                q.put(new Integer(SIZE + 1));
865 >            }});
866  
867 <    /**
868 <     * drainTo(this, n) throws IAE
869 <     */
870 <    public void testDrainToSelfN() {
871 <        ArrayBlockingQueue q = populatedQueue(SIZE);
872 <        try {
873 <            q.drainTo(q, 0);
874 <            shouldThrow();
1035 <        } catch (IllegalArgumentException success) {
1036 <        }
867 >        t.start();
868 >        ArrayList l = new ArrayList();
869 >        q.drainTo(l);
870 >        assertTrue(l.size() >= SIZE);
871 >        for (int i = 0; i < SIZE; ++i)
872 >            assertEquals(l.get(i), new Integer(i));
873 >        t.join();
874 >        assertTrue(q.size() + l.size() >= SIZE);
875      }
876  
877      /**
878 <     * drainTo(c, n) empties first max {n, size} elements of queue into c
878 >     * drainTo(c, n) empties first min(n, size) elements of queue into c
879       */
880      public void testDrainToN() {
881 <        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE*2);
881 >        ArrayBlockingQueue q = new ArrayBlockingQueue(SIZE * 2);
882          for (int i = 0; i < SIZE + 2; ++i) {
883              for (int j = 0; j < SIZE; j++)
884                  assertTrue(q.offer(new Integer(j)));
885              ArrayList l = new ArrayList();
886              q.drainTo(l, i);
887 <            int k = (i < SIZE)? i : SIZE;
888 <            assertEquals(l.size(), k);
889 <            assertEquals(q.size(), SIZE-k);
887 >            int k = (i < SIZE) ? i : SIZE;
888 >            assertEquals(k, l.size());
889 >            assertEquals(SIZE - k, q.size());
890              for (int j = 0; j < k; ++j)
891                  assertEquals(l.get(j), new Integer(j));
892 <            while (q.poll() != null) ;
892 >            do {} while (q.poll() != null);
893          }
894      }
895  
896 +    /**
897 +     * remove(null), contains(null) always return false
898 +     */
899 +    public void testNeverContainsNull() {
900 +        Collection<?>[] qs = {
901 +            new ArrayBlockingQueue<Object>(10),
902 +            populatedQueue(2),
903 +        };
904  
905 +        for (Collection<?> q : qs) {
906 +            assertFalse(q.contains(null));
907 +            assertFalse(q.remove(null));
908 +        }
909 +    }
910   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines