--- jsr166/src/test/tck/ReentrantReadWriteLockTest.java 2015/05/24 01:42:14 1.74 +++ jsr166/src/test/tck/ReentrantReadWriteLockTest.java 2015/07/03 01:25:15 1.77 @@ -155,24 +155,26 @@ public class ReentrantReadWriteLockTest enum AwaitMethod { await, awaitTimed, awaitNanos, awaitUntil } /** - * Awaits condition using the specified AwaitMethod. + * Awaits condition "indefinitely" using the specified AwaitMethod. */ void await(Condition c, AwaitMethod awaitMethod) throws InterruptedException { + long timeoutMillis = 2 * LONG_DELAY_MS; switch (awaitMethod) { case await: c.await(); break; case awaitTimed: - assertTrue(c.await(2 * LONG_DELAY_MS, MILLISECONDS)); + assertTrue(c.await(timeoutMillis, MILLISECONDS)); break; case awaitNanos: - long nanosRemaining = c.awaitNanos(MILLISECONDS.toNanos(2 * LONG_DELAY_MS)); - assertTrue(nanosRemaining > 0); + long timeoutNanos = MILLISECONDS.toNanos(timeoutMillis); + long nanosRemaining = c.awaitNanos(timeoutNanos); + assertTrue(nanosRemaining > timeoutNanos / 2); + assertTrue(nanosRemaining <= timeoutNanos); break; case awaitUntil: - java.util.Date d = new java.util.Date(); - assertTrue(c.awaitUntil(new java.util.Date(d.getTime() + 2 * LONG_DELAY_MS))); + assertTrue(c.awaitUntil(delayedDate(timeoutMillis))); break; default: throw new AssertionError(); @@ -968,10 +970,8 @@ public class ReentrantReadWriteLockTest final Condition c = lock.writeLock().newCondition(); lock.writeLock().lock(); long startTime = System.nanoTime(); - long timeoutMillis = 10; - java.util.Date d = new java.util.Date(); - assertFalse(c.awaitUntil(new java.util.Date(d.getTime() + timeoutMillis))); - assertTrue(millisElapsedSince(startTime) >= timeoutMillis); + assertFalse(c.awaitUntil(delayedDate(timeoutMillis()))); + assertTrue(millisElapsedSince(startTime) >= timeoutMillis()); lock.writeLock().unlock(); } catch (InterruptedException fail) { threadUnexpectedException(fail); } }