Changeset 14685

Timestamp:

15 May 2017, 11:28:32 (7 years ago)

Author:

Henrik Bettermann

Message:

Log ticket creation of previous session payments too.

Location:

main/waeup.kofa/trunk/src/waeup/kofa/students

Files:

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

main/waeup.kofa/trunk/src/waeup/kofa/students/browser.py

@@ -1843 +1843 @@
         self.context[payment.p_id] = payment
         self.flash(_('Payment ticket created.'))
+        self.context.writeLogMessage(self,'added: %s' % payment.p_id)
         self.redirect(self.url(self.context))
         return

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

@@ -1198 +1198 @@
         self.browser.open(self.payments_path)
 
-        # Managers can add previous school fee payment tickets in any state.
+        # Managers can add balance school fee payment tickets in any state.
         IWorkflowState(self.student).setState('courses registered')
         self.browser.open(self.payments_path)
         self.browser.getLink("Add balance payment ticket").click()
 
-        # Previous session payment form is provided
+        # Palance payment form is provided
         self.assertEqual(self.student.current_session, 2004)
         self.browser.getControl(name="form.p_category").value = ['schoolfee']
@@ -2923 +2923 @@
         self.assertMatches('...This is not a previous session...',
                            self.browser.contents)
+
         # Students can pay current session school fee.
         self.browser.getControl(name="form.p_category").value = ['schoolfee']
@@ -2936 +2937 @@
                            self.browser.contents)
         self.assertEqual(self.student['payments'][value].amount_auth, 20000.0)
+
+        # Ticket creation is logged.
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'students.log')
+        logcontent = open(logfile).read()
+        self.assertTrue(
+            'K1000000 - students.browser.PreviousPaymentAddFormPage - '
+            'K1000000 - added: %s' % value
+            in logcontent)
 
         # Payment session is properly set