Skip to content

Commit 55fda57

Browse files
committed
Removed unreachable code found by CodeQL
1 parent cb94acb commit 55fda57

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

tests/workflows/test_processing_job.py

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -94,19 +94,13 @@ def test_run(
9494
if ispyb_session is not None:
9595
mock_transport_object.do_create_ispyb_job.assert_called_once()
9696
if insert_job is not None:
97-
if pj_result is not None:
98-
mock_prom.preporcessed_movies.labels.assert_called_once()
9997
if app_result is not None:
100-
if ispyb_session is not None:
98+
assert result == {"success": True}
99+
else:
100+
if update_status is not None:
101101
assert result == {"success": True}
102102
else:
103-
mock_transport_object.do_update_processing_status.assert_called_once()
104-
if update_status is not None:
105-
assert result == {"success": True}
106-
else:
107-
assert result == {"success": False, "requeue": True}
108-
else:
109-
assert {"success": True}
103+
assert result == {"success": False, "requeue": True}
110104
else:
111105
assert result == {"success": False, "requeue": True}
112106
else:

0 commit comments

Comments
 (0)