Skip to content

Separate tests and coverage in GitHub Actions#10

Merged
xulek merged 2 commits into
mainfrom
claude/split-tests-coverage-actions-01QAk6iyy3zdgdKUt1VjSFyq
Nov 16, 2025
Merged

Separate tests and coverage in GitHub Actions#10
xulek merged 2 commits into
mainfrom
claude/split-tests-coverage-actions-01QAk6iyy3zdgdKUt1VjSFyq

Conversation

@xulek
Copy link
Copy Markdown
Owner

@xulek xulek commented Nov 16, 2025

No description provided.

- Created separate 'test' job: runs tests without coverage for faster feedback
- Created separate 'coverage' job: runs tests with coverage measurement
- Tests in 'test' job no longer install pytest-cov (faster installation)
- Both jobs run in parallel for better CI efficiency
- Updated 'build' job to depend on both test and coverage
- Updated 'coverage-report' job to depend on coverage job
Coverage was running in test job because pytest.ini had --cov flags
in addopts, which are applied to ALL pytest runs by default.

Changes:
- Removed --cov=vnc_lib from default addopts
- Removed --cov-report flags from default addopts
- Removed --cov-fail-under from default addopts
- Kept coverage configuration sections for when --cov is explicitly used
- Now coverage only runs when explicitly requested via command line flags

This ensures the 'test' job runs fast without coverage overhead,
while the 'coverage' job still runs with full coverage measurement.
@xulek xulek merged commit 60a46af into main Nov 16, 2025
12 checks passed
@xulek xulek deleted the claude/split-tests-coverage-actions-01QAk6iyy3zdgdKUt1VjSFyq branch November 16, 2025 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants