javaee7-samples icon indicating copy to clipboard operation
javaee7-samples copied to clipboard

Create unit tests for Concurrency

Open arun-gupta opened this issue 12 years ago • 3 comments

arun-gupta avatar Nov 09 '13 08:11 arun-gupta

Rewrite current Concurrent Tests to use Concurrent Utils; CountDownLatch|CyclicBarrier instead of slow and unsafe Thread.sleep()

e.g. https://github.com/arun-gupta/javaee7-samples/blob/master/concurrency/managedexecutor/src/test/java/org/javaee7/concurrency/managedexecutor/ExecutorInjectTest.java#L72

final CountDownLatch signal = new CountDownLatch(1);
defaultExecutor.submit(new MyRunnableTask(1) {
  void run() {
    super.run();
    signal.countDown();
  }
});
Assert.assertTrue(signal.await(2000, TimeUnit.MILLISECONDS))

aslakknutsen avatar Nov 27 '13 12:11 aslakknutsen

Changed the tests to use CountDownLatch now. Only created unit tests for managedexecutor. Do you want to create for others ?

arun-gupta avatar Nov 28 '13 01:11 arun-gupta

https://github.com/javaee-samples/javaee7-samples/pull/136

aslakknutsen avatar Nov 28 '13 13:11 aslakknutsen