Skip to content

Let base_cli testing write manifest fixtures#962

Merged
codeforester merged 1 commit into
mainfrom
enhancement/942-20260622-testing-manifest-fixture
Jun 22, 2026
Merged

Let base_cli testing write manifest fixtures#962
codeforester merged 1 commit into
mainfrom
enhancement/942-20260622-testing-manifest-fixture

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • Add keyword-only manifest={...} support to base_cli.testing.invoke.
  • Write base_manifest.yaml into cwd before invoking project-aware commands.
  • Document the fixture pattern in the base_cli README.

Validation

  • PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest lib/python/base_cli/tests/test_testing.py -q
  • PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest lib/python/base_cli/tests -q
  • PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pylint --rcfile=.pylintrc lib/python/base_cli/testing.py lib/python/base_cli/tests/test_testing.py
  • git diff --check

Project Fields

  • Priority: P2
  • Size: S
  • Area: Python
  • Initiative: Adoption Polish

Fixes #942

@codeforester codeforester requested a review from a team as a code owner June 22, 2026 06:43
@codeforester codeforester merged commit e65537d into main Jun 22, 2026
18 checks passed
@codeforester codeforester deleted the enhancement/942-20260622-testing-manifest-fixture branch June 22, 2026 06:45
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.

enhancement: let base_cli.testing.invoke write fixture manifests

1 participant