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.5 by dl, Sat Sep 20 18:20:07 2003 UTC vs.
Revision 1.22 by jsr166, Sun Nov 28 20:20:00 2010 UTC

# Line 1 | Line 1
1   /*
2 < * Written by members of JCP JSR-166 Expert Group and released to the
3 < * public domain. Use, modify, and redistribute this code in any way
4 < * without acknowledgement. Other contributors include Andrew Wright,
5 < * Jeffrey Hayes, Pat Fischer, Mike Judd.
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
5 > * Other contributors include Andrew Wright, Jeffrey Hayes,
6 > * Pat Fisher, Mike Judd.
7   */
8  
9   import junit.framework.*;
10   import java.util.concurrent.*;
11 + import static java.util.concurrent.TimeUnit.MILLISECONDS;
12 + import static java.util.concurrent.TimeUnit.SECONDS;
13   import java.util.*;
14  
15   public class FutureTaskTest extends JSR166TestCase {
16  
17      public static void main(String[] args) {
18 <        junit.textui.TestRunner.run (suite());  
18 >        junit.textui.TestRunner.run(suite());
19      }
20      public static Test suite() {
21 <        return new TestSuite(FutureTaskTest.class);
21 >        return new TestSuite(FutureTaskTest.class);
22 >    }
23 >
24 >    void checkNotDone(Future<?> f) {
25 >        assertFalse(f.isDone());
26 >        assertFalse(f.isCancelled());
27 >    }
28 >
29 >    <T> void checkCompletedNormally(Future<T> f, T expected) {
30 >        assertTrue(f.isDone());
31 >        assertFalse(f.isCancelled());
32 >
33 >        try {
34 >            assertSame(expected, f.get());
35 >        } catch (Throwable fail) { threadUnexpectedException(fail); }
36 >        try {
37 >            assertSame(expected, f.get(5L, SECONDS));
38 >        } catch (Throwable fail) { threadUnexpectedException(fail); }
39 >
40 >        assertFalse(f.cancel(false));
41 >        assertFalse(f.cancel(true));
42 >    }
43 >
44 >    void checkCancelled(Future<?> f) {
45 >        assertTrue(f.isDone());
46 >        assertTrue(f.isCancelled());
47 >
48 >        try {
49 >            f.get();
50 >            shouldThrow();
51 >        } catch (CancellationException success) {
52 >        } catch (Throwable fail) { threadUnexpectedException(fail); }
53 >
54 >        try {
55 >            f.get(5L, SECONDS);
56 >            shouldThrow();
57 >        } catch (CancellationException success) {
58 >        } catch (Throwable fail) { threadUnexpectedException(fail); }
59 >
60 >        assertFalse(f.cancel(false));
61 >        assertFalse(f.cancel(true));
62 >    }
63 >
64 >    void checkCompletedAbnormally(Future<?> f, Throwable t) {
65 >        assertTrue(f.isDone());
66 >        assertFalse(f.isCancelled());
67 >
68 >        try {
69 >            f.get();
70 >            shouldThrow();
71 >        } catch (ExecutionException success) {
72 >            assertSame(t, success.getCause());
73 >        } catch (Throwable fail) { threadUnexpectedException(fail); }
74 >
75 >        try {
76 >            f.get(5L, SECONDS);
77 >            shouldThrow();
78 >        } catch (ExecutionException success) {
79 >            assertSame(t, success.getCause());
80 >        } catch (Throwable fail) { threadUnexpectedException(fail); }
81 >
82 >        assertFalse(f.cancel(false));
83 >        assertFalse(f.cancel(true));
84      }
85  
86      /**
87       * Subclass to expose protected methods
88       */
89 <    static class MyFutureTask extends FutureTask {
90 <        public MyFutureTask(Callable r) { super(r); }
91 <        public boolean reset() { return super.reset(); }
27 <        public void setCancelled() { super.setCancelled(); }
28 <        public void setDone() { super.setDone(); }
89 >    static class PublicFutureTask extends FutureTask {
90 >        public PublicFutureTask(Callable r) { super(r); }
91 >        public boolean runAndReset() { return super.runAndReset(); }
92          public void set(Object x) { super.set(x); }
93          public void setException(Throwable t) { super.setException(t); }
94      }
95  
96      /**
97 <     *
97 >     * Creating a future with a null callable throws NPE
98       */
99      public void testConstructor() {
100          try {
101              FutureTask task = new FutureTask(null);
102              shouldThrow();
103 <        }
41 <        catch(NullPointerException success) {
42 <        }
103 >        } catch (NullPointerException success) {}
104      }
105  
106      /**
107 <     *
107 >     * creating a future with null runnable fails
108       */
109      public void testConstructor2() {
110          try {
111              FutureTask task = new FutureTask(null, Boolean.TRUE);
112              shouldThrow();
113 <        }
53 <        catch(NullPointerException success) {
54 <        }
113 >        } catch (NullPointerException success) {}
114      }
115  
116      /**
117 <     *
117 >     * isDone is true when a task completes
118       */
119      public void testIsDone() {
120 <        FutureTask task = new FutureTask( new NoOpCallable());
121 <        task.run();
122 <        assertTrue(task.isDone());
123 <        assertFalse(task.isCancelled());
120 >        FutureTask task = new FutureTask(new NoOpCallable());
121 >        task.run();
122 >        assertTrue(task.isDone());
123 >        checkCompletedNormally(task, Boolean.TRUE);
124      }
125  
126      /**
127 <     *
127 >     * runAndReset of a non-cancelled task succeeds
128       */
129 <    public void testReset() {
130 <        MyFutureTask task = new MyFutureTask(new NoOpCallable());
131 <        task.run();
132 <        assertTrue(task.isDone());
74 <        assertTrue(task.reset());
129 >    public void testRunAndReset() {
130 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
131 >        assertTrue(task.runAndReset());
132 >        checkNotDone(task);
133      }
134  
135      /**
136 <     *
136 >     * runAndReset after cancellation fails
137       */
138      public void testResetAfterCancel() {
139 <        MyFutureTask task = new MyFutureTask(new NoOpCallable());
139 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
140          assertTrue(task.cancel(false));
141 <        task.run();
142 <        assertTrue(task.isDone());
85 <        assertTrue(task.isCancelled());
86 <        assertFalse(task.reset());
87 <    }
88 <
89 <    /**
90 <     *
91 <     */
92 <    public void testSetDone() {
93 <        MyFutureTask task = new MyFutureTask(new NoOpCallable());
94 <        task.setDone();
95 <        assertTrue(task.isDone());
96 <        assertFalse(task.isCancelled());
141 >        assertFalse(task.runAndReset());
142 >        checkCancelled(task);
143      }
144  
99    /**
100     *
101     */
102    public void testSetCancelled() {
103        MyFutureTask task = new MyFutureTask(new NoOpCallable());
104        assertTrue(task.cancel(false));
105        task.setCancelled();
106        assertTrue(task.isDone());
107        assertTrue(task.isCancelled());
108    }
145  
146      /**
147 <     *
147 >     * setting value causes get to return it
148       */
149 <    public void testSet() {
150 <        MyFutureTask task = new MyFutureTask(new NoOpCallable());
149 >    public void testSet() throws Exception {
150 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
151          task.set(one);
152 <        try {
153 <            assertEquals(task.get(), one);
118 <        }
119 <        catch(Exception e) {
120 <            unexpectedException();
121 <        }
152 >        assertSame(task.get(), one);
153 >        checkCompletedNormally(task, one);
154      }
155  
156      /**
157 <     *
157 >     * setException causes get to throw ExecutionException
158       */
159 <    public void testSetException() {
159 >    public void testSetException() throws Exception {
160          Exception nse = new NoSuchElementException();
161 <        MyFutureTask task = new MyFutureTask(new NoOpCallable());
161 >        PublicFutureTask task = new PublicFutureTask(new NoOpCallable());
162          task.setException(nse);
163          try {
164              Object x = task.get();
165              shouldThrow();
166 <        }
167 <        catch(ExecutionException ee) {
168 <            Throwable cause = ee.getCause();
137 <            assertEquals(cause, nse);
138 <        }
139 <        catch(Exception e) {
140 <            unexpectedException();
166 >        } catch (ExecutionException success) {
167 >            assertSame(success.getCause(), nse);
168 >            checkCompletedAbnormally(task, nse);
169          }
170      }
171  
172      /**
173 <     *
173 >     * Cancelling before running succeeds
174       */
175      public void testCancelBeforeRun() {
176 <        FutureTask task = new FutureTask( new NoOpCallable());
176 >        FutureTask task = new FutureTask(new NoOpCallable());
177          assertTrue(task.cancel(false));
178 <        task.run();
179 <        assertTrue(task.isDone());
152 <        assertTrue(task.isCancelled());
178 >        task.run();
179 >        checkCancelled(task);
180      }
181  
182      /**
183 <     *
183 >     * Cancel(true) before run succeeds
184       */
185      public void testCancelBeforeRun2() {
186 <        FutureTask task = new FutureTask( new NoOpCallable());
186 >        FutureTask task = new FutureTask(new NoOpCallable());
187          assertTrue(task.cancel(true));
188 <        task.run();
189 <        assertTrue(task.isDone());
163 <        assertTrue(task.isCancelled());
188 >        task.run();
189 >        checkCancelled(task);
190      }
191  
192      /**
193 <     *
193 >     * cancel of a completed task fails
194       */
195      public void testCancelAfterRun() {
196 <        FutureTask task = new FutureTask( new NoOpCallable());
197 <        task.run();
196 >        FutureTask task = new FutureTask(new NoOpCallable());
197 >        task.run();
198          assertFalse(task.cancel(false));
199 <        assertTrue(task.isDone());
174 <        assertFalse(task.isCancelled());
199 >        checkCompletedNormally(task, Boolean.TRUE);
200      }
201  
202      /**
203 <     *
203 >     * cancel(true) interrupts a running task
204       */
205 <    public void testCancelInterrupt() {
206 <        FutureTask task = new FutureTask( new Callable() {
207 <                public Object call() {
208 <                    try {
209 <                        Thread.sleep(MEDIUM_DELAY_MS);
210 <                        threadShouldThrow();
205 >    public void testCancelInterrupt() throws InterruptedException {
206 >        final CountDownLatch threadStarted = new CountDownLatch(1);
207 >        final FutureTask task =
208 >            new FutureTask(new CheckedCallable<Object>() {
209 >                public Object realCall() {
210 >                    threadStarted.countDown();
211 >                    long t0 = System.nanoTime();
212 >                    for (;;) {
213 >                        if (Thread.interrupted())
214 >                            return Boolean.TRUE;
215 >                        if (millisElapsedSince(t0) > MEDIUM_DELAY_MS)
216 >                            fail("interrupt not delivered");
217 >                        Thread.yield();
218                      }
219 <                    catch (InterruptedException success) {}
220 <                    return Boolean.TRUE;
221 <                } });
222 <        Thread t = new  Thread(task);
223 <        t.start();
224 <        
225 <        try {
226 <            Thread.sleep(SHORT_DELAY_MS);
195 <            assertTrue(task.cancel(true));
196 <            t.join();
197 <            assertTrue(task.isDone());
198 <            assertTrue(task.isCancelled());
199 <        } catch(InterruptedException e){
200 <            unexpectedException();
201 <        }
219 >                }});
220 >
221 >        Thread t = newStartedThread(task);
222 >        threadStarted.await();
223 >        assertTrue(task.cancel(true));
224 >        checkCancelled(task);
225 >        awaitTermination(t, MEDIUM_DELAY_MS);
226 >        checkCancelled(task);
227      }
228  
229 +    /**
230 +     * cancel(false) does not interrupt a running task
231 +     */
232 +    public void testCancelNoInterrupt() throws InterruptedException {
233 +        final CountDownLatch threadStarted = new CountDownLatch(1);
234 +        final CountDownLatch cancelled = new CountDownLatch(1);
235 +        final FutureTask<Boolean> task =
236 +            new FutureTask<Boolean>(new CheckedCallable<Boolean>() {
237 +                public Boolean realCall() throws InterruptedException {
238 +                    threadStarted.countDown();
239 +                    cancelled.await(MEDIUM_DELAY_MS, MILLISECONDS);
240 +                    assertFalse(Thread.interrupted());
241 +                    return Boolean.TRUE;
242 +                }});
243 +
244 +        Thread t = newStartedThread(task);
245 +        threadStarted.await();
246 +        assertTrue(task.cancel(false));
247 +        checkCancelled(task);
248 +        cancelled.countDown();
249 +        awaitTermination(t, MEDIUM_DELAY_MS);
250 +        checkCancelled(task);
251 +    }
252  
253      /**
254 <     *
254 >     * set in one thread causes get in another thread to retrieve value
255       */
256 <    public void testCancelNoInterrupt() {
257 <        FutureTask task = new FutureTask( new Callable() {
258 <                public Object call() {
259 <                    try {
212 <                        Thread.sleep(MEDIUM_DELAY_MS);
213 <                    }
214 <                    catch (InterruptedException success) {
215 <                        threadFail("should not interrupt");
216 <                    }
256 >    public void testGet1() throws InterruptedException {
257 >        final FutureTask task =
258 >            new FutureTask(new CheckedCallable<Object>() {
259 >                public Object realCall() throws InterruptedException {
260                      return Boolean.TRUE;
261 <                } });
262 <        Thread t = new  Thread(task);
263 <        t.start();
264 <        
265 <        try {
266 <            Thread.sleep(SHORT_DELAY_MS);
267 <            assertTrue(task.cancel(false));
268 <            t.join();
269 <            assertTrue(task.isDone());
270 <            assertTrue(task.isCancelled());
271 <        } catch(InterruptedException e){
229 <            unexpectedException();
230 <        }
261 >                }});
262 >        checkNotDone(task);
263 >
264 >        Thread t = newStartedThread(new CheckedRunnable() {
265 >            public void realRun() throws Exception {
266 >                assertSame(Boolean.TRUE, task.get());
267 >            }});
268 >
269 >        task.run();
270 >        checkCompletedNormally(task, Boolean.TRUE);
271 >        awaitTermination(t, MEDIUM_DELAY_MS);
272      }
273  
274      /**
275 <     *
275 >     * set in one thread causes timed get in another thread to retrieve value
276       */
277 <    public void testGet1() {
278 <        final FutureTask ft = new FutureTask(new Callable() {
279 <                public Object call() {
280 <                    try {
240 <                        Thread.sleep(MEDIUM_DELAY_MS);
241 <                    } catch(InterruptedException e){
242 <                        threadUnexpectedException();
243 <                    }
277 >    public void testTimedGet1() throws InterruptedException {
278 >        final FutureTask task =
279 >            new FutureTask(new CheckedCallable<Object>() {
280 >                public Object realCall() throws InterruptedException {
281                      return Boolean.TRUE;
282 <                }
283 <        });
284 <        Thread t = new Thread(new Runnable() {
285 <                public void run() {
286 <                    try {
287 <                        ft.get();
288 <                    } catch(Exception e){
289 <                        threadUnexpectedException();
290 <                    }
291 <                }
292 <            });
256 <        try {
257 <            assertFalse(ft.isDone());
258 <            assertFalse(ft.isCancelled());
259 <            t.start();
260 <            Thread.sleep(SHORT_DELAY_MS);
261 <            ft.run();
262 <            t.join();
263 <            assertTrue(ft.isDone());
264 <            assertFalse(ft.isCancelled());
265 <        } catch(InterruptedException e){
266 <            unexpectedException();
267 <
268 <        }      
282 >                }});
283 >        checkNotDone(task);
284 >
285 >        Thread t = newStartedThread(new CheckedRunnable() {
286 >            public void realRun() throws Exception {
287 >                assertSame(Boolean.TRUE, task.get(SMALL_DELAY_MS, MILLISECONDS));
288 >            }});
289 >
290 >        task.run();
291 >        checkCompletedNormally(task, Boolean.TRUE);
292 >        awaitTermination(t, MEDIUM_DELAY_MS);
293      }
294  
295      /**
296 <     *
297 <     */
298 <    public void testTimedGet1() {
299 <        final FutureTask ft = new FutureTask(new Callable() {
300 <                public Object call() {
301 <                    try {
302 <                        Thread.sleep(MEDIUM_DELAY_MS);
303 <                    } catch(InterruptedException e){
304 <                        threadUnexpectedException();
305 <                    }
296 >     * Cancelling a task causes timed get in another thread to throw
297 >     * CancellationException
298 >     */
299 >    public void testTimedGet_Cancellation() throws InterruptedException {
300 >        final CountDownLatch threadStarted = new CountDownLatch(2);
301 >        final FutureTask task =
302 >            new FutureTask(new CheckedInterruptedCallable<Object>() {
303 >                public Object realCall() throws InterruptedException {
304 >                    threadStarted.countDown();
305 >                    Thread.sleep(LONG_DELAY_MS);
306                      return Boolean.TRUE;
307 <                }
284 <            });
285 <        Thread t = new Thread(new Runnable() {
286 <                public void run() {
287 <                    try {
288 <                        ft.get(SHORT_DELAY_MS, TimeUnit.MILLISECONDS);
289 <                    } catch(TimeoutException success) {
290 <                    } catch(Exception e){
291 <                        threadUnexpectedException();
292 <                    }
293 <                }
294 <            });
295 <        try {
296 <            assertFalse(ft.isDone());
297 <            assertFalse(ft.isCancelled());
298 <            t.start();
299 <            ft.run();
300 <            t.join();
301 <            assertTrue(ft.isDone());
302 <            assertFalse(ft.isCancelled());
303 <        } catch(InterruptedException e){
304 <            unexpectedException();
305 <            
306 <        }      
307 <    }
307 >                }});
308  
309 +        Thread t1 = new ThreadShouldThrow(CancellationException.class) {
310 +            public void realRun() throws Exception {
311 +                threadStarted.countDown();
312 +                task.get(MEDIUM_DELAY_MS, MILLISECONDS);
313 +            }};
314 +        Thread t2 = new Thread(task);
315 +        t1.start();
316 +        t2.start();
317 +        threadStarted.await();
318 +        task.cancel(true);
319 +        awaitTermination(t1, MEDIUM_DELAY_MS);
320 +        awaitTermination(t2, MEDIUM_DELAY_MS);
321 +        checkCancelled(task);
322 +    }
323  
324      /**
325 <     *
326 <     */
327 <    public void testGet_Cancellation() {
328 <        final FutureTask ft = new FutureTask(new Callable() {
329 <                public Object call() {
330 <                    try {
331 <                        Thread.sleep(MEDIUM_DELAY_MS);
332 <                    } catch(InterruptedException e){
333 <                        threadUnexpectedException();
334 <                    }
325 >     * Cancelling a task causes get in another thread to throw
326 >     * CancellationException
327 >     */
328 >    public void testGet_Cancellation() throws InterruptedException {
329 >        final CountDownLatch threadStarted = new CountDownLatch(2);
330 >        final FutureTask task =
331 >            new FutureTask(new CheckedInterruptedCallable<Object>() {
332 >                public Object realCall() throws InterruptedException {
333 >                    threadStarted.countDown();
334 >                    Thread.sleep(LONG_DELAY_MS);
335                      return Boolean.TRUE;
336 <                }
337 <            });
338 <        try {
339 <            Thread.sleep(SHORT_DELAY_MS);
340 <            Thread t = new Thread(new Runnable() {
341 <                    public void run() {
342 <                        try {
343 <                            ft.get();
344 <                            threadShouldThrow();
345 <                        } catch(CancellationException success){
346 <                        }
347 <                        catch(Exception e){
348 <                            threadUnexpectedException();
349 <                        }
350 <                    }
337 <                });
338 <            t.start();
339 <            ft.cancel(true);
340 <            t.join();
341 <        } catch(InterruptedException success){
342 <            unexpectedException();
343 <        }
336 >                }});
337 >
338 >        Thread t1 = new ThreadShouldThrow(CancellationException.class) {
339 >            public void realRun() throws Exception {
340 >                threadStarted.countDown();
341 >                task.get();
342 >            }};
343 >        Thread t2 = new Thread(task);
344 >        t1.start();
345 >        t2.start();
346 >        threadStarted.await();
347 >        task.cancel(true);
348 >        awaitTermination(t1, MEDIUM_DELAY_MS);
349 >        awaitTermination(t2, MEDIUM_DELAY_MS);
350 >        checkCancelled(task);
351      }
352 <    
352 >
353 >
354      /**
355 <     *
356 <     */
357 <    public void testGet_Cancellation2() {
358 <        final FutureTask ft = new FutureTask(new Callable() {
359 <                public Object call() {
360 <                    try {
361 <                        Thread.sleep(SHORT_DELAY_MS);
362 <                    } catch(InterruptedException e) {
363 <                        threadUnexpectedException();
364 <                    }
365 <                    return Boolean.TRUE;
366 <                }
367 <            });
368 <        try {
369 <            Thread.sleep(SHORT_DELAY_MS);
362 <            Thread t = new Thread(new Runnable() {
363 <                    public void run() {
364 <                        try {
365 <                            ft.get(MEDIUM_DELAY_MS, TimeUnit.MILLISECONDS);
366 <                            threadShouldThrow();
367 <                        } catch(CancellationException success) {}
368 <                        catch(Exception e){
369 <                            threadUnexpectedException();
370 <                        }
371 <                    }
372 <                });
373 <            t.start();
374 <            Thread.sleep(SHORT_DELAY_MS);
375 <            ft.cancel(true);
376 <            Thread.sleep(SHORT_DELAY_MS);
377 <            t.join();
378 <        } catch(InterruptedException ie){
379 <            unexpectedException();
355 >     * A runtime exception in task causes get to throw ExecutionException
356 >     */
357 >    public void testGet_ExecutionException() throws InterruptedException {
358 >        final FutureTask task = new FutureTask(new Callable() {
359 >            public Object call() {
360 >                return 5/0;
361 >            }});
362 >
363 >        task.run();
364 >        try {
365 >            task.get();
366 >            shouldThrow();
367 >        } catch (ExecutionException success) {
368 >            assertTrue(success.getCause() instanceof ArithmeticException);
369 >            checkCompletedAbnormally(task, success.getCause());
370          }
371      }
372  
373      /**
374 <     *
374 >     * A runtime exception in task causes timed get to throw ExecutionException
375       */
376 <    public void testGet_ExecutionException() {
377 <        final FutureTask ft = new FutureTask(new Callable() {
378 <                public Object call() {
379 <                    int i = 5/0;
380 <                    return Boolean.TRUE;
381 <                }
382 <            });
393 <        try {
394 <            ft.run();
395 <            ft.get();
396 <            shouldThrow();
397 <        } catch(ExecutionException success){
398 <        }
399 <        catch(Exception e){
400 <            unexpectedException();
401 <        }
402 <    }
403 <  
404 <    /**
405 <     *
406 <     */
407 <    public void testTimedGet_ExecutionException2() {
408 <        final FutureTask ft = new FutureTask(new Callable() {
409 <                public Object call() {
410 <                    int i = 5/0;
411 <                    return Boolean.TRUE;
412 <                }
413 <            });
414 <        try {
415 <            ft.run();
416 <            ft.get(SHORT_DELAY_MS, TimeUnit.MILLISECONDS);
417 <            shouldThrow();
418 <        } catch(ExecutionException success) {
419 <        } catch(TimeoutException success) { } // unlikely but OK
420 <        catch(Exception e){
421 <            unexpectedException();
422 <        }
423 <    }
424 <      
425 <
426 <    /**
427 <     *
428 <     */
429 <    public void testGet_InterruptedException() {
430 <        final FutureTask ft = new FutureTask(new NoOpCallable());
431 <        Thread t = new Thread(new Runnable() {
432 <                public void run() {                
433 <                    try {
434 <                        ft.get();
435 <                        threadShouldThrow();
436 <                    } catch(InterruptedException success){
437 <                    } catch(Exception e){
438 <                        threadUnexpectedException();
439 <                    }
440 <                }
441 <            });
376 >    public void testTimedGet_ExecutionException2() throws Exception {
377 >        final FutureTask task = new FutureTask(new Callable() {
378 >            public Object call() {
379 >                return 5/0;
380 >            }});
381 >
382 >        task.run();
383          try {
384 <            t.start();
385 <            Thread.sleep(SHORT_DELAY_MS);
386 <            t.interrupt();
387 <            t.join();
388 <        } catch(Exception e){
448 <            unexpectedException();
384 >            task.get(SHORT_DELAY_MS, MILLISECONDS);
385 >            shouldThrow();
386 >        } catch (ExecutionException success) {
387 >            assertTrue(success.getCause() instanceof ArithmeticException);
388 >            checkCompletedAbnormally(task, success.getCause());
389          }
390      }
391  
392 +
393 +    /**
394 +     * Interrupting a waiting get causes it to throw InterruptedException
395 +     */
396 +    public void testGet_InterruptedException() throws InterruptedException {
397 +        final CountDownLatch threadStarted = new CountDownLatch(1);
398 +        final FutureTask task = new FutureTask(new NoOpCallable());
399 +        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
400 +            public void realRun() throws Exception {
401 +                threadStarted.countDown();
402 +                task.get();
403 +            }});
404 +
405 +        threadStarted.await();
406 +        t.interrupt();
407 +        awaitTermination(t, MEDIUM_DELAY_MS);
408 +        checkNotDone(task);
409 +    }
410 +
411      /**
412 <     *
412 >     * Interrupting a waiting timed get causes it to throw InterruptedException
413       */
414 <    public void testTimedGet_InterruptedException2() {
415 <        final FutureTask ft = new FutureTask(new NoOpCallable());
416 <        Thread t = new Thread(new Runnable() {
417 <                public void run() {                
418 <                    try {
419 <                        ft.get(LONG_DELAY_MS,TimeUnit.MILLISECONDS);
420 <                        threadShouldThrow();
421 <                    } catch(InterruptedException success){}
422 <                    catch(Exception e){
423 <                        threadUnexpectedException();
424 <                    }
425 <                }
426 <            });
468 <        try {
469 <            t.start();
470 <            Thread.sleep(SHORT_DELAY_MS);
471 <            t.interrupt();
472 <            t.join();
473 <        } catch(Exception e){
474 <            unexpectedException();
475 <        }
414 >    public void testTimedGet_InterruptedException2() throws InterruptedException {
415 >        final CountDownLatch threadStarted = new CountDownLatch(1);
416 >        final FutureTask task = new FutureTask(new NoOpCallable());
417 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
418 >            public void realRun() throws Exception {
419 >                threadStarted.countDown();
420 >                task.get(LONG_DELAY_MS, MILLISECONDS);
421 >            }});
422 >
423 >        threadStarted.await();
424 >        t.interrupt();
425 >        awaitTermination(t, MEDIUM_DELAY_MS);
426 >        checkNotDone(task);
427      }
428 <    
428 >
429      /**
430 <     *
430 >     * A timed out timed get throws TimeoutException
431       */
432 <    public void testGet_TimeoutException() {
433 <        try {
434 <            FutureTask ft = new FutureTask(new NoOpCallable());
435 <            ft.get(1,TimeUnit.MILLISECONDS);
436 <            shouldThrow();
437 <        } catch(TimeoutException success){}
487 <        catch(Exception success){
488 <            unexpectedException();
489 <        }
432 >    public void testGet_TimeoutException() throws Exception {
433 >        try {
434 >            FutureTask task = new FutureTask(new NoOpCallable());
435 >            task.get(1, MILLISECONDS);
436 >            shouldThrow();
437 >        } catch (TimeoutException success) {}
438      }
439 <    
439 >
440   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines