-
Notifications
You must be signed in to change notification settings - Fork 9
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
Streamline fixtures, use pytest-xdist, drop Python3.8 #345
Conversation
@Zeitsperre looks good 👍 ... but probably need some support in adapting PR #342. |
@cehbrecht There's a ton of work to do. I'm completely tearing the testing apart. I might need some help to move |
…tebooks to use new system
0993a40
to
b9ea150
Compare
There are a handful of tests failing that I can't seem to make sense of. It may be due to breaking changes in more recent versions of Within some of the internals of clisops (e.g. One thing for certain is that this affects tests that open paths ending in wildcards (i.e. |
Pull Request Test Coverage Report for Build 11163828659Details
💛 - Coveralls |
@cehbrecht Waiting on your review here. Lots of changes, but things are much smoother here. |
@Zeitsperre thanks for the update. I'm fine with the changes. Moving to |
Pull Request Checklist:
What kind of change does this PR introduce?:
xclim-testdata
repo to prevent breaking changes there from causing issues here.pytest-xdist
to distribute tests across multiple workerspooch
).numpy.random.RandomState
call.pip
,tox
, and testing data to significantly speed up subsequent builds.Does this PR introduce a breaking change?:
Yes. Testing now requires the
pytest-xdist
plugin (has been added topyproject.toml
andenvironment.yml
). By default, runningpytest
will only use one worker, buttox
will use up to eight workers.Other information:
https://pytest-xdist.readthedocs.io/en/stable/