Requested changes

This commit is contained in:
zandercymatics 2023-09-29 14:41:52 -06:00
parent 596f714428
commit 228d37ef39
No known key found for this signature in database
GPG key ID: FF4636ABEC9682B7
3 changed files with 15 additions and 14 deletions

View file

@ -728,14 +728,14 @@ class DomainAdmin(ListHeaderAdmin):
obj.deletedInEpp() obj.deletedInEpp()
obj.save() obj.save()
except RegistryError as err: except RegistryError as err:
# To get past the linter.. # Using variables to get past the linter
l1 = f"Cannot delete Domain when in status {obj.status}" message1 = f"Cannot delete Domain when in status {obj.status}"
l2 = "This subdomain is being used as a hostname on another domain" message2 = "This subdomain is being used as a hostname on another domain"
# Human-readable mappings of ErrorCodes. Can be expanded. # Human-readable mappings of ErrorCodes. Can be expanded.
error_messages = { error_messages = {
# noqa on these items as black wants to reformat to an invalid length # noqa on these items as black wants to reformat to an invalid length
ErrorCode.OBJECT_STATUS_PROHIBITS_OPERATION: l1, ErrorCode.OBJECT_STATUS_PROHIBITS_OPERATION: message1,
ErrorCode.OBJECT_ASSOCIATION_PROHIBITS_OPERATION: l2, ErrorCode.OBJECT_ASSOCIATION_PROHIBITS_OPERATION: message2,
} }
message = "Cannot connect to the registry" message = "Cannot connect to the registry"
@ -756,7 +756,8 @@ class DomainAdmin(ListHeaderAdmin):
self.message_user( self.message_user(
request, request,
"Error deleting this Domain: " "Error deleting this Domain: "
f"Can't switch from state '{obj.state}' to 'deleted'", f"Can't switch from state '{obj.state}' to 'deleted'"
", must be either 'dns_needed' or 'on_hold'",
messages.ERROR, messages.ERROR,
) )
except Exception: except Exception:

View file

@ -15,7 +15,7 @@
{% endif %} {% endif %}
<input id="manageDomainSubmitButton" type="submit" value="Manage Domain" name="_edit_domain"> <input id="manageDomainSubmitButton" type="submit" value="Manage Domain" name="_edit_domain">
<input type="submit" value="get status" name="_get_status"> <input type="submit" value="get status" name="_get_status">
<input type="submit" value="EPP Delete Domain" name="_delete_domain"> <input type="submit" value="Delete Domain in Registry" name="_delete_domain">
</div> </div>
{{ block.super }} {{ block.super }}
{% endblock %} {% endblock %}

View file

@ -108,12 +108,12 @@ class TestDomainAdmin(MockEppLib):
) )
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertContains(response, domain.name) self.assertContains(response, domain.name)
self.assertContains(response, "EPP Delete Domain") self.assertContains(response, "Delete Domain in Registry")
# Test the info dialog # Test the info dialog
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Epp Delete Domain", "name": domain.name}, {"_delete_domain": "Delete Domain in Registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client
@ -148,12 +148,12 @@ class TestDomainAdmin(MockEppLib):
) )
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertContains(response, domain.name) self.assertContains(response, domain.name)
self.assertContains(response, "EPP Delete Domain") self.assertContains(response, "Delete Domain in Registry")
# Test the error # Test the error
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Epp Delete Domain", "name": domain.name}, {"_delete_domain": "Delete Domain in Registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client
@ -192,12 +192,12 @@ class TestDomainAdmin(MockEppLib):
) )
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertContains(response, domain.name) self.assertContains(response, domain.name)
self.assertContains(response, "EPP Delete Domain") self.assertContains(response, "Delete Domain in Registry")
# Test the info dialog # Test the info dialog
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Epp Delete Domain", "name": domain.name}, {"_delete_domain": "Delete Domain in Registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client
@ -219,7 +219,7 @@ class TestDomainAdmin(MockEppLib):
# Test the info dialog # Test the info dialog
request = self.factory.post( request = self.factory.post(
"/admin/registrar/domain/{}/change/".format(domain.pk), "/admin/registrar/domain/{}/change/".format(domain.pk),
{"_delete_domain": "Epp Delete Domain", "name": domain.name}, {"_delete_domain": "Delete Domain in Registry", "name": domain.name},
follow=True, follow=True,
) )
request.user = self.client request.user = self.client