💨 skipping macos tests

Author: jonasvdd

.github/workflows/test.yml

@@ -70,7 +70,7 @@ jobs:
 #         flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
     - name: Test with pytest
       run: |
-        poetry run pytest --cov=plotly_resampler --junitxml=junit/test-results-${{ matrix.python-version }}.xml --cov-report=xml tests/test_figure_resampler_selenium.py
+        poetry run pytest --cov=plotly_resampler --junitxml=junit/test-results-${{ matrix.python-version }}.xml --cov-report=xml tests
     - name: Upload pytest test results
        # Use always() to always run this step to publish test results when there are test failures
       if: ${{ always() && hashFiles('junit/test-results-${{ matrix.python-version }}.xml') != '' }}

tests/conftest.py

@@ -1,7 +1,6 @@
 """Fixtures and helper functions for testing"""
 
 import os
-import platform
 from typing import Union
 
 import numpy as np
@@ -66,22 +65,6 @@ def driver():
         options.add_argument("--disable-gpu")
         # options.add_argument("--no=sandbox")
 
-        # macOS-specific options for better stability
-        if platform.system() == "Darwin":  # macOS
-            options.add_argument("--disable-web-security")
-            options.add_argument("--disable-features=VizDisplayCompositor")
-            options.add_argument("--window-size=1920,1080")
-            options.add_argument("--remote-debugging-port=9222")
-            # Additional stability options for macOS CI
-            options.add_argument("--disable-background-timer-throttling")
-            options.add_argument("--disable-backgrounding-occluded-windows")
-            options.add_argument("--disable-renderer-backgrounding")
-            options.add_argument("--disable-ipc-flooding-protection")
-            options.add_argument("--single-process")  # This can help with CI stability
-            options.add_argument("--disable-extensions")
-            options.add_argument("--disable-default-apps")
-        
-
         driver = webdriver.Chrome(
             options=options,
             desired_capabilities=d,

tests/fr_selenium.py

@@ -230,8 +230,7 @@ def drag_and_zoom(self, div_classname, x0=0.25, x1=0.5, y0=0.25, y1=0.5):
         if not self.on_page:
             self.go_to_page()
 
-        timeout = 30 if not_on_linux() else 10
-        WebDriverWait(self.driver, timeout).until(
+        WebDriverWait(self.driver, 5).until(
             EC.presence_of_element_located((By.CLASS_NAME, div_classname))
         )
 

tests/test_figure_resampler_selenium.py

@@ -1,10 +1,18 @@
 import multiprocessing
+import platform
 import time
 
+import pytest
+
 from plotly_resampler.figure_resampler import FigureResampler
 
 from .fr_selenium import FigureResamplerGUITests, RequestParser
 
+pytestmark = pytest.mark.skipif(
+    platform.system() == "Darwin",
+    reason="Selenium tests are unstable on macOS CI",
+)
+
 
 def test_multiple_tz(driver, multiple_tz_figure):
     from pytest_cov.embed import cleanup_on_sigterm