Redirect users to exit page if not correct age

Author: Themitchell

lung_cancer_screening/core/tests/acceptance/test_questionnaire.py

@@ -1,7 +1,8 @@
 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
 
 class TestQuestionnaire(StaticLiveServerTestCase):
 
@@ -33,12 +34,14 @@ def test_full_questionaire_user_journey(self):
         expect(page.locator("legend")).to_have_text(
             "What is your date of birth?")
 
-        page.fill("input[name='day']", "8")
-        page.fill("input[name='month']", "9")
-        page.fill("input[name='year']", "2000")
+        age = datetime.now() - relativedelta(years=55)
+
+        page.fill("input[name='day']", str(age.day))
+        page.fill("input[name='month']", str(age.month))
+        page.fill("input[name='year']", str(age.year))
 
         page.click("text=Continue")
 
         expect(page).to_have_url(f"{self.live_server_url}/responses")
 
-        expect(page.locator(".responses")).to_have_text("2000-09-08")
+        expect(page.locator(".responses")).to_have_text(age.strftime("%Y-%m-%d"))

lung_cancer_screening/core/tests/acceptance/test_user_out_of_age_range.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
+
+
+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_participant_out_of_age_range(self):
+        participant_id = '123'
+
+        page = self.browser.new_page()
+        page.goto(f"{self.live_server_url}/start")
+
+        page.fill("input[name='participant_id']", participant_id)
+
+        page.click('text=Start now')
+
+        expect(page).to_have_url(f"{self.live_server_url}/date-of-birth")
+
+        expect(page.locator("legend")).to_have_text(
+            "What is your date of birth?")
+
+        age = datetime.now() - relativedelta(years=20)
+
+        page.fill("input[name='day']", str(age.day))
+        page.fill("input[name='month']", str(age.month))
+        page.fill("input[name='year']", str(age.year))
+
+        page.click("text=Continue")
+
+        expect(page).to_have_url(f"{self.live_server_url}/age-range-exit")
+
+        expect(page.locator(".title")).to_have_text(
+            "You do not need an NHS lung health check")

lung_cancer_screening/questions/jinja2/age_range_exit.jinja

@@ -0,0 +1,13 @@
+{% extends 'layout.jinja' %}
+{% from 'button/macro.jinja' import button %}
+{% from 'input/macro.jinja' import input %}
+
+{% block content %}
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-two-thirds">
+      <h2 class="title">You do not need an NHS lung health check</h2>
+
+      <p>The NHS lung health check is for people between the ages of 55 and 74.</p>
+    </div>
+  </div>
+{% endblock %}

lung_cancer_screening/questions/tests/unit/views/test_date_of_birth.py

@@ -1,13 +1,27 @@
 from django.test import TestCase
 from django.urls import reverse
 from datetime import date
+from dateutil.relativedelta import relativedelta
 
 from lung_cancer_screening.questions.models.participant import Participant
 from lung_cancer_screening.questions.models.questionnaire_response import QuestionnaireResponse
 
 class TestPostDateOfBirth(TestCase):
     def setUp(self):
         self.participant = Participant.objects.create(unique_id="12345")
+        self.valid_age = date.today() - relativedelta(years=55)
+        self.valid_params = {
+            "day": self.valid_age.day,
+            "month": self.valid_age.month,
+            "year": self.valid_age.year
+        }
+
+        self.invalid_age = date.today() - relativedelta(years=20)
+        self.invalid_params = {
+            "day": self.invalid_age.day,
+            "month": self.invalid_age.month,
+            "year": self.invalid_age.year
+        }
 
         session = self.client.session
         session['participant_id'] = self.participant.unique_id
@@ -44,33 +58,33 @@ def test_post_redirects_if_the_particpant_does_not_exist(self):
 
         response = self.client.post(
             reverse("questions:date_of_birth"),
-            {"day": "8", "month": "9", "year": "2000"}
+            self.valid_params
         )
 
         self.assertRedirects(response, reverse("questions:start"))
 
     def test_post_stores_a_valid_questionnaire_response_for_the_participant(self):
         self.client.post(
             reverse("questions:date_of_birth"),
-            {"day": "8", "month": "9", "year": "2000"}
+            self.valid_params
         )
 
         questionnaire_response = QuestionnaireResponse.objects.first()
