Do not test on Windows due to conda-forge failed release

conda-forge/pytensor-suite-feedstock#177

Author: ricardoV94

.github/workflows/tests.yml

@@ -178,12 +178,12 @@ jobs:
           name: ${{ matrix.os }} ${{ matrix.floatx }}
           fail_ci_if_error: false
 
-  windows:
+  should_have_been_windows:
     needs: changes
     if: ${{ needs.changes.outputs.changes == 'true' }}
     strategy:
       matrix:
-        os: [windows-latest]
+        os: [ubuntu-latest]  # should revert back to windows-latest
         floatx: [float64]
         python-version: ["3.11"]
         test-subset:
@@ -199,18 +199,18 @@ jobs:
       PYTENSOR_FLAGS: floatX=${{ matrix.floatx }}
     defaults:
       run:
-        shell: cmd /C call {0}
+        shell: bash -leo pipefail {0}
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
         with:
           persist-credentials: false
       - uses: mamba-org/setup-micromamba@v2
         with:
-          environment-file: conda-envs/windows-environment-test.yml
+          environment-file: conda-envs/environment-test.yml
           create-args: >-
             python=${{matrix.python-version}}
           environment-name: pymc-test
-          init-shell: cmd.exe
+          init-shell: bash
           cache-environment: true
       - name: Install-pymc
         run: |
@@ -220,8 +220,8 @@ jobs:
       - name: Run tests
         # This job uses a cmd shell, therefore the environment variable syntax is different!
         # The ">-" in the next line replaces newlines with spaces (see https://stackoverflow.com/a/66809682).
-        run: >-
-          python -m pytest -vv --cov=pymc --cov-report=xml --no-cov-on-fail --cov-report term --durations=50 %TEST_SUBSET%
+        run: |
+          python -m pytest -vv --cov=pymc --cov-report=xml --no-cov-on-fail --cov-report term --durations=50 $TEST_SUBSET
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v5
         with:
@@ -343,7 +343,7 @@ jobs:
     if: ${{ needs.changes.outputs.changes == 'true' }}
     strategy:
       matrix:
-        os: [windows-latest]
+        os: [ubuntu-latest]  # should revert back ot windows-latest
         floatx: [float32]
         python-version: ["3.13"]
         test-subset:
@@ -355,18 +355,18 @@ jobs:
       PYTENSOR_FLAGS: floatX=${{ matrix.floatx }}
     defaults:
       run:
-        shell: cmd /C call {0}
+        shell: bash -leo pipefail {0}
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
         with:
           persist-credentials: false
       - uses: mamba-org/setup-micromamba@v2
         with:
-          environment-file: conda-envs/windows-environment-test.yml
+          environment-file: conda-envs/environment-test.yml
           create-args: >-
             python=${{matrix.python-version}}
           environment-name: pymc-test
-          init-shell: cmd.exe
+          init-shell: bash
           cache-environment: true
       - name: Install-pymc
         run: |
@@ -376,8 +376,8 @@ jobs:
       - name: Run tests
         # This job uses a cmd shell, therefore the environment variable syntax is different!
         # The ">-" in the next line replaces newlines with spaces (see https://stackoverflow.com/a/66809682).
-        run: >-
-          python -m pytest -vv --cov=pymc --cov-report=xml --no-cov-on-fail --cov-report term --durations=50 %TEST_SUBSET%
+        run: |
+          python -m pytest -vv --cov=pymc --cov-report=xml --no-cov-on-fail --cov-report term --durations=50 $TEST_SUBSET
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v5
         with:
@@ -389,12 +389,12 @@ jobs:
   all_tests:
     if: ${{ always() }}
     runs-on: ubuntu-latest
-    needs: [ changes, ubuntu, windows, macos, alternative_backends, float32 ]
+    needs: [ changes, ubuntu, macos, alternative_backends, float32 ]
     steps:
       - name: Check build matrix status
         if: ${{ needs.changes.outputs.changes == 'true' &&
                 ( needs.ubuntu.result != 'success' ||
-                  needs.windows.result != 'success' ||
+                  needs.should_have_been_windows.result != 'success' ||
                   needs.macos.result != 'success' ||
                   needs.alternative_backends.result != 'success' ||
                   needs.float32.result != 'success' ) }}

scripts/check_all_tests_are_covered.py

@@ -10,6 +10,7 @@
 import itertools
 import logging
 import os
+import warnings
 
 from pathlib import Path
 
@@ -110,7 +111,7 @@ def from_yaml():
             f"{len(ignored_by_all)} tests are completely ignored:\n{ignored_by_all}"
         )
     if run_multiple_times:
-        raise AssertionError(
+        warnings.warn(
             f"{len(run_multiple_times)} tests are run multiple times with the same OS and floatX setting:\n{run_multiple_times}"
         )
     return