chore: Update branch

Updating branch and making sure all tests are OK.

tasks/label_analysis.py

@@ -66,7 +66,10 @@ async def run_async(self, db_session, request_id, *args, **kwargs):
                 commit=label_analysis_request.head_commit.commitid,
             ),
         )
+
         if label_analysis_request.state_id == LabelAnalysisRequestState.FINISHED.db_id:
+            # Indicates that this request has been calculated already
+            # We might need to update the requested labels
             return self._handle_larq_already_calculated(label_analysis_request)
 
         try:
@@ -164,7 +167,7 @@ def _handle_larq_already_calculated(self, larq: LabelAnalysisRequest):
         # This means we already calculated everything
         # Except possibly the absent labels
         log.info(
-            "Label analysis was request already calculated",
+            "Label analysis request was already calculated",
             extra=dict(
                 request_id=larq.id,
                 external_id=larq.external_id,
@@ -192,10 +195,11 @@ def _handle_larq_already_calculated(self, larq: LabelAnalysisRequest):
                 commit_sha=larq.head_commit.commitid,
             )
             larq.result = result  # Save the new result
-            return { **result, 'success': True }
-        # No requested labels mean we don't have to calculate again
+            return {**result, "success": True, "errors": []}
+        # No requested labels mean we don't have any new information
+        # So we don't need to calculate again
         # This shouldn't actually happen
-        return { **larq.result, 'success': True }
+        return {**larq.result, "success": True, "errors": []}
 
     def _get_requested_labels(self, label_analysis_request: LabelAnalysisRequest):
         if label_analysis_request.requested_labels:

tasks/tests/unit/test_label_analysis.py

@@ -517,6 +517,7 @@ async def test_simple_call_without_requested_labels_then_with_requested_labels(
         "present_report_labels": expected_present_report_labels,
         "success": True,
         "global_level_labels": [],
+        "errors": [],
     }
     assert larf.result == {
         "absent_labels": expected_absent_labels,
@@ -526,7 +527,6 @@ async def test_simple_call_without_requested_labels_then_with_requested_labels(
     }
 
 
-
 @pytest.mark.asyncio
 async def test_simple_call_with_requested_labels(
     dbsession, mock_storage, mocker, sample_report_with_labels, mock_repo_provider