Skip to content
Permalink
Browse files

Fix tests with missing yields.

  • Loading branch information...
cyrusdaboo committed Jul 26, 2012
1 parent 4e783a0 commit b10203ad8f1128556fa0c1ee2f72212b82f837fb
Showing with 4 additions and 3 deletions.
  1. +4 −3 txdav/carddav/datastore/test/common.py
@@ -288,7 +288,7 @@ def positiveAssertions():
yield self.commit()
home = yield self.homeUnderTest()
addressbook = yield home.addressbookWithName("some_other_name")
positiveAssertions()
yield positiveAssertions()
# FIXME: revert
# FIXME: test for multiple renames
# FIXME: test for conflicting renames (a->b, c->a in the same txn)
@@ -316,6 +316,7 @@ def test_createAddressBookWithName_absent(self):
self.assertIdentical((yield home.addressbookWithName(name)), None)
yield home.createAddressBookWithName(name)
self.assertNotIdentical((yield home.addressbookWithName(name)), None)
@inlineCallbacks
def checkProperties():
addressbookProperties = (yield home.addressbookWithName(name)).properties()
addressbookType = ResourceType.addressbook #@UndefinedVariable
@@ -325,7 +326,7 @@ def checkProperties():
],
addressbookType
)
checkProperties()
yield checkProperties()
yield self.commit()

# Make sure notification fired after commit
@@ -341,7 +342,7 @@ def checkProperties():

# Sanity check: are the properties actually persisted?
# FIXME: no independent testing of this right now
checkProperties()
yield checkProperties()


@inlineCallbacks

0 comments on commit b10203a

Please sign in to comment.
You can’t perform that action at this time.