Merge pull request #1779 from RonnyPfannschmidt/deselect-no-reason

terminal: dont pretend to know the deselection reason
This commit is contained in:
Bruno Oliveira 2016-08-02 08:38:17 -03:00 committed by GitHub
commit ea6191a0cd
3 changed files with 8 additions and 12 deletions

View File

@ -259,7 +259,10 @@ time or change existing behaviors in order to make them less surprising/more use
* Add stderr write for ``pytest.exit(msg)`` during startup. Previously the message was never shown.
Thanks `@BeyondEvil`_ for reporting `#1210`_. Thanks to `@JonathonSonesen`_ and
`@tomviner`_ for PR.
`@tomviner`_ for the PR.
* fix `#1372`_ no longer display the incorrect test deselection reason,
thanks `@ronnypfannschmidt`_ for the PR.
*
@ -311,6 +314,7 @@ time or change existing behaviors in order to make them less surprising/more use
.. _#1210: https://github.com/pytest-dev/pytest/issues/1210
.. _#1235: https://github.com/pytest-dev/pytest/issues/1235
.. _#1351: https://github.com/pytest-dev/pytest/issues/1351
.. _#1372: https://github.com/pytest-dev/pytest/issues/1372
.. _#1421: https://github.com/pytest-dev/pytest/issues/1421
.. _#1426: https://github.com/pytest-dev/pytest/issues/1426
.. _#1428: https://github.com/pytest-dev/pytest/pull/1428

View File

@ -512,16 +512,8 @@ class TerminalReporter:
def summary_deselected(self):
if 'deselected' in self.stats:
l = []
k = self.config.option.keyword
if k:
l.append("-k%s" % k)
m = self.config.option.markexpr
if m:
l.append("-m %r" % m)
if l:
self.write_sep("=", "%d tests deselected by %r" % (
len(self.stats['deselected']), " ".join(l)), bold=True)
self.write_sep("=", "%d tests deselected" % (
len(self.stats['deselected'])), bold=True)
def repr_pythonversion(v=None):
if v is None:

View File

@ -384,7 +384,7 @@ class TestTerminalFunctional:
result = testdir.runpytest("-k", "test_two:", testpath)
result.stdout.fnmatch_lines([
"*test_deselected.py ..",
"=* 1 test*deselected by*test_two:*=",
"=* 1 test*deselected *=",
])
assert result.ret == 0