mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-07-26 20:48:40 +02:00
linter fixes
This commit is contained in:
parent
9c069a8e64
commit
09ba7450b3
2 changed files with 6 additions and 6 deletions
|
@ -11,7 +11,7 @@ class Command(BaseCommand, PopulateScriptTemplate):
|
||||||
help = "Loops through each domain request object and populates the last_status_update and first_submitted_date"
|
help = "Loops through each domain request object and populates the last_status_update and first_submitted_date"
|
||||||
|
|
||||||
def handle(self, **kwargs):
|
def handle(self, **kwargs):
|
||||||
"""Loops through each DomainRequest object and populates
|
"""Loops through each DomainRequest object and populates
|
||||||
its last_status_update and first_submitted_date values"""
|
its last_status_update and first_submitted_date values"""
|
||||||
self.mass_update_records(DomainRequest, None, ["last_status_update", "first_submitted_date"])
|
self.mass_update_records(DomainRequest, None, ["last_status_update", "first_submitted_date"])
|
||||||
|
|
||||||
|
@ -34,12 +34,12 @@ class Command(BaseCommand, PopulateScriptTemplate):
|
||||||
break
|
break
|
||||||
|
|
||||||
logger.info(
|
logger.info(
|
||||||
f"""{TerminalColors.OKCYAN}Updating {record} =>
|
f"""{TerminalColors.OKCYAN}Updating {record} =>
|
||||||
first submitted date: {record.first_submitted_date},
|
first submitted date: {record.first_submitted_date},
|
||||||
last status update: {record.last_status_update}{TerminalColors.ENDC}
|
last status update: {record.last_status_update}{TerminalColors.ENDC}
|
||||||
"""
|
"""
|
||||||
)
|
)
|
||||||
|
|
||||||
def should_skip_record(self, record) -> bool:
|
def should_skip_record(self, record) -> bool:
|
||||||
# make sure the record had some kind of history
|
# make sure the record had some kind of history
|
||||||
return not LogEntry.objects.filter(object_pk=record.pk).exists()
|
return not LogEntry.objects.filter(object_pk=record.pk).exists()
|
||||||
|
|
|
@ -287,7 +287,7 @@ class GetRequestsJsonTest(TestWithUser, WebTest):
|
||||||
|
|
||||||
def test_sorting(self):
|
def test_sorting(self):
|
||||||
"""test that sorting works properly on the result set"""
|
"""test that sorting works properly on the result set"""
|
||||||
response = self.app.get(reverse("get_domain_requests_json"),
|
response = self.app.get(reverse("get_domain_requests_json"),
|
||||||
{"sort_by": "last_submitted_date", "order": "desc"})
|
{"sort_by": "last_submitted_date", "order": "desc"})
|
||||||
self.assertEqual(response.status_code, 200)
|
self.assertEqual(response.status_code, 200)
|
||||||
data = response.json
|
data = response.json
|
||||||
|
@ -307,7 +307,7 @@ class GetRequestsJsonTest(TestWithUser, WebTest):
|
||||||
def test_filter_approved_excluded(self):
|
def test_filter_approved_excluded(self):
|
||||||
"""test that approved requests are excluded from result set."""
|
"""test that approved requests are excluded from result set."""
|
||||||
# sort in reverse chronological order of submission date, since most recent request is approved
|
# sort in reverse chronological order of submission date, since most recent request is approved
|
||||||
response = self.app.get(reverse("get_domain_requests_json"),
|
response = self.app.get(reverse("get_domain_requests_json"),
|
||||||
{"sort_by": "last_submitted_date", "order": "desc"})
|
{"sort_by": "last_submitted_date", "order": "desc"})
|
||||||
self.assertEqual(response.status_code, 200)
|
self.assertEqual(response.status_code, 200)
|
||||||
data = response.json
|
data = response.json
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue