Merge lp:~mikemc/ubuntuone-client/fix-dummy-sync-menu into lp:ubuntuone-client
Proposed by
Mike McCracken
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Mike McCracken | ||||
Approved revision: | 1327 | ||||
Merged at revision: | 1327 | ||||
Proposed branch: | lp:~mikemc/ubuntuone-client/fix-dummy-sync-menu | ||||
Merge into: | lp:ubuntuone-client | ||||
Prerequisite: | lp:~diegosarmentero/ubuntuone-client/ubuntuone-client-timer | ||||
Diff against target: |
72 lines (+55/-1) 2 files modified
tests/platform/sync_menu/test_common.py (+49/-0) ubuntuone/platform/sync_menu/common.py (+6/-1) |
||||
To merge this branch: | bzr merge lp:~mikemc/ubuntuone-client/fix-dummy-sync-menu | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brian Curtin (community) | Approve | ||
Diego Sarmentero (community) | Approve | ||
Review via email: mp+126766@code.launchpad.net |
This proposal supersedes a proposal from 2012-09-24.
Commit message
- Fix dummy sync menu implementation for windows and darwin. (LP: #1055840)
Description of the change
- Fix dummy sync menu implementation for windows and darwin. (LP: #1055840)
Added a test to check that the dummy class has the same API, fixed missing function.
Tests pass on darwin and linux (precise).
(On darwin, I only ran the specific tests that changed, since trunk tests have never worked.)
With this fix, an app built from trunk runs correctly on darwin again.
To post a comment you must log in.
+1