Remove redundant printing

Author: cmaloney

Lib/test/libregrtest/testresult.py

@@ -135,20 +135,13 @@ def printErrorList(self, flavour, errors):
         if not self.GITHUB_ANNOTATIONS:
             super().printErrorList(flavour, errors)
             return
-        stream = self.stream
         for test, err in errors:
-            # Precede each failure report with a GitHub Actions annotation
-            # which has the report as message, so that the annotation links
-            # to the report in the job log.
-            self.stream = unittest.runner._WritelnDecorator(io.StringIO())
-            try:
-                super().printErrorList(flavour, [(test, err)])
-                report = self.stream.getvalue()
-            finally:
-                self.stream = stream
-            stream.writeln(github_annotation(str(test), report))
-            stream.write(report)
-            stream.flush()
+            # Precede each failure report with a GitHub Actions annotation,
+            # so that the annotation links to the report in the job log. The
+            # report follows: don't repeat it in the annotation.
+            self.stream.writeln(github_annotation(str(test), err,
+                                                  include_message=False))
+            super().printErrorList(flavour, [(test, err)])
 
     def get_xml_element(self):
         if not self.USE_XML:

Lib/test/libregrtest/utils.py

@@ -175,10 +175,13 @@ def traceback_location(traceback: str) -> dict[str, str | int] | None:
     return location
 
 
-def github_annotation(title: str, message: str) -> str:
+def github_annotation(title: str, message: str, *,
+                      include_message: bool = True) -> str:
     """Format a GitHub Actions error annotation.
 
-    Locate it at the last frame of the traceback in message, if any.
+    Locate it at the last frame of the traceback in message, if any. If
+    include_message is false, the annotation only has the location and the
+    title.
     """
     def escape(text: str) -> str:
         return (text.replace("%", "%25").replace("\r", "%0D")
@@ -199,7 +202,7 @@ def escape_property(text: str) -> str:
     props["title"] = title
     # The job log only shows the message, not the properties: start the
     # message with the location and the title
-    message = f"{header}\n{message}"
+    message = f"{header}\n{message}" if include_message else header
     props_text = ",".join(f"{key}={escape_property(str(value))}"
                           for key, value in props.items())
     return f"::error {props_text}::{escape(message)}"