summaryrefslogtreecommitdiff
path: root/jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java
diff options
context:
space:
mode:
authorPrzemyslaw Szczepaniak <pszczepaniak@google.com>2016-03-15 09:37:42 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2016-03-15 09:37:42 +0000
commit37f9fec54f50b2e0898ee2b61d058c4de879e38b (patch)
treed7fd9980442e0d647e1454f54f3d037f4b2280e7 /jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java
parent8d28fc1d531b4b39def9598de89311aff112f955 (diff)
parent5328e07d282bef36ac8b757bbee16a761415b2c4 (diff)
Merge "JSR-166 update"
Diffstat (limited to 'jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java')
-rw-r--r--jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java14
1 files changed, 11 insertions, 3 deletions
diff --git a/jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java b/jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java
index b3c4110297..d102fc63c8 100644
--- a/jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java
+++ b/jsr166-tests/src/test/java/jsr166/AbstractQueuedSynchronizerTest.java
@@ -29,7 +29,7 @@ public class AbstractQueuedSynchronizerTest extends JSR166TestCase {
// main(suite(), args);
// }
// public static Test suite() {
- // return new TestSuite(...);
+ // return new TestSuite(AbstractQueuedSynchronizerTest.class);
// }
/**
@@ -244,25 +244,33 @@ public class AbstractQueuedSynchronizerTest extends JSR166TestCase {
*/
void assertAwaitTimesOut(ConditionObject c, AwaitMethod awaitMethod) {
long timeoutMillis = timeoutMillis();
- long startTime = System.nanoTime();
+ long startTime;
try {
switch (awaitMethod) {
case awaitTimed:
+ startTime = System.nanoTime();
assertFalse(c.await(timeoutMillis, MILLISECONDS));
+ assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
break;
case awaitNanos:
+ startTime = System.nanoTime();
long nanosTimeout = MILLISECONDS.toNanos(timeoutMillis);
long nanosRemaining = c.awaitNanos(nanosTimeout);
assertTrue(nanosRemaining <= 0);
+ assertTrue(nanosRemaining > -MILLISECONDS.toNanos(LONG_DELAY_MS));
+ assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
break;
case awaitUntil:
+ // We shouldn't assume that nanoTime and currentTimeMillis
+ // use the same time source, so don't use nanoTime here.
+ java.util.Date delayedDate = delayedDate(timeoutMillis());
assertFalse(c.awaitUntil(delayedDate(timeoutMillis)));
+ assertTrue(new java.util.Date().getTime() >= delayedDate.getTime());
break;
default:
throw new UnsupportedOperationException();
}
} catch (InterruptedException ie) { threadUnexpectedException(ie); }
- assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
}
/**