ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/jsr166/jsr166/src/test/tck/JSR166TestCase.java
(Generate patch)

Comparing jsr166/src/test/tck/JSR166TestCase.java (file contents):
Revision 1.136 by jsr166, Fri Sep 4 18:16:28 2015 UTC vs.
Revision 1.183 by jsr166, Sat Jan 23 20:15:28 2016 UTC

# Line 6 | Line 6
6   * Pat Fisher, Mike Judd.
7   */
8  
9 + /*
10 + * @test
11 + * @summary JSR-166 tck tests
12 + * @build *
13 + * @run junit/othervm/timeout=1000 -Djsr166.testImplementationDetails=true JSR166TestCase
14 + */
15 +
16   import static java.util.concurrent.TimeUnit.MILLISECONDS;
17 + import static java.util.concurrent.TimeUnit.MINUTES;
18   import static java.util.concurrent.TimeUnit.NANOSECONDS;
19  
20   import java.io.ByteArrayInputStream;
# Line 15 | Line 23 | import java.io.ObjectInputStream;
23   import java.io.ObjectOutputStream;
24   import java.lang.management.ManagementFactory;
25   import java.lang.management.ThreadInfo;
26 + import java.lang.management.ThreadMXBean;
27   import java.lang.reflect.Constructor;
28   import java.lang.reflect.Method;
29   import java.lang.reflect.Modifier;
30 + import java.nio.file.Files;
31 + import java.nio.file.Paths;
32   import java.security.CodeSource;
33   import java.security.Permission;
34   import java.security.PermissionCollection;
# Line 37 | Line 48 | import java.util.concurrent.BlockingQueu
48   import java.util.concurrent.Callable;
49   import java.util.concurrent.CountDownLatch;
50   import java.util.concurrent.CyclicBarrier;
51 + import java.util.concurrent.ExecutionException;
52 + import java.util.concurrent.Executors;
53   import java.util.concurrent.ExecutorService;
54 + import java.util.concurrent.ForkJoinPool;
55   import java.util.concurrent.Future;
56   import java.util.concurrent.RecursiveAction;
57   import java.util.concurrent.RecursiveTask;
# Line 47 | Line 61 | import java.util.concurrent.ThreadFactor
61   import java.util.concurrent.ThreadPoolExecutor;
62   import java.util.concurrent.TimeoutException;
63   import java.util.concurrent.atomic.AtomicReference;
64 + import java.util.regex.Matcher;
65   import java.util.regex.Pattern;
66  
67   import junit.framework.AssertionFailedError;
# Line 65 | Line 80 | import junit.framework.TestSuite;
80   *
81   * <ol>
82   *
83 < * <li> All assertions in code running in generated threads must use
83 > * <li>All assertions in code running in generated threads must use
84   * the forms {@link #threadFail}, {@link #threadAssertTrue}, {@link
85   * #threadAssertEquals}, or {@link #threadAssertNull}, (not
86   * {@code fail}, {@code assertTrue}, etc.) It is OK (but not
87   * particularly recommended) for other code to use these forms too.
88   * Only the most typically used JUnit assertion methods are defined
89 < * this way, but enough to live with.</li>
89 > * this way, but enough to live with.
90   *
91 < * <li> If you override {@link #setUp} or {@link #tearDown}, make sure
91 > * <li>If you override {@link #setUp} or {@link #tearDown}, make sure
92   * to invoke {@code super.setUp} and {@code super.tearDown} within
93   * them. These methods are used to clear and check for thread
94 < * assertion failures.</li>
94 > * assertion failures.
95   *
96   * <li>All delays and timeouts must use one of the constants {@code
97   * SHORT_DELAY_MS}, {@code SMALL_DELAY_MS}, {@code MEDIUM_DELAY_MS},
# Line 87 | Line 102 | import junit.framework.TestSuite;
102   * is always discriminable as larger than SHORT and smaller than
103   * MEDIUM.  And so on. These constants are set to conservative values,
104   * but even so, if there is ever any doubt, they can all be increased
105 < * in one spot to rerun tests on slower platforms.</li>
105 > * in one spot to rerun tests on slower platforms.
106   *
107 < * <li> All threads generated must be joined inside each test case
107 > * <li>All threads generated must be joined inside each test case
108   * method (or {@code fail} to do so) before returning from the
109   * method. The {@code joinPool} method can be used to do this when
110 < * using Executors.</li>
110 > * using Executors.
111   *
112   * </ol>
113   *
114   * <p><b>Other notes</b>
115   * <ul>
116   *
117 < * <li> Usually, there is one testcase method per JSR166 method
117 > * <li>Usually, there is one testcase method per JSR166 method
118   * covering "normal" operation, and then as many exception-testing
119   * methods as there are exceptions the method can throw. Sometimes
120   * there are multiple tests per JSR166 method when the different
121   * "normal" behaviors differ significantly. And sometimes testcases
122 < * cover multiple methods when they cannot be tested in
108 < * isolation.</li>
122 > * cover multiple methods when they cannot be tested in isolation.
123   *
124 < * <li> The documentation style for testcases is to provide as javadoc
124 > * <li>The documentation style for testcases is to provide as javadoc
125   * a simple sentence or two describing the property that the testcase
126   * method purports to test. The javadocs do not say anything about how
127 < * the property is tested. To find out, read the code.</li>
127 > * the property is tested. To find out, read the code.
128   *
129 < * <li> These tests are "conformance tests", and do not attempt to
129 > * <li>These tests are "conformance tests", and do not attempt to
130   * test throughput, latency, scalability or other performance factors
131   * (see the separate "jtreg" tests for a set intended to check these
132   * for the most central aspects of functionality.) So, most tests use
133   * the smallest sensible numbers of threads, collection sizes, etc
134 < * needed to check basic conformance.</li>
134 > * needed to check basic conformance.
135   *
136   * <li>The test classes currently do not declare inclusion in
137   * any particular package to simplify things for people integrating
138 < * them in TCK test suites.</li>
138 > * them in TCK test suites.
139   *
140 < * <li> As a convenience, the {@code main} of this class (JSR166TestCase)
141 < * runs all JSR166 unit tests.</li>
140 > * <li>As a convenience, the {@code main} of this class (JSR166TestCase)
141 > * runs all JSR166 unit tests.
142   *
143   * </ul>
144   */
# Line 168 | Line 182 | public class JSR166TestCase extends Test
182      private static final int suiteRuns =
183          Integer.getInteger("jsr166.suiteRuns", 1);
184  
185 +    /**
186 +     * The scaling factor to apply to standard delays used in tests.
187 +     */
188 +    private static final int delayFactor =
189 +        Integer.getInteger("jsr166.delay.factor", 1);
190 +
191      public JSR166TestCase() { super(); }
192      public JSR166TestCase(String name) { super(name); }
193  
# Line 183 | Line 203 | public class JSR166TestCase extends Test
203          return (regex == null) ? null : Pattern.compile(regex);
204      }
205  
206 <    protected void runTest() throws Throwable {
206 >    // Instrumentation to debug very rare, but very annoying hung test runs.
207 >    static volatile TestCase currentTestCase;
208 >    // static volatile int currentRun = 0;
209 >    static {
210 >        Runnable checkForWedgedTest = new Runnable() { public void run() {
211 >            // Avoid spurious reports with enormous runsPerTest.
212 >            // A single test case run should never take more than 1 second.
213 >            // But let's cap it at the high end too ...
214 >            final int timeoutMinutes =
215 >                Math.min(15, Math.max(runsPerTest / 60, 1));
216 >            for (TestCase lastTestCase = currentTestCase;;) {
217 >                try { MINUTES.sleep(timeoutMinutes); }
218 >                catch (InterruptedException unexpected) { break; }
219 >                if (lastTestCase == currentTestCase) {
220 >                    System.err.printf(
221 >                        "Looks like we're stuck running test: %s%n",
222 >                        lastTestCase);
223 > //                     System.err.printf(
224 > //                         "Looks like we're stuck running test: %s (%d/%d)%n",
225 > //                         lastTestCase, currentRun, runsPerTest);
226 > //                     System.err.println("availableProcessors=" +
227 > //                         Runtime.getRuntime().availableProcessors());
228 > //                     System.err.printf("cpu model = %s%n", cpuModel());
229 >                    dumpTestThreads();
230 >                    // one stack dump is probably enough; more would be spam
231 >                    break;
232 >                }
233 >                lastTestCase = currentTestCase;
234 >            }}};
235 >        Thread thread = new Thread(checkForWedgedTest, "checkForWedgedTest");
236 >        thread.setDaemon(true);
237 >        thread.start();
238 >    }
239 >
240 > //     public static String cpuModel() {
241 > //         try {
242 > //             Matcher matcher = Pattern.compile("model name\\s*: (.*)")
243 > //                 .matcher(new String(
244 > //                      Files.readAllBytes(Paths.get("/proc/cpuinfo")), "UTF-8"));
245 > //             matcher.find();
246 > //             return matcher.group(1);
247 > //         } catch (Exception ex) { return null; }
248 > //     }
249 >
250 >    public void runBare() throws Throwable {
251 >        currentTestCase = this;
252          if (methodFilter == null
253 <            || methodFilter.matcher(toString()).find()) {
254 <            for (int i = 0; i < runsPerTest; i++) {
255 <                if (profileTests)
256 <                    runTestProfiled();
257 <                else
258 <                    super.runTest();
259 <            }
253 >            || methodFilter.matcher(toString()).find())
254 >            super.runBare();
255 >    }
256 >
257 >    protected void runTest() throws Throwable {
258 >        for (int i = 0; i < runsPerTest; i++) {
259 >            // currentRun = i;
260 >            if (profileTests)
261 >                runTestProfiled();
262 >            else
263 >                super.runTest();
264          }
265      }
266  
267      protected void runTestProfiled() throws Throwable {
268 <        // Warmup run, notably to trigger all needed classloading.
269 <        super.runTest();
201 <        long t0 = System.nanoTime();
202 <        try {
268 >        for (int i = 0; i < 2; i++) {
269 >            long startTime = System.nanoTime();
270              super.runTest();
271 <        } finally {
272 <            long elapsedMillis = millisElapsedSince(t0);
273 <            if (elapsedMillis >= profileThreshold)
271 >            long elapsedMillis = millisElapsedSince(startTime);
272 >            if (elapsedMillis < profileThreshold)
273 >                break;
274 >            // Never report first run of any test; treat it as a
275 >            // warmup run, notably to trigger all needed classloading,
276 >            if (i > 0)
277                  System.out.printf("%n%s: %d%n", toString(), elapsedMillis);
278          }
279      }
# Line 215 | Line 285 | public class JSR166TestCase extends Test
285          main(suite(), args);
286      }
287  
288 +    static class PithyResultPrinter extends junit.textui.ResultPrinter {
289 +        PithyResultPrinter(java.io.PrintStream writer) { super(writer); }
290 +        long runTime;
291 +        public void startTest(Test test) {}
292 +        protected void printHeader(long runTime) {
293 +            this.runTime = runTime; // defer printing for later
294 +        }
295 +        protected void printFooter(TestResult result) {
296 +            if (result.wasSuccessful()) {
297 +                getWriter().println("OK (" + result.runCount() + " tests)"
298 +                    + "  Time: " + elapsedTimeAsString(runTime));
299 +            } else {
300 +                getWriter().println("Time: " + elapsedTimeAsString(runTime));
301 +                super.printFooter(result);
302 +            }
303 +        }
304 +    }
305 +
306 +    /**
307 +     * Returns a TestRunner that doesn't bother with unnecessary
308 +     * fluff, like printing a "." for each test case.
309 +     */
310 +    static junit.textui.TestRunner newPithyTestRunner() {
311 +        junit.textui.TestRunner runner = new junit.textui.TestRunner();
312 +        runner.setPrinter(new PithyResultPrinter(System.out));
313 +        return runner;
314 +    }
315 +
316      /**
317       * Runs all unit tests in the given test suite.
318       * Actual behavior influenced by jsr166.* system properties.
# Line 226 | Line 324 | public class JSR166TestCase extends Test
324              System.setSecurityManager(new SecurityManager());
325          }
326          for (int i = 0; i < suiteRuns; i++) {
327 <            TestResult result = junit.textui.TestRunner.run(suite);
327 >            TestResult result = newPithyTestRunner().doRun(suite);
328              if (!result.wasSuccessful())
329                  System.exit(1);
330              System.gc();
# Line 282 | Line 380 | public class JSR166TestCase extends Test
380      public static boolean atLeastJava7() { return JAVA_CLASS_VERSION >= 51.0; }
381      public static boolean atLeastJava8() { return JAVA_CLASS_VERSION >= 52.0; }
382      public static boolean atLeastJava9() {
383 <        // As of 2015-09, java9 still uses 52.0 class file version
384 <        return JAVA_SPECIFICATION_VERSION.matches("^(1\\.)?(9|[0-9][0-9])$");
383 >        return JAVA_CLASS_VERSION >= 53.0
384 >            // As of 2015-09, java9 still uses 52.0 class file version
385 >            || JAVA_SPECIFICATION_VERSION.matches("^(1\\.)?(9|[0-9][0-9])$");
386      }
288    // public static boolean atLeastJava9() { return JAVA_CLASS_VERSION >= 53.0; }
387      public static boolean atLeastJava10() {
388 <        return JAVA_SPECIFICATION_VERSION.matches("^(1\\.)?[0-9][0-9]$");
388 >        return JAVA_CLASS_VERSION >= 54.0
389 >            || JAVA_SPECIFICATION_VERSION.matches("^(1\\.)?[0-9][0-9]$");
390      }
391  
392      /**
# Line 376 | Line 475 | public class JSR166TestCase extends Test
475                  "LongAdderTest",
476                  "SplittableRandomTest",
477                  "StampedLockTest",
478 +                "SubmissionPublisherTest",
479                  "ThreadLocalRandom8Test",
480              };
481              addNamedTestClasses(suite, java8TestClassNames);
# Line 384 | Line 484 | public class JSR166TestCase extends Test
484          // Java9+ test classes
485          if (atLeastJava9()) {
486              String[] java9TestClassNames = {
487 <                "ThreadPoolExecutor9Test",
487 >                // Currently empty, but expecting varhandle tests
488              };
489              addNamedTestClasses(suite, java9TestClassNames);
490          }
# Line 451 | Line 551 | public class JSR166TestCase extends Test
551          } else {
552              return new TestSuite();
553          }
454
554      }
555  
556      // Delays for timing-dependent tests, in milliseconds.
# Line 462 | Line 561 | public class JSR166TestCase extends Test
561      public static long LONG_DELAY_MS;
562  
563      /**
564 <     * Returns the shortest timed delay. This could
565 <     * be reimplemented to use for example a Property.
564 >     * Returns the shortest timed delay. This can be scaled up for
565 >     * slow machines using the jsr166.delay.factor system property.
566       */
567      protected long getShortDelay() {
568 <        return 50;
568 >        return 50 * delayFactor;
569      }
570  
571      /**
# Line 509 | Line 608 | public class JSR166TestCase extends Test
608       * the same test have no effect.
609       */
610      public void threadRecordFailure(Throwable t) {
611 +        System.err.println(t);
612 +        dumpTestThreads();
613          threadFailure.compareAndSet(null, t);
614      }
615  
# Line 516 | Line 617 | public class JSR166TestCase extends Test
617          setDelays();
618      }
619  
620 +    void tearDownFail(String format, Object... args) {
621 +        String msg = toString() + ": " + String.format(format, args);
622 +        System.err.println(msg);
623 +        dumpTestThreads();
624 +        throw new AssertionFailedError(msg);
625 +    }
626 +
627      /**
628       * Extra checks that get done for all test cases.
629       *
# Line 543 | Line 651 | public class JSR166TestCase extends Test
651          }
652  
653          if (Thread.interrupted())
654 <            throw new AssertionFailedError("interrupt status set in main thread");
654 >            tearDownFail("interrupt status set in main thread");
655  
656          checkForkJoinPoolThreadLeaks();
657      }
658  
659      /**
660 <     * Finds missing try { ... } finally { joinPool(e); }
660 >     * Finds missing PoolCleaners
661       */
662      void checkForkJoinPoolThreadLeaks() throws InterruptedException {
663 <        Thread[] survivors = new Thread[5];
663 >        Thread[] survivors = new Thread[7];
664          int count = Thread.enumerate(survivors);
665          for (int i = 0; i < count; i++) {
666              Thread thread = survivors[i];
# Line 560 | Line 668 | public class JSR166TestCase extends Test
668              if (name.startsWith("ForkJoinPool-")) {
669                  // give thread some time to terminate
670                  thread.join(LONG_DELAY_MS);
671 <                if (!thread.isAlive()) continue;
672 <                throw new AssertionFailedError
673 <                    (String.format("Found leaked ForkJoinPool thread test=%s thread=%s%n",
566 <                                   toString(), name));
671 >                if (thread.isAlive())
672 >                    tearDownFail("Found leaked ForkJoinPool thread thread=%s",
673 >                                 thread);
674              }
675          }
676 +
677 +        if (!ForkJoinPool.commonPool()
678 +            .awaitQuiescence(LONG_DELAY_MS, MILLISECONDS))
679 +            tearDownFail("ForkJoin common pool thread stuck");
680      }
681  
682      /**
# Line 578 | Line 689 | public class JSR166TestCase extends Test
689              fail(reason);
690          } catch (AssertionFailedError t) {
691              threadRecordFailure(t);
692 <            fail(reason);
692 >            throw t;
693          }
694      }
695  
# Line 705 | Line 816 | public class JSR166TestCase extends Test
816      /**
817       * Delays, via Thread.sleep, for the given millisecond delay, but
818       * if the sleep is shorter than specified, may re-sleep or yield
819 <     * until time elapses.
819 >     * until time elapses.  Ensures that the given time, as measured
820 >     * by System.nanoTime(), has elapsed.
821       */
822      static void delay(long millis) throws InterruptedException {
823 <        long startTime = System.nanoTime();
824 <        long ns = millis * 1000 * 1000;
825 <        for (;;) {
823 >        long nanos = millis * (1000 * 1000);
824 >        final long wakeupTime = System.nanoTime() + nanos;
825 >        do {
826              if (millis > 0L)
827                  Thread.sleep(millis);
828              else // too short to sleep
829                  Thread.yield();
830 <            long d = ns - (System.nanoTime() - startTime);
831 <            if (d > 0L)
832 <                millis = d / (1000 * 1000);
833 <            else
834 <                break;
830 >            nanos = wakeupTime - System.nanoTime();
831 >            millis = nanos / (1000 * 1000);
832 >        } while (nanos >= 0L);
833 >    }
834 >
835 >    /**
836 >     * Allows use of try-with-resources with per-test thread pools.
837 >     */
838 >    class PoolCleaner implements AutoCloseable {
839 >        private final ExecutorService pool;
840 >        public PoolCleaner(ExecutorService pool) { this.pool = pool; }
841 >        public void close() { joinPool(pool); }
842 >    }
843 >
844 >    /**
845 >     * An extension of PoolCleaner that has an action to release the pool.
846 >     */
847 >    class PoolCleanerWithReleaser extends PoolCleaner {
848 >        private final Runnable releaser;
849 >        public PoolCleanerWithReleaser(ExecutorService pool, Runnable releaser) {
850 >            super(pool);
851 >            this.releaser = releaser;
852 >        }
853 >        public void close() {
854 >            try {
855 >                releaser.run();
856 >            } finally {
857 >                super.close();
858 >            }
859          }
860      }
861  
862 +    PoolCleaner cleaner(ExecutorService pool) {
863 +        return new PoolCleaner(pool);
864 +    }
865 +
866 +    PoolCleaner cleaner(ExecutorService pool, Runnable releaser) {
867 +        return new PoolCleanerWithReleaser(pool, releaser);
868 +    }
869 +
870 +    PoolCleaner cleaner(ExecutorService pool, CountDownLatch latch) {
871 +        return new PoolCleanerWithReleaser(pool, releaser(latch));
872 +    }
873 +
874 +    Runnable releaser(final CountDownLatch latch) {
875 +        return new Runnable() { public void run() {
876 +            do { latch.countDown(); }
877 +            while (latch.getCount() > 0);
878 +        }};
879 +    }
880 +
881      /**
882       * Waits out termination of a thread pool or fails doing so.
883       */
884 <    void joinPool(ExecutorService exec) {
884 >    void joinPool(ExecutorService pool) {
885          try {
886 <            exec.shutdown();
887 <            if (!exec.awaitTermination(2 * LONG_DELAY_MS, MILLISECONDS))
888 <                fail("ExecutorService " + exec +
889 <                     " did not terminate in a timely manner");
886 >            pool.shutdown();
887 >            if (!pool.awaitTermination(2 * LONG_DELAY_MS, MILLISECONDS)) {
888 >                try {
889 >                    threadFail("ExecutorService " + pool +
890 >                               " did not terminate in a timely manner");
891 >                } finally {
892 >                    // last resort, for the benefit of subsequent tests
893 >                    pool.shutdownNow();
894 >                    pool.awaitTermination(MEDIUM_DELAY_MS, MILLISECONDS);
895 >                }
896 >            }
897          } catch (SecurityException ok) {
898              // Allowed in case test doesn't have privs
899          } catch (InterruptedException fail) {
900 <            fail("Unexpected InterruptedException");
900 >            threadFail("Unexpected InterruptedException");
901          }
902      }
903  
904 +    /** Like Runnable, but with the freedom to throw anything */
905 +    interface Action { public void run() throws Throwable; }
906 +
907      /**
908 <     * A debugging tool to print all stack traces, as jstack does.
908 >     * Runs all the given actions in parallel, failing if any fail.
909 >     * Useful for running multiple variants of tests that are
910 >     * necessarily individually slow because they must block.
911       */
912 <    static void printAllStackTraces() {
913 <        for (ThreadInfo info :
914 <                 ManagementFactory.getThreadMXBean()
915 <                 .dumpAllThreads(true, true))
912 >    void testInParallel(Action ... actions) {
913 >        ExecutorService pool = Executors.newCachedThreadPool();
914 >        try (PoolCleaner cleaner = cleaner(pool)) {
915 >            ArrayList<Future<?>> futures = new ArrayList<>(actions.length);
916 >            for (final Action action : actions)
917 >                futures.add(pool.submit(new CheckedRunnable() {
918 >                    public void realRun() throws Throwable { action.run();}}));
919 >            for (Future<?> future : futures)
920 >                try {
921 >                    assertNull(future.get(LONG_DELAY_MS, MILLISECONDS));
922 >                } catch (ExecutionException ex) {
923 >                    threadUnexpectedException(ex.getCause());
924 >                } catch (Exception ex) {
925 >                    threadUnexpectedException(ex);
926 >                }
927 >        }
928 >    }
929 >
930 >    /**
931 >     * A debugging tool to print stack traces of most threads, as jstack does.
932 >     * Uninteresting threads are filtered out.
933 >     */
934 >    static void dumpTestThreads() {
935 >        ThreadMXBean threadMXBean = ManagementFactory.getThreadMXBean();
936 >        System.err.println("------ stacktrace dump start ------");
937 >        for (ThreadInfo info : threadMXBean.dumpAllThreads(true, true)) {
938 >            String name = info.getThreadName();
939 >            if ("Signal Dispatcher".equals(name))
940 >                continue;
941 >            if ("Reference Handler".equals(name)
942 >                && info.getLockName().startsWith("java.lang.ref.Reference$Lock"))
943 >                continue;
944 >            if ("Finalizer".equals(name)
945 >                && info.getLockName().startsWith("java.lang.ref.ReferenceQueue$Lock"))
946 >                continue;
947 >            if ("checkForWedgedTest".equals(name))
948 >                continue;
949              System.err.print(info);
950 +        }
951 +        System.err.println("------ stacktrace dump end ------");
952      }
953  
954      /**
# Line 766 | Line 968 | public class JSR166TestCase extends Test
968              delay(millis);
969              assertTrue(thread.isAlive());
970          } catch (InterruptedException fail) {
971 <            fail("Unexpected InterruptedException");
971 >            threadFail("Unexpected InterruptedException");
972          }
973      }
974  
# Line 788 | Line 990 | public class JSR166TestCase extends Test
990              for (Thread thread : threads)
991                  assertTrue(thread.isAlive());
992          } catch (InterruptedException fail) {
993 <            fail("Unexpected InterruptedException");
993 >            threadFail("Unexpected InterruptedException");
994          }
995      }
996  
# Line 1066 | Line 1268 | public class JSR166TestCase extends Test
1268          } finally {
1269              if (t.getState() != Thread.State.TERMINATED) {
1270                  t.interrupt();
1271 <                fail("Test timed out");
1271 >                threadFail("timed out waiting for thread to terminate");
1272              }
1273          }
1274      }
# Line 1191 | Line 1393 | public class JSR166TestCase extends Test
1393      public static final String TEST_STRING = "a test string";
1394  
1395      public static class StringTask implements Callable<String> {
1396 <        public String call() { return TEST_STRING; }
1396 >        final String value;
1397 >        public StringTask() { this(TEST_STRING); }
1398 >        public StringTask(String value) { this.value = value; }
1399 >        public String call() { return value; }
1400      }
1401  
1402      public Callable<String> latchAwaitingStringTask(final CountDownLatch latch) {
# Line 1204 | Line 1409 | public class JSR166TestCase extends Test
1409              }};
1410      }
1411  
1412 <    public Runnable awaiter(final CountDownLatch latch) {
1412 >    public Runnable countDowner(final CountDownLatch latch) {
1413          return new CheckedRunnable() {
1414              public void realRun() throws InterruptedException {
1415 <                await(latch);
1415 >                latch.countDown();
1416              }};
1417      }
1418  
1419 +    class LatchAwaiter extends CheckedRunnable {
1420 +        static final int NEW = 0;
1421 +        static final int RUNNING = 1;
1422 +        static final int DONE = 2;
1423 +        final CountDownLatch latch;
1424 +        int state = NEW;
1425 +        LatchAwaiter(CountDownLatch latch) { this.latch = latch; }
1426 +        public void realRun() throws InterruptedException {
1427 +            state = 1;
1428 +            await(latch);
1429 +            state = 2;
1430 +        }
1431 +    }
1432 +
1433 +    public LatchAwaiter awaiter(CountDownLatch latch) {
1434 +        return new LatchAwaiter(latch);
1435 +    }
1436 +
1437      public void await(CountDownLatch latch) {
1438          try {
1439 <            assertTrue(latch.await(LONG_DELAY_MS, MILLISECONDS));
1439 >            if (!latch.await(LONG_DELAY_MS, MILLISECONDS))
1440 >                fail("timed out waiting for CountDownLatch for "
1441 >                     + (LONG_DELAY_MS/1000) + " sec");
1442          } catch (Throwable fail) {
1443              threadUnexpectedException(fail);
1444          }
# Line 1221 | Line 1446 | public class JSR166TestCase extends Test
1446  
1447      public void await(Semaphore semaphore) {
1448          try {
1449 <            assertTrue(semaphore.tryAcquire(LONG_DELAY_MS, MILLISECONDS));
1449 >            if (!semaphore.tryAcquire(LONG_DELAY_MS, MILLISECONDS))
1450 >                fail("timed out waiting for Semaphore for "
1451 >                     + (LONG_DELAY_MS/1000) + " sec");
1452          } catch (Throwable fail) {
1453              threadUnexpectedException(fail);
1454          }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines