Skip to content

Commit 930685b

Browse files
committed
CI: measure coverage in Python subprocesses via patch = subprocess
Tests such as test_helptext and test_shell_completion run `dandi` as a subprocess, so the CLI code they exercise was never recorded. coverage's `patch = subprocess` starts measurement in child Python processes, writing parallel data files that the existing `coverage combine` picks up. Bump the floor to coverage 7.10.6, which carries fixes for this feature. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UkDjoCcCmBNuhFkmTyyohk
1 parent 06343e3 commit 930685b

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

‎pyproject.toml‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,8 @@ extras = [
105105
test = [
106106
"aiohttp >= 3.8.4, < 3.14", # See https://github.com/kevin1024/vcrpy/issues/995
107107
"anys ~= 0.3",
108-
"coverage >= 7.0.0",
108+
# 7.10.6 has fixes for `patch = subprocess` (see tox.ini)
109+
"coverage >= 7.10.6",
109110
# 4.10.0.84 first built for numpy 2.x ABI (older wheels segfault under numpy 2)
110111
# Workaround for VideoWriter regression in 4.13.0.90 on Intel macOS
111112
"opencv-python >= 4.10.0.84, < 4.13 ; sys_platform == 'darwin' and platform_machine != 'arm64'",

‎tox.ini‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,8 @@ filterwarnings =
9191

9292
[coverage:run]
9393
parallel = True
94+
# Also measure Python subprocesses, e.g. tests invoking the `dandi` CLI
95+
patch = subprocess
9496
source = dandi
9597

9698
[coverage:report]

0 commit comments

Comments
 (0)