Changeset 9520 for main/waeup.uniben/trunk/src/waeup/uniben/students/tests

Timestamp:

4 Nov 2012, 07:48:11 (12 years ago)

Author:

Henrik Bettermann

Message:

Reorganize getReturningData.

Students can pay for previous sessions in all workflow states.
Fresh students are excluded by the update method of the
PreviousPaymentAddFormPage?.

Students are only allowed to pay for the next session if current session payment has really been made, i.e. payment object exists and is paid.

File:

• main/waeup.uniben/trunk/src/waeup/uniben/students/tests/test_browser.py (modified) (13 diffs)

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

@@ -214 +214 @@
         self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
         self.browser.open(self.payments_path)
-        self.browser.getControl("Add online payment ticket").click()
+        self.browser.getLink("Add current session payment ticket").click()
         self.browser.getControl("Create ticket").click()
         self.assertMatches('...Amount could not be determined...',
@@ -248 +248 @@
         self.assertMatches('...ticket created...',
                            self.browser.contents)
+        schoolfee_ticket = self.student['payments'].values()[0]
         self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(name="form.p_category").value = ['gown']
@@ -286 +287 @@
         self.browser.getControl(name="form.p_category").value = ['schoolfee']
         self.browser.getControl("Create ticket").click()
+        # Uups, we forgot to add a session configuration for next session
         self.assertMatches('...Session configuration object is not...',
                            self.browser.contents)
-        # Uups, we forgot to add a session configuration for next session
         configuration = createObject('waeup.SessionConfiguration')
         configuration.academic_session = 2005
         self.app['configuration'].addSessionConfiguration(configuration)
+        self.browser.open(self.payments_path + '/addop')
+        self.browser.getControl(name="form.p_category").value = ['schoolfee']
+        self.browser.getControl("Create ticket").click()
+        self.assertMatches('...You have not yet paid your current/active session...',
+                           self.browser.contents)
+        # Ok, let's pay the first schoolfee ticket.
+        schoolfee_ticket.approve()
         self.browser.open(self.payments_path + '/addop')
         self.browser.getControl(name="form.p_category").value = ['schoolfee']
@@ -309 +317 @@
         del self.app['configuration']['2004']
         self.browser.open(self.payments_path)
-        self.browser.getControl("Add online payment ticket").click()
+        self.browser.getLink("Add current session payment ticket").click()
         self.browser.getControl("Create ticket").click()
         self.assertMatches('...Session configuration object is not...',
@@ -375 +383 @@
             self.student, None, None)
         self.assertEqual(payment, None)
+        # Student is in state 'created' and can thus not pay.
         self.assertTrue(u'Amount could not be determined.' in error)
 
@@ -383 +392 @@
         self.assertTrue(u'The previous session must not fall below' in error)
         error, payment = utils.setPaymentDetails('schoolfee',
-            self.student, 2004, 300)
+            self.student, 2005, 300)
         self.assertEqual(payment, None)
         self.assertTrue(u'This is not a previous session' in error)
 
-        # Previous session payment; fresh and returning
+        # Previous session schoolfee payment; fresh and returning
         # are distinguished by their entry_level
         error, payment = utils.setPaymentDetails('schoolfee',
@@ -402 +411 @@
         self.assertFalse(payment.p_current)
 
-        # Current payment; fresh and returning
+        # Current schoolfee payment; fresh and returning
         # are distinguished by their state
         IWorkflowState(self.student).setState('cleared')
@@ -426 +435 @@
         error, payment = utils.setPaymentDetails('schoolfee',
             self.student, None, None)
+        self.assertTrue(
+            u'You have not yet paid your current/active session.' in error)
+        # Ok, that means we have to add paid payment ticket first.
+        payment = createObject('waeup.StudentOnlinePayment')
+        payment.p_category = u'schoolfee'
+        payment.p_session = self.student.current_session
+        payment.p_item = u'My Certificate'
+        payment.p_id = u'anyid'
+        payment.p_state = u'paid'
+        self.student['payments']['anykey'] = payment
+        error, payment = utils.setPaymentDetails('schoolfee',
+            self.student, None, None)
         self.assertEqual(payment.p_level, 200)
         self.assertEqual(payment.p_session, 2005)
@@ -431 +452 @@
         self.assertEqual(payment.p_item, u'CERT1')
         self.assertEqual(error, None)
+
+        # Staff members pay less.
         self.student.is_staff = True
         error, payment = utils.setPaymentDetails('schoolfee',
@@ -440 +463 @@
         self.assertEqual(error, None)
 
+        # Foreigners pay more.
         IWorkflowState(self.student).setState('cleared')
         self.student.is_staff = False
@@ -451 +475 @@
         self.assertEqual(payment.p_item, u'CERT1')
         self.assertEqual(error, None)
-
         IWorkflowState(self.student).setState('returning')
         self.student.is_staff = False
-        self.student.nationality = u'DE'
         self.certificate.school_fee_4 = 20000.0
         error, payment = utils.setPaymentDetails(
@@ -467 +489 @@
         # if they are ug or pg students.
         IWorkflowState(self.student).setState('school fee paid')
-        self.certificate.study_mode = 'special_pg_pt'
         self.student.is_staff = False
         self.student.nationality = u'NG'
         self.certificate.school_fee_2 = 10000.0
-
+        error, payment = utils.setPaymentDetails(
+            'schoolfee', self.student, None, None)
+        self.assertEqual(payment.p_level, None)
+        self.assertEqual(payment.p_session, 2005)
+        self.assertEqual(payment.amount_auth, 10088.0)
+        self.assertEqual(payment.p_item, u'CERT1')
+        self.assertEqual(error, None)
+        IWorkflowState(self.student).setState('courses registered')
+        self.certificate.study_mode = 'special_pg_pt'
         error, payment = utils.setPaymentDetails(
             'schoolfee', self.student, None, None)
@@ -479 +508 @@
         self.assertEqual(payment.p_item, u'CERT1')
         self.assertEqual(error, None)
-
-        IWorkflowState(self.student).setState('courses registered')
-        error, payment = utils.setPaymentDetails(
-            'schoolfee', self.student, None, None)
-        self.assertEqual(payment.p_level, None)
-        self.assertEqual(payment.p_session, 2005)
-        self.assertEqual(payment.amount_auth, 10000.0)
-        self.assertEqual(payment.p_item, u'CERT1')
-        self.assertEqual(error, None)
-
         IWorkflowState(self.student).setState('courses validated')
         error, payment = utils.setPaymentDetails(