Skip to content
This repository has been archived by the owner on Mar 29, 2022. It is now read-only.

Commit

Permalink
Use pytest --no-cov instead of tools/pytest_all.sh
Browse files Browse the repository at this point in the history
The --no-cov option is added in pytest-cov==2.4.0,
to which I just upgraded.
  • Loading branch information
vfaronov committed Feb 28, 2017
1 parent 06222a5 commit 9459b7e
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 11 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ script:
# Basic stuff.
- python setup.py check --strict --restructuredtext
- tools/pylint_all.sh -j 2
- tools/pytest_all.sh
- pytest

# Build the files to be uploaded to pythonhosted.org.
- tools/build_extra_doc.sh
Expand Down
4 changes: 2 additions & 2 deletions HACKING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ but rather to ensure that no line has been *forgotten*.
Feel free to add ``pragma: no cover`` to code
that would be hard to cover with a natural, functional test.

To run tests with coverage checks locally, use ``tools/pytest_all.sh``.
Use ``pytest --no-cov`` to skip coverage checks when iterating on some code.


Typical workflows
Expand Down Expand Up @@ -148,7 +148,7 @@ Adding a notice
#. If necessary, add more test cases: ``1679_2``, and so on.
#. Run your tests and make sure they fail as expected::

$ pytest -k1679
$ pytest --no-cov -k1679

#. Write the actual checks logic.
Usually it goes into one of the four big functions described above,
Expand Down
2 changes: 1 addition & 1 deletion pytest.ini
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
[pytest]
addopts = --doctest-modules --ignore=setup.py --ignore=doc/
addopts = --doctest-modules --ignore=setup.py --ignore=doc/ --cov=httpolice/ --cov=test/ --cov-fail-under=100 --cov-report=html
7 changes: 0 additions & 7 deletions tools/pytest_all.sh

This file was deleted.

0 comments on commit 9459b7e

Please sign in to comment.