PPHA-264: Add sex-at-birth page

Author: miriam-z

lung_cancer_screening/core/tests/acceptance/helpers/user_interaction_helpers.py

@@ -52,3 +52,11 @@ def fill_in_and_submit_weight_imperial(page, stone, pounds):
     page.get_by_label("Pounds").fill(str(pounds))
 
     page.click("text=Continue")
+
+def fill_in_and_submit_sex_at_birth(page, sex):
+    expect(page.locator("legend")).to_have_text(
+        "What was your sex at birth?")
+
+    page.get_by_label(sex, exact=True).check()
+
+    page.click("text=Continue")

lung_cancer_screening/core/tests/acceptance/test_cannot_change_answers_after_submission.py

@@ -9,7 +9,8 @@
     fill_in_and_submit_weight_metric,
     fill_in_and_submit_participant_id,
     fill_in_and_submit_smoking_eligibility,
-    fill_in_and_submit_date_of_birth
+    fill_in_and_submit_date_of_birth,
+    fill_in_and_submit_sex_at_birth
 )
 
 class TestQuestionnaire(StaticLiveServerTestCase):
@@ -40,6 +41,7 @@ def test_cannot_change_responses_once_checked_and_submitted(self):
         fill_in_and_submit_date_of_birth(page, age)
         fill_in_and_submit_height_metric(page, "170")
         fill_in_and_submit_weight_metric(page, "25.4")
+        fill_in_and_submit_sex_at_birth(page, "Male")
 
         page.click("text=Submit")
 

lung_cancer_screening/core/tests/acceptance/test_questionnaire.py

@@ -11,7 +11,8 @@
     fill_in_and_submit_smoking_eligibility,
     fill_in_and_submit_date_of_birth,
     fill_in_and_submit_weight_metric,
-    fill_in_and_submit_weight_imperial
+    fill_in_and_submit_weight_imperial,
+    fill_in_and_submit_sex_at_birth
 )
 
 from .helpers.assertion_helpers import expect_back_link_to_have_url
@@ -49,43 +50,45 @@ def test_full_questionnaire_user_journey(self):
 
         expect(page).to_have_url(
             f"{self.live_server_url}/have-you-ever-smoked")
-
+        expect_back_link_to_have_url(page, "/start")
         fill_in_and_submit_smoking_eligibility(page, smoking_status)
 
         expect(page).to_have_url(f"{self.live_server_url}/date-of-birth")
         expect_back_link_to_have_url(page, "/have-you-ever-smoked")
-
         fill_in_and_submit_date_of_birth(page, age)
 
         expect(page).to_have_url(f"{self.live_server_url}/height")
-
+        expect_back_link_to_have_url(page, "/date-of-birth")
         fill_in_and_submit_height_metric(page, height)
 
-        expect(page).to_have_url(f"{self.live_server_url}/weight")
-
         page.click("text=Back")
-
         expect(page).to_have_url(f"{self.live_server_url}/height")
-
         page.click("text=Switch to imperial")
-
         fill_in_and_submit_height_imperial(page, feet, inches)
 
         expect(page).to_have_url(f"{self.live_server_url}/weight")
-
+        expect_back_link_to_have_url(page, "/height")
         fill_in_and_submit_weight_metric(page, weight_metric)
-
-        expect(page).to_have_url(f"{self.live_server_url}/responses")
         page.click("text=Back")
+
+        expect(page).to_have_url(f"{self.live_server_url}/weight")
         page.get_by_role("link", name="Switch to stone and pounds").click()
         fill_in_and_submit_weight_imperial(page, weight_stone, weight_pound)
+
+        expect(page).to_have_url(f"{self.live_server_url}/sex-at-birth")
+        expect_back_link_to_have_url(page, "/weight")
+        fill_in_and_submit_sex_at_birth(page, "Male")
+
         expect(page).to_have_url(f"{self.live_server_url}/responses")
+        expect_back_link_to_have_url(page, "/sex-at-birth")
+
         responses = page.locator(".responses")
         expect(responses).to_contain_text("Have you ever smoked? Yes, I used to smoke regularly")
         expect(responses).to_contain_text(
             age.strftime("What is your date of birth? %Y-%m-%d"))
         expect(responses).to_contain_text(f"What is your height? {feet} feet {inches} inches")
         expect(responses).to_contain_text(f"What is your weight? {weight_stone} stone {weight_pound} pound")
+        expect(responses).to_contain_text("What was your sex at birth? Male")
 
         page.click("text=Submit")
 

lung_cancer_screening/questions/forms/sex_at_birth_form.py

@@ -0,0 +1,26 @@
+from django import forms
+
+from ...nhsuk_forms.typed_choice_field import TypedChoiceField
+from ..models.response_set import ResponseSet, SexAtBirthValues
+
+class SexAtBirthForm(forms.ModelForm):
+
+    def __init__(self, *args, **kwargs):
+        self.participant = kwargs.pop('participant')
+        super().__init__(*args, **kwargs)
+        self.instance.participant = self.participant
+
+        self.fields["sex_at_birth"] = TypedChoiceField(
+            choices=SexAtBirthValues.choices,
+            widget=forms.RadioSelect,
+            label="What was your sex at birth?",
+            label_classes="nhsuk-fieldset__legend--m",
+            hint="Your sex may impact your chances of developing lung cancer.",
+            error_messages={
+                'required': 'Select your sex at birth.'
+            }
+        )
+
+    class Meta:
+        model = ResponseSet
+        fields = ['sex_at_birth']

