[issue18702] Report skipped tests as skipped
Terry J. Reedy
report at bugs.python.org
Sat Aug 17 14:27:06 CEST 2013
Terry J. Reedy added the comment:
You changed "NNTP_CLASS = nntplib.NNTP_SSL", which could potentially fail, to "NNTP_CLASS = getattr(nntplib, 'NNTP_SSL', None)", which cannot fail. Since that was the only thing that previously could fail, the change leaves nothing that can fail, so the test is not a test.
I suggested that you either not add the third param, a default, or that you remove the null test completely. If particular, if the only chunk of code in nntplib that is currently being tested is being executed by some other test, then do the latter.
----------
_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue18702>
_______________________________________
More information about the Python-bugs-list
mailing list