Skip to content

Commit 2f67725

Browse files
committed
fix: use detailed_merge_status to check state in conftest.py
Signed-off-by: Tim Knight <tim.knight1@engineering.digital.dwp.gov.uk>
1 parent 117b961 commit 2f67725

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

tests/functional/conftest.py

+5-1
Original file line numberDiff line numberDiff line change
@@ -459,10 +459,14 @@ def _make_merge_request(*, source_branch: str, create_pipeline: bool = False):
459459
mr_iid = mr.iid
460460
for _ in range(60):
461461
mr = project.mergerequests.get(mr_iid)
462-
if mr.merge_status != "checking":
462+
if (
463+
mr.detailed_merge_status != "checking"
464+
and mr.detailed_merge_status != "unchecked"
465+
):
463466
break
464467
time.sleep(0.5)
465468
assert mr.merge_status != "checking"
469+
assert mr.merge_status != "unchecked"
466470

467471
to_delete.extend([mr, mr_branch])
468472
return mr

0 commit comments

Comments
 (0)