lung_cancer_screening/questions/jinja2/have_you_ever_smoked.jinja

@@ -1,6 +1,16 @@
 {% extends 'layout.jinja' %}
 {% from 'nhsuk/components/button/macro.jinja' import button %}
 {% from 'nhsuk/components/radios/macro.jinja' import radios %}
+{% from 'nhsuk/components/back-link/macro.jinja' import backLink %}
+
+{% block beforeContent %}
+  {{
+    backLink({
+      "href": url("questions:start"),
+      "text": "Back"
+    })
+  }}
+{% endblock beforeContent %}
 
 {% block content %}
   <div class="nhsuk-grid-row">

lung_cancer_screening/questions/jinja2/responses.jinja

@@ -5,7 +5,7 @@
 {% block beforeContent %}
   {{
     backLink({
-      "href": url("questions:weight"),
+      "href": url("questions:sex_at_birth"),
       "text": "Back"
     })
   }}
@@ -20,6 +20,7 @@
         <li>What is your date of birth? {{ response_set.date_of_birth }}</li>
         <li>What is your height? {{ response_set.formatted_height }}</li>
         <li>What is your weight? {{ response_set.formatted_weight }}</li>
+        <li>What was your sex at birth? {{ response_set.get_sex_at_birth_display() }}</li>
       </ul>
 
       <form action="{{ request.path }}" method="post">

lung_cancer_screening/questions/jinja2/sex_at_birth.jinja

@@ -0,0 +1,28 @@
+{% extends 'layout.jinja' %}
+{% from 'nhsuk/components/button/macro.jinja' import button %}
+{% from 'nhsuk/components/back-link/macro.jinja' import backLink %}
+
+{% block beforeContent %}
+  {{
+    backLink({
+      "href": url("questions:weight"),
+      "text": "Back"
+    })
+  }}
+{% endblock beforeContent %}
+
+{% block page_content %}
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-two-thirds">
+      <form action="{{ request.path }}" method="POST">
+        {{ csrf_input }}
+
+        {{ form }}
+
+        {{ button({
+          "text": "Continue"
+        }) }}
+      </form>
+    </div>
+  </div>
+{% endblock %}

lung_cancer_screening/questions/migrations/0013_responseset_sex_at_birth.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.2.7 on 2025-10-28 11:09
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('questions', '0012_responseset_weight_imperial_and_more'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='responseset',
+            name='sex_at_birth',
+            field=models.CharField(blank=True, choices=[('F', 'Female'), ('M', 'Male')], max_length=1, null=True),
+        ),
+    ]

lung_cancer_screening/questions/models/response_set.py

@@ -14,6 +14,10 @@ class HaveYouEverSmokedValues(models.IntegerChoices):
     YES_BUT_ONLY_A_FEW_TIMES = 2, 'Yes, but only a few times'
     NO_I_HAVE_NEVER_SMOKED = 3, 'No, I have never smoked'
 
+class SexAtBirthValues(models.TextChoices):
+    FEMALE = "F", 'Female'
+    MALE = "M", 'Male'
+
 class ResponseSet(BaseModel):
     participant = models.ForeignKey(Participant, on_delete=models.CASCADE)
 
@@ -54,6 +58,14 @@ class ResponseSet(BaseModel):
         MaxValueValidator(
             MAX_WEIGHT_IMPERIAL, message="Weight must be between 4 stone and 50 stone"),
     ])
+
+    sex_at_birth = models.CharField(
+        max_length=1,
+        choices=SexAtBirthValues.choices,
+        null=True,
+        blank=True
+    )
+
     submitted_at = models.DateTimeField(null=True, blank=True)
 
     class Meta:

lung_cancer_screening/questions/tests/unit/forms/test_sex_at_birth_form.py

@@ -0,0 +1,48 @@
+from django.test import TestCase
+
+from ....models.response_set import SexAtBirthValues
+from ....models.participant import Participant
+from ....forms.sex_at_birth_form import SexAtBirthForm
+
+class TestSexAtBirthForm(TestCase):
+    def setUp(self):
+        self.participant = Participant.objects.create(unique_id="1234567890")
+
+    def test_is_valid_with_a_valid_value(self):
+        form = SexAtBirthForm(
+            participant=self.participant,
+            data={
+                "sex_at_birth": SexAtBirthValues.FEMALE
+            }
+        )
+        self.assertTrue(form.is_valid())
+        self.assertEqual(
+            form.cleaned_data["sex_at_birth"],
+            SexAtBirthValues.FEMALE.value
+        )
+
+    def test_is_invalid_with_an_invalid_value(self):
+        form = SexAtBirthForm(
+            participant=self.participant,
+            data={
+                "sex_at_birth": "invalid"
+            }
+        )
+        self.assertFalse(form.is_valid())
+        self.assertEqual(
+            form.errors["sex_at_birth"],
+            ["Select a valid choice. invalid is not one of the available choices."]
+        )
+
+    def test_is_invalid_when_no_option_is_selected(self):
+        form = SexAtBirthForm(
+            participant=self.participant,
+            data={
+                "sex_at_birth": None
+            }
+        )
+        self.assertFalse(form.is_valid())
+        self.assertEqual(
+            form.errors["sex_at_birth"],
+            ["Select your sex at birth."]
+        )