-        self.assertEqual(questionnaire_response.value, date(2000, 9, 8))
+        self.assertEqual(questionnaire_response.value, self.valid_age)
         self.assertEqual(questionnaire_response.participant, self.participant)
 
     def test_post_sets_the_participant_id_in_session(self):
         self.client.post(
             reverse("questions:date_of_birth"),
-            {"day": "8", "month": "9", "year": "2000"}
+            self.valid_params
         )
 
         self.assertEqual(self.client.session["participant_id"], "12345")
 
     def test_post_redirects_to_responses_path(self):
         response = self.client.post(
             reverse("questions:date_of_birth"),
-            {"day": "8", "month": "9", "year": "2000"}
+            self.valid_params
         )
 
         self.assertRedirects(response, reverse("questions:responses"))
@@ -82,3 +96,19 @@ def test_post_responds_with_422_if_the_questionnaire_response_fails_to_create(se
         )
 
         self.assertEqual(response.status_code, 422)
+
+    def test_post_does_not_create_a_questionnaire_response_if_the_user_is_not_in_the_correct_age_range(self):
+        response = self.client.post(
+            reverse("questions:date_of_birth"),
+            self.invalid_params
+        )
+
+        self.assertEqual(QuestionnaireResponse.objects.count(), 0)
+
+    def test_post_redirects_if_the_user_is_not_in_the_correct_age_range(self):
+        response = self.client.post(
+            reverse("questions:date_of_birth"),
+            self.invalid_params
+        )
+
+        self.assertRedirects(response, reverse("questions:age_range_exit"))

lung_cancer_screening/questions/urls.py

@@ -18,9 +18,11 @@
 from .views.start import start
 from .views.date_of_birth import date_of_birth
 from .views.responses import responses
+from .views.age_range_exit import age_range_exit
 
 urlpatterns = [
     path('start', start, name='start'),
     path('date-of-birth', date_of_birth, name='date_of_birth'),
-    path('responses', responses, name='responses')
+    path('responses', responses, name='responses'),
+    path('age-range-exit', age_range_exit, name='age_range_exit'),
 ]

lung_cancer_screening/questions/views/age_range_exit.py

@@ -0,0 +1,11 @@
+from django.shortcuts import render, redirect
+from django.urls import reverse
+from django.core.exceptions import ValidationError
+
+from lung_cancer_screening.questions.models.participant import Participant
+
+def age_range_exit(request):
+    return render(
+        request,
+        "age_range_exit.jinja"
+    )

lung_cancer_screening/questions/views/date_of_birth.py

@@ -1,6 +1,7 @@
 from django.shortcuts import render, redirect
 from django.urls import reverse
 from datetime import date
+from dateutil.relativedelta import relativedelta
 
 from ..models.participant import Participant
 from ..models.questionnaire_response import QuestionnaireResponse
@@ -21,12 +22,18 @@ def date_of_birth(request):
                 int(request.POST['day'])
             )
 
-            QuestionnaireResponse.objects.create(
-                participant=participant,
-                value=value
-            )
+            fifty_five_years_ago = date.today() - relativedelta(years=55)
+            seventy_five_years_ago = date.today() - relativedelta(years=75)
+
+            if value in (fifty_five_years_ago, seventy_five_years_ago):
+                QuestionnaireResponse.objects.create(
+                    participant=participant,
+                    value=value
+                )
 
-            return redirect(reverse("questions:responses"))
+                return redirect(reverse("questions:responses"))
+            else:
+                return redirect(reverse("questions:age_range_exit"))
 
         # TODO: Understand how to do validation for dates - either model
         # or Form validator

poetry.lock

@@ -11,6 +11,7 @@ dependencies = [
     "psycopg2-binary (>=2.9.10,<3.0.0)",
     "nhsuk-frontend-jinja (>=0.3.1,<0.4.0)",
     "gunicorn (>=23.0.0,<24.0.0)",
+    "python-dateutil (>=2.9.0.post0,<3.0.0)",
 ]
 
 [tool.poetry]