Skip to content

Commit

Permalink
[hotfix][coordination][tests] Remove error message assertions
Browse files Browse the repository at this point in the history
  • Loading branch information
zentol committed Dec 17, 2020
1 parent 4b87866 commit 8393700
Showing 1 changed file with 0 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -113,12 +113,6 @@ public void testHandleStreamingJobsWhenNotEnoughSlot() throws Exception {
} catch (JobExecutionException e) {
assertTrue(findThrowableWithMessage(e, "Job execution failed.").isPresent());
assertTrue(findThrowable(e, NoResourceAvailableException.class).isPresent());

//TODO: remove the legacy scheduler message check once legacy scheduler is removed
final String legacySchedulerErrorMessage = "Slots required: 2, slots allocated: 1";
final String ngSchedulerErrorMessage = "Could not allocate the required slot within slot request timeout";
assertTrue(findThrowableWithMessage(e, legacySchedulerErrorMessage).isPresent() ||
findThrowableWithMessage(e, ngSchedulerErrorMessage).isPresent());
}
}

Expand All @@ -130,12 +124,6 @@ public void testHandleBatchJobsWhenNotEnoughSlot() throws Exception {
} catch (JobExecutionException e) {
assertTrue(findThrowableWithMessage(e, "Job execution failed.").isPresent());
assertTrue(findThrowable(e, NoResourceAvailableException.class).isPresent());

//TODO: remove the legacy scheduler message check once legacy scheduler is removed
final String legacySchedulerErrorMessage = "Could not allocate enough slots";
final String ngSchedulerErrorMessage = "Could not allocate the required slot within slot request timeout";
assertTrue(findThrowableWithMessage(e, legacySchedulerErrorMessage).isPresent() ||
findThrowableWithMessage(e, ngSchedulerErrorMessage).isPresent());
}
}

Expand Down

0 comments on commit 8393700

Please sign in to comment.