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

Comparing jsr166/src/test/tck/ReentrantReadWriteLockTest.java (file contents):
Revision 1.27 by dl, Thu May 18 10:29:23 2006 UTC vs.
Revision 1.47 by jsr166, Mon May 2 00:04:49 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.io.*;
14   import java.util.*;
15  
16   public class ReentrantReadWriteLockTest 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(ReentrantReadWriteLockTest.class);
21 >        return new TestSuite(ReentrantReadWriteLockTest.class);
22      }
23  
24      /**
25       * A runnable calling lockInterruptibly
26       */
27 <    class InterruptibleLockRunnable implements Runnable {
27 >    class InterruptibleLockRunnable extends CheckedRunnable {
28          final ReentrantReadWriteLock lock;
29          InterruptibleLockRunnable(ReentrantReadWriteLock l) { lock = l; }
30 <        public void run() {
31 <            try {
31 <                lock.writeLock().lockInterruptibly();
32 <            } catch(InterruptedException success){}
30 >        public void realRun() throws InterruptedException {
31 >            lock.writeLock().lockInterruptibly();
32          }
33      }
34  
# Line 38 | Line 37 | public class ReentrantReadWriteLockTest
37       * A runnable calling lockInterruptibly that expects to be
38       * interrupted
39       */
40 <    class InterruptedLockRunnable implements Runnable {
40 >    class InterruptedLockRunnable extends CheckedInterruptedRunnable {
41          final ReentrantReadWriteLock lock;
42          InterruptedLockRunnable(ReentrantReadWriteLock l) { lock = l; }
43 <        public void run() {
44 <            try {
46 <                lock.writeLock().lockInterruptibly();
47 <                threadShouldThrow();
48 <            } catch(InterruptedException success){}
43 >        public void realRun() throws InterruptedException {
44 >            lock.writeLock().lockInterruptibly();
45          }
46      }
47  
# Line 54 | Line 50 | public class ReentrantReadWriteLockTest
50       */
51      static class PublicReentrantReadWriteLock extends ReentrantReadWriteLock {
52          PublicReentrantReadWriteLock() { 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 +     * Releases lock, checking that it had a hold count of 1.
63 +     */
64 +    void releaseLock(ReentrantReadWriteLock.WriteLock lock) {
65 +        assertTrue(lock.isHeldByCurrentThread());
66 +        lock.unlock();
67 +        assertFalse(lock.isHeldByCurrentThread());
68 +    }
69 +
70 +    /**
71       * Constructor sets given fairness, and is in unlocked state
72       */
73 <    public void testConstructor() {
74 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
73 >    public void testConstructor() {
74 >        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
75          assertFalse(rl.isFair());
76          assertFalse(rl.isWriteLocked());
77          assertEquals(0, rl.getReadLockCount());
78 <        ReentrantReadWriteLock r2 = new ReentrantReadWriteLock(true);
78 >        ReentrantReadWriteLock r2 = new ReentrantReadWriteLock(true);
79          assertTrue(r2.isFair());
80          assertFalse(r2.isWriteLocked());
81          assertEquals(0, r2.getReadLockCount());
82 +        ReentrantReadWriteLock r3 = new ReentrantReadWriteLock(false);
83 +        assertFalse(r3.isFair());
84 +        assertFalse(r3.isWriteLocked());
85 +        assertEquals(0, r3.getReadLockCount());
86      }
87  
88      /**
89       * write-locking and read-locking an unlocked lock succeed
90       */
91 <    public void testLock() {
92 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
91 >    public void testLock() {
92 >        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
93          rl.writeLock().lock();
94          assertTrue(rl.isWriteLocked());
95          assertTrue(rl.isWriteLockedByCurrentThread());
# Line 105 | Line 114 | public class ReentrantReadWriteLockTest
114      /**
115       * locking an unlocked fair lock succeeds
116       */
117 <    public void testFairLock() {
118 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock(true);
117 >    public void testFairLock() {
118 >        ReentrantReadWriteLock rl = new ReentrantReadWriteLock(true);
119          rl.writeLock().lock();
120          assertTrue(rl.isWriteLocked());
121          assertTrue(rl.isWriteLockedByCurrentThread());
# Line 131 | Line 140 | public class ReentrantReadWriteLockTest
140       * getWriteHoldCount returns number of recursive holds
141       */
142      public void testGetWriteHoldCount() {
143 <        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
144 <        for(int i = 1; i <= SIZE; i++) {
145 <            lock.writeLock().lock();
146 <            assertEquals(i,lock.getWriteHoldCount());
147 <        }
148 <        for(int i = SIZE; i > 0; i--) {
149 <            lock.writeLock().unlock();
150 <            assertEquals(i-1,lock.getWriteHoldCount());
151 <        }
143 >        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
144 >        for (int i = 1; i <= SIZE; i++) {
145 >            lock.writeLock().lock();
146 >            assertEquals(i,lock.getWriteHoldCount());
147 >        }
148 >        for (int i = SIZE; i > 0; i--) {
149 >            lock.writeLock().unlock();
150 >            assertEquals(i-1,lock.getWriteHoldCount());
151 >        }
152      }
153  
154      /**
155       * WriteLock.getHoldCount returns number of recursive holds
156       */
157      public void testGetHoldCount() {
158 <        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
159 <        for(int i = 1; i <= SIZE; i++) {
160 <            lock.writeLock().lock();
161 <            assertEquals(i,lock.writeLock().getHoldCount());
162 <        }
163 <        for(int i = SIZE; i > 0; i--) {
164 <            lock.writeLock().unlock();
165 <            assertEquals(i-1,lock.writeLock().getHoldCount());
166 <        }
158 >        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
159 >        for (int i = 1; i <= SIZE; i++) {
160 >            lock.writeLock().lock();
161 >            assertEquals(i,lock.writeLock().getHoldCount());
162 >        }
163 >        for (int i = SIZE; i > 0; i--) {
164 >            lock.writeLock().unlock();
165 >            assertEquals(i-1,lock.writeLock().getHoldCount());
166 >        }
167      }
168  
169      /**
170       * getReadHoldCount returns number of recursive holds
171       */
172      public void testGetReadHoldCount() {
173 <        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
174 <        for(int i = 1; i <= SIZE; i++) {
175 <            lock.readLock().lock();
176 <            assertEquals(i,lock.getReadHoldCount());
177 <        }
178 <        for(int i = SIZE; i > 0; i--) {
179 <            lock.readLock().unlock();
180 <            assertEquals(i-1,lock.getReadHoldCount());
181 <        }
173 >        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
174 >        for (int i = 1; i <= SIZE; i++) {
175 >            lock.readLock().lock();
176 >            assertEquals(i,lock.getReadHoldCount());
177 >        }
178 >        for (int i = SIZE; i > 0; i--) {
179 >            lock.readLock().unlock();
180 >            assertEquals(i-1,lock.getReadHoldCount());
181 >        }
182      }
183 <    
183 >
184  
185      /**
186       * write-unlocking an unlocked lock throws IllegalMonitorStateException
187       */
188 <    public void testUnlock_IllegalMonitorStateException() {
189 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
190 <        try {
191 <            rl.writeLock().unlock();
192 <            shouldThrow();
193 <        } catch(IllegalMonitorStateException success){}
188 >    public void testUnlock_IllegalMonitorStateException() {
189 >        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
190 >        try {
191 >            rl.writeLock().unlock();
192 >            shouldThrow();
193 >        } catch (IllegalMonitorStateException success) {}
194      }
195  
196  
197      /**
198       * write-lockInterruptibly is interruptible
199       */
200 <    public void testWriteLockInterruptibly_Interrupted() {
201 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
202 <        Thread t = new Thread(new Runnable() {
203 <                public void run() {
204 <                    try {
205 <                        lock.writeLock().lockInterruptibly();
206 <                        lock.writeLock().unlock();
207 <                        lock.writeLock().lockInterruptibly();
208 <                        lock.writeLock().unlock();
209 <                    } catch(InterruptedException success){}
210 <                }
211 <            });
212 <        try {
204 <            lock.writeLock().lock();
205 <            t.start();
206 <            Thread.sleep(SHORT_DELAY_MS);
207 <            t.interrupt();
208 <            Thread.sleep(SHORT_DELAY_MS);
209 <            lock.writeLock().unlock();
210 <            t.join();
211 <        } catch(Exception e){
212 <            unexpectedException();
213 <        }
214 <    }
200 >    public void testWriteLockInterruptibly_Interrupted() throws Exception {
201 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
202 >        lock.writeLock().lock();
203 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
204 >            public void realRun() throws InterruptedException {
205 >                lock.writeLock().lockInterruptibly();
206 >            }});
207 >
208 >        Thread.sleep(SHORT_DELAY_MS);
209 >        t.interrupt();
210 >        t.join();
211 >        releaseLock(lock.writeLock());
212 >    }
213  
214      /**
215       * timed write-tryLock is interruptible
216       */
217 <    public void testWriteTryLock_Interrupted() {
218 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
219 <        lock.writeLock().lock();
220 <        Thread t = new Thread(new Runnable() {
221 <                public void run() {
222 <                    try {
223 <                        lock.writeLock().tryLock(1000,TimeUnit.MILLISECONDS);
224 <                    } catch(InterruptedException success){}
225 <                }
226 <            });
227 <        try {
228 <            t.start();
231 <            t.interrupt();
232 <            lock.writeLock().unlock();
233 <            t.join();
234 <        } catch(Exception e){
235 <            unexpectedException();
236 <        }
217 >    public void testWriteTryLock_Interrupted() throws InterruptedException {
218 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
219 >        lock.writeLock().lock();
220 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
221 >            public void realRun() throws InterruptedException {
222 >                lock.writeLock().tryLock(SMALL_DELAY_MS, MILLISECONDS);
223 >            }});
224 >
225 >        Thread.sleep(SHORT_DELAY_MS);
226 >        t.interrupt();
227 >        t.join();
228 >        releaseLock(lock.writeLock());
229      }
230  
231      /**
232       * read-lockInterruptibly is interruptible
233       */
234 <    public void testReadLockInterruptibly_Interrupted() {
235 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
236 <        lock.writeLock().lock();
237 <        Thread t = new Thread(new Runnable() {
238 <                public void run() {
239 <                    try {
240 <                        lock.readLock().lockInterruptibly();
241 <                    } catch(InterruptedException success){}
242 <                }
243 <            });
244 <        try {
245 <            t.start();
246 <            Thread.sleep(SHORT_DELAY_MS);
247 <            t.interrupt();
256 <            Thread.sleep(SHORT_DELAY_MS);
257 <            lock.writeLock().unlock();
258 <            t.join();
259 <        } catch(Exception e){
260 <            unexpectedException();
261 <        }
262 <    }
234 >    public void testReadLockInterruptibly_Interrupted() throws InterruptedException {
235 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
236 >        lock.writeLock().lock();
237 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
238 >            public void realRun() throws InterruptedException {
239 >                lock.readLock().lockInterruptibly();
240 >            }});
241 >
242 >        Thread.sleep(SHORT_DELAY_MS);
243 >        t.interrupt();
244 >        Thread.sleep(SHORT_DELAY_MS);
245 >        lock.writeLock().unlock();
246 >        t.join();
247 >    }
248  
249      /**
250       * timed read-tryLock is interruptible
251       */
252 <    public void testReadTryLock_Interrupted() {
253 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
254 <        lock.writeLock().lock();
255 <        Thread t = new Thread(new Runnable() {
256 <                public void run() {
257 <                    try {
258 <                        lock.readLock().tryLock(1000,TimeUnit.MILLISECONDS);
259 <                        threadShouldThrow();
260 <                    } catch(InterruptedException success){}
261 <                }
262 <            });
278 <        try {
279 <            t.start();
280 <            t.interrupt();
281 <            t.join();
282 <        } catch(Exception e){
283 <            unexpectedException();
284 <        }
252 >    public void testReadTryLock_Interrupted() throws InterruptedException {
253 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
254 >        lock.writeLock().lock();
255 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
256 >            public void realRun() throws InterruptedException {
257 >                lock.readLock().tryLock(LONG_DELAY_MS, MILLISECONDS);
258 >            }});
259 >
260 >        Thread.sleep(SHORT_DELAY_MS);
261 >        t.interrupt();
262 >        t.join();
263      }
264  
265 <    
265 >
266      /**
267       * write-tryLock fails if locked
268       */
269 <    public void testWriteTryLockWhenLocked() {
270 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
271 <        lock.writeLock().lock();
272 <        Thread t = new Thread(new Runnable() {
273 <                public void run() {
274 <                    threadAssertFalse(lock.writeLock().tryLock());
275 <                }
276 <            });
277 <        try {
278 <            t.start();
279 <            t.join();
302 <            lock.writeLock().unlock();
303 <        } catch(Exception e){
304 <            unexpectedException();
305 <        }
306 <    }
269 >    public void testWriteTryLockWhenLocked() throws InterruptedException {
270 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
271 >        lock.writeLock().lock();
272 >        Thread t = newStartedThread(new CheckedRunnable() {
273 >            public void realRun() {
274 >                assertFalse(lock.writeLock().tryLock());
275 >            }});
276 >
277 >        t.join();
278 >        lock.writeLock().unlock();
279 >    }
280  
281      /**
282       * read-tryLock fails if locked
283       */
284 <    public void testReadTryLockWhenLocked() {
285 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
286 <        lock.writeLock().lock();
287 <        Thread t = new Thread(new Runnable() {
288 <                public void run() {
289 <                    threadAssertFalse(lock.readLock().tryLock());
290 <                }
291 <            });
292 <        try {
293 <            t.start();
294 <            t.join();
322 <            lock.writeLock().unlock();
323 <        } catch(Exception e){
324 <            unexpectedException();
325 <        }
326 <    }
284 >    public void testReadTryLockWhenLocked() throws InterruptedException {
285 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
286 >        lock.writeLock().lock();
287 >        Thread t = newStartedThread(new CheckedRunnable() {
288 >            public void realRun() {
289 >                assertFalse(lock.readLock().tryLock());
290 >            }});
291 >
292 >        t.join();
293 >        lock.writeLock().unlock();
294 >    }
295  
296      /**
297       * Multiple threads can hold a read lock when not write-locked
298       */
299 <    public void testMultipleReadLocks() {
300 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
301 <        lock.readLock().lock();
302 <        Thread t = new Thread(new Runnable() {
303 <                public void run() {
304 <                    threadAssertTrue(lock.readLock().tryLock());
305 <                    lock.readLock().unlock();
306 <                }
307 <            });
308 <        try {
309 <            t.start();
310 <            t.join();
343 <            lock.readLock().unlock();
344 <        } catch(Exception e){
345 <            unexpectedException();
346 <        }
347 <    }
299 >    public void testMultipleReadLocks() throws InterruptedException {
300 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
301 >        lock.readLock().lock();
302 >        Thread t = newStartedThread(new CheckedRunnable() {
303 >            public void realRun() {
304 >                assertTrue(lock.readLock().tryLock());
305 >                lock.readLock().unlock();
306 >            }});
307 >
308 >        t.join();
309 >        lock.readLock().unlock();
310 >    }
311  
312      /**
313       * A writelock succeeds after reading threads unlock
314       */
315 <    public void testWriteAfterMultipleReadLocks() {
316 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
317 <        lock.readLock().lock();
318 <        Thread t1 = new Thread(new Runnable() {
319 <                public void run() {
320 <                    lock.readLock().lock();
321 <                    lock.readLock().unlock();
322 <                }
323 <            });
324 <        Thread t2 = new Thread(new Runnable() {
325 <                public void run() {
326 <                    lock.writeLock().lock();
327 <                    lock.writeLock().unlock();
365 <                }
366 <            });
315 >    public void testWriteAfterMultipleReadLocks() throws InterruptedException {
316 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
317 >        lock.readLock().lock();
318 >        Thread t1 = newStartedThread(new CheckedRunnable() {
319 >            public void realRun() {
320 >                lock.readLock().lock();
321 >                lock.readLock().unlock();
322 >            }});
323 >        Thread t2 = newStartedThread(new CheckedRunnable() {
324 >            public void realRun() {
325 >                lock.writeLock().lock();
326 >                lock.writeLock().unlock();
327 >            }});
328  
329 <        try {
330 <            t1.start();
331 <            t2.start();
332 <            Thread.sleep(SHORT_DELAY_MS);
333 <            lock.readLock().unlock();
334 <            t1.join(MEDIUM_DELAY_MS);
335 <            t2.join(MEDIUM_DELAY_MS);
375 <            assertTrue(!t1.isAlive());
376 <            assertTrue(!t2.isAlive());
377 <          
378 <        } catch(Exception e){
379 <            unexpectedException();
380 <        }
381 <    }
329 >        Thread.sleep(SHORT_DELAY_MS);
330 >        lock.readLock().unlock();
331 >        t1.join(MEDIUM_DELAY_MS);
332 >        t2.join(MEDIUM_DELAY_MS);
333 >        assertTrue(!t1.isAlive());
334 >        assertTrue(!t2.isAlive());
335 >    }
336  
337      /**
338       * Readlocks succeed after a writing thread unlocks
339       */
340 <    public void testReadAfterWriteLock() {
341 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
342 <        lock.writeLock().lock();
343 <        Thread t1 = new Thread(new Runnable() {
344 <                public void run() {
345 <                    lock.readLock().lock();
346 <                    lock.readLock().unlock();
347 <                }
348 <            });
349 <        Thread t2 = new Thread(new Runnable() {
350 <                public void run() {
351 <                    lock.readLock().lock();
352 <                    lock.readLock().unlock();
399 <                }
400 <            });
340 >    public void testReadAfterWriteLock() throws InterruptedException {
341 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
342 >        lock.writeLock().lock();
343 >        Thread t1 = newStartedThread(new CheckedRunnable() {
344 >            public void realRun() {
345 >                lock.readLock().lock();
346 >                lock.readLock().unlock();
347 >            }});
348 >        Thread t2 = newStartedThread(new CheckedRunnable() {
349 >            public void realRun() {
350 >                lock.readLock().lock();
351 >                lock.readLock().unlock();
352 >            }});
353  
354 <        try {
355 <            t1.start();
356 <            t2.start();
357 <            Thread.sleep(SHORT_DELAY_MS);
358 <            lock.writeLock().unlock();
359 <            t1.join(MEDIUM_DELAY_MS);
360 <            t2.join(MEDIUM_DELAY_MS);
409 <            assertTrue(!t1.isAlive());
410 <            assertTrue(!t2.isAlive());
411 <          
412 <        } catch(Exception e){
413 <            unexpectedException();
414 <        }
415 <    }
354 >        Thread.sleep(SHORT_DELAY_MS);
355 >        lock.writeLock().unlock();
356 >        t1.join(MEDIUM_DELAY_MS);
357 >        t2.join(MEDIUM_DELAY_MS);
358 >        assertTrue(!t1.isAlive());
359 >        assertTrue(!t2.isAlive());
360 >    }
361  
362      /**
363       * Read trylock succeeds if write locked by current thread
364       */
365 <    public void testReadHoldingWriteLock() {
366 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
367 <        lock.writeLock().lock();
365 >    public void testReadHoldingWriteLock() {
366 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
367 >        lock.writeLock().lock();
368          assertTrue(lock.readLock().tryLock());
369          lock.readLock().unlock();
370          lock.writeLock().unlock();
371 <    }
371 >    }
372  
373      /**
374       * Read lock succeeds if write locked by current thread even if
375       * other threads are waiting for readlock
376       */
377 <    public void testReadHoldingWriteLock2() {
378 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
379 <        lock.writeLock().lock();
380 <        Thread t1 = new Thread(new Runnable() {
381 <                public void run() {
382 <                    lock.readLock().lock();
383 <                    lock.readLock().unlock();
384 <                }
385 <            });
386 <        Thread t2 = new Thread(new Runnable() {
387 <                public void run() {
388 <                    lock.readLock().lock();
389 <                    lock.readLock().unlock();
445 <                }
446 <            });
377 >    public void testReadHoldingWriteLock2() throws InterruptedException {
378 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
379 >        lock.writeLock().lock();
380 >        Thread t1 = newStartedThread(new CheckedRunnable() {
381 >            public void realRun() {
382 >                lock.readLock().lock();
383 >                lock.readLock().unlock();
384 >            }});
385 >        Thread t2 = newStartedThread(new CheckedRunnable() {
386 >            public void realRun() {
387 >                lock.readLock().lock();
388 >                lock.readLock().unlock();
389 >            }});
390  
391 <        try {
392 <            t1.start();
393 <            t2.start();
394 <            lock.readLock().lock();
395 <            lock.readLock().unlock();
396 <            Thread.sleep(SHORT_DELAY_MS);
397 <            lock.readLock().lock();
398 <            lock.readLock().unlock();
399 <            lock.writeLock().unlock();
400 <            t1.join(MEDIUM_DELAY_MS);
401 <            t2.join(MEDIUM_DELAY_MS);
459 <            assertTrue(!t1.isAlive());
460 <            assertTrue(!t2.isAlive());
461 <          
462 <        } catch(Exception e){
463 <            unexpectedException();
464 <        }
465 <    }
391 >        lock.readLock().lock();
392 >        lock.readLock().unlock();
393 >        Thread.sleep(SHORT_DELAY_MS);
394 >        lock.readLock().lock();
395 >        lock.readLock().unlock();
396 >        lock.writeLock().unlock();
397 >        t1.join(MEDIUM_DELAY_MS);
398 >        t2.join(MEDIUM_DELAY_MS);
399 >        assertTrue(!t1.isAlive());
400 >        assertTrue(!t2.isAlive());
401 >    }
402  
403      /**
404 <     *  Read lock succeeds if write locked by current thread even if
404 >     * Read lock succeeds if write locked by current thread even if
405       * other threads are waiting for writelock
406       */
407 <    public void testReadHoldingWriteLock3() {
408 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
409 <        lock.writeLock().lock();
410 <        Thread t1 = new Thread(new Runnable() {
411 <                public void run() {
412 <                    lock.writeLock().lock();
413 <                    lock.writeLock().unlock();
414 <                }
415 <            });
416 <        Thread t2 = new Thread(new Runnable() {
417 <                public void run() {
418 <                    lock.writeLock().lock();
419 <                    lock.writeLock().unlock();
484 <                }
485 <            });
407 >    public void testReadHoldingWriteLock3() throws InterruptedException {
408 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
409 >        lock.writeLock().lock();
410 >        Thread t1 = newStartedThread(new CheckedRunnable() {
411 >            public void realRun() {
412 >                lock.writeLock().lock();
413 >                lock.writeLock().unlock();
414 >            }});
415 >        Thread t2 = newStartedThread(new CheckedRunnable() {
416 >            public void realRun() {
417 >                lock.writeLock().lock();
418 >                lock.writeLock().unlock();
419 >            }});
420  
421 <        try {
422 <            t1.start();
423 <            t2.start();
424 <            lock.readLock().lock();
425 <            lock.readLock().unlock();
426 <            Thread.sleep(SHORT_DELAY_MS);
427 <            lock.readLock().lock();
428 <            lock.readLock().unlock();
429 <            lock.writeLock().unlock();
430 <            t1.join(MEDIUM_DELAY_MS);
431 <            t2.join(MEDIUM_DELAY_MS);
498 <            assertTrue(!t1.isAlive());
499 <            assertTrue(!t2.isAlive());
500 <          
501 <        } catch(Exception e){
502 <            unexpectedException();
503 <        }
504 <    }
421 >        lock.readLock().lock();
422 >        lock.readLock().unlock();
423 >        Thread.sleep(SHORT_DELAY_MS);
424 >        lock.readLock().lock();
425 >        lock.readLock().unlock();
426 >        lock.writeLock().unlock();
427 >        t1.join(MEDIUM_DELAY_MS);
428 >        t2.join(MEDIUM_DELAY_MS);
429 >        assertTrue(!t1.isAlive());
430 >        assertTrue(!t2.isAlive());
431 >    }
432  
433  
434      /**
435 <     *  Write lock succeeds if write locked by current thread even if
435 >     * Write lock succeeds if write locked by current thread even if
436       * other threads are waiting for writelock
437       */
438 <    public void testWriteHoldingWriteLock4() {
439 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
440 <        lock.writeLock().lock();
441 <        Thread t1 = new Thread(new Runnable() {
442 <                public void run() {
443 <                    lock.writeLock().lock();
444 <                    lock.writeLock().unlock();
445 <                }
446 <            });
447 <        Thread t2 = new Thread(new Runnable() {
448 <                public void run() {
449 <                    lock.writeLock().lock();
450 <                    lock.writeLock().unlock();
524 <                }
525 <            });
438 >    public void testWriteHoldingWriteLock4() throws InterruptedException {
439 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
440 >        lock.writeLock().lock();
441 >        Thread t1 = newStartedThread(new CheckedRunnable() {
442 >            public void realRun() {
443 >                lock.writeLock().lock();
444 >                lock.writeLock().unlock();
445 >            }});
446 >        Thread t2 = newStartedThread(new CheckedRunnable() {
447 >            public void realRun() {
448 >                lock.writeLock().lock();
449 >                lock.writeLock().unlock();
450 >            }});
451  
452 <        try {
453 <            t1.start();
454 <            t2.start();
455 <            lock.writeLock().lock();
456 <            lock.writeLock().unlock();
457 <            Thread.sleep(SHORT_DELAY_MS);
458 <            lock.writeLock().lock();
459 <            lock.writeLock().unlock();
460 <            lock.writeLock().unlock();
461 <            t1.join(MEDIUM_DELAY_MS);
462 <            t2.join(MEDIUM_DELAY_MS);
538 <            assertTrue(!t1.isAlive());
539 <            assertTrue(!t2.isAlive());
540 <          
541 <        } catch(Exception e){
542 <            unexpectedException();
543 <        }
544 <    }
452 >        lock.writeLock().lock();
453 >        lock.writeLock().unlock();
454 >        Thread.sleep(SHORT_DELAY_MS);
455 >        lock.writeLock().lock();
456 >        lock.writeLock().unlock();
457 >        lock.writeLock().unlock();
458 >        t1.join(MEDIUM_DELAY_MS);
459 >        t2.join(MEDIUM_DELAY_MS);
460 >        assertTrue(!t1.isAlive());
461 >        assertTrue(!t2.isAlive());
462 >    }
463  
464  
465      /**
466       * Fair Read trylock succeeds if write locked by current thread
467       */
468 <    public void testReadHoldingWriteLockFair() {
469 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
470 <        lock.writeLock().lock();
468 >    public void testReadHoldingWriteLockFair() {
469 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
470 >        lock.writeLock().lock();
471          assertTrue(lock.readLock().tryLock());
472          lock.readLock().unlock();
473          lock.writeLock().unlock();
474 <    }
474 >    }
475  
476      /**
477       * Fair Read lock succeeds if write locked by current thread even if
478       * other threads are waiting for readlock
479       */
480 <    public void testReadHoldingWriteLockFair2() {
481 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
482 <        lock.writeLock().lock();
483 <        Thread t1 = new Thread(new Runnable() {
484 <                public void run() {
485 <                    lock.readLock().lock();
486 <                    lock.readLock().unlock();
487 <                }
488 <            });
489 <        Thread t2 = new Thread(new Runnable() {
490 <                public void run() {
491 <                    lock.readLock().lock();
492 <                    lock.readLock().unlock();
575 <                }
576 <            });
480 >    public void testReadHoldingWriteLockFair2() throws InterruptedException {
481 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
482 >        lock.writeLock().lock();
483 >        Thread t1 = newStartedThread(new CheckedRunnable() {
484 >            public void realRun() {
485 >                lock.readLock().lock();
486 >                lock.readLock().unlock();
487 >            }});
488 >        Thread t2 = newStartedThread(new CheckedRunnable() {
489 >            public void realRun() {
490 >                lock.readLock().lock();
491 >                lock.readLock().unlock();
492 >            }});
493  
494 <        try {
495 <            t1.start();
496 <            t2.start();
497 <            lock.readLock().lock();
498 <            lock.readLock().unlock();
499 <            Thread.sleep(SHORT_DELAY_MS);
500 <            lock.readLock().lock();
501 <            lock.readLock().unlock();
502 <            lock.writeLock().unlock();
503 <            t1.join(MEDIUM_DELAY_MS);
504 <            t2.join(MEDIUM_DELAY_MS);
589 <            assertTrue(!t1.isAlive());
590 <            assertTrue(!t2.isAlive());
591 <          
592 <        } catch(Exception e){
593 <            unexpectedException();
594 <        }
595 <    }
494 >        lock.readLock().lock();
495 >        lock.readLock().unlock();
496 >        Thread.sleep(SHORT_DELAY_MS);
497 >        lock.readLock().lock();
498 >        lock.readLock().unlock();
499 >        lock.writeLock().unlock();
500 >        t1.join(MEDIUM_DELAY_MS);
501 >        t2.join(MEDIUM_DELAY_MS);
502 >        assertTrue(!t1.isAlive());
503 >        assertTrue(!t2.isAlive());
504 >    }
505  
506  
507      /**
508       * Fair Read lock succeeds if write locked by current thread even if
509       * other threads are waiting for writelock
510       */
511 <    public void testReadHoldingWriteLockFair3() {
512 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
513 <        lock.writeLock().lock();
514 <        Thread t1 = new Thread(new Runnable() {
515 <                public void run() {
516 <                    lock.writeLock().lock();
517 <                    lock.writeLock().unlock();
518 <                }
519 <            });
520 <        Thread t2 = new Thread(new Runnable() {
521 <                public void run() {
522 <                    lock.writeLock().lock();
523 <                    lock.writeLock().unlock();
615 <                }
616 <            });
511 >    public void testReadHoldingWriteLockFair3() throws InterruptedException {
512 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
513 >        lock.writeLock().lock();
514 >        Thread t1 = newStartedThread(new CheckedRunnable() {
515 >            public void realRun() {
516 >                lock.writeLock().lock();
517 >                lock.writeLock().unlock();
518 >            }});
519 >        Thread t2 = newStartedThread(new CheckedRunnable() {
520 >            public void realRun() {
521 >                lock.writeLock().lock();
522 >                lock.writeLock().unlock();
523 >            }});
524  
525 <        try {
526 <            t1.start();
527 <            t2.start();
528 <            lock.readLock().lock();
529 <            lock.readLock().unlock();
530 <            Thread.sleep(SHORT_DELAY_MS);
531 <            lock.readLock().lock();
532 <            lock.readLock().unlock();
533 <            lock.writeLock().unlock();
534 <            t1.join(MEDIUM_DELAY_MS);
535 <            t2.join(MEDIUM_DELAY_MS);
629 <            assertTrue(!t1.isAlive());
630 <            assertTrue(!t2.isAlive());
631 <          
632 <        } catch(Exception e){
633 <            unexpectedException();
634 <        }
635 <    }
525 >        lock.readLock().lock();
526 >        lock.readLock().unlock();
527 >        Thread.sleep(SHORT_DELAY_MS);
528 >        lock.readLock().lock();
529 >        lock.readLock().unlock();
530 >        lock.writeLock().unlock();
531 >        t1.join(MEDIUM_DELAY_MS);
532 >        t2.join(MEDIUM_DELAY_MS);
533 >        assertTrue(!t1.isAlive());
534 >        assertTrue(!t2.isAlive());
535 >    }
536  
537  
538      /**
539       * Fair Write lock succeeds if write locked by current thread even if
540       * other threads are waiting for writelock
541       */
542 <    public void testWriteHoldingWriteLockFair4() {
543 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
544 <        lock.writeLock().lock();
545 <        Thread t1 = new Thread(new Runnable() {
546 <                public void run() {
547 <                    lock.writeLock().lock();
548 <                    lock.writeLock().unlock();
549 <                }
550 <            });
551 <        Thread t2 = new Thread(new Runnable() {
552 <                public void run() {
553 <                    lock.writeLock().lock();
554 <                    lock.writeLock().unlock();
655 <                }
656 <            });
542 >    public void testWriteHoldingWriteLockFair4() throws InterruptedException {
543 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
544 >        lock.writeLock().lock();
545 >        Thread t1 = newStartedThread(new CheckedRunnable() {
546 >            public void realRun() {
547 >                lock.writeLock().lock();
548 >                lock.writeLock().unlock();
549 >            }});
550 >        Thread t2 = newStartedThread(new CheckedRunnable() {
551 >            public void realRun() {
552 >                lock.writeLock().lock();
553 >                lock.writeLock().unlock();
554 >            }});
555  
556 <        try {
557 <            t1.start();
558 <            t2.start();
559 <            Thread.sleep(SHORT_DELAY_MS);
560 <            assertTrue(lock.isWriteLockedByCurrentThread());
561 <            assertTrue(lock.getWriteHoldCount() == 1);
562 <            lock.writeLock().lock();
563 <            assertTrue(lock.getWriteHoldCount() == 2);
564 <            lock.writeLock().unlock();
565 <            lock.writeLock().lock();
566 <            lock.writeLock().unlock();
567 <            lock.writeLock().unlock();
568 <            t1.join(MEDIUM_DELAY_MS);
569 <            t2.join(MEDIUM_DELAY_MS);
672 <            assertTrue(!t1.isAlive());
673 <            assertTrue(!t2.isAlive());
674 <          
675 <        } catch(Exception e){
676 <            unexpectedException();
677 <        }
678 <    }
556 >        Thread.sleep(SHORT_DELAY_MS);
557 >        assertTrue(lock.isWriteLockedByCurrentThread());
558 >        assertEquals(1, lock.getWriteHoldCount());
559 >        lock.writeLock().lock();
560 >        assertEquals(2, lock.getWriteHoldCount());
561 >        lock.writeLock().unlock();
562 >        lock.writeLock().lock();
563 >        lock.writeLock().unlock();
564 >        lock.writeLock().unlock();
565 >        t1.join(MEDIUM_DELAY_MS);
566 >        t2.join(MEDIUM_DELAY_MS);
567 >        assertTrue(!t1.isAlive());
568 >        assertTrue(!t2.isAlive());
569 >    }
570  
571  
572      /**
573       * Read tryLock succeeds if readlocked but not writelocked
574       */
575 <    public void testTryLockWhenReadLocked() {
576 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
577 <        lock.readLock().lock();
578 <        Thread t = new Thread(new Runnable() {
579 <                public void run() {
580 <                    threadAssertTrue(lock.readLock().tryLock());
581 <                    lock.readLock().unlock();
582 <                }
692 <            });
693 <        try {
694 <            t.start();
695 <            t.join();
696 <            lock.readLock().unlock();
697 <        } catch(Exception e){
698 <            unexpectedException();
699 <        }
700 <    }
575 >    public void testTryLockWhenReadLocked() throws InterruptedException {
576 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
577 >        lock.readLock().lock();
578 >        Thread t = newStartedThread(new CheckedRunnable() {
579 >            public void realRun() {
580 >                assertTrue(lock.readLock().tryLock());
581 >                lock.readLock().unlock();
582 >            }});
583  
584 <    
584 >        t.join();
585 >        lock.readLock().unlock();
586 >    }
587  
588      /**
589       * write tryLock fails when readlocked
590       */
591 <    public void testWriteTryLockWhenReadLocked() {
592 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
593 <        lock.readLock().lock();
594 <        Thread t = new Thread(new Runnable() {
595 <                public void run() {
596 <                    threadAssertFalse(lock.writeLock().tryLock());
597 <                }
598 <            });
599 <        try {
600 <            t.start();
601 <            t.join();
718 <            lock.readLock().unlock();
719 <        } catch(Exception e){
720 <            unexpectedException();
721 <        }
722 <    }
591 >    public void testWriteTryLockWhenReadLocked() throws InterruptedException {
592 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
593 >        lock.readLock().lock();
594 >        Thread t = newStartedThread(new CheckedRunnable() {
595 >            public void realRun() {
596 >                assertFalse(lock.writeLock().tryLock());
597 >            }});
598 >
599 >        t.join();
600 >        lock.readLock().unlock();
601 >    }
602  
603  
604      /**
605       * Fair Read tryLock succeeds if readlocked but not writelocked
606       */
607 <    public void testTryLockWhenReadLockedFair() {
608 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
609 <        lock.readLock().lock();
610 <        Thread t = new Thread(new Runnable() {
611 <                public void run() {
612 <                    threadAssertTrue(lock.readLock().tryLock());
613 <                    lock.readLock().unlock();
614 <                }
615 <            });
616 <        try {
617 <            t.start();
618 <            t.join();
619 <            lock.readLock().unlock();
741 <        } catch(Exception e){
742 <            unexpectedException();
743 <        }
744 <    }
607 >    public void testTryLockWhenReadLockedFair() throws InterruptedException {
608 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
609 >        lock.readLock().lock();
610 >        Thread t = newStartedThread(new CheckedRunnable() {
611 >            public void realRun() {
612 >                assertTrue(lock.readLock().tryLock());
613 >                lock.readLock().unlock();
614 >            }});
615 >
616 >        t.join();
617 >        lock.readLock().unlock();
618 >    }
619 >
620  
746    
621  
622      /**
623       * Fair write tryLock fails when readlocked
624       */
625 <    public void testWriteTryLockWhenReadLockedFair() {
626 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
627 <        lock.readLock().lock();
628 <        Thread t = new Thread(new Runnable() {
629 <                public void run() {
630 <                    threadAssertFalse(lock.writeLock().tryLock());
631 <                }
632 <            });
633 <        try {
634 <            t.start();
635 <            t.join();
636 <            lock.readLock().unlock();
763 <        } catch(Exception e){
764 <            unexpectedException();
765 <        }
766 <    }
625 >    public void testWriteTryLockWhenReadLockedFair() throws InterruptedException {
626 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
627 >        lock.readLock().lock();
628 >        Thread t = newStartedThread(new CheckedRunnable() {
629 >            public void realRun() {
630 >                assertFalse(lock.writeLock().tryLock());
631 >            }});
632 >
633 >        t.join();
634 >        lock.readLock().unlock();
635 >    }
636 >
637  
768    
638  
639      /**
640       * write timed tryLock times out if locked
641       */
642 <    public void testWriteTryLock_Timeout() {
643 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
644 <        lock.writeLock().lock();
645 <        Thread t = new Thread(new Runnable() {
646 <                public void run() {
647 <                    try {
648 <                        threadAssertFalse(lock.writeLock().tryLock(1, TimeUnit.MILLISECONDS));
649 <                    } catch (Exception ex) {
650 <                        threadUnexpectedException();
651 <                    }
652 <                }
653 <            });
785 <        try {
786 <            t.start();
787 <            t.join();
788 <            lock.writeLock().unlock();
789 <        } catch(Exception e){
790 <            unexpectedException();
791 <        }
792 <    }
642 >    public void testWriteTryLock_Timeout() throws InterruptedException {
643 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
644 >        lock.writeLock().lock();
645 >        Thread t = newStartedThread(new CheckedRunnable() {
646 >            public void realRun() throws InterruptedException {
647 >                assertFalse(lock.writeLock().tryLock(1, MILLISECONDS));
648 >            }});
649 >
650 >        t.join();
651 >        assertTrue(lock.writeLock().isHeldByCurrentThread());
652 >        lock.writeLock().unlock();
653 >    }
654  
655      /**
656       * read timed tryLock times out if write-locked
657       */
658 <    public void testReadTryLock_Timeout() {
659 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
660 <        lock.writeLock().lock();
661 <        Thread t = new Thread(new Runnable() {
662 <                public void run() {
663 <                    try {
664 <                        threadAssertFalse(lock.readLock().tryLock(1, TimeUnit.MILLISECONDS));
665 <                    } catch (Exception ex) {
666 <                        threadUnexpectedException();
667 <                    }
668 <                }
669 <            });
809 <        try {
810 <            t.start();
811 <            t.join();
812 <            lock.writeLock().unlock();
813 <        } catch(Exception e){
814 <            unexpectedException();
815 <        }
816 <    }
658 >    public void testReadTryLock_Timeout() throws InterruptedException {
659 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
660 >        lock.writeLock().lock();
661 >        Thread t = newStartedThread(new CheckedRunnable() {
662 >            public void realRun() throws InterruptedException {
663 >                assertFalse(lock.readLock().tryLock(1, MILLISECONDS));
664 >            }});
665 >
666 >        t.join();
667 >        assertTrue(lock.writeLock().isHeldByCurrentThread());
668 >        lock.writeLock().unlock();
669 >    }
670  
671  
672      /**
673       * write lockInterruptibly succeeds if lock free else is interruptible
674       */
675 <    public void testWriteLockInterruptibly() {
676 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
677 <        try {
678 <            lock.writeLock().lockInterruptibly();
679 <        } catch(Exception e) {
680 <            unexpectedException();
681 <        }
682 <        Thread t = new Thread(new Runnable() {
683 <                public void run() {
684 <                    try {
685 <                        lock.writeLock().lockInterruptibly();
686 <                        threadShouldThrow();
687 <                    }
835 <                    catch(InterruptedException success) {
836 <                    }
837 <                }
838 <            });
839 <        try {
840 <            t.start();
841 <            Thread.sleep(SHORT_DELAY_MS);
842 <            t.interrupt();
843 <            Thread.sleep(SHORT_DELAY_MS);
844 <            t.join();
845 <            lock.writeLock().unlock();
846 <        } catch(Exception e){
847 <            unexpectedException();
848 <        }
675 >    public void testWriteLockInterruptibly() throws InterruptedException {
676 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
677 >        lock.writeLock().lockInterruptibly();
678 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
679 >            public void realRun() throws InterruptedException {
680 >                lock.writeLock().lockInterruptibly();
681 >            }});
682 >
683 >        Thread.sleep(SHORT_DELAY_MS);
684 >        t.interrupt();
685 >        Thread.sleep(SHORT_DELAY_MS);
686 >        t.join();
687 >        lock.writeLock().unlock();
688      }
689  
690      /**
691 <     *  read lockInterruptibly succeeds if lock free else is interruptible
691 >     * read lockInterruptibly succeeds if lock free else is interruptible
692       */
693 <    public void testReadLockInterruptibly() {
694 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
695 <        try {
696 <            lock.writeLock().lockInterruptibly();
697 <        } catch(Exception e) {
698 <            unexpectedException();
699 <        }
700 <        Thread t = new Thread(new Runnable() {
701 <                public void run() {
702 <                    try {
703 <                        lock.readLock().lockInterruptibly();
704 <                        threadShouldThrow();
866 <                    }
867 <                    catch(InterruptedException success) {
868 <                    }
869 <                }
870 <            });
871 <        try {
872 <            t.start();
873 <            Thread.sleep(SHORT_DELAY_MS);
874 <            t.interrupt();
875 <            t.join();
876 <            lock.writeLock().unlock();
877 <        } catch(Exception e){
878 <            unexpectedException();
879 <        }
693 >    public void testReadLockInterruptibly() throws InterruptedException {
694 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
695 >        lock.writeLock().lockInterruptibly();
696 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
697 >            public void realRun() throws InterruptedException {
698 >                lock.readLock().lockInterruptibly();
699 >            }});
700 >
701 >        Thread.sleep(SHORT_DELAY_MS);
702 >        t.interrupt();
703 >        t.join();
704 >        lock.writeLock().unlock();
705      }
706  
707      /**
708       * Calling await without holding lock throws IllegalMonitorStateException
709       */
710 <    public void testAwait_IllegalMonitor() {
711 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
710 >    public void testAwait_IllegalMonitor() throws InterruptedException {
711 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
712          final Condition c = lock.writeLock().newCondition();
713          try {
714              c.await();
715              shouldThrow();
716 <        }
892 <        catch (IllegalMonitorStateException success) {
893 <        }
894 <        catch (Exception ex) {
895 <            shouldThrow();
896 <        }
716 >        } catch (IllegalMonitorStateException success) {}
717      }
718  
719      /**
720       * Calling signal without holding lock throws IllegalMonitorStateException
721       */
722      public void testSignal_IllegalMonitor() {
723 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
723 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
724          final Condition c = lock.writeLock().newCondition();
725          try {
726              c.signal();
727              shouldThrow();
728 <        }
909 <        catch (IllegalMonitorStateException success) {
910 <        }
911 <        catch (Exception ex) {
912 <            unexpectedException();
913 <        }
728 >        } catch (IllegalMonitorStateException success) {}
729      }
730  
731      /**
732       * awaitNanos without a signal times out
733       */
734 <    public void testAwaitNanos_Timeout() {
735 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
734 >    public void testAwaitNanos_Timeout() throws InterruptedException {
735 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
736          final Condition c = lock.writeLock().newCondition();
737 <        try {
738 <            lock.writeLock().lock();
739 <            long t = c.awaitNanos(100);
740 <            assertTrue(t <= 0);
741 <            lock.writeLock().unlock();
927 <        }
928 <        catch (Exception ex) {
929 <            unexpectedException();
930 <        }
737 >
738 >        lock.writeLock().lock();
739 >        long t = c.awaitNanos(100);
740 >        assertTrue(t <= 0);
741 >        lock.writeLock().unlock();
742      }
743  
744  
745      /**
746 <     *  timed await without a signal times out
746 >     * timed await without a signal times out
747       */
748 <    public void testAwait_Timeout() {
749 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
748 >    public void testAwait_Timeout() throws InterruptedException {
749 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
750          final Condition c = lock.writeLock().newCondition();
751 <        try {
752 <            lock.writeLock().lock();
753 <            lock.writeLock().unlock();
943 <        }
944 <        catch (Exception ex) {
945 <            unexpectedException();
946 <        }
751 >        lock.writeLock().lock();
752 >        assertFalse(c.await(SHORT_DELAY_MS, MILLISECONDS));
753 >        lock.writeLock().unlock();
754      }
755  
756      /**
757       * awaitUntil without a signal times out
758       */
759 <    public void testAwaitUntil_Timeout() {
760 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
759 >    public void testAwaitUntil_Timeout() throws InterruptedException {
760 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
761          final Condition c = lock.writeLock().newCondition();
762 <        try {
763 <            lock.writeLock().lock();
764 <            java.util.Date d = new java.util.Date();
765 <            lock.writeLock().unlock();
959 <        }
960 <        catch (Exception ex) {
961 <            unexpectedException();
962 <        }
762 >        lock.writeLock().lock();
763 >        java.util.Date d = new java.util.Date();
764 >        assertFalse(c.awaitUntil(new java.util.Date(d.getTime() + 10)));
765 >        lock.writeLock().unlock();
766      }
767  
768      /**
769       * await returns when signalled
770       */
771 <    public void testAwait() {
772 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
771 >    public void testAwait() throws InterruptedException {
772 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
773          final Condition c = lock.writeLock().newCondition();
774 <        Thread t = new Thread(new Runnable() {
775 <                public void run() {
776 <                    try {
777 <                        lock.writeLock().lock();
778 <                        c.await();
779 <                        lock.writeLock().unlock();
977 <                    }
978 <                    catch(InterruptedException e) {
979 <                        threadUnexpectedException();
980 <                    }
981 <                }
982 <            });
774 >        Thread t = newStartedThread(new CheckedRunnable() {
775 >            public void realRun() throws InterruptedException {
776 >                lock.writeLock().lock();
777 >                c.await();
778 >                lock.writeLock().unlock();
779 >            }});
780  
781 <        try {
782 <            t.start();
783 <            Thread.sleep(SHORT_DELAY_MS);
784 <            lock.writeLock().lock();
785 <            c.signal();
786 <            lock.writeLock().unlock();
990 <            t.join(SHORT_DELAY_MS);
991 <            assertFalse(t.isAlive());
992 <        }
993 <        catch (Exception ex) {
994 <            unexpectedException();
995 <        }
781 >        Thread.sleep(SHORT_DELAY_MS);
782 >        lock.writeLock().lock();
783 >        c.signal();
784 >        lock.writeLock().unlock();
785 >        t.join(SHORT_DELAY_MS);
786 >        assertFalse(t.isAlive());
787      }
788  
789      /** A helper class for uninterruptible wait tests */
790      class UninterruptableThread extends Thread {
791          private Lock lock;
792          private Condition c;
793 <        
793 >
794          public volatile boolean canAwake = false;
795          public volatile boolean interrupted = false;
796          public volatile boolean lockStarted = false;
797 <        
797 >
798          public UninterruptableThread(Lock lock, Condition c) {
799              this.lock = lock;
800              this.c = c;
801          }
802 <        
802 >
803          public synchronized void run() {
804              lock.lock();
805              lockStarted = true;
806 <            
806 >
807              while (!canAwake) {
808                  c.awaitUninterruptibly();
809              }
810 <            
810 >
811              interrupted = isInterrupted();
812              lock.unlock();
813          }
# Line 1025 | Line 816 | public class ReentrantReadWriteLockTest
816      /**
817       * awaitUninterruptibly doesn't abort on interrupt
818       */
819 <    public void testAwaitUninterruptibly() {
819 >    public void testAwaitUninterruptibly() throws InterruptedException {
820          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
821          final Condition c = lock.writeLock().newCondition();
822          UninterruptableThread thread = new UninterruptableThread(lock.writeLock(), c);
823  
824 <        try {
1034 <            thread.start();
1035 <
1036 <            while (!thread.lockStarted) {
1037 <                Thread.sleep(100);
1038 <            }
824 >        thread.start();
825  
826 <            lock.writeLock().lock();
827 <            try {
828 <                thread.interrupt();
1043 <                thread.canAwake = true;
1044 <                c.signal();
1045 <            } finally {
1046 <                lock.writeLock().unlock();
1047 <            }
826 >        while (!thread.lockStarted) {
827 >            Thread.sleep(100);
828 >        }
829  
830 <            thread.join();
831 <            assertTrue(thread.interrupted);
832 <            assertFalse(thread.isAlive());
833 <        } catch (Exception ex) {
834 <            unexpectedException();
830 >        lock.writeLock().lock();
831 >        try {
832 >            thread.interrupt();
833 >            thread.canAwake = true;
834 >            c.signal();
835 >        } finally {
836 >            lock.writeLock().unlock();
837          }
838 +
839 +        thread.join();
840 +        assertTrue(thread.interrupted);
841 +        assertFalse(thread.isAlive());
842      }
843  
844      /**
845       * await is interruptible
846       */
847 <    public void testAwait_Interrupt() {
848 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
847 >    public void testAwait_Interrupt() throws InterruptedException {
848 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
849          final Condition c = lock.writeLock().newCondition();
850 <        Thread t = new Thread(new Runnable() {
851 <                public void run() {
852 <                    try {
853 <                        lock.writeLock().lock();
854 <                        c.await();
855 <                        lock.writeLock().unlock();
1069 <                        threadShouldThrow();
1070 <                    }
1071 <                    catch(InterruptedException success) {
1072 <                    }
1073 <                }
1074 <            });
850 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
851 >            public void realRun() throws InterruptedException {
852 >                lock.writeLock().lock();
853 >                c.await();
854 >                lock.writeLock().unlock();
855 >            }});
856  
857 <        try {
858 <            t.start();
859 <            Thread.sleep(SHORT_DELAY_MS);
860 <            t.interrupt();
1080 <            t.join(SHORT_DELAY_MS);
1081 <            assertFalse(t.isAlive());
1082 <        }
1083 <        catch (Exception ex) {
1084 <            unexpectedException();
1085 <        }
857 >        Thread.sleep(SHORT_DELAY_MS);
858 >        t.interrupt();
859 >        t.join(SHORT_DELAY_MS);
860 >        assertFalse(t.isAlive());
861      }
862  
863      /**
864       * awaitNanos is interruptible
865       */
866 <    public void testAwaitNanos_Interrupt() {
867 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
866 >    public void testAwaitNanos_Interrupt() throws InterruptedException {
867 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
868          final Condition c = lock.writeLock().newCondition();
869 <        Thread t = new Thread(new Runnable() {
870 <                public void run() {
871 <                    try {
872 <                        lock.writeLock().lock();
873 <                        c.awaitNanos(SHORT_DELAY_MS * 2 * 1000000);
874 <                        lock.writeLock().unlock();
1100 <                        threadShouldThrow();
1101 <                    }
1102 <                    catch(InterruptedException success) {
1103 <                    }
1104 <                }
1105 <            });
869 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
870 >            public void realRun() throws InterruptedException {
871 >                lock.writeLock().lock();
872 >                c.awaitNanos(MILLISECONDS.toNanos(LONG_DELAY_MS));
873 >                lock.writeLock().unlock();
874 >            }});
875  
876 <        try {
877 <            t.start();
878 <            Thread.sleep(SHORT_DELAY_MS);
879 <            t.interrupt();
1111 <            t.join(SHORT_DELAY_MS);
1112 <            assertFalse(t.isAlive());
1113 <        }
1114 <        catch (Exception ex) {
1115 <            unexpectedException();
1116 <        }
876 >        Thread.sleep(SHORT_DELAY_MS);
877 >        t.interrupt();
878 >        t.join(SHORT_DELAY_MS);
879 >        assertFalse(t.isAlive());
880      }
881  
882      /**
883       * awaitUntil is interruptible
884       */
885 <    public void testAwaitUntil_Interrupt() {
886 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
885 >    public void testAwaitUntil_Interrupt() throws InterruptedException {
886 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
887          final Condition c = lock.writeLock().newCondition();
888 <        Thread t = new Thread(new Runnable() {
889 <                public void run() {
890 <                    try {
891 <                        lock.writeLock().lock();
892 <                        java.util.Date d = new java.util.Date();
893 <                        c.awaitUntil(new java.util.Date(d.getTime() + 10000));
894 <                        lock.writeLock().unlock();
1132 <                        threadShouldThrow();
1133 <                    }
1134 <                    catch(InterruptedException success) {
1135 <                    }
1136 <                }
1137 <            });
888 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
889 >            public void realRun() throws InterruptedException {
890 >                lock.writeLock().lock();
891 >                java.util.Date d = new java.util.Date();
892 >                c.awaitUntil(new java.util.Date(d.getTime() + 10000));
893 >                lock.writeLock().unlock();
894 >            }});
895  
896 <        try {
897 <            t.start();
898 <            Thread.sleep(SHORT_DELAY_MS);
899 <            t.interrupt();
1143 <            t.join(SHORT_DELAY_MS);
1144 <            assertFalse(t.isAlive());
1145 <        }
1146 <        catch (Exception ex) {
1147 <            unexpectedException();
1148 <        }
896 >        Thread.sleep(SHORT_DELAY_MS);
897 >        t.interrupt();
898 >        t.join(SHORT_DELAY_MS);
899 >        assertFalse(t.isAlive());
900      }
901  
902      /**
903       * signalAll wakes up all threads
904       */
905 <    public void testSignalAll() {
906 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
905 >    public void testSignalAll() throws InterruptedException {
906 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
907          final Condition c = lock.writeLock().newCondition();
908 <        Thread t1 = new Thread(new Runnable() {
909 <                public void run() {
910 <                    try {
911 <                        lock.writeLock().lock();
912 <                        c.await();
913 <                        lock.writeLock().unlock();
1163 <                    }
1164 <                    catch(InterruptedException e) {
1165 <                        threadUnexpectedException();
1166 <                    }
1167 <                }
1168 <            });
1169 <
1170 <        Thread t2 = new Thread(new Runnable() {
1171 <                public void run() {
1172 <                    try {
1173 <                        lock.writeLock().lock();
1174 <                        c.await();
1175 <                        lock.writeLock().unlock();
1176 <                    }
1177 <                    catch(InterruptedException e) {
1178 <                        threadUnexpectedException();
1179 <                    }
1180 <                }
1181 <            });
908 >        Thread t1 = newStartedThread(new CheckedRunnable() {
909 >            public void realRun() throws InterruptedException {
910 >                lock.writeLock().lock();
911 >                c.await();
912 >                lock.writeLock().unlock();
913 >            }});
914  
915 <        try {
916 <            t1.start();
917 <            t2.start();
918 <            Thread.sleep(SHORT_DELAY_MS);
919 <            lock.writeLock().lock();
920 <            c.signalAll();
921 <            lock.writeLock().unlock();
922 <            t1.join(SHORT_DELAY_MS);
923 <            t2.join(SHORT_DELAY_MS);
924 <            assertFalse(t1.isAlive());
925 <            assertFalse(t2.isAlive());
926 <        }
927 <        catch (Exception ex) {
928 <            unexpectedException();
929 <        }
915 >        Thread t2 = newStartedThread(new CheckedRunnable() {
916 >            public void realRun() throws InterruptedException {
917 >                lock.writeLock().lock();
918 >                c.await();
919 >                lock.writeLock().unlock();
920 >            }});
921 >
922 >        Thread.sleep(SHORT_DELAY_MS);
923 >        lock.writeLock().lock();
924 >        c.signalAll();
925 >        lock.writeLock().unlock();
926 >        t1.join(SHORT_DELAY_MS);
927 >        t2.join(SHORT_DELAY_MS);
928 >        assertFalse(t1.isAlive());
929 >        assertFalse(t2.isAlive());
930      }
931  
932      /**
933       * A serialized lock deserializes as unlocked
934       */
935 <    public void testSerialization() {
935 >    public void testSerialization() throws Exception {
936          ReentrantReadWriteLock l = new ReentrantReadWriteLock();
937          l.readLock().lock();
938          l.readLock().unlock();
939  
940 <        try {
941 <            ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
942 <            ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
943 <            out.writeObject(l);
944 <            out.close();
945 <
946 <            ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
947 <            ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
948 <            ReentrantReadWriteLock r = (ReentrantReadWriteLock) in.readObject();
949 <            r.readLock().lock();
1218 <            r.readLock().unlock();
1219 <        } catch(Exception e){
1220 <            e.printStackTrace();
1221 <            unexpectedException();
1222 <        }
940 >        ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
941 >        ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
942 >        out.writeObject(l);
943 >        out.close();
944 >
945 >        ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
946 >        ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
947 >        ReentrantReadWriteLock r = (ReentrantReadWriteLock) in.readObject();
948 >        r.readLock().lock();
949 >        r.readLock().unlock();
950      }
951  
952      /**
953       * hasQueuedThreads reports whether there are waiting threads
954       */
955 <    public void testhasQueuedThreads() {
956 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
955 >    public void testhasQueuedThreads() throws InterruptedException {
956 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
957          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
958          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
959 <        try {
960 <            assertFalse(lock.hasQueuedThreads());
961 <            lock.writeLock().lock();
962 <            t1.start();
963 <            Thread.sleep(SHORT_DELAY_MS);
964 <            assertTrue(lock.hasQueuedThreads());
965 <            t2.start();
966 <            Thread.sleep(SHORT_DELAY_MS);
967 <            assertTrue(lock.hasQueuedThreads());
968 <            t1.interrupt();
969 <            Thread.sleep(SHORT_DELAY_MS);
970 <            assertTrue(lock.hasQueuedThreads());
971 <            lock.writeLock().unlock();
972 <            Thread.sleep(SHORT_DELAY_MS);
973 <            assertFalse(lock.hasQueuedThreads());
974 <            t1.join();
975 <            t2.join();
1249 <        } catch(Exception e){
1250 <            unexpectedException();
1251 <        }
1252 <    }
959 >        assertFalse(lock.hasQueuedThreads());
960 >        lock.writeLock().lock();
961 >        t1.start();
962 >        Thread.sleep(SHORT_DELAY_MS);
963 >        assertTrue(lock.hasQueuedThreads());
964 >        t2.start();
965 >        Thread.sleep(SHORT_DELAY_MS);
966 >        assertTrue(lock.hasQueuedThreads());
967 >        t1.interrupt();
968 >        Thread.sleep(SHORT_DELAY_MS);
969 >        assertTrue(lock.hasQueuedThreads());
970 >        lock.writeLock().unlock();
971 >        Thread.sleep(SHORT_DELAY_MS);
972 >        assertFalse(lock.hasQueuedThreads());
973 >        t1.join();
974 >        t2.join();
975 >    }
976  
977      /**
978       * hasQueuedThread(null) throws NPE
979       */
980 <    public void testHasQueuedThreadNPE() {
981 <        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
980 >    public void testHasQueuedThreadNPE() {
981 >        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
982          try {
983              sync.hasQueuedThread(null);
984              shouldThrow();
985 <        } catch (NullPointerException success) {
1263 <        }
985 >        } catch (NullPointerException success) {}
986      }
987  
988      /**
989       * hasQueuedThread reports whether a thread is queued.
990       */
991 <    public void testHasQueuedThread() {
992 <        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
991 >    public void testHasQueuedThread() throws InterruptedException {
992 >        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
993          Thread t1 = new Thread(new InterruptedLockRunnable(sync));
994          Thread t2 = new Thread(new InterruptibleLockRunnable(sync));
995 <        try {
996 <            assertFalse(sync.hasQueuedThread(t1));
997 <            assertFalse(sync.hasQueuedThread(t2));
998 <            sync.writeLock().lock();
999 <            t1.start();
1000 <            Thread.sleep(SHORT_DELAY_MS);
1001 <            assertTrue(sync.hasQueuedThread(t1));
1002 <            t2.start();
1003 <            Thread.sleep(SHORT_DELAY_MS);
1004 <            assertTrue(sync.hasQueuedThread(t1));
1005 <            assertTrue(sync.hasQueuedThread(t2));
1006 <            t1.interrupt();
1007 <            Thread.sleep(SHORT_DELAY_MS);
1008 <            assertFalse(sync.hasQueuedThread(t1));
1009 <            assertTrue(sync.hasQueuedThread(t2));
1010 <            sync.writeLock().unlock();
1011 <            Thread.sleep(SHORT_DELAY_MS);
1012 <            assertFalse(sync.hasQueuedThread(t1));
1013 <            Thread.sleep(SHORT_DELAY_MS);
1014 <            assertFalse(sync.hasQueuedThread(t2));
1015 <            t1.join();
1016 <            t2.join();
1295 <        } catch(Exception e){
1296 <            unexpectedException();
1297 <        }
1298 <    }
995 >        assertFalse(sync.hasQueuedThread(t1));
996 >        assertFalse(sync.hasQueuedThread(t2));
997 >        sync.writeLock().lock();
998 >        t1.start();
999 >        Thread.sleep(SHORT_DELAY_MS);
1000 >        assertTrue(sync.hasQueuedThread(t1));
1001 >        t2.start();
1002 >        Thread.sleep(SHORT_DELAY_MS);
1003 >        assertTrue(sync.hasQueuedThread(t1));
1004 >        assertTrue(sync.hasQueuedThread(t2));
1005 >        t1.interrupt();
1006 >        Thread.sleep(SHORT_DELAY_MS);
1007 >        assertFalse(sync.hasQueuedThread(t1));
1008 >        assertTrue(sync.hasQueuedThread(t2));
1009 >        sync.writeLock().unlock();
1010 >        Thread.sleep(SHORT_DELAY_MS);
1011 >        assertFalse(sync.hasQueuedThread(t1));
1012 >        Thread.sleep(SHORT_DELAY_MS);
1013 >        assertFalse(sync.hasQueuedThread(t2));
1014 >        t1.join();
1015 >        t2.join();
1016 >    }
1017  
1018  
1019      /**
1020       * getQueueLength reports number of waiting threads
1021       */
1022 <    public void testGetQueueLength() {
1023 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1022 >    public void testGetQueueLength() throws InterruptedException {
1023 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1024          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1025          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1026 <        try {
1027 <            assertEquals(0, lock.getQueueLength());
1028 <            lock.writeLock().lock();
1029 <            t1.start();
1030 <            Thread.sleep(SHORT_DELAY_MS);
1031 <            assertEquals(1, lock.getQueueLength());
1032 <            t2.start();
1033 <            Thread.sleep(SHORT_DELAY_MS);
1034 <            assertEquals(2, lock.getQueueLength());
1035 <            t1.interrupt();
1036 <            Thread.sleep(SHORT_DELAY_MS);
1037 <            assertEquals(1, lock.getQueueLength());
1038 <            lock.writeLock().unlock();
1039 <            Thread.sleep(SHORT_DELAY_MS);
1040 <            assertEquals(0, lock.getQueueLength());
1041 <            t1.join();
1042 <            t2.join();
1325 <        } catch(Exception e){
1326 <            unexpectedException();
1327 <        }
1328 <    }
1026 >        assertEquals(0, lock.getQueueLength());
1027 >        lock.writeLock().lock();
1028 >        t1.start();
1029 >        Thread.sleep(SHORT_DELAY_MS);
1030 >        assertEquals(1, lock.getQueueLength());
1031 >        t2.start();
1032 >        Thread.sleep(SHORT_DELAY_MS);
1033 >        assertEquals(2, lock.getQueueLength());
1034 >        t1.interrupt();
1035 >        Thread.sleep(SHORT_DELAY_MS);
1036 >        assertEquals(1, lock.getQueueLength());
1037 >        lock.writeLock().unlock();
1038 >        Thread.sleep(SHORT_DELAY_MS);
1039 >        assertEquals(0, lock.getQueueLength());
1040 >        t1.join();
1041 >        t2.join();
1042 >    }
1043  
1044      /**
1045       * getQueuedThreads includes waiting threads
1046       */
1047 <    public void testGetQueuedThreads() {
1048 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1047 >    public void testGetQueuedThreads() throws InterruptedException {
1048 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1049          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1050          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1051 <        try {
1052 <            assertTrue(lock.getQueuedThreads().isEmpty());
1053 <            lock.writeLock().lock();
1054 <            assertTrue(lock.getQueuedThreads().isEmpty());
1055 <            t1.start();
1056 <            Thread.sleep(SHORT_DELAY_MS);
1057 <            assertTrue(lock.getQueuedThreads().contains(t1));
1058 <            t2.start();
1059 <            Thread.sleep(SHORT_DELAY_MS);
1060 <            assertTrue(lock.getQueuedThreads().contains(t1));
1061 <            assertTrue(lock.getQueuedThreads().contains(t2));
1062 <            t1.interrupt();
1063 <            Thread.sleep(SHORT_DELAY_MS);
1064 <            assertFalse(lock.getQueuedThreads().contains(t1));
1065 <            assertTrue(lock.getQueuedThreads().contains(t2));
1066 <            lock.writeLock().unlock();
1067 <            Thread.sleep(SHORT_DELAY_MS);
1068 <            assertTrue(lock.getQueuedThreads().isEmpty());
1069 <            t1.join();
1070 <            t2.join();
1357 <        } catch(Exception e){
1358 <            unexpectedException();
1359 <        }
1360 <    }
1051 >        assertTrue(lock.getQueuedThreads().isEmpty());
1052 >        lock.writeLock().lock();
1053 >        assertTrue(lock.getQueuedThreads().isEmpty());
1054 >        t1.start();
1055 >        Thread.sleep(SHORT_DELAY_MS);
1056 >        assertTrue(lock.getQueuedThreads().contains(t1));
1057 >        t2.start();
1058 >        Thread.sleep(SHORT_DELAY_MS);
1059 >        assertTrue(lock.getQueuedThreads().contains(t1));
1060 >        assertTrue(lock.getQueuedThreads().contains(t2));
1061 >        t1.interrupt();
1062 >        Thread.sleep(SHORT_DELAY_MS);
1063 >        assertFalse(lock.getQueuedThreads().contains(t1));
1064 >        assertTrue(lock.getQueuedThreads().contains(t2));
1065 >        lock.writeLock().unlock();
1066 >        Thread.sleep(SHORT_DELAY_MS);
1067 >        assertTrue(lock.getQueuedThreads().isEmpty());
1068 >        t1.join();
1069 >        t2.join();
1070 >    }
1071  
1072      /**
1073       * hasWaiters throws NPE if null
1074       */
1075      public void testHasWaitersNPE() {
1076 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1076 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1077          try {
1078              lock.hasWaiters(null);
1079              shouldThrow();
1080 <        } catch (NullPointerException success) {
1371 <        } catch (Exception ex) {
1372 <            unexpectedException();
1373 <        }
1080 >        } catch (NullPointerException success) {}
1081      }
1082  
1083      /**
1084       * getWaitQueueLength throws NPE if null
1085       */
1086      public void testGetWaitQueueLengthNPE() {
1087 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1087 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1088          try {
1089              lock.getWaitQueueLength(null);
1090              shouldThrow();
1091 <        } catch (NullPointerException success) {
1385 <        } catch (Exception ex) {
1386 <            unexpectedException();
1387 <        }
1091 >        } catch (NullPointerException success) {}
1092      }
1093  
1094  
# Line 1392 | Line 1096 | public class ReentrantReadWriteLockTest
1096       * getWaitingThreads throws NPE if null
1097       */
1098      public void testGetWaitingThreadsNPE() {
1099 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1099 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1100          try {
1101              lock.getWaitingThreads(null);
1102              shouldThrow();
1103 <        } catch (NullPointerException success) {
1400 <        } catch (Exception ex) {
1401 <            unexpectedException();
1402 <        }
1103 >        } catch (NullPointerException success) {}
1104      }
1105  
1106      /**
1107       * hasWaiters throws IAE if not owned
1108       */
1109      public void testHasWaitersIAE() {
1110 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1111 <        final Condition c = (lock.writeLock().newCondition());
1112 <        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1110 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1111 >        final Condition c = lock.writeLock().newCondition();
1112 >        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1113          try {
1114              lock2.hasWaiters(c);
1115              shouldThrow();
1116 <        } catch (IllegalArgumentException success) {
1416 <        } catch (Exception ex) {
1417 <            unexpectedException();
1418 <        }
1116 >        } catch (IllegalArgumentException success) {}
1117      }
1118  
1119      /**
1120       * hasWaiters throws IMSE if not locked
1121       */
1122      public void testHasWaitersIMSE() {
1123 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1124 <        final Condition c = (lock.writeLock().newCondition());
1123 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1124 >        final Condition c = lock.writeLock().newCondition();
1125          try {
1126              lock.hasWaiters(c);
1127              shouldThrow();
1128 <        } catch (IllegalMonitorStateException success) {
1431 <        } catch (Exception ex) {
1432 <            unexpectedException();
1433 <        }
1128 >        } catch (IllegalMonitorStateException success) {}
1129      }
1130  
1131  
# Line 1438 | Line 1133 | public class ReentrantReadWriteLockTest
1133       * getWaitQueueLength throws IAE if not owned
1134       */
1135      public void testGetWaitQueueLengthIAE() {
1136 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1137 <        final Condition c = (lock.writeLock().newCondition());
1138 <        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1136 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1137 >        final Condition c = lock.writeLock().newCondition();
1138 >        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1139          try {
1140              lock2.getWaitQueueLength(c);
1141              shouldThrow();
1142 <        } catch (IllegalArgumentException success) {
1448 <        } catch (Exception ex) {
1449 <            unexpectedException();
1450 <        }
1142 >        } catch (IllegalArgumentException success) {}
1143      }
1144  
1145      /**
1146       * getWaitQueueLength throws IMSE if not locked
1147       */
1148      public void testGetWaitQueueLengthIMSE() {
1149 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1150 <        final Condition c = (lock.writeLock().newCondition());
1149 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1150 >        final Condition c = lock.writeLock().newCondition();
1151          try {
1152              lock.getWaitQueueLength(c);
1153              shouldThrow();
1154 <        } catch (IllegalMonitorStateException success) {
1463 <        } catch (Exception ex) {
1464 <            unexpectedException();
1465 <        }
1154 >        } catch (IllegalMonitorStateException success) {}
1155      }
1156  
1157  
# Line 1470 | Line 1159 | public class ReentrantReadWriteLockTest
1159       * getWaitingThreads throws IAE if not owned
1160       */
1161      public void testGetWaitingThreadsIAE() {
1162 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();  
1163 <        final Condition c = (lock.writeLock().newCondition());
1164 <        final PublicReentrantReadWriteLock lock2 = new PublicReentrantReadWriteLock();  
1162 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1163 >        final Condition c = lock.writeLock().newCondition();
1164 >        final PublicReentrantReadWriteLock lock2 = new PublicReentrantReadWriteLock();
1165          try {
1166              lock2.getWaitingThreads(c);
1167              shouldThrow();
1168 <        } catch (IllegalArgumentException success) {
1480 <        } catch (Exception ex) {
1481 <            unexpectedException();
1482 <        }
1168 >        } catch (IllegalArgumentException success) {}
1169      }
1170  
1171      /**
1172       * getWaitingThreads throws IMSE if not locked
1173       */
1174      public void testGetWaitingThreadsIMSE() {
1175 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();  
1176 <        final Condition c = (lock.writeLock().newCondition());
1175 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1176 >        final Condition c = lock.writeLock().newCondition();
1177          try {
1178              lock.getWaitingThreads(c);
1179              shouldThrow();
1180 <        } catch (IllegalMonitorStateException success) {
1495 <        } catch (Exception ex) {
1496 <            unexpectedException();
1497 <        }
1180 >        } catch (IllegalMonitorStateException success) {}
1181      }
1182  
1183  
1184      /**
1185       * hasWaiters returns true when a thread is waiting, else false
1186       */
1187 <    public void testHasWaiters() {
1188 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1189 <        final Condition c = (lock.writeLock().newCondition());
1190 <        Thread t = new Thread(new Runnable() {
1191 <                public void run() {
1192 <                    try {
1193 <                        lock.writeLock().lock();
1194 <                        threadAssertFalse(lock.hasWaiters(c));
1195 <                        threadAssertEquals(0, lock.getWaitQueueLength(c));
1196 <                        c.await();
1197 <                        lock.writeLock().unlock();
1515 <                    }
1516 <                    catch(InterruptedException e) {
1517 <                        threadUnexpectedException();
1518 <                    }
1519 <                }
1520 <            });
1187 >    public void testHasWaiters() throws InterruptedException {
1188 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1189 >        final Condition c = lock.writeLock().newCondition();
1190 >        Thread t = newStartedThread(new CheckedRunnable() {
1191 >            public void realRun() throws InterruptedException {
1192 >                lock.writeLock().lock();
1193 >                assertFalse(lock.hasWaiters(c));
1194 >                assertEquals(0, lock.getWaitQueueLength(c));
1195 >                c.await();
1196 >                lock.writeLock().unlock();
1197 >            }});
1198  
1199 <        try {
1200 <            t.start();
1201 <            Thread.sleep(SHORT_DELAY_MS);
1202 <            lock.writeLock().lock();
1203 <            assertTrue(lock.hasWaiters(c));
1204 <            assertEquals(1, lock.getWaitQueueLength(c));
1205 <            c.signal();
1206 <            lock.writeLock().unlock();
1207 <            Thread.sleep(SHORT_DELAY_MS);
1208 <            lock.writeLock().lock();
1209 <            assertFalse(lock.hasWaiters(c));
1210 <            assertEquals(0, lock.getWaitQueueLength(c));
1211 <            lock.writeLock().unlock();
1535 <            t.join(SHORT_DELAY_MS);
1536 <            assertFalse(t.isAlive());
1537 <        }
1538 <        catch (Exception ex) {
1539 <            unexpectedException();
1540 <        }
1199 >        Thread.sleep(SHORT_DELAY_MS);
1200 >        lock.writeLock().lock();
1201 >        assertTrue(lock.hasWaiters(c));
1202 >        assertEquals(1, lock.getWaitQueueLength(c));
1203 >        c.signal();
1204 >        lock.writeLock().unlock();
1205 >        Thread.sleep(SHORT_DELAY_MS);
1206 >        lock.writeLock().lock();
1207 >        assertFalse(lock.hasWaiters(c));
1208 >        assertEquals(0, lock.getWaitQueueLength(c));
1209 >        lock.writeLock().unlock();
1210 >        t.join(SHORT_DELAY_MS);
1211 >        assertFalse(t.isAlive());
1212      }
1213  
1214      /**
1215       * getWaitQueueLength returns number of waiting threads
1216       */
1217 <    public void testGetWaitQueueLength() {
1218 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1219 <        final Condition c = (lock.writeLock().newCondition());
1220 <        Thread t = new Thread(new Runnable() {
1221 <                public void run() {
1222 <                    try {
1223 <                        lock.writeLock().lock();
1224 <                        threadAssertFalse(lock.hasWaiters(c));
1225 <                        threadAssertEquals(0, lock.getWaitQueueLength(c));
1226 <                        c.await();
1227 <                        lock.writeLock().unlock();
1557 <                    }
1558 <                    catch(InterruptedException e) {
1559 <                        threadUnexpectedException();
1560 <                    }
1561 <                }
1562 <            });
1217 >    public void testGetWaitQueueLength() throws InterruptedException {
1218 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1219 >        final Condition c = lock.writeLock().newCondition();
1220 >        Thread t = newStartedThread(new CheckedRunnable() {
1221 >            public void realRun() throws InterruptedException {
1222 >                lock.writeLock().lock();
1223 >                assertFalse(lock.hasWaiters(c));
1224 >                assertEquals(0, lock.getWaitQueueLength(c));
1225 >                c.await();
1226 >                lock.writeLock().unlock();
1227 >            }});
1228  
1229 <        try {
1230 <            t.start();
1231 <            Thread.sleep(SHORT_DELAY_MS);
1232 <            lock.writeLock().lock();
1233 <            assertTrue(lock.hasWaiters(c));
1234 <            assertEquals(1, lock.getWaitQueueLength(c));
1235 <            c.signal();
1236 <            lock.writeLock().unlock();
1237 <            Thread.sleep(SHORT_DELAY_MS);
1238 <            lock.writeLock().lock();
1239 <            assertFalse(lock.hasWaiters(c));
1240 <            assertEquals(0, lock.getWaitQueueLength(c));
1241 <            lock.writeLock().unlock();
1577 <            t.join(SHORT_DELAY_MS);
1578 <            assertFalse(t.isAlive());
1579 <        }
1580 <        catch (Exception ex) {
1581 <            unexpectedException();
1582 <        }
1229 >        Thread.sleep(SHORT_DELAY_MS);
1230 >        lock.writeLock().lock();
1231 >        assertTrue(lock.hasWaiters(c));
1232 >        assertEquals(1, lock.getWaitQueueLength(c));
1233 >        c.signal();
1234 >        lock.writeLock().unlock();
1235 >        Thread.sleep(SHORT_DELAY_MS);
1236 >        lock.writeLock().lock();
1237 >        assertFalse(lock.hasWaiters(c));
1238 >        assertEquals(0, lock.getWaitQueueLength(c));
1239 >        lock.writeLock().unlock();
1240 >        t.join(SHORT_DELAY_MS);
1241 >        assertFalse(t.isAlive());
1242      }
1243  
1244  
1245      /**
1246       * getWaitingThreads returns only and all waiting threads
1247       */
1248 <    public void testGetWaitingThreads() {
1249 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();  
1248 >    public void testGetWaitingThreads() throws InterruptedException {
1249 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1250          final Condition c = lock.writeLock().newCondition();
1251 <        Thread t1 = new Thread(new Runnable() {
1252 <                public void run() {
1253 <                    try {
1254 <                        lock.writeLock().lock();
1255 <                        threadAssertTrue(lock.getWaitingThreads(c).isEmpty());
1256 <                        c.await();
1257 <                        lock.writeLock().unlock();
1599 <                    }
1600 <                    catch(InterruptedException e) {
1601 <                        threadUnexpectedException();
1602 <                    }
1603 <                }
1604 <            });
1605 <
1606 <        Thread t2 = new Thread(new Runnable() {
1607 <                public void run() {
1608 <                    try {
1609 <                        lock.writeLock().lock();
1610 <                        threadAssertFalse(lock.getWaitingThreads(c).isEmpty());
1611 <                        c.await();
1612 <                        lock.writeLock().unlock();
1613 <                    }
1614 <                    catch(InterruptedException e) {
1615 <                        threadUnexpectedException();
1616 <                    }
1617 <                }
1618 <            });
1251 >        Thread t1 = new Thread(new CheckedRunnable() {
1252 >            public void realRun() throws InterruptedException {
1253 >                lock.writeLock().lock();
1254 >                assertTrue(lock.getWaitingThreads(c).isEmpty());
1255 >                c.await();
1256 >                lock.writeLock().unlock();
1257 >            }});
1258  
1259 <        try {
1260 <            lock.writeLock().lock();
1261 <            assertTrue(lock.getWaitingThreads(c).isEmpty());
1262 <            lock.writeLock().unlock();
1263 <            t1.start();
1264 <            Thread.sleep(SHORT_DELAY_MS);
1265 <            t2.start();
1266 <            Thread.sleep(SHORT_DELAY_MS);
1267 <            lock.writeLock().lock();
1268 <            assertTrue(lock.hasWaiters(c));
1269 <            assertTrue(lock.getWaitingThreads(c).contains(t1));
1270 <            assertTrue(lock.getWaitingThreads(c).contains(t2));
1271 <            c.signalAll();
1272 <            lock.writeLock().unlock();
1273 <            Thread.sleep(SHORT_DELAY_MS);
1274 <            lock.writeLock().lock();
1275 <            assertFalse(lock.hasWaiters(c));
1276 <            assertTrue(lock.getWaitingThreads(c).isEmpty());
1277 <            lock.writeLock().unlock();
1278 <            t1.join(SHORT_DELAY_MS);
1279 <            t2.join(SHORT_DELAY_MS);
1280 <            assertFalse(t1.isAlive());
1281 <            assertFalse(t2.isAlive());
1282 <        }
1283 <        catch (Exception ex) {
1284 <            unexpectedException();
1285 <        }
1259 >        Thread t2 = new Thread(new CheckedRunnable() {
1260 >            public void realRun() throws InterruptedException {
1261 >                lock.writeLock().lock();
1262 >                assertFalse(lock.getWaitingThreads(c).isEmpty());
1263 >                c.await();
1264 >                lock.writeLock().unlock();
1265 >            }});
1266 >
1267 >        lock.writeLock().lock();
1268 >        assertTrue(lock.getWaitingThreads(c).isEmpty());
1269 >        lock.writeLock().unlock();
1270 >        t1.start();
1271 >        Thread.sleep(SHORT_DELAY_MS);
1272 >        t2.start();
1273 >        Thread.sleep(SHORT_DELAY_MS);
1274 >        lock.writeLock().lock();
1275 >        assertTrue(lock.hasWaiters(c));
1276 >        assertTrue(lock.getWaitingThreads(c).contains(t1));
1277 >        assertTrue(lock.getWaitingThreads(c).contains(t2));
1278 >        c.signalAll();
1279 >        lock.writeLock().unlock();
1280 >        Thread.sleep(SHORT_DELAY_MS);
1281 >        lock.writeLock().lock();
1282 >        assertFalse(lock.hasWaiters(c));
1283 >        assertTrue(lock.getWaitingThreads(c).isEmpty());
1284 >        lock.writeLock().unlock();
1285 >        t1.join(SHORT_DELAY_MS);
1286 >        t2.join(SHORT_DELAY_MS);
1287 >        assertFalse(t1.isAlive());
1288 >        assertFalse(t2.isAlive());
1289      }
1290  
1291      /**

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines