Changeset 12979

Timestamp:

21 May 2015, 20:57:55 (10 years ago)

Author:

Henrik Bettermann

Message:

Test if the ticket-expire mechanism works, for both applicant and student payments.

Fix code.

Location:

main/kofacustom.skeleton/trunk/src/kofacustom/skeleton/interswitch

Files:

• browser.py (modified) (2 diffs)
• tests.py (modified) (3 diffs)

main/kofacustom.skeleton/trunk/src/kofacustom/skeleton/interswitch/browser.py

@@ -60 +60 @@
 
     def update(self):
-        super(CustomInterswitchPageStudent, self).update()
+        error = self.init_update()
+        if error:
+            self.flash(error, type='danger')
+            self.redirect(self.url(self.context, '@@index'))
+            return
         student = self.student
         xmldict = self.xmldict
@@ -120 +124 @@
 
     def update(self):
-        super(CustomInterswitchPageApplicant, self).update()
+        error = self.init_update()
+        if error:
+            self.flash(error, type='danger')
+            self.redirect(self.url(self.context, '@@index'))
+            return
         xmldict = {}
         provider_amt = 400.0

main/kofacustom.skeleton/trunk/src/kofacustom/skeleton/interswitch/tests.py

@@ -83 +83 @@
             '...item_name="School Fee" item_amt="4000000" bank_id="00" acct_num="00000000"...',
             self.browser.contents)
+
+    def test_interswitch_form_ticket_expired(self):
+        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+        acc_payment = createObject('waeup.StudentOnlinePayment')
+        acc_payment.p_state = 'unpaid'
+        acc_payment.p_category = 'clearance'
+        acc_payment.p_id = 'xyz'
+        acc_payment.pay_item_id = '123'
+        acc_payment.amount_auth = 876.0
+        self.student['payments']['xyz'] = acc_payment
+        self.browser.open(self.payments_path + '/xyz')
+        self.browser.getLink("CollegePAY", index=0).click()
+        self.assertMatches('...<input type="hidden" name="pay_item_id" value="0000" />...',
+                           self.browser.contents)
+        self.assertMatches('...Total Amount Authorized:...',
+                           self.browser.contents)
+        self.assertEqual(self.student.current_mode, 'ug_ft')
+        self.assertMatches(
+            '...<input type="hidden" name="amount" value="87600" />...',
+            self.browser.contents)
+        delta = timedelta(minutes=61)
+        acc_payment.creation_date -= delta
+        self.browser.open(self.payments_path + '/xyz')
+        self.browser.getLink("CollegePAY", index=0).click()
+        self.assertMatches(
+            '...This payment ticket is too old. Please create a new ticket...',
+            self.browser.contents)
+        delta = timedelta(minutes=2)
+        acc_payment.creation_date += delta
+        self.browser.open(self.payments_path + '/xyz')
+        self.browser.getLink("CollegePAY", index=0).click()
+        self.assertMatches('...Total Amount Authorized:...',
+                           self.browser.contents)
 
     @external_test
@@ -198 +231 @@
 
     def test_interswitch_form(self):
-
         # Manager can access InterswitchForm
         self.browser.getLink("CollegePAY", index=0).click()
@@ -205 +237 @@
         self.assertMatches(
             '...<input type="hidden" name="amount" value="100000" />...',
+            self.browser.contents)
+        delta = timedelta(minutes=61)
+        self.applicant.values()[0].creation_date -= delta
+        self.browser.open(self.payment_url)
+        self.browser.getLink("CollegePAY", index=0).click()
+        self.assertMatches(
+            '...This payment ticket is too old. Please create a new ticket...',
             self.browser.contents)