Add system pytest and use it instead #1422
Merged
+3
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 Description
Fix failing unit tests in GitHub CI by installing
pytestinto the system environment inside the test container. The CI job spins up the container and uses the container's system Python (not the repository venv), so installingpytestsystem-wide ensures the test runner is available and the job behaves the same as the devcontainer.🔍 Related Issues
📋 Type of Change
Please check the relevant option(s):
📷 Screenshots or Logs (if applicable)
See changes made in #1420
🧪 Testing Steps
pytestinside the container (e.g.,pip install pytestruns in the job log).pytestis invoked from the container system Python and that tests complete successfully.pip install pytest && pytest -qto confirm behavior matches CI.✅ Checklist
🙋 Additional Notes
This change mirrors the devcontainer testing environment and prevents CI from failing when
pytestis not available on the container's system interpreter. Consider pinning thepytestversion in a follow-up to avoid unexpected test runner upgrades.