Code review comment for lp:~adeuring/launchpad/bug-594247-unittests-for-searchtasks-3

Revision history for this message
Gavin Panella (allenap) wrote :

Cool. Only one very trivial comment.

[1]

+ assert(bug.default_bugtask.target == bugtarget)

This works, but it's misleading because assert is a statement not a
function. Actually... did you mean to remove it before landing? If
not, perhaps it should be changed to an assertEqual() call.

review: Approve

« Back to merge proposal