upate to same logic

Author: RulaKhaled

graphql_api/types/commit/commit.py

@@ -29,7 +29,6 @@
     queryset_to_connection,
     queryset_to_connection_sync,
 )
-from graphql_api.types import coverage_totals
 from graphql_api.types.comparison.comparison import (
     MissingBaseCommit,
     MissingHeadReport,
@@ -271,13 +270,40 @@ def resolve_commit_bundle_analysis(commit, info):
 
 
 @commit_bindable.field("patchTotals")
-async def resolve_patch_totals(
-    commit: Commit, info: GraphQLResolveInfo
-) -> coverage_totals:  # type: ignore
-    comparison = info.context.get("comparison", None)
+async def resolve_patch_totals(commit: Commit, info: GraphQLResolveInfo):
+    if not commit.parent_commit_id:
+        return None
+
+    comparison_loader = ComparisonLoader.loader(info, commit.repository_id)
+    commit_comparison = await comparison_loader.load(
+        (commit.parent_commit_id, commit.commitid)
+    )
+
+    # Validate the commit comparison but do not return an error
+    validate_commit_comparison(commit_comparison=commit_comparison)
+
+    if commit_comparison and commit_comparison.is_processed:
+        current_owner = info.context["request"].current_owner
+        parent_commit = await CommitLoader.loader(info, commit.repository_id).load(
+            commit.parent_commit_id
+        )
+        comparison = Comparison(
+            user=current_owner, base_commit=parent_commit, head_commit=commit
+        )
+        info.context["comparison"] = comparison
+
+    if commit_comparison:
+        totals = commit_comparison.patch_totals
+        if not totals:
+            return None
+
+        coverage = totals["coverage"]
+        if coverage is not None:
+            # we always return `coverage` as a percentage but it's stored
+            # in the database as 0 <= value <= 1
+            coverage *= 100
 
-    if comparison and isinstance(comparison, ComparisonReport):
-        return comparison.patch_totals
+        return {**totals, "coverage": coverage}
 
     return None