Merge lp:~charlesk/keeper/upate-status-properties-pt-1 into lp:keeper
Proposed by
Charles Kerr
Status: | Merged |
---|---|
Merge reported by: | Charles Kerr |
Merged at revision: | not available |
Proposed branch: | lp:~charlesk/keeper/upate-status-properties-pt-1 |
Merge into: | lp:keeper |
Diff against target: |
373 lines (+71/-77) 7 files modified
src/qdbus-stubs/com.canonical.keeper.User.xml (+13/-15) src/service/backup-choices.cpp (+8/-18) tests/com_canonical_keeper.py (+42/-29) tests/dbusmock/keeper-template-test.cpp (+0/-3) tests/unit/metadata-providers/user-dirs-test.cpp (+1/-3) tests/unit/tar/keeper-tar-create-test.cpp (+0/-1) tests/utils/keeper-dbusmock-fixture.h (+7/-8) |
To merge this branch: | bzr merge lp:~charlesk/keeper/upate-status-properties-pt-1 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Xavi Garcia (community) | Approve | ||
Unity API Team | Pending | ||
Review via email: mp+301424@code.launchpad.net |
Commit message
Update what properties are provided by the com.canonical.
Description of the change
Update what properties are provided by the com.canonical.
The low-hanging fruit from https:/
We don't have anything implementing the state property maps in the service itself, so the card can't be completed yet. But now I can at least test the dbusmock template for an error state when the helper errors out :)
To post a comment you must log in.
I'm not sure if after landing this I will have conflicts with my branch: /code.launchpad .net/~xavi- garcia- mena/keeper/ using-cwd- and-user- startbackup/ +merge/ 301496
https:/
It looks good and your branch was first, so... let's merge it. :)