lp:~tealeg/landscape-client/merge-package-and-hold-handlers

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~landscape/landscape-client/trunk at revision 553

Free Ekanayaka (community): Approve on 2012-04-10

Thomas Herve (community): Approve on 2012-04-10

Diff: 1042 lines (+312/-341)

6 files modified

landscape/__init__.py (+5/-1)
landscape/constants.py (+1/-0)
landscape/package/changer.py (+21/-81)
landscape/package/facade.py (+118/-74)
landscape/package/tests/test_changer.py (+102/-147)
landscape/package/tests/test_facade.py (+65/-38)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #972489: The change-packages handler needs to handle package holds

High

Fix Released

Link a bug report

Related blueprints

577. By Geoff Teale on 2012-04-11

lint

576. By Geoff Teale on 2012-04-11

Bump CLIENT_API to 3.4 and update changelog (From both Free and therve's review).

575. By Geoff Teale on 2012-04-11

Remove unnecessary error case and tests for using holds on a Smart facade. From Free's review.

574. By Geoff Teale on 2012-04-11

Change behaviour when we try to remove a hold on a package the client has never seen and go back to simpler code structure as a result. From Free's review.

573. By Geoff Teale on 2012-04-11

Sanitise and Anglicise hold activity message and method names, following a discussion with Free.

572. By Geoff Teale on 2012-04-11

Bring things back to a known working state.

571. By Geoff Teale on 2012-04-10

Remove else clause (from Free's review)

570. By Geoff Teale on 2012-04-10

Add to test check that holds can be created from two seperate packages with the same version number (from which the hash is calculated). Revert change to use sets for marks as this causes the new test to fail.

569. By Geoff Teale on 2012-04-10

Use a set for marks instead of a list (from therve's review).

568. By Geoff Teale on 2012-04-10

Use list comprehension instead of filter (from therve's review)