Upgrade djstripe (#12148)

Closes #9706

Author: humitos

readthedocs/gold/tests/test_signals.py

@@ -1,3 +1,4 @@
+import requests_mock
 from unittest import mock
 
 import django_dynamic_fixture as fixture
@@ -11,44 +12,21 @@ class GoldSignalTests(TestCase):
     def setUp(self):
         self.user = fixture.get(User)
 
-        # Mocking
-        self.patches = {}
-        self.mocks = {}
-        self.patches["requestor"] = mock.patch("stripe.api_requestor.APIRequestor")
-
-        for patch in self.patches:
-            self.mocks[patch] = self.patches[patch].start()
-
-        self.mocks["request"] = self.mocks["requestor"].return_value
-
-    def tearDown(self):
-        # TODO: refactor to use request_mocks for these. The current mock
-        # pattern breaks tests ran after these if tearDown is not called with
-        # the `.stop` for all the patches. It took me a lot of time to realized
-        # about this problem.
-        for _, patch in self.patches.items():
-            patch.stop()
-
-    def mock_request(self, resp=None):
-        if resp is None:
-            resp = ({}, "")
-        self.mocks["request"].request = mock.Mock(side_effect=resp)
-
-    def test_delete_subscription(self):
+    @requests_mock.Mocker(kw="mock_request")
+    def test_delete_subscription(self, mock_request):
         subscription = fixture.get(GoldUser, user=self.user, stripe_id="cus_123")
         self.assertIsNotNone(subscription)
-        self.mock_request(
-            [
-                ({"id": "cus_123", "object": "customer"}, ""),
-                ({"deleted": True, "customer": "cus_123"}, ""),
-            ]
+        mock_request.get(
+            "https://api.stripe.com/v1/customers/cus_123",
+            json={"id": "cus_123", "object": "customer"},
+        )
+        mock_request.delete(
+            "https://api.stripe.com/v1/customers/cus_123",
+            json={"deleted": True, "customer": "cus_123"},
         )
 
         subscription.delete()
-
-        self.mocks["request"].request.assert_has_calls(
-            [
-                mock.call("get", "/v1/customers/cus_123", {}, mock.ANY),
-                mock.call("delete", "/v1/customers/cus_123", {}, mock.ANY),
-            ]
-        )
+        assert mock_request.request_history[0]._request.method == "GET"
+        assert mock_request.request_history[0]._request.url == "https://api.stripe.com/v1/customers/cus_123"
+        assert mock_request.request_history[1]._request.method == "DELETE"
+        assert mock_request.request_history[1]._request.url == "https://api.stripe.com/v1/customers/cus_123"

readthedocs/subscriptions/tests/test_event_handlers.py

@@ -430,16 +430,3 @@ def test_subscription_precedence(self):
 
             self.organization.refresh_from_db()
             assert self.organization.stripe_subscription == stripe_subscription
-
-    def test_register_events(self):
-        def test_func():
-            pass
-
-        with override_settings(RTD_ALLOW_ORGANIZATIONS=False):
-            event_handlers.handler("event")(test_func)
-
-        self.assertEqual(webhooks.registrations["event"], [])
-
-        with override_settings(RTD_ALLOW_ORGANIZATIONS=True):
-            event_handlers.handler("event")(test_func)
-        self.assertEqual(webhooks.registrations["event"], [test_func])

requirements/deploy.txt

@@ -103,7 +103,7 @@ distlib==0.3.9
     #   virtualenv
 dj-pagination==2.5.0
     # via -r requirements/pip.txt
-dj-stripe==2.6.3
+dj-stripe==2.9.0
     # via -r requirements/pip.txt
 django==4.2.20
     # via
@@ -244,9 +244,7 @@ jmespath==1.0.1
     #   boto3
     #   botocore
 jsonfield==3.1.0
-    # via
-    #   -r requirements/pip.txt
-    #   dj-stripe
+    # via -r requirements/pip.txt
 kombu==5.5.3
     # via
     #   -r requirements/pip.txt
@@ -405,7 +403,7 @@ sqlparse==0.5.3
     #   django-debug-toolbar
 stack-data==0.6.3
     # via ipython
-stripe==4.2.0
+stripe==11.6.0
     # via
     #   -r requirements/pip.txt
     #   dj-stripe
@@ -434,6 +432,7 @@ typing-extensions==4.13.2
     #   pydantic
     #   pydantic-core
     #   pygithub
+    #   stripe
     #   typing-inspection
 typing-inspection==0.4.0
     # via

requirements/docker.txt

@@ -111,7 +111,7 @@ distlib==0.3.9
     #   virtualenv
 dj-pagination==2.5.0
     # via -r requirements/pip.txt
-dj-stripe==2.6.3
+dj-stripe==2.9.0
     # via -r requirements/pip.txt
 django==4.2.20
     # via
@@ -257,9 +257,7 @@ jmespath==1.0.1
     #   boto3
     #   botocore
 jsonfield==3.1.0
-    # via
-    #   -r requirements/pip.txt
-    #   dj-stripe
+    # via -r requirements/pip.txt
 kombu==5.5.3
     # via
     #   -r requirements/pip.txt
@@ -433,7 +431,7 @@ sqlparse==0.5.3
     #   django-debug-toolbar
 stack-data==0.6.3
     # via ipython
-stripe==4.2.0
+stripe==11.6.0
     # via
     #   -r requirements/pip.txt
     #   dj-stripe
@@ -461,6 +459,7 @@ typing-extensions==4.13.2
     #   pydantic
     #   pydantic-core
     #   pygithub
+    #   stripe
     #   typing-inspection
 typing-inspection==0.4.0
     # via

requirements/pip.in

@@ -79,19 +79,14 @@ orjson
 django-gravatar2
 pytz
 
-# We cannot upgrade it until dj-stripe fixes this issue
-# https://github.com/dj-stripe/dj-stripe/issues/1842
-stripe==4.2.0
-
 regex
 
-# NOTE: djstripe overrides the stripe api version
-# globally, when updating make sure our code is compatible
-# with the version used by djstripe.
-# See STRIPE_API_VERSION https://dj-stripe.dev/reference/settings/.
-# dj-stripe==2.7.0 requires some extra actions to be upgrade
-# See https://github.com/readthedocs/readthedocs.org/issues/9706
-dj-stripe==2.6.3
+# Stripe is pinned because 12.0.1 fails with
+#
+# AttributeError: module 'stripe' has no attribute 'UsageRecord'
+stripe<12
+
+dj-stripe==2.9.0
 
 markdown
 

requirements/pip.txt

@@ -66,7 +66,7 @@ distlib==0.3.9
     # via virtualenv
 dj-pagination==2.5.0
     # via -r requirements/pip.in
-dj-stripe==2.6.3
+dj-stripe==2.9.0
     # via -r requirements/pip.in
 django==4.2.20
     # via
@@ -187,9 +187,7 @@ jmespath==1.0.1
     #   boto3
     #   botocore
 jsonfield==3.1.0
-    # via
-    #   -r requirements/pip.in
-    #   dj-stripe
+    # via -r requirements/pip.in
 kombu==5.5.3
     # via celery
 lexid==2021.1006
@@ -296,7 +294,7 @@ sqlparse==0.5.3
     # via
     #   django
     #   django-debug-toolbar
-stripe==4.2.0
+stripe==11.6.0
     # via
     #   -r requirements/pip.in
     #   dj-stripe
@@ -315,6 +313,7 @@ typing-extensions==4.13.2
     #   pydantic
     #   pydantic-core
     #   pygithub
+    #   stripe
     #   typing-inspection
 typing-inspection==0.4.0
     # via pydantic

requirements/testing.txt

@@ -108,7 +108,7 @@ distlib==0.3.9
     #   virtualenv
 dj-pagination==2.5.0
     # via -r requirements/pip.txt
-dj-stripe==2.6.3
+dj-stripe==2.9.0
     # via -r requirements/pip.txt
 django==4.2.20
     # via
@@ -253,9 +253,7 @@ jmespath==1.0.1
     #   boto3
     #   botocore
 jsonfield==3.1.0
-    # via
-    #   -r requirements/pip.txt
-    #   dj-stripe
+    # via -r requirements/pip.txt
 kombu==5.5.3
     # via
     #   -r requirements/pip.txt
@@ -445,7 +443,7 @@ sqlparse==0.5.3
     #   -r requirements/pip.txt
     #   django
     #   django-debug-toolbar
-stripe==4.2.0
+stripe==11.6.0
     # via
     #   -r requirements/pip.txt
     #   dj-stripe
@@ -468,6 +466,7 @@ typing-extensions==4.13.2
     #   pydantic-core
     #   pygithub
     #   sphinx
+    #   stripe
     #   typing-inspection
 typing-inspection==0.4.0
     # via