Skip to content

tox.ini: Tweak test discovery #297

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 2, 2014
Merged

tox.ini: Tweak test discovery #297

merged 1 commit into from
Nov 2, 2014

Conversation

msabramo
Copy link
Contributor

@msabramo msabramo commented Nov 2, 2014

Replace tests/ in addopts with norecursedirs

Better to filter than to add in a default, so as not to force folks to always run all the tests.

This is the same as #296, but applied to the 1.0-alpha branch.

@msabramo
tox.ini: Tweak test discovery
9db08ca 
Replace `tests/` in `addopts` with `norecursedirs`

Better to filter than to add in a default, so as not to force folks to
always run all the tests.
@msabramo
Copy link
Contributor Author

msabramo commented Nov 2, 2014

Proof that it works:

$ tox -e py27 -- -v tests/integration/test_repos_release.py
GLOB sdist-make: /Users/marca/dev/git-repos/github3.py/setup.py
py27 inst-nodeps: /Users/marca/dev/git-repos/github3.py/.tox/dist/github3.py-1.0.0b1.zip
py27 runtests: PYTHONHASHSEED='1019812558'
py27 runtests: commands[0] | py.test -v tests/integration/test_repos_release.py
============================================================================= test session starts ==============================================================================
platform darwin -- Python 2.7.6 -- py-1.4.26 -- pytest-2.6.4
collected 6 items

tests/integration/test_repos_release.py ......

=========================================================================== 6 passed in 0.35 seconds ===========================================================================
___________________________________________________________________________________ summary ____________________________________________________________________________________
  py27: commands succeeded
  congratulations :)

$ tox -e py27
GLOB sdist-make: /Users/marca/dev/git-repos/github3.py/setup.py
py27 inst-nodeps: /Users/marca/dev/git-repos/github3.py/.tox/dist/github3.py-0.9.2.zip
py27 runtests: PYTHONHASHSEED='2676834291'
py27 runtests: commands[0] | py.test
..............................................x...............................................................................................................................................................................................................................................................................................................................................................................................................................................................
493 passed, 1 xfailed in 3.85 seconds
___________________________________________________________________________________ summary ____________________________________________________________________________________
  py27: commands succeeded
  congratulations :)

sigmavirus24 added a commit that referenced this pull request Nov 2, 2014
@sigmavirus24
Merge pull request #297 from msabramo/1.0-alpha__tox.ini_take_tests_o…
c8c9d06 
…ut_of_pytest_addopts

tox.ini: Tweak test discovery
@sigmavirus24 sigmavirus24 merged commit c8c9d06 into sigmavirus24:1.0-alpha Nov 2, 2014
@msabramo
Copy link
Contributor Author

msabramo commented Nov 2, 2014

Cool. Love it! Want to merge #296? Although maybe it doesn't matter if you're going to merge 1.0-alpha to develop soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@msabramo @sigmavirus24