Merge lp:~renatofilho/buteo-syncfw-qml/rename-uoa-service into lp:buteo-syncfw-qml

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Michael Sheldon
Approved revision: 6
Merged at revision: 6
Proposed branch: lp:~renatofilho/buteo-syncfw-qml/rename-uoa-service
Merge into: lp:buteo-syncfw-qml
Diff against target: 61 lines (+6/-6)
2 files modified
tests/qml/buteo-syncfw.py (+4/-4)
tests/qml/tst_ButeoSyncFW.qml (+2/-2)
To merge this branch: bzr merge lp:~renatofilho/buteo-syncfw-qml/rename-uoa-service
Reviewer Review Type Date Requested Status
Michael Sheldon (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+273541@code.launchpad.net

Commit message

Rename uoa contacts service to avoid conflict with old services.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

Are there any related MPs required for this MP to build/function as expected?
NO

Is your branch in sync with latest trunk?
YES

Did you perform an exploratory manual test run of your code change and any related functionality on device or emulator?
YES

Did you successfully run all tests found in your component's Test Plan on device or emulator?
YES

If you changed the UI, was the change specified/approved by design?
NO UI CHANGE.

If you changed UI labels, did you update the pot file?
NO LABEL CHANGE.

If you changed the packaging (debian), did you add a core-dev as a reviewer to this MP?
NO PACKAGE CHANGED

Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

Looks good

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/qml/buteo-syncfw.py'
--- tests/qml/buteo-syncfw.py 2015-09-01 03:50:01 +0000
+++ tests/qml/buteo-syncfw.py 2015-10-06 14:56:23 +0000
@@ -35,7 +35,7 @@
35"""<?xml version=\"1.0\" encoding=\"UTF-8\"?>35"""<?xml version=\"1.0\" encoding=\"UTF-8\"?>
36<profile type=\"sync\" name=\"test-profile\">36<profile type=\"sync\" name=\"test-profile\">
37 <key value=\"45\" name=\"accountid\"/>37 <key value=\"45\" name=\"accountid\"/>
38 <key value=\"contacts\" name=\"category\"/>38 <key value=\"buteo-contacts\" name=\"category\"/>
39 <key value=\"google.Contacts-\" name=\"displayname\"/>39 <key value=\"google.Contacts-\" name=\"displayname\"/>
40 <key value=\"true\" name=\"enabled\"/>40 <key value=\"true\" name=\"enabled\"/>
41 <key value=\"true\" name=\"hidden\"/>41 <key value=\"true\" name=\"hidden\"/>
@@ -58,7 +58,7 @@
58"""<?xml version=\"1.0\" encoding=\"UTF-8\"?>58"""<?xml version=\"1.0\" encoding=\"UTF-8\"?>
59<profile name=\"63807467\" type=\"sync\">59<profile name=\"63807467\" type=\"sync\">
60 <key value=\"110\" name=\"accountid\"/>60 <key value=\"110\" name=\"accountid\"/>
61 <key value=\"contacts\" name=\"category\"/>61 <key value=\"buteo-contacts\" name=\"category\"/>
62 <key value=\"online\" name=\"destinationtype\"/>62 <key value=\"online\" name=\"destinationtype\"/>
63 <key value=\"google-contacts-renato.teste2@gmail.com\" name=\"displayname\"/>63 <key value=\"google-contacts-renato.teste2@gmail.com\" name=\"displayname\"/>
64 <key value=\"true\" name=\"enabled\"/>64 <key value=\"true\" name=\"enabled\"/>
@@ -81,7 +81,7 @@
81</profile>""",81</profile>""",
82"""<?xml version=\"1.0\" encoding=\"UTF-8\"?>82"""<?xml version=\"1.0\" encoding=\"UTF-8\"?>
83<profile type=\"sync\" name=\"template-profile\">83<profile type=\"sync\" name=\"template-profile\">
84 <key value=\"contacts\" name=\"category\"/>84 <key value=\"buteo-contacts\" name=\"category\"/>
85 <key value=\"false\" name=\"enabled\"/>85 <key value=\"false\" name=\"enabled\"/>
86 <schedule syncconfiguredtime=\"\" interval=\"0\" days=\"\" externalsync=\"true\" time=\"\" enabled=\"false\">86 <schedule syncconfiguredtime=\"\" interval=\"0\" days=\"\" externalsync=\"true\" time=\"\" enabled=\"false\">
87 <rush interval=\"0\" days=\"\" externalsync=\"false\" begin=\"\" enabled=\"false\" end=\"\"/>87 <rush interval=\"0\" days=\"\" externalsync=\"false\" begin=\"\" enabled=\"false\" end=\"\"/>
@@ -105,7 +105,7 @@
105 def syncProfilesByKey(self, key, value):105 def syncProfilesByKey(self, key, value):
106 print ("syncProfilesByKey:", key, value)106 print ("syncProfilesByKey:", key, value)
107 if key == 'category':107 if key == 'category':
108 if value == 'contacts':108 if value == 'buteo-contacts':
109 return [ButeoSyncFw.PROFILES[0], ButeoSyncFw.PROFILES[2], ButeoSyncFw.PROFILES[3]]109 return [ButeoSyncFw.PROFILES[0], ButeoSyncFw.PROFILES[2], ButeoSyncFw.PROFILES[3]]
110 if value == 'calendar':110 if value == 'calendar':
111 return [ButeoSyncFw.PROFILES[1]]111 return [ButeoSyncFw.PROFILES[1]]
112112
=== modified file 'tests/qml/tst_ButeoSyncFW.qml'
--- tests/qml/tst_ButeoSyncFW.qml 2015-09-01 12:33:18 +0000
+++ tests/qml/tst_ButeoSyncFW.qml 2015-10-06 14:56:23 +0000
@@ -161,7 +161,7 @@
161161
162 function test_sync_by_profile_by_category()162 function test_sync_by_profile_by_category()
163 {163 {
164 var profiles = buteoComponent.syncProfilesByCategory('contacts')164 var profiles = buteoComponent.syncProfilesByCategory('buteo-contacts')
165 compare(profiles.length, 2)165 compare(profiles.length, 2)
166 compare(profiles[0], '63807467')166 compare(profiles[0], '63807467')
167 compare(profiles[1], 'test-profile')167 compare(profiles[1], 'test-profile')
@@ -173,7 +173,7 @@
173 spy.target = buteoComponent173 spy.target = buteoComponent
174 spy.signalName = "syncStatus"174 spy.signalName = "syncStatus"
175175
176 compare(buteoComponent.startSyncByCategory('contacts'), true)176 compare(buteoComponent.startSyncByCategory('buteo-contacts'), true)
177177
178 // wait for two signals (since we have two contacts profiles)178 // wait for two signals (since we have two contacts profiles)
179 tryCompare(spy, "count", 2)179 tryCompare(spy, "count", 2)

Subscribers

People subscribed via source and target branches