Changeset 12453 for main/waeup.ikoba/trunk/src/waeup/ikoba/payments

Timestamp:

12 Jan 2015, 14:33:22 (10 years ago)

Author:

uli

Message:

Offer a possibility to store credit card data.

Location:

main/waeup.ikoba/trunk/src/waeup/ikoba/payments

Files:

• paypal.py (modified) (1 diff)
• tests/test_paypal.py (modified) (3 diffs)

main/waeup.ikoba/trunk/src/waeup/ikoba/payments/paypal.py

@@ -1139 +1139 @@
         return site['creditcards'].get(payer_id, None)
 
+    def store_credit_card(self, paypal_credit_card):
+        """Store `paypal_credit_card` in vault.
+
+        Returns a credit card token in case of success.
+
+        `paypal_credit_card` must provide `ICreditCard`. The credit
+        card created is stored at PayPal and gets a token stored
+        locally.
+
+        If no `creditcards` are available in local site, we create a
+        container.
+        """
+        site = grok.getSite()
+        if not 'creditcards' in site:
+            site['creditcards'] = grok.Container()
+        return
+
     def create_payment(self, payer, payment_item,  payee=None):
         if not IPayer.providedBy(payer):

main/waeup.ikoba/trunk/src/waeup/ikoba/payments/tests/test_paypal.py

@@ -1100 +1100 @@
         return site
 
+    def get_credit_card(self):
+        # get a valid credit card
+        credit_card = CreditCard(
+            credit_card_type=u"visa",
+            payer_id=u"PAYER_0123456789012345678901",
+            number=u"4417119669820331",
+            expire_month=11,
+            expire_year=2018,
+            cvv2=u"874",
+            first_name=u"Joe",
+            last_name=u"Shopper",
+            )
+        return credit_card
+
     def test_iface(self):
         # we fullfill all interface contracts
@@ -1127 +1141 @@
     def test_get_credit_card(self):
         # we can get a credit card, if one was stored
-        class MyCard(grok.Model):
+        class MyCard(object):
             myid = 'CARD1'
 
@@ -1136 +1150 @@
         site['creditcards'] = grok.Container()
         card = MyCard()
-        card.myid = 'CHANGED ID'
+        card.myid = 'CHANGED CARD ID'
         site['creditcards'][u'CARD1'] = card
         service = PayPalCreditCardService()
         assert service.get_credit_card(u'CARD1') == card
         assert service.get_credit_card(u'CARD2') is None
+
+    def test_store_credit_card_no_credcard_container(self):
+        # a missing creditcards container is created on-the-fly
+        site = self.create_site()
+        assert 'creditcards' not in site
+        service = PayPalCreditCardService()
+        credit_card = self.get_credit_card()
+        service.store_credit_card(credit_card)
+        assert 'creditcards' in site
 
     def test_create_payment_no_credictard(self):