linter fixes

This commit is contained in:
matthewswspence 2024-12-17 13:03:22 -06:00
parent ec39b159ec
commit 24354fd7f1
No known key found for this signature in database
GPG key ID: FB458202A7852BA4
2 changed files with 5 additions and 5 deletions

View file

@ -2442,7 +2442,7 @@ class DomainRequestAdmin(ListHeaderAdmin, ImportExportModelAdmin):
obj_id = domain.id obj_id = domain.id
change_url = reverse("admin:%s_%s_change" % (app_label, model_name), args=[obj_id]) change_url = reverse("admin:%s_%s_change" % (app_label, model_name), args=[obj_id])
message = f"<li>The status of this domain request cannot be changed because it has been joined to a domain in Ready status: " # noqa message = f"<li>The status of this domain request cannot be changed because it has been joined to a domain in Ready status: " # noqa
message += f"<a href='{change_url}'>{domain}</a></li>" message += f"<a href='{change_url}'>{domain}</a></li>"
message_html = mark_safe(message) # nosec message_html = mark_safe(message) # nosec
@ -2450,7 +2450,7 @@ class DomainRequestAdmin(ListHeaderAdmin, ImportExportModelAdmin):
request, request,
message_html, message_html,
) )
obj = self.get_object(request, object_id) obj = self.get_object(request, object_id)
self.display_restricted_warning(request, obj) self.display_restricted_warning(request, obj)
@ -2564,6 +2564,7 @@ class DomainRequestAdmin(ListHeaderAdmin, ImportExportModelAdmin):
qs = qs.filter(portfolio=portfolio_id) qs = qs.filter(portfolio=portfolio_id)
return qs return qs
class TransitionDomainAdmin(ListHeaderAdmin): class TransitionDomainAdmin(ListHeaderAdmin):
"""Custom transition domain admin class.""" """Custom transition domain admin class."""

View file

@ -95,7 +95,6 @@ class TestDomainRequestAdmin(MockEppLib):
User.objects.all().delete() User.objects.all().delete()
AllowedEmail.objects.all().delete() AllowedEmail.objects.all().delete()
@less_console_noise_decorator @less_console_noise_decorator
def test_domain_request_senior_official_is_alphabetically_sorted(self): def test_domain_request_senior_official_is_alphabetically_sorted(self):
"""Tests if the senior offical dropdown is alphanetically sorted in the django admin display""" """Tests if the senior offical dropdown is alphanetically sorted in the django admin display"""
@ -1815,7 +1814,7 @@ class TestDomainRequestAdmin(MockEppLib):
request, request,
"Cannot edit a domain request with a restricted creator.", "Cannot edit a domain request with a restricted creator.",
) )
# @less_console_noise_decorator # @less_console_noise_decorator
def test_approved_domain_request_with_ready_domain_has_warning_message(self): def test_approved_domain_request_with_ready_domain_has_warning_message(self):
"""Tests if the domain request has a warning message when the approved domain is in Ready state""" """Tests if the domain request has a warning message when the approved domain is in Ready state"""
@ -1834,7 +1833,7 @@ class TestDomainRequestAdmin(MockEppLib):
with boto3_mocking.clients.handler_for("sesv2", self.mock_client): with boto3_mocking.clients.handler_for("sesv2", self.mock_client):
with patch("django.contrib.messages.warning") as mock_warning: with patch("django.contrib.messages.warning") as mock_warning:
# Create a request object # Create a request object
self.client.force_login(self.superuser) self.client.force_login(self.superuser)
self.client.get( self.client.get(
"/admin/registrar/domainrequest/{}/change/".format(domain_request.pk), "/admin/registrar/domainrequest/{}/change/".format(domain_request.pk),