fix(mac): improve AltTester connection diagnostics and host discovery

- Try connecting to AltTester on multiple candidate hosts (localhost + interface IPs from Player.log).
- Dump Player.log and AltTester-Server.log tails on failure to pinpoint bind/port issues.
- Keep a bounded connection budget to avoid long CI hangs.

Author: rodrigo-fournier-immutable

sample/Tests/test/test_mac.py

@@ -3,6 +3,7 @@
 import os
 import subprocess
 import socket
+import re
 from pathlib import Path
 
 from selenium import webdriver
@@ -24,6 +25,7 @@
 class MacTest(UnityTest):
 
     altdriver = None
+    _altdriver_host = None
 
     @staticmethod
     def _wait_for_tcp_port(host: str, port: int, *, timeout_seconds: float, poll_seconds: float = 1.0) -> bool:
@@ -73,28 +75,115 @@ def _dump_player_log_tail():
                 print(f"Failed to read Player.log at {path}: {e}")
         print("Player.log not found in known locations.")
 
+    @staticmethod
+    def _dump_alttester_server_log_tail():
+        """
+        Best-effort dump of AltTester server log when connection fails.
+        The Player.log usually prints where this file is stored.
+        """
+        product = os.getenv("UNITY_APP_NAME", "SampleApp")
+        candidates = [
+            os.path.expanduser(f"~/Library/Application Support/Immutable/{product}/AltTester-Server.log"),
+            os.path.expanduser("~/Library/Application Support/Immutable/Sample Unity 6 macOS/AltTester-Server.log"),
+            os.path.expanduser("~/Library/Application Support/Immutable/SampleApp/AltTester-Server.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"----- AltTester-Server.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 AltTester-Server.log tail ({path}) -----")
+                return
+            except Exception as e:
+                print(f"Failed to read AltTester-Server.log at {path}: {e}")
+        print("AltTester-Server.log not found in known locations.")
+
     @classmethod
-    def setUpClass(cls):
-        open_sample_app()
-        # 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.")
+    def _get_candidate_altdriver_hosts(cls):
+        """
+        AltTester on macOS CI sometimes doesn't bind to 127.0.0.1.
+        Parse Player.log for host interface IPs (Unity prints them early),
+        and try those as connection hosts.
+        """
+        hosts = ["127.0.0.1", "localhost"]
 
-        last_err = None
-        for attempt in range(3):
+        candidates = [
+            os.path.expanduser("~/Library/Logs/Unity/Player.log"),
+            os.path.expanduser(f"~/Library/Logs/Immutable/{os.getenv('UNITY_APP_NAME', 'SampleApp')}/Player.log"),
+            os.path.expanduser("~/Library/Logs/Immutable/Sample Unity 6 macOS/Player.log"),
+        ]
+        seen = set()
+        for path in candidates:
+            if not path or path in seen:
+                continue
+            seen.add(path)
             try:
-                cls.altdriver = AltDriver(timeout=120)
-                last_err = None
-                break
-            except Exception as e:
-                last_err = e
-                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}")
+                if not os.path.exists(path):
+                    continue
+                with open(path, "r", encoding="utf-8", errors="ignore") as f:
+                    text = f.read()
+                # Example line:
+                # "Found 2 interfaces on host : 0) 172.16.200.1 1) 169.254.90.9"
+                ips = re.findall(r"\b(?:\d{1,3}\.){3}\d{1,3}\b", text)
+                for ip in ips:
+                    if ip not in hosts:
+                        hosts.append(ip)
+            except Exception:
+                pass
+
+        # De-dupe while preserving order
+        deduped = []
+        for h in hosts:
+            if h not in deduped:
+                deduped.append(h)
+        return deduped
+
+    @classmethod
+    def _connect_altdriver(cls):
+        """
+        Connect to AltTester server using a resilient host discovery strategy.
+        """
+        if cls._altdriver_host:
+            # Fast path if we already found a working host.
+            return AltDriver(cls._altdriver_host, 13000, timeout=120)
+
+        hosts = cls._get_candidate_altdriver_hosts()
+        print(f"AltDriver candidate hosts: {hosts}")
+
+        # Total budget for waiting for the port to appear across all hosts.
+        deadline = time.time() + 90
+        last_host_tried = None
+
+        while time.time() < deadline:
+            for host in hosts:
+                last_host_tried = host
+                if cls._wait_for_tcp_port(host, 13000, timeout_seconds=3, poll_seconds=1.0):
+                    try:
+                        driver = AltDriver(host, 13000, timeout=120)
+                        cls._altdriver_host = host
+                        print(f"AltDriver connected via host: {host}")
+                        return driver
+                    except Exception as e:
+                        print(f"AltDriver connect failed on host {host}: {e}")
+            time.sleep(1)
+
+        cls._dump_player_log_tail()
+        cls._dump_alttester_server_log_tail()
+        raise SystemExit(f"AltTester server port 13000 never opened on macOS (last host tried: {last_host_tried}).")
+
+    @classmethod
+    def setUpClass(cls):
+        open_sample_app()
+        cls.altdriver = cls._connect_altdriver()
         cls.stop_browser()
 
     @classmethod