Skip to content

test(dev): cover linux lab prerequisite paths - #2057

Merged
codeforester merged 1 commit into
mainfrom
ci/2051-20260902-cover-installed-and-failure-paths-for-the-linux-lab-multipas
Sep 2, 2026
Merged

test(dev): cover linux lab prerequisite paths#2057
codeforester merged 1 commit into
mainfrom
ci/2051-20260902-cover-installed-and-failure-paths-for-the-linux-lab-multipas

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • Cover Multipass discovery, timeout, nonzero stdout/stderr diagnostics, and successful version parsing.
  • Exercise Linux-lab setup when Multipass is already installed, Homebrew is absent, delegated installation succeeds, or installation raises ArtifactError.
  • Assert stable BASE-D108, exit codes, messages, and actionable fixes without installing Multipass or creating VMs.

linux_lab.py now has 100% statement and branch coverage in its focused suite (issue targets: 85%/75%).

Issue

Fixes #2051

Validation

  • Focused Linux-lab suite (12 passed, 6 subtests; 100% statements and branches)
  • Pylint on the touched test file
  • Full Python coverage suite and ratchet (1,115 passed, 327 subtests; 90.24% statements, 80.26% branches, 87.79% combined)
  • ./bin/base-test with supported sibling source overrides (1,115 Python tests, 327 subtests, and 899 BATS tests passed)

Demo Impact

None. Test-only hardening; no user-facing behavior changes.

Notes

All executable lookup, subprocess, and Homebrew behavior is mocked. Documentation and AI context are not applicable because production behavior is unchanged.

Checklist

  • Branch name follows <category>/<issue>-<YYYYMMDD>-<slug>, and the prefix matches the issue's single standard category label.
  • PR is scoped to one issue, unless a documented multi-issue exception applies.
  • PR body explains what changed and how it was validated.
  • Validation commands were run from the current checkout or worktree, or unavailable checks are explained.
  • Relevant BATS and Python tests pass.
  • Bug fixes include regression proof or a documented reproduction when practical.
  • Documentation is updated when behavior or user-facing commands change.
  • AI context is updated in .ai-context/, or the PR body explains why it is not applicable.
  • PR includes Fixes #<issue> or Closes #<issue> when it should close the issue.
  • Demo Impact is meaningful for needs-demo work, or explicitly says None.

@codeforester
codeforester requested a review from a team as a code owner September 2, 2026 10:44
@codeforester
codeforester merged commit f00a2b0 into main Sep 2, 2026
19 checks passed
@codeforester
codeforester deleted the ci/2051-20260902-cover-installed-and-failure-paths-for-the-linux-lab-multipas branch September 2, 2026 10:54
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.

Cover installed and failure paths for the linux-lab Multipass profile

1 participant