Skip to content

Split base_setup tests by feature area#305

Merged
codeforester merged 1 commit into
masterfrom
enhancement/289-20260530-split-base-setup-tests
May 30, 2026
Merged

Split base_setup tests by feature area#305
codeforester merged 1 commit into
masterfrom
enhancement/289-20260530-split-base-setup-tests

Conversation

@codeforester
Copy link
Copy Markdown
Owner

Summary

  • Split the oversized base_setup test module into focused feature-area test files.
  • Added shared base_setup.tests.helpers for common test utilities.
  • Removed the broad too-many-lines / too-many-public-methods pylint disable by deleting the monolithic test_engine.py.
  • Updated contributor guidance so larger Python packages can use focused test_*.py modules.

Validation

  • PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest cli/python/base_setup/tests
  • PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pytest
  • git ls-files -z '*.py' | xargs -0 env PYTHONPATH=lib/python:cli/python /Users/rameshhp/.base.d/base/.venv/bin/python -m pylint --rcfile=.pylintrc
  • git diff --check

Closes #289

@codeforester codeforester merged commit 1e0aa08 into master May 30, 2026
16 checks passed
@codeforester codeforester deleted the enhancement/289-20260530-split-base-setup-tests branch May 30, 2026 17:40
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.

Split oversized base_setup Python test suite by feature area

1 participant