Merge pull request #417 from blueyed/fix-admin-customer_user

admin: customer_user: handle customer without user

pinax/stripe/admin.py

@@ -261,6 +261,8 @@ def customer_has_card(obj):
 
 
 def customer_user(obj):
+    if not obj.customer.user:
+        return ""
     User = get_user_model()
     username = getattr(obj.customer.user, User.USERNAME_FIELD)
     email = getattr(obj, "email", "")

pinax/stripe/tests/test_admin.py

@@ -2,7 +2,7 @@
 
 import django
 from django.contrib.auth import get_user_model
-from django.test import Client, TestCase
+from django.test import Client, SimpleTestCase, TestCase
 from django.utils import timezone
 
 from ..models import Customer, Invoice, Plan, Subscription
@@ -146,3 +146,17 @@ def test_charge_admin(self):
         url = reverse("admin:pinax_stripe_charge_changelist")
         response = self.client.get(url)
         self.assertEqual(response.status_code, 200)
+
+
+class AdminSimpleTestCase(SimpleTestCase):
+
+    def test_customer_user_without_user(self):
+        from ..admin import customer_user
+
+        class CustomerWithoutUser(object):
+            user = None
+
+        class Obj(object):
+            customer = CustomerWithoutUser()
+
+        self.assertEqual(customer_user(Obj()), "")