lp:~cmiller/desktopcouch/service-must-not-call-self-over-dbus

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:desktopcouch at revision 264

Natalia Bidart (community): Approve on 2011-02-01

Eric Casteleijn (community): Approve on 2011-01-26

Diff: 336 lines (+82/-34)

6 files modified

desktopcouch/application/plugins/__init__.py (+2/-2)
desktopcouch/application/plugins/tests/test_plugins.py (+6/-1)
desktopcouch/application/plugins/tests/test_ubuntuone_pairing.py (+22/-15)
desktopcouch/application/plugins/ubuntuone_pairing.py (+17/-11)
desktopcouch/application/service.py (+4/-4)
desktopcouch/application/tests/test_service.py (+31/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #706939: service tries to access service over dbus, deadlock

Critical

Fix Released

Link a bug report

Related blueprints

268. By Chad Miller on 2011-01-31

Er, I can't find a way this could swallow exceptions now.

267. By Chad Miller on 2011-01-27

pep8, trailing whitespace.

266. By Chad Miller on 2011-01-27

Merge with trunk to fix conflicts.

265. By Chad Miller on 2011-01-26

Fix tests.

264. By Chad Miller on 2011-01-26

PEP8

263. By Chad Miller on 2011-01-26

Give up on mocker.replace, and instead pass optional test arguments.

262. By Chad Miller on 2011-01-26

Pass the port to be used in tests.

Migration and test-pairing still fail.

261. By Chad Miller on 2011-01-24

Use port also in tests, silly.

260. By Chad Miller on 2011-01-24

When running plugin code in initializing the desktopcouch service, do not call any functions that require that the service be running and answering DBus method calls. In particular, we already know the port couchdb is listening on, so do not ask for that via DBus client call, but explicity pass it into the plugin function calls. (LP: #706939)

259. By Chad Miller on 2011-01-14

If a record id is not in a database, desktopcouch would raise a strange exception when it was used to try to delete a record. Now, return normally, with value None as the deleted-record id. (LP: #702979)