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

Comparing jsr166/src/test/tck/ThreadPoolExecutorSubclassTest.java (file contents):
Revision 1.7 by jsr166, Fri Nov 20 16:02:10 2009 UTC vs.
Revision 1.36 by jsr166, Fri May 15 17:07:27 2015 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 java.util.concurrent.*;
10 < import java.util.concurrent.locks.*;
9 > import static java.util.concurrent.TimeUnit.MILLISECONDS;
10 > import static java.util.concurrent.TimeUnit.SECONDS;
11  
12 < import junit.framework.*;
13 < import java.util.*;
12 > import java.util.ArrayList;
13 > import java.util.List;
14 > import java.util.concurrent.ArrayBlockingQueue;
15 > import java.util.concurrent.BlockingQueue;
16 > import java.util.concurrent.Callable;
17 > import java.util.concurrent.CountDownLatch;
18 > import java.util.concurrent.ExecutionException;
19 > import java.util.concurrent.Executors;
20 > import java.util.concurrent.ExecutorService;
21 > import java.util.concurrent.Future;
22 > import java.util.concurrent.FutureTask;
23 > import java.util.concurrent.LinkedBlockingQueue;
24 > import java.util.concurrent.RejectedExecutionException;
25 > import java.util.concurrent.RejectedExecutionHandler;
26 > import java.util.concurrent.RunnableFuture;
27 > import java.util.concurrent.SynchronousQueue;
28 > import java.util.concurrent.ThreadFactory;
29 > import java.util.concurrent.ThreadPoolExecutor;
30 > import java.util.concurrent.TimeoutException;
31 > import java.util.concurrent.TimeUnit;
32 > import java.util.concurrent.locks.Condition;
33 > import java.util.concurrent.locks.ReentrantLock;
34 >
35 > import junit.framework.Test;
36 > import junit.framework.TestSuite;
37  
38   public class ThreadPoolExecutorSubclassTest extends JSR166TestCase {
39      public static void main(String[] args) {
40 <        junit.textui.TestRunner.run(suite());
40 >        main(suite(), args);
41      }
42      public static Test suite() {
43          return new TestSuite(ThreadPoolExecutorSubclassTest.class);
# Line 36 | Line 59 | public class ThreadPoolExecutorSubclassT
59          CustomTask(final Runnable r, final V res) {
60              if (r == null) throw new NullPointerException();
61              callable = new Callable<V>() {
62 <            public V call() throws Exception { r.run(); return res; }};
62 >                public V call() throws Exception { r.run(); return res; }};
63          }
64          public boolean isDone() {
65              lock.lock(); try { return done; } finally { lock.unlock() ; }
# Line 59 | Line 82 | public class ThreadPoolExecutorSubclassT
82              finally { lock.unlock() ; }
83          }
84          public void run() {
62            boolean runme;
85              lock.lock();
86              try {
87 <                runme = !done;
88 <                if (!runme)
89 <                    thread = Thread.currentThread();
87 >                if (done)
88 >                    return;
89 >                thread = Thread.currentThread();
90              }
91              finally { lock.unlock() ; }
70            if (!runme) return;
92              V v = null;
93              Exception e = null;
94              try {
# Line 116 | Line 137 | public class ThreadPoolExecutorSubclassT
137          }
138      }
139  
119
140      static class CustomTPE extends ThreadPoolExecutor {
141          protected <V> RunnableFuture<V> newTaskFor(Callable<V> c) {
142              return new CustomTask<V>(c);
# Line 163 | Line 183 | public class ThreadPoolExecutorSubclassT
183                workQueue, threadFactory, handler);
184          }
185  
186 <        volatile boolean beforeCalled = false;
187 <        volatile boolean afterCalled = false;
188 <        volatile boolean terminatedCalled = false;
186 >        final CountDownLatch beforeCalled = new CountDownLatch(1);
187 >        final CountDownLatch afterCalled = new CountDownLatch(1);
188 >        final CountDownLatch terminatedCalled = new CountDownLatch(1);
189 >
190          public CustomTPE() {
191 <            super(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new SynchronousQueue<Runnable>());
191 >            super(1, 1, LONG_DELAY_MS, MILLISECONDS, new SynchronousQueue<Runnable>());
192          }
193          protected void beforeExecute(Thread t, Runnable r) {
194 <            beforeCalled = true;
194 >            beforeCalled.countDown();
195          }
196          protected void afterExecute(Runnable r, Throwable t) {
197 <            afterCalled = true;
197 >            afterCalled.countDown();
198          }
199          protected void terminated() {
200 <            terminatedCalled = true;
200 >            terminatedCalled.countDown();
201          }
202  
203 +        public boolean beforeCalled() {
204 +            return beforeCalled.getCount() == 0;
205 +        }
206 +        public boolean afterCalled() {
207 +            return afterCalled.getCount() == 0;
208 +        }
209 +        public boolean terminatedCalled() {
210 +            return terminatedCalled.getCount() == 0;
211 +        }
212      }
213  
214      static class FailingThreadFactory implements ThreadFactory {
# Line 189 | Line 219 | public class ThreadPoolExecutorSubclassT
219          }
220      }
221  
192
222      /**
223 <     *  execute successfully executes a runnable
223 >     * execute successfully executes a runnable
224       */
225      public void testExecute() throws InterruptedException {
226 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
226 >        final ThreadPoolExecutor p =
227 >            new CustomTPE(1, 1,
228 >                          LONG_DELAY_MS, MILLISECONDS,
229 >                          new ArrayBlockingQueue<Runnable>(10));
230 >        final CountDownLatch done = new CountDownLatch(1);
231 >        final Runnable task = new CheckedRunnable() {
232 >            public void realRun() {
233 >                done.countDown();
234 >            }};
235          try {
236 <            p1.execute(new ShortRunnable());
237 <            Thread.sleep(SMALL_DELAY_MS);
236 >            p.execute(task);
237 >            assertTrue(done.await(SMALL_DELAY_MS, MILLISECONDS));
238          } finally {
239 <            joinPool(p1);
239 >            joinPool(p);
240          }
241      }
242  
243      /**
244 <     *  getActiveCount increases but doesn't overestimate, when a
245 <     *  thread becomes active
244 >     * getActiveCount increases but doesn't overestimate, when a
245 >     * thread becomes active
246       */
247      public void testGetActiveCount() throws InterruptedException {
248 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
249 <        assertEquals(0, p2.getActiveCount());
250 <        p2.execute(new MediumRunnable());
251 <        Thread.sleep(SHORT_DELAY_MS);
252 <        assertEquals(1, p2.getActiveCount());
253 <        joinPool(p2);
248 >        final ThreadPoolExecutor p =
249 >            new CustomTPE(2, 2,
250 >                          LONG_DELAY_MS, MILLISECONDS,
251 >                          new ArrayBlockingQueue<Runnable>(10));
252 >        final CountDownLatch threadStarted = new CountDownLatch(1);
253 >        final CountDownLatch done = new CountDownLatch(1);
254 >        try {
255 >            assertEquals(0, p.getActiveCount());
256 >            p.execute(new CheckedRunnable() {
257 >                public void realRun() throws InterruptedException {
258 >                    threadStarted.countDown();
259 >                    assertEquals(1, p.getActiveCount());
260 >                    done.await();
261 >                }});
262 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
263 >            assertEquals(1, p.getActiveCount());
264 >        } finally {
265 >            done.countDown();
266 >            joinPool(p);
267 >        }
268      }
269  
270      /**
271 <     *  prestartCoreThread starts a thread if under corePoolSize, else doesn't
271 >     * prestartCoreThread starts a thread if under corePoolSize, else doesn't
272       */
273      public void testPrestartCoreThread() {
274 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
275 <        assertEquals(0, p2.getPoolSize());
276 <        assertTrue(p2.prestartCoreThread());
277 <        assertEquals(1, p2.getPoolSize());
278 <        assertTrue(p2.prestartCoreThread());
279 <        assertEquals(2, p2.getPoolSize());
280 <        assertFalse(p2.prestartCoreThread());
281 <        assertEquals(2, p2.getPoolSize());
282 <        joinPool(p2);
274 >        ThreadPoolExecutor p = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
275 >        assertEquals(0, p.getPoolSize());
276 >        assertTrue(p.prestartCoreThread());
277 >        assertEquals(1, p.getPoolSize());
278 >        assertTrue(p.prestartCoreThread());
279 >        assertEquals(2, p.getPoolSize());
280 >        assertFalse(p.prestartCoreThread());
281 >        assertEquals(2, p.getPoolSize());
282 >        joinPool(p);
283      }
284  
285      /**
286 <     *  prestartAllCoreThreads starts all corePoolSize threads
286 >     * prestartAllCoreThreads starts all corePoolSize threads
287       */
288      public void testPrestartAllCoreThreads() {
289 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
290 <        assertEquals(0, p2.getPoolSize());
291 <        p2.prestartAllCoreThreads();
292 <        assertEquals(2, p2.getPoolSize());
293 <        p2.prestartAllCoreThreads();
294 <        assertEquals(2, p2.getPoolSize());
295 <        joinPool(p2);
289 >        ThreadPoolExecutor p = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
290 >        assertEquals(0, p.getPoolSize());
291 >        p.prestartAllCoreThreads();
292 >        assertEquals(2, p.getPoolSize());
293 >        p.prestartAllCoreThreads();
294 >        assertEquals(2, p.getPoolSize());
295 >        joinPool(p);
296      }
297  
298      /**
299 <     *   getCompletedTaskCount increases, but doesn't overestimate,
300 <     *   when tasks complete
299 >     * getCompletedTaskCount increases, but doesn't overestimate,
300 >     * when tasks complete
301       */
302      public void testGetCompletedTaskCount() throws InterruptedException {
303 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
304 <        assertEquals(0, p2.getCompletedTaskCount());
305 <        p2.execute(new ShortRunnable());
306 <        Thread.sleep(SMALL_DELAY_MS);
307 <        assertEquals(1, p2.getCompletedTaskCount());
308 <        try { p2.shutdown(); } catch (SecurityException ok) { return; }
309 <        joinPool(p2);
303 >        final ThreadPoolExecutor p =
304 >            new CustomTPE(2, 2,
305 >                          LONG_DELAY_MS, MILLISECONDS,
306 >                          new ArrayBlockingQueue<Runnable>(10));
307 >        final CountDownLatch threadStarted = new CountDownLatch(1);
308 >        final CountDownLatch threadProceed = new CountDownLatch(1);
309 >        final CountDownLatch threadDone = new CountDownLatch(1);
310 >        try {
311 >            assertEquals(0, p.getCompletedTaskCount());
312 >            p.execute(new CheckedRunnable() {
313 >                public void realRun() throws InterruptedException {
314 >                    threadStarted.countDown();
315 >                    assertEquals(0, p.getCompletedTaskCount());
316 >                    threadProceed.await();
317 >                    threadDone.countDown();
318 >                }});
319 >            await(threadStarted);
320 >            assertEquals(0, p.getCompletedTaskCount());
321 >            threadProceed.countDown();
322 >            threadDone.await();
323 >            long startTime = System.nanoTime();
324 >            while (p.getCompletedTaskCount() != 1) {
325 >                if (millisElapsedSince(startTime) > LONG_DELAY_MS)
326 >                    fail("timed out");
327 >                Thread.yield();
328 >            }
329 >        } finally {
330 >            joinPool(p);
331 >        }
332      }
333  
334      /**
335 <     *   getCorePoolSize returns size given in constructor if not otherwise set
335 >     * getCorePoolSize returns size given in constructor if not otherwise set
336       */
337      public void testGetCorePoolSize() {
338 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
339 <        assertEquals(1, p1.getCorePoolSize());
340 <        joinPool(p1);
338 >        ThreadPoolExecutor p = new CustomTPE(1, 1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
339 >        assertEquals(1, p.getCorePoolSize());
340 >        joinPool(p);
341      }
342  
343      /**
344 <     *   getKeepAliveTime returns value given in constructor if not otherwise set
344 >     * getKeepAliveTime returns value given in constructor if not otherwise set
345       */
346      public void testGetKeepAliveTime() {
347 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, 1000, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
348 <        assertEquals(1, p2.getKeepAliveTime(TimeUnit.SECONDS));
349 <        joinPool(p2);
347 >        ThreadPoolExecutor p = new CustomTPE(2, 2, 1000, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
348 >        assertEquals(1, p.getKeepAliveTime(TimeUnit.SECONDS));
349 >        joinPool(p);
350      }
351  
279
352      /**
353       * getThreadFactory returns factory in constructor if not set
354       */
355      public void testGetThreadFactory() {
356          ThreadFactory tf = new SimpleThreadFactory();
357 <        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10), tf, new NoOpREHandler());
357 >        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10), tf, new NoOpREHandler());
358          assertSame(tf, p.getThreadFactory());
359          joinPool(p);
360      }
# Line 291 | Line 363 | public class ThreadPoolExecutorSubclassT
363       * setThreadFactory sets the thread factory returned by getThreadFactory
364       */
365      public void testSetThreadFactory() {
366 <        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
366 >        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
367          ThreadFactory tf = new SimpleThreadFactory();
368          p.setThreadFactory(tf);
369          assertSame(tf, p.getThreadFactory());
370          joinPool(p);
371      }
372  
301
373      /**
374       * setThreadFactory(null) throws NPE
375       */
376      public void testSetThreadFactoryNull() {
377 <        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
377 >        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
378          try {
379              p.setThreadFactory(null);
380              shouldThrow();
# Line 318 | Line 389 | public class ThreadPoolExecutorSubclassT
389       */
390      public void testGetRejectedExecutionHandler() {
391          RejectedExecutionHandler h = new NoOpREHandler();
392 <        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10), h);
392 >        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10), h);
393          assertSame(h, p.getRejectedExecutionHandler());
394          joinPool(p);
395      }
# Line 328 | Line 399 | public class ThreadPoolExecutorSubclassT
399       * getRejectedExecutionHandler
400       */
401      public void testSetRejectedExecutionHandler() {
402 <        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
402 >        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
403          RejectedExecutionHandler h = new NoOpREHandler();
404          p.setRejectedExecutionHandler(h);
405          assertSame(h, p.getRejectedExecutionHandler());
406          joinPool(p);
407      }
408  
338
409      /**
410       * setRejectedExecutionHandler(null) throws NPE
411       */
412      public void testSetRejectedExecutionHandlerNull() {
413 <        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
413 >        ThreadPoolExecutor p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
414          try {
415              p.setRejectedExecutionHandler(null);
416              shouldThrow();
# Line 350 | Line 420 | public class ThreadPoolExecutorSubclassT
420          }
421      }
422  
353
423      /**
424 <     *   getLargestPoolSize increases, but doesn't overestimate, when
425 <     *   multiple threads active
424 >     * getLargestPoolSize increases, but doesn't overestimate, when
425 >     * multiple threads active
426       */
427      public void testGetLargestPoolSize() throws InterruptedException {
428 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
429 <        assertEquals(0, p2.getLargestPoolSize());
430 <        p2.execute(new MediumRunnable());
431 <        p2.execute(new MediumRunnable());
432 <        Thread.sleep(SHORT_DELAY_MS);
433 <        assertEquals(2, p2.getLargestPoolSize());
434 <        joinPool(p2);
428 >        final int THREADS = 3;
429 >        final ThreadPoolExecutor p =
430 >            new CustomTPE(THREADS, THREADS,
431 >                          LONG_DELAY_MS, MILLISECONDS,
432 >                          new ArrayBlockingQueue<Runnable>(10));
433 >        final CountDownLatch threadsStarted = new CountDownLatch(THREADS);
434 >        final CountDownLatch done = new CountDownLatch(1);
435 >        try {
436 >            assertEquals(0, p.getLargestPoolSize());
437 >            for (int i = 0; i < THREADS; i++)
438 >                p.execute(new CheckedRunnable() {
439 >                    public void realRun() throws InterruptedException {
440 >                        threadsStarted.countDown();
441 >                        done.await();
442 >                        assertEquals(THREADS, p.getLargestPoolSize());
443 >                    }});
444 >            assertTrue(threadsStarted.await(SMALL_DELAY_MS, MILLISECONDS));
445 >            assertEquals(THREADS, p.getLargestPoolSize());
446 >        } finally {
447 >            done.countDown();
448 >            joinPool(p);
449 >            assertEquals(THREADS, p.getLargestPoolSize());
450 >        }
451      }
452  
453      /**
454 <     *   getMaximumPoolSize returns value given in constructor if not
455 <     *   otherwise set
454 >     * getMaximumPoolSize returns value given in constructor if not
455 >     * otherwise set
456       */
457      public void testGetMaximumPoolSize() {
458 <        ThreadPoolExecutor p2 = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
459 <        assertEquals(2, p2.getMaximumPoolSize());
460 <        joinPool(p2);
458 >        ThreadPoolExecutor p = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
459 >        assertEquals(2, p.getMaximumPoolSize());
460 >        joinPool(p);
461      }
462  
463      /**
464 <     *   getPoolSize increases, but doesn't overestimate, when threads
465 <     *   become active
464 >     * getPoolSize increases, but doesn't overestimate, when threads
465 >     * become active
466       */
467 <    public void testGetPoolSize() {
468 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
469 <        assertEquals(0, p1.getPoolSize());
470 <        p1.execute(new MediumRunnable());
471 <        assertEquals(1, p1.getPoolSize());
472 <        joinPool(p1);
467 >    public void testGetPoolSize() throws InterruptedException {
468 >        final ThreadPoolExecutor p =
469 >            new CustomTPE(1, 1,
470 >                          LONG_DELAY_MS, MILLISECONDS,
471 >                          new ArrayBlockingQueue<Runnable>(10));
472 >        final CountDownLatch threadStarted = new CountDownLatch(1);
473 >        final CountDownLatch done = new CountDownLatch(1);
474 >        try {
475 >            assertEquals(0, p.getPoolSize());
476 >            p.execute(new CheckedRunnable() {
477 >                public void realRun() throws InterruptedException {
478 >                    threadStarted.countDown();
479 >                    assertEquals(1, p.getPoolSize());
480 >                    done.await();
481 >                }});
482 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
483 >            assertEquals(1, p.getPoolSize());
484 >        } finally {
485 >            done.countDown();
486 >            joinPool(p);
487 >        }
488      }
489  
490      /**
491 <     *  getTaskCount increases, but doesn't overestimate, when tasks submitted
491 >     * getTaskCount increases, but doesn't overestimate, when tasks submitted
492       */
493      public void testGetTaskCount() throws InterruptedException {
494 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
495 <        assertEquals(0, p1.getTaskCount());
496 <        p1.execute(new MediumRunnable());
497 <        Thread.sleep(SHORT_DELAY_MS);
498 <        assertEquals(1, p1.getTaskCount());
499 <        joinPool(p1);
494 >        final ThreadPoolExecutor p =
495 >            new CustomTPE(1, 1,
496 >                          LONG_DELAY_MS, MILLISECONDS,
497 >                          new ArrayBlockingQueue<Runnable>(10));
498 >        final CountDownLatch threadStarted = new CountDownLatch(1);
499 >        final CountDownLatch done = new CountDownLatch(1);
500 >        try {
501 >            assertEquals(0, p.getTaskCount());
502 >            p.execute(new CheckedRunnable() {
503 >                public void realRun() throws InterruptedException {
504 >                    threadStarted.countDown();
505 >                    assertEquals(1, p.getTaskCount());
506 >                    done.await();
507 >                }});
508 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
509 >            assertEquals(1, p.getTaskCount());
510 >        } finally {
511 >            done.countDown();
512 >            joinPool(p);
513 >        }
514      }
515  
516      /**
517 <     *   isShutDown is false before shutdown, true after
517 >     * isShutdown is false before shutdown, true after
518       */
519      public void testIsShutdown() {
520  
521 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
522 <        assertFalse(p1.isShutdown());
523 <        try { p1.shutdown(); } catch (SecurityException ok) { return; }
524 <        assertTrue(p1.isShutdown());
525 <        joinPool(p1);
521 >        ThreadPoolExecutor p = new CustomTPE(1, 1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
522 >        assertFalse(p.isShutdown());
523 >        try { p.shutdown(); } catch (SecurityException ok) { return; }
524 >        assertTrue(p.isShutdown());
525 >        joinPool(p);
526      }
527  
414
528      /**
529 <     *  isTerminated is false before termination, true after
529 >     * isTerminated is false before termination, true after
530       */
531      public void testIsTerminated() throws InterruptedException {
532 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
533 <        assertFalse(p1.isTerminated());
534 <        try {
535 <            p1.execute(new MediumRunnable());
536 <        } finally {
537 <            try { p1.shutdown(); } catch (SecurityException ok) { return; }
538 <        }
539 <        assertTrue(p1.awaitTermination(LONG_DELAY_MS, TimeUnit.MILLISECONDS));
540 <        assertTrue(p1.isTerminated());
532 >        final ThreadPoolExecutor p =
533 >            new CustomTPE(1, 1,
534 >                          LONG_DELAY_MS, MILLISECONDS,
535 >                          new ArrayBlockingQueue<Runnable>(10));
536 >        final CountDownLatch threadStarted = new CountDownLatch(1);
537 >        final CountDownLatch done = new CountDownLatch(1);
538 >        try {
539 >            assertFalse(p.isTerminating());
540 >            p.execute(new CheckedRunnable() {
541 >                public void realRun() throws InterruptedException {
542 >                    assertFalse(p.isTerminating());
543 >                    threadStarted.countDown();
544 >                    done.await();
545 >                }});
546 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
547 >            assertFalse(p.isTerminating());
548 >            done.countDown();
549 >        } finally {
550 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
551 >        }
552 >        assertTrue(p.awaitTermination(LONG_DELAY_MS, MILLISECONDS));
553 >        assertTrue(p.isTerminated());
554 >        assertFalse(p.isTerminating());
555      }
556  
557      /**
558 <     *  isTerminating is not true when running or when terminated
558 >     * isTerminating is not true when running or when terminated
559       */
560      public void testIsTerminating() throws InterruptedException {
561 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
562 <        assertFalse(p1.isTerminating());
563 <        try {
564 <            p1.execute(new SmallRunnable());
565 <            assertFalse(p1.isTerminating());
566 <        } finally {
567 <            try { p1.shutdown(); } catch (SecurityException ok) { return; }
568 <        }
569 <        assertTrue(p1.awaitTermination(LONG_DELAY_MS, TimeUnit.MILLISECONDS));
570 <        assertTrue(p1.isTerminated());
571 <        assertFalse(p1.isTerminating());
561 >        final ThreadPoolExecutor p =
562 >            new CustomTPE(1, 1,
563 >                          LONG_DELAY_MS, MILLISECONDS,
564 >                          new ArrayBlockingQueue<Runnable>(10));
565 >        final CountDownLatch threadStarted = new CountDownLatch(1);
566 >        final CountDownLatch done = new CountDownLatch(1);
567 >        try {
568 >            assertFalse(p.isTerminating());
569 >            p.execute(new CheckedRunnable() {
570 >                public void realRun() throws InterruptedException {
571 >                    assertFalse(p.isTerminating());
572 >                    threadStarted.countDown();
573 >                    done.await();
574 >                }});
575 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
576 >            assertFalse(p.isTerminating());
577 >            done.countDown();
578 >        } finally {
579 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
580 >        }
581 >        assertTrue(p.awaitTermination(LONG_DELAY_MS, MILLISECONDS));
582 >        assertTrue(p.isTerminated());
583 >        assertFalse(p.isTerminating());
584      }
585  
586      /**
587       * getQueue returns the work queue, which contains queued tasks
588       */
589      public void testGetQueue() throws InterruptedException {
590 <        BlockingQueue<Runnable> q = new ArrayBlockingQueue<Runnable>(10);
591 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, q);
592 <        FutureTask[] tasks = new FutureTask[5];
593 <        for (int i = 0; i < 5; i++) {
594 <            tasks[i] = new FutureTask(new MediumPossiblyInterruptedRunnable(), Boolean.TRUE);
595 <            p1.execute(tasks[i]);
596 <        }
597 <        try {
598 <            Thread.sleep(SHORT_DELAY_MS);
599 <            BlockingQueue<Runnable> wq = p1.getQueue();
600 <            assertSame(q, wq);
601 <            assertFalse(wq.contains(tasks[0]));
602 <            assertTrue(wq.contains(tasks[4]));
603 <            for (int i = 1; i < 5; ++i)
604 <                tasks[i].cancel(true);
605 <            p1.shutdownNow();
590 >        final BlockingQueue<Runnable> q = new ArrayBlockingQueue<Runnable>(10);
591 >        final ThreadPoolExecutor p =
592 >            new CustomTPE(1, 1,
593 >                          LONG_DELAY_MS, MILLISECONDS,
594 >                          q);
595 >        final CountDownLatch threadStarted = new CountDownLatch(1);
596 >        final CountDownLatch done = new CountDownLatch(1);
597 >        try {
598 >            FutureTask[] tasks = new FutureTask[5];
599 >            for (int i = 0; i < tasks.length; i++) {
600 >                Callable task = new CheckedCallable<Boolean>() {
601 >                    public Boolean realCall() throws InterruptedException {
602 >                        threadStarted.countDown();
603 >                        assertSame(q, p.getQueue());
604 >                        done.await();
605 >                        return Boolean.TRUE;
606 >                    }};
607 >                tasks[i] = new FutureTask(task);
608 >                p.execute(tasks[i]);
609 >            }
610 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
611 >            assertSame(q, p.getQueue());
612 >            assertFalse(q.contains(tasks[0]));
613 >            assertTrue(q.contains(tasks[tasks.length - 1]));
614 >            assertEquals(tasks.length - 1, q.size());
615          } finally {
616 <            joinPool(p1);
616 >            done.countDown();
617 >            joinPool(p);
618          }
619      }
620  
# Line 474 | Line 623 | public class ThreadPoolExecutorSubclassT
623       */
624      public void testRemove() throws InterruptedException {
625          BlockingQueue<Runnable> q = new ArrayBlockingQueue<Runnable>(10);
626 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, q);
627 <        FutureTask[] tasks = new FutureTask[5];
628 <        for (int i = 0; i < 5; i++) {
629 <            tasks[i] = new FutureTask(new MediumPossiblyInterruptedRunnable(), Boolean.TRUE);
630 <            p1.execute(tasks[i]);
631 <        }
632 <        try {
633 <            Thread.sleep(SHORT_DELAY_MS);
634 <            assertFalse(p1.remove(tasks[0]));
626 >        final ThreadPoolExecutor p =
627 >            new CustomTPE(1, 1,
628 >                          LONG_DELAY_MS, MILLISECONDS,
629 >                          q);
630 >        Runnable[] tasks = new Runnable[6];
631 >        final CountDownLatch threadStarted = new CountDownLatch(1);
632 >        final CountDownLatch done = new CountDownLatch(1);
633 >        try {
634 >            for (int i = 0; i < tasks.length; i++) {
635 >                tasks[i] = new CheckedRunnable() {
636 >                        public void realRun() throws InterruptedException {
637 >                            threadStarted.countDown();
638 >                            done.await();
639 >                        }};
640 >                p.execute(tasks[i]);
641 >            }
642 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
643 >            assertFalse(p.remove(tasks[0]));
644              assertTrue(q.contains(tasks[4]));
645              assertTrue(q.contains(tasks[3]));
646 <            assertTrue(p1.remove(tasks[4]));
647 <            assertFalse(p1.remove(tasks[4]));
646 >            assertTrue(p.remove(tasks[4]));
647 >            assertFalse(p.remove(tasks[4]));
648              assertFalse(q.contains(tasks[4]));
649              assertTrue(q.contains(tasks[3]));
650 <            assertTrue(p1.remove(tasks[3]));
650 >            assertTrue(p.remove(tasks[3]));
651              assertFalse(q.contains(tasks[3]));
652          } finally {
653 <            joinPool(p1);
653 >            done.countDown();
654 >            joinPool(p);
655          }
656      }
657  
658      /**
659 <     *   purge removes cancelled tasks from the queue
659 >     * purge removes cancelled tasks from the queue
660       */
661 <    public void testPurge() {
662 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
661 >    public void testPurge() throws InterruptedException {
662 >        final CountDownLatch threadStarted = new CountDownLatch(1);
663 >        final CountDownLatch done = new CountDownLatch(1);
664 >        final BlockingQueue<Runnable> q = new ArrayBlockingQueue<Runnable>(10);
665 >        final ThreadPoolExecutor p =
666 >            new CustomTPE(1, 1,
667 >                          LONG_DELAY_MS, MILLISECONDS,
668 >                          q);
669          FutureTask[] tasks = new FutureTask[5];
670 <        for (int i = 0; i < 5; i++) {
671 <            tasks[i] = new FutureTask(new MediumPossiblyInterruptedRunnable(), Boolean.TRUE);
672 <            p1.execute(tasks[i]);
670 >        try {
671 >            for (int i = 0; i < tasks.length; i++) {
672 >                Callable task = new CheckedCallable<Boolean>() {
673 >                    public Boolean realCall() throws InterruptedException {
674 >                        threadStarted.countDown();
675 >                        done.await();
676 >                        return Boolean.TRUE;
677 >                    }};
678 >                tasks[i] = new FutureTask(task);
679 >                p.execute(tasks[i]);
680 >            }
681 >            assertTrue(threadStarted.await(SMALL_DELAY_MS, MILLISECONDS));
682 >            assertEquals(tasks.length, p.getTaskCount());
683 >            assertEquals(tasks.length - 1, q.size());
684 >            assertEquals(1L, p.getActiveCount());
685 >            assertEquals(0L, p.getCompletedTaskCount());
686 >            tasks[4].cancel(true);
687 >            tasks[3].cancel(false);
688 >            p.purge();
689 >            assertEquals(tasks.length - 3, q.size());
690 >            assertEquals(tasks.length - 2, p.getTaskCount());
691 >            p.purge();         // Nothing to do
692 >            assertEquals(tasks.length - 3, q.size());
693 >            assertEquals(tasks.length - 2, p.getTaskCount());
694 >        } finally {
695 >            done.countDown();
696 >            joinPool(p);
697          }
509        tasks[4].cancel(true);
510        tasks[3].cancel(true);
511        p1.purge();
512        long count = p1.getTaskCount();
513        assertTrue(count >= 2 && count < 5);
514        joinPool(p1);
698      }
699  
700      /**
701 <     *  shutDownNow returns a list containing tasks that were not run
701 >     * shutdownNow returns a list containing tasks that were not run
702       */
703 <    public void testShutDownNow() {
704 <        ThreadPoolExecutor p1 = new CustomTPE(1, 1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
703 >    public void testShutdownNow() {
704 >        ThreadPoolExecutor p = new CustomTPE(1, 1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
705          List l;
706          try {
707              for (int i = 0; i < 5; i++)
708 <                p1.execute(new MediumPossiblyInterruptedRunnable());
708 >                p.execute(new MediumPossiblyInterruptedRunnable());
709          }
710          finally {
711              try {
712 <                l = p1.shutdownNow();
712 >                l = p.shutdownNow();
713              } catch (SecurityException ok) { return; }
531
714          }
715 <        assertTrue(p1.isShutdown());
716 <        assertTrue(l.size() <= 4);
715 >        assertTrue(p.isShutdown());
716 >        assertTrue(l.size() <= 4);
717      }
718  
719      // Exception Tests
720  
539
721      /**
722       * Constructor throws if corePoolSize argument is less than zero
723       */
724      public void testConstructor1() {
725          try {
726 <            new CustomTPE(-1,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
726 >            new CustomTPE(-1, 1, 1L, SECONDS,
727 >                          new ArrayBlockingQueue<Runnable>(10));
728              shouldThrow();
729          } catch (IllegalArgumentException success) {}
730      }
# Line 552 | Line 734 | public class ThreadPoolExecutorSubclassT
734       */
735      public void testConstructor2() {
736          try {
737 <            new CustomTPE(1,-1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
737 >            new CustomTPE(1, -1, 1L, SECONDS,
738 >                          new ArrayBlockingQueue<Runnable>(10));
739              shouldThrow();
740          } catch (IllegalArgumentException success) {}
741      }
# Line 562 | Line 745 | public class ThreadPoolExecutorSubclassT
745       */
746      public void testConstructor3() {
747          try {
748 <            new CustomTPE(1,0,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
748 >            new CustomTPE(1, 0, 1L, SECONDS,
749 >                          new ArrayBlockingQueue<Runnable>(10));
750              shouldThrow();
751          } catch (IllegalArgumentException success) {}
752      }
# Line 572 | Line 756 | public class ThreadPoolExecutorSubclassT
756       */
757      public void testConstructor4() {
758          try {
759 <            new CustomTPE(1,2,-1L,TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
759 >            new CustomTPE(1, 2, -1L, SECONDS,
760 >                          new ArrayBlockingQueue<Runnable>(10));
761              shouldThrow();
762          } catch (IllegalArgumentException success) {}
763      }
# Line 582 | Line 767 | public class ThreadPoolExecutorSubclassT
767       */
768      public void testConstructor5() {
769          try {
770 <            new CustomTPE(2,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
770 >            new CustomTPE(2, 1, 1L, SECONDS,
771 >                          new ArrayBlockingQueue<Runnable>(10));
772              shouldThrow();
773          } catch (IllegalArgumentException success) {}
774      }
# Line 592 | Line 778 | public class ThreadPoolExecutorSubclassT
778       */
779      public void testConstructorNullPointerException() {
780          try {
781 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,null);
781 >            new CustomTPE(1, 2, 1L, SECONDS, null);
782              shouldThrow();
783          } catch (NullPointerException success) {}
784      }
785  
600
601
786      /**
787       * Constructor throws if corePoolSize argument is less than zero
788       */
789      public void testConstructor6() {
790          try {
791 <            new CustomTPE(-1,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory());
791 >            new CustomTPE(-1, 1, 1L, SECONDS,
792 >                          new ArrayBlockingQueue<Runnable>(10),
793 >                          new SimpleThreadFactory());
794              shouldThrow();
795          } catch (IllegalArgumentException success) {}
796      }
# Line 614 | Line 800 | public class ThreadPoolExecutorSubclassT
800       */
801      public void testConstructor7() {
802          try {
803 <            new CustomTPE(1,-1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory());
803 >            new CustomTPE(1,-1, 1L, SECONDS,
804 >                          new ArrayBlockingQueue<Runnable>(10),
805 >                          new SimpleThreadFactory());
806              shouldThrow();
807          } catch (IllegalArgumentException success) {}
808      }
# Line 624 | Line 812 | public class ThreadPoolExecutorSubclassT
812       */
813      public void testConstructor8() {
814          try {
815 <            new CustomTPE(1,0,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory());
815 >            new CustomTPE(1, 0, 1L, SECONDS,
816 >                          new ArrayBlockingQueue<Runnable>(10),
817 >                          new SimpleThreadFactory());
818              shouldThrow();
819          } catch (IllegalArgumentException success) {}
820      }
# Line 634 | Line 824 | public class ThreadPoolExecutorSubclassT
824       */
825      public void testConstructor9() {
826          try {
827 <            new CustomTPE(1,2,-1L,TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory());
827 >            new CustomTPE(1, 2, -1L, SECONDS,
828 >                          new ArrayBlockingQueue<Runnable>(10),
829 >                          new SimpleThreadFactory());
830              shouldThrow();
831          } catch (IllegalArgumentException success) {}
832      }
# Line 644 | Line 836 | public class ThreadPoolExecutorSubclassT
836       */
837      public void testConstructor10() {
838          try {
839 <            new CustomTPE(2,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory());
839 >            new CustomTPE(2, 1, 1L, SECONDS,
840 >                          new ArrayBlockingQueue<Runnable>(10),
841 >                          new SimpleThreadFactory());
842              shouldThrow();
843          } catch (IllegalArgumentException success) {}
844      }
# Line 654 | Line 848 | public class ThreadPoolExecutorSubclassT
848       */
849      public void testConstructorNullPointerException2() {
850          try {
851 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,null,new SimpleThreadFactory());
851 >            new CustomTPE(1, 2, 1L, SECONDS, null, new SimpleThreadFactory());
852              shouldThrow();
853          } catch (NullPointerException success) {}
854      }
# Line 664 | Line 858 | public class ThreadPoolExecutorSubclassT
858       */
859      public void testConstructorNullPointerException3() {
860          try {
861 <            ThreadFactory f = null;
862 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10),f);
861 >            new CustomTPE(1, 2, 1L, SECONDS,
862 >                          new ArrayBlockingQueue<Runnable>(10),
863 >                          (ThreadFactory) null);
864              shouldThrow();
865          } catch (NullPointerException success) {}
866      }
867  
673
868      /**
869       * Constructor throws if corePoolSize argument is less than zero
870       */
871      public void testConstructor11() {
872          try {
873 <            new CustomTPE(-1,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new NoOpREHandler());
873 >            new CustomTPE(-1, 1, 1L, SECONDS,
874 >                          new ArrayBlockingQueue<Runnable>(10),
875 >                          new NoOpREHandler());
876              shouldThrow();
877          } catch (IllegalArgumentException success) {}
878      }
# Line 686 | Line 882 | public class ThreadPoolExecutorSubclassT
882       */
883      public void testConstructor12() {
884          try {
885 <            new CustomTPE(1,-1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new NoOpREHandler());
885 >            new CustomTPE(1, -1, 1L, SECONDS,
886 >                          new ArrayBlockingQueue<Runnable>(10),
887 >                          new NoOpREHandler());
888              shouldThrow();
889          } catch (IllegalArgumentException success) {}
890      }
# Line 696 | Line 894 | public class ThreadPoolExecutorSubclassT
894       */
895      public void testConstructor13() {
896          try {
897 <            new CustomTPE(1,0,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new NoOpREHandler());
897 >            new CustomTPE(1, 0, 1L, SECONDS,
898 >                          new ArrayBlockingQueue<Runnable>(10),
899 >                          new NoOpREHandler());
900              shouldThrow();
901          } catch (IllegalArgumentException success) {}
902      }
# Line 706 | Line 906 | public class ThreadPoolExecutorSubclassT
906       */
907      public void testConstructor14() {
908          try {
909 <            new CustomTPE(1,2,-1L,TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new NoOpREHandler());
909 >            new CustomTPE(1, 2, -1L, SECONDS,
910 >                          new ArrayBlockingQueue<Runnable>(10),
911 >                          new NoOpREHandler());
912              shouldThrow();
913          } catch (IllegalArgumentException success) {}
914      }
# Line 716 | Line 918 | public class ThreadPoolExecutorSubclassT
918       */
919      public void testConstructor15() {
920          try {
921 <            new CustomTPE(2,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new NoOpREHandler());
921 >            new CustomTPE(2, 1, 1L, SECONDS,
922 >                          new ArrayBlockingQueue<Runnable>(10),
923 >                          new NoOpREHandler());
924              shouldThrow();
925          } catch (IllegalArgumentException success) {}
926      }
# Line 726 | Line 930 | public class ThreadPoolExecutorSubclassT
930       */
931      public void testConstructorNullPointerException4() {
932          try {
933 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,null,new NoOpREHandler());
933 >            new CustomTPE(1, 2, 1L, SECONDS,
934 >                          null,
935 >                          new NoOpREHandler());
936              shouldThrow();
937          } catch (NullPointerException success) {}
938      }
# Line 736 | Line 942 | public class ThreadPoolExecutorSubclassT
942       */
943      public void testConstructorNullPointerException5() {
944          try {
945 <            RejectedExecutionHandler r = null;
946 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10),r);
945 >            new CustomTPE(1, 2, 1L, SECONDS,
946 >                          new ArrayBlockingQueue<Runnable>(10),
947 >                          (RejectedExecutionHandler) null);
948              shouldThrow();
949          } catch (NullPointerException success) {}
950      }
951  
745
952      /**
953       * Constructor throws if corePoolSize argument is less than zero
954       */
955      public void testConstructor16() {
956          try {
957 <            new CustomTPE(-1,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory(),new NoOpREHandler());
957 >            new CustomTPE(-1, 1, 1L, SECONDS,
958 >                          new ArrayBlockingQueue<Runnable>(10),
959 >                          new SimpleThreadFactory(),
960 >                          new NoOpREHandler());
961              shouldThrow();
962          } catch (IllegalArgumentException success) {}
963      }
# Line 758 | Line 967 | public class ThreadPoolExecutorSubclassT
967       */
968      public void testConstructor17() {
969          try {
970 <            new CustomTPE(1,-1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory(),new NoOpREHandler());
970 >            new CustomTPE(1, -1, 1L, SECONDS,
971 >                          new ArrayBlockingQueue<Runnable>(10),
972 >                          new SimpleThreadFactory(),
973 >                          new NoOpREHandler());
974              shouldThrow();
975          } catch (IllegalArgumentException success) {}
976      }
# Line 768 | Line 980 | public class ThreadPoolExecutorSubclassT
980       */
981      public void testConstructor18() {
982          try {
983 <            new CustomTPE(1,0,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory(),new NoOpREHandler());
983 >            new CustomTPE(1, 0, 1L, SECONDS,
984 >                          new ArrayBlockingQueue<Runnable>(10),
985 >                          new SimpleThreadFactory(),
986 >                          new NoOpREHandler());
987              shouldThrow();
988          } catch (IllegalArgumentException success) {}
989      }
# Line 778 | Line 993 | public class ThreadPoolExecutorSubclassT
993       */
994      public void testConstructor19() {
995          try {
996 <            new CustomTPE(1,2,-1L,TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory(),new NoOpREHandler());
996 >            new CustomTPE(1, 2, -1L, SECONDS,
997 >                          new ArrayBlockingQueue<Runnable>(10),
998 >                          new SimpleThreadFactory(),
999 >                          new NoOpREHandler());
1000              shouldThrow();
1001          } catch (IllegalArgumentException success) {}
1002      }
# Line 788 | Line 1006 | public class ThreadPoolExecutorSubclassT
1006       */
1007      public void testConstructor20() {
1008          try {
1009 <            new CustomTPE(2,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory(),new NoOpREHandler());
1009 >            new CustomTPE(2, 1, 1L, SECONDS,
1010 >                          new ArrayBlockingQueue<Runnable>(10),
1011 >                          new SimpleThreadFactory(),
1012 >                          new NoOpREHandler());
1013              shouldThrow();
1014          } catch (IllegalArgumentException success) {}
1015      }
1016  
1017      /**
1018 <     * Constructor throws if workQueue is set to null
1018 >     * Constructor throws if workQueue is null
1019       */
1020      public void testConstructorNullPointerException6() {
1021          try {
1022 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,null,new SimpleThreadFactory(),new NoOpREHandler());
1022 >            new CustomTPE(1, 2, 1L, SECONDS,
1023 >                          null,
1024 >                          new SimpleThreadFactory(),
1025 >                          new NoOpREHandler());
1026              shouldThrow();
1027          } catch (NullPointerException success) {}
1028      }
1029  
1030      /**
1031 <     * Constructor throws if handler is set to null
1031 >     * Constructor throws if handler is null
1032       */
1033      public void testConstructorNullPointerException7() {
1034          try {
1035 <            RejectedExecutionHandler r = null;
1036 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10),new SimpleThreadFactory(),r);
1035 >            new CustomTPE(1, 2, 1L, SECONDS,
1036 >                          new ArrayBlockingQueue<Runnable>(10),
1037 >                          new SimpleThreadFactory(),
1038 >                          (RejectedExecutionHandler) null);
1039              shouldThrow();
1040          } catch (NullPointerException success) {}
1041      }
1042  
1043      /**
1044 <     * Constructor throws if ThreadFactory is set top null
1044 >     * Constructor throws if ThreadFactory is null
1045       */
1046      public void testConstructorNullPointerException8() {
1047          try {
1048 <            ThreadFactory f = null;
1049 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10),f,new NoOpREHandler());
1048 >            new CustomTPE(1, 2, 1L, SECONDS,
1049 >                          new ArrayBlockingQueue<Runnable>(10),
1050 >                          (ThreadFactory) null,
1051 >                          new NoOpREHandler());
1052              shouldThrow();
1053          } catch (NullPointerException success) {}
1054      }
1055  
828
1056      /**
1057 <     *  execute throws RejectedExecutionException
831 <     *  if saturated.
1057 >     * execute throws RejectedExecutionException if saturated.
1058       */
1059      public void testSaturatedExecute() {
1060 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1));
1061 <        try {
1062 <
1063 <            for (int i = 0; i < 5; ++i) {
1064 <                p.execute(new MediumRunnable());
1060 >        ThreadPoolExecutor p =
1061 >            new CustomTPE(1, 1,
1062 >                          LONG_DELAY_MS, MILLISECONDS,
1063 >                          new ArrayBlockingQueue<Runnable>(1));
1064 >        final CountDownLatch done = new CountDownLatch(1);
1065 >        try {
1066 >            Runnable task = new CheckedRunnable() {
1067 >                public void realRun() throws InterruptedException {
1068 >                    done.await();
1069 >                }};
1070 >            for (int i = 0; i < 2; ++i)
1071 >                p.execute(task);
1072 >            for (int i = 0; i < 2; ++i) {
1073 >                try {
1074 >                    p.execute(task);
1075 >                    shouldThrow();
1076 >                } catch (RejectedExecutionException success) {}
1077 >                assertTrue(p.getTaskCount() <= 2);
1078              }
1079 <            shouldThrow();
1080 <        } catch (RejectedExecutionException success) {}
1081 <        joinPool(p);
1079 >        } finally {
1080 >            done.countDown();
1081 >            joinPool(p);
1082 >        }
1083      }
1084  
1085      /**
1086 <     *  executor using CallerRunsPolicy runs task if saturated.
1086 >     * executor using CallerRunsPolicy runs task if saturated.
1087       */
1088      public void testSaturatedExecute2() {
1089          RejectedExecutionHandler h = new CustomTPE.CallerRunsPolicy();
1090 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1090 >        ThreadPoolExecutor p = new CustomTPE(1, 1,
1091 >                                             LONG_DELAY_MS, MILLISECONDS,
1092 >                                             new ArrayBlockingQueue<Runnable>(1),
1093 >                                             h);
1094          try {
852
1095              TrackedNoOpRunnable[] tasks = new TrackedNoOpRunnable[5];
1096 <            for (int i = 0; i < 5; ++i) {
1096 >            for (int i = 0; i < tasks.length; ++i)
1097                  tasks[i] = new TrackedNoOpRunnable();
856            }
1098              TrackedLongRunnable mr = new TrackedLongRunnable();
1099              p.execute(mr);
1100 <            for (int i = 0; i < 5; ++i) {
1100 >            for (int i = 0; i < tasks.length; ++i)
1101                  p.execute(tasks[i]);
1102 <            }
862 <            for (int i = 1; i < 5; ++i) {
1102 >            for (int i = 1; i < tasks.length; ++i)
1103                  assertTrue(tasks[i].done);
864            }
1104              try { p.shutdownNow(); } catch (SecurityException ok) { return; }
1105          } finally {
1106              joinPool(p);
# Line 869 | Line 1108 | public class ThreadPoolExecutorSubclassT
1108      }
1109  
1110      /**
1111 <     *  executor using DiscardPolicy drops task if saturated.
1111 >     * executor using DiscardPolicy drops task if saturated.
1112       */
1113      public void testSaturatedExecute3() {
1114          RejectedExecutionHandler h = new CustomTPE.DiscardPolicy();
1115 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1115 >        ThreadPoolExecutor p =
1116 >            new CustomTPE(1, 1,
1117 >                          LONG_DELAY_MS, MILLISECONDS,
1118 >                          new ArrayBlockingQueue<Runnable>(1),
1119 >                          h);
1120          try {
878
1121              TrackedNoOpRunnable[] tasks = new TrackedNoOpRunnable[5];
1122 <            for (int i = 0; i < 5; ++i) {
1122 >            for (int i = 0; i < tasks.length; ++i)
1123                  tasks[i] = new TrackedNoOpRunnable();
882            }
1124              p.execute(new TrackedLongRunnable());
1125 <            for (int i = 0; i < 5; ++i) {
1126 <                p.execute(tasks[i]);
1127 <            }
1128 <            for (int i = 0; i < 5; ++i) {
888 <                assertFalse(tasks[i].done);
889 <            }
1125 >            for (TrackedNoOpRunnable task : tasks)
1126 >                p.execute(task);
1127 >            for (TrackedNoOpRunnable task : tasks)
1128 >                assertFalse(task.done);
1129              try { p.shutdownNow(); } catch (SecurityException ok) { return; }
1130          } finally {
1131              joinPool(p);
# Line 894 | Line 1133 | public class ThreadPoolExecutorSubclassT
1133      }
1134  
1135      /**
1136 <     *  executor using DiscardOldestPolicy drops oldest task if saturated.
1136 >     * executor using DiscardOldestPolicy drops oldest task if saturated.
1137       */
1138      public void testSaturatedExecute4() {
1139          RejectedExecutionHandler h = new CustomTPE.DiscardOldestPolicy();
1140 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1140 >        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1141          try {
1142              p.execute(new TrackedLongRunnable());
1143              TrackedLongRunnable r2 = new TrackedLongRunnable();
# Line 915 | Line 1154 | public class ThreadPoolExecutorSubclassT
1154      }
1155  
1156      /**
1157 <     *  execute throws RejectedExecutionException if shutdown
1157 >     * execute throws RejectedExecutionException if shutdown
1158       */
1159      public void testRejectedExecutionExceptionOnShutdown() {
1160 <        ThreadPoolExecutor tpe =
1161 <            new CustomTPE(1,1,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(1));
1162 <        try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1163 <        try {
1164 <            tpe.execute(new NoOpRunnable());
1165 <            shouldThrow();
1166 <        } catch (RejectedExecutionException success) {}
1160 >        ThreadPoolExecutor p =
1161 >            new CustomTPE(1,1,LONG_DELAY_MS, MILLISECONDS,new ArrayBlockingQueue<Runnable>(1));
1162 >        try { p.shutdown(); } catch (SecurityException ok) { return; }
1163 >        try {
1164 >            p.execute(new NoOpRunnable());
1165 >            shouldThrow();
1166 >        } catch (RejectedExecutionException success) {}
1167  
1168 <        joinPool(tpe);
1168 >        joinPool(p);
1169      }
1170  
1171      /**
1172 <     *  execute using CallerRunsPolicy drops task on shutdown
1172 >     * execute using CallerRunsPolicy drops task on shutdown
1173       */
1174      public void testCallerRunsOnShutdown() {
1175          RejectedExecutionHandler h = new CustomTPE.CallerRunsPolicy();
1176 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1176 >        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1177  
1178          try { p.shutdown(); } catch (SecurityException ok) { return; }
1179 <        try {
1179 >        try {
1180              TrackedNoOpRunnable r = new TrackedNoOpRunnable();
1181 <            p.execute(r);
1181 >            p.execute(r);
1182              assertFalse(r.done);
1183          } finally {
1184              joinPool(p);
# Line 947 | Line 1186 | public class ThreadPoolExecutorSubclassT
1186      }
1187  
1188      /**
1189 <     *  execute using DiscardPolicy drops task on shutdown
1189 >     * execute using DiscardPolicy drops task on shutdown
1190       */
1191      public void testDiscardOnShutdown() {
1192          RejectedExecutionHandler h = new CustomTPE.DiscardPolicy();
1193 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1193 >        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1194  
1195          try { p.shutdown(); } catch (SecurityException ok) { return; }
1196 <        try {
1196 >        try {
1197              TrackedNoOpRunnable r = new TrackedNoOpRunnable();
1198 <            p.execute(r);
1198 >            p.execute(r);
1199              assertFalse(r.done);
1200          } finally {
1201              joinPool(p);
1202          }
1203      }
1204  
966
1205      /**
1206 <     *  execute using DiscardOldestPolicy drops task on shutdown
1206 >     * execute using DiscardOldestPolicy drops task on shutdown
1207       */
1208      public void testDiscardOldestOnShutdown() {
1209          RejectedExecutionHandler h = new CustomTPE.DiscardOldestPolicy();
1210 <        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1210 >        ThreadPoolExecutor p = new CustomTPE(1,1, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(1), h);
1211  
1212          try { p.shutdown(); } catch (SecurityException ok) { return; }
1213 <        try {
1213 >        try {
1214              TrackedNoOpRunnable r = new TrackedNoOpRunnable();
1215 <            p.execute(r);
1215 >            p.execute(r);
1216              assertFalse(r.done);
1217          } finally {
1218              joinPool(p);
1219          }
1220      }
1221  
984
1222      /**
1223 <     *  execute (null) throws NPE
1223 >     * execute(null) throws NPE
1224       */
1225      public void testExecuteNull() {
1226 <        ThreadPoolExecutor tpe = null;
1226 >        ThreadPoolExecutor p = null;
1227          try {
1228 <            tpe = new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1229 <            tpe.execute(null);
1228 >            p = new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1229 >            p.execute(null);
1230              shouldThrow();
1231 <        } catch (NullPointerException success) {}
1231 >        } catch (NullPointerException success) {}
1232  
1233 <        joinPool(tpe);
1233 >        joinPool(p);
1234      }
1235  
1236      /**
1237 <     *  setCorePoolSize of negative value throws IllegalArgumentException
1237 >     * setCorePoolSize of negative value throws IllegalArgumentException
1238       */
1239      public void testCorePoolSizeIllegalArgumentException() {
1240 <        ThreadPoolExecutor tpe =
1241 <            new CustomTPE(1,2,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1242 <        try {
1243 <            tpe.setCorePoolSize(-1);
1244 <            shouldThrow();
1245 <        } catch (IllegalArgumentException success) {
1240 >        ThreadPoolExecutor p =
1241 >            new CustomTPE(1,2,LONG_DELAY_MS, MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1242 >        try {
1243 >            p.setCorePoolSize(-1);
1244 >            shouldThrow();
1245 >        } catch (IllegalArgumentException success) {
1246          } finally {
1247 <            try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1247 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
1248          }
1249 <        joinPool(tpe);
1249 >        joinPool(p);
1250      }
1251  
1252      /**
1253 <     *  setMaximumPoolSize(int) throws IllegalArgumentException if
1254 <     *  given a value less the core pool size
1253 >     * setMaximumPoolSize(int) throws IllegalArgumentException
1254 >     * if given a value less the core pool size
1255       */
1256      public void testMaximumPoolSizeIllegalArgumentException() {
1257 <        ThreadPoolExecutor tpe = null;
1258 <        try {
1022 <            tpe = new CustomTPE(2,3,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1023 <        } catch (Exception e) {}
1257 >        ThreadPoolExecutor p =
1258 >            new CustomTPE(2,3,LONG_DELAY_MS, MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1259          try {
1260 <            tpe.setMaximumPoolSize(1);
1260 >            p.setMaximumPoolSize(1);
1261              shouldThrow();
1262          } catch (IllegalArgumentException success) {
1263          } finally {
1264 <            try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1264 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
1265          }
1266 <        joinPool(tpe);
1266 >        joinPool(p);
1267      }
1268  
1269      /**
1270 <     *  setMaximumPoolSize throws IllegalArgumentException
1271 <     *  if given a negative value
1270 >     * setMaximumPoolSize throws IllegalArgumentException
1271 >     * if given a negative value
1272       */
1273      public void testMaximumPoolSizeIllegalArgumentException2() {
1274 <        ThreadPoolExecutor tpe = null;
1274 >        ThreadPoolExecutor p =
1275 >            new CustomTPE(2,3,LONG_DELAY_MS, MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1276          try {
1277 <            tpe = new CustomTPE(2,3,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1042 <        } catch (Exception e) {}
1043 <        try {
1044 <            tpe.setMaximumPoolSize(-1);
1277 >            p.setMaximumPoolSize(-1);
1278              shouldThrow();
1279          } catch (IllegalArgumentException success) {
1280          } finally {
1281 <            try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1281 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
1282          }
1283 <        joinPool(tpe);
1283 >        joinPool(p);
1284      }
1285  
1053
1286      /**
1287 <     *  setKeepAliveTime  throws IllegalArgumentException
1288 <     *  when given a negative value
1287 >     * setKeepAliveTime throws IllegalArgumentException
1288 >     * when given a negative value
1289       */
1290      public void testKeepAliveTimeIllegalArgumentException() {
1291 <        ThreadPoolExecutor tpe = null;
1292 <        try {
1061 <            tpe = new CustomTPE(2,3,LONG_DELAY_MS, TimeUnit.MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1062 <        } catch (Exception e) {}
1291 >        ThreadPoolExecutor p =
1292 >            new CustomTPE(2,3,LONG_DELAY_MS, MILLISECONDS,new ArrayBlockingQueue<Runnable>(10));
1293  
1294 <        try {
1295 <            tpe.setKeepAliveTime(-1,TimeUnit.MILLISECONDS);
1294 >        try {
1295 >            p.setKeepAliveTime(-1,MILLISECONDS);
1296              shouldThrow();
1297          } catch (IllegalArgumentException success) {
1298          } finally {
1299 <            try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1299 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
1300          }
1301 <        joinPool(tpe);
1301 >        joinPool(p);
1302      }
1303  
1304      /**
1305       * terminated() is called on termination
1306       */
1307      public void testTerminated() {
1308 <        CustomTPE tpe = new CustomTPE();
1309 <        try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1310 <        assertTrue(tpe.terminatedCalled);
1311 <        joinPool(tpe);
1308 >        CustomTPE p = new CustomTPE();
1309 >        try { p.shutdown(); } catch (SecurityException ok) { return; }
1310 >        assertTrue(p.terminatedCalled());
1311 >        joinPool(p);
1312      }
1313  
1314      /**
1315       * beforeExecute and afterExecute are called when executing task
1316       */
1317      public void testBeforeAfter() throws InterruptedException {
1318 <        CustomTPE tpe = new CustomTPE();
1318 >        CustomTPE p = new CustomTPE();
1319          try {
1320 <            TrackedNoOpRunnable r = new TrackedNoOpRunnable();
1321 <            tpe.execute(r);
1322 <            Thread.sleep(SHORT_DELAY_MS);
1323 <            assertTrue(r.done);
1324 <            assertTrue(tpe.beforeCalled);
1325 <            assertTrue(tpe.afterCalled);
1326 <            try { tpe.shutdown(); } catch (SecurityException ok) { return; }
1320 >            final CountDownLatch done = new CountDownLatch(1);
1321 >            p.execute(new CheckedRunnable() {
1322 >                public void realRun() {
1323 >                    done.countDown();
1324 >                }});
1325 >            await(p.afterCalled);
1326 >            assertEquals(0, done.getCount());
1327 >            assertTrue(p.afterCalled());
1328 >            assertTrue(p.beforeCalled());
1329 >            try { p.shutdown(); } catch (SecurityException ok) { return; }
1330          } finally {
1331 <            joinPool(tpe);
1331 >            joinPool(p);
1332          }
1333      }
1334  
# Line 1103 | Line 1336 | public class ThreadPoolExecutorSubclassT
1336       * completed submit of callable returns result
1337       */
1338      public void testSubmitCallable() throws Exception {
1339 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1339 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1340          try {
1341              Future<String> future = e.submit(new StringTask());
1342              String result = future.get();
# Line 1117 | Line 1350 | public class ThreadPoolExecutorSubclassT
1350       * completed submit of runnable returns successfully
1351       */
1352      public void testSubmitRunnable() throws Exception {
1353 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1353 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1354          try {
1355              Future<?> future = e.submit(new NoOpRunnable());
1356              future.get();
# Line 1131 | Line 1364 | public class ThreadPoolExecutorSubclassT
1364       * completed submit of (runnable, result) returns result
1365       */
1366      public void testSubmitRunnable2() throws Exception {
1367 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1367 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1368          try {
1369              Future<String> future = e.submit(new NoOpRunnable(), TEST_STRING);
1370              String result = future.get();
# Line 1141 | Line 1374 | public class ThreadPoolExecutorSubclassT
1374          }
1375      }
1376  
1144
1377      /**
1378       * invokeAny(null) throws NPE
1379       */
1380      public void testInvokeAny1() throws Exception {
1381 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1381 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1382          try {
1383              e.invokeAny(null);
1384              shouldThrow();
# Line 1160 | Line 1392 | public class ThreadPoolExecutorSubclassT
1392       * invokeAny(empty collection) throws IAE
1393       */
1394      public void testInvokeAny2() throws Exception {
1395 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1395 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1396          try {
1397              e.invokeAny(new ArrayList<Callable<String>>());
1398              shouldThrow();
# Line 1174 | Line 1406 | public class ThreadPoolExecutorSubclassT
1406       * invokeAny(c) throws NPE if c has null elements
1407       */
1408      public void testInvokeAny3() throws Exception {
1409 <        final CountDownLatch latch = new CountDownLatch(1);
1410 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1409 >        CountDownLatch latch = new CountDownLatch(1);
1410 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1411 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1412 >        l.add(latchAwaitingStringTask(latch));
1413 >        l.add(null);
1414          try {
1180            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1181            l.add(new Callable<String>() {
1182                      public String call() {
1183                          try {
1184                              latch.await();
1185                          } catch (InterruptedException ok) {}
1186                          return TEST_STRING;
1187                      }});
1188            l.add(null);
1415              e.invokeAny(l);
1416              shouldThrow();
1417          } catch (NullPointerException success) {
# Line 1199 | Line 1425 | public class ThreadPoolExecutorSubclassT
1425       * invokeAny(c) throws ExecutionException if no task completes
1426       */
1427      public void testInvokeAny4() throws Exception {
1428 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1428 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1429 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1430 >        l.add(new NPETask());
1431          try {
1204            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1205            l.add(new NPETask());
1432              e.invokeAny(l);
1433              shouldThrow();
1434          } catch (ExecutionException success) {
1435 +            assertTrue(success.getCause() instanceof NullPointerException);
1436          } finally {
1437              joinPool(e);
1438          }
# Line 1215 | Line 1442 | public class ThreadPoolExecutorSubclassT
1442       * invokeAny(c) returns result of some task
1443       */
1444      public void testInvokeAny5() throws Exception {
1445 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1445 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1446          try {
1447 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1447 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
1448              l.add(new StringTask());
1449              l.add(new StringTask());
1450              String result = e.invokeAny(l);
# Line 1231 | Line 1458 | public class ThreadPoolExecutorSubclassT
1458       * invokeAll(null) throws NPE
1459       */
1460      public void testInvokeAll1() throws Exception {
1461 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1461 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1462          try {
1463              e.invokeAll(null);
1464              shouldThrow();
# Line 1245 | Line 1472 | public class ThreadPoolExecutorSubclassT
1472       * invokeAll(empty collection) returns empty collection
1473       */
1474      public void testInvokeAll2() throws Exception {
1475 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1475 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1476          try {
1477              List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>());
1478              assertTrue(r.isEmpty());
# Line 1258 | Line 1485 | public class ThreadPoolExecutorSubclassT
1485       * invokeAll(c) throws NPE if c has null elements
1486       */
1487      public void testInvokeAll3() throws Exception {
1488 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1488 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1489 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1490 >        l.add(new StringTask());
1491 >        l.add(null);
1492          try {
1263            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1264            l.add(new StringTask());
1265            l.add(null);
1493              e.invokeAll(l);
1494              shouldThrow();
1495          } catch (NullPointerException success) {
# Line 1275 | Line 1502 | public class ThreadPoolExecutorSubclassT
1502       * get of element of invokeAll(c) throws exception on failed task
1503       */
1504      public void testInvokeAll4() throws Exception {
1505 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1505 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1506 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1507 >        l.add(new NPETask());
1508 >        List<Future<String>> futures = e.invokeAll(l);
1509 >        assertEquals(1, futures.size());
1510          try {
1511 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1281 <            l.add(new NPETask());
1282 <            List<Future<String>> result = e.invokeAll(l);
1283 <            assertEquals(1, result.size());
1284 <            for (Future<String> future : result)
1285 <                future.get();
1511 >            futures.get(0).get();
1512              shouldThrow();
1513          } catch (ExecutionException success) {
1514 +            assertTrue(success.getCause() instanceof NullPointerException);
1515          } finally {
1516              joinPool(e);
1517          }
# Line 1294 | Line 1521 | public class ThreadPoolExecutorSubclassT
1521       * invokeAll(c) returns results of all completed tasks
1522       */
1523      public void testInvokeAll5() throws Exception {
1524 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1524 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1525          try {
1526 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1526 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
1527              l.add(new StringTask());
1528              l.add(new StringTask());
1529 <            List<Future<String>> result = e.invokeAll(l);
1530 <            assertEquals(2, result.size());
1531 <            for (Future<String> future : result)
1529 >            List<Future<String>> futures = e.invokeAll(l);
1530 >            assertEquals(2, futures.size());
1531 >            for (Future<String> future : futures)
1532                  assertSame(TEST_STRING, future.get());
1533          } finally {
1534              joinPool(e);
1535          }
1536      }
1537  
1311
1312
1538      /**
1539       * timed invokeAny(null) throws NPE
1540       */
1541      public void testTimedInvokeAny1() throws Exception {
1542 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1542 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1543          try {
1544 <            e.invokeAny(null, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1544 >            e.invokeAny(null, MEDIUM_DELAY_MS, MILLISECONDS);
1545              shouldThrow();
1546          } catch (NullPointerException success) {
1547          } finally {
# Line 1328 | Line 1553 | public class ThreadPoolExecutorSubclassT
1553       * timed invokeAny(,,null) throws NPE
1554       */
1555      public void testTimedInvokeAnyNullTimeUnit() throws Exception {
1556 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1556 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1557 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1558 >        l.add(new StringTask());
1559          try {
1333            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1334            l.add(new StringTask());
1560              e.invokeAny(l, MEDIUM_DELAY_MS, null);
1561              shouldThrow();
1562          } catch (NullPointerException success) {
# Line 1344 | Line 1569 | public class ThreadPoolExecutorSubclassT
1569       * timed invokeAny(empty collection) throws IAE
1570       */
1571      public void testTimedInvokeAny2() throws Exception {
1572 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1572 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1573          try {
1574 <            e.invokeAny(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1574 >            e.invokeAny(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, MILLISECONDS);
1575              shouldThrow();
1576          } catch (IllegalArgumentException success) {
1577          } finally {
# Line 1358 | Line 1583 | public class ThreadPoolExecutorSubclassT
1583       * timed invokeAny(c) throws NPE if c has null elements
1584       */
1585      public void testTimedInvokeAny3() throws Exception {
1586 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1586 >        CountDownLatch latch = new CountDownLatch(1);
1587 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1588 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1589 >        l.add(latchAwaitingStringTask(latch));
1590 >        l.add(null);
1591          try {
1592 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1364 <            l.add(new StringTask());
1365 <            l.add(null);
1366 <            e.invokeAny(l, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1592 >            e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1593              shouldThrow();
1594          } catch (NullPointerException success) {
1595          } finally {
1596 +            latch.countDown();
1597              joinPool(e);
1598          }
1599      }
# Line 1375 | Line 1602 | public class ThreadPoolExecutorSubclassT
1602       * timed invokeAny(c) throws ExecutionException if no task completes
1603       */
1604      public void testTimedInvokeAny4() throws Exception {
1605 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1605 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1606 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1607 >        l.add(new NPETask());
1608          try {
1609 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1381 <            l.add(new NPETask());
1382 <            e.invokeAny(l, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1609 >            e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1610              shouldThrow();
1611          } catch (ExecutionException success) {
1612 +            assertTrue(success.getCause() instanceof NullPointerException);
1613          } finally {
1614              joinPool(e);
1615          }
# Line 1391 | Line 1619 | public class ThreadPoolExecutorSubclassT
1619       * timed invokeAny(c) returns result of some task
1620       */
1621      public void testTimedInvokeAny5() throws Exception {
1622 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1622 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1623          try {
1624 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1624 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
1625              l.add(new StringTask());
1626              l.add(new StringTask());
1627 <            String result = e.invokeAny(l, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1627 >            String result = e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1628              assertSame(TEST_STRING, result);
1629          } finally {
1630              joinPool(e);
# Line 1407 | Line 1635 | public class ThreadPoolExecutorSubclassT
1635       * timed invokeAll(null) throws NPE
1636       */
1637      public void testTimedInvokeAll1() throws Exception {
1638 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1638 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1639          try {
1640 <            e.invokeAll(null, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1640 >            e.invokeAll(null, MEDIUM_DELAY_MS, MILLISECONDS);
1641              shouldThrow();
1642          } catch (NullPointerException success) {
1643          } finally {
# Line 1421 | Line 1649 | public class ThreadPoolExecutorSubclassT
1649       * timed invokeAll(,,null) throws NPE
1650       */
1651      public void testTimedInvokeAllNullTimeUnit() throws Exception {
1652 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1652 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1653 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1654 >        l.add(new StringTask());
1655          try {
1426            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1427            l.add(new StringTask());
1656              e.invokeAll(l, MEDIUM_DELAY_MS, null);
1657              shouldThrow();
1658          } catch (NullPointerException success) {
# Line 1437 | Line 1665 | public class ThreadPoolExecutorSubclassT
1665       * timed invokeAll(empty collection) returns empty collection
1666       */
1667      public void testTimedInvokeAll2() throws Exception {
1668 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1668 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1669          try {
1670 <            List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1670 >            List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, MILLISECONDS);
1671              assertTrue(r.isEmpty());
1672          } finally {
1673              joinPool(e);
# Line 1450 | Line 1678 | public class ThreadPoolExecutorSubclassT
1678       * timed invokeAll(c) throws NPE if c has null elements
1679       */
1680      public void testTimedInvokeAll3() throws Exception {
1681 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1681 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1682 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1683 >        l.add(new StringTask());
1684 >        l.add(null);
1685          try {
1686 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1456 <            l.add(new StringTask());
1457 <            l.add(null);
1458 <            e.invokeAll(l, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1686 >            e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
1687              shouldThrow();
1688          } catch (NullPointerException success) {
1689          } finally {
# Line 1467 | Line 1695 | public class ThreadPoolExecutorSubclassT
1695       * get of element of invokeAll(c) throws exception on failed task
1696       */
1697      public void testTimedInvokeAll4() throws Exception {
1698 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1698 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1699 >        List<Callable<String>> l = new ArrayList<Callable<String>>();
1700 >        l.add(new NPETask());
1701 >        List<Future<String>> futures =
1702 >            e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
1703 >        assertEquals(1, futures.size());
1704          try {
1705 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1473 <            l.add(new NPETask());
1474 <            List<Future<String>> result = e.invokeAll(l, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1475 <            assertEquals(1, result.size());
1476 <            for (Future<String> future : result)
1477 <                future.get();
1705 >            futures.get(0).get();
1706              shouldThrow();
1707          } catch (ExecutionException success) {
1708              assertTrue(success.getCause() instanceof NullPointerException);
# Line 1487 | Line 1715 | public class ThreadPoolExecutorSubclassT
1715       * timed invokeAll(c) returns results of all completed tasks
1716       */
1717      public void testTimedInvokeAll5() throws Exception {
1718 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1718 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1719          try {
1720 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1720 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
1721              l.add(new StringTask());
1722              l.add(new StringTask());
1723 <            List<Future<String>> result = e.invokeAll(l, MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
1724 <            assertEquals(2, result.size());
1725 <            for (Future<String> future : result)
1723 >            List<Future<String>> futures =
1724 >                e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
1725 >            assertEquals(2, futures.size());
1726 >            for (Future<String> future : futures)
1727                  assertSame(TEST_STRING, future.get());
1499        } catch (ExecutionException success) {
1728          } finally {
1729              joinPool(e);
1730          }
# Line 1506 | Line 1734 | public class ThreadPoolExecutorSubclassT
1734       * timed invokeAll(c) cancels tasks not completed by timeout
1735       */
1736      public void testTimedInvokeAll6() throws Exception {
1737 <        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1737 >        ExecutorService e = new CustomTPE(2, 2, LONG_DELAY_MS, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1738          try {
1739 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
1739 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
1740              l.add(new StringTask());
1741              l.add(Executors.callable(new MediumPossiblyInterruptedRunnable(), TEST_STRING));
1742              l.add(new StringTask());
1743 <            List<Future<String>> result = e.invokeAll(l, SHORT_DELAY_MS, TimeUnit.MILLISECONDS);
1744 <            assertEquals(3, result.size());
1745 <            Iterator<Future<String>> it = result.iterator();
1746 <            Future<String> f1 = it.next();
1747 <            Future<String> f2 = it.next();
1748 <            Future<String> f3 = it.next();
1749 <            assertTrue(f1.isDone());
1522 <            assertTrue(f2.isDone());
1523 <            assertTrue(f3.isDone());
1524 <            assertFalse(f1.isCancelled());
1525 <            assertTrue(f2.isCancelled());
1743 >            List<Future<String>> futures =
1744 >                e.invokeAll(l, SHORT_DELAY_MS, MILLISECONDS);
1745 >            assertEquals(l.size(), futures.size());
1746 >            for (Future future : futures)
1747 >                assertTrue(future.isDone());
1748 >            assertFalse(futures.get(0).isCancelled());
1749 >            assertTrue(futures.get(1).isCancelled());
1750          } finally {
1751              joinPool(e);
1752          }
# Line 1533 | Line 1757 | public class ThreadPoolExecutorSubclassT
1757       * thread factory fails to create more
1758       */
1759      public void testFailingThreadFactory() throws InterruptedException {
1760 <        ExecutorService e = new CustomTPE(100, 100, LONG_DELAY_MS, TimeUnit.MILLISECONDS, new LinkedBlockingQueue<Runnable>(), new FailingThreadFactory());
1761 <        try {
1762 <            for (int k = 0; k < 100; ++k) {
1763 <                e.execute(new NoOpRunnable());
1764 <            }
1765 <            Thread.sleep(LONG_DELAY_MS);
1760 >        final ExecutorService e =
1761 >            new CustomTPE(100, 100,
1762 >                          LONG_DELAY_MS, MILLISECONDS,
1763 >                          new LinkedBlockingQueue<Runnable>(),
1764 >                          new FailingThreadFactory());
1765 >        try {
1766 >            final int TASKS = 100;
1767 >            final CountDownLatch done = new CountDownLatch(TASKS);
1768 >            for (int k = 0; k < TASKS; ++k)
1769 >                e.execute(new CheckedRunnable() {
1770 >                    public void realRun() {
1771 >                        done.countDown();
1772 >                    }});
1773 >            assertTrue(done.await(LONG_DELAY_MS, MILLISECONDS));
1774          } finally {
1775              joinPool(e);
1776          }
# Line 1548 | Line 1780 | public class ThreadPoolExecutorSubclassT
1780       * allowsCoreThreadTimeOut is by default false.
1781       */
1782      public void testAllowsCoreThreadTimeOut() {
1783 <        ThreadPoolExecutor tpe = new CustomTPE(2, 2, 1000, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1784 <        assertFalse(tpe.allowsCoreThreadTimeOut());
1785 <        joinPool(tpe);
1783 >        ThreadPoolExecutor p = new CustomTPE(2, 2, 1000, MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1784 >        assertFalse(p.allowsCoreThreadTimeOut());
1785 >        joinPool(p);
1786      }
1787  
1788      /**
1789       * allowCoreThreadTimeOut(true) causes idle threads to time out
1790       */
1791 <    public void testAllowCoreThreadTimeOut_true() throws InterruptedException {
1792 <        ThreadPoolExecutor tpe = new CustomTPE(2, 10, 10, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1793 <        tpe.allowCoreThreadTimeOut(true);
1794 <        tpe.execute(new NoOpRunnable());
1795 <        try {
1796 <            Thread.sleep(MEDIUM_DELAY_MS);
1797 <            assertEquals(0, tpe.getPoolSize());
1791 >    public void testAllowCoreThreadTimeOut_true() throws Exception {
1792 >        long coreThreadTimeOut = SHORT_DELAY_MS;
1793 >        final ThreadPoolExecutor p =
1794 >            new CustomTPE(2, 10,
1795 >                          coreThreadTimeOut, MILLISECONDS,
1796 >                          new ArrayBlockingQueue<Runnable>(10));
1797 >        final CountDownLatch threadStarted = new CountDownLatch(1);
1798 >        try {
1799 >            p.allowCoreThreadTimeOut(true);
1800 >            p.execute(new CheckedRunnable() {
1801 >                public void realRun() throws InterruptedException {
1802 >                    threadStarted.countDown();
1803 >                    assertEquals(1, p.getPoolSize());
1804 >                }});
1805 >            await(threadStarted);
1806 >            delay(coreThreadTimeOut);
1807 >            long startTime = System.nanoTime();
1808 >            while (p.getPoolSize() > 0
1809 >                   && millisElapsedSince(startTime) < LONG_DELAY_MS)
1810 >                Thread.yield();
1811 >            assertTrue(millisElapsedSince(startTime) < LONG_DELAY_MS);
1812 >            assertEquals(0, p.getPoolSize());
1813          } finally {
1814 <            joinPool(tpe);
1814 >            joinPool(p);
1815          }
1816      }
1817  
1818      /**
1819       * allowCoreThreadTimeOut(false) causes idle threads not to time out
1820       */
1821 <    public void testAllowCoreThreadTimeOut_false() throws InterruptedException {
1822 <        ThreadPoolExecutor tpe = new CustomTPE(2, 10, 10, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10));
1823 <        tpe.allowCoreThreadTimeOut(false);
1824 <        tpe.execute(new NoOpRunnable());
1825 <        try {
1826 <            Thread.sleep(MEDIUM_DELAY_MS);
1827 <            assertTrue(tpe.getPoolSize() >= 1);
1821 >    public void testAllowCoreThreadTimeOut_false() throws Exception {
1822 >        long coreThreadTimeOut = SHORT_DELAY_MS;
1823 >        final ThreadPoolExecutor p =
1824 >            new CustomTPE(2, 10,
1825 >                          coreThreadTimeOut, MILLISECONDS,
1826 >                          new ArrayBlockingQueue<Runnable>(10));
1827 >        final CountDownLatch threadStarted = new CountDownLatch(1);
1828 >        try {
1829 >            p.allowCoreThreadTimeOut(false);
1830 >            p.execute(new CheckedRunnable() {
1831 >                public void realRun() throws InterruptedException {
1832 >                    threadStarted.countDown();
1833 >                    assertTrue(p.getPoolSize() >= 1);
1834 >                }});
1835 >            delay(2 * coreThreadTimeOut);
1836 >            assertTrue(p.getPoolSize() >= 1);
1837          } finally {
1838 <            joinPool(tpe);
1838 >            joinPool(p);
1839          }
1840      }
1841  

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines