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.23 by dl, Mon Aug 1 19:53:01 2005 UTC vs.
Revision 1.51 by jsr166, Mon May 2 00:14:09 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());
96 +        assertTrue(rl.writeLock().isHeldByCurrentThread());
97          assertEquals(0, rl.getReadLockCount());
98          rl.writeLock().unlock();
99          assertFalse(rl.isWriteLocked());
100          assertFalse(rl.isWriteLockedByCurrentThread());
101 +        assertFalse(rl.writeLock().isHeldByCurrentThread());
102          assertEquals(0, rl.getReadLockCount());
103          rl.readLock().lock();
104          assertFalse(rl.isWriteLocked());
# Line 103 | 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());
122 +        assertTrue(rl.writeLock().isHeldByCurrentThread());
123          assertEquals(0, rl.getReadLockCount());
124          rl.writeLock().unlock();
125          assertFalse(rl.isWriteLocked());
126          assertFalse(rl.isWriteLockedByCurrentThread());
127 +        assertFalse(rl.writeLock().isHeldByCurrentThread());
128          assertEquals(0, rl.getReadLockCount());
129          rl.readLock().lock();
130          assertFalse(rl.isWriteLocked());
# Line 127 | 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 >        }
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 {
185 <            lock.writeLock().lock();
186 <            t.start();
187 <            t.interrupt();
188 <            lock.writeLock().unlock();
189 <            t.join();
190 <        } catch(Exception e){
191 <            unexpectedException();
192 <        }
193 <    }
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();
210 <            t.interrupt();
211 <            lock.writeLock().unlock();
212 <            t.join();
213 <        } catch(Exception e){
214 <            unexpectedException();
215 <        }
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 <            t.interrupt();
234 <            lock.writeLock().unlock();
235 <            t.join();
236 <        } catch(Exception e){
237 <            unexpectedException();
238 <        }
239 <    }
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 >        t.join();
245 >        releaseLock(lock.writeLock());
246 >    }
247  
248      /**
249       * timed read-tryLock is interruptible
250       */
251 <    public void testReadTryLock_Interrupted() {
252 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
253 <        lock.writeLock().lock();
254 <        Thread t = new Thread(new Runnable() {
255 <                public void run() {
256 <                    try {
257 <                        lock.readLock().tryLock(1000,TimeUnit.MILLISECONDS);
258 <                        threadShouldThrow();
259 <                    } catch(InterruptedException success){}
260 <                }
261 <            });
262 <        try {
256 <            t.start();
257 <            t.interrupt();
258 <            t.join();
259 <        } catch(Exception e){
260 <            unexpectedException();
261 <        }
251 >    public void testReadTryLock_Interrupted() throws InterruptedException {
252 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
253 >        lock.writeLock().lock();
254 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
255 >            public void realRun() throws InterruptedException {
256 >                lock.readLock().tryLock(LONG_DELAY_MS, MILLISECONDS);
257 >            }});
258 >
259 >        Thread.sleep(SHORT_DELAY_MS);
260 >        t.interrupt();
261 >        t.join();
262 >        releaseLock(lock.writeLock());
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();
279 <            lock.writeLock().unlock();
280 <        } catch(Exception e){
281 <            unexpectedException();
282 <        }
283 <    }
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();
299 <            lock.writeLock().unlock();
300 <        } catch(Exception e){
301 <            unexpectedException();
302 <        }
303 <    }
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();
320 <            lock.readLock().unlock();
321 <        } catch(Exception e){
322 <            unexpectedException();
323 <        }
324 <    }
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();
342 <                }
343 <            });
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);
352 <            assertTrue(!t1.isAlive());
353 <            assertTrue(!t2.isAlive());
354 <          
355 <        } catch(Exception e){
356 <            unexpectedException();
357 <        }
358 <    }
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();
376 <                }
377 <            });
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);
386 <            assertTrue(!t1.isAlive());
387 <            assertTrue(!t2.isAlive());
388 <          
389 <        } catch(Exception e){
390 <            unexpectedException();
391 <        }
392 <    }
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();
422 <                }
423 <            });
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);
436 <            assertTrue(!t1.isAlive());
437 <            assertTrue(!t2.isAlive());
438 <          
439 <        } catch(Exception e){
440 <            unexpectedException();
441 <        }
442 <    }
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();
461 <                }
462 <            });
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);
475 <            assertTrue(!t1.isAlive());
476 <            assertTrue(!t2.isAlive());
477 <          
478 <        } catch(Exception e){
479 <            unexpectedException();
480 <        }
481 <    }
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();
501 <                }
502 <            });
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);
515 <            assertTrue(!t1.isAlive());
516 <            assertTrue(!t2.isAlive());
517 <          
518 <        } catch(Exception e){
519 <            unexpectedException();
520 <        }
521 <    }
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();
552 <                }
553 <            });
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);
566 <            assertTrue(!t1.isAlive());
567 <            assertTrue(!t2.isAlive());
568 <          
569 <        } catch(Exception e){
570 <            unexpectedException();
571 <        }
572 <    }
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();
592 <                }
593 <            });
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);
606 <            assertTrue(!t1.isAlive());
607 <            assertTrue(!t2.isAlive());
608 <          
609 <        } catch(Exception e){
610 <            unexpectedException();
611 <        }
612 <    }
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();
632 <                }
633 <            });
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);
649 <            assertTrue(!t1.isAlive());
650 <            assertTrue(!t2.isAlive());
651 <          
652 <        } catch(Exception e){
653 <            unexpectedException();
654 <        }
655 <    }
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 <                }
669 <            });
670 <        try {
671 <            t.start();
672 <            t.join();
673 <            lock.readLock().unlock();
674 <        } catch(Exception e){
675 <            unexpectedException();
676 <        }
677 <    }
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();
602 <            lock.readLock().unlock();
603 <        } catch(Exception e){
604 <            unexpectedException();
605 <        }
606 <    }
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() 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 >
621 >
622 >    /**
623 >     * Fair write tryLock fails when readlocked
624 >     */
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  
701    
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 <            });
718 <        try {
719 <            t.start();
720 <            t.join();
721 <            lock.writeLock().unlock();
722 <        } catch(Exception e){
723 <            unexpectedException();
724 <        }
725 <    }
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 <            });
742 <        try {
743 <            t.start();
744 <            t.join();
745 <            lock.writeLock().unlock();
746 <        } catch(Exception e){
747 <            unexpectedException();
748 <        }
749 <    }
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();
767 <                    }
768 <                    catch(InterruptedException success) {
769 <                    }
770 <                }
771 <            });
772 <        try {
773 <            t.start();
774 <            t.interrupt();
775 <            t.join();
776 <            lock.writeLock().unlock();
777 <        } catch(Exception e){
778 <            unexpectedException();
779 <        }
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 >        t.join();
686 >        releaseLock(lock.writeLock());
687      }
688  
689      /**
690 <     *  read lockInterruptibly succeeds if lock free else is interruptible
690 >     * read lockInterruptibly succeeds if lock free else is interruptible
691       */
692 <    public void testReadLockInterruptibly() {
693 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
694 <        try {
695 <            lock.writeLock().lockInterruptibly();
696 <        } catch(Exception e) {
697 <            unexpectedException();
698 <        }
699 <        Thread t = new Thread(new Runnable() {
700 <                public void run() {
701 <                    try {
702 <                        lock.readLock().lockInterruptibly();
703 <                        threadShouldThrow();
797 <                    }
798 <                    catch(InterruptedException success) {
799 <                    }
800 <                }
801 <            });
802 <        try {
803 <            t.start();
804 <            t.interrupt();
805 <            t.join();
806 <            lock.writeLock().unlock();
807 <        } catch(Exception e){
808 <            unexpectedException();
809 <        }
692 >    public void testReadLockInterruptibly() throws InterruptedException {
693 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
694 >        lock.writeLock().lockInterruptibly();
695 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
696 >            public void realRun() throws InterruptedException {
697 >                lock.readLock().lockInterruptibly();
698 >            }});
699 >
700 >        Thread.sleep(SHORT_DELAY_MS);
701 >        t.interrupt();
702 >        t.join();
703 >        releaseLock(lock.writeLock());
704      }
705  
706      /**
707       * Calling await without holding lock throws IllegalMonitorStateException
708       */
709 <    public void testAwait_IllegalMonitor() {
710 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
709 >    public void testAwait_IllegalMonitor() throws InterruptedException {
710 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
711          final Condition c = lock.writeLock().newCondition();
712          try {
713              c.await();
714              shouldThrow();
715 <        }
822 <        catch (IllegalMonitorStateException success) {
823 <        }
824 <        catch (Exception ex) {
825 <            shouldThrow();
826 <        }
715 >        } catch (IllegalMonitorStateException success) {}
716      }
717  
718      /**
719       * Calling signal without holding lock throws IllegalMonitorStateException
720       */
721      public void testSignal_IllegalMonitor() {
722 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
722 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
723          final Condition c = lock.writeLock().newCondition();
724          try {
725              c.signal();
726              shouldThrow();
727 <        }
839 <        catch (IllegalMonitorStateException success) {
840 <        }
841 <        catch (Exception ex) {
842 <            unexpectedException();
843 <        }
727 >        } catch (IllegalMonitorStateException success) {}
728      }
729  
730      /**
731       * awaitNanos without a signal times out
732       */
733 <    public void testAwaitNanos_Timeout() {
734 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
733 >    public void testAwaitNanos_Timeout() throws InterruptedException {
734 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
735          final Condition c = lock.writeLock().newCondition();
736 <        try {
737 <            lock.writeLock().lock();
738 <            long t = c.awaitNanos(100);
739 <            assertTrue(t <= 0);
740 <            lock.writeLock().unlock();
857 <        }
858 <        catch (Exception ex) {
859 <            unexpectedException();
860 <        }
736 >
737 >        lock.writeLock().lock();
738 >        long t = c.awaitNanos(100);
739 >        assertTrue(t <= 0);
740 >        lock.writeLock().unlock();
741      }
742  
743  
744      /**
745 <     *  timed await without a signal times out
745 >     * timed await without a signal times out
746       */
747 <    public void testAwait_Timeout() {
748 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
747 >    public void testAwait_Timeout() throws InterruptedException {
748 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
749          final Condition c = lock.writeLock().newCondition();
750 <        try {
751 <            lock.writeLock().lock();
752 <            lock.writeLock().unlock();
873 <        }
874 <        catch (Exception ex) {
875 <            unexpectedException();
876 <        }
750 >        lock.writeLock().lock();
751 >        assertFalse(c.await(SHORT_DELAY_MS, MILLISECONDS));
752 >        lock.writeLock().unlock();
753      }
754  
755      /**
756       * awaitUntil without a signal times out
757       */
758 <    public void testAwaitUntil_Timeout() {
759 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
758 >    public void testAwaitUntil_Timeout() throws InterruptedException {
759 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
760          final Condition c = lock.writeLock().newCondition();
761 <        try {
762 <            lock.writeLock().lock();
763 <            java.util.Date d = new java.util.Date();
764 <            lock.writeLock().unlock();
889 <        }
890 <        catch (Exception ex) {
891 <            unexpectedException();
892 <        }
761 >        lock.writeLock().lock();
762 >        java.util.Date d = new java.util.Date();
763 >        assertFalse(c.awaitUntil(new java.util.Date(d.getTime() + 10)));
764 >        lock.writeLock().unlock();
765      }
766  
767      /**
768       * await returns when signalled
769       */
770 <    public void testAwait() {
771 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
770 >    public void testAwait() throws InterruptedException {
771 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
772          final Condition c = lock.writeLock().newCondition();
773 <        Thread t = new Thread(new Runnable() {
774 <                public void run() {
775 <                    try {
776 <                        lock.writeLock().lock();
777 <                        c.await();
778 <                        lock.writeLock().unlock();
907 <                    }
908 <                    catch(InterruptedException e) {
909 <                        threadUnexpectedException();
910 <                    }
911 <                }
912 <            });
773 >        Thread t = newStartedThread(new CheckedRunnable() {
774 >            public void realRun() throws InterruptedException {
775 >                lock.writeLock().lock();
776 >                c.await();
777 >                lock.writeLock().unlock();
778 >            }});
779  
780 <        try {
781 <            t.start();
782 <            Thread.sleep(SHORT_DELAY_MS);
783 <            lock.writeLock().lock();
784 <            c.signal();
785 <            lock.writeLock().unlock();
920 <            t.join(SHORT_DELAY_MS);
921 <            assertFalse(t.isAlive());
922 <        }
923 <        catch (Exception ex) {
924 <            unexpectedException();
925 <        }
780 >        Thread.sleep(SHORT_DELAY_MS);
781 >        lock.writeLock().lock();
782 >        c.signal();
783 >        lock.writeLock().unlock();
784 >        t.join(SHORT_DELAY_MS);
785 >        assertFalse(t.isAlive());
786      }
787  
788      /** A helper class for uninterruptible wait tests */
789      class UninterruptableThread extends Thread {
790          private Lock lock;
791          private Condition c;
792 <        
792 >
793          public volatile boolean canAwake = false;
794          public volatile boolean interrupted = false;
795          public volatile boolean lockStarted = false;
796 <        
796 >
797          public UninterruptableThread(Lock lock, Condition c) {
798              this.lock = lock;
799              this.c = c;
800          }
801 <        
801 >
802          public synchronized void run() {
803              lock.lock();
804              lockStarted = true;
805 <            
805 >
806              while (!canAwake) {
807                  c.awaitUninterruptibly();
808              }
809 <            
809 >
810              interrupted = isInterrupted();
811              lock.unlock();
812          }
# Line 955 | Line 815 | public class ReentrantReadWriteLockTest
815      /**
816       * awaitUninterruptibly doesn't abort on interrupt
817       */
818 <    public void testAwaitUninterruptibly() {
818 >    public void testAwaitUninterruptibly() throws InterruptedException {
819          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
820          final Condition c = lock.writeLock().newCondition();
821          UninterruptableThread thread = new UninterruptableThread(lock.writeLock(), c);
822  
823 <        try {
964 <            thread.start();
965 <
966 <            while (!thread.lockStarted) {
967 <                Thread.sleep(100);
968 <            }
823 >        thread.start();
824  
825 <            lock.writeLock().lock();
826 <            try {
827 <                thread.interrupt();
973 <                thread.canAwake = true;
974 <                c.signal();
975 <            } finally {
976 <                lock.writeLock().unlock();
977 <            }
825 >        while (!thread.lockStarted) {
826 >            Thread.sleep(100);
827 >        }
828  
829 <            thread.join();
830 <            assertTrue(thread.interrupted);
831 <            assertFalse(thread.isAlive());
832 <        } catch (Exception ex) {
833 <            unexpectedException();
829 >        lock.writeLock().lock();
830 >        try {
831 >            thread.interrupt();
832 >            thread.canAwake = true;
833 >            c.signal();
834 >        } finally {
835 >            lock.writeLock().unlock();
836          }
837 +
838 +        thread.join();
839 +        assertTrue(thread.interrupted);
840 +        assertFalse(thread.isAlive());
841      }
842  
843      /**
844       * await is interruptible
845       */
846 <    public void testAwait_Interrupt() {
847 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
846 >    public void testAwait_Interrupt() throws InterruptedException {
847 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
848          final Condition c = lock.writeLock().newCondition();
849 <        Thread t = new Thread(new Runnable() {
850 <                public void run() {
851 <                    try {
852 <                        lock.writeLock().lock();
853 <                        c.await();
854 <                        lock.writeLock().unlock();
999 <                        threadShouldThrow();
1000 <                    }
1001 <                    catch(InterruptedException success) {
1002 <                    }
1003 <                }
1004 <            });
849 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
850 >            public void realRun() throws InterruptedException {
851 >                lock.writeLock().lock();
852 >                c.await();
853 >                lock.writeLock().unlock();
854 >            }});
855  
856 <        try {
857 <            t.start();
858 <            Thread.sleep(SHORT_DELAY_MS);
859 <            t.interrupt();
1010 <            t.join(SHORT_DELAY_MS);
1011 <            assertFalse(t.isAlive());
1012 <        }
1013 <        catch (Exception ex) {
1014 <            unexpectedException();
1015 <        }
856 >        Thread.sleep(SHORT_DELAY_MS);
857 >        t.interrupt();
858 >        t.join(SHORT_DELAY_MS);
859 >        assertFalse(t.isAlive());
860      }
861  
862      /**
863       * awaitNanos is interruptible
864       */
865 <    public void testAwaitNanos_Interrupt() {
866 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
865 >    public void testAwaitNanos_Interrupt() throws InterruptedException {
866 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
867          final Condition c = lock.writeLock().newCondition();
868 <        Thread t = new Thread(new Runnable() {
869 <                public void run() {
870 <                    try {
871 <                        lock.writeLock().lock();
872 <                        c.awaitNanos(SHORT_DELAY_MS * 2 * 1000000);
873 <                        lock.writeLock().unlock();
1030 <                        threadShouldThrow();
1031 <                    }
1032 <                    catch(InterruptedException success) {
1033 <                    }
1034 <                }
1035 <            });
868 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
869 >            public void realRun() throws InterruptedException {
870 >                lock.writeLock().lock();
871 >                c.awaitNanos(MILLISECONDS.toNanos(LONG_DELAY_MS));
872 >                lock.writeLock().unlock();
873 >            }});
874  
875 <        try {
876 <            t.start();
877 <            Thread.sleep(SHORT_DELAY_MS);
878 <            t.interrupt();
1041 <            t.join(SHORT_DELAY_MS);
1042 <            assertFalse(t.isAlive());
1043 <        }
1044 <        catch (Exception ex) {
1045 <            unexpectedException();
1046 <        }
875 >        Thread.sleep(SHORT_DELAY_MS);
876 >        t.interrupt();
877 >        t.join(SHORT_DELAY_MS);
878 >        assertFalse(t.isAlive());
879      }
880  
881      /**
882       * awaitUntil is interruptible
883       */
884 <    public void testAwaitUntil_Interrupt() {
885 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
884 >    public void testAwaitUntil_Interrupt() throws InterruptedException {
885 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
886          final Condition c = lock.writeLock().newCondition();
887 <        Thread t = new Thread(new Runnable() {
888 <                public void run() {
889 <                    try {
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();
1062 <                        threadShouldThrow();
1063 <                    }
1064 <                    catch(InterruptedException success) {
1065 <                    }
1066 <                }
1067 <            });
887 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
888 >            public void realRun() throws InterruptedException {
889 >                lock.writeLock().lock();
890 >                java.util.Date d = new java.util.Date();
891 >                c.awaitUntil(new java.util.Date(d.getTime() + 10000));
892 >                lock.writeLock().unlock();
893 >            }});
894  
895 <        try {
896 <            t.start();
897 <            Thread.sleep(SHORT_DELAY_MS);
898 <            t.interrupt();
1073 <            t.join(SHORT_DELAY_MS);
1074 <            assertFalse(t.isAlive());
1075 <        }
1076 <        catch (Exception ex) {
1077 <            unexpectedException();
1078 <        }
895 >        Thread.sleep(SHORT_DELAY_MS);
896 >        t.interrupt();
897 >        t.join(SHORT_DELAY_MS);
898 >        assertFalse(t.isAlive());
899      }
900  
901      /**
902       * signalAll wakes up all threads
903       */
904 <    public void testSignalAll() {
905 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();      
904 >    public void testSignalAll() throws InterruptedException {
905 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
906          final Condition c = lock.writeLock().newCondition();
907 <        Thread t1 = new Thread(new Runnable() {
908 <                public void run() {
909 <                    try {
910 <                        lock.writeLock().lock();
911 <                        c.await();
912 <                        lock.writeLock().unlock();
1093 <                    }
1094 <                    catch(InterruptedException e) {
1095 <                        threadUnexpectedException();
1096 <                    }
1097 <                }
1098 <            });
1099 <
1100 <        Thread t2 = new Thread(new Runnable() {
1101 <                public void run() {
1102 <                    try {
1103 <                        lock.writeLock().lock();
1104 <                        c.await();
1105 <                        lock.writeLock().unlock();
1106 <                    }
1107 <                    catch(InterruptedException e) {
1108 <                        threadUnexpectedException();
1109 <                    }
1110 <                }
1111 <            });
907 >        Thread t1 = newStartedThread(new CheckedRunnable() {
908 >            public void realRun() throws InterruptedException {
909 >                lock.writeLock().lock();
910 >                c.await();
911 >                lock.writeLock().unlock();
912 >            }});
913  
914 <        try {
915 <            t1.start();
916 <            t2.start();
917 <            Thread.sleep(SHORT_DELAY_MS);
918 <            lock.writeLock().lock();
919 <            c.signalAll();
920 <            lock.writeLock().unlock();
921 <            t1.join(SHORT_DELAY_MS);
922 <            t2.join(SHORT_DELAY_MS);
923 <            assertFalse(t1.isAlive());
924 <            assertFalse(t2.isAlive());
925 <        }
926 <        catch (Exception ex) {
927 <            unexpectedException();
928 <        }
914 >        Thread t2 = newStartedThread(new CheckedRunnable() {
915 >            public void realRun() throws InterruptedException {
916 >                lock.writeLock().lock();
917 >                c.await();
918 >                lock.writeLock().unlock();
919 >            }});
920 >
921 >        Thread.sleep(SHORT_DELAY_MS);
922 >        lock.writeLock().lock();
923 >        c.signalAll();
924 >        lock.writeLock().unlock();
925 >        t1.join(SHORT_DELAY_MS);
926 >        t2.join(SHORT_DELAY_MS);
927 >        assertFalse(t1.isAlive());
928 >        assertFalse(t2.isAlive());
929      }
930  
931      /**
932       * A serialized lock deserializes as unlocked
933       */
934 <    public void testSerialization() {
934 >    public void testSerialization() throws Exception {
935          ReentrantReadWriteLock l = new ReentrantReadWriteLock();
936          l.readLock().lock();
937          l.readLock().unlock();
938  
939 <        try {
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();
1148 <            r.readLock().unlock();
1149 <        } catch(Exception e){
1150 <            e.printStackTrace();
1151 <            unexpectedException();
1152 <        }
939 >        ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
940 >        ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
941 >        out.writeObject(l);
942 >        out.close();
943 >
944 >        ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
945 >        ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
946 >        ReentrantReadWriteLock r = (ReentrantReadWriteLock) in.readObject();
947 >        r.readLock().lock();
948 >        r.readLock().unlock();
949      }
950  
951      /**
952       * hasQueuedThreads reports whether there are waiting threads
953       */
954 <    public void testhasQueuedThreads() {
955 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
954 >    public void testhasQueuedThreads() throws InterruptedException {
955 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
956          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
957          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
958 <        try {
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();
1179 <        } catch(Exception e){
1180 <            unexpectedException();
1181 <        }
1182 <    }
958 >        assertFalse(lock.hasQueuedThreads());
959 >        lock.writeLock().lock();
960 >        t1.start();
961 >        Thread.sleep(SHORT_DELAY_MS);
962 >        assertTrue(lock.hasQueuedThreads());
963 >        t2.start();
964 >        Thread.sleep(SHORT_DELAY_MS);
965 >        assertTrue(lock.hasQueuedThreads());
966 >        t1.interrupt();
967 >        Thread.sleep(SHORT_DELAY_MS);
968 >        assertTrue(lock.hasQueuedThreads());
969 >        lock.writeLock().unlock();
970 >        Thread.sleep(SHORT_DELAY_MS);
971 >        assertFalse(lock.hasQueuedThreads());
972 >        t1.join();
973 >        t2.join();
974 >    }
975  
976      /**
977       * hasQueuedThread(null) throws NPE
978       */
979 <    public void testHasQueuedThreadNPE() {
980 <        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
979 >    public void testHasQueuedThreadNPE() {
980 >        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
981          try {
982              sync.hasQueuedThread(null);
983              shouldThrow();
984 <        } catch (NullPointerException success) {
1193 <        }
984 >        } catch (NullPointerException success) {}
985      }
986  
987      /**
988       * hasQueuedThread reports whether a thread is queued.
989       */
990 <    public void testHasQueuedThread() {
991 <        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
990 >    public void testHasQueuedThread() throws InterruptedException {
991 >        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
992          Thread t1 = new Thread(new InterruptedLockRunnable(sync));
993          Thread t2 = new Thread(new InterruptibleLockRunnable(sync));
994 <        try {
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();
1225 <        } catch(Exception e){
1226 <            unexpectedException();
1227 <        }
1228 <    }
994 >        assertFalse(sync.hasQueuedThread(t1));
995 >        assertFalse(sync.hasQueuedThread(t2));
996 >        sync.writeLock().lock();
997 >        t1.start();
998 >        Thread.sleep(SHORT_DELAY_MS);
999 >        assertTrue(sync.hasQueuedThread(t1));
1000 >        t2.start();
1001 >        Thread.sleep(SHORT_DELAY_MS);
1002 >        assertTrue(sync.hasQueuedThread(t1));
1003 >        assertTrue(sync.hasQueuedThread(t2));
1004 >        t1.interrupt();
1005 >        Thread.sleep(SHORT_DELAY_MS);
1006 >        assertFalse(sync.hasQueuedThread(t1));
1007 >        assertTrue(sync.hasQueuedThread(t2));
1008 >        sync.writeLock().unlock();
1009 >        Thread.sleep(SHORT_DELAY_MS);
1010 >        assertFalse(sync.hasQueuedThread(t1));
1011 >        Thread.sleep(SHORT_DELAY_MS);
1012 >        assertFalse(sync.hasQueuedThread(t2));
1013 >        t1.join();
1014 >        t2.join();
1015 >    }
1016  
1017  
1018      /**
1019       * getQueueLength reports number of waiting threads
1020       */
1021 <    public void testGetQueueLength() {
1022 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1021 >    public void testGetQueueLength() throws InterruptedException {
1022 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1023          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1024          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1025 <        try {
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();
1255 <        } catch(Exception e){
1256 <            unexpectedException();
1257 <        }
1258 <    }
1025 >        assertEquals(0, lock.getQueueLength());
1026 >        lock.writeLock().lock();
1027 >        t1.start();
1028 >        Thread.sleep(SHORT_DELAY_MS);
1029 >        assertEquals(1, lock.getQueueLength());
1030 >        t2.start();
1031 >        Thread.sleep(SHORT_DELAY_MS);
1032 >        assertEquals(2, lock.getQueueLength());
1033 >        t1.interrupt();
1034 >        Thread.sleep(SHORT_DELAY_MS);
1035 >        assertEquals(1, lock.getQueueLength());
1036 >        lock.writeLock().unlock();
1037 >        Thread.sleep(SHORT_DELAY_MS);
1038 >        assertEquals(0, lock.getQueueLength());
1039 >        t1.join();
1040 >        t2.join();
1041 >    }
1042  
1043      /**
1044       * getQueuedThreads includes waiting threads
1045       */
1046 <    public void testGetQueuedThreads() {
1047 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1046 >    public void testGetQueuedThreads() throws InterruptedException {
1047 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1048          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1049          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1050 <        try {
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();
1287 <        } catch(Exception e){
1288 <            unexpectedException();
1289 <        }
1290 <    }
1050 >        assertTrue(lock.getQueuedThreads().isEmpty());
1051 >        lock.writeLock().lock();
1052 >        assertTrue(lock.getQueuedThreads().isEmpty());
1053 >        t1.start();
1054 >        Thread.sleep(SHORT_DELAY_MS);
1055 >        assertTrue(lock.getQueuedThreads().contains(t1));
1056 >        t2.start();
1057 >        Thread.sleep(SHORT_DELAY_MS);
1058 >        assertTrue(lock.getQueuedThreads().contains(t1));
1059 >        assertTrue(lock.getQueuedThreads().contains(t2));
1060 >        t1.interrupt();
1061 >        Thread.sleep(SHORT_DELAY_MS);
1062 >        assertFalse(lock.getQueuedThreads().contains(t1));
1063 >        assertTrue(lock.getQueuedThreads().contains(t2));
1064 >        lock.writeLock().unlock();
1065 >        Thread.sleep(SHORT_DELAY_MS);
1066 >        assertTrue(lock.getQueuedThreads().isEmpty());
1067 >        t1.join();
1068 >        t2.join();
1069 >    }
1070  
1071      /**
1072       * hasWaiters throws NPE if null
1073       */
1074      public void testHasWaitersNPE() {
1075 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1075 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1076          try {
1077              lock.hasWaiters(null);
1078              shouldThrow();
1079 <        } catch (NullPointerException success) {
1301 <        } catch (Exception ex) {
1302 <            unexpectedException();
1303 <        }
1079 >        } catch (NullPointerException success) {}
1080      }
1081  
1082      /**
1083       * getWaitQueueLength throws NPE if null
1084       */
1085      public void testGetWaitQueueLengthNPE() {
1086 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1086 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1087          try {
1088              lock.getWaitQueueLength(null);
1089              shouldThrow();
1090 <        } catch (NullPointerException success) {
1315 <        } catch (Exception ex) {
1316 <            unexpectedException();
1317 <        }
1090 >        } catch (NullPointerException success) {}
1091      }
1092  
1093  
# Line 1322 | Line 1095 | public class ReentrantReadWriteLockTest
1095       * getWaitingThreads throws NPE if null
1096       */
1097      public void testGetWaitingThreadsNPE() {
1098 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1098 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1099          try {
1100              lock.getWaitingThreads(null);
1101              shouldThrow();
1102 <        } catch (NullPointerException success) {
1330 <        } catch (Exception ex) {
1331 <            unexpectedException();
1332 <        }
1102 >        } catch (NullPointerException success) {}
1103      }
1104  
1105      /**
1106       * hasWaiters throws IAE if not owned
1107       */
1108      public void testHasWaitersIAE() {
1109 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1110 <        final Condition c = (lock.writeLock().newCondition());
1111 <        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1109 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1110 >        final Condition c = lock.writeLock().newCondition();
1111 >        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1112          try {
1113              lock2.hasWaiters(c);
1114              shouldThrow();
1115 <        } catch (IllegalArgumentException success) {
1346 <        } catch (Exception ex) {
1347 <            unexpectedException();
1348 <        }
1115 >        } catch (IllegalArgumentException success) {}
1116      }
1117  
1118      /**
1119       * hasWaiters throws IMSE if not locked
1120       */
1121      public void testHasWaitersIMSE() {
1122 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1123 <        final Condition c = (lock.writeLock().newCondition());
1122 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1123 >        final Condition c = lock.writeLock().newCondition();
1124          try {
1125              lock.hasWaiters(c);
1126              shouldThrow();
1127 <        } catch (IllegalMonitorStateException success) {
1361 <        } catch (Exception ex) {
1362 <            unexpectedException();
1363 <        }
1127 >        } catch (IllegalMonitorStateException success) {}
1128      }
1129  
1130  
# Line 1368 | Line 1132 | public class ReentrantReadWriteLockTest
1132       * getWaitQueueLength throws IAE if not owned
1133       */
1134      public void testGetWaitQueueLengthIAE() {
1135 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1136 <        final Condition c = (lock.writeLock().newCondition());
1137 <        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1135 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1136 >        final Condition c = lock.writeLock().newCondition();
1137 >        final ReentrantReadWriteLock lock2 = new ReentrantReadWriteLock();
1138          try {
1139              lock2.getWaitQueueLength(c);
1140              shouldThrow();
1141 <        } catch (IllegalArgumentException success) {
1378 <        } catch (Exception ex) {
1379 <            unexpectedException();
1380 <        }
1141 >        } catch (IllegalArgumentException success) {}
1142      }
1143  
1144      /**
1145       * getWaitQueueLength throws IMSE if not locked
1146       */
1147      public void testGetWaitQueueLengthIMSE() {
1148 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1149 <        final Condition c = (lock.writeLock().newCondition());
1148 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1149 >        final Condition c = lock.writeLock().newCondition();
1150          try {
1151              lock.getWaitQueueLength(c);
1152              shouldThrow();
1153 <        } catch (IllegalMonitorStateException success) {
1393 <        } catch (Exception ex) {
1394 <            unexpectedException();
1395 <        }
1153 >        } catch (IllegalMonitorStateException success) {}
1154      }
1155  
1156  
# Line 1400 | Line 1158 | public class ReentrantReadWriteLockTest
1158       * getWaitingThreads throws IAE if not owned
1159       */
1160      public void testGetWaitingThreadsIAE() {
1161 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();  
1162 <        final Condition c = (lock.writeLock().newCondition());
1163 <        final PublicReentrantReadWriteLock lock2 = new PublicReentrantReadWriteLock();  
1161 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1162 >        final Condition c = lock.writeLock().newCondition();
1163 >        final PublicReentrantReadWriteLock lock2 = new PublicReentrantReadWriteLock();
1164          try {
1165              lock2.getWaitingThreads(c);
1166              shouldThrow();
1167 <        } catch (IllegalArgumentException success) {
1410 <        } catch (Exception ex) {
1411 <            unexpectedException();
1412 <        }
1167 >        } catch (IllegalArgumentException success) {}
1168      }
1169  
1170      /**
1171       * getWaitingThreads throws IMSE if not locked
1172       */
1173      public void testGetWaitingThreadsIMSE() {
1174 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();  
1175 <        final Condition c = (lock.writeLock().newCondition());
1174 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1175 >        final Condition c = lock.writeLock().newCondition();
1176          try {
1177              lock.getWaitingThreads(c);
1178              shouldThrow();
1179 <        } catch (IllegalMonitorStateException success) {
1425 <        } catch (Exception ex) {
1426 <            unexpectedException();
1427 <        }
1179 >        } catch (IllegalMonitorStateException success) {}
1180      }
1181  
1182  
1183      /**
1184       * hasWaiters returns true when a thread is waiting, else false
1185       */
1186 <    public void testHasWaiters() {
1187 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1188 <        final Condition c = (lock.writeLock().newCondition());
1189 <        Thread t = new Thread(new Runnable() {
1190 <                public void run() {
1191 <                    try {
1192 <                        lock.writeLock().lock();
1193 <                        threadAssertFalse(lock.hasWaiters(c));
1194 <                        threadAssertEquals(0, lock.getWaitQueueLength(c));
1195 <                        c.await();
1196 <                        lock.writeLock().unlock();
1445 <                    }
1446 <                    catch(InterruptedException e) {
1447 <                        threadUnexpectedException();
1448 <                    }
1449 <                }
1450 <            });
1186 >    public void testHasWaiters() throws InterruptedException {
1187 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1188 >        final Condition c = lock.writeLock().newCondition();
1189 >        Thread t = newStartedThread(new CheckedRunnable() {
1190 >            public void realRun() throws InterruptedException {
1191 >                lock.writeLock().lock();
1192 >                assertFalse(lock.hasWaiters(c));
1193 >                assertEquals(0, lock.getWaitQueueLength(c));
1194 >                c.await();
1195 >                lock.writeLock().unlock();
1196 >            }});
1197  
1198 <        try {
1199 <            t.start();
1200 <            Thread.sleep(SHORT_DELAY_MS);
1201 <            lock.writeLock().lock();
1202 <            assertTrue(lock.hasWaiters(c));
1203 <            assertEquals(1, lock.getWaitQueueLength(c));
1204 <            c.signal();
1205 <            lock.writeLock().unlock();
1206 <            Thread.sleep(SHORT_DELAY_MS);
1207 <            lock.writeLock().lock();
1208 <            assertFalse(lock.hasWaiters(c));
1209 <            assertEquals(0, lock.getWaitQueueLength(c));
1210 <            lock.writeLock().unlock();
1465 <            t.join(SHORT_DELAY_MS);
1466 <            assertFalse(t.isAlive());
1467 <        }
1468 <        catch (Exception ex) {
1469 <            unexpectedException();
1470 <        }
1198 >        Thread.sleep(SHORT_DELAY_MS);
1199 >        lock.writeLock().lock();
1200 >        assertTrue(lock.hasWaiters(c));
1201 >        assertEquals(1, lock.getWaitQueueLength(c));
1202 >        c.signal();
1203 >        lock.writeLock().unlock();
1204 >        Thread.sleep(SHORT_DELAY_MS);
1205 >        lock.writeLock().lock();
1206 >        assertFalse(lock.hasWaiters(c));
1207 >        assertEquals(0, lock.getWaitQueueLength(c));
1208 >        lock.writeLock().unlock();
1209 >        t.join(SHORT_DELAY_MS);
1210 >        assertFalse(t.isAlive());
1211      }
1212  
1213      /**
1214       * getWaitQueueLength returns number of waiting threads
1215       */
1216 <    public void testGetWaitQueueLength() {
1217 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1218 <        final Condition c = (lock.writeLock().newCondition());
1219 <        Thread t = new Thread(new Runnable() {
1220 <                public void run() {
1221 <                    try {
1222 <                        lock.writeLock().lock();
1223 <                        threadAssertFalse(lock.hasWaiters(c));
1224 <                        threadAssertEquals(0, lock.getWaitQueueLength(c));
1225 <                        c.await();
1226 <                        lock.writeLock().unlock();
1487 <                    }
1488 <                    catch(InterruptedException e) {
1489 <                        threadUnexpectedException();
1490 <                    }
1491 <                }
1492 <            });
1216 >    public void testGetWaitQueueLength() throws InterruptedException {
1217 >        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1218 >        final Condition c = lock.writeLock().newCondition();
1219 >        Thread t = newStartedThread(new CheckedRunnable() {
1220 >            public void realRun() throws InterruptedException {
1221 >                lock.writeLock().lock();
1222 >                assertFalse(lock.hasWaiters(c));
1223 >                assertEquals(0, lock.getWaitQueueLength(c));
1224 >                c.await();
1225 >                lock.writeLock().unlock();
1226 >            }});
1227  
1228 <        try {
1229 <            t.start();
1230 <            Thread.sleep(SHORT_DELAY_MS);
1231 <            lock.writeLock().lock();
1232 <            assertTrue(lock.hasWaiters(c));
1233 <            assertEquals(1, lock.getWaitQueueLength(c));
1234 <            c.signal();
1235 <            lock.writeLock().unlock();
1236 <            Thread.sleep(SHORT_DELAY_MS);
1237 <            lock.writeLock().lock();
1238 <            assertFalse(lock.hasWaiters(c));
1239 <            assertEquals(0, lock.getWaitQueueLength(c));
1240 <            lock.writeLock().unlock();
1507 <            t.join(SHORT_DELAY_MS);
1508 <            assertFalse(t.isAlive());
1509 <        }
1510 <        catch (Exception ex) {
1511 <            unexpectedException();
1512 <        }
1228 >        Thread.sleep(SHORT_DELAY_MS);
1229 >        lock.writeLock().lock();
1230 >        assertTrue(lock.hasWaiters(c));
1231 >        assertEquals(1, lock.getWaitQueueLength(c));
1232 >        c.signal();
1233 >        lock.writeLock().unlock();
1234 >        Thread.sleep(SHORT_DELAY_MS);
1235 >        lock.writeLock().lock();
1236 >        assertFalse(lock.hasWaiters(c));
1237 >        assertEquals(0, lock.getWaitQueueLength(c));
1238 >        lock.writeLock().unlock();
1239 >        t.join(SHORT_DELAY_MS);
1240 >        assertFalse(t.isAlive());
1241      }
1242  
1243  
1244      /**
1245       * getWaitingThreads returns only and all waiting threads
1246       */
1247 <    public void testGetWaitingThreads() {
1248 <        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();  
1247 >    public void testGetWaitingThreads() throws InterruptedException {
1248 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1249          final Condition c = lock.writeLock().newCondition();
1250 <        Thread t1 = new Thread(new Runnable() {
1251 <                public void run() {
1252 <                    try {
1253 <                        lock.writeLock().lock();
1254 <                        threadAssertTrue(lock.getWaitingThreads(c).isEmpty());
1255 <                        c.await();
1256 <                        lock.writeLock().unlock();
1529 <                    }
1530 <                    catch(InterruptedException e) {
1531 <                        threadUnexpectedException();
1532 <                    }
1533 <                }
1534 <            });
1535 <
1536 <        Thread t2 = new Thread(new Runnable() {
1537 <                public void run() {
1538 <                    try {
1539 <                        lock.writeLock().lock();
1540 <                        threadAssertFalse(lock.getWaitingThreads(c).isEmpty());
1541 <                        c.await();
1542 <                        lock.writeLock().unlock();
1543 <                    }
1544 <                    catch(InterruptedException e) {
1545 <                        threadUnexpectedException();
1546 <                    }
1547 <                }
1548 <            });
1250 >        Thread t1 = new Thread(new CheckedRunnable() {
1251 >            public void realRun() throws InterruptedException {
1252 >                lock.writeLock().lock();
1253 >                assertTrue(lock.getWaitingThreads(c).isEmpty());
1254 >                c.await();
1255 >                lock.writeLock().unlock();
1256 >            }});
1257  
1258 <        try {
1259 <            lock.writeLock().lock();
1260 <            assertTrue(lock.getWaitingThreads(c).isEmpty());
1261 <            lock.writeLock().unlock();
1262 <            t1.start();
1263 <            Thread.sleep(SHORT_DELAY_MS);
1264 <            t2.start();
1265 <            Thread.sleep(SHORT_DELAY_MS);
1266 <            lock.writeLock().lock();
1267 <            assertTrue(lock.hasWaiters(c));
1268 <            assertTrue(lock.getWaitingThreads(c).contains(t1));
1269 <            assertTrue(lock.getWaitingThreads(c).contains(t2));
1270 <            c.signalAll();
1271 <            lock.writeLock().unlock();
1272 <            Thread.sleep(SHORT_DELAY_MS);
1273 <            lock.writeLock().lock();
1274 <            assertFalse(lock.hasWaiters(c));
1275 <            assertTrue(lock.getWaitingThreads(c).isEmpty());
1276 <            lock.writeLock().unlock();
1277 <            t1.join(SHORT_DELAY_MS);
1278 <            t2.join(SHORT_DELAY_MS);
1279 <            assertFalse(t1.isAlive());
1280 <            assertFalse(t2.isAlive());
1281 <        }
1282 <        catch (Exception ex) {
1283 <            unexpectedException();
1284 <        }
1258 >        Thread t2 = new Thread(new CheckedRunnable() {
1259 >            public void realRun() throws InterruptedException {
1260 >                lock.writeLock().lock();
1261 >                assertFalse(lock.getWaitingThreads(c).isEmpty());
1262 >                c.await();
1263 >                lock.writeLock().unlock();
1264 >            }});
1265 >
1266 >        lock.writeLock().lock();
1267 >        assertTrue(lock.getWaitingThreads(c).isEmpty());
1268 >        lock.writeLock().unlock();
1269 >        t1.start();
1270 >        Thread.sleep(SHORT_DELAY_MS);
1271 >        t2.start();
1272 >        Thread.sleep(SHORT_DELAY_MS);
1273 >        lock.writeLock().lock();
1274 >        assertTrue(lock.hasWaiters(c));
1275 >        assertTrue(lock.getWaitingThreads(c).contains(t1));
1276 >        assertTrue(lock.getWaitingThreads(c).contains(t2));
1277 >        c.signalAll();
1278 >        lock.writeLock().unlock();
1279 >        Thread.sleep(SHORT_DELAY_MS);
1280 >        lock.writeLock().lock();
1281 >        assertFalse(lock.hasWaiters(c));
1282 >        assertTrue(lock.getWaitingThreads(c).isEmpty());
1283 >        lock.writeLock().unlock();
1284 >        t1.join(SHORT_DELAY_MS);
1285 >        t2.join(SHORT_DELAY_MS);
1286 >        assertFalse(t1.isAlive());
1287 >        assertFalse(t2.isAlive());
1288      }
1289  
1290      /**

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines