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

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

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines