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

Comparing jsr166/src/test/tck/FutureTaskTest.java (file contents):
Revision 1.26 by jsr166, Tue May 31 16:16:23 2011 UTC vs.
Revision 1.35 by jsr166, Mon Jan 14 21:54:42 2013 UTC

# Line 14 | Line 14 | import java.util.concurrent.ExecutionExc
14   import java.util.concurrent.Future;
15   import java.util.concurrent.FutureTask;
16   import java.util.concurrent.TimeoutException;
17 + import java.util.concurrent.atomic.AtomicInteger;
18   import static java.util.concurrent.TimeUnit.MILLISECONDS;
19   import static java.util.concurrent.TimeUnit.SECONDS;
20   import java.util.*;
# Line 27 | Line 28 | public class FutureTaskTest extends JSR1
28          return new TestSuite(FutureTaskTest.class);
29      }
30  
31 +    void checkIsDone(Future<?> f) {
32 +        assertTrue(f.isDone());
33 +        assertFalse(f.cancel(false));
34 +        assertFalse(f.cancel(true));
35 +        if (f instanceof PublicFutureTask) {
36 +            PublicFutureTask pf = (PublicFutureTask) f;
37 +            assertEquals(1, pf.doneCount());
38 +            assertFalse(pf.runAndReset());
39 +            assertEquals(1, pf.doneCount());
40 +
41 +            // Check that run and runAndReset have no effect.
42 +            int savedRunCount = pf.runCount();
43 +            int savedSetCount = pf.setCount();
44 +            int savedSetExceptionCount = pf.setExceptionCount();
45 +            pf.run();
46 +            pf.runAndReset();
47 +            assertEquals(savedRunCount, pf.runCount());
48 +            assertEquals(savedSetCount, pf.setCount());
49 +            assertEquals(savedSetExceptionCount, pf.setExceptionCount());
50 +            assertTrue(f.isDone());
51 +        }
52 +    }
53 +
54      void checkNotDone(Future<?> f) {
55          assertFalse(f.isDone());
56          assertFalse(f.isCancelled());
57 +        if (f instanceof PublicFutureTask) {
58 +            PublicFutureTask pf = (PublicFutureTask) f;
59 +            assertEquals(0, pf.doneCount());
60 +            assertEquals(0, pf.setCount());
61 +            assertEquals(0, pf.setExceptionCount());
62 +        }
63 +    }
64 +
65 +    void checkIsRunning(Future<?> f) {
66 +        checkNotDone(f);
67 +        if (f instanceof FutureTask) {
68 +            FutureTask ft = (FutureTask<?>) f;
69 +            // Check that run methods do nothing
70 +            ft.run();
71 +            if (f instanceof PublicFutureTask)
72 +                assertFalse(((PublicFutureTask) f).runAndReset());
73 +            checkNotDone(f);
74 +        }
75      }
76  
77      <T> void checkCompletedNormally(Future<T> f, T expected) {
78 <        assertTrue(f.isDone());
78 >        checkIsDone(f);
79          assertFalse(f.isCancelled());
80  
81          try {
# Line 42 | Line 84 | public class FutureTaskTest extends JSR1
84          try {
85              assertSame(expected, f.get(5L, SECONDS));
86          } catch (Throwable fail) { threadUnexpectedException(fail); }
45
46        assertFalse(f.cancel(false));
47        assertFalse(f.cancel(true));
87      }
88  
89      void checkCancelled(Future<?> f) {
90 <        assertTrue(f.isDone());
90 >        checkIsDone(f);
91          assertTrue(f.isCancelled());
92  
93          try {
# Line 62 | Line 101 | public class FutureTaskTest extends JSR1
101              shouldThrow();
102          } catch (CancellationException success) {
103          } catch (Throwable fail) { threadUnexpectedException(fail); }
104 +    }
105  
106 <        assertFalse(f.cancel(false));
107 <        assertFalse(f.cancel(true));
106 >    void tryToConfuseDoneTask(PublicFutureTask pf) {
107 >        pf.set(new Object());
108 >        pf.setException(new Error());
109 >        for (boolean mayInterruptIfRunning : new boolean[] { true, false }) {
110 >            pf.cancel(true);
111 >        }
112      }
113  
114      void checkCompletedAbnormally(Future<?> f, Throwable t) {
115 <        assertTrue(f.isDone());
115 >        checkIsDone(f);
116          assertFalse(f.isCancelled());
117  
118          try {
# Line 84 | Line 128 | public class FutureTaskTest extends JSR1
128          } catch (ExecutionException success) {
129              assertSame(t, success.getCause());
130          } catch (Throwable fail) { threadUnexpectedException(fail); }
87
88        assertFalse(f.cancel(false));
89        assertFalse(f.cancel(true));
131      }
132  
133      /**
134       * Subclass to expose protected methods
135       */
136      static class PublicFutureTask extends FutureTask {
137 <        public PublicFutureTask(Callable r) { super(r); }
138 <        public boolean runAndReset() { return super.runAndReset(); }
139 <        public void set(Object x) { super.set(x); }
140 <        public void setException(Throwable t) { super.setException(t); }
137 >        private final AtomicInteger runCount;
138 >        private final AtomicInteger doneCount = new AtomicInteger(0);
139 >        private final AtomicInteger runAndResetCount = new AtomicInteger(0);
140 >        private final AtomicInteger setCount = new AtomicInteger(0);
141 >        private final AtomicInteger setExceptionCount = new AtomicInteger(0);
142 >        public int runCount() { return runCount.get(); }
143 >        public int doneCount() { return doneCount.get(); }
144 >        public int runAndResetCount() { return runAndResetCount.get(); }
145 >        public int setCount() { return setCount.get(); }
146 >        public int setExceptionCount() { return setExceptionCount.get(); }
147 >
148 >        PublicFutureTask(Runnable runnable) {
149 >            this(runnable, seven);
150 >        }
151 >        PublicFutureTask(Runnable runnable, Object result) {
152 >            this(runnable, result, new AtomicInteger(0));
153 >        }
154 >        private PublicFutureTask(final Runnable runnable, Object result,
155 >                                 final AtomicInteger runCount) {
156 >            super(new Runnable() {
157 >                public void run() {
158 >                    runCount.getAndIncrement();
159 >                    runnable.run();
160 >                }}, result);
161 >            this.runCount = runCount;
162 >        }
163 >        PublicFutureTask(Callable callable) {
164 >            this(callable, new AtomicInteger(0));
165 >        }
166 >        private PublicFutureTask(final Callable callable,
167 >                                 final AtomicInteger runCount) {
168 >            super(new Callable() {
169 >                public Object call() throws Exception {
170 >                    runCount.getAndIncrement();
171 >                    return callable.call();
172 >                }});
173 >            this.runCount = runCount;
174 >        }
175 >        @Override public void done() {
176 >            assertTrue(isDone());
177 >            doneCount.incrementAndGet();
178 >            super.done();
179 >        }
180 >        @Override public boolean runAndReset() {
181 >            runAndResetCount.incrementAndGet();
182 >            return super.runAndReset();
183 >        }
184 >        @Override public void set(Object x) {
185 >            setCount.incrementAndGet();
186 >            super.set(x);
187 >        }
188 >        @Override public void setException(Throwable t) {
189 >            setExceptionCount.incrementAndGet();
190 >            super.setException(t);
191 >        }
192 >    }
193 >
194 >    class Counter extends CheckedRunnable {
195 >        final AtomicInteger count = new AtomicInteger(0);
196 >        public int get() { return count.get(); }
197 >        public void realRun() {
198 >            count.getAndIncrement();
199 >        }
200      }
201  
202      /**
203 <     * Creating a future with a null callable throws NPE
203 >     * creating a future with a null callable throws NullPointerException
204       */
205      public void testConstructor() {
206          try {
207 <            FutureTask task = new FutureTask(null);
207 >            new FutureTask(null);
208              shouldThrow();
209          } catch (NullPointerException success) {}
210      }
211  
212      /**
213 <     * creating a future with null runnable fails
213 >     * creating a future with null runnable throws NullPointerException
214       */
215      public void testConstructor2() {
216          try {
217 <            FutureTask task = new FutureTask(null, Boolean.TRUE);
217 >            new FutureTask(null, Boolean.TRUE);
218              shouldThrow();
219          } catch (NullPointerException success) {}
220      }
# Line 123 | Line 223 | public class FutureTaskTest extends JSR1
223       * isDone is true when a task completes
224       */
225      public void testIsDone() {
226 <        FutureTask task = new FutureTask(new NoOpCallable());
226 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
227 >        assertFalse(task.isDone());
228          task.run();
229          assertTrue(task.isDone());
230          checkCompletedNormally(task, Boolean.TRUE);
231 +        assertEquals(1, task.runCount());
232      }
233  
234      /**
# Line 134 | Line 236 | public class FutureTaskTest extends JSR1
236       */
237      public void testRunAndReset() {
238          PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
239 <        assertTrue(task.runAndReset());
240 <        checkNotDone(task);
239 >        for (int i = 0; i < 3; i++) {
240 >            assertTrue(task.runAndReset());
241 >            checkNotDone(task);
242 >            assertEquals(i+1, task.runCount());
243 >            assertEquals(i+1, task.runAndResetCount());
244 >            assertEquals(0, task.setCount());
245 >            assertEquals(0, task.setExceptionCount());
246 >        }
247      }
248  
249      /**
250       * runAndReset after cancellation fails
251       */
252 <    public void testResetAfterCancel() {
253 <        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
254 <        assertTrue(task.cancel(false));
255 <        assertFalse(task.runAndReset());
256 <        checkCancelled(task);
252 >    public void testRunAndResetAfterCancel() {
253 >        for (boolean mayInterruptIfRunning : new boolean[] { true, false }) {
254 >            PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
255 >            assertTrue(task.cancel(mayInterruptIfRunning));
256 >            for (int i = 0; i < 3; i++) {
257 >                assertFalse(task.runAndReset());
258 >                assertEquals(0, task.runCount());
259 >                assertEquals(i+1, task.runAndResetCount());
260 >                assertEquals(0, task.setCount());
261 >                assertEquals(0, task.setExceptionCount());
262 >            }
263 >            tryToConfuseDoneTask(task);
264 >            checkCancelled(task);
265 >        }
266      }
267  
151
268      /**
269       * setting value causes get to return it
270       */
271      public void testSet() throws Exception {
272          PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
273          task.set(one);
274 <        assertSame(task.get(), one);
274 >        for (int i = 0; i < 3; i++) {
275 >            assertSame(one, task.get());
276 >            assertSame(one, task.get(LONG_DELAY_MS, MILLISECONDS));
277 >            assertEquals(1, task.setCount());
278 >        }
279 >        tryToConfuseDoneTask(task);
280          checkCompletedNormally(task, one);
281 +        assertEquals(0, task.runCount());
282      }
283  
284      /**
285       * setException causes get to throw ExecutionException
286       */
287 <    public void testSetException() throws Exception {
287 >    public void testSetException_get() throws Exception {
288          Exception nse = new NoSuchElementException();
289          PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
290          task.setException(nse);
291 +
292 +        try {
293 +            task.get();
294 +            shouldThrow();
295 +        } catch (ExecutionException success) {
296 +            assertSame(nse, success.getCause());
297 +            checkCompletedAbnormally(task, nse);
298 +        }
299 +
300          try {
301 <            Object x = task.get();
301 >            task.get(LONG_DELAY_MS, MILLISECONDS);
302              shouldThrow();
303          } catch (ExecutionException success) {
304 <            assertSame(success.getCause(), nse);
304 >            assertSame(nse, success.getCause());
305              checkCompletedAbnormally(task, nse);
306          }
307 +
308 +        assertEquals(1, task.setExceptionCount());
309 +        assertEquals(0, task.setCount());
310 +        tryToConfuseDoneTask(task);
311 +        checkCompletedAbnormally(task, nse);
312 +        assertEquals(0, task.runCount());
313      }
314  
315      /**
316 <     * Cancelling before running succeeds
316 >     * cancel(false) before run succeeds
317       */
318      public void testCancelBeforeRun() {
319 <        FutureTask task = new FutureTask(new NoOpCallable());
319 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
320          assertTrue(task.cancel(false));
321          task.run();
322 +        assertEquals(0, task.runCount());
323 +        assertEquals(0, task.setCount());
324 +        assertEquals(0, task.setExceptionCount());
325 +        assertTrue(task.isCancelled());
326 +        assertTrue(task.isDone());
327 +        tryToConfuseDoneTask(task);
328 +        assertEquals(0, task.runCount());
329          checkCancelled(task);
330      }
331  
332      /**
333 <     * Cancel(true) before run succeeds
333 >     * cancel(true) before run succeeds
334       */
335      public void testCancelBeforeRun2() {
336 <        FutureTask task = new FutureTask(new NoOpCallable());
336 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
337          assertTrue(task.cancel(true));
338          task.run();
339 +        assertEquals(0, task.runCount());
340 +        assertEquals(0, task.setCount());
341 +        assertEquals(0, task.setExceptionCount());
342 +        assertTrue(task.isCancelled());
343 +        assertTrue(task.isDone());
344 +        tryToConfuseDoneTask(task);
345 +        assertEquals(0, task.runCount());
346          checkCancelled(task);
347      }
348  
349      /**
350 <     * cancel of a completed task fails
350 >     * cancel(false) of a completed task fails
351       */
352      public void testCancelAfterRun() {
353 <        FutureTask task = new FutureTask(new NoOpCallable());
353 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
354          task.run();
355          assertFalse(task.cancel(false));
356 +        assertEquals(1, task.runCount());
357 +        assertEquals(1, task.setCount());
358 +        assertEquals(0, task.setExceptionCount());
359 +        tryToConfuseDoneTask(task);
360          checkCompletedNormally(task, Boolean.TRUE);
361 +        assertEquals(1, task.runCount());
362      }
363  
364      /**
365 <     * cancel(true) interrupts a running task
365 >     * cancel(true) of a completed task fails
366       */
367 <    public void testCancelInterrupt() throws InterruptedException {
368 <        final CountDownLatch threadStarted = new CountDownLatch(1);
369 <        final FutureTask task =
370 <            new FutureTask(new CheckedCallable<Object>() {
371 <                public Object realCall() {
372 <                    threadStarted.countDown();
373 <                    long t0 = System.nanoTime();
374 <                    for (;;) {
375 <                        if (Thread.interrupted())
376 <                            return Boolean.TRUE;
377 <                        if (millisElapsedSince(t0) > MEDIUM_DELAY_MS)
378 <                            fail("interrupt not delivered");
379 <                        Thread.yield();
380 <                    }
367 >    public void testCancelAfterRun2() {
368 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
369 >        task.run();
370 >        assertFalse(task.cancel(true));
371 >        assertEquals(1, task.runCount());
372 >        assertEquals(1, task.setCount());
373 >        assertEquals(0, task.setExceptionCount());
374 >        tryToConfuseDoneTask(task);
375 >        checkCompletedNormally(task, Boolean.TRUE);
376 >        assertEquals(1, task.runCount());
377 >    }
378 >
379 >    /**
380 >     * cancel(true) interrupts a running task that subsequently succeeds
381 >     */
382 >    public void testCancelInterrupt() {
383 >        final CountDownLatch pleaseCancel = new CountDownLatch(1);
384 >        final PublicFutureTask task =
385 >            new PublicFutureTask(new CheckedRunnable() {
386 >                public void realRun() {
387 >                    pleaseCancel.countDown();
388 >                    try {
389 >                        delay(LONG_DELAY_MS);
390 >                        shouldThrow();
391 >                    } catch (InterruptedException success) {}
392                  }});
393  
394          Thread t = newStartedThread(task);
395 <        threadStarted.await();
395 >        await(pleaseCancel);
396          assertTrue(task.cancel(true));
397 <        checkCancelled(task);
398 <        awaitTermination(t, MEDIUM_DELAY_MS);
397 >        assertTrue(task.isCancelled());
398 >        assertTrue(task.isDone());
399 >        awaitTermination(t);
400 >        assertEquals(1, task.runCount());
401 >        assertEquals(1, task.setCount());
402 >        assertEquals(0, task.setExceptionCount());
403 >        tryToConfuseDoneTask(task);
404          checkCancelled(task);
405      }
406  
407      /**
408 <     * cancel(false) does not interrupt a running task
408 >     * cancel(true) tries to interrupt a running task, but
409 >     * Thread.interrupt throws (simulating a restrictive security
410 >     * manager)
411       */
412 <    public void testCancelNoInterrupt() throws InterruptedException {
413 <        final CountDownLatch threadStarted = new CountDownLatch(1);
412 >    public void testCancelInterrupt_ThrowsSecurityException() {
413 >        final CountDownLatch pleaseCancel = new CountDownLatch(1);
414          final CountDownLatch cancelled = new CountDownLatch(1);
415 <        final FutureTask<Boolean> task =
416 <            new FutureTask<Boolean>(new CheckedCallable<Boolean>() {
417 <                public Boolean realCall() throws InterruptedException {
418 <                    threadStarted.countDown();
419 <                    cancelled.await(MEDIUM_DELAY_MS, MILLISECONDS);
415 >        final PublicFutureTask task =
416 >            new PublicFutureTask(new CheckedRunnable() {
417 >                public void realRun() {
418 >                    pleaseCancel.countDown();
419 >                    await(cancelled);
420                      assertFalse(Thread.interrupted());
247                    return Boolean.TRUE;
421                  }});
422  
423 <        Thread t = newStartedThread(task);
424 <        threadStarted.await();
425 <        assertTrue(task.cancel(false));
426 <        checkCancelled(task);
423 >        final Thread t = new Thread(task) {
424 >            // Simulate a restrictive security manager.
425 >            @Override public void interrupt() {
426 >                throw new SecurityException();
427 >            }};
428 >        t.setDaemon(true);
429 >        t.start();
430 >
431 >        await(pleaseCancel);
432 >        try {
433 >            task.cancel(true);
434 >            shouldThrow();
435 >        } catch (SecurityException expected) {}
436 >
437 >        // We failed to deliver the interrupt, but the world retains
438 >        // its sanity, as if we had done task.cancel(false)
439 >        assertTrue(task.isCancelled());
440 >        assertTrue(task.isDone());
441 >        assertEquals(1, task.runCount());
442 >        assertEquals(1, task.doneCount());
443 >        assertEquals(0, task.setCount());
444 >        assertEquals(0, task.setExceptionCount());
445          cancelled.countDown();
446 <        awaitTermination(t, MEDIUM_DELAY_MS);
446 >        awaitTermination(t);
447 >        assertEquals(1, task.setCount());
448 >        assertEquals(0, task.setExceptionCount());
449 >        tryToConfuseDoneTask(task);
450          checkCancelled(task);
451      }
452  
453      /**
454 <     * run in one thread causes get in another thread to retrieve value
454 >     * cancel(true) interrupts a running task that subsequently throws
455       */
456 <    public void testGetRun() throws InterruptedException {
457 <        final CountDownLatch threadStarted = new CountDownLatch(1);
458 <
459 <        final FutureTask task =
460 <            new FutureTask(new CheckedCallable<Object>() {
461 <                public Object realCall() throws InterruptedException {
462 <                    return Boolean.TRUE;
456 >    public void testCancelInterrupt_taskFails() {
457 >        final CountDownLatch pleaseCancel = new CountDownLatch(1);
458 >        final PublicFutureTask task =
459 >            new PublicFutureTask(new Runnable() {
460 >                public void run() {
461 >                    try {
462 >                        pleaseCancel.countDown();
463 >                        delay(LONG_DELAY_MS);
464 >                        shouldThrow();
465 >                    } catch (Throwable t) {
466 >                        assertTrue(t instanceof InterruptedException);
467 >                    }
468 >                    throw new RuntimeException();
469                  }});
470  
471 <        Thread t = newStartedThread(new CheckedRunnable() {
472 <            public void realRun() throws Exception {
473 <                threadStarted.countDown();
474 <                assertSame(Boolean.TRUE, task.get());
475 <            }});
476 <
477 <        threadStarted.await();
478 <        checkNotDone(task);
479 <        assertTrue(t.isAlive());
480 <        task.run();
281 <        checkCompletedNormally(task, Boolean.TRUE);
282 <        awaitTermination(t, MEDIUM_DELAY_MS);
471 >        Thread t = newStartedThread(task);
472 >        await(pleaseCancel);
473 >        assertTrue(task.cancel(true));
474 >        assertTrue(task.isCancelled());
475 >        awaitTermination(t);
476 >        assertEquals(1, task.runCount());
477 >        assertEquals(0, task.setCount());
478 >        assertEquals(1, task.setExceptionCount());
479 >        tryToConfuseDoneTask(task);
480 >        checkCancelled(task);
481      }
482  
483      /**
484 <     * set in one thread causes get in another thread to retrieve value
484 >     * cancel(false) does not interrupt a running task
485       */
486 <    public void testGetSet() throws InterruptedException {
487 <        final CountDownLatch threadStarted = new CountDownLatch(1);
488 <
486 >    public void testCancelNoInterrupt() {
487 >        final CountDownLatch pleaseCancel = new CountDownLatch(1);
488 >        final CountDownLatch cancelled = new CountDownLatch(1);
489          final PublicFutureTask task =
490 <            new PublicFutureTask(new CheckedCallable<Object>() {
491 <                public Object realCall() throws InterruptedException {
490 >            new PublicFutureTask(new CheckedCallable<Boolean>() {
491 >                public Boolean realCall() {
492 >                    pleaseCancel.countDown();
493 >                    await(cancelled);
494 >                    assertFalse(Thread.interrupted());
495                      return Boolean.TRUE;
496                  }});
497  
498 <        Thread t = newStartedThread(new CheckedRunnable() {
499 <            public void realRun() throws Exception {
500 <                threadStarted.countDown();
501 <                assertSame(Boolean.FALSE, task.get());
502 <            }});
503 <
504 <        threadStarted.await();
505 <        checkNotDone(task);
506 <        assertTrue(t.isAlive());
507 <        task.set(Boolean.FALSE);
508 <        checkCompletedNormally(task, Boolean.FALSE);
308 <        awaitTermination(t, MEDIUM_DELAY_MS);
498 >        Thread t = newStartedThread(task);
499 >        await(pleaseCancel);
500 >        assertTrue(task.cancel(false));
501 >        assertTrue(task.isCancelled());
502 >        cancelled.countDown();
503 >        awaitTermination(t);
504 >        assertEquals(1, task.runCount());
505 >        assertEquals(1, task.setCount());
506 >        assertEquals(0, task.setExceptionCount());
507 >        tryToConfuseDoneTask(task);
508 >        checkCancelled(task);
509      }
510  
511      /**
512 <     * run in one thread causes timed get in another thread to retrieve value
512 >     * run in one thread causes get in another thread to retrieve value
513       */
514 <    public void testTimedGetRun() throws InterruptedException {
515 <        final CountDownLatch threadStarted = new CountDownLatch(1);
514 >    public void testGetRun() {
515 >        final CountDownLatch pleaseRun = new CountDownLatch(2);
516  
517 <        final FutureTask task =
518 <            new FutureTask(new CheckedCallable<Object>() {
519 <                public Object realCall() throws InterruptedException {
520 <                    return Boolean.TRUE;
517 >        final PublicFutureTask task =
518 >            new PublicFutureTask(new CheckedCallable<Object>() {
519 >                public Object realCall() {
520 >                    return two;
521                  }});
522  
523 <        Thread t = newStartedThread(new CheckedRunnable() {
523 >        Thread t1 = newStartedThread(new CheckedRunnable() {
524              public void realRun() throws Exception {
525 <                threadStarted.countDown();
526 <                assertSame(Boolean.TRUE,
327 <                           task.get(MEDIUM_DELAY_MS, MILLISECONDS));
525 >                pleaseRun.countDown();
526 >                assertSame(two, task.get());
527              }});
528  
529 <        threadStarted.await();
529 >        Thread t2 = newStartedThread(new CheckedRunnable() {
530 >            public void realRun() throws Exception {
531 >                pleaseRun.countDown();
532 >                assertSame(two, task.get(2*LONG_DELAY_MS, MILLISECONDS));
533 >            }});
534 >
535 >        await(pleaseRun);
536          checkNotDone(task);
537 <        assertTrue(t.isAlive());
537 >        assertTrue(t1.isAlive());
538 >        assertTrue(t2.isAlive());
539          task.run();
540 <        checkCompletedNormally(task, Boolean.TRUE);
541 <        awaitTermination(t, MEDIUM_DELAY_MS);
540 >        checkCompletedNormally(task, two);
541 >        assertEquals(1, task.runCount());
542 >        assertEquals(1, task.setCount());
543 >        assertEquals(0, task.setExceptionCount());
544 >        awaitTermination(t1);
545 >        awaitTermination(t2);
546 >        tryToConfuseDoneTask(task);
547 >        checkCompletedNormally(task, two);
548      }
549  
550      /**
551 <     * set in one thread causes timed get in another thread to retrieve value
551 >     * set in one thread causes get in another thread to retrieve value
552       */
553 <    public void testTimedGetSet() throws InterruptedException {
554 <        final CountDownLatch threadStarted = new CountDownLatch(1);
553 >    public void testGetSet() {
554 >        final CountDownLatch pleaseSet = new CountDownLatch(2);
555  
556          final PublicFutureTask task =
557              new PublicFutureTask(new CheckedCallable<Object>() {
558                  public Object realCall() throws InterruptedException {
559 <                    return Boolean.TRUE;
559 >                    return two;
560                  }});
561  
562 <        Thread t = newStartedThread(new CheckedRunnable() {
562 >        Thread t1 = newStartedThread(new CheckedRunnable() {
563 >            public void realRun() throws Exception {
564 >                pleaseSet.countDown();
565 >                assertSame(two, task.get());
566 >            }});
567 >
568 >        Thread t2 = newStartedThread(new CheckedRunnable() {
569              public void realRun() throws Exception {
570 <                threadStarted.countDown();
571 <                assertSame(Boolean.FALSE,
354 <                           task.get(MEDIUM_DELAY_MS, MILLISECONDS));
570 >                pleaseSet.countDown();
571 >                assertSame(two, task.get(2*LONG_DELAY_MS, MILLISECONDS));
572              }});
573  
574 <        threadStarted.await();
574 >        await(pleaseSet);
575          checkNotDone(task);
576 <        assertTrue(t.isAlive());
577 <        task.set(Boolean.FALSE);
578 <        checkCompletedNormally(task, Boolean.FALSE);
579 <        awaitTermination(t, MEDIUM_DELAY_MS);
576 >        assertTrue(t1.isAlive());
577 >        assertTrue(t2.isAlive());
578 >        task.set(two);
579 >        assertEquals(0, task.runCount());
580 >        assertEquals(1, task.setCount());
581 >        assertEquals(0, task.setExceptionCount());
582 >        tryToConfuseDoneTask(task);
583 >        checkCompletedNormally(task, two);
584 >        awaitTermination(t1);
585 >        awaitTermination(t2);
586      }
587  
588      /**
589       * Cancelling a task causes timed get in another thread to throw
590       * CancellationException
591       */
592 <    public void testTimedGet_Cancellation() throws InterruptedException {
593 <        final CountDownLatch threadStarted = new CountDownLatch(2);
594 <        final FutureTask task =
595 <            new FutureTask(new CheckedInterruptedCallable<Object>() {
596 <                public Object realCall() throws InterruptedException {
597 <                    threadStarted.countDown();
598 <                    delay(LONG_DELAY_MS);
599 <                    return Boolean.TRUE;
600 <                }});
601 <
602 <        Thread t1 = new ThreadShouldThrow(CancellationException.class) {
603 <            public void realRun() throws Exception {
604 <                threadStarted.countDown();
605 <                task.get(MEDIUM_DELAY_MS, MILLISECONDS);
606 <            }};
607 <        Thread t2 = new Thread(task);
608 <        t1.start();
609 <        t2.start();
610 <        threadStarted.await();
611 <        task.cancel(true);
612 <        awaitTermination(t1, MEDIUM_DELAY_MS);
613 <        awaitTermination(t2, MEDIUM_DELAY_MS);
614 <        checkCancelled(task);
615 <    }
616 <
617 <    /**
618 <     * Cancelling a task causes get in another thread to throw
619 <     * CancellationException
620 <     */
621 <    public void testGet_Cancellation() throws InterruptedException {
622 <        final CountDownLatch threadStarted = new CountDownLatch(2);
623 <        final FutureTask task =
624 <            new FutureTask(new CheckedInterruptedCallable<Object>() {
625 <                public Object realCall() throws InterruptedException {
626 <                    threadStarted.countDown();
627 <                    delay(LONG_DELAY_MS);
628 <                    return Boolean.TRUE;
629 <                }});
630 <
631 <        Thread t1 = new ThreadShouldThrow(CancellationException.class) {
632 <            public void realRun() throws Exception {
633 <                threadStarted.countDown();
634 <                task.get();
635 <            }};
636 <        Thread t2 = new Thread(task);
637 <        t1.start();
415 <        t2.start();
416 <        threadStarted.await();
417 <        task.cancel(true);
418 <        awaitTermination(t1, MEDIUM_DELAY_MS);
419 <        awaitTermination(t2, MEDIUM_DELAY_MS);
420 <        checkCancelled(task);
592 >    public void testTimedGet_Cancellation() {
593 >        for (final boolean mayInterruptIfRunning :
594 >                 new boolean[] { true, false }) {
595 >            final CountDownLatch pleaseCancel = new CountDownLatch(3);
596 >            final CountDownLatch cancelled = new CountDownLatch(1);
597 >            final PublicFutureTask task =
598 >                new PublicFutureTask(new CheckedCallable<Object>() {
599 >                    public Object realCall() throws InterruptedException {
600 >                        pleaseCancel.countDown();
601 >                        if (mayInterruptIfRunning) {
602 >                            try {
603 >                                delay(2*LONG_DELAY_MS);
604 >                            } catch (InterruptedException success) {}
605 >                        } else {
606 >                            await(cancelled);
607 >                        }
608 >                        return two;
609 >                    }});
610 >
611 >            Thread t1 = new ThreadShouldThrow(CancellationException.class) {
612 >                public void realRun() throws Exception {
613 >                    pleaseCancel.countDown();
614 >                    task.get();
615 >                }};
616 >            Thread t2 = new ThreadShouldThrow(CancellationException.class) {
617 >                public void realRun() throws Exception {
618 >                    pleaseCancel.countDown();
619 >                    task.get(2*LONG_DELAY_MS, MILLISECONDS);
620 >                }};
621 >            t1.start();
622 >            t2.start();
623 >            Thread t3 = newStartedThread(task);
624 >            await(pleaseCancel);
625 >            checkIsRunning(task);
626 >            task.cancel(mayInterruptIfRunning);
627 >            checkCancelled(task);
628 >            awaitTermination(t1);
629 >            awaitTermination(t2);
630 >            cancelled.countDown();
631 >            awaitTermination(t3);
632 >            assertEquals(1, task.runCount());
633 >            assertEquals(1, task.setCount());
634 >            assertEquals(0, task.setExceptionCount());
635 >            tryToConfuseDoneTask(task);
636 >            checkCancelled(task);
637 >        }
638      }
639  
423
640      /**
641       * A runtime exception in task causes get to throw ExecutionException
642       */
643      public void testGet_ExecutionException() throws InterruptedException {
644 <        final FutureTask task = new FutureTask(new Callable() {
644 >        final ArithmeticException e = new ArithmeticException();
645 >        final PublicFutureTask task = new PublicFutureTask(new Callable() {
646              public Object call() {
647 <                return 5/0;
647 >                throw e;
648              }});
649  
650          task.run();
651 +        assertEquals(1, task.runCount());
652 +        assertEquals(0, task.setCount());
653 +        assertEquals(1, task.setExceptionCount());
654          try {
655              task.get();
656              shouldThrow();
657          } catch (ExecutionException success) {
658 <            assertTrue(success.getCause() instanceof ArithmeticException);
658 >            assertSame(e, success.getCause());
659 >            tryToConfuseDoneTask(task);
660              checkCompletedAbnormally(task, success.getCause());
661          }
662      }
# Line 444 | Line 665 | public class FutureTaskTest extends JSR1
665       * A runtime exception in task causes timed get to throw ExecutionException
666       */
667      public void testTimedGet_ExecutionException2() throws Exception {
668 <        final FutureTask task = new FutureTask(new Callable() {
668 >        final ArithmeticException e = new ArithmeticException();
669 >        final PublicFutureTask task = new PublicFutureTask(new Callable() {
670              public Object call() {
671 <                return 5/0;
671 >                throw e;
672              }});
673  
674          task.run();
675          try {
676 <            task.get(SHORT_DELAY_MS, MILLISECONDS);
676 >            task.get(LONG_DELAY_MS, MILLISECONDS);
677              shouldThrow();
678          } catch (ExecutionException success) {
679 <            assertTrue(success.getCause() instanceof ArithmeticException);
679 >            assertSame(e, success.getCause());
680 >            tryToConfuseDoneTask(task);
681              checkCompletedAbnormally(task, success.getCause());
682          }
683      }
684  
462
685      /**
686 <     * Interrupting a waiting get causes it to throw InterruptedException
686 >     * get is interruptible
687       */
688 <    public void testGet_InterruptedException() throws InterruptedException {
689 <        final CountDownLatch threadStarted = new CountDownLatch(1);
688 >    public void testGet_interruptible() {
689 >        final CountDownLatch pleaseInterrupt = new CountDownLatch(1);
690          final FutureTask task = new FutureTask(new NoOpCallable());
691 <        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
691 >        Thread t = newStartedThread(new CheckedRunnable() {
692              public void realRun() throws Exception {
693 <                threadStarted.countDown();
694 <                task.get();
693 >                Thread.currentThread().interrupt();
694 >                try {
695 >                    task.get();
696 >                    shouldThrow();
697 >                } catch (InterruptedException success) {}
698 >                assertFalse(Thread.interrupted());
699 >
700 >                pleaseInterrupt.countDown();
701 >                try {
702 >                    task.get();
703 >                    shouldThrow();
704 >                } catch (InterruptedException success) {}
705 >                assertFalse(Thread.interrupted());
706              }});
707  
708 <        threadStarted.await();
708 >        await(pleaseInterrupt);
709          t.interrupt();
710 <        awaitTermination(t, MEDIUM_DELAY_MS);
710 >        awaitTermination(t);
711          checkNotDone(task);
712      }
713  
714      /**
715 <     * Interrupting a waiting timed get causes it to throw InterruptedException
715 >     * timed get is interruptible
716       */
717 <    public void testTimedGet_InterruptedException2() throws InterruptedException {
718 <        final CountDownLatch threadStarted = new CountDownLatch(1);
717 >    public void testTimedGet_interruptible() {
718 >        final CountDownLatch pleaseInterrupt = new CountDownLatch(1);
719          final FutureTask task = new FutureTask(new NoOpCallable());
720 <        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
720 >        Thread t = newStartedThread(new CheckedRunnable() {
721              public void realRun() throws Exception {
722 <                threadStarted.countDown();
723 <                task.get(LONG_DELAY_MS, MILLISECONDS);
722 >                Thread.currentThread().interrupt();
723 >                try {
724 >                    task.get(2*LONG_DELAY_MS, MILLISECONDS);
725 >                    shouldThrow();
726 >                } catch (InterruptedException success) {}
727 >                assertFalse(Thread.interrupted());
728 >
729 >                pleaseInterrupt.countDown();
730 >                try {
731 >                    task.get(2*LONG_DELAY_MS, MILLISECONDS);
732 >                    shouldThrow();
733 >                } catch (InterruptedException success) {}
734 >                assertFalse(Thread.interrupted());
735              }});
736  
737 <        threadStarted.await();
737 >        await(pleaseInterrupt);
738          t.interrupt();
739 <        awaitTermination(t, MEDIUM_DELAY_MS);
739 >        awaitTermination(t);
740          checkNotDone(task);
741      }
742  
# Line 500 | Line 744 | public class FutureTaskTest extends JSR1
744       * A timed out timed get throws TimeoutException
745       */
746      public void testGet_TimeoutException() throws Exception {
747 +        FutureTask task = new FutureTask(new NoOpCallable());
748 +        long startTime = System.nanoTime();
749          try {
750 <            FutureTask task = new FutureTask(new NoOpCallable());
505 <            task.get(1, MILLISECONDS);
750 >            task.get(timeoutMillis(), MILLISECONDS);
751              shouldThrow();
752 <        } catch (TimeoutException success) {}
752 >        } catch (TimeoutException success) {
753 >            assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
754 >        }
755 >    }
756 >
757 >    /**
758 >     * timed get with null TimeUnit throws NullPointerException
759 >     */
760 >    public void testGet_NullTimeUnit() throws Exception {
761 >        FutureTask task = new FutureTask(new NoOpCallable());
762 >        long[] timeouts = { Long.MIN_VALUE, 0L, Long.MAX_VALUE };
763 >
764 >        for (long timeout : timeouts) {
765 >            try {
766 >                task.get(timeout, null);
767 >                shouldThrow();
768 >            } catch (NullPointerException success) {}
769 >        }
770 >
771 >        task.run();
772 >
773 >        for (long timeout : timeouts) {
774 >            try {
775 >                task.get(timeout, null);
776 >                shouldThrow();
777 >            } catch (NullPointerException success) {}
778 >        }
779      }
780  
781   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines