Add dedicated test.yml workflow for PR validation#395
Conversation
|
Merging to
|
Co-authored-by: abhimehro <84992105+abhimehro@users.noreply.github.com>
|
👋 Development Partner is reviewing this PR. Will provide feedback shortly. |
|
👋 Development Partner is reviewing this PR. Will provide feedback shortly. |
There was a problem hiding this comment.
Pull request overview
This pull request adds a dedicated test workflow for automated PR validation, addressing a gap where the comprehensive test suite (18 test files) was not being run on pull requests. Previously, tests only ran via a custom composite action in daily agent workflows, invisible to PR checks.
Changes:
- Creates a new
.github/workflows/test.ymlworkflow that runs on push/PR to main using uv for dependency management and pytest for test execution - Adds a Tests badge to README.md alongside existing CI badges
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.github/workflows/test.yml |
New workflow with uv-based setup and pytest execution for PR validation |
README.md |
Adds Tests workflow badge to display test status |
| run: uv sync --all-extras | ||
|
|
||
| - name: Run tests | ||
| run: uv run pytest tests/ -v --tb=short |
There was a problem hiding this comment.
The test command is missing the -n auto flag for parallel test execution. The existing custom action in .github/actions/daily-perf-improver/build-steps/action.yml uses pytest tests/ -n auto -v to run tests in parallel, and the repository includes pytest-xdist>=3.0.0 in dev dependencies specifically for this purpose. Running tests in parallel will significantly reduce CI execution time, especially as the test suite grows (currently 18 test files).
| run: uv run pytest tests/ -v --tb=short | |
| run: uv run pytest tests/ -n auto -v --tb=short |
No CI workflow ran tests on pull requests — the only automated test execution was buried inside a custom composite action used by daily agent workflows, invisible to PR checks.
Changes
.github/workflows/test.yml(new) — Runs onpush/pull_requesttomainusingastral-sh/setup-uv@v4+uv sync --all-extras+uv run pytest tests/ -v --tb=short. Matches the documented developer workflow. Minimalcontents: readpermissions.README.md— Adds aTestsbadge alongside the existing Sync/Bandit/Codacy badges.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.