Merge pull request #56 from NHSDigital/ask-for-medical-info

Ask for medical information page

Author: MatMoore

manage_breast_screening/record_a_mammogram/forms.py

@@ -18,8 +18,8 @@ def save(self):
 class AskForMedicalInformationForm(forms.Form):
     decision = forms.ChoiceField(
         choices=(
-            ("continue", "Yes"),
-            ("dropout", "No - proceed to imaging"),
+            ("yes", "Yes"),
+            ("no", "No - proceed to imaging"),
         ),
         required=True,
         widget=forms.RadioSelect(),

manage_breast_screening/record_a_mammogram/tests/test_views.py

@@ -5,66 +5,108 @@
 from manage_breast_screening.clinics.tests.factories import AppointmentFactory
 
 
+@pytest.fixture
+def appointment():
+    return AppointmentFactory.create()
+
+
 @pytest.mark.django_db
 class TestStartScreening:
-    @pytest.fixture(autouse=True)
-    def setup(self):
-        self.appointment = AppointmentFactory.create()
-
-    def test_appointment_continued(self, client):
+    def test_appointment_continued(self, client, appointment):
         response = client.post(
             reverse(
-                "record_a_mammogram:start_screening", kwargs={"id": self.appointment.pk}
+                "record_a_mammogram:start_screening", kwargs={"id": appointment.pk}
             ),
             {"decision": "continue"},
         )
         assertRedirects(
             response,
             reverse(
                 "record_a_mammogram:ask_for_medical_information",
-                kwargs={"id": self.appointment.pk},
+                kwargs={"id": appointment.pk},
             ),
         )
 
-    def test_appointment_stopped(self, client):
+    def test_appointment_stopped(self, client, appointment):
         response = client.post(
             reverse(
-                "record_a_mammogram:start_screening", kwargs={"id": self.appointment.pk}
+                "record_a_mammogram:start_screening", kwargs={"id": appointment.pk}
             ),
             {"decision": "dropout"},
         )
         assertRedirects(
             response,
             reverse(
                 "record_a_mammogram:appointment_cannot_go_ahead",
-                kwargs={"id": self.appointment.pk},
+                kwargs={"id": appointment.pk},
             ),
         )
 
-    def test_renders_invalid_form(self, client):
+    def test_renders_invalid_form(self, client, appointment):
         response = client.post(
             reverse(
-                "record_a_mammogram:start_screening", kwargs={"id": self.appointment.pk}
+                "record_a_mammogram:start_screening", kwargs={"id": appointment.pk}
             ),
             {},
         )
         assertContains(response, "There is a problem")
 
 
 @pytest.mark.django_db
-class TestCheckIn:
-    @pytest.fixture(autouse=True)
-    def setup(self):
-        self.appointment = AppointmentFactory.create()
+class TestAskForMedicalInformation:
+    def test_continue_to_record(self, client, appointment):
+        response = client.post(
+            reverse(
+                "record_a_mammogram:ask_for_medical_information",
+                kwargs={"id": appointment.pk},
+            ),
+            {"decision": "yes"},
+        )
+        assertRedirects(
+            response,
+            reverse(
+                "record_a_mammogram:record_medical_information",
+                kwargs={"id": appointment.pk},
+            ),
+        )
+
+    def test_continue_to_imaging(self, client, appointment):
+        response = client.post(
+            reverse(
+                "record_a_mammogram:ask_for_medical_information",
+                kwargs={"id": appointment.pk},
+            ),
+            {"decision": "no"},
+        )
+        assertRedirects(
+            response,
+            reverse(
+                "record_a_mammogram:awaiting_images",
+                kwargs={"id": appointment.pk},
+            ),
+        )
+
+    def test_renders_invalid_form(self, client, appointment):
+        response = client.post(
+            reverse(
+                "record_a_mammogram:ask_for_medical_information",
+                kwargs={"id": appointment.pk},
+            ),
+            {},
+        )
+        assertContains(response, "There is a problem")
+
 
-    def test_known_redirect(self, client):
+@pytest.mark.django_db
+class TestCheckIn:
+    def test_known_redirect(self, client, appointment):
         response = client.post(
-            reverse("record_a_mammogram:check_in", kwargs={"id": self.appointment.pk}),
+            reverse("record_a_mammogram:check_in", kwargs={"id": appointment.pk}),
             {"next": "start-screening"},
         )
         assertRedirects(
             response,
             reverse(
-                "record_a_mammogram:start_screening", kwargs={"id": self.appointment.pk}
+                "record_a_mammogram:start_screening", kwargs={"id": appointment.pk}
             ),
         )

manage_breast_screening/record_a_mammogram/views.py

@@ -6,6 +6,7 @@
 from django.views.generic import FormView
 
 from manage_breast_screening.clinics.models import Appointment
+from manage_breast_screening.participants.models import Participant
 
 from ..clinics.models import Appointment
 from .forms import (
@@ -18,6 +19,8 @@
 
 Status = Appointment.Status
 
+APPOINTMENT_CANNOT_PROCEED = "Appointment cannot proceed"
+
 logger = logging.getLogger(__name__)
 
 
@@ -111,12 +114,35 @@ def get_context_data(self, **kwargs):
         )
         return context
 
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data(**kwargs)
+        id = self.kwargs["id"]
+        participant = Participant.objects.get(screeningepisode__appointment__id=id)
+
+        context.update(
+            {
+                "participant": participant,
+                "caption": participant.full_name,
+                "title": "Medical information",
+                "decision_legend": "Has the participant shared any relevant medical information?",
+                "cannot_continue_link": {
+                    "href": reverse(
+                        "record_a_mammogram:appointment_cannot_go_ahead",
+                        kwargs={"id": id},
+                    ),
+                    "text": APPOINTMENT_CANNOT_PROCEED,
+                },
+            }
+        )
+
+        return context
+
     def form_valid(self, form):
         form.save()
 
         appointment = self.get_appointment()
 
-        if form.cleaned_data["decision"] == "continue":
+        if form.cleaned_data["decision"] == "yes":
             return redirect(
                 "record_a_mammogram:record_medical_information", id=appointment.pk
             )
@@ -170,6 +196,7 @@ def appointment_cannot_go_ahead(request, id):
             "title": "Appointment cannot go ahead",
             "caption": participant.full_name,
             "form": form,
+            "decision_legend": "Does the appointment need to be rescheduled?",
         },
     )
 

