Changeset 13873

Timestamp:

4 Jun 2016, 04:43:01 (8 years ago)

Author:

Henrik Bettermann

Message:

Add checkConversion test.

Location:

main/waeup.kofa/trunk/src/waeup/kofa/applicants

Files:

• batching.py (modified) (3 diffs)
• tests/test_batching.py (modified) (2 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/applicants/batching.py

@@ -20 +20 @@
 import unicodecsv as csv # XXX: csv ops should move to dedicated module.
 import grok
+from time import time
 from zope.schema import getFields
 from zope.interface import Interface
@@ -341 +342 @@
     """The Applicant Online Payment Processor imports applicant payment tickets.
     The tickets are located in the applicant container.
-    The only additional locator is `p_id`, the object id.
 
     The `checkConversion` method checks the format of the payment identifier.
@@ -365 +365 @@
     def available_fields(self):
         af = sorted(list(set(
-            self.location_fields + getFields(self.iface).keys())) + ['p_id',])
+            self.location_fields + getFields(self.iface).keys())))
         af.remove('display_item')
         return af

main/waeup.kofa/trunk/src/waeup/kofa/applicants/tests/test_batching.py

@@ -25 +25 @@
 import unittest
 import grok
+from time import time
 from hurry.workflow.interfaces import IWorkflowState
 from zope.component.hooks import setSite, clearSite
@@ -498 +499 @@
             dict(applicant_id=self.applicant2.applicant_id, p_id='p456'), self.app)
 
+    def test_checkConversion(self):
+        errs, inv_errs, conv_dict = self.processor.checkConversion(
+            dict(p_id='<IGNORE>'), mode='create')
+        self.assertEqual(len(errs),0)
+        errs, inv_errs, conv_dict = self.processor.checkConversion(
+            dict(p_id='<IGNORE>'), mode='update')
+        self.assertEqual(len(errs),1)
+        self.assertEqual(errs[0], ('p_id', u'missing'))
+        errs, inv_errs, conv_dict = self.processor.checkConversion(
+            dict(p_id='p1266236341955'))
+        self.assertEqual(len(errs),0)
+        errs, inv_errs, conv_dict = self.processor.checkConversion(
+            dict(p_id='nonsense'))
+        self.assertEqual(len(errs),1)
+        self.assertEqual(errs[0], ('p_id', u'invalid length'))
+        timestamp = ("%d" % int(time()*10000))[1:]
+        p_id = "p%s" % timestamp
+        errs, inv_errs, conv_dict = self.processor.checkConversion(
+            dict(p_id=p_id))
+        self.assertEqual(len(errs),0)
+        dup_payment = createObject(u'waeup.ApplicantOnlinePayment')
+        dup_payment.p_id = 'p1266236341955'
+        self.applicant2[dup_payment.p_id] = dup_payment
+        errs, inv_errs, conv_dict = self.processor.checkConversion(
+            dict(p_id='p1266236341955'), mode='create')
+        self.assertEqual(len(errs),1)
+        self.assertEqual(errs[0], ('p_id', u'p_id exists in dp2011_1234 '))
+
     def test_import(self):
         num, num_warns, fin_file, fail_file = self.processor.doImport(