[Checkins] SVN: Products.CMFDefault/trunk/Products/CMFDefault/tests/test_DiscussionReply.py - don't use has_permission for permission checks

Yvo Schubbe y.2011 at wcm-solutions.de
Wed Feb 16 09:47:57 EST 2011


Log message for revision 120382:
  - don't use has_permission for permission checks

Changed:
  U   Products.CMFDefault/trunk/Products/CMFDefault/tests/test_DiscussionReply.py

-=-
Modified: Products.CMFDefault/trunk/Products/CMFDefault/tests/test_DiscussionReply.py
===================================================================
--- Products.CMFDefault/trunk/Products/CMFDefault/tests/test_DiscussionReply.py	2011-02-16 14:24:58 UTC (rev 120381)
+++ Products.CMFDefault/trunk/Products/CMFDefault/tests/test_DiscussionReply.py	2011-02-16 14:47:57 UTC (rev 120382)
@@ -20,6 +20,8 @@
 from AccessControl.SecurityManagement import getSecurityManager
 from zope.site.hooks import setSite
 
+from Products.CMFDefault.permissions import AccessContentsInformation
+from Products.CMFDefault.permissions import View
 from Products.CMFDefault.testing import FunctionalLayer
 
 
@@ -59,10 +61,9 @@
         # https://bugs.launchpad.net/zope-cmf/+bug/161720
         state = self.portal.portal_workflow.getInfoFor(reply, 'review_state')
         self.assertEqual(state, 'published')
-        member = self.portal.portal_membership.getAuthenticatedMember()
-        self.failUnless(member.has_permission('View', reply))
-        self.failUnless(member.has_permission('Access contents information',
-                         reply))
+        sm = getSecurityManager()
+        self.assertTrue(sm.checkPermission(View, reply))
+        self.assertTrue(sm.checkPermission(AccessContentsInformation, reply))
 
 
 class DiscussionReplyTestMember(DiscussionReplyTest):



More information about the checkins mailing list