error if expected test is skipped

AVSLab · Dec 15, 2024 · 7748e1b · 7748e1b
1 parent b38433d
commit 7748e1b
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 requirements_dev.txt"
-        run: source .venv/bin/activate && pip3 install -r requirements_dev.txt
+        run: source .venv/bin/activate && pip3 install -r requirements_dev.txt 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
@@ -189,7 +189,7 @@ jobs:
         shell: pwsh
         run: |
             venv\Scripts\activate
-            pip install -r requirements_dev.txt
+            pip install -r requirements_dev.txt pytest-error-for-skips
       - name: "Add basilisk and cmake path to env path"
         shell: pwsh
         run: |
@@ -207,7 +207,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() }}
@@ -241,7 +241,7 @@ jobs:
       - name: "Install requirements_dev.txt"
         run: |
           source .venv/bin/activate
-          pip3 install cmake -r requirements_dev.txt
+          pip3 install cmake -r requirements_dev.txt pytest-error-for-skips
 
       - name: "Build basilisk with OpNav"
         run: source .venv/bin/activate && python3 conanfile.py --opNav True --allOptPkg
@@ -250,7 +250,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