Changeset 15148 for main/waeup.kofa/branches/henrik-transcript-workflow/src/waeup/kofa/students

Timestamp:

18 Sep 2018, 20:48:41 (6 years ago)

Author:

Henrik Bettermann

Message:

Add logging and history messages after signing.

Location:

main/waeup.kofa/branches/henrik-transcript-workflow/src/waeup/kofa/students

Files:

• browser.py (modified) (1 diff)
• tests/test_browser.py (modified) (3 diffs)

main/waeup.kofa/branches/henrik-transcript-workflow/src/waeup/kofa/students/browser.py

@@ -1178 +1178 @@
             return
         self.flash(_('Transcript signed.'))
+        ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
+        self.context.student.__parent__.logger.info(
+            '%s - %s - Transcript signed'
+            % (ob_class, self.context.student.student_id))
+        self.context.student.history.addMessage('Transcript signed')
         self.redirect(self.url(self.context))
         return

main/waeup.kofa/branches/henrik-transcript-workflow/src/waeup/kofa/students/tests/test_browser.py

@@ -2146 +2146 @@
         # Officer can search for students in state 'transcript requested'
         self.browser.open(self.container_path)
-        self.browser.getControl(name="searchtype").value = ['transcript requested']
+        self.browser.getControl(name="searchtype").value = [
+            'transcript requested']
         self.browser.getControl("Find student(s)").click()
         self.assertTrue('Anna Tester' in self.browser.contents)
@@ -2153 +2154 @@
         # Officers can still edit studycourse, studylevel and course tickets.
         self.browser.open(self.studycourse_path + '/manage')
-        self.assertTrue('Undergraduate Full-Time</option>' in self.browser.contents)
+        self.assertTrue('Undergraduate Full-Time</option>'
+            in self.browser.contents)
         self.browser.getControl(name="form.certificate").value = ['CERT1']
         self.browser.getControl(name="form.current_session").value = ['2004']
@@ -2178 +2180 @@
         self.browser.open(self.studycourse_path + '/manage')
         self.assertTrue('The requested form is locked' in self.browser.contents)
-        self.assertFalse('Undergraduate Full-Time</option>' in self.browser.contents)
+        self.assertFalse('Undergraduate Full-Time</option>'
+            in self.browser.contents)
         self.browser.open(self.studycourse_path + '/100/manage')
         self.assertTrue('The requested form is locked' in self.browser.contents)
         self.browser.open(self.studycourse_path + '/100/COURSE1/manage')
         self.assertTrue('The requested form is locked' in self.browser.contents)
+        # Transcript can be signed if officer has the permission to sign
+        self.browser.open(self.studycourse_path + '/transcript')
+        self.assertFalse('Sign transcript' in self.browser.contents)
+        prmglobal = IPrincipalRoleManager(self.app)
+        prmglobal.assignRoleToPrincipal('waeup.TranscriptSignee', 'mrtranscript')
+        self.browser.open(self.studycourse_path + '/transcript')
+        self.browser.getLink("Sign transcript").click()
+        # Transcript signing has been logged ...
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'students.log')
+        logcontent = open(logfile).read()
+        self.assertTrue(
+            'mrtranscript - students.browser.StudentTranscriptSignView - '
+            'K1000000 - Transcript signed' in logcontent)
+        # ... and appears in the student's history
+        self.browser.open(self.history_path)
+        self.assertTrue('Transcript signed by Ruth Gordon'
+            in self.browser.contents)
         # Officer can release the transcript
         self.browser.open(self.student_path)