refactor(reporting): improve test class/name reporting in un/quarantined list

We needed a more spaced reporting for the test lists in order
to prevent confusion. We now report the class and test name separately for better clarity.

Fixes MRGFY-5937

Author: Syffe

mergify_cli/ci/junit_processing/junit.py

@@ -194,17 +194,15 @@ async def junit_to_spans(
 
         for testcase in testsuite.findall("testcase"):
             classname = testcase.get("classname")
-            if classname is not None:
-                test_name = classname + "." + testcase.get("name", "unnamed test")
-            else:
-                test_name = testcase.get("name", "unnamed test")
+            test_case_name = testcase.get("name", "unnamed test")
             start_time = now - int(float(testcase.get("time", 0)) * 10e9)
             min_start_time = min(min_start_time, start_time)
 
             attributes: dict[str, str | bool] = {
                 "test.scope": "case",
-                "test.case.name": test_name,
-                "code.function.name": test_name,
+                "test.class.name": classname,
+                "test.case.name": test_case_name,
+                "code.function.name": test_case_name,
                 "cicd.test.quarantined": False,
             }
 
@@ -249,7 +247,7 @@ async def junit_to_spans(
                 )
 
             span = ReadableSpan(
-                name=test_name,
+                name=test_case_name,
                 start_time=start_time,
                 end_time=now,
                 context=opentelemetry.trace.span.SpanContext(

mergify_cli/ci/junit_processing/quarantine.py

@@ -98,12 +98,16 @@ async def check_and_update_failing_spans(
     if quarantined_tests_spans:
         click.echo("  - 🔒 Quarantined:")
         for qt_span in quarantined_tests_spans:
-            click.echo(f"      · {qt_span.name}")
+            click.echo("      · Test:")
+            click.echo(f"         Class: {qt_span.attributes.get('test.class.name')}")
+            click.echo(f"         Name: {qt_span.attributes.get('test.case.name')}")
 
     if non_quarantined_tests_spans:
         click.echo("  - ❌ Unquarantined:")
         for nqt_span in non_quarantined_tests_spans:
-            click.echo(f"      · {nqt_span.name}")
+            click.echo("      · Test:")
+            click.echo(f"         Class: {nqt_span.attributes.get('test.class.name')}")
+            click.echo(f"         Name: {nqt_span.attributes.get('test.case.name')}")
 
     return failing_tests_not_quarantined_count