mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-05-29 17:00:02 +02:00
PR suggestions pt. 2
This commit is contained in:
parent
d934897323
commit
2f5559a72f
3 changed files with 3 additions and 3 deletions
|
@ -223,7 +223,7 @@ class CsvReportsTest(TestCase):
|
||||||
self.assertEqual(expected_file_content, response.content)
|
self.assertEqual(expected_file_content, response.content)
|
||||||
|
|
||||||
|
|
||||||
class ExportDataTest(MockEppLib):
|
class ExportDataTest(TestCase):
|
||||||
def setUp(self):
|
def setUp(self):
|
||||||
super().setUp()
|
super().setUp()
|
||||||
username = "test_user"
|
username = "test_user"
|
||||||
|
|
|
@ -86,7 +86,7 @@ class TestPatchAgencyInfo(TestCase):
|
||||||
def test_patch_agency_info_skip_updates_data(self):
|
def test_patch_agency_info_skip_updates_data(self):
|
||||||
"""
|
"""
|
||||||
Tests that the `patch_federal_agency_info` command logs a warning but
|
Tests that the `patch_federal_agency_info` command logs a warning but
|
||||||
updates the DomainInformation object, because an record exists in the
|
updates the DomainInformation object, because a record exists in the
|
||||||
provided current-full.csv file.
|
provided current-full.csv file.
|
||||||
"""
|
"""
|
||||||
# Set federal_agency to None to simulate a skip
|
# Set federal_agency to None to simulate a skip
|
||||||
|
|
|
@ -37,7 +37,7 @@ def write_row(writer, columns, domain_info):
|
||||||
if security_contacts:
|
if security_contacts:
|
||||||
security_email = security_contacts[0].email
|
security_email = security_contacts[0].email
|
||||||
|
|
||||||
invalid_emails = {"registrar@dotgov.gov", "dotgov@cisa.dhs.gov"}
|
invalid_emails = {"registrar@dotgov.gov"}
|
||||||
# These are default emails that should not be displayed in the csv report
|
# These are default emails that should not be displayed in the csv report
|
||||||
if security_email is not None and security_email.lower() in invalid_emails:
|
if security_email is not None and security_email.lower() in invalid_emails:
|
||||||
security_email = "(blank)"
|
security_email = "(blank)"
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue