Skip to content

Configure Tox and Use It to Drive Travis #1173

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

Merged
merged 9 commits into from
Aug 28, 2013
Merged

Conversation

dstufft
Copy link
Member

@dstufft dstufft commented Aug 28, 2013

  • Configures Tox to correctly run all of the items in our test matrix
  • Updates .travis.ymlso it dispatches the testing logic to Tox
  • Work around the bug where pytest-xdist doesn't detect if there was an error while collecting tests by making an explicit test collection test
  • Test that our documentation builds correctly and without warnings

Essentially I got tired of remembering that I needed to run py.test instead of py.test -n 8 on Python 3.2 and that I needed to run PYTHONHASHSEED=0 py.test -n 8 on Python 3.3. This makes it so the command is always tox -e pyNN. Additionally it consolidates the testing configuration in one place (tox.ini) so that we only need to maintain one configuration. Finally we make sure that our documentation always builds because we want good documentation.

* Uses parallel testing where it's able to
* Sets PYTHONHASHSEED=0 Where it's required
Using tox to drive the tests instead of travis itself means that
we only need to configure how to run tests in one location. We
use tox for that location because tox can run on developer machines
as well as on travis so it has the best reusability.
A trailing slash in a directory name is essentially a noop unless
the directory name itself contains a trailing slash.
We need the docs directory for ``python setup.py``
This is to workaround pytest issues pypa#149 / pypa#180 where distributed
testing doesn't error out if it fails to collect tests.
@qwcode
Copy link
Contributor

qwcode commented Aug 28, 2013

merge 👍

dstufft added a commit that referenced this pull request Aug 28, 2013
Configure Tox and Use It to Drive Travis
@dstufft dstufft merged commit b4981da into pypa:develop Aug 28, 2013
@dstufft dstufft deleted the use-tox branch August 28, 2013 16:08
@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 5, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants