fix(ci): exclude cancelled jobs from pr-test-summary failed count (#1558)

## Summary

- `failed` was computed as "anything not passed or skipped", so cancelled jobs (e.g. downstream jobs queued when one job fails) were counted as failures.
- Fix: only count jobs with `conclusion in ("failure", "timed_out")`.

Closes #1223

Authors:
  - Ramakrishna Prabhu (https://github.com/ramakrishnap-nv)

Approvers:
  - Trevor McKay (https://github.com/tmckayus)
  - James Lamb (https://github.com/jameslamb)

URL: #1558

Author: ramakrishnap-nv

ci/utils/pr_test_summary.py

@@ -131,13 +131,21 @@ def _add(test_id):
     return failed[:_MAX_TESTS]
 
 
-def _build_body(failed, passed, skipped, job_analysis):
+def _build_body(failed, passed, skipped, cancelled, job_analysis):
     lines = [_MARKER, "## CI Test Summary", ""]
     if not failed:
-        lines.append(f"✅ All {len(passed)} test job(s) passed.")
+        if cancelled:
+            parts = [f"✅ {len(passed)} passed"]
+            if skipped:
+                parts.append(f"{len(skipped)} skipped")
+            parts.append(f"{len(cancelled)} cancelled / not completed")
+            lines.append(" · ".join(parts))
+        else:
+            lines.append(f"✅ All {len(passed)} test job(s) passed.")
     else:
         lines.append(
             f"**{len(failed)} failed** · {len(passed)} passed · {len(skipped)} skipped"
+            + (f" · {len(cancelled)} cancelled" if cancelled else "")
         )
         for job in failed:
             tests = job_analysis[job["id"]]
@@ -177,13 +185,20 @@ def main():
 
     passed = [j for j in test_jobs if j["conclusion"] == "success"]
     skipped = [j for j in test_jobs if j["conclusion"] == "skipped"]
-    failed = [j for j in test_jobs if j not in passed and j not in skipped]
+    failed = [
+        j for j in test_jobs if j["conclusion"] in ("failure", "timed_out")
+    ]
+    cancelled = [
+        j
+        for j in test_jobs
+        if j not in passed and j not in skipped and j not in failed
+    ]
 
     job_analysis = {
         job["id"]: _analyze_job_log(job["id"], repo, token) for job in failed
     }
 
-    body = _build_body(failed, passed, skipped, job_analysis)
+    body = _build_body(failed, passed, skipped, cancelled, job_analysis)
 
     comments = list(
         _paginate(f"/repos/{repo}/issues/{pr_number}/comments", token)