[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