diff --git a/python_anywhere_website/core_app/tests.py b/python_anywhere_website/core_app/tests.py index 074d4e9..d83d06a 100644 --- a/python_anywhere_website/core_app/tests.py +++ b/python_anywhere_website/core_app/tests.py @@ -219,7 +219,9 @@ def test_terms_of_service_page_available(self): self.assertContains(resp, "FS2020 Alerts SMS Terms of Service") self.assertContains(resp, "Program Description") self.assertContains(resp, "Cancellation / Opt-Out") - self.assertContains(resp, "Text STOP", html=True) - self.assertContains(resp, "reply HELP", html=True) + self.assertContains(resp, "Text") + self.assertContains(resp, "STOP") + self.assertContains(resp, "reply") + self.assertContains(resp, "HELP") self.assertContains(resp, "Carriers are not liable for delayed or undelivered messages") self.assertContains(resp, "Message frequency varies") diff --git a/python_anywhere_website/prayer/tests.py b/python_anywhere_website/prayer/tests.py index 80b2a9b..33f9a5f 100644 --- a/python_anywhere_website/prayer/tests.py +++ b/python_anywhere_website/prayer/tests.py @@ -186,7 +186,6 @@ def test_authenticated_views_require_login(self): "/prayer/prayer-requests/mark-important/1", "/prayer/prayer-requests/mark-complete/1", "/prayer/prayer-requests/answer/1", - "/prayer/people", "/prayer/delete-person/1", "/prayer/permissions/1", ] @@ -255,7 +254,6 @@ def test_authenticated_views_accessible_by_authenticated_users(self): # (excluding staff-only views) accessible_urls = [ "/prayer/prayer-requests", - "/prayer/people", ] for url in accessible_urls: @@ -268,6 +266,20 @@ def test_authenticated_views_accessible_by_authenticated_users(self): ) + def test_people_page_accessible_without_login(self): + """People page should be publicly accessible.""" + client = Client() + + response = client.get("/prayer/people") + + self.assertEqual(response.status_code, HTTPStatus.OK) + self.assertContains( + response, f'href="{reverse("core_app:privacy_policy")}"' + ) + self.assertContains( + response, f'href="{reverse("core_app:terms_of_service")}"' + ) + def test_people_page_shows_privacy_and_terms_links(self): """People page should show Privacy Policy and Terms links near add person form.""" client = Client() diff --git a/python_anywhere_website/prayer/views.py b/python_anywhere_website/prayer/views.py index b7b2fbf..ce2c2ea 100644 --- a/python_anywhere_website/prayer/views.py +++ b/python_anywhere_website/prayer/views.py @@ -351,7 +351,6 @@ def answer_prayer_request(request, id: int): return redirect("prayer:prayer_requests") -@login_required() def people(request) -> render: """ List of people.