[Checkins] SVN: Products.CMFDefault/branches/rossp-intid/Products/CMFDefault/tests/test_Discussions.py Catalog no longer uses paths as uids
Ross Patterson
me at rpatterson.net
Thu Nov 3 04:22:21 UTC 2011
Log message for revision 123273:
Catalog no longer uses paths as uids
Changed:
U Products.CMFDefault/branches/rossp-intid/Products/CMFDefault/tests/test_Discussions.py
-=-
Modified: Products.CMFDefault/branches/rossp-intid/Products/CMFDefault/tests/test_Discussions.py
===================================================================
--- Products.CMFDefault/branches/rossp-intid/Products/CMFDefault/tests/test_Discussions.py 2011-11-03 04:22:16 UTC (rev 123272)
+++ Products.CMFDefault/branches/rossp-intid/Products/CMFDefault/tests/test_Discussions.py 2011-11-03 04:22:20 UTC (rev 123273)
@@ -33,10 +33,8 @@
from Products.CMFDefault.DiscussionTool import DiscussionTool
from Products.CMFDefault.exceptions import DiscussionNotAllowed
-def has_path(catalog, path):
- if isinstance(path, tuple):
- path = '/'.join(path)
- return bool(catalog.getrid(path))
+def has_path(catalog, obj):
+ return bool(catalog.getrid(obj))
class DiscussionItemTests(unittest.TestCase):
@@ -179,7 +177,7 @@
test.allow_discussion = 1
self.assertEqual( len(ctool), 1 )
- self.failUnless(has_path(ctool, test.getPhysicalPath()))
+ self.failUnless(has_path(ctool, test))
talkback = dtool.getDiscussionFor(test)
self.assertEqual( talkback.getPhysicalPath(),
('', 'bar', 'site', 'test', 'talkback') )
More information about the checkins
mailing list