processor readability and error fix

Author: anacarolinats

maxipago/resources/payment.py

@@ -13,6 +13,10 @@ def process(self):
         self.captured = False
 
         tree = etree.parse(StringIO(self.data))
+        error_code = tree.find('errorCode')
+        if error_code is not None and error_code != '0':
+            error_message = tree.find('errorMsg').text
+            raise PaymentException(message=error_message)
 
         processor_code = tree.find('processorCode')
         if processor_code.text.lower() == 'a':

maxipago/utils.py

@@ -3,6 +3,7 @@
 import unittest
 from datetime import date
 from maxipago import Maxipago, exceptions
+from maxipago.utils import payment_processors
 from random import randint
 
 
@@ -157,7 +158,7 @@ def test_payment_authorize(self):
         REFERENCE = randint(1, 100000)
 
         response = self.maxipago.payment.authorize(
-            processor_id=1,
+            processor_id=payment_processors.TEST,
             reference_num=REFERENCE,
 
             billing_name=u'Fulano de Tal',
@@ -184,7 +185,7 @@ def test_payment_direct(self):
         REFERENCE = randint(1, 100000)
 
         response = self.maxipago.payment.direct(
-            processor_id=1,
+            processor_id=payment_processors.TEST,
             reference_num=REFERENCE,
 
             billing_name=u'Fulano de Tal',
@@ -211,7 +212,7 @@ def test_payment_direct_declined(self):
         REFERENCE = randint(1, 100000)
 
         response = self.maxipago.payment.direct(
-            processor_id=1,
+            processor_id=payment_processors.TEST,
             reference_num=REFERENCE,
 
             billing_name=u'Fulano de Tal',
@@ -234,5 +235,94 @@ def test_payment_direct_declined(self):
         self.assertFalse(response.authorized)
         self.assertFalse(response.captured)
 
+    def test_payment_direct_with_token(self):
+        CUSTOMER_ID = randint(1, 100000)
+
+        response = self.maxipago.customer.add(
+            customer_id=CUSTOMER_ID,
+            first_name=u'Fulano',
+            last_name=u'de Tal',
+        )
+
+        self.assertTrue(hasattr(response, 'id'))
+
+        maxipago_customer_id = response.id
+
+        response = self.maxipago.card.add(
+            customer_id=maxipago_customer_id,
+            number=u'4111111111111111',
+            expiration_month=u'02',
+            expiration_year=date.today().year + 3,
+            billing_name=u'Fulano de Tal',
+            billing_address1=u'Rua das Alamedas, 123',
+            billing_city=u'Rio de Janeiro',
+            billing_state=u'RJ',
+            billing_zip=u'20123456',
+            billing_country=u'BR',
+            billing_phone=u'552140634666',
+            billing_email=u'[email protected]',
+        )
+
+        self.assertTrue(getattr(response, 'token', False))
+        REFERENCE = randint(1, 100000)
+
+        response = self.maxipago.payment.direct(
+            processor_id=payment_processors.TEST,
+            reference_num=REFERENCE,
+
+            customer_id=maxipago_customer_id,
+            token=response.token,
+
+            charge_total='100.00',
+        )
+
+        self.assertTrue(response.authorized)
+        self.assertTrue(response.captured)
+
+    def test_payment_direct_with_token_decline(self):
+        CUSTOMER_ID = randint(1, 100000)
+
+        response = self.maxipago.customer.add(
+            customer_id=CUSTOMER_ID,
+            first_name=u'Fulano',
+            last_name=u'de Tal',
+        )
+
+        self.assertTrue(hasattr(response, 'id'))
+
+        maxipago_customer_id = response.id
+
+        response = self.maxipago.card.add(
+            customer_id=maxipago_customer_id,
+            number=u'4111111111111111',
+            expiration_month=u'02',
+            expiration_year=date.today().year + 3,
+            billing_name=u'Fulano de Tal',
+            billing_address1=u'Rua das Alamedas, 123',
+            billing_city=u'Rio de Janeiro',
+            billing_state=u'RJ',
+            billing_zip=u'20123456',
+            billing_country=u'BR',
+            billing_phone=u'552140634666',
+            billing_email=u'[email protected]',
+        )
+
+        self.assertTrue(getattr(response, 'token', False))
+        REFERENCE = randint(1, 100000)
+
+        response = self.maxipago.payment.direct(
+            processor_id=payment_processors.TEST,
+            reference_num=REFERENCE,
+
+            customer_id=maxipago_customer_id,
+            token=response.token,
+
+            charge_total='100.01',
+        )
+
+        self.assertFalse(response.authorized)
+        self.assertFalse(response.captured)
+
+
 if __name__ == '__main__':
     unittest.main()