Code review comment for lp:~ack/landscape-client/package-description-decode-fix

Revision history for this message
Alberto Donato (ack) wrote :

> The change looks good, but your test doesn't seem to be calling
> build_skeleton(). And quick coverage run tells me your code is not exercised
> through test.

Good catch, I think something got messed up while I was experimenting with the test.
It's fixed now. I also dropped the self.build_skeleton() helper in the test class, since it's a usless indirection.
Thanks!

« Back to merge proposal