Skip to content

Restore pytest-asyncio to base test requirements#355

Merged
shaypal5 merged 2 commits intotesting-improvementsfrom
copilot/sub-pr-349
Feb 28, 2026
Merged

Restore pytest-asyncio to base test requirements#355
shaypal5 merged 2 commits intotesting-improvementsfrom
copilot/sub-pr-349

Conversation

Copy link
Contributor

Copilot AI commented Feb 28, 2026

When pytest-xdist and pytest-rerunfailures were added to tests/requirements.txt, pytest-asyncio was accidentally dropped. Since tests/sql_tests/conftest.py imports pytest_asyncio at the top level, this caused collection-time failures across all backend jobs that don't install requirements_postgres.txt:

ERROR tests/sql_tests - ModuleNotFoundError: No module named 'pytest_asyncio'
Interrupted: 1 error during collection

Fix

  • Adds pytest-asyncio back to tests/requirements.txt alongside the newly added parallel/retry dependencies

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI self-assigned this Feb 28, 2026
Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix precommit-ci errors in testing setup Restore pytest-asyncio to base test requirements Feb 28, 2026
@shaypal5 shaypal5 marked this pull request as ready for review February 28, 2026 22:20
@shaypal5 shaypal5 merged commit 576bb9e into testing-improvements Feb 28, 2026
1 check passed
@shaypal5 shaypal5 deleted the copilot/sub-pr-349 branch February 28, 2026 22:20
shaypal5 added a commit that referenced this pull request Mar 1, 2026
* Initial plan

* Add pytest-asyncio back to tests/requirements.txt

Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shaypal5 <917954+shaypal5@users.noreply.github.com>
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.

2 participants