[CMF-checkins] CVS: CMF/DCWorkflow/tests - test_roles.py:1.2

[Shane Hathaway]

Update of /cvs-repository/CMF/DCWorkflow/tests
In directory cvs.zope.org:/tmp/cvs-serv706/tests

Modified Files:
	test_roles.py 
Log Message:
The workflow can now expand abstract groups into true groups.

Here is the idea behind this: there are many specific groups, but they
are all global; there are also many general groups, but they should all
be local.  DCWorkflow becomes the main component that maps between
global and local groups.

Also changed to storing group names instead of monikers, to avoid the
need to explain what a security moniker is. ;-)



=== CMF/DCWorkflow/tests/test_roles.py 1.1 => 1.2 ===
--- CMF/DCWorkflow/tests/test_roles.py:1.1	Thu Jan 15 12:14:44 2004
+++ CMF/DCWorkflow/tests/test_roles.py	Thu Jan 15 15:52:07 2004
@@ -38,19 +38,19 @@
 
     def testModifyRolesForGroup(self):
         modifyRolesForGroup(
-            self.ob, '(Group) Administrators', ['Owner'], ['Member', 'Owner'])
+            self.ob, 'Administrators', ['Owner'], ['Member', 'Owner'])
         modifyRolesForGroup(
-            self.ob, '(Group) Users', [], ['Member'])
+            self.ob, 'Users', [], ['Member'])
         self.assertEqual(self.ob.__ac_local_roles__, {
-            '(Group) Administrators': ('Manager', 'Owner'),
+            '(Group) Administrators': ['Manager', 'Owner'],
             })
         modifyRolesForGroup(
-            self.ob, '(Group) Administrators', ['Member'], ['Member', 'Owner'])
+            self.ob, 'Administrators', ['Member'], ['Member', 'Owner'])
         modifyRolesForGroup(
-            self.ob, '(Group) Users', ['Member'], ['Member'])
+            self.ob, 'Users', ['Member'], ['Member'])
         self.assertEqual(self.ob.__ac_local_roles__, {
-            '(Group) Administrators': ('Manager', 'Member'),
-            '(Group) Users': ('Member',),
+            '(Group) Administrators': ['Manager', 'Member'],
+            '(Group) Users': ['Member'],
             })
 
     def testModifyRolesForPermission(self):