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

Comparing jsr166/src/test/tck/ScheduledExecutorTest.java (file contents):
Revision 1.89 by jsr166, Tue Mar 28 18:13:10 2017 UTC vs.
Revision 1.96 by jsr166, Mon Jan 8 02:04:15 2018 UTC

# Line 11 | Line 11 | import static java.util.concurrent.TimeU
11   import static java.util.concurrent.TimeUnit.SECONDS;
12  
13   import java.util.ArrayList;
14 + import java.util.Collection;
15 + import java.util.Collections;
16   import java.util.HashSet;
17   import java.util.List;
18   import java.util.concurrent.BlockingQueue;
# Line 73 | Line 75 | public class ScheduledExecutorTest exten
75              Future f = p.schedule(task, timeoutMillis(), MILLISECONDS);
76              assertSame(Boolean.TRUE, f.get());
77              assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
78 <            assertTrue(done.await(0L, MILLISECONDS));
78 >            assertEquals(0L, done.getCount());
79          }
80      }
81  
# Line 222 | Line 224 | public class ScheduledExecutorTest exten
224      }
225  
226      /**
227 <     * execute(null) throws NPE
227 >     * Submitting null tasks throws NullPointerException
228       */
229 <    public void testExecuteNull() throws InterruptedException {
229 >    public void testNullTaskSubmission() {
230          final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
231          try (PoolCleaner cleaner = cleaner(p)) {
232 <            try {
231 <                p.execute(null);
232 <                shouldThrow();
233 <            } catch (NullPointerException success) {}
232 >            assertNullTaskSubmissionThrowsNullPointerException(p);
233          }
234      }
235  
236      /**
237 <     * schedule(null) throws NPE
237 >     * Submitted tasks are rejected when shutdown
238       */
239 <    public void testScheduleNull() throws InterruptedException {
239 >    public void testSubmittedTasksRejectedWhenShutdown() throws InterruptedException {
240          final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
241 <        try (PoolCleaner cleaner = cleaner(p)) {
242 <            try {
243 <                TrackedCallable callable = null;
244 <                Future f = p.schedule(callable, SHORT_DELAY_MS, MILLISECONDS);
245 <                shouldThrow();
246 <            } catch (NullPointerException success) {}
247 <        }
248 <    }
241 >        final ThreadLocalRandom rnd = ThreadLocalRandom.current();
242 >        final CountDownLatch threadsStarted = new CountDownLatch(p.getCorePoolSize());
243 >        final CountDownLatch done = new CountDownLatch(1);
244 >        final Runnable r = () -> {
245 >            threadsStarted.countDown();
246 >            for (;;) {
247 >                try {
248 >                    done.await();
249 >                    return;
250 >                } catch (InterruptedException shutdownNowDeliberatelyIgnored) {}
251 >            }};
252 >        final Callable<Boolean> c = () -> {
253 >            threadsStarted.countDown();
254 >            for (;;) {
255 >                try {
256 >                    done.await();
257 >                    return Boolean.TRUE;
258 >                } catch (InterruptedException shutdownNowDeliberatelyIgnored) {}
259 >            }};
260  
261 <    /**
262 <     * execute throws RejectedExecutionException if shutdown
263 <     */
264 <    public void testSchedule1_RejectedExecutionException() throws InterruptedException {
265 <        final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
266 <        try (PoolCleaner cleaner = cleaner(p)) {
267 <            try {
268 <                p.shutdown();
269 <                p.schedule(new NoOpRunnable(),
260 <                           MEDIUM_DELAY_MS, MILLISECONDS);
261 <                shouldThrow();
262 <            } catch (RejectedExecutionException success) {
263 <            } catch (SecurityException ok) {}
264 <        }
265 <    }
261 >        try (PoolCleaner cleaner = cleaner(p, done)) {
262 >            for (int i = p.getCorePoolSize(); i--> 0; ) {
263 >                switch (rnd.nextInt(4)) {
264 >                case 0: p.execute(r); break;
265 >                case 1: assertFalse(p.submit(r).isDone()); break;
266 >                case 2: assertFalse(p.submit(r, Boolean.TRUE).isDone()); break;
267 >                case 3: assertFalse(p.submit(c).isDone()); break;
268 >                }
269 >            }
270  
271 <    /**
272 <     * schedule throws RejectedExecutionException if shutdown
269 <     */
270 <    public void testSchedule2_RejectedExecutionException() throws InterruptedException {
271 <        final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
272 <        try (PoolCleaner cleaner = cleaner(p)) {
273 <            try {
274 <                p.shutdown();
275 <                p.schedule(new NoOpCallable(),
276 <                           MEDIUM_DELAY_MS, MILLISECONDS);
277 <                shouldThrow();
278 <            } catch (RejectedExecutionException success) {
279 <            } catch (SecurityException ok) {}
280 <        }
281 <    }
271 >            // ScheduledThreadPoolExecutor has an unbounded queue, so never saturated.
272 >            await(threadsStarted);
273  
274 <    /**
275 <     * schedule callable throws RejectedExecutionException if shutdown
276 <     */
286 <    public void testSchedule3_RejectedExecutionException() throws InterruptedException {
287 <        final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
288 <        try (PoolCleaner cleaner = cleaner(p)) {
289 <            try {
274 >            if (rnd.nextBoolean())
275 >                p.shutdownNow();
276 >            else
277                  p.shutdown();
278 <                p.schedule(new NoOpCallable(),
279 <                           MEDIUM_DELAY_MS, MILLISECONDS);
280 <                shouldThrow();
294 <            } catch (RejectedExecutionException success) {
295 <            } catch (SecurityException ok) {}
296 <        }
297 <    }
278 >            // Pool is shutdown, but not yet terminated
279 >            assertTaskSubmissionsAreRejected(p);
280 >            assertFalse(p.isTerminated());
281  
282 <    /**
283 <     * scheduleAtFixedRate throws RejectedExecutionException if shutdown
301 <     */
302 <    public void testScheduleAtFixedRate1_RejectedExecutionException() throws InterruptedException {
303 <        final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
304 <        try (PoolCleaner cleaner = cleaner(p)) {
305 <            try {
306 <                p.shutdown();
307 <                p.scheduleAtFixedRate(new NoOpRunnable(),
308 <                                      MEDIUM_DELAY_MS, MEDIUM_DELAY_MS, MILLISECONDS);
309 <                shouldThrow();
310 <            } catch (RejectedExecutionException success) {
311 <            } catch (SecurityException ok) {}
312 <        }
313 <    }
282 >            done.countDown();   // release blocking tasks
283 >            assertTrue(p.awaitTermination(LONG_DELAY_MS, MILLISECONDS));
284  
285 <    /**
316 <     * scheduleWithFixedDelay throws RejectedExecutionException if shutdown
317 <     */
318 <    public void testScheduleWithFixedDelay1_RejectedExecutionException() throws InterruptedException {
319 <        final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
320 <        try (PoolCleaner cleaner = cleaner(p)) {
321 <            try {
322 <                p.shutdown();
323 <                p.scheduleWithFixedDelay(new NoOpRunnable(),
324 <                                         MEDIUM_DELAY_MS, MEDIUM_DELAY_MS, MILLISECONDS);
325 <                shouldThrow();
326 <            } catch (RejectedExecutionException success) {
327 <            } catch (SecurityException ok) {}
285 >            assertTaskSubmissionsAreRejected(p);
286          }
287 +        assertEquals(p.getCorePoolSize(), p.getCompletedTaskCount());
288      }
289  
290      /**
# Line 745 | Line 704 | public class ScheduledExecutorTest exten
704       * - setExecuteExistingDelayedTasksAfterShutdownPolicy
705       * - setContinueExistingPeriodicTasksAfterShutdownPolicy
706       */
707 +    @SuppressWarnings("FutureReturnValueIgnored")
708      public void testShutdown_cancellation() throws Exception {
709 <        final int poolSize = 6;
709 >        final int poolSize = 4;
710          final ScheduledThreadPoolExecutor p
711              = new ScheduledThreadPoolExecutor(poolSize);
712          final BlockingQueue<Runnable> q = p.getQueue();
713          final ThreadLocalRandom rnd = ThreadLocalRandom.current();
714 +        final long delay = rnd.nextInt(2);
715 +        final int rounds = rnd.nextInt(1, 3);
716          final boolean effectiveDelayedPolicy;
717          final boolean effectivePeriodicPolicy;
718          final boolean effectiveRemovePolicy;
# Line 779 | Line 741 | public class ScheduledExecutorTest exten
741          assertEquals(effectiveRemovePolicy,
742                       p.getRemoveOnCancelPolicy());
743  
744 <        // System.err.println("effectiveDelayedPolicy="+effectiveDelayedPolicy);
783 <        // System.err.println("effectivePeriodicPolicy="+effectivePeriodicPolicy);
784 <        // System.err.println("effectiveRemovePolicy="+effectiveRemovePolicy);
744 >        final boolean periodicTasksContinue = effectivePeriodicPolicy && rnd.nextBoolean();
745  
746          // Strategy: Wedge the pool with one wave of "blocker" tasks,
747 <        // then add a second wave that waits in the queue.
747 >        // then add a second wave that waits in the queue until unblocked.
748          final AtomicInteger ran = new AtomicInteger(0);
749          final CountDownLatch poolBlocked = new CountDownLatch(poolSize);
750          final CountDownLatch unblock = new CountDownLatch(1);
751 +        final RuntimeException exception = new RuntimeException();
752  
753 <        class Task extends CheckedRunnable {
754 <            public void realRun() throws InterruptedException {
755 <                ran.getAndIncrement();
756 <                poolBlocked.countDown();
757 <                await(unblock);
753 >        class Task implements Runnable {
754 >            public void run() {
755 >                try {
756 >                    ran.getAndIncrement();
757 >                    poolBlocked.countDown();
758 >                    await(unblock);
759 >                } catch (Throwable fail) { threadUnexpectedException(fail); }
760              }
761          }
762  
763          class PeriodicTask extends Task {
764              PeriodicTask(int rounds) { this.rounds = rounds; }
765              int rounds;
766 <            public void realRun() throws InterruptedException {
767 <                if (--rounds == 0) super.realRun();
766 >            public void run() {
767 >                if (--rounds == 0) super.run();
768 >                // throw exception to surely terminate this periodic task,
769 >                // but in a separate execution and in a detectable way.
770 >                if (rounds == -1) throw exception;
771              }
772          }
773  
# Line 812 | Line 778 | public class ScheduledExecutorTest exten
778          List<Future<?>> periodics  = new ArrayList<>();
779  
780          immediates.add(p.submit(task));
781 <        delayeds.add(p.schedule(task, 1, MILLISECONDS));
782 <        for (int rounds : new int[] { 1, 2 }) {
783 <            periodics.add(p.scheduleAtFixedRate(
784 <                              new PeriodicTask(rounds), 1, 1, MILLISECONDS));
785 <            periodics.add(p.scheduleWithFixedDelay(
820 <                              new PeriodicTask(rounds), 1, 1, MILLISECONDS));
821 <        }
781 >        delayeds.add(p.schedule(task, delay, MILLISECONDS));
782 >        periodics.add(p.scheduleAtFixedRate(
783 >                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
784 >        periodics.add(p.scheduleWithFixedDelay(
785 >                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
786  
787          await(poolBlocked);
788  
789          assertEquals(poolSize, ran.get());
790 +        assertEquals(poolSize, p.getActiveCount());
791          assertTrue(q.isEmpty());
792  
793          // Add second wave of tasks.
794          immediates.add(p.submit(task));
795 <        long delay_ms = effectiveDelayedPolicy ? 1 : LONG_DELAY_MS;
796 <        delayeds.add(p.schedule(task, delay_ms, MILLISECONDS));
797 <        for (int rounds : new int[] { 1, 2 }) {
798 <            periodics.add(p.scheduleAtFixedRate(
799 <                              new PeriodicTask(rounds), 1, 1, MILLISECONDS));
835 <            periodics.add(p.scheduleWithFixedDelay(
836 <                              new PeriodicTask(rounds), 1, 1, MILLISECONDS));
837 <        }
795 >        delayeds.add(p.schedule(task, effectiveDelayedPolicy ? delay : LONG_DELAY_MS, MILLISECONDS));
796 >        periodics.add(p.scheduleAtFixedRate(
797 >                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
798 >        periodics.add(p.scheduleWithFixedDelay(
799 >                          new PeriodicTask(rounds), delay, 1, MILLISECONDS));
800  
801          assertEquals(poolSize, q.size());
802          assertEquals(poolSize, ran.get());
# Line 842 | Line 804 | public class ScheduledExecutorTest exten
804          immediates.forEach(
805              f -> assertTrue(((ScheduledFuture)f).getDelay(NANOSECONDS) <= 0L));
806  
807 <        Stream.of(immediates, delayeds, periodics).flatMap(c -> c.stream())
807 >        Stream.of(immediates, delayeds, periodics).flatMap(Collection::stream)
808              .forEach(f -> assertFalse(f.isDone()));
809  
810          try { p.shutdown(); } catch (SecurityException ok) { return; }
# Line 864 | Line 826 | public class ScheduledExecutorTest exten
826          assertTrue(!effectiveDelayedPolicy
827                     ^ q.contains(delayeds.get(1)));
828          assertTrue(!effectivePeriodicPolicy
829 <                   ^ q.containsAll(periodics.subList(4, 8)));
829 >                   ^ q.containsAll(periodics.subList(2, 4)));
830  
831          immediates.forEach(f -> assertFalse(f.isDone()));
832  
# Line 874 | Line 836 | public class ScheduledExecutorTest exten
836          else
837              assertTrue(delayeds.get(1).isCancelled());
838  
839 <        if (testImplementationDetails) {
840 <            if (effectivePeriodicPolicy)
841 <                // TODO: ensure periodic tasks continue executing
842 <                periodics.forEach(
843 <                    f -> {
882 <                        assertFalse(f.isDone());
839 >        if (effectivePeriodicPolicy)
840 >            periodics.forEach(
841 >                f -> {
842 >                    assertFalse(f.isDone());
843 >                    if (!periodicTasksContinue) {
844                          assertTrue(f.cancel(false));
845 <                    });
846 <            else {
847 <                periodics.subList(0, 4).forEach(f -> assertFalse(f.isDone()));
848 <                periodics.subList(4, 8).forEach(f -> assertTrue(f.isCancelled()));
849 <            }
845 >                        assertTrue(f.isCancelled());
846 >                    }
847 >                });
848 >        else {
849 >            periodics.subList(0, 2).forEach(f -> assertFalse(f.isDone()));
850 >            periodics.subList(2, 4).forEach(f -> assertTrue(f.isCancelled()));
851          }
852  
853          unblock.countDown();    // Release all pool threads
# Line 896 | Line 858 | public class ScheduledExecutorTest exten
858  
859          assertTrue(q.isEmpty());
860  
861 +        Stream.of(immediates, delayeds, periodics).flatMap(Collection::stream)
862 +            .forEach(f -> assertTrue(f.isDone()));
863 +
864          for (Future<?> f : immediates) assertNull(f.get());
865  
866          assertNull(delayeds.get(0).get());
# Line 904 | Line 869 | public class ScheduledExecutorTest exten
869          else
870              assertTrue(delayeds.get(1).isCancelled());
871  
872 <        periodics.forEach(f -> assertTrue(f.isDone()));
873 <        periodics.forEach(f -> assertTrue(f.isCancelled()));
872 >        if (periodicTasksContinue)
873 >            periodics.forEach(
874 >                f -> {
875 >                    try { f.get(); }
876 >                    catch (ExecutionException success) {
877 >                        assertSame(exception, success.getCause());
878 >                    }
879 >                    catch (Throwable fail) { threadUnexpectedException(fail); }
880 >                });
881 >        else
882 >            periodics.forEach(f -> assertTrue(f.isCancelled()));
883  
884 <        assertEquals(poolSize + 1 + (effectiveDelayedPolicy ? 1 : 0), ran.get());
884 >        assertEquals(poolSize + 1
885 >                     + (effectiveDelayedPolicy ? 1 : 0)
886 >                     + (periodicTasksContinue ? 2 : 0),
887 >                     ran.get());
888      }
889  
890      /**
# Line 947 | Line 924 | public class ScheduledExecutorTest exten
924      }
925  
926      /**
927 <     * invokeAny(null) throws NPE
927 >     * invokeAny(null) throws NullPointerException
928       */
929      public void testInvokeAny1() throws Exception {
930          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
# Line 960 | Line 937 | public class ScheduledExecutorTest exten
937      }
938  
939      /**
940 <     * invokeAny(empty collection) throws IAE
940 >     * invokeAny(empty collection) throws IllegalArgumentException
941       */
942      public void testInvokeAny2() throws Exception {
943          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
# Line 973 | Line 950 | public class ScheduledExecutorTest exten
950      }
951  
952      /**
953 <     * invokeAny(c) throws NPE if c has null elements
953 >     * invokeAny(c) throws NullPointerException if c has null elements
954       */
955      public void testInvokeAny3() throws Exception {
956          CountDownLatch latch = new CountDownLatch(1);
# Line 1035 | Line 1012 | public class ScheduledExecutorTest exten
1012      }
1013  
1014      /**
1015 <     * invokeAll(empty collection) returns empty collection
1015 >     * invokeAll(empty collection) returns empty list
1016       */
1017      public void testInvokeAll2() throws Exception {
1018          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
1019 +        final Collection<Callable<String>> emptyCollection
1020 +            = Collections.emptyList();
1021          try (PoolCleaner cleaner = cleaner(e)) {
1022 <            List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>());
1022 >            List<Future<String>> r = e.invokeAll(emptyCollection);
1023              assertTrue(r.isEmpty());
1024          }
1025      }
# Line 1103 | Line 1082 | public class ScheduledExecutorTest exten
1082          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
1083          try (PoolCleaner cleaner = cleaner(e)) {
1084              try {
1085 <                e.invokeAny(null, MEDIUM_DELAY_MS, MILLISECONDS);
1085 >                e.invokeAny(null, randomTimeout(), randomTimeUnit());
1086                  shouldThrow();
1087              } catch (NullPointerException success) {}
1088          }
1089      }
1090  
1091      /**
1092 <     * timed invokeAny(,,null) throws NPE
1092 >     * timed invokeAny(,,null) throws NullPointerException
1093       */
1094      public void testTimedInvokeAnyNullTimeUnit() throws Exception {
1095          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
# Line 1118 | Line 1097 | public class ScheduledExecutorTest exten
1097              List<Callable<String>> l = new ArrayList<>();
1098              l.add(new StringTask());
1099              try {
1100 <                e.invokeAny(l, MEDIUM_DELAY_MS, null);
1100 >                e.invokeAny(l, randomTimeout(), null);
1101                  shouldThrow();
1102              } catch (NullPointerException success) {}
1103          }
1104      }
1105  
1106      /**
1107 <     * timed invokeAny(empty collection) throws IAE
1107 >     * timed invokeAny(empty collection) throws IllegalArgumentException
1108       */
1109      public void testTimedInvokeAny2() throws Exception {
1110          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
1111 +        final Collection<Callable<String>> emptyCollection
1112 +            = Collections.emptyList();
1113          try (PoolCleaner cleaner = cleaner(e)) {
1114              try {
1115 <                e.invokeAny(new ArrayList<Callable<String>>(), MEDIUM_DELAY_MS, MILLISECONDS);
1115 >                e.invokeAny(emptyCollection, randomTimeout(), randomTimeUnit());
1116                  shouldThrow();
1117              } catch (IllegalArgumentException success) {}
1118          }
# Line 1148 | Line 1129 | public class ScheduledExecutorTest exten
1129              l.add(latchAwaitingStringTask(latch));
1130              l.add(null);
1131              try {
1132 <                e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
1132 >                e.invokeAny(l, randomTimeout(), randomTimeUnit());
1133                  shouldThrow();
1134              } catch (NullPointerException success) {}
1135              latch.countDown();
# Line 1197 | Line 1178 | public class ScheduledExecutorTest exten
1178          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
1179          try (PoolCleaner cleaner = cleaner(e)) {
1180              try {
1181 <                e.invokeAll(null, MEDIUM_DELAY_MS, MILLISECONDS);
1181 >                e.invokeAll(null, randomTimeout(), randomTimeUnit());
1182                  shouldThrow();
1183              } catch (NullPointerException success) {}
1184          }
# Line 1212 | Line 1193 | public class ScheduledExecutorTest exten
1193              List<Callable<String>> l = new ArrayList<>();
1194              l.add(new StringTask());
1195              try {
1196 <                e.invokeAll(l, MEDIUM_DELAY_MS, null);
1196 >                e.invokeAll(l, randomTimeout(), null);
1197                  shouldThrow();
1198              } catch (NullPointerException success) {}
1199          }
1200      }
1201  
1202      /**
1203 <     * timed invokeAll(empty collection) returns empty collection
1203 >     * timed invokeAll(empty collection) returns empty list
1204       */
1205      public void testTimedInvokeAll2() throws Exception {
1206          final ExecutorService e = new ScheduledThreadPoolExecutor(2);
1207 +        final Collection<Callable<String>> emptyCollection
1208 +            = Collections.emptyList();
1209          try (PoolCleaner cleaner = cleaner(e)) {
1210 <            List<Future<String>> r = e.invokeAll(new ArrayList<Callable<String>>(),
1211 <                                                 MEDIUM_DELAY_MS, MILLISECONDS);
1210 >            List<Future<String>> r =
1211 >                e.invokeAll(emptyCollection, randomTimeout(), randomTimeUnit());
1212              assertTrue(r.isEmpty());
1213          }
1214      }
# Line 1240 | Line 1223 | public class ScheduledExecutorTest exten
1223              l.add(new StringTask());
1224              l.add(null);
1225              try {
1226 <                e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
1226 >                e.invokeAll(l, randomTimeout(), randomTimeUnit());
1227                  shouldThrow();
1228              } catch (NullPointerException success) {}
1229          }
# Line 1326 | Line 1309 | public class ScheduledExecutorTest exten
1309       * one-shot task from executing.
1310       * https://bugs.openjdk.java.net/browse/JDK-8051859
1311       */
1312 +    @SuppressWarnings("FutureReturnValueIgnored")
1313      public void testScheduleWithFixedDelay_overflow() throws Exception {
1314          final CountDownLatch delayedDone = new CountDownLatch(1);
1315          final CountDownLatch immediateDone = new CountDownLatch(1);
1316          final ScheduledThreadPoolExecutor p = new ScheduledThreadPoolExecutor(1);
1317          try (PoolCleaner cleaner = cleaner(p)) {
1318 <            final Runnable immediate = new Runnable() { public void run() {
1335 <                immediateDone.countDown();
1336 <            }};
1337 <            final Runnable delayed = new Runnable() { public void run() {
1318 >            final Runnable delayed = () -> {
1319                  delayedDone.countDown();
1320 <                p.submit(immediate);
1321 <            }};
1320 >                p.submit(() -> immediateDone.countDown());
1321 >            };
1322              p.scheduleWithFixedDelay(delayed, 0L, Long.MAX_VALUE, SECONDS);
1323              await(delayedDone);
1324              await(immediateDone);

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines