Skip to content

Setup coverage (windows) #249

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
gvanrossum merged 2 commits into from
Apr 24, 2026
Merged

Setup coverage (windows) #249

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b3c63f0
fix setup of coverage and add support on windows
bmerkle Apr 23, 2026
8e5dea3
fixed source location for coverage
bmerkle Apr 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .coveragerc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
[run]
branch = true
parallel = true
source = test
source = src

[report]
sort = cover
13 changes: 12 additions & 1 deletion make.bat
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ if "%~1"=="" goto help
if /I "%~1"=="format" goto format
if /I "%~1"=="check" goto check
if /I "%~1"=="test" goto test
if /I "%~1"=="coverage" goto coverage
if /I "%~1"=="demo" goto demo
if /I "%~1"=="build" goto build
if /I "%~1"=="venv" goto venv
Expand Down Expand Up @@ -43,6 +44,16 @@ echo Running unit tests...
uv run pytest
goto end

:coverage
if not exist ".venv\" call make.bat venv
echo Running test coverage...
uv run coverage erase
uv run coverage run -m pytest
uv run coverage combine
uv run coverage report
Comment on lines +50 to +53
Copy link

Copilot AI Apr 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Windows coverage target doesn't set COVERAGE_PROCESS_START. Some tests spawn the MCP server in a subprocess and only propagate COVERAGE_PROCESS_START if it’s already present, so coverage for subprocesses won’t be collected on Windows (unlike the Makefile target). Set COVERAGE_PROCESS_START=.coveragerc before coverage run (and optionally unset it afterwards) to match the Unix behavior.

Suggested change
uv run coverage erase
uv run coverage run -m pytest
uv run coverage combine
uv run coverage report
uv run coverage erase
set "COVERAGE_PROCESS_START=.coveragerc"
uv run coverage run -m pytest
uv run coverage combine
uv run coverage report
set "COVERAGE_PROCESS_START="

Copilot uses AI. Check for mistakes.
goto end


:demo
if not exist ".venv\" call make.bat venv
echo Running query tool...
Expand Down Expand Up @@ -88,7 +99,7 @@ if exist .pytest_cache rmdir /s /q .pytest_cache
goto end

:help
echo Usage: .\make [format^|check^|test^|build^|venv^|sync^|install-uv^|clean^|help]
echo Usage: .\make [format^|check^|test^|coverage^|demo^|build^|venv^|sync^|install-uv^|clean^|help]
goto end

:end
Loading