Skip to content

Commit 8137719

Browse files
XD-DENGkaxil
authored and
Chris Fei
committed
Fix test in PR 12663 (apache#12667)
The order of the two TI keys in test_process_task_instances_depends_on_past() is not guaranteed, in such a case, directly assert two lists (both with two elements) will cause 'random' failure This PR addresses this issue. Co-authored-by: Kaxil Naik <[email protected]> Co-authored-by: Kaxil Naik <[email protected]>
1 parent 86ed1cd commit 8137719

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/jobs/test_scheduler_job.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -570,10 +570,10 @@ def test_process_task_instances_depends_on_past(self, state, start_date, end_dat
570570
ti_to_schedule = []
571571
scheduler_job._process_task_instances(dag, task_instances_list=ti_to_schedule)
572572

573-
assert ti_to_schedule == [
573+
assert sorted(ti_to_schedule) == sorted([
574574
(dag.dag_id, dag_task1.task_id, DEFAULT_DATE, TRY_NUMBER),
575575
(dag.dag_id, dag_task2.task_id, DEFAULT_DATE, TRY_NUMBER),
576-
]
576+
])
577577

578578
def test_find_executable_task_instances_concurrency(self):
579579
dag_id = 'SchedulerJobTest.test_find_executable_task_instances_concurrency'

0 commit comments

Comments
 (0)