From d286fb6709a8013d7411b2a8457b958fd18f1f10 Mon Sep 17 00:00:00 2001 From: zandercymatics <141044360+zandercymatics@users.noreply.github.com> Date: Thu, 28 Sep 2023 08:01:37 -0600 Subject: [PATCH] Lint --- src/registrar/models/domain.py | 10 ++-------- src/registrar/tests/test_views.py | 9 ++------- 2 files changed, 4 insertions(+), 15 deletions(-) diff --git a/src/registrar/models/domain.py b/src/registrar/models/domain.py index 0e3cff325..03544e5c9 100644 --- a/src/registrar/models/domain.py +++ b/src/registrar/models/domain.py @@ -652,12 +652,8 @@ class Domain(TimeStampedModel, DomainHelper): def isActive(self): return self.state == Domain.State.CREATED - def map_epp_contact_to_public_contact( - self, - contact: eppInfo.InfoContactResultData, - contact_id, - contact_type + self, contact: eppInfo.InfoContactResultData, contact_id, contact_type ): """Maps the Epp contact representation to a PublicContact object. @@ -827,9 +823,7 @@ class Domain(TimeStampedModel, DomainHelper): # If the for loop didn't do a return, # then we know that it doesn't exist within cache - logger.info( - f"Requested contact {contact.registry_id} does not exist in cache." - ) + logger.info(f"Requested contact {contact.registry_id} does not exist in cache.") return None # ForeignKey on UserDomainRole creates a "permissions" member for diff --git a/src/registrar/tests/test_views.py b/src/registrar/tests/test_views.py index c71669dfd..996d49792 100644 --- a/src/registrar/tests/test_views.py +++ b/src/registrar/tests/test_views.py @@ -1414,11 +1414,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib): domain_contact, _ = Domain.objects.get_or_create(name="freeman.gov") # Add current user to this domain - _ = UserDomainRole( - user=self.user, - domain = domain_contact, - role = "admin" - ).save() + _ = UserDomainRole(user=self.user, domain=domain_contact, role="admin").save() page = self.client.get( reverse("domain-security-email", kwargs={"pk": domain_contact.id}) ) @@ -1434,7 +1430,7 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib): self.mockSendPatch = patch("registrar.models.domain.registry.send") self.mockedSendFunction = self.mockSendPatch.start() self.mockedSendFunction.side_effect = self.mockSend - + page = self.client.get( reverse("domain-security-email", kwargs={"pk": self.domain.id}) ) @@ -1454,7 +1450,6 @@ class TestDomainDetail(TestWithDomainPermissions, WebTest, MockEppLib): @skip("Ticket 912 needs to fix this one") def test_domain_security_email_form(self): """Adding a security email works. - Uses self.app WebTest because we need to interact with forms. """ security_email_page = self.app.get(