Skip to content
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

Pytest without doctests #198

Closed

Conversation

bsravanin
Copy link

  1. Switching from unittest assertions to basic assert statements.
  2. Removing boilerplate code related to test suites.
  3. Adding a TestCommand to setup.py, to run tests using pytest by default.
  4. Fixing the fabfile changes -- print in dodo.py needs to be imported from future.

Doctests aren't enabled in this because of few minor issues being discussed elsewhere.

file=sys.stderr option of print command needs the print function to be
imported from future.
Tests still run using pytest, by using TestCommand in setup.py.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.24%) to 91.38% when pulling c9d6d6a on bsravanin:pytest_without_doctests into 1fd7c59 on wackou:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.24%) to 91.38% when pulling c9d6d6a on bsravanin:pytest_without_doctests into 1fd7c59 on wackou:master.

@bsravanin
Copy link
Author

Based on my comparison with the the previous build that passed, I suspect the coverage failure has to do with that fact that doctests weren't running in this. I will try to get that patch fixed and include in this pull request.

@bsravanin
Copy link
Author

I got this working along with doctests. Will close this (as I'm not sure how to edit a pull request) and create a new pull request.

@bsravanin bsravanin closed this Apr 21, 2015
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.

None yet

2 participants