Merge pull request #1125 from NHSDigital/DTOSS-12373-breast-pain

Added breast pain symptom

Author: swebberuk

manage_breast_screening/mammograms/forms/symptom_forms.py

@@ -485,3 +485,47 @@ def __init__(self, instance=None, **kwargs):
         self.given_field_value("investigated", YesNo.YES).require_field(
             "investigation_details"
         )
+
+
+class BreastPainForm(SymptomForm):
+    BREAST_PAIN_NAME = "pain"
+
+    area = CommonFields.area_radios(symptom_name=BREAST_PAIN_NAME)
+    area_description_right_breast = CommonFields.area_description(
+        BREAST_PAIN_NAME, visually_hidden_label_suffix=": right breast"
+    )
+    area_description_left_breast = CommonFields.area_description(
+        BREAST_PAIN_NAME, visually_hidden_label_suffix=": left breast"
+    )
+    area_description_other = CommonFields.area_description(
+        BREAST_PAIN_NAME,
+        visually_hidden_label_suffix=": other",
+        classes="nhsuk-u-width-two-thirds",
+    )
+    when_started = CommonFields.when_started
+    specific_date = CommonFields.specific_date
+    intermittent = CommonFields.intermittent
+    recently_resolved = CommonFields.recently_resolved
+    when_resolved = CommonFields.when_resolved
+    investigated = CommonFields.investigated
+    investigation_details = CommonFields.investigation_details
+    additional_information = CommonFields.additional_information
+
+    def __init__(self, instance=None, **kwargs):
+        super().__init__(
+            symptom_type=SymptomType.BREAST_PAIN,
+            instance=instance,
+            **kwargs,
+        )
+
+        self.given_field("area").require_field_with_prefix("area_description")
+
+        self.given_field_value(
+            "when_started", RelativeDateChoices.SINCE_A_SPECIFIC_DATE
+        ).require_field("specific_date")
+
+        self.given_field_value("recently_resolved", True).require_field("when_resolved")
+
+        self.given_field_value("investigated", YesNo.YES).require_field(
+            "investigation_details"
+        )

manage_breast_screening/mammograms/jinja2/mammograms/medical_information/symptoms/forms/breast_pain.jinja

@@ -0,0 +1,48 @@
+{% extends "layout-form.jinja" %}
+{% from "nhsuk/components/button/macro.jinja" import button %}
+{% from "nhsuk/components/fieldset/macro.jinja" import fieldset %}
+
+{% block form %}
+  {% include "mammograms/medical_information/symptoms/forms/inline_area_radios.jinja" %}
+
+  {% do form.when_started.add_divider_after("OVER_THREE_YEARS", "or") %}
+  {% do form.when_started.add_divider_after("SINCE_A_SPECIFIC_DATE", "or") %}
+  {% do form.when_started.add_conditional_html(
+    'SINCE_A_SPECIFIC_DATE',
+    form.specific_date.as_field_group()
+  ) %}
+
+  {% do form.recently_resolved.add_conditional_html('true', form.when_resolved.as_field_group()) %}
+
+  {% do form.investigated.add_conditional_html('YES', form.investigation_details.as_field_group()) %}
+
+  {% call fieldset({
+    "legend": {
+      "text": form.when_started.label,
+      "classes": "nhsuk-fieldset__legend--m"
+    }
+  }) %}
+    {{ form.when_started.as_field_group() }}
+    <br>
+    {{ form.intermittent.as_field_group() }}
+    {{ form.recently_resolved.as_field_group() }}
+  {% endcall %}
+
+  {{ form.investigated.as_field_group() }}
+
+  {{ form.additional_information.as_field_group() }}
+
+  <div class="nhsuk-button-group">
+    {{ button({
+      "text": "Save symptom"
+    }) }}
+  </div>
+
+  {% if delete_link %}
+    <p class="nhsuk-u-margin-top-4">
+      <a href="{{ delete_link.href }}" class="{{ delete_link.class }}">
+        {{ delete_link.text }}
+      </a>
+    </p>
+  {% endif %}
+{% endblock %}

manage_breast_screening/mammograms/presenters/medical_information_presenter.py

@@ -182,6 +182,7 @@ def symptom_buttons(self):
             self.add_swelling_or_shape_change_button,
             self.add_skin_change_button,
             self.add_nipple_change_button,
+            self.add_breast_pain_button,
             self.add_other_symptom_button,
         ]
 
@@ -232,6 +233,10 @@ def add_nipple_change_button(self):
             "mammograms:add_symptom_nipple_change", "Nipple change"
         )
 
+    @property
+    def add_breast_pain_button(self):
+        return self._subpage_button("mammograms:add_symptom_breast_pain", "Breast pain")
+
     @property
     def add_other_symptom_button(self):
         return self._subpage_button("mammograms:add_symptom_other", "Other")

manage_breast_screening/mammograms/presenters/symptom_presenter.py

@@ -165,17 +165,7 @@ def build_summary_list_row(self, include_actions=True):
         return result
 
     def change_view(self):
