Skip to content

test: add unit tests for pet-pyenv crate (Fixes #389)#441

Open
karthiknadig wants to merge 3 commits intomainfrom
test/issue-389-pyenv
Open

test: add unit tests for pet-pyenv crate (Fixes #389)#441
karthiknadig wants to merge 3 commits intomainfrom
test/issue-389-pyenv

Conversation

@karthiknadig
Copy link
Copy Markdown
Member

Add 22 new unit tests to pet-pyenv (27 total, up from 5 integration tests):

  • environments.rs (14 tests): get_version parsing (stable, dev, alpha, win32, non-version, partial, rc limitation), get_generic_python_environment (version extraction, win32 arch, no-arch, manager, unrecognized folder), get_virtual_env_environment (with/without pyvenv.cfg)
  • environment_locations.rs (8 tests): get_pyenv_dir (PYENV_ROOT priority, PYENV fallback, no vars), get_home_pyenv_dir (no home, expected path), get_binary_from_known_paths (found, not found, empty)

Fixes #389

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Performance Report (Linux) ➖

Metric PR (P50) PR (P95) Baseline (P50) Delta Change
Server Startup 1ms 1ms 0ms 1ms N/A%
Full Refresh 65ms 225ms 53ms 12ms 20.0%

Results based on 10 iterations. P50 = median, P95 = 95th percentile.


Legend
  • 🚀 Significant speedup (>100ms faster)
  • ✅ Faster than baseline
  • ➖ No significant change
  • 🔺 Slower than baseline (>100ms)
  • ⚠️ Significant slowdown (>500ms)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Test Coverage Report (Linux)

Metric Value
Current Coverage 75.9%
Base Branch Coverage 75.6%
Delta .3% ✅

Coverage increased! Great work!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Performance Report (Windows) ✅

Metric PR (P50) PR (P95) Baseline (P50) Delta Change
Server Startup 9ms 12ms 11ms -2ms -18.2%
Full Refresh 147ms 401ms 202ms -55ms -27.2%

Results based on 10 iterations. P50 = median, P95 = 95th percentile.


Legend
  • 🚀 Significant speedup (>100ms faster)
  • ✅ Faster than baseline
  • ➖ No significant change
  • 🔺 Slower than baseline (>100ms)
  • ⚠️ Significant slowdown (>500ms)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Performance Report (macOS)

Metric PR (P50) PR (P95) Baseline (P50) Delta
Server Startup 57ms 727ms 85ms -28ms
Full Refresh 105ms 30660ms 175ms -70ms

Results based on 10 iterations. P50 = median, P95 = 95th percentile.


Legend
  • 🚀 Significant speedup (>100ms faster)
  • ✅ Faster than baseline
  • ➖ No significant change
  • 🔺 Slower than baseline (>100ms)
  • ⚠️ Significant slowdown (>500ms)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 22, 2026

Test Coverage Report (Windows)

Metric Value
Current Coverage 72.93%
Base Branch Coverage 72.49%
Delta 0.44% ✅

Coverage increased! Great work!

@karthiknadig karthiknadig force-pushed the test/issue-389-pyenv branch from 6936d56 to e495eb0 Compare April 23, 2026 15:48
@karthiknadig karthiknadig marked this pull request as ready for review April 23, 2026 16:33
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.

Coverage: ranked plan to move workspace coverage toward 90%

2 participants