Merge pull request #2730 from freakboy3742/macos-ci-stability

Speculative workaround for macOS test reporting issue.
beeware · Jul 25, 2024 · a33dc91 · a33dc91
2 parents cdff0dd + b2adb67
commit a33dc91
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/changes/2689.misc.rst b/changes/2689.misc.rst
@@ -0,0 +1 @@
+A potential workaround for macOS CI result reporting was added.
diff --git a/testbed/tests/testbed.py b/testbed/tests/testbed.py
@@ -102,9 +102,12 @@ def run_tests(app, cov, args, report_coverage, run_slow, running_in_ci):
         traceback.print_exc()
         app.returncode = 1
     finally:
-        print(f">>>>>>>>>> EXIT {app.returncode} <<<<<<<<<<")
-        # Add a short pause to make sure any log tailing gets a chance to flush
-        time.sleep(0.5)
+        # Add a short pause to make sure any log tailing gets a chance to flush. Run a
+        # couple of times to make sure any log streaming dropouts don't prevent
+        # Briefcase from seeing the output.
+        for i in range(0, 6):
+            print(f">>>>>>>>>> EXIT {app.returncode} <<<<<<<<<<")
+            time.sleep(0.5)
         app.loop.call_soon_threadsafe(app.exit)
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		A potential workaround for macOS CI result reporting was added.