Update test runner to use pytest#495
Conversation
|
@jbrockmendel @mariocj89 I think this is fine, any comments? I'd like to merge ASAP so that everyone else can start using pytest. |
|
Looks great! I'd personally add a -vvvv for pytest to get the history of test run on the builds |
|
@mariocj89 Ah, good call, I didn't notice that that was missing. |
|
What is the difference between |
|
Just realized, it actually changes the way some diffs are reported. -v will add the test name in the reports. having multiple v just increases the level of verbosity, which adds extra information on some assertion failures. Example, if you assert two diffs which are different but big, adding extra verbosity will print the whole dict whilst just -v won't. |
|
OK, we can change it later if |
|
|
||
| script: | ||
| - coverage run --omit=setup.py,dateutil/test/* setup.py test | ||
| - coverage run --omit=setup.py,dateutil/test/* -m pytest -v |
There was a problem hiding this comment.
There is a pytest-cov plugin, not sure to what extent there is a "preferred" usage of that vs this usage.
|
Little late, but LGTM. |
3 participants
This changes the test runner over to pytest.