Skip to content

Cover base_dev artifact checks#316

Merged
codeforester merged 1 commit into
masterfrom
enhancement/300-20260530-base-dev-tests
May 30, 2026
Merged

Cover base_dev artifact checks#316
codeforester merged 1 commit into
masterfrom
enhancement/300-20260530-base-dev-tests

Conversation

@codeforester
Copy link
Copy Markdown
Owner

Summary

  • Add direct unit coverage for base_dev Homebrew artifact checks across installed, missing, missing Homebrew, unsupported version, and unsupported manager states.
  • Add non-dry-run setup_dev_artifacts coverage for successful reconciliation and per-artifact failure reporting.
  • Add GitHub CLI auth coverage for missing gh, unauthenticated gh, and authenticated gh, and return structured findings when brew or gh are missing from PATH.

Validation

  • env PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest cli/python/base_dev/tests/test_engine.py
  • env -u BASE_HOME HOME=/private/tmp/base-review-home BASE_TEST_PYTHON=/Users/rameshhp/.base.d/base/.venv/bin/python bin/base-test
  • git diff --check

Closes #300

@codeforester codeforester force-pushed the enhancement/300-20260530-base-dev-tests branch from 6d37261 to 89cf7e9 Compare May 30, 2026 22:41
@codeforester codeforester merged commit b5fa554 into master May 30, 2026
16 checks passed
@codeforester codeforester deleted the enhancement/300-20260530-base-dev-tests branch May 30, 2026 22:42
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.

Add unit tests for check_homebrew_artifact and setup_dev_artifacts in base_dev

1 participant