Changeset 12982 for main/kofacustom.wdu/trunk/src/kofacustom/wdu/interswitch

Timestamp:

22 May 2015, 07:19:27 (10 years ago)

Author:

Henrik Bettermann

Message:

Fix exporter and test if the ticket-expire mechanism works, for both applicant and student payments.

Location:

main/kofacustom.wdu/trunk/src/kofacustom/wdu/interswitch

Files:

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

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

@@ -62 +62 @@
 
     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
@@ -125 +129 @@
 
     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 = 500.0

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

@@ -85 +85 @@
             '...item_name="School Fees" item_amt="23000000" bank_id="47" acct_num="4742014159"...',
             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 = 'schoolfee'
+        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="101" />...',
+                           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
@@ -208 +242 @@
             '...<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)
 
     @external_test