--- jsr166/src/test/tck/CompletableFutureTest.java 2018/01/30 04:07:09 1.193 +++ jsr166/src/test/tck/CompletableFutureTest.java 2018/07/22 20:17:46 1.196 @@ -59,9 +59,13 @@ public class CompletableFutureTest exten assertFalse(f.isDone()); assertFalse(f.isCancelled()); assertTrue(f.toString().matches(".*\\[.*Not completed.*\\]")); + + Object result = null; try { - assertNull(f.getNow(null)); + result = f.getNow(null); } catch (Throwable fail) { threadUnexpectedException(fail); } + assertNull(result); + try { f.get(randomExpiredTimeout(), randomTimeUnit()); shouldThrow(); @@ -73,11 +77,15 @@ public class CompletableFutureTest exten void checkCompletedNormally(CompletableFuture f, T value) { checkTimedGet(f, value); + assertEquals(value, f.join()); + assertEquals(value, f.getNow(null)); + + T result = null; try { - assertEquals(value, f.join()); - assertEquals(value, f.getNow(null)); - assertEquals(value, f.get()); + result = f.get(); } catch (Throwable fail) { threadUnexpectedException(fail); } + assertEquals(value, result); + assertTrue(f.isDone()); assertFalse(f.isCancelled()); assertFalse(f.isCompletedExceptionally()); @@ -4388,7 +4396,7 @@ public class CompletableFutureTest exten f.complete(null); f = new CompletableFuture<>(); - CompletableFuture.anyOf(new CompletableFuture[] { f, incomplete }); + CompletableFuture.anyOf(f, incomplete); f.complete(null); } @@ -4406,7 +4414,7 @@ public class CompletableFutureTest exten f.complete(null); f = new CompletableFuture<>(); - CompletableFuture.anyOf(new CompletableFuture[] { incomplete, f }); + CompletableFuture.anyOf(incomplete, f); f.complete(null); } }