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

Make multi-threaded progress meter daemon unit test more robust

This commit is contained in:
Mark DePristo 2013-01-05 12:59:18 -05:00
parent 2ab55e4ee7
commit b403c269e9
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -96,7 +96,11 @@ public class ProgressMeterDaemonUnitTest extends BaseTest {
daemon.done(); daemon.done();
Assert.assertTrue(daemon.isDone()); Assert.assertTrue(daemon.isDone());
Assert.assertEquals(meter.progressCalls.size(), ticks, Assert.assertTrue(meter.progressCalls.size() >= 1,
"Expected " + ticks + " progress calls from daemon thread, but only got " + meter.progressCalls.size() + " with exact calls " + meter.progressCalls); "Expected at least one progress update call from daemon thread, but only got " + meter.progressCalls.size() + " with exact calls " + meter.progressCalls);
final int tolerance = (int)Math.ceil(0.8 * meter.progressCalls.size());
Assert.assertTrue(Math.abs(meter.progressCalls.size() - ticks) <= tolerance,
"Expected " + ticks + " progress calls from daemon thread, but got " + meter.progressCalls.size() + " and a tolerance of only " + tolerance);
} }
} }