manage_breast_screening/templates/record_a_mammogram/ask_for_medical_information.jinja

@@ -1,2 +1,13 @@
 {% extends "wizard_step.jinja" %}
 
+{% block stepContent %}
+  <p>Ask them about any:</p>
+  <ul class="nhsuk-list nhsuk-list--bullet">
+    <li>history of breast cancer</li>
+    <li>lump examinations or biopsies</li>
+    <li>other procedures in the breast or chest area</li>
+    <li>recent breast changes or symptoms</li>
+    <li>breast features (such as moles) that may appear on an x-ray</li>
+    <li>relevant medication (such as HRT)</li>
+  </ul>
+{% endblock %}

manage_breast_screening/templates/wizard_step.jinja

@@ -66,7 +66,7 @@
               "errorMessage": errorMessage,
               "hint": {
                 "html": decision_hint|e
-              },
+              } if decision_hint,
               "items": [
                 {
                   "value": form.decision.field.choices[0][0],
@@ -90,8 +90,12 @@
         </div>
     </div>
   </form>
+  {% endif %}
 
+  {% if cannot_continue_link %}
+    <p><a class="nhsuk-link nhsuk-link--no-visited-state" href="{{ cannot_continue_link.href }}">{{ cannot_continue_link.text }}</a></p>
   {% endif %}
   </div>
+
 </div>
 {% endblock pageContent %}