Changeset 9924 for main/waeup.kofa/trunk/src/waeup/kofa/students/tests

Timestamp:

30 Jan 2013, 08:58:37 (12 years ago)

Author:

Henrik Bettermann

Message:

Add new permission waeup.editStudyLevel to allow course advisers to edit study levels after rejection.

Improve logging. Creation and deletion of course tickets is now properly logged.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/students/tests/test_browser.py (modified) (5 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/students/tests/test_browser.py

@@ -750 +750 @@
         logcontent = open(logfile).read()
         self.assertTrue('zope.mgr - students.browser.StudyLevelManageFormPage '
-        '- K1000000 - removed: COURSE1' in logcontent)
+        '- K1000000 - removed: COURSE1 at 100' in logcontent)
         self.browser.getLink("here").click()
         self.browser.getControl(name="form.course").value = ['COURSE1']
@@ -1422 +1422 @@
             cert,studylevel)
         L110_student_path = self.studycourse_path + '/110'
-        # Only in state courses registered and only if the current level
-        # corresponds with the name of the study level object
-        # the 100L CA does see the 'Validate' button
+        # The CA can neither see the Validate nor the Edit button
         self.browser.open(L110_student_path)
         self.assertFalse('Validate courses' in self.browser.contents)
+        self.assertFalse('Edit' in self.browser.contents)
         IWorkflowInfo(self.student).fireTransition('register_courses')
         self.browser.open(L110_student_path)
         self.assertFalse('Validate courses' in self.browser.contents)
+        self.assertFalse('Edit' in self.browser.contents)
+        # Only in state courses registered and only if the current level
+        # corresponds with the name of the study level object
+        # the 100L CA does see the 'Validate' button but not
+        # the edit button
         self.student['studycourse'].current_level = 110
         self.browser.open(L110_student_path)
+        self.assertFalse('Edit' in self.browser.contents)
         self.assertTrue('Validate courses' in self.browser.contents)
-        # ... but a 100L CA does not see the button on other levels
+        # But a 100L CA does not see the button at other levels
         studylevel2 = StudentStudyLevel()
         studylevel2.level = 200
@@ -1440 +1445 @@
         L200_student_path = self.studycourse_path + '/200'
         self.browser.open(L200_student_path)
+        self.assertFalse('Edit' in self.browser.contents)
         self.assertFalse('Validate courses' in self.browser.contents)
         self.browser.open(L110_student_path)
@@ -1471 +1477 @@
         self.browser.getControl("Send message now").click()
         self.assertTrue('Your message has been sent' in self.browser.contents)
+        # The CA does now see the Edit button and can edit
+        # current study level
+        self.browser.open(L110_student_path)
+        self.browser.getLink("Edit").click()
+        self.assertTrue('Edit course list of 100 (Year 1) on 1st probation'
+            in self.browser.contents)
         # The CA can't validate courses if not in state
         # courses registered
@@ -2069 +2081 @@
         self.browser.getControl("Register course list").click()
         self.assertTrue('Maximum credits of 50 exceeded' in self.browser.contents)
-        # Student can now remove the ticket ...
+        # Student can now remove the ticket
         ctrl = self.browser.getControl(name='val_id')
         ctrl.getControl(value='COURSE1').selected = True
         self.browser.getControl("Remove selected", index=0).click()
         self.assertTrue('Successfully removed' in self.browser.contents)
-        # ... and add the same ticket using the edit page directly.
+        # Removing course tickets is properly logged
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'students.log')
+        logcontent = open(logfile).read()
+        self.assertTrue('K1000000 - students.browser.StudyLevelEditFormPage '
+        '- K1000000 - removed: COURSE1 at 200' in logcontent)
+        # They can add the same ticket using the edit page directly.
         # We can do the same by adding the course on the manage page directly
         self.browser.getControl(name="course").value = 'COURSE1'
         self.browser.getControl("Add course ticket").click()
+        # Adding course tickets is logged
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'students.log')
+        logcontent = open(logfile).read()
+        self.assertTrue('K1000000 - students.browser.StudyLevelEditFormPage - '
+            'K1000000 - added: COURSE1|200|2004' in logcontent)
         # Course list can be registered
         self.browser.getControl("Register course list").click()