add missing test for ChangeLumpView

Author: MatMoore

manage_breast_screening/mammograms/tests/views/test_symptom_views.py

@@ -18,7 +18,7 @@ def lump(appointment):
 
 
 @pytest.mark.django_db
-class TestAddLump:
+class TestAddLumpView:
     def test_renders_response(self, clinical_user_client, appointment):
         response = clinical_user_client.get(
             reverse(
@@ -78,6 +78,92 @@ def test_invvalid_post_renders_response_with_errors(
         )
 
 
+@pytest.mark.django_db
+class TestChangeLumpView:
+    @pytest.fixture
+    def lump(self, appointment):
+        return SymptomFactory.create(lump=True, appointment=appointment)
+
+    def test_renders_response(self, clinical_user_client, appointment, lump):
+        response = clinical_user_client.get(
+            reverse(
+                "mammograms:change_symptom_lump",
+                kwargs={"pk": appointment.pk, "symptom_pk": lump.pk},
+            )
+        )
+        assert response.status_code == 200
+
+    def test_non_existant_or_deleted_symptom_id_is_a_404(
+        self, clinical_user_client, appointment
+    ):
+        """
+        Note: the behaviour we probably want here is to redirect back to
+        the "parent page" when a child entity is not found, and use flash
+        messages to explain the error. However, none of this is
+        implemented yet.
+        """
+        response = clinical_user_client.get(
+            reverse(
+                "mammograms:change_symptom_lump",
+                kwargs={
+                    "pk": appointment.pk,
+                    "symptom_pk": "beefbeef-beef-beef-beef-beefbeefbeef",
+                },
+            )
+        )
+
+        assert response.status_code == 404
+
+    def test_valid_post_redirects_to_appointment(
+        self, clinical_user_client, appointment, lump
+    ):
+        response = clinical_user_client.post(
+            reverse(
+                "mammograms:change_symptom_lump",
+                kwargs={"pk": appointment.pk, "symptom_pk": lump.pk},
+            ),
+            {
+                "area": SymptomAreas.RIGHT_BREAST.value,
+                "when_started": RelativeDateChoices.LESS_THAN_THREE_MONTHS.value,
+                "investigated": YesNo.NO.value,
+            },
+        )
+        assertRedirects(
+            response,
+            reverse(
+                "mammograms:record_medical_information",
+                kwargs={"pk": appointment.pk},
+            ),
+        )
+
+    def test_invvalid_post_renders_response_with_errors(
+        self, clinical_user_client, appointment, lump
+    ):
+        response = clinical_user_client.post(
+            reverse(
+                "mammograms:change_symptom_lump",
+                kwargs={"pk": appointment.pk, "symptom_pk": lump.pk},
+            ),
+            {},
+        )
+        assert response.status_code == 200
+        assertInHTML(
+            """
+                <div class="nhsuk-error-summary" aria-labelledby="error-summary-title" role="alert" tabindex="-1" data-module="nhsuk-error-summary">
+                    <h2 class="nhsuk-error-summary__title" id="error-summary-title">There is a problem</h2>
+                      <div class="nhsuk-error-summary__body">
+                        <ul class="nhsuk-list nhsuk-error-summary__list" role="list">
+                            <li><a href="#id_area">Select the location of the lump</a></li>
+                            <li><a href="#id_when_started">Select how long the symptom has existed</a></li>
+                            <li><a href="#id_investigated">Select whether the symptom has been investigated or not</a></li>
+                        </ul>
+                    </div>
+                </div>
+            """,
+            response.text,
+        )
+
+
 @pytest.mark.django_db
 class TestDeleteSymptomView:
     def test_get_renders_response(self, clinical_user_client, appointment, lump):