Changeset 7184 for main/waeup.sirp/trunk/src/waeup/sirp/applicants/tests

Timestamp:

24 Nov 2011, 09:20:13 (13 years ago)

Author:

Henrik Bettermann

Message:

Reorganise permissions:

The navigation viewlets now manage the permission requirements themselves. No need to do this in pagetemplates.

Rename permission waeup.View to waeup.viewAcademics because it only refers to the academic section.

Add permission waeup.Authenticated (which is used in students). The StudentRecordOwner? explicitly needs this permission. Otherwise the MyData? tab disappears when changing the password.

Roles do not need to get the waeup.Public permission. This is already guaranteed in site.zcml.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/applicants/tests/test_browser.py (modified) (3 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/applicants/tests/test_browser.py

@@ -138 +138 @@
             'Manage' in self.browser.contents)
 
+    
     def test_anonymous_no_actions(self):
         # Make sure anonymous users cannot access actions
@@ -173 +174 @@
         return
 
-    def test_local_roles_add_delete(self):
-        # Managers can assign and delete local roles of applicants root
-        myusers = self.app['users']
-        myusers.addUser('bob', 'bobssecret')
-        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
-        self.browser.open(self.manage_root_path)
-        self.browser.getControl(name="user").value = ['bob']
-        self.browser.getControl(name="local_role").value = [
-            'waeup.ApplicationsOfficer']
-        self.browser.getControl("Add local role").click()
-        self.assertTrue('<td>bob</td>' in self.browser.contents)
-        # Remove the role assigned
-        ctrl = self.browser.getControl(name='role_id')
-        ctrl.getControl(value='bob|waeup.ApplicationsOfficer').selected = True
-        self.browser.getControl("Remove selected local roles").click()
-        self.assertTrue('Successfully removed:' in self.browser.contents)
-        self.assertFalse('<td>bob</td>' in self.browser.contents)
-        return
+    # We have no local roles yet
+    #def test_local_roles_add_delete(self):
+    #    # Managers can assign and delete local roles of applicants root
+    #    myusers = self.app['users']
+    #    myusers.addUser('bob', 'bobssecret')
+    #    self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+    #    self.browser.open(self.manage_root_path)
+    #    self.browser.getControl(name="user").value = ['bob']
+    #    self.browser.getControl(name="local_role").value = [
+    #        'waeup.local.ApplicationsOfficer']
+    #    self.browser.getControl("Add local role").click()
+    #    self.assertTrue('<td>bob</td>' in self.browser.contents)
+    #    # Remove the role assigned
+    #    ctrl = self.browser.getControl(name='role_id')
+    #    ctrl.getControl(value='bob|waeup.ApplicationsOfficer').selected = True
+    #    self.browser.getControl("Remove selected local roles").click()
+    #    self.assertTrue('Successfully removed:' in self.browser.contents)
+    #    self.assertFalse('<td>bob</td>' in self.browser.contents)
+    #    return
 
     def test_add_delete_container(self):
@@ -423 +425 @@
         return
 
-    def test_local_roles_add_delete(self):
-        # Managers can assign and delete local roles of applicants containers
-        myusers = self.app['users']
-        myusers.addUser('bob', 'bobssecret')
-        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
-        self.browser.open(self.manage_container_path)
-        self.browser.getControl(name="user").value = ['bob']
-        self.browser.getControl(name="local_role").value = [
-            'waeup.ApplicationsOfficer']
-        self.browser.getControl("Add local role").click()
-        self.assertTrue('<td>bob</td>' in self.browser.contents)
-        ctrl = self.browser.getControl(name='role_id')
-        ctrl.getControl(value='bob|waeup.ApplicationsOfficer').selected = True
-        self.browser.getControl("Remove selected local roles").click()
-        self.assertTrue('Successfully removed:' in self.browser.contents)
-        self.assertFalse('<td>bob</td>' in self.browser.contents)
-        return
+    # We have no local roles yet
+    #def test_local_roles_add_delete(self):
+    #    # Managers can assign and delete local roles of applicants containers
+    #    myusers = self.app['users']
+    #    myusers.addUser('bob', 'bobssecret')
+    #    self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+    #    self.browser.open(self.manage_container_path)
+    #    self.browser.getControl(name="user").value = ['bob']
+    #    self.browser.getControl(name="local_role").value = [
+    #        'waeup.local.ApplicationsOfficer']
+    #    self.browser.getControl("Add local role").click()
+    #    self.assertTrue('<td>bob</td>' in self.browser.contents)
+    #    ctrl = self.browser.getControl(name='role_id')
+    #    ctrl.getControl(value='bob|waeup.ApplicationsOfficer').selected = True
+    #    self.browser.getControl("Remove selected local roles").click()
+    #    self.assertTrue('Successfully removed:' in self.browser.contents)
+    #    self.assertFalse('<td>bob</td>' in self.browser.contents)
+    #    return
 
 class LoginTest(FunctionalTestCase):