Issue 4014 - Publish python-abp on PyPI

tox.ini

Index: tox.ini
===================================================================
--- a/tox.ini
+++ b/tox.ini
@@ -1,25 +1,21 @@
 [tox]
-envlist = py27,py35,py36,pypy,coverage,flake8
+envlist = py27,py35,py36,pypy,flake8
 
 [flake8]
 ignore = D107, N801, W503
 
 [testenv]
 deps =
-    pytest
-    mock
-commands =
-    py.test tests
-
-[testenv:coverage]
-deps =
     pytest-cov
     mock
 commands =
-    py.test --cov=abp --cov-report=term-missing tests
+    # For accurate coverage reports, we write a `pragma: no py{}` instruction
+    #   containing the current Python version to a .coveragerc file

[Sebastian Noack, 2019/01/03 05:22:03]

Nit: The additional trailing spaces in the comment here seem inconsistent.
(FWIW, IMO this comment is redundant anyway.)

[rhowell, 2019/01/03 21:32:29]

I'm guessing you mean the leading spaces here? Yeah, I can remove them for
consistency. This comment may help save someone's time in the future trying to
figure out why the line is here, but I don't feel too strongly about it either
way.

[Sebastian Noack, 2019/01/03 21:44:23]

Sorry, I meant leading spaces of course.
This comment just reiterates the code in the line below. In fact, it takes me
more time to comprehend this comment than the code it describes. IMO, this is
equivalent to adding a comment like "add one to the variable i" above "i += 1".
But I don't care enough to keep arguing about this.

+    python -c 'import sys; open(".coveragerc", "w").write("[report]\nexclude_lines = pragma: no py\{\} cover".format(sys.version_info[0]))'
+    py.test --cov=abp --cov-report=term-missing --cov-fail-under=100 tests
 
-# We have to install our package in development mode
-# so that pytest-cov can find the .coverage file.
+# We have to install our package in development mode so that pytest-cov can
+# find the .coverage and .coveragerc files.
 usedevelop = true
 
 [testenv:flake8]