Skip to content

Commit cf09aa8

Browse files
Merge pull request #2713 from VWS-Python/cov-simple
Fix and simplify coverage config
2 parents 57a0f01 + 75fa85f commit cf09aa8

File tree

2 files changed

+25
-34
lines changed

2 files changed

+25
-34
lines changed

.github/workflows/test.yml

Lines changed: 22 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -153,16 +153,12 @@ jobs:
153153
- name: Run tests
154154
run: |
155155
uv run --extra=dev \
156-
coverage run \
157-
--parallel-mode \
158-
--source=src/ \
159-
--source=tests/ \
160-
-m pytest \
161-
-s \
162-
-vvv \
163-
--showlocals \
164-
--exitfirst \
165-
${{ matrix.ci_pattern }}
156+
coverage run -m pytest \
157+
-s \
158+
-vvv \
159+
--showlocals \
160+
--exitfirst \
161+
${{ matrix.ci_pattern }}
166162
env:
167163
UV_PYTHON: ${{ matrix.python-version }}
168164

@@ -204,19 +200,15 @@ jobs:
204200
- name: Run tests
205201
run: |
206202
uv run --extra=dev \
207-
coverage run \
208-
--parallel-mode \
209-
--source=src/ \
210-
--source=tests/ \
211-
-m pytest \
212-
--skip-docker_build_tests \
213-
--skip-docker_in_memory \
214-
--skip-mock \
215-
--skip-real \
216-
--capture=no \
217-
-vvv \
218-
--exitfirst \
219-
.
203+
coverage run -m pytest \
204+
--skip-docker_build_tests \
205+
--skip-docker_in_memory \
206+
--skip-mock \
207+
--skip-real \
208+
--capture=no \
209+
-vvv \
210+
--exitfirst \
211+
.
220212
env:
221213
UV_PYTHON: ${{ matrix.python-version }}
222214

@@ -260,16 +252,12 @@ jobs:
260252
#
261253
# We therefore have a few ``# pragma: no cover`` statements.
262254
uv run --extra=dev \
263-
coverage run \
264-
--parallel-mode \
265-
--source=src/ \
266-
--source=tests/ \
267-
-m pytest \
268-
--skip-real \
269-
-vvv \
270-
--exitfirst \
271-
-n auto \
272-
.
255+
coverage run -m pytest \
256+
--skip-real \
257+
-vvv \
258+
--exitfirst \
259+
-n auto \
260+
.
273261
env:
274262
UV_PYTHON: ${{ matrix.python-version }}
275263

@@ -305,7 +293,7 @@ jobs:
305293
coverage report --format=markdown >> "$GITHUB_STEP_SUMMARY"
306294
307295
# Report again and fail if under 100%.
308-
coverage report --fail-under=100
296+
coverage report
309297
310298
- name: Upload HTML report if check failed
311299
uses: actions/upload-artifact@v4

pyproject.toml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -365,13 +365,16 @@ branch = true
365365
omit = [
366366
"src/mock_vws/_flask_server/healthcheck.py",
367367
]
368+
parallel = true
369+
source = [ "src/", "tests/" ]
368370

369371
[tool.coverage.report]
370372

371373
exclude_also = [
372374
"if TYPE_CHECKING:",
373375
"class .*\\bProtocol\\):",
374376
]
377+
fail_under = 100
375378

376379
[tool.mypy]
377380

0 commit comments

Comments
 (0)