Skip to content

Conversation

@harshil21
Copy link
Member

@harshil21 harshil21 commented Oct 4, 2022

Runs the test suite in parallel by using the pytest-xdist plugin. In my case, the test suite went from taking ~14 mins to complete to just 3m 28s!
That's roughly a 75% improvement!

@harshil21 harshil21 added the ⚙️ tests affected functionality: tests label Oct 4, 2022
Copy link
Member

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

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

Sorry for getting back to this late.

The speedup in the CI doesn't seem as big as for you locally, but it's nice nonetheless. Tests seem to run rather stable, too, so let's add it :) Can you also add a note to the contrib guide that tells users how they can use pytest-xdist? Probably also add a warning that this may effect the test results in rare edge cases :D

Copy link
Member

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

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

LGTM 👍 if there's nothing left from your side, I can merge once the tests pass :)

@Bibo-Joshi Bibo-Joshi merged commit 87b2e29 into master Oct 23, 2022
@Bibo-Joshi Bibo-Joshi deleted the faster-tests branch October 23, 2022 12:25
@github-actions github-actions bot locked and limited conversation to collaborators Oct 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⚙️ tests affected functionality: tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants