From bdc03c66a91b6e441dcc7683d84117b1d3c46b11 Mon Sep 17 00:00:00 2001
From: zandercymatics <141044360+zandercymatics@users.noreply.github.com>
Date: Wed, 20 Mar 2024 12:33:28 -0600
Subject: [PATCH] Final unit test

---
 .../includes/domain_request_fieldset.html     | 12 ++++++++++
 src/registrar/tests/test_admin.py             | 24 ++++++++++++++++++-
 2 files changed, 35 insertions(+), 1 deletion(-)

diff --git a/src/registrar/templates/django/admin/includes/domain_request_fieldset.html b/src/registrar/templates/django/admin/includes/domain_request_fieldset.html
index efd90b909..980fe0bfd 100644
--- a/src/registrar/templates/django/admin/includes/domain_request_fieldset.html
+++ b/src/registrar/templates/django/admin/includes/domain_request_fieldset.html
@@ -22,6 +22,18 @@ This is using a custom implementation fieldset.html (see admin/fieldset.html)
                                             <a href="#" class="other-contact__{{forloop.counter}}">{{ contact }}</a>{% if not forloop.last %}, {% endif %}
                                         {% endfor %}
                                     </div>
+                                {% elif field.field.name == "current_websites" %}
+                                    {% comment %}
+                                    The "website" model is essentially just a text field.
+                                    It is not useful to be redirected to the object definition,
+                                    rather it is more useful in this scenario to be redirected to the
+                                    actual website (as its just a plaintext string otherwise).
+
+                                    This ONLY applies to analysts. For superusers, its business as usual.
+                                    {% endcomment %}
+                                    {% for website in field.contents|split:", " %}
+                                        <a href="{{ website }}" class="padding-top-1 current-website__{{forloop.counter}}">{{ website }}</a>{% if not forloop.last %}, {% endif %}
+                                    {% endfor %}
                                 {% else %}
                                     <div class="readonly">{{ field.contents }}</div>
                                 {% endif %}
diff --git a/src/registrar/tests/test_admin.py b/src/registrar/tests/test_admin.py
index 970742ae3..b4b12c56f 100644
--- a/src/registrar/tests/test_admin.py
+++ b/src/registrar/tests/test_admin.py
@@ -1247,7 +1247,29 @@ class TestDomainRequestAdmin(MockEppLib):
         # Check that the page contains the link we expect.
         # Since the url is dynamic (populated by JS), we can test for its existence
         # by checking for the end tag.
-        expected_url = f"Testy Tester</a>"
+        expected_url = "Testy Tester</a>"
+        self.assertContains(response, expected_url)
+
+    @less_console_noise_decorator
+    def test_other_websites_has_readonly_link(self):
+        """Tests if the readonly other_websites field has links"""
+
+        # Create a fake domain request
+        domain_request = completed_domain_request(status=DomainRequest.DomainRequestStatus.IN_REVIEW)
+
+        p = "userpass"
+        self.client.login(username="staffuser", password=p)
+        response = self.client.get(
+            "/admin/registrar/domainrequest/{}/change/".format(domain_request.pk),
+            follow=True,
+        )
+
+        # Make sure the page loaded, and that we're on the right page
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, domain_request.requested_domain.name)
+
+        # Check that the page contains the link we expect.
+        expected_url = '<a href="city.com" class="padding-top-1 current-website__1">city.com</a>'
         self.assertContains(response, expected_url)
 
     @less_console_noise_decorator