Merge pull request #30 from NHSDigital/8441-appointment-cannot-proceed

Add appointment cannot go ahead page

Author: malcolmbaig

.github/workflows/stage-2-test.yaml

@@ -66,12 +66,13 @@ jobs:
           cache-dependency-path: ./poetry.lock
 
       - name: Install dependencies
-        run:
-          make dependencies
+        run: make dependencies
+
+      - name: Install Playwright browsers
+        run: poetry run playwright install chromium --with-deps
 
       - name: "Run unit test suite"
-        run: |
-          make test-unit
+        run: make test-unit
         env:
           DATABASE_NAME: postgres
           DATABASE_PASSWORD: postgres

manage_breast_screening/clinics/migrations/0007_appointment_reinvite.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.1.8 on 2025-05-01 14:36
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('clinics', '0006_alter_appointment_status'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='appointment',
+            name='reinvite',
+            field=models.BooleanField(default=False),
+        ),
+    ]

manage_breast_screening/clinics/migrations/0008_appointment_stopped_reasons.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.1.8 on 2025-05-06 12:18
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('clinics', '0007_appointment_reinvite'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='appointment',
+            name='stopped_reasons',
+            field=models.JSONField(blank=True, null=True),
+        ),
+    ]

manage_breast_screening/clinics/migrations/0009_merge_20250514_1140.py

@@ -0,0 +1,14 @@
+# Generated by Django 5.2.1 on 2025-05-14 11:40
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('clinics', '0007_alter_appointment_status'),
+        ('clinics', '0008_appointment_stopped_reasons'),
+    ]
+
+    operations = [
+    ]

manage_breast_screening/clinics/models.py

@@ -162,3 +162,6 @@ class Status:
     status = models.CharField(
         choices=STATUS_CHOICES, max_length=50, default=Status.CONFIRMED
     )
+    reinvite = models.BooleanField(default=False)
+    stopped_reasons = models.JSONField(null=True, blank=True)
+

manage_breast_screening/config/jinja2_env.py

@@ -75,7 +75,7 @@ def format_age(value: int) -> str:
 
 
 def environment(**options):
-    env = Environment(**options)
+    env = Environment(**options, extensions=["jinja2.ext.do"])
     if env.loader:
         env.loader = ChoiceLoader([PackageLoader("nhsuk_frontend_jinja"), env.loader])
 

manage_breast_screening/config/system_test_setup.py

@@ -0,0 +1,23 @@
+from django.contrib.staticfiles.testing import StaticLiveServerTestCase
+import os
+from playwright.sync_api import sync_playwright, expect
+
+class SystemTestCase(StaticLiveServerTestCase):
+    @classmethod
+    def setUpClass(cls):
+        os.environ["DJANGO_ALLOW_ASYNC_UNSAFE"] = "true"
+        super().setUpClass()
+        cls.playwright = sync_playwright().start()
+        cls.browser = cls.playwright.chromium.launch()
+
+    @classmethod
+    def tearDownClass(cls):
+        super().tearDownClass()
+        cls.browser.close()
+        cls.playwright.stop()
+
+    def setUp(self):
+        self.page = self.browser.new_page()
+
+    def tearDown(self):
+        self.page.close()

manage_breast_screening/record_a_mammogram/forms.py

@@ -1,6 +1,5 @@
 from django import forms
 
