diff --git a/src/registrar/tests/test_views_requests_json.py b/src/registrar/tests/test_views_requests_json.py index 1f59ad70b..157a49705 100644 --- a/src/registrar/tests/test_views_requests_json.py +++ b/src/registrar/tests/test_views_requests_json.py @@ -3,6 +3,7 @@ from django.urls import reverse from .test_views import TestWithUser from django_webtest import WebTest # type: ignore from django.utils.dateparse import parse_datetime +from django.urls import reverse class GetRequestsJsonTest(TestWithUser, WebTest): @@ -158,14 +159,14 @@ class GetRequestsJsonTest(TestWithUser, WebTest): # Check action_url action_url_expected = ( - f"/domain-request/{self.domain_requests[i].id}/edit" + reverse('domain-request//edit/', kwargs={'pk': self.domain_requests[i].id}) if self.domain_requests[i].status in [ DomainRequest.DomainRequestStatus.STARTED, DomainRequest.DomainRequestStatus.ACTION_NEEDED, DomainRequest.DomainRequestStatus.WITHDRAWN, ] - else f"/domain-request/{self.domain_requests[i].id}" + else reverse('domain-request', kwargs={'pk': self.domain_requests[i].id}) ) self.assertEqual(action_url_expected, action_urls[i]) diff --git a/src/registrar/views/domains_json.py b/src/registrar/views/domains_json.py index c00a926cd..d5b686af8 100644 --- a/src/registrar/views/domains_json.py +++ b/src/registrar/views/domains_json.py @@ -2,6 +2,7 @@ from django.http import JsonResponse from django.core.paginator import Paginator from registrar.models import UserDomainRole, Domain from django.contrib.auth.decorators import login_required +from django.urls import reverse @login_required @@ -40,7 +41,7 @@ def get_domains_json(request): "state": domain.state, "state_display": domain.state_display(), "get_state_help_text": domain.get_state_help_text(), - "action_url": f"/domain/{domain.id}", + "action_url": reverse('domain', kwargs={'pk': domain.id}), "action_label": ("View" if domain.state in [Domain.State.DELETED, Domain.State.ON_HOLD] else "Manage"), "svg_icon": ("visibility" if domain.state in [Domain.State.DELETED, Domain.State.ON_HOLD] else "settings"), }