zzh
/
gatk-3.8
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add join call to Progress meter unit test so we actually know the daemon thread has finished

This commit is contained in:
Mark DePristo 2013-01-27 16:52:45 -05:00
parent f5473285d5
commit 63913d516f
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
public/java/test/org/broadinstitute/sting/utils/progressmeter/ProgressMeterDaemonUnitTest.java
View File

@ -84,7 +84,7 @@ public class ProgressMeterDaemonUnitTest extends BaseTest {
}
@Test(dataProvider = "PollingData", invocationCount = 10, successPercentage = 90)
public void testMe(final long poll, final int ticks) throws InterruptedException {
public void testProgressMeterDaemon(final long poll, final int ticks) throws InterruptedException {
final TestingProgressMeter meter = new TestingProgressMeter(poll);
final ProgressMeterDaemon daemon = meter.getProgressMeterDaemon();
Assert.assertTrue(daemon.isDaemon());
@ -96,6 +96,9 @@ public class ProgressMeterDaemonUnitTest extends BaseTest {
daemon.done();
Assert.assertTrue(daemon.isDone());
// wait for the thread to actually finish
daemon.join();
Assert.assertTrue(meter.progressCalls.size() >= 1,
"Expected at least one progress update call from daemon thread, but only got " + meter.progressCalls.size() + " with exact calls " + meter.progressCalls);