-
 class ScreeningAppointmentForm(forms.Form):
     decision = forms.ChoiceField(
         choices=(
@@ -44,4 +43,67 @@ def save(self):
 
 
 class AppointmentCannotGoAheadForm(forms.Form):
-    pass
+    STOPPED_REASON_CHOICES = (
+        ("participant_did_not_attend", "Participant did not attend"),
+        ("failed_identity_check", "Failed identity check"),
+        ("language_difficulties", "Language difficulties"),
+        ("physical_health_issue", "Physical health issue"),
+        ("mental_health_issue", "Mental health issue"),
+        ("last_mammogram_within_6_months", "Last mammogram within 6 months"),
+        ("breast_implant_risks", "Breast implant risks"),
+        ("pain_during_screening", "Pain during screening"),
+        ("technical_issues", "Technical issues"),
+        ("participant_withdrew_consent", "Participant withdrew consent"),
+        ("other", "Other"),
+    )
+
+    def __init__(self, *args, **kwargs):
+        if 'instance' not in kwargs:
+            raise ValueError("AppointmentCannotGoAheadForm requires an instance")
+        self.instance = kwargs.pop('instance')
+        super().__init__(*args, **kwargs)
+
+        # Dynamically add detail fields for each choice
+        for field_name, _ in self.STOPPED_REASON_CHOICES:
+            self.fields[f"{field_name}_details"] = forms.CharField(required=False)
+
+    stopped_reasons = forms.MultipleChoiceField(
+        choices=STOPPED_REASON_CHOICES,
+        required=True,
+        error_messages={
+            "required": "A reason for why this appointment cannot continue must be provided"
+        }
+    )
+
+    decision = forms.ChoiceField(
+        choices=(
+            ("True", "Yes, add participant to reinvite list"),
+            ("False", "No"),
+        ),
+        required=True,
+        widget=forms.RadioSelect(),
+        error_messages={
+            "required": "Select whether the participant needs to be invited for another appointment"
+        }
+    )
+
+    def clean(self):
+        cleaned_data = super().clean()
+
+        if 'stopped_reasons' in cleaned_data and 'other' in cleaned_data['stopped_reasons']:
+            if not cleaned_data.get('other_details'):
+                self.add_error('other_details', 'Explain why this appointment cannot proceed')
+        return cleaned_data
+
+    def save(self):
+        reasons_json = {}
+        reasons_json["stopped_reasons"] = self.cleaned_data["stopped_reasons"]
+        for field_name, value in self.cleaned_data.items():
+            if field_name.endswith('_details') and value:
+                reasons_json[field_name] = value
+        self.instance.stopped_reasons = reasons_json
+        self.instance.reinvite = self.cleaned_data["decision"]
+        self.instance.status = self.instance.Status.ATTENDED_NOT_SCREENED
+        self.instance.save()
+
+        return self.instance

manage_breast_screening/record_a_mammogram/tests/system/__init__.py

@@ -0,0 +1,74 @@
+import re
+from django.urls import reverse
+from playwright.sync_api import expect
+
+from manage_breast_screening.config.system_test_setup import SystemTestCase
+from manage_breast_screening.clinics.tests.factories import AppointmentFactory, ScreeningEpisodeFactory
+from manage_breast_screening.participants.tests.factories import ParticipantFactory
+from manage_breast_screening.clinics.models import Appointment
+
+class UserSubmitsCannotGoAheadForm(SystemTestCase):
+    def setUp(self):
+        super().setUp()
+        self.participant = ParticipantFactory()
+        self.screening_episode = ScreeningEpisodeFactory(participant=self.participant)
+        self.appointment = AppointmentFactory(screening_episode=self.screening_episode)
+
+    def test_user_submits_cannot_go_ahead_form(self):
+        self.given_i_am_on_the_cannot_go_ahead_form()
+        self.when_i_submit_the_form()
+        self.then_i_should_see_validation_errors()
+
+        self.when_i_select_a_reason_for_the_appointment_being_stopped()
+        self.and_i_select_other_as_a_reason()
+        self.and_i_choose_to_add_the_participant_to_the_reinvite_list()
+        self.when_i_submit_the_form()
+        self.then_i_see_an_error_for_other_details()
+
+        self.when_i_fill_in_other_details()
+        self.when_i_submit_the_form()
+        self.then_i_see_the_clinics_page()
+        self.and_the_appointment_is_updated()
+
+
+    def given_i_am_on_the_cannot_go_ahead_form(self):
+        self.page.goto(
+            self.live_server_url + reverse(
+                "record_a_mammogram:appointment_cannot_go_ahead", kwargs={"id": self.appointment.pk}
+            )
+        )
+
+    def when_i_submit_the_form(self):
+        self.page.get_by_role("button", name="Continue").click()
+
+    def then_i_should_see_validation_errors(self):
+        expect(
+            self.page.locator("#stopped_reasons-error")
+        ).to_have_text(re.compile("A reason for why this appointment cannot continue must be provided"))
+
+    def when_i_select_a_reason_for_the_appointment_being_stopped(self):
+        self.page.get_by_label("Failed identity check").check()
+
+    def and_i_select_other_as_a_reason(self):
+        self.page.get_by_label("Other").check()
+
+    def and_i_choose_to_add_the_participant_to_the_reinvite_list(self):
+        self.page.get_by_label("Yes, add participant to reinvite list").click()
+        expect(self.page.get_by_label("Yes, add participant to reinvite list")).to_be_checked()
+
+    def then_i_see_an_error_for_other_details(self):
+        expect(
+            self.page.locator("#other_details-error")
+        ).to_have_text(re.compile("Explain why this appointment cannot proceed"))
+
+    def when_i_fill_in_other_details(self):
+        self.page.locator("#other_details").fill("Explain other choice")
+
+    def then_i_see_the_clinics_page(self):
+        expect(self.page).to_have_url(
+            re.compile(reverse("clinics:index"))
+        )
+
+    def and_the_appointment_is_updated(self):
+        self.appointment.refresh_from_db()
+        self.assertEqual(self.appointment.status, Appointment.Status.ATTENDED_NOT_SCREENED)