Code review comment for lp:~mbp/bzr/scripts

Revision history for this message
Vincent Ladeuil (vila) wrote :

I agree, having the test writer think about -q and '...' sounds like a good plan.
The only drawback I could think of is that it will require a bit more expertise
than a mere copy/paste. No big loss here, I'm still waiting for bug reporters to
do that anyway :-D

127 - test_case.assertEqualDiff(expected, actual)
128 + test_case.assertEquals(expected, actual)

Why ??? EqualDiff provides valuable info, please don't remove that !

review: Needs Fixing

« Back to merge proposal