fix: fixup tests and formatting

Signed-off-by: Henry Schreiner <[email protected]>

Author: henryiii

cibuildwheel/logger.py

@@ -1,4 +1,6 @@
 import codecs
+import functools
+import io
 import os
 import re
 import sys
@@ -197,20 +199,24 @@ def error(self, error: BaseException | str) -> None:
             print(f"cibuildwheel: {c.bright_red}error{c.end}: {error}\n", file=sys.stderr)
 
     def print_summary(self) -> None:
-        summary = "## 🎡: Wheels\n\n| Identifier | Wheel | Size | Time |\n|===|===|===|===|\n"
+        string_io = io.StringIO()
+        ioprint = functools.partial(print, file=string_io)
+        ioprint("## 🎡: Wheels\n")
+        ioprint("| Identifier | Wheel | Size | Time |")
+        ioprint("|===|===|===|===|")
         for ident, filename, duration in self.summary:
             if filename:
                 size_mb = filename.stat().st_size / 1024**2
-                summary += f"| {ident} | {filename.name} | {size_mb:.2f} MB | {duration} |"
+                ioprint(f"| {ident} | {filename.name} | {size_mb:.2f} MB | {duration} |")
             else:
-                summary += f"| {ident} | test only | --- | {duration} |"
+                ioprint(f"| {ident} | test only | --- | {duration} |")
 
-        match self.summary_mode:
-            case "github":
-                Path(os.environ["GITHUB_STEP_SUMMARY"]).write_text(summary, encoding="utf-8")
-                print(summary)
-            case _:
-                print(summary)
+        if self.summary_mode == "github":
+            Path(os.environ["GITHUB_STEP_SUMMARY"]).write_text(
+                string_io.getvalue(), encoding="utf-8"
+            )
+
+        print(string_io.getvalue())
 
         self.summary = []
 

unit_test/wheel_print_test.py

@@ -12,9 +12,6 @@ def test_printout_wheels(tmp_path, capsys):
     captured = capsys.readouterr()
     assert captured.err == ""
 
-    assert "example.0" not in captured.out
-    assert "example.1   1 kB\n" in captured.out
-    assert "example.2   2 kB\n" in captured.out
     assert "TEST_MSG:" in captured.out
     assert "TEST_MSG: 2\n" in captured.out
 
@@ -28,7 +25,4 @@ def test_no_printout_on_error(tmp_path, capsys):
     captured = capsys.readouterr()
     assert captured.err == ""
 
-    assert "example.0" not in captured.out
-    assert "example.1" not in captured.out
-    assert "example.2" not in captured.out
     assert "TEST_MSG:" not in captured.out