Merge pull request #515 from blueyed/charge-outcome-upstream

Add Charge.outcome

pinax/stripe/actions/charges.py

@@ -203,6 +203,7 @@ def sync_charge_from_stripe_data(data):
         )
         obj.fee_currency = balance_transaction["currency"]
     obj.transfer_group = data.get("transfer_group")
+    obj.outcome = data.get("outcome")
     obj.save()
     return obj
 

pinax/stripe/migrations/0013_charge_outcome.py

@@ -0,0 +1,21 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.6 on 2017-11-22 15:40
+from __future__ import unicode_literals
+
+from django.db import migrations
+import jsonfield.fields
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('pinax_stripe', '0014_blank_with_null'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='charge',
+            name='outcome',
+            field=jsonfield.fields.JSONField(blank=True, null=True),
+        ),
+    ]

pinax/stripe/models.py

@@ -484,6 +484,7 @@ class Charge(StripeAccountFromCustomerMixin, StripeObject):
     fee_currency = models.CharField(max_length=10, null=True, blank=True)
 
     transfer_group = models.TextField(null=True, blank=True)
+    outcome = JSONField(null=True, blank=True)
 
     objects = ChargeManager()
 

pinax/stripe/tests/test_actions.py

@@ -2155,6 +2155,7 @@ def test_sync_charge_from_stripe_data_failed(self):
         charge = Charge.objects.get(stripe_id=data["id"])
         self.assertEqual(charge.amount, decimal.Decimal("2"))
         self.assertEqual(charge.customer, None)
+        self.assertEqual(charge.outcome["risk_level"], "normal")
 
     @patch("stripe.Subscription.retrieve")
     def test_retrieve_stripe_subscription(self, RetrieveMock):