[issue15802] Nonsensical test for mailbox

Chris Jerdonek report at bugs.python.org
Wed Aug 29 12:39:06 CEST 2012


Chris Jerdonek added the comment:

> In other words, (int(groups[0]), int(groups[1])) >= (int(previous_groups[0]), int(previous_groups[1])).

Why not use a single self.assertGreaterEqual() on the pairs (with an appropriate change in the message)?

It currently hand-codes the lexicographic comparison, which seems an unnecessary complication (and may have increased the likelihood of error).

----------
nosy: +cjerdonek

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue15802>
_______________________________________


More information about the Python-bugs-list mailing list