launchpad-reviewers team mailing list archive

[Michael Nelson <michael.nelson@xxxxxxxxxxxxx>]

Robert,

I removed the useless vars, and updated to use the IsInstance matcher, but although I see HasLength on the testtools docs, I don't see it in the LP code or the version currently used by LP:

http://paste.ubuntu.com/5572967/

so I've gone with assertThat(len(urls), Equals(1)) instead.

Thanks!
-- 
https://code.launchpad.net/~michael.nelson/launchpad/include-binary-size/+merge/150831
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~michael.nelson/launchpad/include-binary-size into lp:launchpad.