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

Comparing jsr166/src/test/tck/ForkJoinPoolTest.java (file contents):
Revision 1.8 by jsr166, Tue Aug 4 21:10:19 2009 UTC vs.
Revision 1.65 by jsr166, Tue Oct 6 21:22:54 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   */
6  
7
8 import junit.framework.*;
9 import java.util.*;
10 import java.util.concurrent.*;
7   import static java.util.concurrent.TimeUnit.MILLISECONDS;
8 < import java.util.concurrent.locks.*;
9 < import java.security.*;
8 > import static java.util.concurrent.TimeUnit.NANOSECONDS;
9 >
10 > import java.security.PrivilegedAction;
11 > import java.security.PrivilegedExceptionAction;
12 > import java.util.ArrayList;
13 > import java.util.Collection;
14 > import java.util.List;
15 > import java.util.concurrent.Callable;
16 > import java.util.concurrent.CountDownLatch;
17 > import java.util.concurrent.ExecutionException;
18 > import java.util.concurrent.Executors;
19 > import java.util.concurrent.ExecutorService;
20 > import java.util.concurrent.ForkJoinPool;
21 > import java.util.concurrent.ForkJoinTask;
22 > import java.util.concurrent.ForkJoinWorkerThread;
23 > import java.util.concurrent.Future;
24 > import java.util.concurrent.RecursiveTask;
25 > import java.util.concurrent.RejectedExecutionException;
26 > import java.util.concurrent.atomic.AtomicBoolean;
27 > import java.util.concurrent.locks.ReentrantLock;
28 >
29 > import junit.framework.AssertionFailedError;
30 > import junit.framework.Test;
31 > import junit.framework.TestSuite;
32  
33   public class ForkJoinPoolTest extends JSR166TestCase {
34      public static void main(String[] args) {
35 <        junit.textui.TestRunner.run (suite());
35 >        main(suite(), args);
36      }
37 +
38      public static Test suite() {
39          return new TestSuite(ForkJoinPoolTest.class);
40      }
41  
42 <    /**
42 >    /*
43       * Testing coverage notes:
44       *
45       * 1. shutdown and related methods are tested via super.joinPool.
# Line 39 | Line 58 | public class ForkJoinPoolTest extends JS
58      // Some classes to test extension and factory methods
59  
60      static class MyHandler implements Thread.UncaughtExceptionHandler {
61 <        int catches = 0;
61 >        volatile int catches = 0;
62          public void uncaughtException(Thread t, Throwable e) {
63              ++catches;
64          }
# Line 48 | Line 67 | public class ForkJoinPoolTest extends JS
67      // to test handlers
68      static class FailingFJWSubclass extends ForkJoinWorkerThread {
69          public FailingFJWSubclass(ForkJoinPool p) { super(p) ; }
70 <        protected void onStart() { throw new Error(); }
70 >        protected void onStart() { super.onStart(); throw new Error(); }
71      }
72  
73      static class FailingThreadFactory
74              implements ForkJoinPool.ForkJoinWorkerThreadFactory {
75 <        int calls = 0;
75 >        volatile int calls = 0;
76          public ForkJoinWorkerThread newThread(ForkJoinPool p) {
77              if (++calls > 1) return null;
78              return new FailingFJWSubclass(p);
# Line 88 | Line 107 | public class ForkJoinPoolTest extends JS
107      static final class FibTask extends RecursiveTask<Integer> {
108          final int number;
109          FibTask(int n) { number = n; }
110 <        public Integer compute() {
110 >        protected Integer compute() {
111              int n = number;
112              if (n <= 1)
113                  return n;
# Line 116 | Line 135 | public class ForkJoinPoolTest extends JS
135              this.locker = locker;
136              this.lock = lock;
137          }
138 <        public Integer compute() {
138 >        protected Integer compute() {
139              int n;
140              LockingFibTask f1 = null;
141              LockingFibTask f2 = null;
# Line 142 | Line 161 | public class ForkJoinPoolTest extends JS
161       * tasks, and quiescent running state.
162       */
163      public void testDefaultInitialState() {
164 <        ForkJoinPool p = null;
165 <        try {
166 <            p = new ForkJoinPool(1);
167 <            assertTrue(p.getFactory() ==
149 <                       ForkJoinPool.defaultForkJoinWorkerThreadFactory);
150 <            assertTrue(p.isQuiescent());
151 <            assertTrue(p.getMaintainsParallelism());
164 >        ForkJoinPool p = new ForkJoinPool(1);
165 >        try (PoolCleaner cleaner = cleaner(p)) {
166 >            assertSame(ForkJoinPool.defaultForkJoinWorkerThreadFactory,
167 >                       p.getFactory());
168              assertFalse(p.getAsyncMode());
169 <            assertTrue(p.getActiveThreadCount() == 0);
170 <            assertTrue(p.getStealCount() == 0);
171 <            assertTrue(p.getQueuedTaskCount() == 0);
172 <            assertTrue(p.getQueuedSubmissionCount() == 0);
169 >            assertEquals(0, p.getActiveThreadCount());
170 >            assertEquals(0, p.getStealCount());
171 >            assertEquals(0, p.getQueuedTaskCount());
172 >            assertEquals(0, p.getQueuedSubmissionCount());
173              assertFalse(p.hasQueuedSubmissions());
174              assertFalse(p.isShutdown());
175              assertFalse(p.isTerminating());
176              assertFalse(p.isTerminated());
161        } finally {
162            joinPool(p);
177          }
178      }
179  
# Line 170 | Line 184 | public class ForkJoinPoolTest extends JS
184          try {
185              new ForkJoinPool(-1);
186              shouldThrow();
187 <        }
174 <        catch (IllegalArgumentException success) {}
187 >        } catch (IllegalArgumentException success) {}
188      }
189  
190      /**
# Line 179 | Line 192 | public class ForkJoinPoolTest extends JS
192       */
193      public void testConstructor2() {
194          try {
195 <            new ForkJoinPool(1, null);
195 >            new ForkJoinPool(1, null, null, false);
196              shouldThrow();
197 <        } catch (NullPointerException success) {
185 <        }
197 >        } catch (NullPointerException success) {}
198      }
199  
188
200      /**
201       * getParallelism returns size set in constructor
202       */
203      public void testGetParallelism() {
204 <        ForkJoinPool p = null;
205 <        try {
206 <            p = new ForkJoinPool(1);
196 <            assertTrue(p.getParallelism() == 1);
197 <        } finally {
198 <            joinPool(p);
199 <        }
200 <    }
201 <
202 <    /**
203 <     * setParallelism changes reported parallelism level.
204 <     */
205 <    public void testSetParallelism() {
206 <        ForkJoinPool p = null;
207 <        try {
208 <            p = new ForkJoinPool(1);
209 <            assertTrue(p.getParallelism() == 1);
210 <            p.setParallelism(2);
211 <            assertTrue(p.getParallelism() == 2);
212 <        } finally {
213 <            joinPool(p);
214 <        }
215 <    }
216 <
217 <    /**
218 <     * setParallelism with argument <= 0 throws exception
219 <     */
220 <    public void testSetParallelism2() {
221 <        ForkJoinPool p = null;
222 <        try {
223 <            p = new ForkJoinPool(1);
224 <            assertTrue(p.getParallelism() == 1);
225 <            p.setParallelism(-2);
226 <            shouldThrow();
227 <        } catch (IllegalArgumentException success) {
228 <        } finally {
229 <            joinPool(p);
204 >        ForkJoinPool p = new ForkJoinPool(1);
205 >        try (PoolCleaner cleaner = cleaner(p)) {
206 >            assertEquals(1, p.getParallelism());
207          }
208      }
209  
# Line 234 | Line 211 | public class ForkJoinPoolTest extends JS
211       * getPoolSize returns number of started workers.
212       */
213      public void testGetPoolSize() {
214 <        ForkJoinPool p = null;
215 <        try {
216 <            p = new ForkJoinPool(1);
240 <            assertTrue(p.getPoolSize() == 0);
214 >        ForkJoinPool p = new ForkJoinPool(1);
215 >        try (PoolCleaner cleaner = cleaner(p)) {
216 >            assertEquals(0, p.getActiveThreadCount());
217              Future<String> future = p.submit(new StringTask());
218 <            assertTrue(p.getPoolSize() == 1);
243 <
244 <        } finally {
245 <            joinPool(p);
246 <        }
247 <    }
248 <
249 <    /**
250 <     * setMaximumPoolSize changes size reported by getMaximumPoolSize.
251 <     */
252 <    public void testSetMaximumPoolSize() {
253 <        ForkJoinPool p = null;
254 <        try {
255 <            p = new ForkJoinPool(1);
256 <            p.setMaximumPoolSize(2);
257 <            assertTrue(p.getMaximumPoolSize() == 2);
258 <        } finally {
259 <            joinPool(p);
260 <        }
261 <    }
262 <
263 <    /**
264 <     * setMaximumPoolSize with argument <= 0 throws exception
265 <     */
266 <    public void testSetMaximumPoolSize2() {
267 <        ForkJoinPool p = null;
268 <        try {
269 <            p = new ForkJoinPool(1);
270 <            p.setMaximumPoolSize(-2);
271 <            shouldThrow();
272 <        } catch (IllegalArgumentException success) {
273 <        } finally {
274 <            joinPool(p);
275 <        }
276 <    }
277 <
278 <    /**
279 <     * setMaintainsParallelism changes policy reported by
280 <     * getMaintainsParallelism.
281 <     */
282 <    public void testSetMaintainsParallelism() {
283 <        ForkJoinPool p = null;
284 <        try {
285 <            p = new ForkJoinPool(1);
286 <            p.setMaintainsParallelism(false);
287 <            assertFalse(p.getMaintainsParallelism());
288 <        } finally {
289 <            joinPool(p);
218 >            assertEquals(1, p.getPoolSize());
219          }
220      }
221  
222      /**
223 <     * setAsyncMode changes policy reported by
295 <     * getAsyncMode.
223 >     * awaitTermination on a non-shutdown pool times out
224       */
225 <    public void testSetAsyncMode() {
226 <        ForkJoinPool p = null;
227 <        try {
228 <            p = new ForkJoinPool(1);
229 <            p.setAsyncMode(true);
230 <            assertTrue(p.getAsyncMode());
231 <        } finally {
232 <            joinPool(p);
225 >    public void testAwaitTermination_timesOut() throws InterruptedException {
226 >        ForkJoinPool p = new ForkJoinPool(1);
227 >        try (PoolCleaner cleaner = cleaner(p)) {
228 >            assertFalse(p.isTerminated());
229 >            assertFalse(p.awaitTermination(Long.MIN_VALUE, NANOSECONDS));
230 >            assertFalse(p.awaitTermination(Long.MIN_VALUE, MILLISECONDS));
231 >            assertFalse(p.awaitTermination(-1L, NANOSECONDS));
232 >            assertFalse(p.awaitTermination(-1L, MILLISECONDS));
233 >            assertFalse(p.awaitTermination(0L, NANOSECONDS));
234 >            assertFalse(p.awaitTermination(0L, MILLISECONDS));
235 >            long timeoutNanos = 999999L;
236 >            long startTime = System.nanoTime();
237 >            assertFalse(p.awaitTermination(timeoutNanos, NANOSECONDS));
238 >            assertTrue(System.nanoTime() - startTime >= timeoutNanos);
239 >            assertFalse(p.isTerminated());
240 >            startTime = System.nanoTime();
241 >            long timeoutMillis = timeoutMillis();
242 >            assertFalse(p.awaitTermination(timeoutMillis, MILLISECONDS));
243 >            assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
244 >            assertFalse(p.isTerminated());
245 >            p.shutdown();
246 >            assertTrue(p.awaitTermination(LONG_DELAY_MS, MILLISECONDS));
247 >            assertTrue(p.isTerminated());
248          }
249      }
250  
# Line 312 | Line 255 | public class ForkJoinPoolTest extends JS
255       * performs its defined action
256       */
257      public void testSetUncaughtExceptionHandler() throws InterruptedException {
258 <        ForkJoinPool p = null;
259 <        try {
260 <            p = new ForkJoinPool(1, new FailingThreadFactory());
261 <            MyHandler eh = new MyHandler();
262 <            p.setUncaughtExceptionHandler(eh);
263 <            assertEquals(eh, p.getUncaughtExceptionHandler());
264 <            p.execute(new FailingTask());
265 <            Thread.sleep(MEDIUM_DELAY_MS);
266 <            assertTrue(eh.catches > 0);
267 <        } finally {
268 <            joinPool(p);
258 >        final CountDownLatch uehInvoked = new CountDownLatch(1);
259 >        final Thread.UncaughtExceptionHandler ueh =
260 >            new Thread.UncaughtExceptionHandler() {
261 >                public void uncaughtException(Thread t, Throwable e) {
262 >                    uehInvoked.countDown();
263 >                }};
264 >        ForkJoinPool p = new ForkJoinPool(1, new FailingThreadFactory(),
265 >                                          ueh, false);
266 >        try (PoolCleaner cleaner = cleaner(p)) {
267 >            assertSame(ueh, p.getUncaughtExceptionHandler());
268 >            try {
269 >                try {
270 >                    p.execute(new FibTask(8));
271 >                    await(uehInvoked);
272 >                } catch (RejectedExecutionException ok) {}
273 >            } finally {
274 >                p.shutdownNow(); // failure might have prevented processing task
275 >            }
276          }
277      }
278  
279      /**
280 <     * setUncaughtExceptionHandler of null removes handler
281 <     */
282 <    public void testSetUncaughtExceptionHandler2() {
283 <        ForkJoinPool p = null;
284 <        try {
285 <            p = new ForkJoinPool(1);
286 <            p.setUncaughtExceptionHandler(null);
287 <            assertNull(p.getUncaughtExceptionHandler());
288 <        } finally {
289 <            joinPool(p);
290 <        }
291 <    }
292 <
280 >     * After invoking a single task, isQuiescent eventually becomes
281 >     * true, at which time queues are empty, threads are not active,
282 >     * the task has completed successfully, and construction
283 >     * parameters continue to hold
284 >     */
285 >    public void testIsQuiescent() throws Exception {
286 >        ForkJoinPool p = new ForkJoinPool(2);
287 >        try (PoolCleaner cleaner = cleaner(p)) {
288 >            assertTrue(p.isQuiescent());
289 >            long startTime = System.nanoTime();
290 >            FibTask f = new FibTask(20);
291 >            p.invoke(f);
292 >            assertSame(ForkJoinPool.defaultForkJoinWorkerThreadFactory,
293 >                       p.getFactory());
294 >            while (! p.isQuiescent()) {
295 >                if (millisElapsedSince(startTime) > LONG_DELAY_MS)
296 >                    throw new AssertionFailedError("timed out");
297 >                assertFalse(p.getAsyncMode());
298 >                assertFalse(p.isShutdown());
299 >                assertFalse(p.isTerminating());
300 >                assertFalse(p.isTerminated());
301 >                Thread.yield();
302 >            }
303  
344    /**
345     * After invoking a single task, isQuiescent is true,
346     * queues are empty, threads are not active, and
347     * construction parameters continue to hold
348     */
349    public void testisQuiescent() throws InterruptedException {
350        ForkJoinPool p = null;
351        try {
352            p = new ForkJoinPool(2);
353            p.invoke(new FibTask(20));
354            assertTrue(p.getFactory() ==
355                       ForkJoinPool.defaultForkJoinWorkerThreadFactory);
356            Thread.sleep(MEDIUM_DELAY_MS);
304              assertTrue(p.isQuiescent());
358            assertTrue(p.getMaintainsParallelism());
305              assertFalse(p.getAsyncMode());
306 <            assertTrue(p.getActiveThreadCount() == 0);
307 <            assertTrue(p.getQueuedTaskCount() == 0);
308 <            assertTrue(p.getQueuedSubmissionCount() == 0);
306 >            assertEquals(0, p.getActiveThreadCount());
307 >            assertEquals(0, p.getQueuedTaskCount());
308 >            assertEquals(0, p.getQueuedSubmissionCount());
309              assertFalse(p.hasQueuedSubmissions());
310              assertFalse(p.isShutdown());
311              assertFalse(p.isTerminating());
312              assertFalse(p.isTerminated());
313 <        } finally {
314 <            joinPool(p);
313 >            assertTrue(f.isDone());
314 >            assertEquals(6765, (int) f.get());
315          }
316      }
317  
# Line 373 | Line 319 | public class ForkJoinPoolTest extends JS
319       * Completed submit(ForkJoinTask) returns result
320       */
321      public void testSubmitForkJoinTask() throws Throwable {
322 <        ForkJoinPool p = null;
323 <        try {
378 <            p = new ForkJoinPool(1);
322 >        ForkJoinPool p = new ForkJoinPool(1);
323 >        try (PoolCleaner cleaner = cleaner(p)) {
324              ForkJoinTask<Integer> f = p.submit(new FibTask(8));
325 <            int r = f.get();
381 <            assertTrue(r == 21);
382 <        } finally {
383 <            joinPool(p);
325 >            assertEquals(21, (int) f.get());
326          }
327      }
328  
# Line 388 | Line 330 | public class ForkJoinPoolTest extends JS
330       * A task submitted after shutdown is rejected
331       */
332      public void testSubmitAfterShutdown() {
333 <        ForkJoinPool p = null;
334 <        try {
393 <            p = new ForkJoinPool(1);
333 >        ForkJoinPool p = new ForkJoinPool(1);
334 >        try (PoolCleaner cleaner = cleaner(p)) {
335              p.shutdown();
336              assertTrue(p.isShutdown());
337 <            ForkJoinTask<Integer> f = p.submit(new FibTask(8));
338 <            shouldThrow();
339 <        } catch (RejectedExecutionException success) {
340 <        } finally {
400 <            joinPool(p);
337 >            try {
338 >                ForkJoinTask<Integer> f = p.submit(new FibTask(8));
339 >                shouldThrow();
340 >            } catch (RejectedExecutionException success) {}
341          }
342      }
343  
# Line 405 | Line 345 | public class ForkJoinPoolTest extends JS
345       * Pool maintains parallelism when using ManagedBlocker
346       */
347      public void testBlockingForkJoinTask() throws Throwable {
348 <        ForkJoinPool p = null;
348 >        ForkJoinPool p = new ForkJoinPool(4);
349          try {
410            p = new ForkJoinPool(4);
350              ReentrantLock lock = new ReentrantLock();
351              ManagedLocker locker = new ManagedLocker(lock);
352 <            ForkJoinTask<Integer> f = new LockingFibTask(30, locker, lock);
352 >            ForkJoinTask<Integer> f = new LockingFibTask(20, locker, lock);
353              p.execute(f);
354 <            assertTrue(p.getPoolSize() >= 4);
416 <            int r = f.get();
417 <            assertTrue(r ==  832040);
354 >            assertEquals(6765, (int) f.get());
355          } finally {
356              p.shutdownNow(); // don't wait out shutdown
357          }
# Line 424 | Line 361 | public class ForkJoinPoolTest extends JS
361       * pollSubmission returns unexecuted submitted task, if present
362       */
363      public void testPollSubmission() {
364 <        SubFJP p = null;
365 <        try {
366 <            p = new SubFJP();
367 <            ForkJoinTask a = p.submit(new MediumRunnable());
368 <            ForkJoinTask b = p.submit(new MediumRunnable());
369 <            ForkJoinTask c = p.submit(new MediumRunnable());
364 >        final CountDownLatch done = new CountDownLatch(1);
365 >        SubFJP p = new SubFJP();
366 >        try (PoolCleaner cleaner = cleaner(p)) {
367 >            ForkJoinTask a = p.submit(awaiter(done));
368 >            ForkJoinTask b = p.submit(awaiter(done));
369 >            ForkJoinTask c = p.submit(awaiter(done));
370              ForkJoinTask r = p.pollSubmission();
371              assertTrue(r == a || r == b || r == c);
372              assertFalse(r.isDone());
373 <        } finally {
437 <            joinPool(p);
373 >            done.countDown();
374          }
375      }
376  
# Line 442 | Line 378 | public class ForkJoinPoolTest extends JS
378       * drainTasksTo transfers unexecuted submitted tasks, if present
379       */
380      public void testDrainTasksTo() {
381 <        SubFJP p = null;
382 <        try {
383 <            p = new SubFJP();
384 <            ForkJoinTask a = p.submit(new MediumRunnable());
385 <            ForkJoinTask b = p.submit(new MediumRunnable());
386 <            ForkJoinTask c = p.submit(new MediumRunnable());
381 >        final CountDownLatch done = new CountDownLatch(1);
382 >        SubFJP p = new SubFJP();
383 >        try (PoolCleaner cleaner = cleaner(p)) {
384 >            ForkJoinTask a = p.submit(awaiter(done));
385 >            ForkJoinTask b = p.submit(awaiter(done));
386 >            ForkJoinTask c = p.submit(awaiter(done));
387              ArrayList<ForkJoinTask> al = new ArrayList();
388              p.drainTasksTo(al);
389              assertTrue(al.size() > 0);
# Line 455 | Line 391 | public class ForkJoinPoolTest extends JS
391                  assertTrue(r == a || r == b || r == c);
392                  assertFalse(r.isDone());
393              }
394 <        } finally {
459 <            joinPool(p);
394 >            done.countDown();
395          }
396      }
397  
463
398      // FJ Versions of AbstractExecutorService tests
399  
400      /**
# Line 468 | Line 402 | public class ForkJoinPoolTest extends JS
402       */
403      public void testExecuteRunnable() throws Throwable {
404          ExecutorService e = new ForkJoinPool(1);
405 <        TrackedShortRunnable task = new TrackedShortRunnable();
406 <        assertFalse(task.done);
407 <        Future<?> future = e.submit(task);
408 <        future.get();
409 <        assertTrue(task.done);
405 >        try (PoolCleaner cleaner = cleaner(e)) {
406 >            final AtomicBoolean done = new AtomicBoolean(false);
407 >            Future<?> future = e.submit(new CheckedRunnable() {
408 >                public void realRun() {
409 >                    done.set(true);
410 >                }});
411 >            assertNull(future.get());
412 >            assertNull(future.get(0, MILLISECONDS));
413 >            assertTrue(done.get());
414 >            assertTrue(future.isDone());
415 >            assertFalse(future.isCancelled());
416 >        }
417      }
418  
478
419      /**
420       * Completed submit(callable) returns result
421       */
422      public void testSubmitCallable() throws Throwable {
423          ExecutorService e = new ForkJoinPool(1);
424 <        Future<String> future = e.submit(new StringTask());
425 <        String result = future.get();
426 <        assertSame(TEST_STRING, result);
424 >        try (PoolCleaner cleaner = cleaner(e)) {
425 >            Future<String> future = e.submit(new StringTask());
426 >            assertSame(TEST_STRING, future.get());
427 >            assertTrue(future.isDone());
428 >            assertFalse(future.isCancelled());
429 >        }
430      }
431  
432      /**
# Line 491 | Line 434 | public class ForkJoinPoolTest extends JS
434       */
435      public void testSubmitRunnable() throws Throwable {
436          ExecutorService e = new ForkJoinPool(1);
437 <        Future<?> future = e.submit(new NoOpRunnable());
438 <        future.get();
439 <        assertTrue(future.isDone());
437 >        try (PoolCleaner cleaner = cleaner(e)) {
438 >            Future<?> future = e.submit(new NoOpRunnable());
439 >            assertNull(future.get());
440 >            assertTrue(future.isDone());
441 >            assertFalse(future.isCancelled());
442 >        }
443      }
444  
445      /**
# Line 501 | Line 447 | public class ForkJoinPoolTest extends JS
447       */
448      public void testSubmitRunnable2() throws Throwable {
449          ExecutorService e = new ForkJoinPool(1);
450 <        Future<String> future = e.submit(new NoOpRunnable(), TEST_STRING);
451 <        String result = future.get();
452 <        assertSame(TEST_STRING, result);
450 >        try (PoolCleaner cleaner = cleaner(e)) {
451 >            Future<String> future = e.submit(new NoOpRunnable(), TEST_STRING);
452 >            assertSame(TEST_STRING, future.get());
453 >            assertTrue(future.isDone());
454 >            assertFalse(future.isCancelled());
455 >        }
456      }
457  
509
458      /**
459 <     * A submitted privileged action to completion
459 >     * A submitted privileged action runs to completion
460       */
461 <    public void testSubmitPrivilegedAction() throws Throwable {
462 <        Policy savedPolicy = null;
463 <        try {
464 <            savedPolicy = Policy.getPolicy();
465 <            AdjustablePolicy policy = new AdjustablePolicy();
518 <            policy.addPermission(new RuntimePermission("getContextClassLoader"));
519 <            policy.addPermission(new RuntimePermission("setContextClassLoader"));
520 <            Policy.setPolicy(policy);
521 <        } catch (AccessControlException ok) {
522 <            return;
523 <        }
524 <        try {
461 >    public void testSubmitPrivilegedAction() throws Exception {
462 >        final Callable callable = Executors.callable(new PrivilegedAction() {
463 >                public Object run() { return TEST_STRING; }});
464 >        Runnable r = new CheckedRunnable() {
465 >        public void realRun() throws Exception {
466              ExecutorService e = new ForkJoinPool(1);
467 <            Future future = e.submit(Executors.callable(new PrivilegedAction() {
468 <                    public Object run() {
469 <                        return TEST_STRING;
470 <                    }}));
467 >            try (PoolCleaner cleaner = cleaner(e)) {
468 >                Future future = e.submit(callable);
469 >                assertSame(TEST_STRING, future.get());
470 >            }
471 >        }};
472  
473 <            Object result = future.get();
532 <            assertSame(TEST_STRING, result);
533 <        }
534 <        finally {
535 <            Policy.setPolicy(savedPolicy);
536 <        }
473 >        runWithPermissions(r, new RuntimePermission("modifyThread"));
474      }
475  
476      /**
477 <     * A submitted a privileged exception action runs to completion
477 >     * A submitted privileged exception action runs to completion
478       */
479 <    public void testSubmitPrivilegedExceptionAction() throws Throwable {
480 <        Policy savedPolicy = null;
481 <        try {
482 <            savedPolicy = Policy.getPolicy();
483 <            AdjustablePolicy policy = new AdjustablePolicy();
484 <            policy.addPermission(new RuntimePermission("getContextClassLoader"));
548 <            policy.addPermission(new RuntimePermission("setContextClassLoader"));
549 <            Policy.setPolicy(policy);
550 <        } catch (AccessControlException ok) {
551 <            return;
552 <        }
553 <
554 <        try {
479 >    public void testSubmitPrivilegedExceptionAction() throws Exception {
480 >        final Callable callable =
481 >            Executors.callable(new PrivilegedExceptionAction() {
482 >                public Object run() { return TEST_STRING; }});
483 >        Runnable r = new CheckedRunnable() {
484 >        public void realRun() throws Exception {
485              ExecutorService e = new ForkJoinPool(1);
486 <            Future future = e.submit(Executors.callable(new PrivilegedExceptionAction() {
487 <                    public Object run() {
488 <                        return TEST_STRING;
489 <                    }}));
486 >            try (PoolCleaner cleaner = cleaner(e)) {
487 >                Future future = e.submit(callable);
488 >                assertSame(TEST_STRING, future.get());
489 >            }
490 >        }};
491  
492 <            Object result = future.get();
562 <            assertSame(TEST_STRING, result);
563 <        }
564 <        finally {
565 <            Policy.setPolicy(savedPolicy);
566 <        }
492 >        runWithPermissions(r, new RuntimePermission("modifyThread"));
493      }
494  
495      /**
496       * A submitted failed privileged exception action reports exception
497       */
498 <    public void testSubmitFailedPrivilegedExceptionAction() throws Throwable {
499 <        Policy savedPolicy = null;
500 <        try {
501 <            savedPolicy = Policy.getPolicy();
502 <            AdjustablePolicy policy = new AdjustablePolicy();
503 <            policy.addPermission(new RuntimePermission("getContextClassLoader"));
578 <            policy.addPermission(new RuntimePermission("setContextClassLoader"));
579 <            Policy.setPolicy(policy);
580 <        } catch (AccessControlException ok) {
581 <            return;
582 <        }
583 <
584 <
585 <        try {
498 >    public void testSubmitFailedPrivilegedExceptionAction() throws Exception {
499 >        final Callable callable =
500 >            Executors.callable(new PrivilegedExceptionAction() {
501 >                public Object run() { throw new IndexOutOfBoundsException(); }});
502 >        Runnable r = new CheckedRunnable() {
503 >        public void realRun() throws Exception {
504              ExecutorService e = new ForkJoinPool(1);
505 <            Future future = e.submit(Executors.callable(new PrivilegedExceptionAction() {
506 <                    public Object run() throws Exception {
507 <                        throw new IndexOutOfBoundsException();
508 <                    }}));
505 >            try (PoolCleaner cleaner = cleaner(e)) {
506 >                Future future = e.submit(callable);
507 >                try {
508 >                    future.get();
509 >                    shouldThrow();
510 >                } catch (ExecutionException success) {
511 >                    assertTrue(success.getCause() instanceof IndexOutOfBoundsException);
512 >                }
513 >            }
514 >        }};
515  
516 <            Object result = future.get();
593 <            shouldThrow();
594 <        } catch (ExecutionException success) {
595 <        } finally {
596 <            Policy.setPolicy(savedPolicy);
597 <        }
516 >        runWithPermissions(r, new RuntimePermission("modifyThread"));
517      }
518  
519      /**
520       * execute(null runnable) throws NullPointerException
521       */
522      public void testExecuteNullRunnable() {
523 <        try {
524 <            ExecutorService e = new ForkJoinPool(1);
525 <            TrackedShortRunnable task = null;
526 <            Future<?> future = e.submit(task);
527 <            shouldThrow();
528 <        } catch (NullPointerException success) {
523 >        ExecutorService e = new ForkJoinPool(1);
524 >        try (PoolCleaner cleaner = cleaner(e)) {
525 >            try {
526 >                Future<?> future = e.submit((Runnable) null);
527 >                shouldThrow();
528 >            } catch (NullPointerException success) {}
529          }
530      }
531  
613
532      /**
533       * submit(null callable) throws NullPointerException
534       */
535      public void testSubmitNullCallable() {
536 <        try {
537 <            ExecutorService e = new ForkJoinPool(1);
538 <            StringTask t = null;
539 <            Future<String> future = e.submit(t);
540 <            shouldThrow();
541 <        } catch (NullPointerException success) {
536 >        ExecutorService e = new ForkJoinPool(1);
537 >        try (PoolCleaner cleaner = cleaner(e)) {
538 >            try {
539 >                Future<String> future = e.submit((Callable) null);
540 >                shouldThrow();
541 >            } catch (NullPointerException success) {}
542          }
543      }
544  
627
545      /**
546 <     * Blocking on submit(callable) throws InterruptedException if
630 <     * caller interrupted.
546 >     * submit(callable).get() throws InterruptedException if interrupted
547       */
548      public void testInterruptedSubmit() throws InterruptedException {
549 <        final ForkJoinPool p = new ForkJoinPool(1);
550 <
551 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
552 <            void realRun() throws Throwable {
553 <                p.submit(new CheckedCallable<Object>() {
554 <                    public Object realCall() throws Throwable {
555 <                        try {
556 <                            Thread.sleep(MEDIUM_DELAY_MS);
557 <                        } catch (InterruptedException ok) {
558 <                        }
559 <                        return null;
560 <                    }}).get();
561 <            }});
562 <
563 <        t.start();
564 <        Thread.sleep(SHORT_DELAY_MS);
565 <        t.interrupt();
566 <        t.join();
567 <        p.shutdownNow();
568 <        joinPool(p);
549 >        final CountDownLatch submitted    = new CountDownLatch(1);
550 >        final CountDownLatch quittingTime = new CountDownLatch(1);
551 >        final Callable<Void> awaiter = new CheckedCallable<Void>() {
552 >            public Void realCall() throws InterruptedException {
553 >                assertTrue(quittingTime.await(2*LONG_DELAY_MS, MILLISECONDS));
554 >                return null;
555 >            }};
556 >        final ExecutorService p = new ForkJoinPool(1);
557 >        try (PoolCleaner cleaner = cleaner(p, quittingTime)) {
558 >            Thread t = new Thread(new CheckedInterruptedRunnable() {
559 >                public void realRun() throws Exception {
560 >                    Future<Void> future = p.submit(awaiter);
561 >                    submitted.countDown();
562 >                    future.get();
563 >                }});
564 >            t.start();
565 >            await(submitted);
566 >            t.interrupt();
567 >            awaitTermination(t);
568 >        }
569      }
570  
571      /**
# Line 658 | Line 574 | public class ForkJoinPoolTest extends JS
574       */
575      public void testSubmitEE() throws Throwable {
576          ForkJoinPool p = new ForkJoinPool(1);
577 <        try {
578 <            p.submit(new Callable() {
579 <                public Object call() {
580 <                    int i = 5/0;
581 <                    return Boolean.TRUE;
582 <                }}).get();
583 <            shouldThrow();
584 <        } catch (ExecutionException success) {
577 >        try (PoolCleaner cleaner = cleaner(p)) {
578 >            try {
579 >                p.submit(new Callable() {
580 >                        public Object call() { throw new ArithmeticException(); }})
581 >                    .get();
582 >                shouldThrow();
583 >            } catch (ExecutionException success) {
584 >                assertTrue(success.getCause() instanceof ArithmeticException);
585 >            }
586          }
670        joinPool(p);
587      }
588  
589      /**
# Line 675 | Line 591 | public class ForkJoinPoolTest extends JS
591       */
592      public void testInvokeAny1() throws Throwable {
593          ExecutorService e = new ForkJoinPool(1);
594 <        try {
595 <            e.invokeAny(null);
596 <            shouldThrow();
597 <        } catch (NullPointerException success) {
598 <        } finally {
683 <            joinPool(e);
594 >        try (PoolCleaner cleaner = cleaner(e)) {
595 >            try {
596 >                e.invokeAny(null);
597 >                shouldThrow();
598 >            } catch (NullPointerException success) {}
599          }
600      }
601  
# Line 689 | Line 604 | public class ForkJoinPoolTest extends JS
604       */
605      public void testInvokeAny2() throws Throwable {
606          ExecutorService e = new ForkJoinPool(1);
607 <        try {
608 <            e.invokeAny(new ArrayList<Callable<String>>());
609 <            shouldThrow();
610 <        } catch (IllegalArgumentException success) {
611 <        } finally {
697 <            joinPool(e);
607 >        try (PoolCleaner cleaner = cleaner(e)) {
608 >            try {
609 >                e.invokeAny(new ArrayList<Callable<String>>());
610 >                shouldThrow();
611 >            } catch (IllegalArgumentException success) {}
612          }
613      }
614  
615      /**
616 <     * invokeAny(c) throws NullPointerException if c has null elements
616 >     * invokeAny(c) throws NullPointerException if c has a single null element
617       */
618      public void testInvokeAny3() throws Throwable {
619          ExecutorService e = new ForkJoinPool(1);
620 <        try {
621 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
708 <            l.add(new StringTask());
620 >        try (PoolCleaner cleaner = cleaner(e)) {
621 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
622              l.add(null);
623 <            e.invokeAny(l);
624 <            shouldThrow();
625 <        } catch (NullPointerException success) {
626 <        } finally {
714 <            joinPool(e);
623 >            try {
624 >                e.invokeAny(l);
625 >                shouldThrow();
626 >            } catch (NullPointerException success) {}
627          }
628      }
629  
630      /**
631 <     * invokeAny(c) throws ExecutionException if no task in c completes
631 >     * invokeAny(c) throws NullPointerException if c has null elements
632       */
633      public void testInvokeAny4() throws Throwable {
634 +        CountDownLatch latch = new CountDownLatch(1);
635          ExecutorService e = new ForkJoinPool(1);
636 <        try {
637 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
636 >        try (PoolCleaner cleaner = cleaner(e)) {
637 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
638 >            l.add(latchAwaitingStringTask(latch));
639 >            l.add(null);
640 >            try {
641 >                e.invokeAny(l);
642 >                shouldThrow();
643 >            } catch (NullPointerException success) {}
644 >            latch.countDown();
645 >        }
646 >    }
647 >
648 >    /**
649 >     * invokeAny(c) throws ExecutionException if no task in c completes
650 >     */
651 >    public void testInvokeAny5() throws Throwable {
652 >        ExecutorService e = new ForkJoinPool(1);
653 >        try (PoolCleaner cleaner = cleaner(e)) {
654 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
655              l.add(new NPETask());
656 <            e.invokeAny(l);
657 <            shouldThrow();
658 <        } catch (ExecutionException success) {
659 <        } finally {
660 <            joinPool(e);
656 >            try {
657 >                e.invokeAny(l);
658 >                shouldThrow();
659 >            } catch (ExecutionException success) {
660 >                assertTrue(success.getCause() instanceof NullPointerException);
661 >            }
662          }
663      }
664  
665      /**
666       * invokeAny(c) returns result of some task in c if at least one completes
667       */
668 <    public void testInvokeAny5() throws Throwable {
668 >    public void testInvokeAny6() throws Throwable {
669          ExecutorService e = new ForkJoinPool(1);
670 <        try {
671 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
670 >        try (PoolCleaner cleaner = cleaner(e)) {
671 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
672              l.add(new StringTask());
673              l.add(new StringTask());
674              String result = e.invokeAny(l);
675              assertSame(TEST_STRING, result);
745        } finally {
746            joinPool(e);
676          }
677      }
678  
# Line 752 | Line 681 | public class ForkJoinPoolTest extends JS
681       */
682      public void testInvokeAll1() throws Throwable {
683          ExecutorService e = new ForkJoinPool(1);
684 <        try {
685 <            e.invokeAll(null);
686 <            shouldThrow();
687 <        } catch (NullPointerException success) {
688 <        } finally {
760 <            joinPool(e);
684 >        try (PoolCleaner cleaner = cleaner(e)) {
685 >            try {
686 >                e.invokeAll(null);
687 >                shouldThrow();
688 >            } catch (NullPointerException success) {}
689          }
690      }
691  
# Line 766 | Line 694 | public class ForkJoinPoolTest extends JS
694       */
695      public void testInvokeAll2() throws InterruptedException {
696          ExecutorService e = new ForkJoinPool(1);
697 <        try {
697 >        try (PoolCleaner cleaner = cleaner(e)) {
698              List<Future<String>> r
699                  = e.invokeAll(new ArrayList<Callable<String>>());
700              assertTrue(r.isEmpty());
773        } finally {
774            joinPool(e);
701          }
702      }
703  
# Line 780 | Line 706 | public class ForkJoinPoolTest extends JS
706       */
707      public void testInvokeAll3() throws InterruptedException {
708          ExecutorService e = new ForkJoinPool(1);
709 <        try {
710 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
709 >        try (PoolCleaner cleaner = cleaner(e)) {
710 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
711              l.add(new StringTask());
712              l.add(null);
713 <            e.invokeAll(l);
714 <            shouldThrow();
715 <        } catch (NullPointerException success) {
716 <        } finally {
791 <            joinPool(e);
713 >            try {
714 >                e.invokeAll(l);
715 >                shouldThrow();
716 >            } catch (NullPointerException success) {}
717          }
718      }
719  
# Line 798 | Line 723 | public class ForkJoinPoolTest extends JS
723       */
724      public void testInvokeAll4() throws Throwable {
725          ExecutorService e = new ForkJoinPool(1);
726 <        try {
727 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
726 >        try (PoolCleaner cleaner = cleaner(e)) {
727 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
728              l.add(new NPETask());
729 <            List<Future<String>> result = e.invokeAll(l);
730 <            assertEquals(1, result.size());
731 <            for (Future<String> future : result)
732 <                future.get();
733 <            shouldThrow();
734 <        } catch (ExecutionException success) {
735 <        } finally {
736 <            joinPool(e);
729 >            List<Future<String>> futures = e.invokeAll(l);
730 >            assertEquals(1, futures.size());
731 >            try {
732 >                futures.get(0).get();
733 >                shouldThrow();
734 >            } catch (ExecutionException success) {
735 >                assertTrue(success.getCause() instanceof NullPointerException);
736 >            }
737          }
738      }
739  
# Line 817 | Line 742 | public class ForkJoinPoolTest extends JS
742       */
743      public void testInvokeAll5() throws Throwable {
744          ExecutorService e = new ForkJoinPool(1);
745 <        try {
746 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
745 >        try (PoolCleaner cleaner = cleaner(e)) {
746 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
747              l.add(new StringTask());
748              l.add(new StringTask());
749 <            List<Future<String>> result = e.invokeAll(l);
750 <            assertEquals(2, result.size());
751 <            for (Future<String> future : result)
749 >            List<Future<String>> futures = e.invokeAll(l);
750 >            assertEquals(2, futures.size());
751 >            for (Future<String> future : futures)
752                  assertSame(TEST_STRING, future.get());
828        } finally {
829            joinPool(e);
753          }
754      }
755  
833
756      /**
757       * timed invokeAny(null) throws NullPointerException
758       */
759      public void testTimedInvokeAny1() throws Throwable {
760          ExecutorService e = new ForkJoinPool(1);
761 <        try {
762 <            e.invokeAny(null, MEDIUM_DELAY_MS, MILLISECONDS);
763 <            shouldThrow();
764 <        } catch (NullPointerException success) {
765 <        } finally {
844 <            joinPool(e);
761 >        try (PoolCleaner cleaner = cleaner(e)) {
762 >            try {
763 >                e.invokeAny(null, MEDIUM_DELAY_MS, MILLISECONDS);
764 >                shouldThrow();
765 >            } catch (NullPointerException success) {}
766          }
767      }
768  
# Line 850 | Line 771 | public class ForkJoinPoolTest extends JS
771       */
772      public void testTimedInvokeAnyNullTimeUnit() throws Throwable {
773          ExecutorService e = new ForkJoinPool(1);
774 <        try {
775 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
774 >        try (PoolCleaner cleaner = cleaner(e)) {
775 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
776              l.add(new StringTask());
777 <            e.invokeAny(l, MEDIUM_DELAY_MS, null);
778 <            shouldThrow();
779 <        } catch (NullPointerException success) {
780 <        } finally {
860 <            joinPool(e);
777 >            try {
778 >                e.invokeAny(l, MEDIUM_DELAY_MS, null);
779 >                shouldThrow();
780 >            } catch (NullPointerException success) {}
781          }
782      }
783  
# Line 866 | Line 786 | public class ForkJoinPoolTest extends JS
786       */
787      public void testTimedInvokeAny2() throws Throwable {
788          ExecutorService e = new ForkJoinPool(1);
789 <        try {
790 <            e.invokeAny(new ArrayList<Callable<String>>(),
791 <                        MEDIUM_DELAY_MS, MILLISECONDS);
792 <            shouldThrow();
793 <        } catch (IllegalArgumentException success) {
794 <        } finally {
875 <            joinPool(e);
789 >        try (PoolCleaner cleaner = cleaner(e)) {
790 >            try {
791 >                e.invokeAny(new ArrayList<Callable<String>>(),
792 >                            MEDIUM_DELAY_MS, MILLISECONDS);
793 >                shouldThrow();
794 >            } catch (IllegalArgumentException success) {}
795          }
796      }
797  
# Line 880 | Line 799 | public class ForkJoinPoolTest extends JS
799       * timed invokeAny(c) throws NullPointerException if c has null elements
800       */
801      public void testTimedInvokeAny3() throws Throwable {
802 +        CountDownLatch latch = new CountDownLatch(1);
803          ExecutorService e = new ForkJoinPool(1);
804 <        try {
805 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
806 <            l.add(new StringTask());
804 >        try (PoolCleaner cleaner = cleaner(e)) {
805 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
806 >            l.add(latchAwaitingStringTask(latch));
807              l.add(null);
808 <            e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
809 <            shouldThrow();
810 <        } catch (NullPointerException success) {
811 <        } finally {
812 <            joinPool(e);
808 >            try {
809 >                e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
810 >                shouldThrow();
811 >            } catch (NullPointerException success) {}
812 >            latch.countDown();
813          }
814      }
815  
# Line 898 | Line 818 | public class ForkJoinPoolTest extends JS
818       */
819      public void testTimedInvokeAny4() throws Throwable {
820          ExecutorService e = new ForkJoinPool(1);
821 <        try {
822 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
821 >        try (PoolCleaner cleaner = cleaner(e)) {
822 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
823              l.add(new NPETask());
824 <            e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
825 <            shouldThrow();
826 <        } catch (ExecutionException success) {
827 <        } finally {
828 <            joinPool(e);
824 >            try {
825 >                e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
826 >                shouldThrow();
827 >            } catch (ExecutionException success) {
828 >                assertTrue(success.getCause() instanceof NullPointerException);
829 >            }
830          }
831      }
832  
# Line 914 | Line 835 | public class ForkJoinPoolTest extends JS
835       */
836      public void testTimedInvokeAny5() throws Throwable {
837          ExecutorService e = new ForkJoinPool(1);
838 <        try {
839 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
838 >        try (PoolCleaner cleaner = cleaner(e)) {
839 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
840              l.add(new StringTask());
841              l.add(new StringTask());
842              String result = e.invokeAny(l, MEDIUM_DELAY_MS, MILLISECONDS);
843              assertSame(TEST_STRING, result);
923        } finally {
924            joinPool(e);
844          }
845      }
846  
# Line 930 | Line 849 | public class ForkJoinPoolTest extends JS
849       */
850      public void testTimedInvokeAll1() throws Throwable {
851          ExecutorService e = new ForkJoinPool(1);
852 <        try {
853 <            e.invokeAll(null, MEDIUM_DELAY_MS, MILLISECONDS);
854 <            shouldThrow();
855 <        } catch (NullPointerException success) {
856 <        } finally {
938 <            joinPool(e);
852 >        try (PoolCleaner cleaner = cleaner(e)) {
853 >            try {
854 >                e.invokeAll(null, MEDIUM_DELAY_MS, MILLISECONDS);
855 >                shouldThrow();
856 >            } catch (NullPointerException success) {}
857          }
858      }
859  
# Line 944 | Line 862 | public class ForkJoinPoolTest extends JS
862       */
863      public void testTimedInvokeAllNullTimeUnit() throws Throwable {
864          ExecutorService e = new ForkJoinPool(1);
865 <        try {
866 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
865 >        try (PoolCleaner cleaner = cleaner(e)) {
866 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
867              l.add(new StringTask());
868 <            e.invokeAll(l, MEDIUM_DELAY_MS, null);
869 <            shouldThrow();
870 <        } catch (NullPointerException success) {
871 <        } finally {
954 <            joinPool(e);
868 >            try {
869 >                e.invokeAll(l, MEDIUM_DELAY_MS, null);
870 >                shouldThrow();
871 >            } catch (NullPointerException success) {}
872          }
873      }
874  
# Line 960 | Line 877 | public class ForkJoinPoolTest extends JS
877       */
878      public void testTimedInvokeAll2() throws InterruptedException {
879          ExecutorService e = new ForkJoinPool(1);
880 <        try {
880 >        try (PoolCleaner cleaner = cleaner(e)) {
881              List<Future<String>> r
882                  = e.invokeAll(new ArrayList<Callable<String>>(),
883                                MEDIUM_DELAY_MS, MILLISECONDS);
884              assertTrue(r.isEmpty());
968        } finally {
969            joinPool(e);
885          }
886      }
887  
# Line 975 | Line 890 | public class ForkJoinPoolTest extends JS
890       */
891      public void testTimedInvokeAll3() throws InterruptedException {
892          ExecutorService e = new ForkJoinPool(1);
893 <        try {
894 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
893 >        try (PoolCleaner cleaner = cleaner(e)) {
894 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
895              l.add(new StringTask());
896              l.add(null);
897 <            e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
898 <            shouldThrow();
899 <        } catch (NullPointerException success) {
900 <        } finally {
986 <            joinPool(e);
897 >            try {
898 >                e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
899 >                shouldThrow();
900 >            } catch (NullPointerException success) {}
901          }
902      }
903  
# Line 992 | Line 906 | public class ForkJoinPoolTest extends JS
906       */
907      public void testTimedInvokeAll4() throws Throwable {
908          ExecutorService e = new ForkJoinPool(1);
909 <        try {
910 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
909 >        try (PoolCleaner cleaner = cleaner(e)) {
910 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
911              l.add(new NPETask());
912 <            List<Future<String>> result
913 <                = e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
914 <            assertEquals(1, result.size());
915 <            for (Future<String> future : result)
916 <                future.get();
917 <            shouldThrow();
918 <        } catch (ExecutionException success) {
919 <        } finally {
920 <            joinPool(e);
912 >            List<Future<String>> futures
913 >                = e.invokeAll(l, LONG_DELAY_MS, MILLISECONDS);
914 >            assertEquals(1, futures.size());
915 >            try {
916 >                futures.get(0).get();
917 >                shouldThrow();
918 >            } catch (ExecutionException success) {
919 >                assertTrue(success.getCause() instanceof NullPointerException);
920 >            }
921          }
922      }
923  
# Line 1011 | Line 925 | public class ForkJoinPoolTest extends JS
925       * timed invokeAll(c) returns results of all completed tasks in c
926       */
927      public void testTimedInvokeAll5() throws Throwable {
928 <        ExecutorService e = new ForkJoinPool(1);
929 <        try {
930 <            ArrayList<Callable<String>> l = new ArrayList<Callable<String>>();
928 >        ForkJoinPool e = new ForkJoinPool(1);
929 >        try (PoolCleaner cleaner = cleaner(e)) {
930 >            List<Callable<String>> l = new ArrayList<Callable<String>>();
931              l.add(new StringTask());
932              l.add(new StringTask());
933 <            List<Future<String>> result
934 <                = e.invokeAll(l, MEDIUM_DELAY_MS, MILLISECONDS);
935 <            assertEquals(2, result.size());
936 <            for (Future<String> future : result)
933 >            List<Future<String>> futures
934 >                = e.invokeAll(l, LONG_DELAY_MS, MILLISECONDS);
935 >            assertEquals(2, futures.size());
936 >            for (Future<String> future : futures)
937                  assertSame(TEST_STRING, future.get());
1024        } finally {
1025            joinPool(e);
938          }
939      }
940  

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines