Skip to content

test: add shared pytest manifest fixtures - #1967

Merged
codeforester merged 2 commits into
mainfrom
enhancement/1912-20260819-introduce-shared-pytest-fixtures-for-manifest-project-root-t
Aug 19, 2026
Merged

test: add shared pytest manifest fixtures#1967
codeforester merged 2 commits into
mainfrom
enhancement/1912-20260819-introduce-shared-pytest-fixtures-for-manifest-project-root-t

Conversation

@codeforester

Copy link
Copy Markdown
Collaborator

Summary

  • add repository-wide pytest fixtures for isolated project roots and common manifest variants
  • migrate duplicated manifest setup in base_trust and base_projects engine tests
  • add direct fixture smoke coverage

This is the first incremental slice of #1912. The remaining base_pr_policy and base_release migrations are intentionally left for follow-up work.

Validation

  • BASE_CLI_SOURCE_DIR=/Users/rameshhp/work/base-cli/lib/python PYTHONPATH=/Users/rameshhp/work/base-cli/lib/python:lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest -q tests/test_shared_fixtures.py cli/python/base_trust/tests/test_engine.py cli/python/base_projects/tests/test_engine.py
  • TZ=UTC BASE_BASH_LIBS_DIR=/Users/rameshhp/work/base-bash-libs/lib/bash BASE_CLI_SOURCE_DIR=/Users/rameshhp/work/base-cli/lib/python env -u BASE_HOME ./bin/base-test
  • git diff --check

The default local timezone is Asia/Kolkata, while the existing host-timezone history test assumes UTC; the full harness passes with TZ=UTC.

AI context

No update needed: this is test-only infrastructure with no product behavior or architecture change.

Demo Impact

None.

Refactor the duplicated Base manifest setup in the trust and projects test suites onto shared pytest fixtures.\n\nFixes #1912
@codeforester
codeforester requested a review from a team as a code owner August 19, 2026 10:33
@codeforester
codeforester merged commit 3291e48 into main Aug 19, 2026
28 checks passed
@codeforester
codeforester deleted the enhancement/1912-20260819-introduce-shared-pytest-fixtures-for-manifest-project-root-t branch August 19, 2026 11:22
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.

1 participant