error if expected test is skipped

AVSLab · Dec 14, 2024 · 529e0bb · 529e0bb
1 parent 619e6ff
commit 529e0bb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -121,7 +121,7 @@ jobs:
       - name: "Run Python Tests"
         run: |
           source .venv/bin/activate
-          cd src && pytest -n auto -m "not ciSkip" -rs --fail-on-skip
+          cd src && pytest -n auto -m "not ciSkip" -rs --error-for-skips
 
       - name: "Run C/C++ Tests"
         working-directory: ./dist3
@@ -206,7 +206,7 @@ jobs:
           Set-ItemProperty -Path 'Registry::HKEY_LOCAL_MACHINE\System\CurrentControlSet\Control\Session Manager\Environment' -Name MPLBACKEND -Value ${MPLBACKEND}
           venv\Scripts\activate
           cd src
-          pytest -n auto -m "not ciSkip" -rs --fail-on-skip
+          pytest -n auto -m "not ciSkip" -rs --error-for-skips
           if(($LastExitCode -ne 0) -and ($LastExitCode -ne 5)) {exit 1}
       - name: "C/C++ Tests"
         if: ${{ always() }}
@@ -249,7 +249,7 @@ jobs:
         run: |
           source .venv/bin/activate
           cd src
-          pytest -n auto -m "not ciSkip" -rs --fail-on-skip
+          pytest -n auto -m "not ciSkip" -rs --error-for-skips
         if: ${{ always() }}
       - name: "Run C/C++ Tests"
         working-directory: ./dist3

diff --git a/requirements.txt b/requirements.txt
@@ -7,4 +7,5 @@ Pillow
 pytest
 pytest-html
 pytest-xdist
+pytest-error-for-skips
 requests