Prevent status updates from overwriting test results

canonical · Jul 22, 2024 · c6d5472 · c6d5472
1 parent 2f1a582
commit c6d5472
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/backend/test_observer/controllers/test_executions/status_update.py b/backend/test_observer/controllers/test_executions/status_update.py
@@ -57,7 +57,9 @@ def put_status_update(
     event_parser.process_events(test_execution.test_events)
     if event_parser.resource_url is not None:
         test_execution.resource_url = event_parser.resource_url
-    if event_parser.is_completed:
+    if (event_parser.is_completed
+    and test_execution.status is not TestExecutionStatus.FAILED
+    and test_execution.status is not TestExecutionStatus.PASSED):
         test_execution.status = TestExecutionStatus.ENDED
     db.commit()