[DRE-59]No se puede eliminar un PaymentSource (#72)

* fix delete payment sources issue

* add pip install nose

* add pip install -e .

Author: fcarrero

.github/workflows/test.yml

@@ -21,5 +21,8 @@ jobs:
       uses: actions/setup-python@v2
       with:
         python-version: ${{ matrix.python-version }}
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip && pip install -e .
     - name: Run tests
       run: nosetests

conekta/__init__.py

@@ -634,7 +634,7 @@ def instance_url(self):
         return "orders/%s/line_items" % (self.parent.id)
 
     def delete(self, params={}, api_key=None):
-        return super(LineItem, self).delete(params, self.parent.line_items)
+        return super(LineItem, self).delete(params, self.parent.line_items, api_key=api_key)
 
 
 class TaxLine(_CreatableResource, _UpdatableResource, _DeletableResource, _FindableResource, _EventableResource):
@@ -643,7 +643,7 @@ def instance_url(self):
         return "orders/%s/tax_lines" % (self.parent.id)
 
     def delete(self, params={}, api_key=None):
-        return super(TaxLine, self).delete(params, self.parent.tax_lines)
+        return super(TaxLine, self).delete(params, self.parent.tax_lines, api_key=api_key)
 
 
 class ShippingLine(_CreatableResource, _UpdatableResource, _DeletableResource, _FindableResource, _EventableResource):
@@ -652,7 +652,7 @@ def instance_url(self):
         return "orders/%s/shipping_lines" % (self.parent.id)
 
     def delete(self, params={}, api_key=None):
-        return super(ShippingLine, self).delete(params, self.parent.shipping_lines)
+        return super(ShippingLine, self).delete(params, self.parent.shipping_lines, api_key=api_key)
 
 
 class DiscountLine(_CreatableResource, _UpdatableResource, _DeletableResource, _FindableResource, _EventableResource):
@@ -661,15 +661,15 @@ def instance_url(self):
         return "orders/%s/discount_lines" % (self.parent.id)
 
     def delete(self, params={}, api_key=None):
-        return super(DiscountLine, self).delete(params, self.parent.discount_lines)
+        return super(DiscountLine, self).delete(params, self.parent.discount_lines, api_key=api_key)
 
 
 class PaymentSource(_CreatableResource, _UpdatableResource, _DeletableResource, _FindableResource):
     def instance_url(self):
         return "customers/%s/payment_sources" % (self.parent.id)
 
-    def delete(self, params={}, api_key=None):
-        return super(PaymentSource, self).delete(params, self.parent.payment_sources)
+    def delete(self, params={}, list_to_remove=None, uri=None, api_key=None):
+        return super(PaymentSource, self).delete(params, self.parent.payment_sources, api_key=api_key)
 
     def events(self, params={}, api_key=None):
         uri = "%s/payment_sources/%s/events" % (
@@ -689,7 +689,7 @@ def update(self, params={}, api_key=None):
 
     def delete(self, params={}, api_key=None):
         uri = "%s/shipping_contacts/%s" % (self.parent.instance_url(), self.id)
-        return super(ShippingContact, self).delete(params, self.parent.shipping_contacts, uri)
+        return super(ShippingContact, self).delete(params, self.parent.shipping_contacts, uri, api_key=api_key)
 
     def events(self, params={}, api_key=None):
         uri = "%s/shipping_contacts/%s/events" % (

tests/test_customer.py

@@ -75,3 +75,12 @@ def test_09_customer_update_plan(self):
         customer2 = self.client.Customer.find(customer.id)
 
         assert customer2.subscription.plan_id == "opal-plan"
+
+    def test_21_delete_payment_source(self):
+        self.client.api_key = 'key_ZLy4aP2szht1HqzkCezDEA'
+        customer = self.client.Customer.create(self.customer_object.copy())
+        payment_source = customer.payment_sources[0]
+
+        response = payment_source.delete()
+
+        assert payment_source.id == response.id

tests/test_orders.py

@@ -283,8 +283,7 @@ def test_19_order_create_with_customer_id(self):
         order_params["customer_info"] = {}
         order_params["customer_info"]["customer_id"] = customer.id
 
-        charge_params = {}
-        charge_params["payment_method"] = {}
+        charge_params = {"payment_method": {}}
         charge_params["payment_method"]["type"] = "card"
         charge_params["payment_method"]["token_id"] = "tok_test_visa_4242"
         order_params["charges"] = [charge_params]