-        match self._symptom.symptom_type_id:
-            case SymptomType.LUMP:
-                return "mammograms:change_symptom_lump"
-            case SymptomType.SWELLING_OR_SHAPE_CHANGE:
-                return "mammograms:change_symptom_swelling_or_shape_change"
-            case SymptomType.SKIN_CHANGE:
-                return "mammograms:change_symptom_skin_change"
-            case SymptomType.NIPPLE_CHANGE:
-                return "mammograms:change_symptom_nipple_change"
-            case _:
-                return "mammograms:change_symptom_other"
+        return f"mammograms:change_symptom_{self._symptom.symptom_type_id.lower()}"
 
     @property
     def delete_message_html(self):

manage_breast_screening/mammograms/tests/forms/test_symptom_forms.py

@@ -5,6 +5,7 @@
 from django.test import RequestFactory
 
 from manage_breast_screening.mammograms.forms.symptom_forms import (
+    BreastPainForm,
     LumpForm,
     NippleChangeForm,
     OtherSymptomForm,
@@ -638,3 +639,74 @@ def test_valid_form_with_conditionally_required_fields(self):
             )
         )
         assert form.is_valid()
+
+
+class TestBreastPainForm:
+    def test_valid_form(self):
+        form = BreastPainForm(
+            data=QueryDict(
+                urlencode(
+                    {
+                        "area": RightLeftOtherChoices.LEFT_BREAST,
+                        "area_description_left_breast": "uoq",
+                        "when_started": RelativeDateChoices.LESS_THAN_THREE_MONTHS,
+                        "investigated": YesNo.NO,
+                    }
+                )
+            )
+        )
+        assert form.is_valid()
+
+    def test_missing_required_fields(self):
+        form = BreastPainForm(data=QueryDict())
+
+        assert not form.is_valid()
+        assert form.errors == {
+            "when_started": ["Select how long the symptom has existed"],
+            "investigated": ["Select whether the symptom has been investigated or not"],
+            "area": ["Select the location of the pain"],
+        }
+
+    def test_missing_conditionally_required_fields(self):
+        form = BreastPainForm(
+            data=QueryDict(
+                urlencode(
+                    {
+                        "area": RightLeftOtherChoices.OTHER,
+                        "when_started": RelativeDateChoices.SINCE_A_SPECIFIC_DATE,
+                        "investigated": YesNo.YES,
+                        "recently_resolved": True,
+                    }
+                )
+            )
+        )
+
+        assert not form.is_valid()
+        assert form.errors == {
+            "area_description_other": [
+                "Describe the specific area where the pain is located"
+            ],
+            "specific_date": ["Enter the date the symptom started"],
+            "investigation_details": ["Enter details of any investigations"],
+            "when_resolved": ["Enter when the symptom was resolved"],
+        }
+
+    def test_valid_form_with_conditionally_required_fields(self):
+        form = BreastPainForm(
+            data=QueryDict(
+                urlencode(
+                    {
+                        "area": RightLeftOtherChoices.OTHER,
+                        "area_description_other": "abc",
+                        "when_started": RelativeDateChoices.SINCE_A_SPECIFIC_DATE,
+                        "investigated": YesNo.YES,
+                        "specific_date_0": "2",
+                        "specific_date_1": "2025",
+                        "investigation_details": "def",
+                        "recently_resolved": True,
+                        "when_resolved": "3 months ago",
+                    }
+                )
+            )
+        )
+        assert form.is_valid()

manage_breast_screening/mammograms/tests/presenters/test_check_medical_information_presenter.py

@@ -864,6 +864,15 @@ def test_many_medical_history(self):
                     "when_started": RelativeDateChoices.LESS_THAN_THREE_MONTHS,
                 },
             ),
+            (
+                "Breast pain (both breasts)",
+                {
+                    "symptom_type_id": SymptomType.BREAST_PAIN,
+                    "symptom_sub_type_details": "user provided details",
+                    "area": SymptomAreas.BOTH_BREASTS,
+                    "when_started": RelativeDateChoices.LESS_THAN_THREE_MONTHS,
+                },
+            ),
             (
                 "user provided details (both breasts)",
                 {

manage_breast_screening/mammograms/tests/presenters/test_medical_information_presenter.py

@@ -108,6 +108,14 @@ def test_add_nipple_change_button(self):
             "text": "Nipple change",
         }
 
+    def test_add_breast_pain_button(self):
+        appointment = AppointmentFactory()
+
+        assert MedicalInformationPresenter(appointment).add_breast_pain_button == {
+            "href": f"/mammograms/{appointment.pk}/record-medical-information/breast-pain/",
+            "text": "Breast pain",
+        }
+
     def test_add_breast_cancer_history_button(self):
         appointment = AppointmentFactory()
 
@@ -499,3 +507,18 @@ def test_review_button(self, in_progress_appointment):
             "href": f"/mammograms/{in_progress_appointment.pk}/record-medical-information/mark-reviewed/OTHER_INFORMATION/",
             "text": "Mark as reviewed",
         }
