Merge lp://qastaging/~urbanape/desktopcouch/fix-516128 into lp://qastaging/desktopcouch
Proposed by
Zachery Bir
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Tim Cole | ||||
Approved revision: | 147 | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp://qastaging/~urbanape/desktopcouch/fix-516128 | ||||
Merge into: | lp://qastaging/desktopcouch | ||||
Diff against target: |
157 lines (+140/-0) 3 files modified
desktopcouch/bookmarks/__init__.py (+20/-0) desktopcouch/bookmarks/record.py (+60/-0) desktopcouch/bookmarks/tests/test_record.py (+60/-0) |
||||
To merge this branch: | bzr merge lp://qastaging/~urbanape/desktopcouch/fix-516128 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Cole (community) | Approve | ||
Eric Casteleijn (community) | Approve | ||
Review via email:
|
Commit message
Add bookmark records to desktopcouch.
Description of the change
Add the minimal stuff for Bookmarks records.
To post a comment you must log in.
Looks good, tests give one error, looks like a pasto in the import:
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= ======= == bookmarks. tests.test_ record
[ERROR]: desktopcouch.
Traceback (most recent call last): python2. 6/dist- packages/ twisted/ trial/runner. py", line 563, in loadPackage python2. 6/dist- packages/ twisted/ python/ modules. py", line 381, in load pythonPath. moduleLoader( self.name) python2. 6/dist- packages/ twisted/ python/ reflect. py", line 464, in namedAny Stack(trialname ) eric/canonical/ desktopcouch/ fix-516128/ desktopcouch/ bookmarks/ tests/test_ record. py", line 24, in <module> contacts. record import \ ImportError: cannot import name Bookmark ------- ------- ------- ------- ------- ------- ------- ------- ------- ------- --
File "/usr/lib/
module = modinfo.load()
File "/usr/lib/
return self.pathEntry.
File "/usr/lib/
topLevelPackage = _importAndCheck
File "/home/
from desktopcouch.
exceptions.
-------