[Merge] lp:~evfool/update-manager/fix665173 into lp:update-manager
Barry Warsaw
barry at canonical.com
Fri Apr 1 16:56:11 UTC 2011
> I have tested it manually on a source not supporting changelogs, and the
> message is only shown once, as expected. Do you mean a unit-test? I think I
> could do that with a hard-coded launchpad changelog URL, that reports that
> changelogs are not supported. I'll check, I haven't written any Python unit-
> tests so fat, but I assume that the right place for this would be another
> method in the tests/test_changelog.py
I think that would be a good place for it. There should be lots of examples in the package to cargo cult from :). Give it a shot! Thanks.
--
https://code.launchpad.net/~evfool/update-manager/fix665173/+merge/55663
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~evfool/update-manager/fix665173 into lp:update-manager.
More information about the Ubuntu-reviews
mailing list