Skip to content

Commit 11462fe

Browse files
committed
have the CI tests show the reason for the skipped tests
Mark test runs that should not be skipping a test as a failure if a test is skipped
1 parent 71f24f1 commit 11462fe

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

.github/workflows/pull-request.yml

+7-7
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ jobs:
5252
if: ${{ always() }}
5353
run: |
5454
source .venv/bin/activate
55-
cd src && pytest -n auto -m "not ciSkip"
55+
cd src && pytest -n auto -m "not ciSkip" -rs
5656
- name: Run C/C++ Tests
5757
if: ${{ always() }}
5858
working-directory: ./dist3
@@ -85,7 +85,7 @@ jobs:
8585
- name: "Run Python Tests"
8686
run: |
8787
source .venv/bin/activate
88-
cd src && pytest -n auto -m "not ciSkip"
88+
cd src && pytest -n auto -m "not ciSkip" -rs
8989
9090
- name: "Run C/C++ Tests"
9191
working-directory: ./dist3
@@ -121,7 +121,7 @@ jobs:
121121
- name: "Run Python Tests"
122122
run: |
123123
source .venv/bin/activate
124-
cd src && pytest -n auto -m "not ciSkip"
124+
cd src && pytest -n auto -m "not ciSkip" -rs --fail-on-skip
125125
126126
- name: "Run C/C++ Tests"
127127
working-directory: ./dist3
@@ -156,7 +156,7 @@ jobs:
156156
- name: "Run Python Tests"
157157
run: |
158158
source .venv/bin/activate
159-
cd src && pytest -n auto -m "not ciSkip"
159+
cd src && pytest -n auto -m "not ciSkip" -rs
160160
161161
162162
build-windows:
@@ -206,7 +206,7 @@ jobs:
206206
Set-ItemProperty -Path 'Registry::HKEY_LOCAL_MACHINE\System\CurrentControlSet\Control\Session Manager\Environment' -Name MPLBACKEND -Value ${MPLBACKEND}
207207
venv\Scripts\activate
208208
cd src
209-
pytest -n auto -m "not ciSkip"
209+
pytest -n auto -m "not ciSkip" -rs --fail-on-skip
210210
if(($LastExitCode -ne 0) -and ($LastExitCode -ne 5)) {exit 1}
211211
- name: "C/C++ Tests"
212212
if: ${{ always() }}
@@ -249,7 +249,7 @@ jobs:
249249
run: |
250250
source .venv/bin/activate
251251
cd src
252-
pytest -n auto -m "not ciSkip"
252+
pytest -n auto -m "not ciSkip" -rs --fail-on-skip
253253
if: ${{ always() }}
254254
- name: "Run C/C++ Tests"
255255
working-directory: ./dist3
@@ -295,6 +295,6 @@ jobs:
295295
run: |
296296
source .venv/bin/activate
297297
cd src
298-
pytest -n auto -m "not ciSkip"
298+
pytest -n auto -m "not ciSkip" -rs
299299
300300
if: ${{ always() }}

0 commit comments

Comments
 (0)