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
Reviewer Review Type Date Requested Status
Tim Cole (community) Approve
Eric Casteleijn (community) Approve
Review via email: mp+22909@code.qastaging.launchpad.net

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.
Revision history for this message
Eric Casteleijn (thisfred) wrote :

Looks good, tests give one error, looks like a pasto in the import:

===============================================================================
[ERROR]: desktopcouch.bookmarks.tests.test_record

Traceback (most recent call last):
  File "/usr/lib/python2.6/dist-packages/twisted/trial/runner.py", line 563, in loadPackage
    module = modinfo.load()
  File "/usr/lib/python2.6/dist-packages/twisted/python/modules.py", line 381, in load
    return self.pathEntry.pythonPath.moduleLoader(self.name)
  File "/usr/lib/python2.6/dist-packages/twisted/python/reflect.py", line 464, in namedAny
    topLevelPackage = _importAndCheckStack(trialname)
  File "/home/eric/canonical/desktopcouch/fix-516128/desktopcouch/bookmarks/tests/test_record.py", line 24, in <module>
    from desktopcouch.contacts.record import \
exceptions.ImportError: cannot import name Bookmark
-------------------------------------------------------------------------------

review: Needs Fixing
146. By Zachery Bir

woops, missed a s/contacts/bookmarks/

147. By Zachery Bir

le sigh. Copy/paste code strikes again.

Revision history for this message
Eric Casteleijn (thisfred) wrote :

All fixed! :)

review: Approve
Revision history for this message
Tim Cole (tcole) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches