[Erp5-report] r34065 jerome - /erp5/trunk/products/ERP5Type/tests/SecurityTestCase.py

[nobody at svn.erp5.org]

Author: jerome
Date: Wed Mar 24 17:52:28 2010
New Revision: 34065

URL: http://svn.erp5.org?rev=34065&view=rev
Log:
use pprint for easier display of existing local roles

Modified:
    erp5/trunk/products/ERP5Type/tests/SecurityTestCase.py

Modified: erp5/trunk/products/ERP5Type/tests/SecurityTestCase.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/tests/SecurityTestCase.py?rev=34065&r1=34064&r2=34065&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/tests/SecurityTestCase.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/tests/SecurityTestCase.py [utf8] Wed Mar 24 17:52:28 2010
@@ -29,6 +29,7 @@
 """Base Class for security tests using ERP5Security and DCWorkflow
 """
 
+from pprint import pformat
 import transaction
 from AccessControl.SecurityManagement import newSecurityManager
 from AccessControl.SecurityManagement import getSecurityManager
@@ -71,14 +72,14 @@
           groups = user.getGroups()
         self._instance.fail(
           'User %s does NOT have %s permission on %s %s (user roles: [%s], '
-          'roles needed: [%s], existing local roles: %s, '
+          'roles needed: [%s], existing local roles:\n%s\n'
           'your user groups: [%s])' %
           (username, self._permission_name, document.getPortalTypeName(),
             document, ', '.join(user.getRolesInContext(document)),
            ', '.join([x['name'] for x in
                       document.rolesOfPermission(self._permission_name)
                       if x['selected']]),
-           repr(document.get_local_roles()),
+           pformat(document.get_local_roles()),
            ', '.join(groups)))
     finally:
       setSecurityManager(sm)