[CalendarServer-changes] [5793] CalendarServer/branches/new-store/txdav/propertystore
source_changes at macosforge.org
source_changes at macosforge.org
Mon Jun 21 09:47:36 PDT 2010
Revision: 5793
http://trac.macosforge.org/projects/calendarserver/changeset/5793
Author: cdaboo at apple.com
Date: 2010-06-21 09:47:35 -0700 (Mon, 21 Jun 2010)
Log Message:
-----------
Don't fail when removing a missing property.
Modified Paths:
--------------
CalendarServer/branches/new-store/txdav/propertystore/none.py
CalendarServer/branches/new-store/txdav/propertystore/test/base.py
Modified: CalendarServer/branches/new-store/txdav/propertystore/none.py
===================================================================
--- CalendarServer/branches/new-store/txdav/propertystore/none.py 2010-06-21 16:44:01 UTC (rev 5792)
+++ CalendarServer/branches/new-store/txdav/propertystore/none.py 2010-06-21 16:47:35 UTC (rev 5793)
@@ -47,8 +47,6 @@
if key in self.modified:
del self.modified[key]
- else:
- raise KeyError(key)
def __getitem__(self, key):
validKey(key)
Modified: CalendarServer/branches/new-store/txdav/propertystore/test/base.py
===================================================================
--- CalendarServer/branches/new-store/txdav/propertystore/test/base.py 2010-06-21 16:44:01 UTC (rev 5792)
+++ CalendarServer/branches/new-store/txdav/propertystore/test/base.py 2010-06-21 16:47:35 UTC (rev 5793)
@@ -82,7 +82,7 @@
def doDelete():
del self.propertyStore[propertyName("xyzzy")]
- self.assertRaises(KeyError, doDelete)
+ doDelete()
def test_keyInPropertyName(self):
store = self.propertyStore
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20100621/d35cd681/attachment.html>
More information about the calendarserver-changes
mailing list