Ensure participants can only submit one ResponseSet per year

Author: Themitchell

lung_cancer_screening/core/tests/acceptance/test_cannot_change_answers_after_submission.py

@@ -0,0 +1,49 @@
+import os
+from django.contrib.staticfiles.testing import StaticLiveServerTestCase
+from playwright.sync_api import sync_playwright, expect
+from datetime import datetime
+from dateutil.relativedelta import relativedelta
+
+from .helpers.user_interaction_helpers import (
+    fill_in_and_submit_participant_id,
+    fill_in_and_submit_smoking_elligibility,
+    fill_in_and_submit_date_of_birth
+)
+
+
+class TestQuestionnaire(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 test_cannot_change_responses_once_checked_and_submitted(self):
+        participant_id = '123'
+        smoking_status = 'Yes, I used to smoke regularly'
+        age = datetime.now() - relativedelta(years=55)
+
+        page = self.browser.new_page()
+        page.goto(f"{self.live_server_url}/start")
+
+        fill_in_and_submit_participant_id(page, participant_id)
+        fill_in_and_submit_smoking_elligibility(page, smoking_status)
+        fill_in_and_submit_date_of_birth(page, age)
+
+        page.click("text=Submit")
+
+        page.goto(f"{self.live_server_url}/start")
+
+        fill_in_and_submit_participant_id(page, participant_id)
+
+        expect(page.locator('#maincontent')).to_contain_text(
+            "Responses have already been submitted for this participant"
+        )

lung_cancer_screening/core/tests/acceptance/test_questionnaire.py

@@ -49,3 +49,7 @@ def test_full_questionaire_user_journey(self):
         expect(page.locator(".responses")).to_contain_text(
             age.strftime("Have you ever smoked? Yes, I used to smoke regularly"))
         expect(page.locator(".responses")).to_contain_text(age.strftime("What is your date of birth? %Y-%m-%d"))
+
+        page.click("text=Submit")
+
+        expect(page).to_have_url(f"{self.live_server_url}/your-results")

lung_cancer_screening/core/tests/acceptance/test_questionnaire_validation_errors.py

@@ -4,8 +4,7 @@
 
 from .helpers.user_interaction_helpers import (
     fill_in_and_submit_participant_id,
-    fill_in_and_submit_smoking_elligibility,
-    fill_in_and_submit_date_of_birth
+    fill_in_and_submit_smoking_elligibility
 )
 
 class TestQuestionnaireValidationErrors(StaticLiveServerTestCase):

lung_cancer_screening/questions/forms/date_of_birth_form.py

@@ -6,7 +6,9 @@
 
 class DateOfBirthForm(forms.ModelForm):
     def __init__(self, *args, **kwargs):
+        self.participant = kwargs.pop('participant')
         super().__init__(*args, **kwargs)
+        self.instance.participant = self.participant
 
         self.fields["date_of_birth"] = SplitDateField(
             max_value=date.today(),

lung_cancer_screening/questions/forms/have_you_ever_smoked_form.py

@@ -6,7 +6,9 @@
 class HaveYouEverSmokedForm(forms.ModelForm):
 
     def __init__(self, *args, **kwargs):
+        self.participant = kwargs.pop('participant')
         super().__init__(*args, **kwargs)
+        self.instance.participant = self.participant
 
         self.fields["have_you_ever_smoked"] = TypedChoiceField(
             choices=HaveYouEverSmokedValues.choices,

lung_cancer_screening/questions/jinja2/responses.jinja

@@ -1,5 +1,5 @@
 {% extends 'layout.jinja' %}
-
+{% from 'button/macro.jinja' import button %}
 
 {% block content %}
   <div class="nhsuk-grid-row">
@@ -9,6 +9,14 @@
         <li>Have you ever smoked? {{ response_set.get_have_you_ever_smoked_display() }}</li>
         <li>What is your date of birth? {{ response_set.date_of_birth }}</li>
       </ul>
+
+      <form action="{{ request.path }}" method="post">
+        {{ csrf_input }}
+
+        {{ button({
+          "text": "Submit"
+        }) }}
+      </form>
     </div>
   </div>
 {% endblock %}

lung_cancer_screening/questions/jinja2/start.jinja

@@ -1,8 +1,17 @@
 {% extends 'layout.jinja' %}
+{% from 'error-summary/macro.jinja' import errorSummary %}
 {% from 'button/macro.jinja' import button %}
 {% from 'input/macro.jinja' import input %}
 
 {% block content %}
+
+  {% if error_messages %}
+    {{ errorSummary({
+      "titleText": "There is a problem",
+      "errorList": error_messages
+    }) }}
+  {% endif %}
+
   <div class="nhsuk-grid-row">
     <div class="nhsuk-grid-column-two-thirds">
       <form action="{{ request.path }}" method="POST">

lung_cancer_screening/questions/jinja2/your_results.jinja

@@ -0,0 +1,9 @@
+{% extends 'layout.jinja' %}
+
+{% block content %}
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-two-thirds">
+      Thank you for submitting your responses
+    </div>
+  </div>
+{% endblock %}

lung_cancer_screening/questions/migrations/0007_responseset_submitted_and_more.py

@@ -0,0 +1,28 @@
+# Generated by Django 5.2.6 on 2025-09-08 15:57
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('questions', '0006_remove_dateresponse_participant_and_more'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='responseset',
+            name='submitted',
+            field=models.BooleanField(default=False),
+        ),
+        migrations.AlterField(
+            model_name='responseset',
+            name='date_of_birth',
+            field=models.DateField(blank=True, null=True),
+        ),
+        migrations.AlterField(
+            model_name='responseset',
+            name='have_you_ever_smoked',
+            field=models.IntegerField(blank=True, choices=[(0, 'Yes, I currently smoke'), (1, 'Yes, I used to smoke regularly'), (2, 'Yes, but only a few times'), (3, 'No, I have never smoked')], null=True),
+        ),
+    ]

lung_cancer_screening/questions/migrations/0008_remove_responseset_submitted_and_more.py

@@ -0,0 +1,22 @@
+# Generated by Django 5.2.6 on 2025-09-09 09:34
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('questions', '0007_responseset_submitted_and_more'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='responseset',
+            name='submitted',
+        ),
+        migrations.AddField(
+            model_name='responseset',
+            name='submitted_at',
+            field=models.DateTimeField(blank=True, null=True),
+        ),
+    ]