+
+    def test_symptom_buttons(self):
+        appointment = AppointmentFactory()
+        presenter = MedicalInformationPresenter(appointment)
+        buttons = presenter.symptom_buttons
+
+        assert [button["text"] for button in buttons] == [
+            "Lump",
+            "Swelling or shape change",
+            "Skin change",
+            "Nipple change",
+            "Breast pain",
+            "Other",
+        ]
+        assert all(button.get("href") for button in buttons)

manage_breast_screening/mammograms/tests/presenters/test_symptom_presenter.py

@@ -181,3 +181,8 @@ def test_add_message_html(self):
             presenter.add_message_html
             == '<h3 class="nhsuk-notification-banner__heading">Symptom added</h3><p>Added lump.</p>'
         )
+
+    def test_change_view(self):
+        symptom = SymptomFactory.create(symptom_type_id=SymptomType.BREAST_PAIN)
+        presenter = SymptomPresenter(symptom)
+        assert presenter.change_view() == "mammograms:change_symptom_breast_pain"

manage_breast_screening/mammograms/tests/views/test_symptom_views.py

@@ -10,6 +10,7 @@
     SkinChangeChoices,
     Symptom,
     SymptomAreas,
+    SymptomType,
 )
 from manage_breast_screening.participants.tests.factories import (
     AppointmentFactory,
@@ -446,6 +447,95 @@ def test_valid_post_redirects_to_appointment(
         )
 
 
+@pytest.mark.django_db
+class TestAddBreastPainView:
+    def test_renders_response(self, clinical_user_client):
+        appointment = AppointmentFactory.create(
+            clinic_slot__clinic__setting__provider=clinical_user_client.current_provider
+        )
+        response = clinical_user_client.http.get(
+            reverse(
+                "mammograms:add_symptom_breast_pain",
+                kwargs={"pk": appointment.pk},
+            )
+        )
+        assert response.status_code == 200
+
+    def test_valid_post_redirects_to_appointment(self, clinical_user_client):
+        appointment = AppointmentFactory.create(
+            clinic_slot__clinic__setting__provider=clinical_user_client.current_provider
+        )
+        response = clinical_user_client.http.post(
+            reverse(
+                "mammograms:add_symptom_breast_pain",
+                kwargs={"pk": appointment.pk},
+            ),
+            {
+                "area": SymptomAreas.RIGHT_BREAST.value,
+                "area_description_right_breast": "uiq",
+                "when_started": RelativeDateChoices.LESS_THAN_THREE_MONTHS.value,
+                "investigated": YesNo.NO.value,
+            },
+        )
+        assertRedirects(
+            response,
+            reverse(
+                "mammograms:record_medical_information",
+                kwargs={"pk": appointment.pk},
+            ),
+        )
+
+
+@pytest.mark.django_db
+class TestChangeBreastPainView:
+    @pytest.fixture
+    def breast_pain(self, clinical_user_client):
+        appointment = AppointmentFactory.create(
+            clinic_slot__clinic__setting__provider=clinical_user_client.current_provider
+        )
+        return SymptomFactory.create(
+            symptom_type_id=SymptomType.BREAST_PAIN, appointment=appointment
+        )
+
+    def test_renders_response(self, clinical_user_client, breast_pain):
+        response = clinical_user_client.http.get(
+            reverse(
+                "mammograms:change_symptom_breast_pain",
+                kwargs={
+                    "pk": breast_pain.appointment.pk,
+                    "symptom_pk": breast_pain.pk,
+                },
+            )
+        )
+        assert response.status_code == 200
+
+    def test_valid_post_redirects_to_appointment(
+        self, clinical_user_client, breast_pain
+    ):
+        response = clinical_user_client.http.post(
+            reverse(
+                "mammograms:change_symptom_breast_pain",
+                kwargs={
+                    "pk": breast_pain.appointment.pk,
+                    "symptom_pk": breast_pain.pk,
+                },
+            ),
+            {
+                "area": SymptomAreas.RIGHT_BREAST.value,
+                "area_description_right_breast": "uiq",
+                "when_started": RelativeDateChoices.LESS_THAN_THREE_MONTHS.value,
+                "investigated": YesNo.NO.value,
+            },
+        )
+        assertRedirects(
+            response,
+            reverse(
+                "mammograms:record_medical_information",
+                kwargs={"pk": breast_pain.appointment.pk},
+            ),
+        )
+
+
 @pytest.mark.django_db
 class TestDeleteSymptomView:
     def test_get_renders_response(self, clinical_user_client, lump):

manage_breast_screening/mammograms/urls.py

@@ -165,6 +165,16 @@ def not_implemented_view(request, pk):
         symptom_views.ChangeNippleChangeView.as_view(),
         name="change_symptom_nipple_change",
     ),
+    path(
+        "<uuid:pk>/record-medical-information/breast-pain/",
+        symptom_views.AddBreastPainView.as_view(),
+        name="add_symptom_breast_pain",
+    ),
+    path(
+        "<uuid:pk>/record-medical-information/breast-pain/<uuid:symptom_pk>/",
+        symptom_views.ChangeBreastPainView.as_view(),
+        name="change_symptom_breast_pain",
+    ),
     path(
         "<uuid:pk>/record-medical-information/other/",
         symptom_views.AddOtherSymptomView.as_view(),