ci(pytest): Be OS-explicit in the pytest regression tests

This makes it easier to add support for MS-Windows and macOS
Only upload to coveralls from the main repo, not from the forks

Author: amilcarlucas

.github/workflows/pytest.yml

@@ -29,13 +29,16 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        # os: [ubuntu-latest, macos-latest, windows-latest]
-        # python-version: ["3.9", "3.10", "3.11", "3.12", "pypy3.9", "pypy3.10"]
         os: [ubuntu-latest]
-        python-version: ["3.9", "3.14", "3.14t"]
+        python-version: ["3.14"]  # to make sure the latest AMC supported python version does not break
+      include:
+        - os: ubuntu-latest
+          python-version: "3.9"  # to make sure the minimum AMC supported python version does not break
+          # this is also the version used to report test coverage, other versions do not report coverage
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
+        if: matrix.os != 'windows-latest'
         uses: step-security/harden-runner@5ef0c079ce82195b2a36a210272d6b661572d83e # v2.14.2
         with:
           egress-policy: audit
@@ -50,7 +53,8 @@ jobs:
           python-version: ${{ matrix.python-version }}
           activate-environment: true
 
-      - name: Install system dependencies for GUI testing
+      - name: Install system dependencies for GUI testing on linux
+        if: matrix.os == 'ubuntu-latest'
         run: |
           sudo apt-get update
           # Only install system Tcl/Tk for Python < 3.13 (newer versions bundle their own)
@@ -65,6 +69,7 @@ jobs:
           python3 --version && python3 -c "import tkinter; print(tkinter.TclVersion, tkinter.TkVersion)"
 
       - name: Ensure Tcl/Tk search paths
+        if: matrix.os == 'ubuntu-latest'
         run: |
           # Only set Tcl/Tk paths for Python < 3.13
           if [[ "${{ matrix.python-version }}" < "3.13" ]]; then
@@ -81,6 +86,7 @@ jobs:
           uv pip install --editable .[dev,ci_headless_tests]
 
       - name: Download ArduCopter SITL (if available)
+        if: matrix.os == 'ubuntu-latest'
         run: |
           # Create cache key based on current quarter (YYYY-Q)
           CURRENT_YEAR=$(date +%Y)
@@ -160,31 +166,38 @@ jobs:
         if: ${{ always() }}
 
       - name: Upload coverage xml report
+        # Use always() to always run this step to publish test results when there are test failures
+        if: matrix.os == 'ubuntu-latest' && always()
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v6.0.0
         with:
             name: coverage-${{ matrix.python-version }}-xml
             path: tests/*.xml
             retention-days: 1
-        # Use always() to always run this step to publish test results when there are test failures
-        if: ${{ always() }}
 
       - name: Upload coverage report
+        # Use always() to always run this step to publish test results when there are test failures
+        if: matrix.os == 'ubuntu-latest' && always()
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v6.0.0
         with:
             name: coverage-${{ matrix.python-version }}
             path: .coverage
             include-hidden-files: true
             retention-days: 1
-        # Use always() to always run this step to publish test results when there are test failures
-        if: ${{ always() }}
 
   upload_coverage_to_coveralls:
-    if: (github.event_name == 'push' && github.ref == 'refs/heads/master') && (success() || failure())
+    if: >
+      github.repository == 'ArduPilot/MethodicConfigurator' &&
+      matrix.os == 'ubuntu-latest' &&
+      matrix.python-version == '3.9' &&
+      github.event_name == 'push' &&
+      github.ref == 'refs/heads/master' &&
+      (success() || failure())
     runs-on: ubuntu-latest
     needs: pytest
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
+        if: matrix.os != 'windows-latest'
         uses: step-security/harden-runner@5ef0c079ce82195b2a36a210272d6b661572d83e # v2.14.2
         with:
           egress-policy: audit
@@ -212,6 +225,7 @@ jobs:
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
+        if: matrix.os != 'windows-latest'
         uses: step-security/harden-runner@5ef0c079ce82195b2a36a210272d6b661572d83e # v2.14.2
         with:
           egress-policy: audit
@@ -257,6 +271,7 @@ jobs:
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
+        if: matrix.os != 'windows-latest'
         uses: step-security/harden-runner@5ef0c079ce82195b2a36a210272d6b661572d83e # v2.14.2
         with:
           egress-policy: audit
@@ -316,6 +331,7 @@ jobs:
 
     steps:
       - name: Harden the runner (Audit all outbound calls)
+        if: matrix.os != 'windows-latest'
         uses: step-security/harden-runner@5ef0c079ce82195b2a36a210272d6b661572d83e # v2.14.2
         with:
           egress-policy: audit