error if expected test is skipped

AVSLab · Dec 15, 2024 · f826306 · f826306
1 parent 2bb76da
commit f826306
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -114,14 +114,14 @@ jobs:
       - name: "Create virtual Environment"
         run: python3 -m venv .venv
       - name: "Install wheel and conan package"
-        run: source .venv/bin/activate && pip3 install wheel conan
+        run: source .venv/bin/activate && pip3 install wheel conan pytest-error-for-skips
       - name: "Build basilisk"
         run: source .venv/bin/activate && python3 conanfile.py --opNav True --allOptPkg
 
       - name: "Run Python Tests"
         run: |
           source .venv/bin/activate
-          cd src && pytest -n auto -m "not ciSkip" -rs
+          cd src && pytest -n auto -m "not ciSkip" -rs --error-for-skips
 
       - name: "Run C/C++ Tests"
         working-directory: ./dist3
@@ -188,7 +188,7 @@ jobs:
         shell: pwsh
         run: |
             venv\Scripts\activate
-            pip install wheel conan
+            pip install wheel conan pytest-error-for-skips
       - name: "Add basilisk and cmake path to env path"
         shell: pwsh
         run: |
@@ -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
+          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() }}
@@ -240,7 +240,7 @@ jobs:
       - name: "Install wheel and conan package"
         run: |
           source .venv/bin/activate
-          pip3 install wheel conan cmake
+          pip3 install wheel conan cmake pytest-error-for-skips
 
       - name: "Build basilisk with OpNav"
         run: source .venv/bin/activate && python3 conanfile.py --opNav True --allOptPkg
@@ -249,7 +249,7 @@ jobs:
         run: |
           source .venv/bin/activate
           cd src
-          pytest -n auto -m "not ciSkip" -rs
+          pytest -n auto -m "not ciSkip" -rs --error-for-skips
         if: ${{ always() }}
       - name: "Run C/C++ Tests"
         working-directory: ./dist3