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

Comparing jsr166/src/test/tck/ScheduledExecutorSubclassTest.java (file contents):
Revision 1.46 by jsr166, Sun Oct 4 16:03:12 2015 UTC vs.
Revision 1.74 by jsr166, Wed Jan 27 01:57:24 2021 UTC

# Line 5 | Line 5
5   */
6  
7   import static java.util.concurrent.TimeUnit.MILLISECONDS;
8 + import static java.util.concurrent.TimeUnit.NANOSECONDS;
9   import static java.util.concurrent.TimeUnit.SECONDS;
10  
11   import java.util.ArrayList;
12 + import java.util.Collection;
13 + import java.util.Collections;
14   import java.util.HashSet;
15   import java.util.List;
16   import java.util.concurrent.BlockingQueue;
# Line 16 | Line 19 | import java.util.concurrent.Cancellation
19   import java.util.concurrent.CountDownLatch;
20   import java.util.concurrent.Delayed;
21   import java.util.concurrent.ExecutionException;
19 import java.util.concurrent.Executors;
22   import java.util.concurrent.ExecutorService;
23   import java.util.concurrent.Future;
24   import java.util.concurrent.RejectedExecutionException;
# Line 25 | Line 27 | import java.util.concurrent.RunnableSche
27   import java.util.concurrent.ScheduledFuture;
28   import java.util.concurrent.ScheduledThreadPoolExecutor;
29   import java.util.concurrent.ThreadFactory;
30 + import java.util.concurrent.ThreadLocalRandom;
31   import java.util.concurrent.ThreadPoolExecutor;
32   import java.util.concurrent.TimeoutException;
33   import java.util.concurrent.TimeUnit;
34 + import java.util.concurrent.atomic.AtomicBoolean;
35   import java.util.concurrent.atomic.AtomicInteger;
36 + import java.util.concurrent.atomic.AtomicLong;
37 + import java.util.stream.Stream;
38  
39   import junit.framework.Test;
40   import junit.framework.TestSuite;
# Line 42 | Line 48 | public class ScheduledExecutorSubclassTe
48      }
49  
50      static class CustomTask<V> implements RunnableScheduledFuture<V> {
51 <        RunnableScheduledFuture<V> task;
51 >        private final RunnableScheduledFuture<V> task;
52          volatile boolean ran;
53 <        CustomTask(RunnableScheduledFuture<V> t) { task = t; }
53 >        CustomTask(RunnableScheduledFuture<V> task) { this.task = task; }
54          public boolean isPeriodic() { return task.isPeriodic(); }
55          public void run() {
56              ran = true;
# Line 105 | Line 111 | public class ScheduledExecutorSubclassTe
111              final Runnable task = new CheckedRunnable() {
112                  public void realRun() { done.countDown(); }};
113              p.execute(task);
114 <            assertTrue(done.await(LONG_DELAY_MS, MILLISECONDS));
114 >            await(done);
115          }
116      }
117  
# Line 113 | Line 119 | public class ScheduledExecutorSubclassTe
119       * delayed schedule of callable successfully executes after delay
120       */
121      public void testSchedule1() throws Exception {
122 +        final CountDownLatch done = new CountDownLatch(1);
123          final CustomExecutor p = new CustomExecutor(1);
124 <        try (PoolCleaner cleaner = cleaner(p)) {
124 >        try (PoolCleaner cleaner = cleaner(p, done)) {
125              final long startTime = System.nanoTime();
126 <            final CountDownLatch done = new CountDownLatch(1);
120 <            Callable task = new CheckedCallable<Boolean>() {
126 >            Callable<Boolean> task = new CheckedCallable<>() {
127                  public Boolean realCall() {
128                      done.countDown();
129                      assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
130                      return Boolean.TRUE;
131                  }};
132 <            Future f = p.schedule(task, timeoutMillis(), MILLISECONDS);
132 >            Future<Boolean> f = p.schedule(task, timeoutMillis(), MILLISECONDS);
133              assertSame(Boolean.TRUE, f.get());
134              assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
129            assertTrue(done.await(0L, MILLISECONDS));
135          }
136      }
137  
# Line 143 | Line 148 | public class ScheduledExecutorSubclassTe
148                      done.countDown();
149                      assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
150                  }};
151 <            Future f = p.schedule(task, timeoutMillis(), MILLISECONDS);
151 >            Future<?> f = p.schedule(task, timeoutMillis(), MILLISECONDS);
152              await(done);
153              assertNull(f.get(LONG_DELAY_MS, MILLISECONDS));
154              assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
# Line 163 | Line 168 | public class ScheduledExecutorSubclassTe
168                      done.countDown();
169                      assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
170                  }};
171 <            ScheduledFuture f =
171 >            ScheduledFuture<?> f =
172                  p.scheduleAtFixedRate(task, timeoutMillis(),
173                                        LONG_DELAY_MS, MILLISECONDS);
174              await(done);
# Line 185 | Line 190 | public class ScheduledExecutorSubclassTe
190                      done.countDown();
191                      assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
192                  }};
193 <            ScheduledFuture f =
193 >            ScheduledFuture<?> f =
194                  p.scheduleWithFixedDelay(task, timeoutMillis(),
195                                           LONG_DELAY_MS, MILLISECONDS);
196              await(done);
# Line 200 | Line 205 | public class ScheduledExecutorSubclassTe
205      }
206  
207      /**
208 <     * scheduleAtFixedRate executes series of tasks at given rate
208 >     * scheduleAtFixedRate executes series of tasks at given rate.
209 >     * Eventually, it must hold that:
210 >     *   cycles - 1 <= elapsedMillis/delay < cycles
211       */
212      public void testFixedRateSequence() throws InterruptedException {
213          final CustomExecutor p = new CustomExecutor(1);
214          try (PoolCleaner cleaner = cleaner(p)) {
215              for (int delay = 1; delay <= LONG_DELAY_MS; delay *= 3) {
216 <                long startTime = System.nanoTime();
217 <                int cycles = 10;
216 >                final long startTime = System.nanoTime();
217 >                final int cycles = 8;
218                  final CountDownLatch done = new CountDownLatch(cycles);
219 <                Runnable task = new CheckedRunnable() {
219 >                final Runnable task = new CheckedRunnable() {
220                      public void realRun() { done.countDown(); }};
221 <                ScheduledFuture h =
221 >                final ScheduledFuture<?> periodicTask =
222                      p.scheduleAtFixedRate(task, 0, delay, MILLISECONDS);
223 <                done.await();
224 <                h.cancel(true);
225 <                double normalizedTime =
226 <                    (double) millisElapsedSince(startTime) / delay;
227 <                if (normalizedTime >= cycles - 1 &&
228 <                    normalizedTime <= cycles)
223 >                final int totalDelayMillis = (cycles - 1) * delay;
224 >                await(done, totalDelayMillis + LONG_DELAY_MS);
225 >                periodicTask.cancel(true);
226 >                final long elapsedMillis = millisElapsedSince(startTime);
227 >                assertTrue(elapsedMillis >= totalDelayMillis);
228 >                if (elapsedMillis <= cycles * delay)
229                      return;
230 +                // else retry with longer delay
231              }
232 <            throw new AssertionError("unexpected execution rate");
232 >            fail("unexpected execution rate");
233          }
234      }
235  
236      /**
237 <     * scheduleWithFixedDelay executes series of tasks with given period
237 >     * scheduleWithFixedDelay executes series of tasks with given period.
238 >     * Eventually, it must hold that each task starts at least delay and at
239 >     * most 2 * delay after the termination of the previous task.
240       */
241      public void testFixedDelaySequence() throws InterruptedException {
242          final CustomExecutor p = new CustomExecutor(1);
243          try (PoolCleaner cleaner = cleaner(p)) {
244              for (int delay = 1; delay <= LONG_DELAY_MS; delay *= 3) {
245 <                long startTime = System.nanoTime();
246 <                int cycles = 10;
245 >                final long startTime = System.nanoTime();
246 >                final AtomicLong previous = new AtomicLong(startTime);
247 >                final AtomicBoolean tryLongerDelay = new AtomicBoolean(false);
248 >                final int cycles = 8;
249                  final CountDownLatch done = new CountDownLatch(cycles);
250 <                Runnable task = new CheckedRunnable() {
251 <                    public void realRun() { done.countDown(); }};
252 <                ScheduledFuture h =
250 >                final int d = delay;
251 >                final Runnable task = new CheckedRunnable() {
252 >                    public void realRun() {
253 >                        long now = System.nanoTime();
254 >                        long elapsedMillis
255 >                            = NANOSECONDS.toMillis(now - previous.get());
256 >                        if (done.getCount() == cycles) { // first execution
257 >                            if (elapsedMillis >= d)
258 >                                tryLongerDelay.set(true);
259 >                        } else {
260 >                            assertTrue(elapsedMillis >= d);
261 >                            if (elapsedMillis >= 2 * d)
262 >                                tryLongerDelay.set(true);
263 >                        }
264 >                        previous.set(now);
265 >                        done.countDown();
266 >                    }};
267 >                final ScheduledFuture<?> periodicTask =
268                      p.scheduleWithFixedDelay(task, 0, delay, MILLISECONDS);
269 <                done.await();
270 <                h.cancel(true);
271 <                double normalizedTime =
272 <                    (double) millisElapsedSince(startTime) / delay;
273 <                if (normalizedTime >= cycles - 1 &&
274 <                    normalizedTime <= cycles)
269 >                final int totalDelayMillis = (cycles - 1) * delay;
270 >                await(done, totalDelayMillis + cycles * LONG_DELAY_MS);
271 >                periodicTask.cancel(true);
272 >                final long elapsedMillis = millisElapsedSince(startTime);
273 >                assertTrue(elapsedMillis >= totalDelayMillis);
274 >                if (!tryLongerDelay.get())
275                      return;
276 +                // else retry with longer delay
277              }
278 <            throw new AssertionError("unexpected execution rate");
278 >            fail("unexpected execution rate");
279          }
280      }
281  
282      /**
283 <     * execute(null) throws NPE
283 >     * Submitting null tasks throws NullPointerException
284       */
285 <    public void testExecuteNull() throws InterruptedException {
285 >    public void testNullTaskSubmission() {
286          final CustomExecutor p = new CustomExecutor(1);
287          try (PoolCleaner cleaner = cleaner(p)) {
288 <            try {
261 <                p.execute(null);
262 <                shouldThrow();
263 <            } catch (NullPointerException success) {}
288 >            assertNullTaskSubmissionThrowsNullPointerException(p);
289          }
290      }
291  
292      /**
293 <     * schedule(null) throws NPE
293 >     * Submitted tasks are rejected when shutdown
294       */
295 <    public void testScheduleNull() throws InterruptedException {
296 <        final CustomExecutor p = new CustomExecutor(1);
297 <        try (PoolCleaner cleaner = cleaner(p)) {
298 <            try {
299 <                TrackedCallable callable = null;
300 <                Future f = p.schedule(callable, SHORT_DELAY_MS, MILLISECONDS);
301 <                shouldThrow();
302 <            } catch (NullPointerException success) {}
303 <        }
304 <    }
295 >    public void testSubmittedTasksRejectedWhenShutdown() throws InterruptedException {
296 >        final CustomExecutor p = new CustomExecutor(2);
297 >        final ThreadLocalRandom rnd = ThreadLocalRandom.current();
298 >        final CountDownLatch threadsStarted = new CountDownLatch(p.getCorePoolSize());
299 >        final CountDownLatch done = new CountDownLatch(1);
300 >        final Runnable r = () -> {
301 >            threadsStarted.countDown();
302 >            for (;;) {
303 >                try {
304 >                    done.await();
305 >                    return;
306 >                } catch (InterruptedException shutdownNowDeliberatelyIgnored) {}
307 >            }};
308 >        final Callable<Boolean> c = () -> {
309 >            threadsStarted.countDown();
310 >            for (;;) {
311 >                try {
312 >                    done.await();
313 >                    return Boolean.TRUE;
314 >                } catch (InterruptedException shutdownNowDeliberatelyIgnored) {}
315 >            }};
316  
317 <    /**
318 <     * execute throws RejectedExecutionException if shutdown
319 <     */
320 <    public void testSchedule1_RejectedExecutionException() {
321 <        final CustomExecutor p = new CustomExecutor(1);
322 <        try (PoolCleaner cleaner = cleaner(p)) {
323 <            try {
324 <                p.shutdown();
325 <                p.schedule(new NoOpRunnable(),
290 <                           MEDIUM_DELAY_MS, MILLISECONDS);
291 <                shouldThrow();
292 <            } catch (RejectedExecutionException success) {
293 <            } catch (SecurityException ok) {}
294 <        }
295 <    }
317 >        try (PoolCleaner cleaner = cleaner(p, done)) {
318 >            for (int i = p.getCorePoolSize(); i--> 0; ) {
319 >                switch (rnd.nextInt(4)) {
320 >                case 0: p.execute(r); break;
321 >                case 1: assertFalse(p.submit(r).isDone()); break;
322 >                case 2: assertFalse(p.submit(r, Boolean.TRUE).isDone()); break;
323 >                case 3: assertFalse(p.submit(c).isDone()); break;
324 >                }
325 >            }
326  
327 <    /**
328 <     * schedule throws RejectedExecutionException if shutdown
299 <     */
300 <    public void testSchedule2_RejectedExecutionException() {
301 <        final CustomExecutor p = new CustomExecutor(1);
302 <        try (PoolCleaner cleaner = cleaner(p)) {
303 <            try {
304 <                p.shutdown();
305 <                p.schedule(new NoOpCallable(),
306 <                           MEDIUM_DELAY_MS, MILLISECONDS);
307 <                shouldThrow();
308 <            } catch (RejectedExecutionException success) {
309 <            } catch (SecurityException ok) {}
310 <        }
311 <    }
327 >            // ScheduledThreadPoolExecutor has an unbounded queue, so never saturated.
328 >            await(threadsStarted);
329  
330 <    /**
331 <     * schedule callable throws RejectedExecutionException if shutdown
332 <     */
316 <    public void testSchedule3_RejectedExecutionException() {
317 <        final CustomExecutor p = new CustomExecutor(1);
318 <        try (PoolCleaner cleaner = cleaner(p)) {
319 <            try {
330 >            if (rnd.nextBoolean())
331 >                p.shutdownNow();
332 >            else
333                  p.shutdown();
334 <                p.schedule(new NoOpCallable(),
335 <                           MEDIUM_DELAY_MS, MILLISECONDS);
336 <                shouldThrow();
324 <            } catch (RejectedExecutionException success) {
325 <            } catch (SecurityException ok) {}
326 <        }
327 <    }
334 >            // Pool is shutdown, but not yet terminated
335 >            assertTaskSubmissionsAreRejected(p);
336 >            assertFalse(p.isTerminated());
337  
338 <    /**
339 <     * scheduleAtFixedRate throws RejectedExecutionException if shutdown
331 <     */
332 <    public void testScheduleAtFixedRate1_RejectedExecutionException() {
333 <        final CustomExecutor p = new CustomExecutor(1);
334 <        try (PoolCleaner cleaner = cleaner(p)) {
335 <            try {
336 <                p.shutdown();
337 <                p.scheduleAtFixedRate(new NoOpRunnable(),
338 <                                      MEDIUM_DELAY_MS, MEDIUM_DELAY_MS, MILLISECONDS);
339 <                shouldThrow();
340 <            } catch (RejectedExecutionException success) {
341 <            } catch (SecurityException ok) {}
342 <        }
343 <    }
338 >            done.countDown();   // release blocking tasks
339 >            assertTrue(p.awaitTermination(LONG_DELAY_MS, MILLISECONDS));
340  
341 <    /**
346 <     * scheduleWithFixedDelay throws RejectedExecutionException if shutdown
347 <     */
348 <    public void testScheduleWithFixedDelay1_RejectedExecutionException() {
349 <        final CustomExecutor p = new CustomExecutor(1);
350 <        try (PoolCleaner cleaner = cleaner(p)) {
351 <            try {
352 <                p.shutdown();
353 <                p.scheduleWithFixedDelay(new NoOpRunnable(),
354 <                                         MEDIUM_DELAY_MS, MEDIUM_DELAY_MS, MILLISECONDS);
355 <                shouldThrow();
356 <            } catch (RejectedExecutionException success) {
357 <            } catch (SecurityException ok) {}
341 >            assertTaskSubmissionsAreRejected(p);
342          }
343 +        assertEquals(p.getCorePoolSize(), p.getCompletedTaskCount());
344      }
345  
346      /**
# Line 363 | Line 348 | public class ScheduledExecutorSubclassTe
348       * thread becomes active
349       */
350      public void testGetActiveCount() throws InterruptedException {
351 +        final CountDownLatch done = new CountDownLatch(1);
352          final ThreadPoolExecutor p = new CustomExecutor(2);
353 <        try (PoolCleaner cleaner = cleaner(p)) {
353 >        try (PoolCleaner cleaner = cleaner(p, done)) {
354              final CountDownLatch threadStarted = new CountDownLatch(1);
369            final CountDownLatch done = new CountDownLatch(1);
355              assertEquals(0, p.getActiveCount());
356              p.execute(new CheckedRunnable() {
357                  public void realRun() throws InterruptedException {
358                      threadStarted.countDown();
359                      assertEquals(1, p.getActiveCount());
360 <                    done.await();
360 >                    await(done);
361                  }});
362 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
362 >            await(threadStarted);
363              assertEquals(1, p.getActiveCount());
379            done.countDown();
364          }
365      }
366  
# Line 395 | Line 379 | public class ScheduledExecutorSubclassTe
379                  public void realRun() throws InterruptedException {
380                      threadStarted.countDown();
381                      assertEquals(0, p.getCompletedTaskCount());
382 <                    threadProceed.await();
382 >                    await(threadProceed);
383                      threadDone.countDown();
384                  }});
385              await(threadStarted);
386              assertEquals(0, p.getCompletedTaskCount());
387              threadProceed.countDown();
388 <            threadDone.await();
388 >            await(threadDone);
389              long startTime = System.nanoTime();
390              while (p.getCompletedTaskCount() != 1) {
391                  if (millisElapsedSince(startTime) > LONG_DELAY_MS)
# Line 427 | Line 411 | public class ScheduledExecutorSubclassTe
411       */
412      public void testGetLargestPoolSize() throws InterruptedException {
413          final int THREADS = 3;
414 +        final CountDownLatch done = new CountDownLatch(1);
415          final ThreadPoolExecutor p = new CustomExecutor(THREADS);
416 <        try (PoolCleaner cleaner = cleaner(p)) {
416 >        try (PoolCleaner cleaner = cleaner(p, done)) {
417              final CountDownLatch threadsStarted = new CountDownLatch(THREADS);
433            final CountDownLatch done = new CountDownLatch(1);
418              assertEquals(0, p.getLargestPoolSize());
419              for (int i = 0; i < THREADS; i++)
420                  p.execute(new CheckedRunnable() {
421                      public void realRun() throws InterruptedException {
422                          threadsStarted.countDown();
423 <                        done.await();
423 >                        await(done);
424                          assertEquals(THREADS, p.getLargestPoolSize());
425                      }});
426 <            assertTrue(threadsStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
426 >            await(threadsStarted);
427              assertEquals(THREADS, p.getLargestPoolSize());
444            done.countDown();
428          }
429          assertEquals(THREADS, p.getLargestPoolSize());
430      }
# Line 451 | Line 434 | public class ScheduledExecutorSubclassTe
434       * become active
435       */
436      public void testGetPoolSize() throws InterruptedException {
437 +        final CountDownLatch done = new CountDownLatch(1);
438          final ThreadPoolExecutor p = new CustomExecutor(1);
439 <        try (PoolCleaner cleaner = cleaner(p)) {
439 >        try (PoolCleaner cleaner = cleaner(p, done)) {
440              final CountDownLatch threadStarted = new CountDownLatch(1);
457            final CountDownLatch done = new CountDownLatch(1);
441              assertEquals(0, p.getPoolSize());
442              p.execute(new CheckedRunnable() {
443                  public void realRun() throws InterruptedException {
444                      threadStarted.countDown();
445                      assertEquals(1, p.getPoolSize());
446 <                    done.await();
446 >                    await(done);
447                  }});
448 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
448 >            await(threadStarted);
449              assertEquals(1, p.getPoolSize());
467            done.countDown();
450          }
451      }
452  
# Line 473 | Line 455 | public class ScheduledExecutorSubclassTe
455       * submitted
456       */
457      public void testGetTaskCount() throws InterruptedException {
458 +        final int TASKS = 3;
459 +        final CountDownLatch done = new CountDownLatch(1);
460          final ThreadPoolExecutor p = new CustomExecutor(1);
461 <        try (PoolCleaner cleaner = cleaner(p)) {
461 >        try (PoolCleaner cleaner = cleaner(p, done)) {
462              final CountDownLatch threadStarted = new CountDownLatch(1);
479            final CountDownLatch done = new CountDownLatch(1);
480            final int TASKS = 5;
463              assertEquals(0, p.getTaskCount());
464 <            for (int i = 0; i < TASKS; i++)
464 >            assertEquals(0, p.getCompletedTaskCount());
465 >            p.execute(new CheckedRunnable() {
466 >                public void realRun() throws InterruptedException {
467 >                    threadStarted.countDown();
468 >                    await(done);
469 >                }});
470 >            await(threadStarted);
471 >            assertEquals(1, p.getTaskCount());
472 >            assertEquals(0, p.getCompletedTaskCount());
473 >            for (int i = 0; i < TASKS; i++) {
474 >                assertEquals(1 + i, p.getTaskCount());
475                  p.execute(new CheckedRunnable() {
476                      public void realRun() throws InterruptedException {
477                          threadStarted.countDown();
478 <                        done.await();
478 >                        assertEquals(1 + TASKS, p.getTaskCount());
479 >                        await(done);
480                      }});
481 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
482 <            assertEquals(TASKS, p.getTaskCount());
483 <            done.countDown();
481 >            }
482 >            assertEquals(1 + TASKS, p.getTaskCount());
483 >            assertEquals(0, p.getCompletedTaskCount());
484          }
485 +        assertEquals(1 + TASKS, p.getTaskCount());
486 +        assertEquals(1 + TASKS, p.getCompletedTaskCount());
487      }
488  
489      /**
# Line 543 | Line 538 | public class ScheduledExecutorSubclassTe
538       * isTerminated is false before termination, true after
539       */
540      public void testIsTerminated() throws InterruptedException {
541 +        final CountDownLatch done = new CountDownLatch(1);
542          final ThreadPoolExecutor p = new CustomExecutor(1);
543          try (PoolCleaner cleaner = cleaner(p)) {
544              final CountDownLatch threadStarted = new CountDownLatch(1);
549            final CountDownLatch done = new CountDownLatch(1);
550            assertFalse(p.isTerminated());
545              p.execute(new CheckedRunnable() {
546                  public void realRun() throws InterruptedException {
547                      assertFalse(p.isTerminated());
548                      threadStarted.countDown();
549 <                    done.await();
549 >                    await(done);
550                  }});
551 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
551 >            await(threadStarted);
552 >            assertFalse(p.isTerminated());
553              assertFalse(p.isTerminating());
554              done.countDown();
555              try { p.shutdown(); } catch (SecurityException ok) { return; }
# Line 567 | Line 562 | public class ScheduledExecutorSubclassTe
562       * isTerminating is not true when running or when terminated
563       */
564      public void testIsTerminating() throws InterruptedException {
565 +        final CountDownLatch done = new CountDownLatch(1);
566          final ThreadPoolExecutor p = new CustomExecutor(1);
567          try (PoolCleaner cleaner = cleaner(p)) {
568              final CountDownLatch threadStarted = new CountDownLatch(1);
573            final CountDownLatch done = new CountDownLatch(1);
569              assertFalse(p.isTerminating());
570              p.execute(new CheckedRunnable() {
571                  public void realRun() throws InterruptedException {
572                      assertFalse(p.isTerminating());
573                      threadStarted.countDown();
574 <                    done.await();
574 >                    await(done);
575                  }});
576 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
576 >            await(threadStarted);
577              assertFalse(p.isTerminating());
578              done.countDown();
579              try { p.shutdown(); } catch (SecurityException ok) { return; }
# Line 592 | Line 587 | public class ScheduledExecutorSubclassTe
587       * getQueue returns the work queue, which contains queued tasks
588       */
589      public void testGetQueue() throws InterruptedException {
590 +        final CountDownLatch done = new CountDownLatch(1);
591          final ScheduledThreadPoolExecutor p = new CustomExecutor(1);
592 <        try (PoolCleaner cleaner = cleaner(p)) {
592 >        try (PoolCleaner cleaner = cleaner(p, done)) {
593              final CountDownLatch threadStarted = new CountDownLatch(1);
594 <            final CountDownLatch done = new CountDownLatch(1);
595 <            ScheduledFuture[] tasks = new ScheduledFuture[5];
594 >            @SuppressWarnings("unchecked")
595 >            ScheduledFuture<?>[] tasks = (ScheduledFuture<?>[])new ScheduledFuture[5];
596              for (int i = 0; i < tasks.length; i++) {
597                  Runnable r = new CheckedRunnable() {
598                      public void realRun() throws InterruptedException {
599                          threadStarted.countDown();
600 <                        done.await();
600 >                        await(done);
601                      }};
602                  tasks[i] = p.schedule(r, 1, MILLISECONDS);
603              }
604 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
604 >            await(threadStarted);
605              BlockingQueue<Runnable> q = p.getQueue();
606              assertTrue(q.contains(tasks[tasks.length - 1]));
607              assertFalse(q.contains(tasks[0]));
612            done.countDown();
608          }
609      }
610  
# Line 617 | Line 612 | public class ScheduledExecutorSubclassTe
612       * remove(task) removes queued task, and fails to remove active task
613       */
614      public void testRemove() throws InterruptedException {
615 +        final CountDownLatch done = new CountDownLatch(1);
616          final ScheduledThreadPoolExecutor p = new CustomExecutor(1);
617 <        try (PoolCleaner cleaner = cleaner(p)) {
618 <            ScheduledFuture[] tasks = new ScheduledFuture[5];
617 >        try (PoolCleaner cleaner = cleaner(p, done)) {
618 >            @SuppressWarnings("unchecked")
619 >            ScheduledFuture<?>[] tasks = (ScheduledFuture<?>[])new ScheduledFuture[5];
620              final CountDownLatch threadStarted = new CountDownLatch(1);
624            final CountDownLatch done = new CountDownLatch(1);
621              for (int i = 0; i < tasks.length; i++) {
622                  Runnable r = new CheckedRunnable() {
623                      public void realRun() throws InterruptedException {
624                          threadStarted.countDown();
625 <                        done.await();
625 >                        await(done);
626                      }};
627                  tasks[i] = p.schedule(r, 1, MILLISECONDS);
628              }
629 <            assertTrue(threadStarted.await(MEDIUM_DELAY_MS, MILLISECONDS));
629 >            await(threadStarted);
630              BlockingQueue<Runnable> q = p.getQueue();
631              assertFalse(p.remove((Runnable)tasks[0]));
632              assertTrue(q.contains((Runnable)tasks[4]));
# Line 641 | Line 637 | public class ScheduledExecutorSubclassTe
637              assertTrue(q.contains((Runnable)tasks[3]));
638              assertTrue(p.remove((Runnable)tasks[3]));
639              assertFalse(q.contains((Runnable)tasks[3]));
644            done.countDown();
640          }
641      }
642  
# Line 649 | Line 644 | public class ScheduledExecutorSubclassTe
644       * purge removes cancelled tasks from the queue
645       */
646      public void testPurge() throws InterruptedException {
647 +        @SuppressWarnings("unchecked")
648 +        ScheduledFuture<?>[] tasks = (ScheduledFuture<?>[])new ScheduledFuture[5];
649 +        final Runnable releaser = new Runnable() { public void run() {
650 +            for (ScheduledFuture<?> task : tasks)
651 +                if (task != null) task.cancel(true); }};
652          final CustomExecutor p = new CustomExecutor(1);
653 <        ScheduledFuture[] tasks = new ScheduledFuture[5];
654 <        for (int i = 0; i < tasks.length; i++)
655 <            tasks[i] = p.schedule(new SmallPossiblyInterruptedRunnable(),
656 <                                  LONG_DELAY_MS, MILLISECONDS);
657 <        try {
653 >        try (PoolCleaner cleaner = cleaner(p, releaser)) {
654 >            for (int i = 0; i < tasks.length; i++)
655 >                tasks[i] = p.schedule(possiblyInterruptedRunnable(SMALL_DELAY_MS),
656 >                                      LONG_DELAY_MS, MILLISECONDS);
657              int max = tasks.length;
658              if (tasks[4].cancel(true)) --max;
659              if (tasks[3].cancel(true)) --max;
# Line 666 | Line 665 | public class ScheduledExecutorSubclassTe
665                  long count = p.getTaskCount();
666                  if (count == max)
667                      return;
668 <            } while (millisElapsedSince(startTime) < MEDIUM_DELAY_MS);
668 >            } while (millisElapsedSince(startTime) < LONG_DELAY_MS);
669              fail("Purge failed to remove cancelled tasks");
671        } finally {
672            for (ScheduledFuture task : tasks)
673                task.cancel(true);
674            joinPool(p);
670          }
671      }
672  
# Line 684 | Line 679 | public class ScheduledExecutorSubclassTe
679          final int count = 5;
680          final AtomicInteger ran = new AtomicInteger(0);
681          final CustomExecutor p = new CustomExecutor(poolSize);
682 <        CountDownLatch threadsStarted = new CountDownLatch(poolSize);
682 >        final CountDownLatch threadsStarted = new CountDownLatch(poolSize);
683          Runnable waiter = new CheckedRunnable() { public void realRun() {
684              threadsStarted.countDown();
685              try {
686 <                MILLISECONDS.sleep(2 * LONG_DELAY_MS);
686 >                MILLISECONDS.sleep(LONGER_DELAY_MS);
687              } catch (InterruptedException success) {}
688              ran.getAndIncrement();
689          }};
690          for (int i = 0; i < count; i++)
691              p.execute(waiter);
692 <        assertTrue(threadsStarted.await(LONG_DELAY_MS, MILLISECONDS));
692 >        await(threadsStarted);
693          assertEquals(poolSize, p.getActiveCount());
694          assertEquals(0, p.getCompletedTaskCount());
695          final List<Runnable> queuedTasks;
# Line 718 | Line 713 | public class ScheduledExecutorSubclassTe
713       */
714      public void testShutdownNow_delayedTasks() throws InterruptedException {
715          final CustomExecutor p = new CustomExecutor(1);
716 <        List<ScheduledFuture> tasks = new ArrayList<>();
716 >        List<ScheduledFuture<?>> tasks = new ArrayList<>();
717          for (int i = 0; i < 3; i++) {
718              Runnable r = new NoOpRunnable();
719              tasks.add(p.schedule(r, 9, SECONDS));
# Line 726 | Line 721 | public class ScheduledExecutorSubclassTe
721              tasks.add(p.scheduleWithFixedDelay(r, 9, 9, SECONDS));
722          }
723          if (testImplementationDetails)
724 <            assertEquals(new HashSet(tasks), new HashSet(p.getQueue()));
724 >            assertEquals(new HashSet<Object>(tasks), new HashSet<Object>(p.getQueue()));
725          final List<Runnable> queuedTasks;
726          try {
727              queuedTasks = p.shutdownNow();
# Line 736 | Line 731 | public class ScheduledExecutorSubclassTe
731          assertTrue(p.isShutdown());
732          assertTrue(p.getQueue().isEmpty());
733          if (testImplementationDetails)
734 <            assertEquals(new HashSet(tasks), new HashSet(queuedTasks));
734 >            assertEquals(new HashSet<Object>(tasks), new HashSet<Object>(queuedTasks));
735          assertEquals(tasks.size(), queuedTasks.size());
736 <        for (ScheduledFuture task : tasks) {
736 >        for (ScheduledFuture<?> task : tasks) {
737              assertFalse(((CustomTask)task).ran);
738              assertFalse(task.isDone());
739              assertFalse(task.isCancelled());
# Line 754 | Line 749 | public class ScheduledExecutorSubclassTe
749       * - setExecuteExistingDelayedTasksAfterShutdownPolicy
750       * - setContinueExistingPeriodicTasksAfterShutdownPolicy
751       */
752 +    @SuppressWarnings("FutureReturnValueIgnored")
753      public void testShutdown_cancellation() throws Exception {
754 <        Boolean[] allBooleans = { null, Boolean.FALSE, Boolean.TRUE };
759 <        for (Boolean policy : allBooleans)
760 <    {
761 <        final int poolSize = 2;
754 >        final int poolSize = 4;
755          final CustomExecutor p = new CustomExecutor(poolSize);
756 <        final boolean effectiveDelayedPolicy = (policy != Boolean.FALSE);
757 <        final boolean effectivePeriodicPolicy = (policy == Boolean.TRUE);
758 <        final boolean effectiveRemovePolicy = (policy == Boolean.TRUE);
759 <        if (policy != null) {
760 <            p.setExecuteExistingDelayedTasksAfterShutdownPolicy(policy);
761 <            p.setContinueExistingPeriodicTasksAfterShutdownPolicy(policy);
762 <            p.setRemoveOnCancelPolicy(policy);
763 <        }
756 >        final BlockingQueue<Runnable> q = p.getQueue();
757 >        final ThreadLocalRandom rnd = ThreadLocalRandom.current();
758 >        final long delay = rnd.nextInt(2);
759 >        final int rounds = rnd.nextInt(1, 3);
760 >        final boolean effectiveDelayedPolicy;
761 >        final boolean effectivePeriodicPolicy;
762 >        final boolean effectiveRemovePolicy;
763 >
764 >        if (rnd.nextBoolean())
765 >            p.setExecuteExistingDelayedTasksAfterShutdownPolicy(
766 >                effectiveDelayedPolicy = rnd.nextBoolean());
767 >        else
768 >            effectiveDelayedPolicy = true;
769          assertEquals(effectiveDelayedPolicy,
770                       p.getExecuteExistingDelayedTasksAfterShutdownPolicy());
771 +
772 +        if (rnd.nextBoolean())
773 +            p.setContinueExistingPeriodicTasksAfterShutdownPolicy(
774 +                effectivePeriodicPolicy = rnd.nextBoolean());
775 +        else
776 +            effectivePeriodicPolicy = false;
777          assertEquals(effectivePeriodicPolicy,
778                       p.getContinueExistingPeriodicTasksAfterShutdownPolicy());
779 +
780 +        if (rnd.nextBoolean())
781 +            p.setRemoveOnCancelPolicy(
782 +                effectiveRemovePolicy = rnd.nextBoolean());
783 +        else
784 +            effectiveRemovePolicy = false;
785          assertEquals(effectiveRemovePolicy,
786                       p.getRemoveOnCancelPolicy());
787 <        // Strategy: Wedge the pool with poolSize "blocker" threads
787 >
788 >        final boolean periodicTasksContinue = effectivePeriodicPolicy && rnd.nextBoolean();
789 >
790 >        // Strategy: Wedge the pool with one wave of "blocker" tasks,
791 >        // then add a second wave that waits in the queue until unblocked.
792          final AtomicInteger ran = new AtomicInteger(0);
793          final CountDownLatch poolBlocked = new CountDownLatch(poolSize);
794          final CountDownLatch unblock = new CountDownLatch(1);
795 <        final CountDownLatch periodicLatch1 = new CountDownLatch(2);
782 <        final CountDownLatch periodicLatch2 = new CountDownLatch(2);
783 <        Runnable task = new CheckedRunnable() { public void realRun()
784 <                                                    throws InterruptedException {
785 <            poolBlocked.countDown();
786 <            assertTrue(unblock.await(LONG_DELAY_MS, MILLISECONDS));
787 <            ran.getAndIncrement();
788 <        }};
789 <        List<Future<?>> blockers = new ArrayList<>();
790 <        List<Future<?>> periodics = new ArrayList<>();
791 <        List<Future<?>> delayeds = new ArrayList<>();
792 <        for (int i = 0; i < poolSize; i++)
793 <            blockers.add(p.submit(task));
794 <        assertTrue(poolBlocked.await(LONG_DELAY_MS, MILLISECONDS));
795 <
796 <        periodics.add(p.scheduleAtFixedRate(countDowner(periodicLatch1),
797 <                                            1, 1, MILLISECONDS));
798 <        periodics.add(p.scheduleWithFixedDelay(countDowner(periodicLatch2),
799 <                                               1, 1, MILLISECONDS));
800 <        delayeds.add(p.schedule(task, 1, MILLISECONDS));
795 >        final RuntimeException exception = new RuntimeException();
796  
797 <        assertTrue(p.getQueue().containsAll(periodics));
798 <        assertTrue(p.getQueue().containsAll(delayeds));
799 <        try { p.shutdown(); } catch (SecurityException ok) { return; }
800 <        assertTrue(p.isShutdown());
801 <        assertFalse(p.isTerminated());
802 <        for (Future<?> periodic : periodics) {
803 <            assertTrue(effectivePeriodicPolicy ^ periodic.isCancelled());
809 <            assertTrue(effectivePeriodicPolicy ^ periodic.isDone());
810 <        }
811 <        for (Future<?> delayed : delayeds) {
812 <            assertTrue(effectiveDelayedPolicy ^ delayed.isCancelled());
813 <            assertTrue(effectiveDelayedPolicy ^ delayed.isDone());
814 <        }
815 <        if (testImplementationDetails) {
816 <            assertEquals(effectivePeriodicPolicy,
817 <                         p.getQueue().containsAll(periodics));
818 <            assertEquals(effectiveDelayedPolicy,
819 <                         p.getQueue().containsAll(delayeds));
820 <        }
821 <        // Release all pool threads
822 <        unblock.countDown();
823 <
824 <        for (Future<?> delayed : delayeds) {
825 <            if (effectiveDelayedPolicy) {
826 <                assertNull(delayed.get());
797 >        class Task implements Runnable {
798 >            public void run() {
799 >                try {
800 >                    ran.getAndIncrement();
801 >                    poolBlocked.countDown();
802 >                    await(unblock);
803 >                } catch (Throwable fail) { threadUnexpectedException(fail); }
804              }
805          }
806 <        if (effectivePeriodicPolicy) {
807 <            assertTrue(periodicLatch1.await(LONG_DELAY_MS, MILLISECONDS));
808 <            assertTrue(periodicLatch2.await(LONG_DELAY_MS, MILLISECONDS));
809 <            for (Future<?> periodic : periodics) {
810 <                assertTrue(periodic.cancel(false));
811 <                assertTrue(periodic.isCancelled());
812 <                assertTrue(periodic.isDone());
806 >
807 >        class PeriodicTask extends Task {
808 >            PeriodicTask(int rounds) { this.rounds = rounds; }
809 >            int rounds;
810 >            public void run() {
811 >                if (--rounds == 0) super.run();
812 >                // throw exception to surely terminate this periodic task,
813 >                // but in a separate execution and in a detectable way.
814 >                if (rounds == -1) throw exception;
815              }
816          }
817 +
818 +        Runnable task = new Task();
819 +
820 +        List<Future<?>> immediates = new ArrayList<>();
821 +        List<Future<?>> delayeds   = new ArrayList<>();
822 +        List<Future<?>> periodics  = new ArrayList<>();
823 +
824 +        immediates.add(p.submit(task));
825 +        delayeds.add(p.schedule(task, delay, MILLISECONDS));
826 +        periodics.add(p.scheduleAtFixedRate(
827 +                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
828 +        periodics.add(p.scheduleWithFixedDelay(
829 +                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
830 +
831 +        await(poolBlocked);
832 +
833 +        assertEquals(poolSize, ran.get());
834 +        assertEquals(poolSize, p.getActiveCount());
835 +        assertTrue(q.isEmpty());
836 +
837 +        // Add second wave of tasks.
838 +        immediates.add(p.submit(task));
839 +        delayeds.add(p.schedule(task, effectiveDelayedPolicy ? delay : LONG_DELAY_MS, MILLISECONDS));
840 +        periodics.add(p.scheduleAtFixedRate(
841 +                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
842 +        periodics.add(p.scheduleWithFixedDelay(
843 +                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
844 +
845 +        assertEquals(poolSize, q.size());
846 +        assertEquals(poolSize, ran.get());
847 +
848 +        immediates.forEach(
849 +            f -> assertTrue(((ScheduledFuture)f).getDelay(NANOSECONDS) <= 0L));
850 +
851 +        Stream.of(immediates, delayeds, periodics).flatMap(Collection::stream)
852 +            .forEach(f -> assertFalse(f.isDone()));
853 +
854 +        try { p.shutdown(); } catch (SecurityException ok) { return; }
855 +        assertTrue(p.isShutdown());
856 +        assertTrue(p.isTerminating());
857 +        assertFalse(p.isTerminated());
858 +
859 +        if (rnd.nextBoolean())
860 +            assertThrows(
861 +                RejectedExecutionException.class,
862 +                () -> p.submit(task),
863 +                () -> p.schedule(task, 1, SECONDS),
864 +                () -> p.scheduleAtFixedRate(
865 +                    new PeriodicTask(1), 1, 1, SECONDS),
866 +                () -> p.scheduleWithFixedDelay(
867 +                    new PeriodicTask(2), 1, 1, SECONDS));
868 +
869 +        assertTrue(q.contains(immediates.get(1)));
870 +        assertTrue(!effectiveDelayedPolicy
871 +                   ^ q.contains(delayeds.get(1)));
872 +        assertTrue(!effectivePeriodicPolicy
873 +                   ^ q.containsAll(periodics.subList(2, 4)));
874 +
875 +        immediates.forEach(f -> assertFalse(f.isDone()));
876 +
877 +        assertFalse(delayeds.get(0).isDone());
878 +        if (effectiveDelayedPolicy)
879 +            assertFalse(delayeds.get(1).isDone());
880 +        else
881 +            assertTrue(delayeds.get(1).isCancelled());
882 +
883 +        if (effectivePeriodicPolicy)
884 +            periodics.forEach(
885 +                f -> {
886 +                    assertFalse(f.isDone());
887 +                    if (!periodicTasksContinue) {
888 +                        assertTrue(f.cancel(false));
889 +                        assertTrue(f.isCancelled());
890 +                    }
891 +                });
892 +        else {
893 +            periodics.subList(0, 2).forEach(f -> assertFalse(f.isDone()));
894 +            periodics.subList(2, 4).forEach(f -> assertTrue(f.isCancelled()));
895 +        }
896 +
897 +        unblock.countDown();    // Release all pool threads
898 +
899          assertTrue(p.awaitTermination(LONG_DELAY_MS, MILLISECONDS));
900 +        assertFalse(p.isTerminating());
901          assertTrue(p.isTerminated());
902 <        assertEquals(2 + (effectiveDelayedPolicy ? 1 : 0), ran.get());
903 <    }}
902 >
903 >        assertTrue(q.isEmpty());
904 >
905 >        Stream.of(immediates, delayeds, periodics).flatMap(Collection::stream)
906 >            .forEach(f -> assertTrue(f.isDone()));
907 >
908 >        for (Future<?> f : immediates) assertNull(f.get());
909 >
910 >        assertNull(delayeds.get(0).get());
911 >        if (effectiveDelayedPolicy)
912 >            assertNull(delayeds.get(1).get());
913 >        else
914 >            assertTrue(delayeds.get(1).isCancelled());
915 >
916 >        if (periodicTasksContinue)
917 >            periodics.forEach(
918 >                f -> {
919 >                    try { f.get(); }
920 >                    catch (ExecutionException success) {
921 >                        assertSame(exception, success.getCause());
922 >                    }
923 >                    catch (Throwable fail) { threadUnexpectedException(fail); }
924 >                });
925 >        else
926 >            periodics.forEach(f -> assertTrue(f.isCancelled()));
927 >
928 >        assertEquals(poolSize + 1
929 >                     + (effectiveDelayedPolicy ? 1 : 0)
930 >                     + (periodicTasksContinue ? 2 : 0),
931 >                     ran.get());
932 >    }
933  
934      /**
935       * completed submit of callable returns result
# Line 890 | Line 981 | public class ScheduledExecutorSubclassTe
981      }
982  
983      /**
984 <     * invokeAny(empty collection) throws IAE
984 >     * invokeAny(empty collection) throws IllegalArgumentException
985       */
986      public void testInvokeAny2() throws Exception {
987          final ExecutorService e = new CustomExecutor(2);
# Line 909 | Line 1000 | public class ScheduledExecutorSubclassTe
1000          final CountDownLatch latch = new CountDownLatch(1);
1001          final ExecutorService e = new CustomExecutor(2);
1002          try (PoolCleaner cleaner = cleaner(e)) {
1003 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1003 >            List<Callable<String>> l = new ArrayList<>();
1004              l.add(latchAwaitingStringTask(latch));
1005              l.add(null);
1006              try {
# Line 926 | Line 1017 | public class ScheduledExecutorSubclassTe
1017      public void testInvokeAny4() throws Exception {
1018          final ExecutorService e = new CustomExecutor(2);
1019          try (PoolCleaner cleaner = cleaner(e)) {
1020 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1020 >            List<Callable<String>> l = new ArrayList<>();
1021              l.add(new NPETask());
1022              try {
1023                  e.invokeAny(l);
# Line 943 | Line 1034 | public class ScheduledExecutorSubclassTe
1034      public void testInvokeAny5() throws Exception {
1035          final ExecutorService e = new CustomExecutor(2);
1036          try (PoolCleaner cleaner = cleaner(e)) {
1037 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1037 >            List<Callable<String>> l = new ArrayList<>();
1038              l.add(new StringTask());
1039              l.add(new StringTask());
1040              String result = e.invokeAny(l);
# Line 965 | Line 1056 | public class ScheduledExecutorSubclassTe
1056      }
1057  
1058      /**
1059 <     * invokeAll(empty collection) returns empty collection
1059 >     * invokeAll(empty collection) returns empty list
1060       */
1061      public void testInvokeAll2() throws Exception {
1062          final ExecutorService e = new CustomExecutor(2);
1063 +        final Collection<Callable<String>> emptyCollection
1064 +            = Collections.emptyList();
1065          try (PoolCleaner cleaner = cleaner(e)) {
1066 <            List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>());
1066 >            List<Future<String>> r = e.invokeAll(emptyCollection);
1067              assertTrue(r.isEmpty());
1068          }
1069      }
# Line 981 | Line 1074 | public class ScheduledExecutorSubclassTe
1074      public void testInvokeAll3() throws Exception {
1075          final ExecutorService e = new CustomExecutor(2);
1076          try (PoolCleaner cleaner = cleaner(e)) {
1077 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1077 >            List<Callable<String>> l = new ArrayList<>();
1078              l.add(new StringTask());
1079              l.add(null);
1080              try {
# Line 997 | Line 1090 | public class ScheduledExecutorSubclassTe
1090      public void testInvokeAll4() throws Exception {
1091          final ExecutorService e = new CustomExecutor(2);
1092          try (PoolCleaner cleaner = cleaner(e)) {
1093 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1093 >            List<Callable<String>> l = new ArrayList<>();
1094              l.add(new NPETask());
1095              List<Future<String>> futures = e.invokeAll(l);
1096              assertEquals(1, futures.size());
# Line 1016 | Line 1109 | public class ScheduledExecutorSubclassTe
1109      public void testInvokeAll5() throws Exception {
1110          final ExecutorService e = new CustomExecutor(2);
1111          try (PoolCleaner cleaner = cleaner(e)) {
1112 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1112 >            List<Callable<String>> l = new ArrayList<>();
1113              l.add(new StringTask());
1114              l.add(new StringTask());
1115              List<Future<String>> futures = e.invokeAll(l);
# Line 1033 | Line 1126 | public class ScheduledExecutorSubclassTe
1126          final ExecutorService e = new CustomExecutor(2);
1127          try (PoolCleaner cleaner = cleaner(e)) {
1128              try {
1129 <                e.invokeAny(null, MEDIUM_DELAY_MS, MILLISECONDS);
1129 >                e.invokeAny(null, randomTimeout(), randomTimeUnit());
1130                  shouldThrow();
1131              } catch (NullPointerException success) {}
1132          }
# Line 1045 | Line 1138 | public class ScheduledExecutorSubclassTe
1138      public void testTimedInvokeAnyNullTimeUnit() throws Exception {
1139          final ExecutorService e = new CustomExecutor(2);
1140          try (PoolCleaner cleaner = cleaner(e)) {
1141 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1141 >            List<Callable<String>> l = new ArrayList<>();
1142              l.add(new StringTask());
1143              try {
1144 <                e.invokeAny(l, MEDIUM_DELAY_MS, null);
1144 >                e.invokeAny(l, randomTimeout(), null);
1145                  shouldThrow();
1146              } catch (NullPointerException success) {}
1147          }
1148      }
1149  
1150      /**
1151 <     * timed invokeAny(empty collection) throws IAE
1151 >     * timed invokeAny(empty collection) throws IllegalArgumentException
1152       */
1153      public void testTimedInvokeAny2() throws Exception {
1154          final ExecutorService e = new CustomExecutor(2);
1155 +        final Collection<Callable<String>> emptyCollection
1156 +            = Collections.emptyList();
1157          try (PoolCleaner cleaner = cleaner(e)) {
1158              try {
1159 <                e.invokeAny(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, MILLISECONDS);
1159 >                e.invokeAny(emptyCollection, randomTimeout(), randomTimeUnit());
1160                  shouldThrow();
1161              } catch (IllegalArgumentException success) {}
1162          }
# Line 1074 | Line 1169 | public class ScheduledExecutorSubclassTe
1169          CountDownLatch latch = new CountDownLatch(1);
1170          final ExecutorService e = new CustomExecutor(2);
1171          try (PoolCleaner cleaner = cleaner(e)) {
1172 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1172 >            List<Callable<String>> l = new ArrayList<>();
1173              l.add(latchAwaitingStringTask(latch));
1174              l.add(null);
1175              try {
1176 <                e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1176 >                e.invokeAny(l, randomTimeout(), randomTimeUnit());
1177                  shouldThrow();
1178              } catch (NullPointerException success) {}
1179              latch.countDown();
# Line 1091 | Line 1186 | public class ScheduledExecutorSubclassTe
1186      public void testTimedInvokeAny4() throws Exception {
1187          final ExecutorService e = new CustomExecutor(2);
1188          try (PoolCleaner cleaner = cleaner(e)) {
1189 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1189 >            long startTime = System.nanoTime();
1190 >            List<Callable<String>> l = new ArrayList<>();
1191              l.add(new NPETask());
1192              try {
1193 <                e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1193 >                e.invokeAny(l, LONG_DELAY_MS, MILLISECONDS);
1194                  shouldThrow();
1195              } catch (ExecutionException success) {
1196                  assertTrue(success.getCause() instanceof NullPointerException);
1197              }
1198 +            assertTrue(millisElapsedSince(startTime) < LONG_DELAY_MS);
1199          }
1200      }
1201  
# Line 1108 | Line 1205 | public class ScheduledExecutorSubclassTe
1205      public void testTimedInvokeAny5() throws Exception {
1206          final ExecutorService e = new CustomExecutor(2);
1207          try (PoolCleaner cleaner = cleaner(e)) {
1208 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1208 >            long startTime = System.nanoTime();
1209 >            List<Callable<String>> l = new ArrayList<>();
1210              l.add(new StringTask());
1211              l.add(new StringTask());
1212 <            String result = e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1212 >            String result = e.invokeAny(l, LONG_DELAY_MS, MILLISECONDS);
1213              assertSame(TEST_STRING, result);
1214 +            assertTrue(millisElapsedSince(startTime) < LONG_DELAY_MS);
1215          }
1216      }
1217  
1218      /**
1219 <     * timed invokeAll(null) throws NPE
1219 >     * timed invokeAll(null) throws NullPointerException
1220       */
1221      public void testTimedInvokeAll1() throws Exception {
1222          final ExecutorService e = new CustomExecutor(2);
1223          try (PoolCleaner cleaner = cleaner(e)) {
1224              try {
1225 <                e.invokeAll(null, MEDIUM_DELAY_MS, MILLISECONDS);
1225 >                e.invokeAll(null, randomTimeout(), randomTimeUnit());
1226                  shouldThrow();
1227              } catch (NullPointerException success) {}
1228          }
1229      }
1230  
1231      /**
1232 <     * timed invokeAll(,,null) throws NPE
1232 >     * timed invokeAll(,,null) throws NullPointerException
1233       */
1234      public void testTimedInvokeAllNullTimeUnit() throws Exception {
1235          final ExecutorService e = new CustomExecutor(2);
1236          try (PoolCleaner cleaner = cleaner(e)) {
1237 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1237 >            List<Callable<String>> l = new ArrayList<>();
1238              l.add(new StringTask());
1239              try {
1240 <                e.invokeAll(l, MEDIUM_DELAY_MS, null);
1240 >                e.invokeAll(l, randomTimeout(), null);
1241                  shouldThrow();
1242              } catch (NullPointerException success) {}
1243          }
1244      }
1245  
1246      /**
1247 <     * timed invokeAll(empty collection) returns empty collection
1247 >     * timed invokeAll(empty collection) returns empty list
1248       */
1249      public void testTimedInvokeAll2() throws Exception {
1250          final ExecutorService e = new CustomExecutor(2);
1251 +        final Collection<Callable<String>> emptyCollection
1252 +            = Collections.emptyList();
1253          try (PoolCleaner cleaner = cleaner(e)) {
1254 <            List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, MILLISECONDS);
1254 >            List<Future<String>> r =
1255 >                e.invokeAll(emptyCollection, randomTimeout(), randomTimeUnit());
1256              assertTrue(r.isEmpty());
1257          }
1258      }
# Line 1161 | Line 1263 | public class ScheduledExecutorSubclassTe
1263      public void testTimedInvokeAll3() throws Exception {
1264          final ExecutorService e = new CustomExecutor(2);
1265          try (PoolCleaner cleaner = cleaner(e)) {
1266 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1266 >            List<Callable<String>> l = new ArrayList<>();
1267              l.add(new StringTask());
1268              l.add(null);
1269              try {
1270 <                e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
1270 >                e.invokeAll(l, randomTimeout(), randomTimeUnit());
1271                  shouldThrow();
1272              } catch (NullPointerException success) {}
1273          }
# Line 1176 | Line 1278 | public class ScheduledExecutorSubclassTe
1278       */
1279      public void testTimedInvokeAll4() throws Exception {
1280          final ExecutorService e = new CustomExecutor(2);
1281 +        final Collection<Callable<String>> c = new ArrayList<>();
1282 +        c.add(new NPETask());
1283          try (PoolCleaner cleaner = cleaner(e)) {
1180            List<Callable<String>> l = new ArrayList<Callable<String>>();
1181            l.add(new NPETask());
1284              List<Future<String>> futures =
1285 <                e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
1285 >                e.invokeAll(c, LONG_DELAY_MS, MILLISECONDS);
1286              assertEquals(1, futures.size());
1287              try {
1288                  futures.get(0).get();
# Line 1197 | Line 1299 | public class ScheduledExecutorSubclassTe
1299      public void testTimedInvokeAll5() throws Exception {
1300          final ExecutorService e = new CustomExecutor(2);
1301          try (PoolCleaner cleaner = cleaner(e)) {
1302 <            List<Callable<String>> l = new ArrayList<Callable<String>>();
1302 >            List<Callable<String>> l = new ArrayList<>();
1303              l.add(new StringTask());
1304              l.add(new StringTask());
1305              List<Future<String>> futures =
# Line 1212 | Line 1314 | public class ScheduledExecutorSubclassTe
1314       * timed invokeAll(c) cancels tasks not completed by timeout
1315       */
1316      public void testTimedInvokeAll6() throws Exception {
1317 <        final ExecutorService e = new CustomExecutor(2);
1318 <        try (PoolCleaner cleaner = cleaner(e)) {
1319 <            for (long timeout = timeoutMillis();;) {
1317 >        for (long timeout = timeoutMillis();;) {
1318 >            final CountDownLatch done = new CountDownLatch(1);
1319 >            final Callable<String> waiter = new CheckedCallable<>() {
1320 >                public String realCall() {
1321 >                    try { done.await(LONG_DELAY_MS, MILLISECONDS); }
1322 >                    catch (InterruptedException ok) {}
1323 >                    return "1"; }};
1324 >            final ExecutorService p = new CustomExecutor(2);
1325 >            try (PoolCleaner cleaner = cleaner(p, done)) {
1326                  List<Callable<String>> tasks = new ArrayList<>();
1327                  tasks.add(new StringTask("0"));
1328 <                tasks.add(Executors.callable(new LongPossiblyInterruptedRunnable(), TEST_STRING));
1328 >                tasks.add(waiter);
1329                  tasks.add(new StringTask("2"));
1330                  long startTime = System.nanoTime();
1331                  List<Future<String>> futures =
1332 <                    e.invokeAll(tasks, timeout, MILLISECONDS);
1332 >                    p.invokeAll(tasks, timeout, MILLISECONDS);
1333                  assertEquals(tasks.size(), futures.size());
1334                  assertTrue(millisElapsedSince(startTime) >= timeout);
1335 <                for (Future future : futures)
1335 >                for (Future<?> future : futures)
1336                      assertTrue(future.isDone());
1337                  assertTrue(futures.get(1).isCancelled());
1338                  try {

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines