fix(mac): fail fast when AltTester server never starts

- Wait for AltTester port 13000 to open before attempting AltDriver.
- Limit AltDriver connect retries to avoid long CI hangs.
- Dump Player.log tail when the server never comes up.

Author: rodrigo-fournier-immutable

sample/Tests/test/test_mac.py

@@ -2,6 +2,7 @@
 import time
 import os
 import subprocess
+import socket
 from pathlib import Path
 
 from selenium import webdriver
@@ -24,22 +25,75 @@ class MacTest(UnityTest):
 
     altdriver = None
 
+    @staticmethod
+    def _wait_for_tcp_port(host: str, port: int, *, timeout_seconds: float, poll_seconds: float = 1.0) -> bool:
+        """
+        Wait until a TCP port is accepting connections.
+        This avoids long hangs inside AltDriver connection logic when the AltTester
+        server hasn't started (common on macOS CI).
+        """
+        deadline = time.time() + float(timeout_seconds)
+        while time.time() < deadline:
+            try:
+                with socket.create_connection((host, int(port)), timeout=1.0):
+                    return True
+            except OSError:
+                time.sleep(float(poll_seconds))
+        return False
+
+    @staticmethod
+    def _dump_player_log_tail():
+        """
+        Best-effort dump of Player.log to understand why AltTester server didn't start.
+        """
+        candidates = [
+            os.path.expanduser("~/Library/Logs/Unity/Player.log"),
+            # Standalone Player logs often land under ~/Library/Logs/<Company>/<Product>/Player.log
+            os.path.expanduser(f"~/Library/Logs/Immutable/{os.getenv('UNITY_APP_NAME', 'SampleApp')}/Player.log"),
+            os.path.expanduser(f"~/Library/Logs/Immutable/{os.getenv('UNITY_APP_NAME', 'Sample Unity 6 macOS')}/Player.log"),
+            os.path.expanduser(f"~/Library/Logs/Immutable/{os.getenv('UNITY_APP_NAME', '')}/Player.log"),
+        ]
+        seen = set()
+        for path in candidates:
+            if not path or path in seen:
+                continue
+            seen.add(path)
+            try:
+                if not os.path.exists(path):
+                    continue
+                print(f"----- Player.log tail ({path}) -----")
+                with open(path, "r", encoding="utf-8", errors="ignore") as f:
+                    lines = f.read().splitlines()
+                tail = lines[-250:] if len(lines) > 250 else lines
+                for line in tail:
+                    print(line)
+                print(f"----- end Player.log tail ({path}) -----")
+                return
+            except Exception as e:
+                print(f"Failed to read Player.log at {path}: {e}")
+        print("Player.log not found in known locations.")
+
     @classmethod
     def setUpClass(cls):
         open_sample_app()
-        # macOS runners can take a while to start the AltTester server inside Unity.
-        # Retry connection instead of failing immediately with "Connection refused".
+        # Avoid 20-30 minute hangs: first wait for the AltTester server port to open,
+        # then attempt a small number of AltDriver connections.
+        if not cls._wait_for_tcp_port("127.0.0.1", 13000, timeout_seconds=90, poll_seconds=1.0):
+            cls._dump_player_log_tail()
+            raise SystemExit("AltTester server port 13000 never opened on macOS.")
+
         last_err = None
-        for attempt in range(12):  # ~60s total
+        for attempt in range(3):
             try:
                 cls.altdriver = AltDriver(timeout=120)
                 last_err = None
                 break
             except Exception as e:
                 last_err = e
-                print(f"AltDriver connect attempt {attempt + 1}/12 failed: {e}")
-                time.sleep(5)
+                print(f"AltDriver connect attempt {attempt + 1}/3 failed: {e}")
+                time.sleep(2)
         if last_err is not None:
+            cls._dump_player_log_tail()
             raise SystemExit(f"Failed to connect AltDriver on macOS: {last_err}")
         cls.stop_browser()