Support for running under setup.py/manage.py and travis setup. #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 3 commits
February 19, 2015 13:15
- .travis.yml to run CI tests on travis - .gitignore: add vim tmp files, eggs and others installed by setup.py - nosepipe.py: ProcessIsolationPlugin.configure, if coverage plugin isn't installed don't try to configure for it - setup.py: remove nose==dev dependency as nose-dev seems to distribute 'dev' with a version number. Testing with dev is done by pre-installing a dev version in the travis setup script.
- nosepipe.py: If we're not running as `nosetests` change the sub-process to run as nosetests. Other non-nosetests scripts like setup.py/manage.py obscure the output of tests such that nosetests can't parse the output correctly. - add test-support/django sample project for testing django-nose
|
This looks good to me. Thanks! |
dmccombs
pushed a commit
that referenced
this pull request
Feb 24, 2015
Support for running under setup.py/manage.py and travis setup.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm closing out #7 and moving the commits to this PR. I basically rearranged the commits to make more sense. The only additional changes on this PR are fully working tests for running under django_nose and as
setup.py nosetests.The nosepipe plugin wasn't working when running under setup.py or manage.py (django with django_nose). It would fail because the subprocesses would be run as setup.py/manage.py instead of nosetests processes. The output from the sub-process was confusing nosetests. So, when running under something other than nosetests, nosetests needs to make sure the sub-process command is nosetests... instead of setup.py nosetests...
I also had dependency problems with nose==dev in setup.py. Getting the following error:
This seems to be due to the dependency being ==dev AND >=0.1.0 which doesn't seem to be able to be true. I removed the ==dev but added a travis test case to specifically use the dev version of nose.
Finally, I added a .travis.yml which will make it simple to add automated CI jobs using http://travis-ci.org if you 'd like. See the tests running from my fork here:
https://travis-ci.org/mdprewitt/nosepipe