Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pytest-xdist #464

Merged
merged 9 commits into from
Sep 19, 2024
Merged

Add pytest-xdist #464

merged 9 commits into from
Sep 19, 2024

Conversation

Zeitsperre
Copy link
Contributor

@Zeitsperre Zeitsperre commented Sep 19, 2024

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes #xyz
  • (If applicable) Documentation has been added / updated (for bug fixes / features).
  • (If applicable) Tests have been added.
  • This PR does not seem to break the templates.
  • CHANGELOG.rst has been updated (with summary of main changes).
    • Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

  • Adjusts the code base to potentially benefit from pytest-xdist
  • Removes several old tox-conda configurations (sadly, the library is no longer maintained).

Does this PR introduce a breaking change?

pytest-xdist is now a required development dependency, even if users don't use it.

Other information:

https://pytest-xdist.readthedocs.io/en/stable/

pytest-xdist is a tool for distributing tests across workers. Assuming that the setup is not too heavy, this greatly reduces the amount of time that pytest needs to run tests. The testing code base is now free of potential data races, so there should be no issues.

pytest-xdist is disabled by default as the spin-up time doesn't merit using it yet, but once tests become more numerous, it may be worthwhile to enable it.

@Zeitsperre Zeitsperre changed the title Faster testing Add pyest-xdist Sep 19, 2024
@Zeitsperre Zeitsperre changed the title Add pyest-xdist Add pytest-xdist Sep 19, 2024
Base automatically changed from use-helper-bot to main September 19, 2024 18:59
@Zeitsperre Zeitsperre merged commit 46a27c9 into main Sep 19, 2024
16 of 17 checks passed
@Zeitsperre Zeitsperre deleted the faster-testing branch September 19, 2024 19:56
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