Skip to content

Conversation

@RoryBarnes
Copy link
Contributor

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

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>
@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.13 on ubuntu-24.04)

39 tests   38 ✅  13s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.10 on ubuntu-24.04)

39 tests   38 ✅  14s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.10 on ubuntu-22.04)

39 tests   38 ✅  14s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.11 on ubuntu-22.04)

39 tests   38 ✅  14s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.11 on ubuntu-24.04)

39 tests   38 ✅  14s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.12 on ubuntu-24.04)

39 tests   38 ✅  16s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.13 on ubuntu-22.04)

39 tests   38 ✅  16s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.12 on ubuntu-22.04)

39 tests   38 ✅  16s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.14 on ubuntu-22.04)

39 tests   38 ✅  19s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.9 on ubuntu-22.04)

39 tests   38 ✅  14s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.14 on ubuntu-24.04)

39 tests   38 ✅  18s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.9 on ubuntu-24.04)

39 tests   38 ✅  14s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@codecov
Copy link

codecov bot commented Jan 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.81%. Comparing base (9a6526b) to head (f2f990e).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             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     
Flag Coverage Δ
ubuntu-22.04-py3.9 74.81% <ø> (+44.21%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.11 on macos-15-intel)

39 tests   38 ✅  32s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.13 on macos-15-intel)

39 tests   38 ✅  30s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.12 on macos-15-intel)

39 tests   38 ✅  33s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.10 on macos-15-intel)

39 tests   38 ✅  42s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.9 on macos-15-intel)

39 tests   38 ✅  42s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.10 on macos-26)

39 tests   38 ✅  18s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.11 on macos-26)

39 tests   38 ✅  36s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.13 on macos-26)

39 tests   38 ✅  20s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.14 on macos-15-intel)

39 tests   38 ✅  31s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.14 on macos-26)

39 tests   38 ✅  19s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.9 on macos-26)

39 tests   38 ✅  19s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Test Results (py3.12 on macos-26)

39 tests   38 ✅  22s ⏱️
 1 suites   1 💤
 1 files     0 ❌

Results for commit f2f990e.

@RoryBarnes RoryBarnes merged commit d8bebc0 into main Jan 4, 2026
50 checks passed
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