Skip to content

Enable parallel test execution in CI with -n auto#359

Merged
Borda merged 3 commits intotesting-improvementsfrom
copilot/sub-pr-349-again
Mar 6, 2026
Merged

Enable parallel test execution in CI with -n auto#359
Borda merged 3 commits intotesting-improvementsfrom
copilot/sub-pr-349-again

Conversation

Copy link
Contributor

Copilot AI commented Mar 6, 2026

CI was running all pytest commands serially despite pytest-xdist already being in tests/requirements.txt and isolation fixtures being in place in conftest.py.

Changes

  • .github/workflows/ci-test.yml: Added -n auto to all five backend pytest commands (local, mongodb, postgres, redis, s3)
  • tests/README.md: Updated CI/CD Integration section to reflect parallel execution; removed the stale note claiming local tests don't use xdist in CI

Why it's safe

The existing conftest.py fixtures already handle xdist isolation:

  • isolated_cache_directory — gives each worker its own cache dir for pickle/maxage tests
  • inject_worker_schema_for_sql_tests — creates per-worker PostgreSQL schemas for SQL tests
  • MongoDB, Redis, and S3 backends are naturally isolated per cached function namespace

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@Borda Borda marked this pull request as ready for review March 6, 2026 08:57
@Borda Borda requested a review from shaypal5 as a code owner March 6, 2026 08:57
Copilot AI review requested due to automatic review settings March 6, 2026 08:57
@Borda Borda marked this pull request as draft March 6, 2026 08:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR is scoped under the ongoing effort to support parallel test execution, but the current diff only applies minor formatting adjustments within the parallel-test fixtures.

Changes:

  • Reformat urlunparse(...) calls in the SQL worker schema injection and schema cleanup fixtures.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Borda <6035284+Borda@users.noreply.github.com>
Copilot AI changed the title [WIP] Add parallel testing for faster local testing Enable parallel test execution in CI with -n auto Mar 6, 2026
Copilot finished work on behalf of Borda March 6, 2026 09:00
@Borda Borda marked this pull request as ready for review March 6, 2026 09:08
@Borda Borda merged commit 441ad80 into testing-improvements Mar 6, 2026
@Borda Borda deleted the copilot/sub-pr-349-again branch March 6, 2026 09:11
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.

3 participants