Test owner email

Author: spalmurray

billing/tests/test_views.py

@@ -194,6 +194,7 @@ def test_invoice_payment_failed_sends_email_to_admins(self, mocked_send_email):
         admin_2 = OwnerFactory(email="[email protected]")
         self.owner.admins = [admin_1.ownerid, admin_2.ownerid]
         self.owner.plan_activated_users = [non_admin.ownerid]
+        self.owner.email = "[email protected]"
         self.owner.save()
 
         response = self._send_event(
@@ -218,6 +219,17 @@ def test_invoice_payment_failed_sends_email_to_admins(self, mocked_send_email):
         assert self.owner.delinquent is True
 
         expected_calls = [
+            call(
+                to_addr=self.owner.email,
+                subject="Your Codecov payment failed",
+                template_name="failed-payment",
+                name=self.owner.username,
+                amount=240,
+                card_type="visa",
+                last_four=1234,
+                cta_link="https://stripe.com",
+                date=datetime.now().strftime("%B %-d, %Y"),
+            ),
             call(
                 to_addr=admin_1.email,
                 subject="Your Codecov payment failed",
@@ -252,6 +264,7 @@ def test_invoice_payment_failed_sends_email_to_admins_no_card(
         admin_2 = OwnerFactory(email="[email protected]")
         self.owner.admins = [admin_1.ownerid, admin_2.ownerid]
         self.owner.plan_activated_users = [non_admin.ownerid]
+        self.owner.email = "[email protected]"
         self.owner.save()
 
         response = self._send_event(
@@ -274,6 +287,17 @@ def test_invoice_payment_failed_sends_email_to_admins_no_card(
         assert self.owner.delinquent is True
 
         expected_calls = [
+            call(
+                to_addr=self.owner.email,
+                subject="Your Codecov payment failed",
+                template_name="failed-payment",
+                name=self.owner.username,
+                amount=240,
+                card_type=None,
+                last_four=None,
+                cta_link="https://stripe.com",
+                date=datetime.now().strftime("%B %-d, %Y"),
+            ),
             call(
                 to_addr=admin_1.email,
                 subject="Your Codecov payment failed",

billing/views.py

@@ -73,7 +73,7 @@ def invoice_payment_failed(self, invoice: stripe.Invoice) -> None:
 
             # Add the owner's email as well - for user owners, admins is empty.
             if owner.email:
-                admin_ids.add(owner)
+                admin_ids.add(owner.ownerid)
 
         admins: QuerySet[Owner] = Owner.objects.filter(pk__in=admin_ids)