Code review comment for lp:~salgado/launchpad/refactor-blueprints-tests

Revision history for this message
Guilherme Salgado (salgado) wrote :

On Mon, 2010-11-29 at 16:59 +0000, Henning Eggers wrote:
> Review: Approve code
> Thanks for this clean-up. Please consider using assertContentEqual instead of "sorted" in assertNamesOfSpecificationsAre.

Oops, I took your r=me on IRC as "I've no remarks, go ahead and land
it", so I didn't do that change. But I'll make sure to include it in my
next branch.

Thanks for the review!

« Back to merge proposal