From a95ff2bf53fc5e3c088a9556ec1e958ff76d1ca8 Mon Sep 17 00:00:00 2001 From: rachidatecs Date: Fri, 19 May 2023 10:48:34 -0400 Subject: [PATCH] add test --- src/registrar/tests/test_views.py | 6 ++++++ src/registrar/views/domain.py | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/src/registrar/tests/test_views.py b/src/registrar/tests/test_views.py index ee83034e1..959183f34 100644 --- a/src/registrar/tests/test_views.py +++ b/src/registrar/tests/test_views.py @@ -1095,6 +1095,12 @@ class TestDomainPermissions(TestWithDomainPermissions): ) self.assertEqual(response.status_code, 403) + with less_console_noise(): + response = self.client.get( + reverse("domain-security-email", kwargs={"pk": self.domain.id}) + ) + self.assertEqual(response.status_code, 403) + class TestDomainDetail(TestWithDomainPermissions, WebTest): def setUp(self): diff --git a/src/registrar/views/domain.py b/src/registrar/views/domain.py index d0fa8952e..b5962c398 100644 --- a/src/registrar/views/domain.py +++ b/src/registrar/views/domain.py @@ -130,7 +130,7 @@ class DomainSecurityEmailView(DomainPermission, FormMixin, DetailView): """The form is valid, call setter in model.""" # Set the security email from the form - new_email = form.cleaned_data["security_email"] + new_email = form.cleaned_data.get("security_email", "") domain = self.get_object() domain.set_security_email(new_email)