Merge pull request #651 from stripe/remi/codegen-824d494

Add support for `billing_portal` namespace and `Session` resource and APIs

.travis.yml

@@ -26,7 +26,7 @@ cache:
 env:
   global:
     # If changing this number, please also change it in `tests/conftest.py`.
-    - STRIPE_MOCK_VERSION=0.76.0
+    - STRIPE_MOCK_VERSION=0.88.0
 
 before_install:
   # Unpack and start stripe-mock so that the test suite can talk to it

stripe/api_resources/__init__.py

@@ -5,6 +5,7 @@
 from stripe.api_resources.error_object import ErrorObject, OAuthErrorObject
 from stripe.api_resources.list_object import ListObject
 
+from stripe.api_resources import billing_portal
 from stripe.api_resources import checkout
 from stripe.api_resources import issuing
 from stripe.api_resources import radar

stripe/api_resources/billing_portal/__init__.py

@@ -0,0 +1,5 @@
+from __future__ import absolute_import, division, print_function
+
+# flake8: noqa
+
+from stripe.api_resources.billing_portal.session import Session

stripe/api_resources/billing_portal/session.py

@@ -0,0 +1,7 @@
+from __future__ import absolute_import, division, print_function
+
+from stripe.api_resources.abstract import CreateableAPIResource
+
+
+class Session(CreateableAPIResource):
+    OBJECT_NAME = "billing_portal.session"

stripe/object_classes.py

@@ -16,6 +16,7 @@
     api_resources.Balance.OBJECT_NAME: api_resources.Balance,
     api_resources.BalanceTransaction.OBJECT_NAME: api_resources.BalanceTransaction,
     api_resources.BankAccount.OBJECT_NAME: api_resources.BankAccount,
+    api_resources.billing_portal.Session.OBJECT_NAME: api_resources.billing_portal.Session,
     api_resources.BitcoinReceiver.OBJECT_NAME: api_resources.BitcoinReceiver,
     api_resources.BitcoinTransaction.OBJECT_NAME: api_resources.BitcoinTransaction,
     api_resources.Capability.OBJECT_NAME: api_resources.Capability,

tests/api_resources/billing_portal/test_session.py

@@ -0,0 +1,15 @@
+from __future__ import absolute_import, division, print_function
+
+import stripe
+
+
+TEST_RESOURCE_ID = "pts_123"
+
+
+class TestSession(object):
+    def test_is_creatable(self, request_mock):
+        resource = stripe.billing_portal.Session.create(
+            customer="cus_123", return_url="https://stripe.com/return"
+        )
+        request_mock.assert_requested("post", "/v1/billing_portal/sessions")
+        assert isinstance(resource, stripe.billing_portal.Session)

tests/api_resources/issuing/test_card.py

@@ -45,14 +45,30 @@ def test_is_saveable(self, request_mock):
         assert resource is card
 
     def test_can_retrieve_details(self, request_mock):
-        resource = stripe.issuing.Card.retrieve(TEST_RESOURCE_ID)
-        card_details = resource.details()
+        # stripe-mock does not handle this anymore so we stub instead.
+        request_mock.stub_request(
+            "get",
+            "/v1/issuing/cards/%s/details" % TEST_RESOURCE_ID,
+            {"object": "issuing.card_details"},
+        )
+        card = stripe.issuing.Card.construct_from(
+            {"id": "%s" % TEST_RESOURCE_ID, "object": "issuing.card"},
+            stripe.api_key,
+        )
+        card_details = card.details()
         request_mock.assert_requested(
             "get", "/v1/issuing/cards/%s/details" % TEST_RESOURCE_ID
         )
         assert isinstance(card_details, stripe.issuing.CardDetails)
 
     def test_can_retrieve_details_classmethod(self, request_mock):
+        # stripe-mock does not handle this anymore so we stub instead.
+        request_mock.stub_request(
+            "get",
+            "/v1/issuing/cards/%s/details" % TEST_RESOURCE_ID,
+            {"object": "issuing.card_details"},
+        )
+
         card_details = stripe.issuing.Card.details(TEST_RESOURCE_ID)
         request_mock.assert_requested(
             "get", "/v1/issuing/cards/%s/details" % TEST_RESOURCE_ID

tests/api_resources/issuing/test_dispute.py

@@ -8,9 +8,7 @@
 
 class TestDispute(object):
     def test_is_creatable(self, request_mock):
-        resource = stripe.issuing.Dispute.create(
-            reason="fraudulent", disputed_transaction="ipi_123"
-        )
+        resource = stripe.issuing.Dispute.create()
         request_mock.assert_requested("post", "/v1/issuing/disputes")
         assert isinstance(resource, stripe.issuing.Dispute)
 
@@ -21,9 +19,7 @@ def test_is_listable(self, request_mock):
         assert isinstance(resources.data[0], stripe.issuing.Dispute)
 
     def test_is_modifiable(self, request_mock):
-        resource = stripe.issuing.Dispute.modify(
-            TEST_RESOURCE_ID, metadata={"key": "value"}
-        )
+        resource = stripe.issuing.Dispute.modify(TEST_RESOURCE_ID)
         request_mock.assert_requested(
             "post", "/v1/issuing/disputes/%s" % TEST_RESOURCE_ID
         )
@@ -35,13 +31,3 @@ def test_is_retrievable(self, request_mock):
             "get", "/v1/issuing/disputes/%s" % TEST_RESOURCE_ID
         )
         assert isinstance(resource, stripe.issuing.Dispute)
-
-    def test_is_saveable(self, request_mock):
-        resource = stripe.issuing.Dispute.retrieve(TEST_RESOURCE_ID)
-        resource.metadata["key"] = "value"
-        dispute = resource.save()
-        request_mock.assert_requested(
-            "post", "/v1/issuing/disputes/%s" % TEST_RESOURCE_ID
-        )
-        assert isinstance(resource, stripe.issuing.Dispute)
-        assert resource is dispute

tests/conftest.py

@@ -16,7 +16,7 @@
 
 
 # When changing this number, don't forget to change it in `.travis.yml` too.
-MOCK_MINIMUM_VERSION = "0.76.0"
+MOCK_MINIMUM_VERSION = "0.88.0"
 
 # Starts stripe-mock if an OpenAPI spec override is found in `openapi/`, and
 # otherwise fall back to `STRIPE_MOCK_PORT` or 12111.