Add event querying and payment_intent events

These features are technically separate but it's easier to test them together.

The Stripe event query API lets us filter by event type and timestamp.

Meanwhile payment_intent successes and failures generate events (this is
particularly important for sepa_direct_debit which on the real Stripe API
is typically pending for days, and in localstripe is pending for 0.5
seconds).

Author: Ben Creech

localstripe/resources.py

@@ -1115,6 +1115,47 @@ def _api_update(cls, id, **data):
     def _api_delete(cls, id):
         raise UserError(405, 'Method Not Allowed')
 
+    @classmethod
+    def _api_list_all(cls, url, type=None, created=None, limit=None,
+                      starting_after=None, **kwargs):
+        if kwargs:
+            raise UserError(400, 'Unexpected ' + ', '.join(kwargs.keys()))
+
+        filters = []
+        try:
+            if type is not None:
+                assert _type(type) is str
+                filters.append(lambda obj: obj.type == type)
+            if created is not None:
+                assert _type(created) is dict
+                gt = try_convert_to_int(created.pop('gt', None))
+                if gt is not None:
+                    filters.append(lambda obj: obj.created > gt)
+
+                gte = try_convert_to_int(created.pop('gte', None))
+                if gte is not None:
+                    filters.append(lambda obj: obj.created >= gte)
+
+                lt = try_convert_to_int(created.pop('lt', None))
+                if lt is not None:
+                    filters.append(lambda obj: obj.created < lt)
+
+                lte = try_convert_to_int(created.pop('lte', None))
+                if lte is not None:
+                    filters.append(lambda obj: obj.created <= lte)
+
+                assert not created  # no other params are supported
+        except AssertionError:
+            raise UserError(400, 'Bad request')
+
+        li = super()._api_list_all(
+            url, limit=limit, starting_after=starting_after
+        )
+
+        li._list = [obj for obj in li._list if all(f(obj) for f in filters)]
+
+        return li
+
 
 class Invoice(StripeObject):
     object = 'invoice'
@@ -1877,18 +1918,21 @@ def __init__(self, amount=None, currency=None, customer=None,
         self._authentication_failed = False
 
     def _on_success(self):
+        schedule_webhook(Event('payment_intent.succeeded', self))
         if self.invoice:
             invoice = Invoice._api_retrieve(self.invoice)
             invoice._on_payment_success()
 
     def _report_failure(self):
+        schedule_webhook(Event('payment_intent.payment_failed', self))
         if self.invoice:
             invoice = Invoice._api_retrieve(self.invoice)
             invoice._on_payment_failure_now()
 
         self.latest_charge._raise_failure()
 
-    def _on_failure_later(self):
+    def _report_async_failure(self):
+        schedule_webhook(Event('payment_intent.payment_failed', self))
         if self.invoice:
             invoice = Invoice._api_retrieve(self.invoice)
             invoice._on_payment_failure_later()
@@ -1905,7 +1949,7 @@ def _create_charge(self, on_failure_now):
         charge.payment_intent = self.id
         self.latest_charge = charge
         charge._initialize_charge(self._on_success, on_failure_now,
-                                  self._on_failure_later)
+                                  self._report_async_failure)
 
     @property
     def status(self):

test.sh

@@ -1068,6 +1068,12 @@ payment_intent=$(
        -d capture_method=manual \
   | grep -oE 'pi_\w+' | head -n 1)
 
+# we don't get a payment_intent.succeeded event from the pre-auth:
+succeded_event=$(
+  curl -sSfg -u $SK: "$HOST/v1/events?type=payment_intent.succeeded" \
+  | grep -oE "^          \"id\": \"$payment_intent\"" || true)
+[ -z "$succeded_event" ]
+
 # payment_intent was not captured
 captured=$(
   curl -sSfg -u $SK: $HOST/v1/payment_intents/$payment_intent \
@@ -1088,6 +1094,12 @@ captured=$(
   | grep -oE '"status": "succeeded"')
 [ -n "$captured" ]
 
+# we do get a payment_intent.succeeded event from the capture:
+succeded_event=$(
+  curl -sSfg -u $SK: "$HOST/v1/events?type=payment_intent.succeeded" \
+  | grep -oE "^          \"id\": \"$payment_intent\"" || true)
+[ -n "$succeded_event" ]
+
 # difference between pre-auth and capture is refunded
 refunded=$(
   curl -sSfg -u $SK: $HOST/v1/payment_intents/$payment_intent \
@@ -1165,6 +1177,37 @@ code=$(
        -X POST -o /dev/null -w "%{http_code}")
 [ "$code" = 402 ]
 
+# we get a payment_intent.payment_failed event:
+failed_event=$(
+  curl -sSfg -u $SK: "$HOST/v1/events?type=payment_intent.payment_failed" \
+  | grep -oE "^          \"id\": \"$payment_intent\"" || true)
+[ -n "$failed_event" ]
+
+# we don't get a payment_intent.succeeded event:
+cus=$(curl -sSfg -u $SK: $HOST/v1/customers \
+          -d email=james.robinson@example.com \
+      | grep -oE 'cus_\w+' | head -n 1)
+
+## test event timestamp filtering:
+first_created=$(
+  curl -sSfg -u $SK: "$HOST/v1/events" \
+  | grep -oP -m 1 'created": \K([0-9]+)' || true)
+[ -n "$first_created" ]
+
+total_count=$(curl -sSfg -u $SK: $HOST/v1/events \
+              | grep -oP '^  "total_count": \K([0-9]+)')
+[ "$total_count" -gt 1 ]
+
+count=$(
+  curl -sSfg -u $SK: "$HOST/v1/events?created[lte]=$first_created" \
+  | grep -oP '^  "total_count": \K([0-9]+)')
+[ "$count" -le "$total_count" ]
+
+count=$(
+  curl -sSfg -u $SK: "$HOST/v1/events?created[lte]=$first_created&created[gt]=9999999999" \
+  | grep -oP '^  "total_count": \K([0-9]+)')
+[ "$count" -eq 0 ]
+
 # Create a customer with card 4000000000000341 (that fails upon payment) and
 # make sure creating the subscription doesn't fail (although it creates it with
 # status 'incomplete'). This how Stripe behaves, see