Fixed tests to run on MacOS

Author: franporcel

modules/01-operating-room/tests/test_dds_communication.py

@@ -28,6 +28,8 @@
     wait_for_process_ready,
 )
 
+_IS_MACOS = sys.platform == "darwin"
+
 # Ensure generated types are importable
 if str(SRC_DIR) not in sys.path:
     sys.path.insert(0, str(SRC_DIR))
@@ -262,7 +264,7 @@ class TestAllAppsStatus:
     """All five apps should report DeviceStatus ON when running."""
 
     QT_ENV = {"QT_QPA_PLATFORM": "offscreen"}
-    GTK_ENV = {"GDK_BACKEND": "x11"}
+    GTK_ENV = {} if _IS_MACOS else {"GDK_BACKEND": "x11"}
 
     EXPECTED_DEVICES = {
         "PATIENT_SENSOR",

modules/01-operating-room/tests/test_demo_flow.py

@@ -23,11 +23,14 @@
 
 import importlib
 import sys
+import threading
 import time
 from pathlib import Path
 
 import pytest
 
+_IS_MACOS = sys.platform == "darwin"
+
 TESTS_DIR = Path(__file__).resolve().parent
 
 if str(TESTS_DIR) not in sys.path:
@@ -46,7 +49,7 @@
     sys.path.insert(0, str(SRC_DIR))
 
 QT_ENV = {"QT_QPA_PLATFORM": "offscreen"}
-GTK_ENV = {"GDK_BACKEND": "x11"}
+GTK_ENV = {} if _IS_MACOS else {"GDK_BACKEND": "x11"}
 
 
 # ---------------------------------------------------------------------------
@@ -304,9 +307,14 @@ def test_secure_launch(self, proc_manager_secure, dds_env_secure):
             "ArmController": proc_manager_secure.start_app("ArmController", extra_env=GTK_ENV),
         }
 
-        # Wait for security handshake and DDS initialization
-        for p in apps.values():
-            wait_for_process_ready(p, timeout_sec=15)
+        # Wait for security handshake and DDS initialization (parallel).
+        # Apps that fail the handshake crash within the first 1-2 s; 3 s is
+        # enough to catch startup failures while keeping the test fast.
+        threads = [threading.Thread(target=wait_for_process_ready, args=(p, 3)) for p in apps.values()]
+        for t in threads:
+            t.start()
+        for t in threads:
+            t.join()
 
         crashed = {}
         for name, p in apps.items():

modules/01-operating-room/tests/test_launch.py

@@ -15,13 +15,17 @@
 GUI applications are marked so they can be skipped on headless systems.
 """
 
+import sys
+
 import pytest
 from module01_test_support import (
     create_reader,
     wait_for_device_status,
     wait_for_process_ready,
 )
 
+_IS_MACOS = sys.platform == "darwin"
+
 
 class TestPatientSensor:
     """PatientSensor is a headless C++ app — no display needed."""
@@ -52,7 +56,7 @@ def test_prints_launch_message(self, proc_manager):
 class TestOrchestrator:
     """Orchestrator is a C++ GTK application."""
 
-    GTK_ENV = {"GDK_BACKEND": "x11"}
+    GTK_ENV = {} if _IS_MACOS else {"GDK_BACKEND": "x11"}
 
     def test_starts_and_stays_alive(self, proc_manager):
         proc = proc_manager.start_app("Orchestrator", extra_env=self.GTK_ENV)
@@ -64,7 +68,7 @@ def test_starts_and_stays_alive(self, proc_manager):
 class TestArmController:
     """ArmController is a C++ GTK application."""
 
-    GTK_ENV = {"GDK_BACKEND": "x11"}
+    GTK_ENV = {} if _IS_MACOS else {"GDK_BACKEND": "x11"}
 
     def test_starts_and_stays_alive(self, proc_manager):
         proc = proc_manager.start_app("ArmController", extra_env=self.GTK_ENV)
@@ -101,7 +105,7 @@ class TestAllApps:
     """Launch all five applications simultaneously."""
 
     QT_ENV = {"QT_QPA_PLATFORM": "offscreen"}
-    GTK_ENV = {"GDK_BACKEND": "x11"}
+    GTK_ENV = {} if _IS_MACOS else {"GDK_BACKEND": "x11"}
 
     def test_all_apps_launch_together(self, proc_manager, dds_participant):
         from Types import Common, Common_DeviceStatus