Install xvfb beforehand

Deltares · Jan 24, 2024 · ef40de8 · ef40de8
1 parent 443c1f8
commit ef40de8
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 12 deletions.
diff --git a/.github/workflows/qgis.yml b/.github/workflows/qgis.yml
@@ -22,6 +22,10 @@ jobs:
           - windows-latest
     steps:
       - uses: actions/checkout@v4
+      - name: Install xvfb
+        if: runner.os == 'Linux'
+        shell: bash
+        run: sudo apt update && sudo apt install -y xvfb
       - uses: prefix-dev/[email protected]
         with:
           pixi-version: "latest"

diff --git a/ribasim_qgis/scripts/run_qgis_tests.py b/ribasim_qgis/scripts/run_qgis_tests.py
@@ -1,18 +1,20 @@
 import subprocess
+import sys
+
+xvfb = ["xvfb-run", "-a"] if sys.platform == "linux" else []
+qgis_command = [
+    "qgis",
+    "--profiles-path",
+    ".pixi/qgis_env",
+    "--version-migration",
+    "--nologo",
+    "--code",
+    "ribasim_qgis/scripts/qgis_testrunner.py",
+    "ribasim_qgis.tests",
+]
 
 qgis_process = subprocess.run(
-    [
-        "xvfb-run",
-        "-a",
-        "qgis",
-        "--profiles-path",
-        ".pixi/qgis_env",
-        "--version-migration",
-        "--nologo",
-        "--code",
-        "ribasim_qgis/scripts/qgis_testrunner.py",
-        "ribasim_qgis.tests",
-    ],
+    xvfb + qgis_command,
     stdout=subprocess.PIPE,
     stderr=subprocess.STDOUT,
     text=True,