Code review comment for lp:~stevenk/launchpad/uploaded-packages-wrong-link

Revision history for this message
Robert Collins (lifeless) wrote :

My only comment is that this:

35 + for suffix in ('+bugs', '+questions'):
36 + self.assertIn('<a href="%s%s">' % (expected_base, suffix), html)

is no shorter, and may well be harder to debug that two separate assertions.

« Back to merge proposal