-
Notifications
You must be signed in to change notification settings - Fork 115
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix faulty and random failing IJobManagerTest.testOrder #628
The test case IJobManagerTest.testOrder is randomly failing. It is supposed to validate that Jobs are execute in the order in which they are scheduled. To this end, it asserts that the run methods of the Jobs are executed in the order in which the have been scheduled. There is, however, no guarantee for an execution order of the run methods but only for the order in which the Jobs are picked by a Worker to be executed. This change rewrites the test case to only validate that whenever a Job's run method is executed the previously scheduled jobs have started running or have already finished as well, i.e., are either in the state RUNNING or NONE. Fixes #628
- Loading branch information
1 parent
3baaffe
commit 6dee57c
Showing
1 changed file
with
25 additions
and
21 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters