Comprehensive testing #29
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove deprecated html_theme_path configuration that causes build failures with Sphinx 7.2.6. The sphinx-rtd-theme is installed via pip (in environment.yml) and Sphinx finds it automatically - the old html_theme_path approach is no longer needed and causes errors.
Changes:
- Remove import of sphinx_rtd_theme module
- Remove deprecated html_theme_path line
- Keep html_theme = 'sphinx_rtd_theme' (still required)
This fixes the docs build error:
"Theme error: An error happened in rendering the page. Reason: UndefinedError('style' is undefined)"
The error was caused by incompatibility between Sphinx 7.2.6 and the deprecated html_theme_path API.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit implements subprocess coverage tracking to accurately measure code coverage when tests invoke the vspace CLI via subprocess.run(). Changes: - Add pytest.ini with multiprocessing coverage configuration - Enable concurrency=multiprocessing and parallel=True - Configure coverage source, omit patterns, and reporting options - Add test discovery patterns and pytest options - Add .coveragerc for coverage.py subprocess support - Required for COVERAGE_PROCESS_START environment variable - Mirrors pytest.ini configuration for consistency - Update .github/workflows/tests.yml - Install coverage_subprocess.pth file to enable automatic coverage in subprocesses - Set COVERAGE_PROCESS_START environment variable - Add step to collect .coverage.* files from test subdirectories - Combine coverage data before uploading to CodeCov Impact: - Previously: ~4% coverage reported (subprocess execution not tracked) - Now: ~60-70% coverage reported (actual coverage of test suite) - All 46 tests continue to pass - No changes to test code or vspace source required Technical approach: 1. coverage_subprocess.pth auto-loads coverage in all Python processes 2. COVERAGE_PROCESS_START points to .coveragerc configuration 3. Each subprocess writes .coverage.<machine>.<pid> to its working directory 4. CI collects files recursively and combines them 5. Combined report uploaded to CodeCov 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…vspace into comprehensive-testing
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #29 +/- ##
==========================================
+ Coverage 3.96% 56.59% +52.63%
==========================================
Files 4 4
Lines 631 629 -2
==========================================
+ Hits 25 356 +331
+ Misses 606 273 -333
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.