--- jsr166/src/test/tck/ScheduledExecutorTest.java 2017/03/28 23:21:24 1.90 +++ jsr166/src/test/tck/ScheduledExecutorTest.java 2017/03/29 16:53:20 1.91 @@ -827,6 +827,7 @@ public class ScheduledExecutorTest exten await(poolBlocked); assertEquals(poolSize, ran.get()); + assertEquals(poolSize, p.getActiveCount()); assertTrue(q.isEmpty()); // Add second wave of tasks. @@ -878,18 +879,18 @@ public class ScheduledExecutorTest exten else assertTrue(delayeds.get(1).isCancelled()); - if (testImplementationDetails) { - if (effectivePeriodicPolicy) - periodics.forEach( - f -> { - assertFalse(f.isDone()); - if (!periodicTasksContinue) - assertTrue(f.cancel(false)); - }); - else { - periodics.subList(0, 4).forEach(f -> assertFalse(f.isDone())); - periodics.subList(4, 8).forEach(f -> assertTrue(f.isCancelled())); - } + if (effectivePeriodicPolicy) + periodics.forEach( + f -> { + assertFalse(f.isDone()); + if (!periodicTasksContinue) { + assertTrue(f.cancel(false)); + assertTrue(f.isCancelled()); + } + }); + else { + periodics.subList(0, 4).forEach(f -> assertFalse(f.isDone())); + periodics.subList(4, 8).forEach(f -> assertTrue(f.isCancelled())); } unblock.countDown(); // Release all pool threads