addon-unit: skip test_windows_* on the Ubuntu platform (#4)

Mirrors the filename convention introduced in addons-source (ci.yml):

    test_*.py              general — every platform
    test_linux_*.py        Linux-only
    test_windows_*.py      Windows-only
    test_integration_*.py  Linux-only, full-pipeline/DB-backed

The Ubuntu runner and its CI counterpart now skip test_windows_*.py so
a Windows-only test file doesn't fail spuriously on Linux. A future
scripts/windows/run-addon-unit.sh will do the inverse filter.

Until a Windows image exists in the testbed, test_windows_*.py runs
nowhere here — Windows coverage remains in addons-source's own Windows
CI job. Verified locally: TMGimporter still passes all 175 tests, and
a synthesised test_windows_fake.py was correctly skipped.

Author: eduralph

.github/workflows/addon-unit-tests.yml

@@ -214,9 +214,20 @@ jobs:
             echo "=== $addon ==="
             out_dir="$GITHUB_WORKSPACE/gramps-testbed/test-results/$addon"
             mkdir -p "$out_dir"
+            # Filename convention (mirrors
+            # addons-source/.github/workflows/ci.yml):
+            #   test_*.py              general — every platform
+            #   test_linux_*.py        Linux-only
+            #   test_windows_*.py      Windows-only (skipped here)
+            #   test_integration_*.py  Linux-only, full-pipeline/DB-backed
+            # This job runs on ubuntu-24.04. A sibling Windows job would
+            # do the inverse (skip test_linux_*/test_integration_*).
             modules=()
             for f in "$GITHUB_WORKSPACE"/addons-source/"$addon"/tests/test_*.py; do
               [ -f "$f" ] || continue
+              case "$(basename "$f")" in
+                test_windows_*) continue ;;
+              esac
               rel="${f#$GITHUB_WORKSPACE/addons-source/}"
               mod="${rel%.py}"
               mod="${mod//\//.}"

CLAUDE.md

@@ -35,6 +35,17 @@ If `../addons-source/AGENTS.md` exists, it applies inside that repo:
   class under dotted-path loading.
 - Platform-specific scripts live under ./scripts/<platform>/; today only
   scripts/ubuntu/ exists (Fedora/Arch/macOS/Windows equivalents are planned)
+- Addon test filename convention (mirrors addons-source/.github/workflows/ci.yml):
+  - `test_*.py` — general, runs on every platform
+  - `test_linux_*.py` — Linux-only
+  - `test_windows_*.py` — Windows-only
+  - `test_integration_*.py` — Linux-only, full-pipeline/DB-backed
+  Each platform's `run-addon-unit.sh` (and the corresponding CI job) filters
+  out the prefixes that don't match its OS. The Ubuntu runner skips
+  `test_windows_*` and runs everything else; a future `scripts/windows/`
+  runner would do the inverse. Until a Windows image/runner exists in the
+  testbed, `test_windows_*.py` runs nowhere here — Windows coverage lives
+  in addons-source's own Windows CI job.
 - Docker image: gramps-testbed:ubuntu-<gramps-version> (e.g. gramps-testbed:ubuntu-6.0.8),
   built from docker/Dockerfile.ubuntu; version is auto-read from gramps/version.py
   by the wrapper scripts so different Gramps versions get different tags

scripts/ubuntu/run-addon-unit.sh

@@ -158,9 +158,20 @@ PY
       # surfaces package-shadowing bugs like bug 0012691, where
       # "from <Addon> import <Addon>" binds the submodule instead of the
       # class. Discover-from-tests/ hides the trap.
+      # Filename convention (mirrors addons-source/.github/workflows/ci.yml):
+      #   test_*.py              general — every platform
+      #   test_linux_*.py        Linux-only
+      #   test_windows_*.py      Windows-only (skipped here)
+      #   test_integration_*.py  Linux-only, full-pipeline/DB-backed
+      # This is the Ubuntu runner, so test_windows_* is excluded. When
+      # scripts/windows/run-addon-unit.sh is added it will do the inverse
+      # (skip test_linux_*/test_integration_*).
       modules=()
       for f in /workspace/addons-source/"$addon"/tests/test_*.py; do
         [ -f "$f" ] || continue
+        case "$(basename "$f")" in
+          test_windows_*) continue ;;
+        esac
         rel="${f#/workspace/addons-source/}"
         mod="${rel%.py}"
         mod="${mod//\//.}"