[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