Skip to content

Fix tox.ini to allow specifying pytest tests #323

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 2 commits into from
Sep 1, 2017
Merged

Fix tox.ini to allow specifying pytest tests #323

merged 2 commits into from
Sep 1, 2017

Conversation

willkg
Copy link
Contributor

@willkg willkg commented Feb 23, 2017

This fixes tox.ini so that you can pass arguments to py.test.

For example:

    tox -e py34-base html5lib/tests/test_serializer.py

will now only run the serializer tests.

    tox -e py34-base -- -vv

will run with super verbose mode and tons of data will go flying by in your
terminal.

@willkg
Copy link
Contributor Author

willkg commented Feb 23, 2017

This doesn't affect travis, but it does make development iterations much faster.

Another fix we can do in the tox file is to move the flake8 pass to a separate "environment" so that it runs once in a tox run and not once for every environment.

Does that sound helpful? If so, I can make that change and tack it on to this one.

willkg and others added 2 commits February 23, 2017 13:33
This fixes tox.ini so that you can pass arguments to py.test.

For example:

    tox -e py34-base html5lib/tests/test_serializer.py

will now only run the serializer tests.

    tox -e py34-base -- -vv

will run with super verbose mode and tons of data will go flying by in your
terminal.
@willkg willkg requested a review from gsnedders September 1, 2017 16:44
@willkg
Copy link
Contributor Author

willkg commented Sep 1, 2017

This is super minor. It's just an ergonomic issue related to development.

@jgraham jgraham merged commit 590d9a5 into html5lib:master Sep 1, 2017
@willkg willkg deleted the tox-fix branch December 4, 2017 14:03
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