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

Speed up test suite #637

Merged
merged 12 commits into from
May 1, 2018
Merged

Speed up test suite #637

merged 12 commits into from
May 1, 2018

Conversation

pv
Copy link
Collaborator

@pv pv commented Apr 30, 2018

Run tests in parallel with pytest-xdist.
Fix timing issues that caused problems with pytest-xdist.
Use --bench= in tests to avoid running the whole slow benchmark suite.
Prefer close+join when using multiprocessing, to avoid some issues on python2 on windows.
Fix test_web:basic_html fixture to work with pytest-xdist.

Add --offline option for running tests without pip/conda downloading.
Pre-download necessary packages before running tests, which also mitigates some issues with conda's parallel execution safety.

@pv pv force-pushed the ci-parallel branch 6 times, most recently from a2219b5 to 965c754 Compare April 30, 2018 22:15
@pv pv changed the title CI: use pytest-xdist Speed up test suite Apr 30, 2018
@pv pv force-pushed the ci-parallel branch 19 times, most recently from 1f946ed to a9e53d1 Compare May 1, 2018 20:31
pv added 4 commits May 1, 2018 23:25
This is more robust vs. PYTHONPATH containing '' (current directory),
which occurs when running under pytest-xdist.

Running in a temporary directory is also consistent with how benchmarks
themselves are run.
Disable pytest-timeout, as it appears incompatible with xdist
pv added 8 commits May 1, 2018 23:25
…in thread

Setting signal handlers works only in the main thread.

Typically, asv is single-threaded, but when running tests, pytest-xdist
doesn't seem to always guarantee the tests are run in the main thread.
This may avoid sending signals to workers, which can bump into
multiprocessing bugs on python 2
Use only the currently installed Python version, plus a second version.
@pv pv merged commit e3a6d72 into airspeed-velocity:master May 1, 2018
@pv pv deleted the ci-parallel branch August 19, 2018 20:07
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.

None yet

1 participant