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

Comparing jsr166/src/test/tck/ReentrantLockTest.java (file contents):
Revision 1.24 by dl, Fri Feb 24 00:03:16 2006 UTC vs.
Revision 1.44 by jsr166, Sat May 7 03:40:45 2011 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
5 < * Other contributors include Andrew Wright, Jeffrey Hayes,
6 < * Pat Fisher, Mike Judd.
4 > * http://creativecommons.org/publicdomain/zero/1.0/
5 > * Other contributors include Andrew Wright, Jeffrey Hayes,
6 > * Pat Fisher, Mike Judd.
7   */
8  
9   import junit.framework.*;
10   import java.util.concurrent.locks.*;
11   import java.util.concurrent.*;
12 + import static java.util.concurrent.TimeUnit.MILLISECONDS;
13   import java.util.*;
14   import java.io.*;
15  
16   public class ReentrantLockTest extends JSR166TestCase {
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(ReentrantLockTest.class);
21 >        return new TestSuite(ReentrantLockTest.class);
22      }
23  
24      /**
25       * A runnable calling lockInterruptibly
26       */
27 <    class InterruptibleLockRunnable implements Runnable {
27 >    class InterruptibleLockRunnable extends CheckedRunnable {
28          final ReentrantLock lock;
29          InterruptibleLockRunnable(ReentrantLock l) { lock = l; }
30 <        public void run() {
31 <            try {
31 <                lock.lockInterruptibly();
32 <            } catch(InterruptedException success){}
30 >        public void realRun() throws InterruptedException {
31 >            lock.lockInterruptibly();
32          }
33      }
34  
# Line 38 | Line 37 | public class ReentrantLockTest extends J
37       * A runnable calling lockInterruptibly that expects to be
38       * interrupted
39       */
40 <    class InterruptedLockRunnable implements Runnable {
40 >    class InterruptedLockRunnable extends CheckedInterruptedRunnable {
41          final ReentrantLock lock;
42          InterruptedLockRunnable(ReentrantLock l) { lock = l; }
43 <        public void run() {
44 <            try {
46 <                lock.lockInterruptibly();
47 <                threadShouldThrow();
48 <            } catch(InterruptedException success){}
43 >        public void realRun() throws InterruptedException {
44 >            lock.lockInterruptibly();
45          }
46      }
47  
# Line 54 | Line 50 | public class ReentrantLockTest extends J
50       */
51      static class PublicReentrantLock extends ReentrantLock {
52          PublicReentrantLock() { super(); }
53 <        public Collection<Thread> getQueuedThreads() {
54 <            return super.getQueuedThreads();
53 >        public Collection<Thread> getQueuedThreads() {
54 >            return super.getQueuedThreads();
55          }
56 <        public Collection<Thread> getWaitingThreads(Condition c) {
57 <            return super.getWaitingThreads(c);
56 >        public Collection<Thread> getWaitingThreads(Condition c) {
57 >            return super.getWaitingThreads(c);
58          }
59 +    }
60  
61 +    /**
62 +     * Checks that condition c has no waiters.
63 +     */
64 +    void assertHasNoWaiters(PublicReentrantLock lock, Condition c) {
65 +        assertHasWaiters(lock, c, new Thread[] {});
66 +    }
67  
68 +    /**
69 +     * Checks that condition c has exactly the given waiter threads.
70 +     */
71 +    void assertHasWaiters(PublicReentrantLock lock, Condition c,
72 +                          Thread... threads) {
73 +        lock.lock();
74 +        assertEquals(threads.length > 0, lock.hasWaiters(c));
75 +        assertEquals(threads.length, lock.getWaitQueueLength(c));
76 +        assertEquals(threads.length == 0, lock.getWaitingThreads(c).isEmpty());
77 +        assertEquals(threads.length, lock.getWaitingThreads(c).size());
78 +        assertEquals(new HashSet<Thread>(lock.getWaitingThreads(c)),
79 +                     new HashSet<Thread>(Arrays.asList(threads)));
80 +        lock.unlock();
81      }
82  
83      /**
84       * Constructor sets given fairness
85       */
86 <    public void testConstructor() {
87 <        ReentrantLock rl = new ReentrantLock();
88 <        assertFalse(rl.isFair());
89 <        ReentrantLock r2 = new ReentrantLock(true);
74 <        assertTrue(r2.isFair());
86 >    public void testConstructor() {
87 >        assertFalse(new ReentrantLock().isFair());
88 >        assertFalse(new ReentrantLock(false).isFair());
89 >        assertTrue(new ReentrantLock(true).isFair());
90      }
91  
92      /**
93       * locking an unlocked lock succeeds
94       */
95 <    public void testLock() {
96 <        ReentrantLock rl = new ReentrantLock();
95 >    public void testLock() {
96 >        ReentrantLock rl = new ReentrantLock();
97          rl.lock();
98          assertTrue(rl.isLocked());
99          rl.unlock();
100 +        assertFalse(rl.isLocked());
101      }
102  
103      /**
104       * locking an unlocked fair lock succeeds
105       */
106 <    public void testFairLock() {
107 <        ReentrantLock rl = new ReentrantLock(true);
106 >    public void testFairLock() {
107 >        ReentrantLock rl = new ReentrantLock(true);
108          rl.lock();
109          assertTrue(rl.isLocked());
110          rl.unlock();
# Line 97 | Line 113 | public class ReentrantLockTest extends J
113      /**
114       * Unlocking an unlocked lock throws IllegalMonitorStateException
115       */
116 <    public void testUnlock_IllegalMonitorStateException() {
117 <        ReentrantLock rl = new ReentrantLock();
118 <        try {
119 <            rl.unlock();
120 <            shouldThrow();
121 <
106 <        } catch(IllegalMonitorStateException success){}
116 >    public void testUnlock_IllegalMonitorStateException() {
117 >        ReentrantLock rl = new ReentrantLock();
118 >        try {
119 >            rl.unlock();
120 >            shouldThrow();
121 >        } catch (IllegalMonitorStateException success) {}
122      }
123  
124      /**
125       * tryLock on an unlocked lock succeeds
126       */
127 <    public void testTryLock() {
128 <        ReentrantLock rl = new ReentrantLock();
127 >    public void testTryLock() {
128 >        ReentrantLock rl = new ReentrantLock();
129          assertTrue(rl.tryLock());
130          assertTrue(rl.isLocked());
131          rl.unlock();
# Line 120 | Line 135 | public class ReentrantLockTest extends J
135      /**
136       * hasQueuedThreads reports whether there are waiting threads
137       */
138 <    public void testhasQueuedThreads() {
139 <        final ReentrantLock lock = new ReentrantLock();
138 >    public void testhasQueuedThreads() throws InterruptedException {
139 >        final ReentrantLock lock = new ReentrantLock();
140          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
141          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
142 <        try {
143 <            assertFalse(lock.hasQueuedThreads());
144 <            lock.lock();
145 <            t1.start();
146 <            Thread.sleep(SHORT_DELAY_MS);
147 <            assertTrue(lock.hasQueuedThreads());
148 <            t2.start();
149 <            Thread.sleep(SHORT_DELAY_MS);
150 <            assertTrue(lock.hasQueuedThreads());
151 <            t1.interrupt();
152 <            Thread.sleep(SHORT_DELAY_MS);
153 <            assertTrue(lock.hasQueuedThreads());
154 <            lock.unlock();
155 <            Thread.sleep(SHORT_DELAY_MS);
156 <            assertFalse(lock.hasQueuedThreads());
157 <            t1.join();
158 <            t2.join();
144 <        } catch(Exception e){
145 <            unexpectedException();
146 <        }
147 <    }
142 >        assertFalse(lock.hasQueuedThreads());
143 >        lock.lock();
144 >        t1.start();
145 >        delay(SHORT_DELAY_MS);
146 >        assertTrue(lock.hasQueuedThreads());
147 >        t2.start();
148 >        delay(SHORT_DELAY_MS);
149 >        assertTrue(lock.hasQueuedThreads());
150 >        t1.interrupt();
151 >        delay(SHORT_DELAY_MS);
152 >        assertTrue(lock.hasQueuedThreads());
153 >        lock.unlock();
154 >        delay(SHORT_DELAY_MS);
155 >        assertFalse(lock.hasQueuedThreads());
156 >        awaitTermination(t1);
157 >        awaitTermination(t2);
158 >    }
159  
160      /**
161       * getQueueLength reports number of waiting threads
162       */
163 <    public void testGetQueueLength() {
164 <        final ReentrantLock lock = new ReentrantLock();
163 >    public void testGetQueueLength() throws InterruptedException {
164 >        final ReentrantLock lock = new ReentrantLock();
165          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
166          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
167 <        try {
168 <            assertEquals(0, lock.getQueueLength());
169 <            lock.lock();
170 <            t1.start();
171 <            Thread.sleep(SHORT_DELAY_MS);
172 <            assertEquals(1, lock.getQueueLength());
173 <            t2.start();
174 <            Thread.sleep(SHORT_DELAY_MS);
175 <            assertEquals(2, lock.getQueueLength());
176 <            t1.interrupt();
177 <            Thread.sleep(SHORT_DELAY_MS);
178 <            assertEquals(1, lock.getQueueLength());
179 <            lock.unlock();
180 <            Thread.sleep(SHORT_DELAY_MS);
181 <            assertEquals(0, lock.getQueueLength());
182 <            t1.join();
183 <            t2.join();
173 <        } catch(Exception e){
174 <            unexpectedException();
175 <        }
176 <    }
167 >        assertEquals(0, lock.getQueueLength());
168 >        lock.lock();
169 >        t1.start();
170 >        delay(SHORT_DELAY_MS);
171 >        assertEquals(1, lock.getQueueLength());
172 >        t2.start();
173 >        delay(SHORT_DELAY_MS);
174 >        assertEquals(2, lock.getQueueLength());
175 >        t1.interrupt();
176 >        delay(SHORT_DELAY_MS);
177 >        assertEquals(1, lock.getQueueLength());
178 >        lock.unlock();
179 >        delay(SHORT_DELAY_MS);
180 >        assertEquals(0, lock.getQueueLength());
181 >        awaitTermination(t1);
182 >        awaitTermination(t2);
183 >    }
184  
185      /**
186       * getQueueLength reports number of waiting threads
187       */
188 <    public void testGetQueueLength_fair() {
189 <        final ReentrantLock lock = new ReentrantLock(true);
188 >    public void testGetQueueLength_fair() throws InterruptedException {
189 >        final ReentrantLock lock = new ReentrantLock(true);
190          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
191          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
192 <        try {
193 <            assertEquals(0, lock.getQueueLength());
194 <            lock.lock();
195 <            t1.start();
196 <            Thread.sleep(SHORT_DELAY_MS);
197 <            assertEquals(1, lock.getQueueLength());
198 <            t2.start();
199 <            Thread.sleep(SHORT_DELAY_MS);
200 <            assertEquals(2, lock.getQueueLength());
201 <            t1.interrupt();
202 <            Thread.sleep(SHORT_DELAY_MS);
203 <            assertEquals(1, lock.getQueueLength());
204 <            lock.unlock();
205 <            Thread.sleep(SHORT_DELAY_MS);
206 <            assertEquals(0, lock.getQueueLength());
207 <            t1.join();
208 <            t2.join();
202 <        } catch(Exception e){
203 <            unexpectedException();
204 <        }
205 <    }
192 >        assertEquals(0, lock.getQueueLength());
193 >        lock.lock();
194 >        t1.start();
195 >        delay(SHORT_DELAY_MS);
196 >        assertEquals(1, lock.getQueueLength());
197 >        t2.start();
198 >        delay(SHORT_DELAY_MS);
199 >        assertEquals(2, lock.getQueueLength());
200 >        t1.interrupt();
201 >        delay(SHORT_DELAY_MS);
202 >        assertEquals(1, lock.getQueueLength());
203 >        lock.unlock();
204 >        delay(SHORT_DELAY_MS);
205 >        assertEquals(0, lock.getQueueLength());
206 >        awaitTermination(t1);
207 >        awaitTermination(t2);
208 >    }
209  
210      /**
211       * hasQueuedThread(null) throws NPE
212       */
213 <    public void testHasQueuedThreadNPE() {
214 <        final ReentrantLock sync = new ReentrantLock();
213 >    public void testHasQueuedThreadNPE() {
214 >        final ReentrantLock sync = new ReentrantLock();
215          try {
216              sync.hasQueuedThread(null);
217              shouldThrow();
218 <        } catch (NullPointerException success) {
216 <        }
218 >        } catch (NullPointerException success) {}
219      }
220  
221      /**
222       * hasQueuedThread reports whether a thread is queued.
223       */
224 <    public void testHasQueuedThread() {
225 <        final ReentrantLock sync = new ReentrantLock();
224 >    public void testHasQueuedThread() throws InterruptedException {
225 >        final ReentrantLock sync = new ReentrantLock();
226          Thread t1 = new Thread(new InterruptedLockRunnable(sync));
227          Thread t2 = new Thread(new InterruptibleLockRunnable(sync));
228 <        try {
229 <            assertFalse(sync.hasQueuedThread(t1));
230 <            assertFalse(sync.hasQueuedThread(t2));
231 <            sync.lock();
232 <            t1.start();
233 <            Thread.sleep(SHORT_DELAY_MS);
234 <            assertTrue(sync.hasQueuedThread(t1));
235 <            t2.start();
236 <            Thread.sleep(SHORT_DELAY_MS);
237 <            assertTrue(sync.hasQueuedThread(t1));
238 <            assertTrue(sync.hasQueuedThread(t2));
239 <            t1.interrupt();
240 <            Thread.sleep(SHORT_DELAY_MS);
241 <            assertFalse(sync.hasQueuedThread(t1));
242 <            assertTrue(sync.hasQueuedThread(t2));
243 <            sync.unlock();
244 <            Thread.sleep(SHORT_DELAY_MS);
245 <            assertFalse(sync.hasQueuedThread(t1));
246 <            Thread.sleep(SHORT_DELAY_MS);
247 <            assertFalse(sync.hasQueuedThread(t2));
248 <            t1.join();
249 <            t2.join();
248 <        } catch(Exception e){
249 <            unexpectedException();
250 <        }
251 <    }
228 >        assertFalse(sync.hasQueuedThread(t1));
229 >        assertFalse(sync.hasQueuedThread(t2));
230 >        sync.lock();
231 >        t1.start();
232 >        delay(SHORT_DELAY_MS);
233 >        assertTrue(sync.hasQueuedThread(t1));
234 >        t2.start();
235 >        delay(SHORT_DELAY_MS);
236 >        assertTrue(sync.hasQueuedThread(t1));
237 >        assertTrue(sync.hasQueuedThread(t2));
238 >        t1.interrupt();
239 >        delay(SHORT_DELAY_MS);
240 >        assertFalse(sync.hasQueuedThread(t1));
241 >        assertTrue(sync.hasQueuedThread(t2));
242 >        sync.unlock();
243 >        delay(SHORT_DELAY_MS);
244 >        assertFalse(sync.hasQueuedThread(t1));
245 >        delay(SHORT_DELAY_MS);
246 >        assertFalse(sync.hasQueuedThread(t2));
247 >        awaitTermination(t1);
248 >        awaitTermination(t2);
249 >    }
250  
251  
252      /**
253       * getQueuedThreads includes waiting threads
254       */
255 <    public void testGetQueuedThreads() {
256 <        final PublicReentrantLock lock = new PublicReentrantLock();
255 >    public void testGetQueuedThreads() throws InterruptedException {
256 >        final PublicReentrantLock lock = new PublicReentrantLock();
257          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
258          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
259 <        try {
260 <            assertTrue(lock.getQueuedThreads().isEmpty());
261 <            lock.lock();
262 <            assertTrue(lock.getQueuedThreads().isEmpty());
263 <            t1.start();
264 <            Thread.sleep(SHORT_DELAY_MS);
265 <            assertTrue(lock.getQueuedThreads().contains(t1));
266 <            t2.start();
267 <            Thread.sleep(SHORT_DELAY_MS);
268 <            assertTrue(lock.getQueuedThreads().contains(t1));
269 <            assertTrue(lock.getQueuedThreads().contains(t2));
270 <            t1.interrupt();
271 <            Thread.sleep(SHORT_DELAY_MS);
272 <            assertFalse(lock.getQueuedThreads().contains(t1));
273 <            assertTrue(lock.getQueuedThreads().contains(t2));
274 <            lock.unlock();
275 <            Thread.sleep(SHORT_DELAY_MS);
276 <            assertTrue(lock.getQueuedThreads().isEmpty());
277 <            t1.join();
278 <            t2.join();
281 <        } catch(Exception e){
282 <            unexpectedException();
283 <        }
284 <    }
259 >        assertTrue(lock.getQueuedThreads().isEmpty());
260 >        lock.lock();
261 >        assertTrue(lock.getQueuedThreads().isEmpty());
262 >        t1.start();
263 >        delay(SHORT_DELAY_MS);
264 >        assertTrue(lock.getQueuedThreads().contains(t1));
265 >        t2.start();
266 >        delay(SHORT_DELAY_MS);
267 >        assertTrue(lock.getQueuedThreads().contains(t1));
268 >        assertTrue(lock.getQueuedThreads().contains(t2));
269 >        t1.interrupt();
270 >        delay(SHORT_DELAY_MS);
271 >        assertFalse(lock.getQueuedThreads().contains(t1));
272 >        assertTrue(lock.getQueuedThreads().contains(t2));
273 >        lock.unlock();
274 >        delay(SHORT_DELAY_MS);
275 >        assertTrue(lock.getQueuedThreads().isEmpty());
276 >        awaitTermination(t1);
277 >        awaitTermination(t2);
278 >    }
279  
280  
281      /**
282       * timed tryLock is interruptible.
283       */
284 <    public void testInterruptedException2() {
285 <        final ReentrantLock lock = new ReentrantLock();
286 <        lock.lock();
287 <        Thread t = new Thread(new Runnable() {
288 <                public void run() {
289 <                    try {
290 <                        lock.tryLock(MEDIUM_DELAY_MS,TimeUnit.MILLISECONDS);
291 <                        threadShouldThrow();
292 <                    } catch(InterruptedException success){}
293 <                }
294 <            });
301 <        try {
302 <            t.start();
303 <            t.interrupt();
304 <        } catch(Exception e){
305 <            unexpectedException();
306 <        }
284 >    public void testInterruptedException2() throws InterruptedException {
285 >        final ReentrantLock lock = new ReentrantLock();
286 >        lock.lock();
287 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
288 >            public void realRun() throws InterruptedException {
289 >                lock.tryLock(MEDIUM_DELAY_MS,MILLISECONDS);
290 >            }});
291 >
292 >        delay(SHORT_DELAY_MS);
293 >        t.interrupt();
294 >        awaitTermination(t);
295      }
296  
297  
298      /**
299       * TryLock on a locked lock fails
300       */
301 <    public void testTryLockWhenLocked() {
302 <        final ReentrantLock lock = new ReentrantLock();
303 <        lock.lock();
304 <        Thread t = new Thread(new Runnable() {
305 <                public void run() {
306 <                    threadAssertFalse(lock.tryLock());
307 <                }
308 <            });
309 <        try {
310 <            t.start();
311 <            t.join();
324 <            lock.unlock();
325 <        } catch(Exception e){
326 <            unexpectedException();
327 <        }
328 <    }
301 >    public void testTryLockWhenLocked() throws InterruptedException {
302 >        final ReentrantLock lock = new ReentrantLock();
303 >        lock.lock();
304 >        Thread t = newStartedThread(new CheckedRunnable() {
305 >            public void realRun() {
306 >                assertFalse(lock.tryLock());
307 >            }});
308 >
309 >        awaitTermination(t);
310 >        lock.unlock();
311 >    }
312  
313      /**
314       * Timed tryLock on a locked lock times out
315       */
316 <    public void testTryLock_Timeout() {
317 <        final ReentrantLock lock = new ReentrantLock();
318 <        lock.lock();
319 <        Thread t = new Thread(new Runnable() {
320 <                public void run() {
321 <                    try {
322 <                        threadAssertFalse(lock.tryLock(1, TimeUnit.MILLISECONDS));
323 <                    } catch (Exception ex) {
324 <                        threadUnexpectedException();
325 <                    }
326 <                }
327 <            });
345 <        try {
346 <            t.start();
347 <            t.join();
348 <            lock.unlock();
349 <        } catch(Exception e){
350 <            unexpectedException();
351 <        }
352 <    }
353 <    
316 >    public void testTryLock_Timeout() throws InterruptedException {
317 >        final ReentrantLock lock = new ReentrantLock();
318 >        lock.lock();
319 >        Thread t = newStartedThread(new CheckedRunnable() {
320 >            public void realRun() throws InterruptedException {
321 >                assertFalse(lock.tryLock(1, MILLISECONDS));
322 >            }});
323 >
324 >        awaitTermination(t);
325 >        lock.unlock();
326 >    }
327 >
328      /**
329       * getHoldCount returns number of recursive holds
330       */
331      public void testGetHoldCount() {
332 <        ReentrantLock lock = new ReentrantLock();
333 <        for(int i = 1; i <= SIZE; i++) {
334 <            lock.lock();
335 <            assertEquals(i,lock.getHoldCount());
336 <        }
337 <        for(int i = SIZE; i > 0; i--) {
338 <            lock.unlock();
339 <            assertEquals(i-1,lock.getHoldCount());
340 <        }
332 >        ReentrantLock lock = new ReentrantLock();
333 >        for (int i = 1; i <= SIZE; i++) {
334 >            lock.lock();
335 >            assertEquals(i, lock.getHoldCount());
336 >        }
337 >        for (int i = SIZE; i > 0; i--) {
338 >            lock.unlock();
339 >            assertEquals(i-1, lock.getHoldCount());
340 >        }
341      }
342 <    
343 <  
342 >
343 >
344      /**
345       * isLocked is true when locked and false when not
346       */
347 <    public void testIsLocked() {
348 <        final ReentrantLock lock = new ReentrantLock();
349 <        lock.lock();
350 <        assertTrue(lock.isLocked());
351 <        lock.unlock();
352 <        assertFalse(lock.isLocked());
353 <        Thread t = new Thread(new Runnable() {
354 <                public void run() {
355 <                    lock.lock();
356 <                    try {
357 <                        Thread.sleep(SMALL_DELAY_MS);
358 <                    }
359 <                    catch(Exception e) {
360 <                        threadUnexpectedException();
361 <                    }
362 <                    lock.unlock();
363 <                }
390 <            });
391 <        try {
392 <            t.start();
393 <            Thread.sleep(SHORT_DELAY_MS);
394 <            assertTrue(lock.isLocked());
395 <            t.join();
396 <            assertFalse(lock.isLocked());
397 <        } catch(Exception e){
398 <            unexpectedException();
399 <        }
347 >    public void testIsLocked() throws InterruptedException {
348 >        final ReentrantLock lock = new ReentrantLock();
349 >        lock.lock();
350 >        assertTrue(lock.isLocked());
351 >        lock.unlock();
352 >        assertFalse(lock.isLocked());
353 >        Thread t = newStartedThread(new CheckedRunnable() {
354 >            public void realRun() throws InterruptedException {
355 >                lock.lock();
356 >                delay(SMALL_DELAY_MS);
357 >                lock.unlock();
358 >            }});
359 >
360 >        delay(SHORT_DELAY_MS);
361 >        assertTrue(lock.isLocked());
362 >        awaitTermination(t);
363 >        assertFalse(lock.isLocked());
364      }
365  
366  
367      /**
368       * lockInterruptibly is interruptible.
369       */
370 <    public void testLockInterruptibly1() {
371 <        final ReentrantLock lock = new ReentrantLock();
372 <        lock.lock();
373 <        Thread t = new Thread(new InterruptedLockRunnable(lock));
374 <        try {
375 <            t.start();
376 <            Thread.sleep(SHORT_DELAY_MS);
377 <            t.interrupt();
378 <            lock.unlock();
379 <            t.join();
416 <        } catch(Exception e){
417 <            unexpectedException();
418 <        }
419 <    }
370 >    public void testLockInterruptibly1() throws InterruptedException {
371 >        final ReentrantLock lock = new ReentrantLock();
372 >        lock.lock();
373 >        Thread t = newStartedThread(new InterruptedLockRunnable(lock));
374 >        delay(SHORT_DELAY_MS);
375 >        t.interrupt();
376 >        delay(SHORT_DELAY_MS);
377 >        lock.unlock();
378 >        awaitTermination(t);
379 >    }
380  
381      /**
382       * lockInterruptibly succeeds when unlocked, else is interruptible
383       */
384 <    public void testLockInterruptibly2() {
385 <        final ReentrantLock lock = new ReentrantLock();
386 <        try {
387 <            lock.lockInterruptibly();
388 <        } catch(Exception e) {
389 <            unexpectedException();
390 <        }
391 <        Thread t = new Thread(new InterruptedLockRunnable(lock));
392 <        try {
433 <            t.start();
434 <            t.interrupt();
435 <            assertTrue(lock.isLocked());
436 <            assertTrue(lock.isHeldByCurrentThread());
437 <            t.join();
438 <        } catch(Exception e){
439 <            unexpectedException();
440 <        }
384 >    public void testLockInterruptibly2() throws InterruptedException {
385 >        final ReentrantLock lock = new ReentrantLock();
386 >        lock.lockInterruptibly();
387 >        Thread t = newStartedThread(new InterruptedLockRunnable(lock));
388 >        delay(SHORT_DELAY_MS);
389 >        t.interrupt();
390 >        assertTrue(lock.isLocked());
391 >        assertTrue(lock.isHeldByCurrentThread());
392 >        awaitTermination(t);
393      }
394  
395      /**
396       * Calling await without holding lock throws IllegalMonitorStateException
397       */
398 <    public void testAwait_IllegalMonitor() {
399 <        final ReentrantLock lock = new ReentrantLock();
398 >    public void testAwait_IllegalMonitor() throws InterruptedException {
399 >        final ReentrantLock lock = new ReentrantLock();
400          final Condition c = lock.newCondition();
401          try {
402              c.await();
403              shouldThrow();
404 <        }
453 <        catch (IllegalMonitorStateException success) {
454 <        }
455 <        catch (Exception ex) {
456 <            unexpectedException();
457 <        }
404 >        } catch (IllegalMonitorStateException success) {}
405      }
406  
407      /**
408       * Calling signal without holding lock throws IllegalMonitorStateException
409       */
410      public void testSignal_IllegalMonitor() {
411 <        final ReentrantLock lock = new ReentrantLock();
411 >        final ReentrantLock lock = new ReentrantLock();
412          final Condition c = lock.newCondition();
413          try {
414              c.signal();
415              shouldThrow();
416 <        }
470 <        catch (IllegalMonitorStateException success) {
471 <        }
472 <        catch (Exception ex) {
473 <            unexpectedException();
474 <        }
416 >        } catch (IllegalMonitorStateException success) {}
417      }
418  
419      /**
420       * awaitNanos without a signal times out
421       */
422 <    public void testAwaitNanos_Timeout() {
423 <        final ReentrantLock lock = new ReentrantLock();
422 >    public void testAwaitNanos_Timeout() throws InterruptedException {
423 >        final ReentrantLock lock = new ReentrantLock();
424          final Condition c = lock.newCondition();
425 <        try {
426 <            lock.lock();
427 <            long t = c.awaitNanos(100);
428 <            assertTrue(t <= 0);
487 <            lock.unlock();
488 <        }
489 <        catch (Exception ex) {
490 <            unexpectedException();
491 <        }
425 >        lock.lock();
426 >        long t = c.awaitNanos(100);
427 >        assertTrue(t <= 0);
428 >        lock.unlock();
429      }
430  
431      /**
432 <     *  timed await without a signal times out
432 >     * timed await without a signal times out
433       */
434 <    public void testAwait_Timeout() {
435 <        final ReentrantLock lock = new ReentrantLock();
434 >    public void testAwait_Timeout() throws InterruptedException {
435 >        final ReentrantLock lock = new ReentrantLock();
436          final Condition c = lock.newCondition();
437 <        try {
438 <            lock.lock();
439 <            c.await(SHORT_DELAY_MS, TimeUnit.MILLISECONDS);
503 <            lock.unlock();
504 <        }
505 <        catch (Exception ex) {
506 <            unexpectedException();
507 <        }
437 >        lock.lock();
438 >        assertFalse(c.await(SHORT_DELAY_MS, MILLISECONDS));
439 >        lock.unlock();
440      }
441  
442      /**
443       * awaitUntil without a signal times out
444       */
445 <    public void testAwaitUntil_Timeout() {
446 <        final ReentrantLock lock = new ReentrantLock();
445 >    public void testAwaitUntil_Timeout() throws InterruptedException {
446 >        final ReentrantLock lock = new ReentrantLock();
447          final Condition c = lock.newCondition();
448 <        try {
449 <            lock.lock();
450 <            java.util.Date d = new java.util.Date();
451 <            c.awaitUntil(new java.util.Date(d.getTime() + 10));
520 <            lock.unlock();
521 <        }
522 <        catch (Exception ex) {
523 <            unexpectedException();
524 <        }
448 >        lock.lock();
449 >        java.util.Date d = new java.util.Date();
450 >        assertFalse(c.awaitUntil(new java.util.Date(d.getTime() + 10)));
451 >        lock.unlock();
452      }
453  
454      /**
455       * await returns when signalled
456       */
457 <    public void testAwait() {
458 <        final ReentrantLock lock = new ReentrantLock();
457 >    public void testAwait() throws InterruptedException {
458 >        final ReentrantLock lock = new ReentrantLock();
459          final Condition c = lock.newCondition();
460 <        Thread t = new Thread(new Runnable() {
461 <                public void run() {
462 <                    try {
463 <                        lock.lock();
464 <                        c.await();
465 <                        lock.unlock();
539 <                    }
540 <                    catch(InterruptedException e) {
541 <                        threadUnexpectedException();
542 <                    }
543 <                }
544 <            });
460 >        Thread t = newStartedThread(new CheckedRunnable() {
461 >            public void realRun() throws InterruptedException {
462 >                lock.lock();
463 >                c.await();
464 >                lock.unlock();
465 >            }});
466  
467 <        try {
468 <            t.start();
469 <            Thread.sleep(SHORT_DELAY_MS);
470 <            lock.lock();
471 <            c.signal();
551 <            lock.unlock();
552 <            t.join(SHORT_DELAY_MS);
553 <            assertFalse(t.isAlive());
554 <        }
555 <        catch (Exception ex) {
556 <            unexpectedException();
557 <        }
467 >        delay(SHORT_DELAY_MS);
468 >        lock.lock();
469 >        c.signal();
470 >        lock.unlock();
471 >        awaitTermination(t);
472      }
473  
474      /**
475       * hasWaiters throws NPE if null
476       */
477      public void testHasWaitersNPE() {
478 <        final ReentrantLock lock = new ReentrantLock();
478 >        final ReentrantLock lock = new ReentrantLock();
479          try {
480              lock.hasWaiters(null);
481              shouldThrow();
482 <        } catch (NullPointerException success) {
569 <        } catch (Exception ex) {
570 <            unexpectedException();
571 <        }
482 >        } catch (NullPointerException success) {}
483      }
484  
485      /**
486       * getWaitQueueLength throws NPE if null
487       */
488      public void testGetWaitQueueLengthNPE() {
489 <        final ReentrantLock lock = new ReentrantLock();
489 >        final ReentrantLock lock = new ReentrantLock();
490          try {
491              lock.getWaitQueueLength(null);
492              shouldThrow();
493 <        } catch (NullPointerException success) {
583 <        } catch (Exception ex) {
584 <            unexpectedException();
585 <        }
493 >        } catch (NullPointerException success) {}
494      }
495  
496  
# Line 590 | Line 498 | public class ReentrantLockTest extends J
498       * getWaitingThreads throws NPE if null
499       */
500      public void testGetWaitingThreadsNPE() {
501 <        final PublicReentrantLock lock = new PublicReentrantLock();
501 >        final PublicReentrantLock lock = new PublicReentrantLock();
502          try {
503              lock.getWaitingThreads(null);
504              shouldThrow();
505 <        } catch (NullPointerException success) {
598 <        } catch (Exception ex) {
599 <            unexpectedException();
600 <        }
505 >        } catch (NullPointerException success) {}
506      }
507  
508  
# Line 605 | Line 510 | public class ReentrantLockTest extends J
510       * hasWaiters throws IAE if not owned
511       */
512      public void testHasWaitersIAE() {
513 <        final ReentrantLock lock = new ReentrantLock();
514 <        final Condition c = (lock.newCondition());
515 <        final ReentrantLock lock2 = new ReentrantLock();
513 >        final ReentrantLock lock = new ReentrantLock();
514 >        final Condition c = lock.newCondition();
515 >        final ReentrantLock lock2 = new ReentrantLock();
516          try {
517              lock2.hasWaiters(c);
518              shouldThrow();
519 <        } catch (IllegalArgumentException success) {
615 <        } catch (Exception ex) {
616 <            unexpectedException();
617 <        }
519 >        } catch (IllegalArgumentException success) {}
520      }
521  
522      /**
523       * hasWaiters throws IMSE if not locked
524       */
525      public void testHasWaitersIMSE() {
526 <        final ReentrantLock lock = new ReentrantLock();
527 <        final Condition c = (lock.newCondition());
526 >        final ReentrantLock lock = new ReentrantLock();
527 >        final Condition c = lock.newCondition();
528          try {
529              lock.hasWaiters(c);
530              shouldThrow();
531 <        } catch (IllegalMonitorStateException success) {
630 <        } catch (Exception ex) {
631 <            unexpectedException();
632 <        }
531 >        } catch (IllegalMonitorStateException success) {}
532      }
533  
534  
# Line 637 | Line 536 | public class ReentrantLockTest extends J
536       * getWaitQueueLength throws IAE if not owned
537       */
538      public void testGetWaitQueueLengthIAE() {
539 <        final ReentrantLock lock = new ReentrantLock();
540 <        final Condition c = (lock.newCondition());
541 <        final ReentrantLock lock2 = new ReentrantLock();
539 >        final ReentrantLock lock = new ReentrantLock();
540 >        final Condition c = lock.newCondition();
541 >        final ReentrantLock lock2 = new ReentrantLock();
542          try {
543              lock2.getWaitQueueLength(c);
544              shouldThrow();
545 <        } catch (IllegalArgumentException success) {
647 <        } catch (Exception ex) {
648 <            unexpectedException();
649 <        }
545 >        } catch (IllegalArgumentException success) {}
546      }
547  
548      /**
549       * getWaitQueueLength throws IMSE if not locked
550       */
551      public void testGetWaitQueueLengthIMSE() {
552 <        final ReentrantLock lock = new ReentrantLock();
553 <        final Condition c = (lock.newCondition());
552 >        final ReentrantLock lock = new ReentrantLock();
553 >        final Condition c = lock.newCondition();
554          try {
555              lock.getWaitQueueLength(c);
556              shouldThrow();
557 <        } catch (IllegalMonitorStateException success) {
662 <        } catch (Exception ex) {
663 <            unexpectedException();
664 <        }
557 >        } catch (IllegalMonitorStateException success) {}
558      }
559  
560  
# Line 669 | Line 562 | public class ReentrantLockTest extends J
562       * getWaitingThreads throws IAE if not owned
563       */
564      public void testGetWaitingThreadsIAE() {
565 <        final PublicReentrantLock lock = new PublicReentrantLock();    
566 <        final Condition c = (lock.newCondition());
567 <        final PublicReentrantLock lock2 = new PublicReentrantLock();    
565 >        final PublicReentrantLock lock = new PublicReentrantLock();
566 >        final Condition c = lock.newCondition();
567 >        final PublicReentrantLock lock2 = new PublicReentrantLock();
568          try {
569              lock2.getWaitingThreads(c);
570              shouldThrow();
571 <        } catch (IllegalArgumentException success) {
679 <        } catch (Exception ex) {
680 <            unexpectedException();
681 <        }
571 >        } catch (IllegalArgumentException success) {}
572      }
573  
574      /**
575       * getWaitingThreads throws IMSE if not locked
576       */
577      public void testGetWaitingThreadsIMSE() {
578 <        final PublicReentrantLock lock = new PublicReentrantLock();    
579 <        final Condition c = (lock.newCondition());
578 >        final PublicReentrantLock lock = new PublicReentrantLock();
579 >        final Condition c = lock.newCondition();
580          try {
581              lock.getWaitingThreads(c);
582              shouldThrow();
583 <        } catch (IllegalMonitorStateException success) {
694 <        } catch (Exception ex) {
695 <            unexpectedException();
696 <        }
583 >        } catch (IllegalMonitorStateException success) {}
584      }
585  
586  
700
587      /**
588       * hasWaiters returns true when a thread is waiting, else false
589       */
590 <    public void testHasWaiters() {
591 <        final ReentrantLock lock = new ReentrantLock();
590 >    public void testHasWaiters() throws InterruptedException {
591 >        final ReentrantLock lock = new ReentrantLock();
592          final Condition c = lock.newCondition();
593 <        Thread t = new Thread(new Runnable() {
594 <                public void run() {
595 <                    try {
596 <                        lock.lock();
597 <                        threadAssertFalse(lock.hasWaiters(c));
598 <                        threadAssertEquals(0, lock.getWaitQueueLength(c));
599 <                        c.await();
600 <                        lock.unlock();
715 <                    }
716 <                    catch(InterruptedException e) {
717 <                        threadUnexpectedException();
718 <                    }
719 <                }
720 <            });
593 >        Thread t = newStartedThread(new CheckedRunnable() {
594 >            public void realRun() throws InterruptedException {
595 >                lock.lock();
596 >                assertFalse(lock.hasWaiters(c));
597 >                assertEquals(0, lock.getWaitQueueLength(c));
598 >                c.await();
599 >                lock.unlock();
600 >            }});
601  
602 <        try {
603 <            t.start();
604 <            Thread.sleep(SHORT_DELAY_MS);
605 <            lock.lock();
606 <            assertTrue(lock.hasWaiters(c));
607 <            assertEquals(1, lock.getWaitQueueLength(c));
608 <            c.signal();
609 <            lock.unlock();
610 <            Thread.sleep(SHORT_DELAY_MS);
611 <            lock.lock();
612 <            assertFalse(lock.hasWaiters(c));
613 <            assertEquals(0, lock.getWaitQueueLength(c));
734 <            lock.unlock();
735 <            t.join(SHORT_DELAY_MS);
736 <            assertFalse(t.isAlive());
737 <        }
738 <        catch (Exception ex) {
739 <            unexpectedException();
740 <        }
602 >        delay(SHORT_DELAY_MS);
603 >        lock.lock();
604 >        assertTrue(lock.hasWaiters(c));
605 >        assertEquals(1, lock.getWaitQueueLength(c));
606 >        c.signal();
607 >        lock.unlock();
608 >        delay(SHORT_DELAY_MS);
609 >        lock.lock();
610 >        assertFalse(lock.hasWaiters(c));
611 >        assertEquals(0, lock.getWaitQueueLength(c));
612 >        lock.unlock();
613 >        awaitTermination(t);
614      }
615  
616      /**
617       * getWaitQueueLength returns number of waiting threads
618       */
619 <    public void testGetWaitQueueLength() {
620 <        final ReentrantLock lock = new ReentrantLock();
619 >    public void testGetWaitQueueLength() throws InterruptedException {
620 >        final ReentrantLock lock = new ReentrantLock();
621          final Condition c = lock.newCondition();
622 <        Thread t1 = new Thread(new Runnable() {
623 <                public void run() {
624 <                    try {
625 <                        lock.lock();
626 <                        threadAssertFalse(lock.hasWaiters(c));
627 <                        threadAssertEquals(0, lock.getWaitQueueLength(c));
628 <                        c.await();
629 <                        lock.unlock();
757 <                    }
758 <                    catch(InterruptedException e) {
759 <                        threadUnexpectedException();
760 <                    }
761 <                }
762 <            });
763 <
764 <        Thread t2 = new Thread(new Runnable() {
765 <                public void run() {
766 <                    try {
767 <                        lock.lock();
768 <                        threadAssertTrue(lock.hasWaiters(c));
769 <                        threadAssertEquals(1, lock.getWaitQueueLength(c));
770 <                        c.await();
771 <                        lock.unlock();
772 <                    }
773 <                    catch(InterruptedException e) {
774 <                        threadUnexpectedException();
775 <                    }
776 <                }
777 <            });
622 >        Thread t1 = newStartedThread(new CheckedRunnable() {
623 >            public void realRun() throws InterruptedException {
624 >                lock.lock();
625 >                assertFalse(lock.hasWaiters(c));
626 >                assertEquals(0, lock.getWaitQueueLength(c));
627 >                c.await();
628 >                lock.unlock();
629 >            }});
630  
631 <        try {
632 <            t1.start();
633 <            Thread.sleep(SHORT_DELAY_MS);
634 <            t2.start();
635 <            Thread.sleep(SHORT_DELAY_MS);
636 <            lock.lock();
637 <            assertTrue(lock.hasWaiters(c));
638 <            assertEquals(2, lock.getWaitQueueLength(c));
639 <            c.signalAll();
640 <            lock.unlock();
641 <            Thread.sleep(SHORT_DELAY_MS);
642 <            lock.lock();
643 <            assertFalse(lock.hasWaiters(c));
644 <            assertEquals(0, lock.getWaitQueueLength(c));
645 <            lock.unlock();
646 <            t1.join(SHORT_DELAY_MS);
647 <            t2.join(SHORT_DELAY_MS);
648 <            assertFalse(t1.isAlive());
649 <            assertFalse(t2.isAlive());
650 <        }
651 <        catch (Exception ex) {
652 <            unexpectedException();
653 <        }
631 >        delay(SHORT_DELAY_MS);
632 >
633 >        Thread t2 = newStartedThread(new CheckedRunnable() {
634 >            public void realRun() throws InterruptedException {
635 >                lock.lock();
636 >                assertTrue(lock.hasWaiters(c));
637 >                assertEquals(1, lock.getWaitQueueLength(c));
638 >                c.await();
639 >                lock.unlock();
640 >            }});
641 >
642 >        delay(SHORT_DELAY_MS);
643 >        lock.lock();
644 >        assertTrue(lock.hasWaiters(c));
645 >        assertEquals(2, lock.getWaitQueueLength(c));
646 >        c.signalAll();
647 >        lock.unlock();
648 >        delay(SHORT_DELAY_MS);
649 >        lock.lock();
650 >        assertFalse(lock.hasWaiters(c));
651 >        assertEquals(0, lock.getWaitQueueLength(c));
652 >        lock.unlock();
653 >        awaitTermination(t1);
654 >        awaitTermination(t2);
655      }
656  
657      /**
658       * getWaitingThreads returns only and all waiting threads
659       */
660 <    public void testGetWaitingThreads() {
661 <        final PublicReentrantLock lock = new PublicReentrantLock();    
660 >    public void testGetWaitingThreads() throws InterruptedException {
661 >        final PublicReentrantLock lock = new PublicReentrantLock();
662          final Condition c = lock.newCondition();
663 <        Thread t1 = new Thread(new Runnable() {
664 <                public void run() {
665 <                    try {
666 <                        lock.lock();
667 <                        threadAssertTrue(lock.getWaitingThreads(c).isEmpty());
668 <                        c.await();
669 <                        lock.unlock();
817 <                    }
818 <                    catch(InterruptedException e) {
819 <                        threadUnexpectedException();
820 <                    }
821 <                }
822 <            });
823 <
824 <        Thread t2 = new Thread(new Runnable() {
825 <                public void run() {
826 <                    try {
827 <                        lock.lock();
828 <                        threadAssertFalse(lock.getWaitingThreads(c).isEmpty());
829 <                        c.await();
830 <                        lock.unlock();
831 <                    }
832 <                    catch(InterruptedException e) {
833 <                        threadUnexpectedException();
834 <                    }
835 <                }
836 <            });
663 >        Thread t1 = new Thread(new CheckedRunnable() {
664 >            public void realRun() throws InterruptedException {
665 >                lock.lock();
666 >                assertTrue(lock.getWaitingThreads(c).isEmpty());
667 >                c.await();
668 >                lock.unlock();
669 >            }});
670  
671 <        try {
672 <            lock.lock();
673 <            assertTrue(lock.getWaitingThreads(c).isEmpty());
674 <            lock.unlock();
675 <            t1.start();
676 <            Thread.sleep(SHORT_DELAY_MS);
677 <            t2.start();
678 <            Thread.sleep(SHORT_DELAY_MS);
679 <            lock.lock();
680 <            assertTrue(lock.hasWaiters(c));
681 <            assertTrue(lock.getWaitingThreads(c).contains(t1));
682 <            assertTrue(lock.getWaitingThreads(c).contains(t2));
683 <            c.signalAll();
684 <            lock.unlock();
685 <            Thread.sleep(SHORT_DELAY_MS);
686 <            lock.lock();
687 <            assertFalse(lock.hasWaiters(c));
688 <            assertTrue(lock.getWaitingThreads(c).isEmpty());
689 <            lock.unlock();
690 <            t1.join(SHORT_DELAY_MS);
691 <            t2.join(SHORT_DELAY_MS);
692 <            assertFalse(t1.isAlive());
693 <            assertFalse(t2.isAlive());
694 <        }
695 <        catch (Exception ex) {
696 <            unexpectedException();
697 <        }
671 >        Thread t2 = new Thread(new CheckedRunnable() {
672 >            public void realRun() throws InterruptedException {
673 >                lock.lock();
674 >                assertFalse(lock.getWaitingThreads(c).isEmpty());
675 >                c.await();
676 >                lock.unlock();
677 >            }});
678 >
679 >        lock.lock();
680 >        assertTrue(lock.getWaitingThreads(c).isEmpty());
681 >        lock.unlock();
682 >        t1.start();
683 >        delay(SHORT_DELAY_MS);
684 >        t2.start();
685 >        delay(SHORT_DELAY_MS);
686 >        lock.lock();
687 >        assertTrue(lock.hasWaiters(c));
688 >        assertTrue(lock.getWaitingThreads(c).contains(t1));
689 >        assertTrue(lock.getWaitingThreads(c).contains(t2));
690 >        c.signalAll();
691 >        lock.unlock();
692 >        delay(SHORT_DELAY_MS);
693 >        lock.lock();
694 >        assertFalse(lock.hasWaiters(c));
695 >        assertTrue(lock.getWaitingThreads(c).isEmpty());
696 >        lock.unlock();
697 >        awaitTermination(t1);
698 >        awaitTermination(t2);
699      }
700  
701      /** A helper class for uninterruptible wait tests */
702 <    class UninterruptableThread extends Thread {
702 >    class UninterruptibleThread extends Thread {
703          private ReentrantLock lock;
704          private Condition c;
705 <        
705 >
706          public volatile boolean canAwake = false;
707          public volatile boolean interrupted = false;
708          public volatile boolean lockStarted = false;
709 <        
710 <        public UninterruptableThread(ReentrantLock lock, Condition c) {
709 >
710 >        public UninterruptibleThread(ReentrantLock lock, Condition c) {
711              this.lock = lock;
712              this.c = c;
713          }
714 <        
714 >
715          public synchronized void run() {
716              lock.lock();
717              lockStarted = true;
718 <            
718 >
719              while (!canAwake) {
720                  c.awaitUninterruptibly();
721              }
722 <            
722 >
723              interrupted = isInterrupted();
724              lock.unlock();
725          }
# Line 894 | Line 728 | public class ReentrantLockTest extends J
728      /**
729       * awaitUninterruptibly doesn't abort on interrupt
730       */
731 <    public void testAwaitUninterruptibly() {
731 >    public void testAwaitUninterruptibly() throws InterruptedException {
732          final ReentrantLock lock = new ReentrantLock();
733          final Condition c = lock.newCondition();
734 <        UninterruptableThread thread = new UninterruptableThread(lock, c);
734 >        UninterruptibleThread thread = new UninterruptibleThread(lock, c);
735  
736 <        try {
903 <            thread.start();
736 >        thread.start();
737  
738 <            while (!thread.lockStarted) {
739 <                Thread.sleep(100);
740 <            }
908 <
909 <            lock.lock();
910 <            try {
911 <                thread.interrupt();
912 <                thread.canAwake = true;
913 <                c.signal();
914 <            } finally {
915 <                lock.unlock();
916 <            }
738 >        while (!thread.lockStarted) {
739 >            delay(100);
740 >        }
741  
742 <            thread.join();
743 <            assertTrue(thread.interrupted);
744 <            assertFalse(thread.isAlive());
745 <        } catch (Exception ex) {
746 <            unexpectedException();
742 >        lock.lock();
743 >        try {
744 >            thread.interrupt();
745 >            thread.canAwake = true;
746 >            c.signal();
747 >        } finally {
748 >            lock.unlock();
749          }
750 +
751 +        thread.join();
752 +        assertTrue(thread.interrupted);
753 +        assertFalse(thread.isAlive());
754      }
755  
756      /**
757       * await is interruptible
758       */
759 <    public void testAwait_Interrupt() {
760 <        final ReentrantLock lock = new ReentrantLock();
759 >    public void testAwait_Interrupt() throws InterruptedException {
760 >        final PublicReentrantLock lock = new PublicReentrantLock();
761          final Condition c = lock.newCondition();
762 <        Thread t = new Thread(new Runnable() {
763 <                public void run() {
764 <                    try {
765 <                        lock.lock();
766 <                        c.await();
767 <                        lock.unlock();
768 <                        threadShouldThrow();
769 <                    }
770 <                    catch(InterruptedException success) {
771 <                    }
772 <                }
773 <            });
774 <
775 <        try {
776 <            t.start();
777 <            Thread.sleep(SHORT_DELAY_MS);
778 <            t.interrupt();
779 <            t.join(SHORT_DELAY_MS);
780 <            assertFalse(t.isAlive());
781 <        }
782 <        catch (Exception ex) {
783 <            unexpectedException();
784 <        }
762 >        final CountDownLatch locked = new CountDownLatch(1);
763 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
764 >            public void realRun() throws InterruptedException {
765 >                lock.lock();
766 >                assertTrue(lock.isLocked());
767 >                assertTrue(lock.isHeldByCurrentThread());
768 >                assertHasNoWaiters(lock, c);
769 >                locked.countDown();
770 >                try {
771 >                    c.await();
772 >                } finally {
773 >                    assertTrue(lock.isLocked());
774 >                    assertTrue(lock.isHeldByCurrentThread());
775 >                    assertHasNoWaiters(lock, c);
776 >                    lock.unlock();
777 >                    assertFalse(Thread.interrupted());
778 >                }
779 >            }});
780 >
781 >        locked.await();
782 >        assertHasWaiters(lock, c, t);
783 >        t.interrupt();
784 >        awaitTermination(t);
785 >        assertFalse(lock.isLocked());
786      }
787  
788      /**
789       * awaitNanos is interruptible
790       */
791 <    public void testAwaitNanos_Interrupt() {
792 <        final ReentrantLock lock = new ReentrantLock();
791 >    public void testAwaitNanos_Interrupt() throws InterruptedException {
792 >        final PublicReentrantLock lock = new PublicReentrantLock();
793          final Condition c = lock.newCondition();
794 <        Thread t = new Thread(new Runnable() {
795 <                public void run() {
796 <                    try {
797 <                        lock.lock();
798 <                        c.awaitNanos(1000 * 1000 * 1000); // 1 sec
799 <                        lock.unlock();
800 <                        threadShouldThrow();
801 <                    }
802 <                    catch(InterruptedException success) {
803 <                    }
804 <                }
805 <            });
806 <
807 <        try {
808 <            t.start();
809 <            Thread.sleep(SHORT_DELAY_MS);
810 <            t.interrupt();
811 <            t.join(SHORT_DELAY_MS);
812 <            assertFalse(t.isAlive());
813 <        }
814 <        catch (Exception ex) {
815 <            unexpectedException();
816 <        }
794 >        final CountDownLatch locked = new CountDownLatch(1);
795 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
796 >            public void realRun() throws InterruptedException {
797 >                lock.lock();
798 >                assertTrue(lock.isLocked());
799 >                assertTrue(lock.isHeldByCurrentThread());
800 >                assertHasNoWaiters(lock, c);
801 >                locked.countDown();
802 >                try {
803 >                    c.awaitNanos(MILLISECONDS.toNanos(LONG_DELAY_MS));
804 >                } finally {
805 >                    assertTrue(lock.isLocked());
806 >                    assertTrue(lock.isHeldByCurrentThread());
807 >                    assertHasNoWaiters(lock, c);
808 >                    lock.unlock();
809 >                    assertFalse(Thread.interrupted());
810 >                }
811 >            }});
812 >
813 >        locked.await();
814 >        assertHasWaiters(lock, c, t);
815 >        t.interrupt();
816 >        awaitTermination(t);
817 >        assertFalse(lock.isLocked());
818      }
819  
820      /**
821       * awaitUntil is interruptible
822       */
823 <    public void testAwaitUntil_Interrupt() {
824 <        final ReentrantLock lock = new ReentrantLock();
823 >    public void testAwaitUntil_Interrupt() throws InterruptedException {
824 >        final PublicReentrantLock lock = new PublicReentrantLock();
825          final Condition c = lock.newCondition();
826 <        Thread t = new Thread(new Runnable() {
827 <                public void run() {
828 <                    try {
829 <                        lock.lock();
830 <                        java.util.Date d = new java.util.Date();
831 <                        c.awaitUntil(new java.util.Date(d.getTime() + 10000));
832 <                        lock.unlock();
833 <                        threadShouldThrow();
834 <                    }
835 <                    catch(InterruptedException success) {
836 <                    }
837 <                }
838 <            });
839 <
840 <        try {
841 <            t.start();
842 <            Thread.sleep(SHORT_DELAY_MS);
843 <            t.interrupt();
844 <            t.join(SHORT_DELAY_MS);
845 <            assertFalse(t.isAlive());
846 <        }
847 <        catch (Exception ex) {
848 <            unexpectedException();
849 <        }
826 >        final CountDownLatch locked = new CountDownLatch(1);
827 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
828 >            public void realRun() throws InterruptedException {
829 >                lock.lock();
830 >                assertTrue(lock.isLocked());
831 >                assertTrue(lock.isHeldByCurrentThread());
832 >                assertHasNoWaiters(lock, c);
833 >                locked.countDown();
834 >                java.util.Date d = new java.util.Date();
835 >                try {
836 >                    c.awaitUntil(new java.util.Date(d.getTime() + 10000));
837 >                } finally {
838 >                    assertTrue(lock.isLocked());
839 >                    assertTrue(lock.isHeldByCurrentThread());
840 >                    assertHasNoWaiters(lock, c);
841 >                    lock.unlock();
842 >                    assertFalse(Thread.interrupted());
843 >                }
844 >            }});
845 >
846 >        locked.await();
847 >        assertHasWaiters(lock, c, t);
848 >        t.interrupt();
849 >        awaitTermination(t);
850 >        assertFalse(lock.isLocked());
851      }
852  
853      /**
854       * signalAll wakes up all threads
855       */
856 <    public void testSignalAll() {
857 <        final ReentrantLock lock = new ReentrantLock();
856 >    public void testSignalAll() throws InterruptedException {
857 >        final ReentrantLock lock = new ReentrantLock();
858          final Condition c = lock.newCondition();
859 <        Thread t1 = new Thread(new Runnable() {
860 <                public void run() {
861 <                    try {
862 <                        lock.lock();
863 <                        c.await();
864 <                        lock.unlock();
1032 <                    }
1033 <                    catch(InterruptedException e) {
1034 <                        threadUnexpectedException();
1035 <                    }
1036 <                }
1037 <            });
1038 <
1039 <        Thread t2 = new Thread(new Runnable() {
1040 <                public void run() {
1041 <                    try {
1042 <                        lock.lock();
1043 <                        c.await();
1044 <                        lock.unlock();
1045 <                    }
1046 <                    catch(InterruptedException e) {
1047 <                        threadUnexpectedException();
1048 <                    }
1049 <                }
1050 <            });
859 >        Thread t1 = newStartedThread(new CheckedRunnable() {
860 >            public void realRun() throws InterruptedException {
861 >                lock.lock();
862 >                c.await();
863 >                lock.unlock();
864 >            }});
865  
866 <        try {
867 <            t1.start();
868 <            t2.start();
869 <            Thread.sleep(SHORT_DELAY_MS);
870 <            lock.lock();
871 <            c.signalAll();
872 <            lock.unlock();
873 <            t1.join(SHORT_DELAY_MS);
874 <            t2.join(SHORT_DELAY_MS);
875 <            assertFalse(t1.isAlive());
876 <            assertFalse(t2.isAlive());
877 <        }
878 <        catch (Exception ex) {
1065 <            unexpectedException();
1066 <        }
866 >        Thread t2 = newStartedThread(new CheckedRunnable() {
867 >            public void realRun() throws InterruptedException {
868 >                lock.lock();
869 >                c.await();
870 >                lock.unlock();
871 >            }});
872 >
873 >        delay(SHORT_DELAY_MS);
874 >        lock.lock();
875 >        c.signalAll();
876 >        lock.unlock();
877 >        awaitTermination(t1);
878 >        awaitTermination(t2);
879      }
880  
881      /**
882       * await after multiple reentrant locking preserves lock count
883       */
884 <    public void testAwaitLockCount() {
885 <        final ReentrantLock lock = new ReentrantLock();
884 >    public void testAwaitLockCount() throws InterruptedException {
885 >        final ReentrantLock lock = new ReentrantLock();
886          final Condition c = lock.newCondition();
887 <        Thread t1 = new Thread(new Runnable() {
888 <                public void run() {
889 <                    try {
890 <                        lock.lock();
891 <                        threadAssertEquals(1, lock.getHoldCount());
892 <                        c.await();
893 <                        threadAssertEquals(1, lock.getHoldCount());
894 <                        lock.unlock();
1083 <                    }
1084 <                    catch(InterruptedException e) {
1085 <                        threadUnexpectedException();
1086 <                    }
1087 <                }
1088 <            });
1089 <
1090 <        Thread t2 = new Thread(new Runnable() {
1091 <                public void run() {
1092 <                    try {
1093 <                        lock.lock();
1094 <                        lock.lock();
1095 <                        threadAssertEquals(2, lock.getHoldCount());
1096 <                        c.await();
1097 <                        threadAssertEquals(2, lock.getHoldCount());
1098 <                        lock.unlock();
1099 <                        lock.unlock();
1100 <                    }
1101 <                    catch(InterruptedException e) {
1102 <                        threadUnexpectedException();
1103 <                    }
1104 <                }
1105 <            });
887 >        Thread t1 = newStartedThread(new CheckedRunnable() {
888 >            public void realRun() throws InterruptedException {
889 >                lock.lock();
890 >                assertEquals(1, lock.getHoldCount());
891 >                c.await();
892 >                assertEquals(1, lock.getHoldCount());
893 >                lock.unlock();
894 >            }});
895  
896 <        try {
897 <            t1.start();
898 <            t2.start();
899 <            Thread.sleep(SHORT_DELAY_MS);
900 <            lock.lock();
901 <            c.signalAll();
902 <            lock.unlock();
903 <            t1.join(SHORT_DELAY_MS);
904 <            t2.join(SHORT_DELAY_MS);
905 <            assertFalse(t1.isAlive());
906 <            assertFalse(t2.isAlive());
907 <        }
908 <        catch (Exception ex) {
909 <            unexpectedException();
910 <        }
896 >        Thread t2 = newStartedThread(new CheckedRunnable() {
897 >            public void realRun() throws InterruptedException {
898 >                lock.lock();
899 >                lock.lock();
900 >                assertEquals(2, lock.getHoldCount());
901 >                c.await();
902 >                assertEquals(2, lock.getHoldCount());
903 >                lock.unlock();
904 >                lock.unlock();
905 >            }});
906 >
907 >        delay(SHORT_DELAY_MS);
908 >        lock.lock();
909 >        c.signalAll();
910 >        lock.unlock();
911 >        awaitTermination(t1);
912 >        awaitTermination(t2);
913      }
914  
915      /**
916       * A serialized lock deserializes as unlocked
917       */
918 <    public void testSerialization() {
918 >    public void testSerialization() throws Exception {
919          ReentrantLock l = new ReentrantLock();
920          l.lock();
921          l.unlock();
922  
923 <        try {
924 <            ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
925 <            ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
926 <            out.writeObject(l);
927 <            out.close();
928 <
929 <            ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
930 <            ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
931 <            ReentrantLock r = (ReentrantLock) in.readObject();
932 <            r.lock();
933 <            r.unlock();
934 <        } catch(Exception e){
935 <            e.printStackTrace();
1145 <            unexpectedException();
1146 <        }
923 >        ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
924 >        ObjectOutputStream out =
925 >            new ObjectOutputStream(new BufferedOutputStream(bout));
926 >        out.writeObject(l);
927 >        out.close();
928 >
929 >        ByteArrayInputStream bin =
930 >            new ByteArrayInputStream(bout.toByteArray());
931 >        ObjectInputStream in =
932 >            new ObjectInputStream(new BufferedInputStream(bin));
933 >        ReentrantLock r = (ReentrantLock) in.readObject();
934 >        r.lock();
935 >        r.unlock();
936      }
937  
938      /**

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines