Merge pull request #191 from NHSDigital/DTOSS-9816-screening-protocol

[DTOSS-9816] Add screening protocol

Author: malcolmbaig

docs/diagrams/erd.md

@@ -123,6 +123,7 @@ ScreeningEpisode {
     DateTimeField created_at
     DateTimeField updated_at
     ForeignKey participant
+    CharField protocol
 }
 Appointment {
     UUIDField id

manage_breast_screening/mammograms/jinja2/mammograms/start_screening.jinja

@@ -56,6 +56,10 @@
   {{ card({
     "headingHtml": " ",
     "headingLevel": "2",
-    "descriptionHtml": participant_details(presented_participant=presented_participant, presented_mammograms=presented_mammograms, return_url=request.path)
+    "descriptionHtml": participant_details(
+      presented_participant=presented_participant,
+      return_url=request.path,
+      screening_protocol=presented_appointment.screening_protocol,
+    )
   }) }}
 {% endblock %}

manage_breast_screening/mammograms/presenters.py

@@ -33,6 +33,7 @@ def __init__(self, appointment):
         self.participant = ParticipantPresenter(
             appointment.screening_episode.participant
         )
+        self.screening_protocol = appointment.screening_episode.get_protocol_display()
 
     @cached_property
     def participant_url(self):

manage_breast_screening/mammograms/tests/test_presenters.py

@@ -82,6 +82,15 @@ def test_participant_url(self, mock_appointment):
             == "/participants/ac1b68ec-06a4-40a0-a016-7108dffe4397/"
         )
 
+    def test_screening_protocol(self, mock_appointment):
+        mock_appointment.screening_episode.get_protocol_display.return_value = (
+            "Family history"
+        )
+
+        result = AppointmentPresenter(mock_appointment)
+
+        assert result.screening_protocol == "Family history"
+
 
 class TestLastKnownMammogramPresenter:
     @pytest.fixture()

manage_breast_screening/participants/jinja2/components/participant-details/macro.jinja

@@ -1,4 +1,4 @@
-{% macro participant_details(presented_participant, presented_mammograms, return_url=none, full_details=false) %}
+{% macro participant_details(presented_participant, presented_mammograms, return_url=none, full_details=false, screening_protocol=none) %}
   {% if return_url is none %}
     {{ raise('return_url is required') }}
   {% endif %}

manage_breast_screening/participants/jinja2/components/participant-details/template.jinja

@@ -113,6 +113,14 @@
        "html": presented_participant.phone
      }
    },
+   {
+     "key": {
+       "text": "Screening protocol"
+     },
+     "value": {
+       "html": screening_protocol
+     }
+   } if screening_protocol else undefined,
    {
      "key": {
        "text": "Email"

manage_breast_screening/participants/migrations/0017_screeningepisode_protocol.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.2.3 on 2025-07-08 08:03
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('participants', '0016_participant_any_other_background_details'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='screeningepisode',
+            name='protocol',
+            field=models.CharField(choices=[('FAMILY_HISTORY', 'Family history')], default='FAMILY_HISTORY', max_length=50),
+        ),
+    ]

manage_breast_screening/participants/migrations/0018_merge_20250708_1015.py

@@ -0,0 +1,14 @@
+# Generated by Django 5.2.3 on 2025-07-08 10:15
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('participants', '0017_merge_20250707_0856'),
+        ('participants', '0017_screeningepisode_protocol'),
+    ]
+
+    operations = [
+    ]

manage_breast_screening/participants/models.py

@@ -160,7 +160,17 @@ class ParticipantAddress(models.Model):
 
 
 class ScreeningEpisode(BaseModel):
+    class Protocol:
+        FAMILY_HISTORY = "FAMILY_HISTORY"
+
+    PROTOCOL_CHOICES = {
+        Protocol.FAMILY_HISTORY: "Family history",
+    }
+
     participant = models.ForeignKey(Participant, on_delete=models.PROTECT)
+    protocol = models.CharField(
+        choices=PROTOCOL_CHOICES, default=Protocol.FAMILY_HISTORY, max_length=50
+    )
 
     def screening_history(self):
         """

manage_breast_screening/participants/tests/test_forms.py

@@ -162,7 +162,7 @@ def test_no_provider_selected(self, participant, current_provider):
         )
         assert not form.is_valid()
         assert form.errors == {
-            "provider": ["Select another brest screening unit."],
+            "provider": ["Select another breast screening unit"],
         }
 
     def test_mammogram_in_same_provider(self, participant, current_provider):