Skip to content

Commit

Permalink
[HWKMETRICS-52] try to make test less prone to failure on CI builds
Browse files Browse the repository at this point in the history
  • Loading branch information
John Sanda committed May 2, 2015
1 parent 688d993 commit 4d2e8f4
Showing 1 changed file with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,9 @@ public void startScheduler() throws Exception {

List<DateTime> actualTimeSlices = new ArrayList<>();
List<DateTime> expectedTimeSlices = asList(
dateTimeService.getTimeSlice(now().plusSeconds(1), standardSeconds(2)),
dateTimeService.getTimeSlice(now().plusSeconds(2), standardSeconds(3)),
dateTimeService.getTimeSlice(now().plusSeconds(2), standardSeconds(3))
dateTimeService.getTimeSlice(now().plusSeconds(2), standardSeconds(1)),
dateTimeService.getTimeSlice(now().plusSeconds(3), standardSeconds(1)),
dateTimeService.getTimeSlice(now().plusSeconds(4), standardSeconds(1))
);

TaskServiceImpl taskService = new TaskServiceImpl(session, queries, leaseService, taskTypes) {
Expand All @@ -63,12 +63,12 @@ public void executeTasks(DateTime timeSlice) {
taskService.setTimeUnit(TimeUnit.SECONDS);
taskService.start();

Thread.sleep(4000);
Thread.sleep(5000);
taskService.shutdown();

assertTrue(actualTimeSlices.size() >= 3, "Expected task execution to be scheduled at least 3 times but it " +
"was scheduled " + actualTimeSlices.size() + " times.");
assertTrue(actualTimeSlices.size() <= 5, "Exepected no more that 5 task executions to be scheduled since " +
assertTrue(actualTimeSlices.size() <= 6, "Expected no more that 5 task executions to be scheduled since " +
"shutdown was called, but it was scheduled " + actualTimeSlices.size() + " times.");
assertTaskExecutionScheduleForTimeSlices(actualTimeSlices, expectedTimeSlices);
}
Expand Down

0 comments on commit 4d2e8f4

Please sign in to comment.