[CI-test] Split OS-specific addon tests via filename convention

.github/docker/gramps-ci/Dockerfile

@@ -37,13 +37,17 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
         xauth \
     && rm -rf /var/lib/apt/lists/*
 
+# Addon runtime deps (dbf, networkx, lxml, svgwrite, boto3, etc.) are
+# NOT baked in here — ci.yml's "Install addon runtime deps (derived from
+# requires_mod)" step pip-installs them at CI runtime from every
+# .gpr.py's requires_mod list, matching what Gramps' Addon Manager does
+# for an end user. Keeps .gpr.py the single source of truth.
 RUN pip install --no-cache-dir \
         PyGObject \
         pycairo \
         "gramps>=6.0,<6.1" \
         orjson \
-        ruff \
-        dbf
+        ruff
 
 RUN apt-get purge -y gcc python3-dev pkg-config && apt-get autoremove -y
 

.github/environment.yml

@@ -6,7 +6,10 @@ dependencies:
   - pygobject
   - gtk3
   - pip
+  # Addon runtime deps (dbf, networkx, lxml, svgwrite, boto3, etc.) are
+  # installed at CI runtime by ci.yml's auto-derive step from .gpr.py
+  # requires_mod — single source of truth. Keep only the stable base
+  # here (Gramps + orjson for plugin registration).
   - pip:
       - "gramps>=6.0,<6.1"
       - orjson
-      - dbf

.github/workflows/ci.yml

@@ -100,7 +100,58 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
+      - name: Install addon runtime deps (derived from requires_mod)
+        # Auto-derive the union of requires_mod across every .gpr.py in
+        # the repo. Mirrors Gramps' Addon Manager install path
+        # (gramps/gui/plug/_windows.py __on_install_clicked → req.install →
+        # gen/utils/requirements.py). Keeps .gpr.py files as the single
+        # source of truth for addon deps — no parallel list to maintain
+        # in the image or workflow. Best-effort: a package needing exotic
+        # system deps (pygraphviz → graphviz-dev, psycopg2 → libpq-dev)
+        # may fail here; the affected addon's tests will skip or fail in
+        # isolation without blocking the rest.
+        shell: bash
+        run: |
+          addon_mods=$(python3 - <<'PY'
+          import ast, glob, re
+          pat = re.compile(r"requires_mod\s*=\s*(\[[^\]]*\])")
+          mods = set()
+          for f in glob.glob("*/*.gpr.py"):
+              try:
+                  text = open(f, encoding="utf-8").read()
+              except OSError:
+                  continue
+              for m in pat.finditer(text):
+                  try:
+                      mods.update(ast.literal_eval(m.group(1)))
+                  except (ValueError, SyntaxError):
+                      pass
+          print(" ".join(sorted(mods)))
+          PY
+          )
+          if [ -n "$addon_mods" ]; then
+            echo "→ addon deps: $addon_mods"
+            for mod in $addon_mods; do
+              pip install "$mod" || echo "× $mod failed to install (continuing)"
+            done
+          else
+            echo "no requires_mod declarations found"
+          fi
+
       - name: Run per-addon unit tests
+        # Filename convention (all OSes):
+        #   test_*.py              — general (any OS)
+        #   test_linux_*.py        — Linux-only
+        #   test_windows_*.py      — Windows-only
+        #   test_integration_*.py  — Linux-only, full-pipeline/DB-backed
+        # The Linux job runs test_*.py except the Windows-only and
+        # integration buckets. Integration tests run in their own job.
+        #
+        # shell: bash — the container's default shell is /bin/sh
+        # (dash on python:3.12-slim), which does not support the
+        # ${var//pattern/repl} and ${var%.py} parameter expansions
+        # used below. Falls silently under continue-on-error.
+        shell: bash
         env:
           PYTHONPATH: .
         run: |
@@ -109,6 +160,7 @@ jobs:
             [ -f "$f" ] || continue
             case "$(basename "$f")" in
               test_integration*) continue ;;
+              test_windows_*)    continue ;;
             esac
             case "$f" in
               Sqlite/tests/test_sqlite.py) continue ;;
@@ -152,7 +204,39 @@ jobs:
           mamba list | head -30
           python -c "import gramps, gi; print('deps OK')"
 
+      - name: Install addon runtime deps (derived from requires_mod)
+        # See unit-test-linux for rationale. Uses `python` (conda-forge
+        # env) to match the surrounding Windows job style.
+        run: |
+          addon_mods=$(python - <<'PY'
+          import ast, glob, re
+          pat = re.compile(r"requires_mod\s*=\s*(\[[^\]]*\])")
+          mods = set()
+          for f in glob.glob("*/*.gpr.py"):
+              try:
+                  text = open(f, encoding="utf-8").read()
+              except OSError:
+                  continue
+              for m in pat.finditer(text):
+                  try:
+                      mods.update(ast.literal_eval(m.group(1)))
+                  except (ValueError, SyntaxError):
+                      pass
+          print(" ".join(sorted(mods)))
+          PY
+          )
+          if [ -n "$addon_mods" ]; then
+            echo "→ addon deps: $addon_mods"
+            for mod in $addon_mods; do
+              pip install "$mod" || echo "× $mod failed to install (continuing)"
+            done
+          else
+            echo "no requires_mod declarations found"
+          fi
+
       - name: Run per-addon unit tests
+        # See filename-convention note in unit-test-linux. The Windows
+        # job runs test_*.py except test_linux_* and test_integration_*.
         env:
           PYTHONPATH: .
         run: |
@@ -161,6 +245,7 @@ jobs:
             [ -f "$f" ] || continue
             case "$(basename "$f")" in
               test_integration*) continue ;;
+              test_linux_*)      continue ;;
             esac
             case "$f" in
               Sqlite/tests/test_sqlite.py) continue ;;
@@ -189,12 +274,52 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
+      - name: Install addon runtime deps (derived from requires_mod)
+        # See unit-test-linux for rationale. The plugin registration test
+        # subprocess-loads each addon's module, which imports its
+        # requires_mod packages.
+        shell: bash
+        run: |
+          addon_mods=$(python3 - <<'PY'
+          import ast, glob, re
+          pat = re.compile(r"requires_mod\s*=\s*(\[[^\]]*\])")
+          mods = set()
+          for f in glob.glob("*/*.gpr.py"):
+              try:
+                  text = open(f, encoding="utf-8").read()
+              except OSError:
+                  continue
+              for m in pat.finditer(text):
+                  try:
+                      mods.update(ast.literal_eval(m.group(1)))
+                  except (ValueError, SyntaxError):
+                      pass
+          print(" ".join(sorted(mods)))
+          PY
+          )
+          if [ -n "$addon_mods" ]; then
+            echo "→ addon deps: $addon_mods"
+            for mod in $addon_mods; do
+              pip install "$mod" || echo "× $mod failed to install (continuing)"
+            done
+          else
+            echo "no requires_mod declarations found"
+          fi
+
       - name: Run plugin registration tests
+        # shell: bash for consistency with the surrounding steps; the
+        # current command uses no bashisms, but keeps this block safe
+        # against future edits. Container default is /bin/sh → dash.
+        shell: bash
         env:
           PYTHONPATH: .
         run: python3 -m unittest discover -s tests -p "test_*.py" -t . -v
 
       - name: Run per-addon integration tests
+        # shell: bash — see unit-test-linux for rationale; the
+        # ${var//pattern/repl} and ${var%.py} expansions below are
+        # bash-only.
+        shell: bash
         env:
           PYTHONPATH: .
         run: |