-
Notifications
You must be signed in to change notification settings - Fork 1
Improve test coverage from 30.6% to 74.81% #26
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.
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
Conversation
This commit implements a comprehensive test coverage improvement strategy, adding 29 new unit tests across three test files. The coverage increased from 30.6% (82/268 statements) to 74.81% (199/266 statements) when excluding the unused multiplanet_refactored.py file. New test files: - tests/unit/test_helpers.py: Tests for GetSNames, GetSims, GetDir (10 tests) - tests/unit/test_mpstatus.py: Tests for mpstatus module (5 tests) - tests/fixtures/conftest.py: Shared pytest fixtures for test utilities Expanded test files: - tests/unit/test_multiplanet.py: Added CreateCP, ReCreateCP, and Arguments tests (14 new tests), expanding from 123 to 462 lines Configuration updates: - pyproject.toml: Added coverage configuration to exclude unused code - Removed .coveragerc (backed up to .coveragerc.bak) in favor of pyproject.toml Coverage by module: - multiplanet/__init__.py: 100% (6/6 statements) - multiplanet/mpstatus.py: 97.14% (34/35 statements) - multiplanet/multiplanet.py: 69.63% (149/214 statements) - multiplanet/multiplanet_module.py: 90.91% (10/11 statements) All tests pass (34 unit tests + existing integration tests). The remaining uncovered code consists primarily of multiprocessing worker functions that are already thoroughly tested by the existing integration test suite. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Test Results (py3.13 on ubuntu-24.04)39 tests 38 ✅ 13s ⏱️ Results for commit f2f990e. |
Test Results (py3.10 on ubuntu-24.04)39 tests 38 ✅ 14s ⏱️ Results for commit f2f990e. |
Test Results (py3.10 on ubuntu-22.04)39 tests 38 ✅ 14s ⏱️ Results for commit f2f990e. |
Test Results (py3.11 on ubuntu-22.04)39 tests 38 ✅ 14s ⏱️ Results for commit f2f990e. |
Test Results (py3.11 on ubuntu-24.04)39 tests 38 ✅ 14s ⏱️ Results for commit f2f990e. |
Test Results (py3.12 on ubuntu-24.04)39 tests 38 ✅ 16s ⏱️ Results for commit f2f990e. |
Test Results (py3.13 on ubuntu-22.04)39 tests 38 ✅ 16s ⏱️ Results for commit f2f990e. |
Test Results (py3.12 on ubuntu-22.04)39 tests 38 ✅ 16s ⏱️ Results for commit f2f990e. |
Test Results (py3.14 on ubuntu-22.04)39 tests 38 ✅ 19s ⏱️ Results for commit f2f990e. |
Test Results (py3.9 on ubuntu-22.04)39 tests 38 ✅ 14s ⏱️ Results for commit f2f990e. |
Test Results (py3.14 on ubuntu-24.04)39 tests 38 ✅ 18s ⏱️ Results for commit f2f990e. |
Test Results (py3.9 on ubuntu-24.04)39 tests 38 ✅ 14s ⏱️ Results for commit f2f990e. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #26 +/- ##
===========================================
+ Coverage 30.59% 74.81% +44.21%
===========================================
Files 4 4
Lines 268 266 -2
===========================================
+ Hits 82 199 +117
+ Misses 186 67 -119
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Test Results (py3.11 on macos-15-intel)39 tests 38 ✅ 32s ⏱️ Results for commit f2f990e. |
Test Results (py3.13 on macos-15-intel)39 tests 38 ✅ 30s ⏱️ Results for commit f2f990e. |
Test Results (py3.12 on macos-15-intel)39 tests 38 ✅ 33s ⏱️ Results for commit f2f990e. |
Test Results (py3.10 on macos-15-intel)39 tests 38 ✅ 42s ⏱️ Results for commit f2f990e. |
Test Results (py3.9 on macos-15-intel)39 tests 38 ✅ 42s ⏱️ Results for commit f2f990e. |
Test Results (py3.10 on macos-26)39 tests 38 ✅ 18s ⏱️ Results for commit f2f990e. |
Test Results (py3.11 on macos-26)39 tests 38 ✅ 36s ⏱️ Results for commit f2f990e. |
Test Results (py3.13 on macos-26)39 tests 38 ✅ 20s ⏱️ Results for commit f2f990e. |
Test Results (py3.14 on macos-15-intel)39 tests 38 ✅ 31s ⏱️ Results for commit f2f990e. |
Test Results (py3.14 on macos-26)39 tests 38 ✅ 19s ⏱️ Results for commit f2f990e. |
Test Results (py3.9 on macos-26)39 tests 38 ✅ 19s ⏱️ Results for commit f2f990e. |
Test Results (py3.12 on macos-26)39 tests 38 ✅ 22s ⏱️ Results for commit f2f990e. |
This commit implements a comprehensive test coverage improvement strategy, adding 29 new unit tests across three test files. The coverage increased from 30.6% (82/268 statements) to 74.81% (199/266 statements) when excluding the unused multiplanet_refactored.py file.
New test files:
Expanded test files:
Configuration updates:
Coverage by module:
All tests pass (34 unit tests + existing integration tests). The remaining uncovered code consists primarily of multiprocessing worker functions that are already thoroughly tested by the existing integration test suite.
🤖 Generated with Claude Code