PPHA-525: Add Periods when you stopped smoking page

Author: Themitchell

features/family_history_lung_cancer.feature

@@ -18,7 +18,7 @@ Feature: Family history of lung cancer page
     When I go to "/family-history-lung-cancer"
     Then I see a back link to "/cancer-diagnosis"
     When I fill in and submit my asbestos exposure with "No"
-    Then I am on "/check-your-answers"
+    Then I am on "/periods-when-you-stopped-smoking"
     When I click "Back"
     When I fill in and submit my family history lung cancer with "Yes"
     Then I am on "/relatives-age-when-diagnosed"

features/periods_when_you_stopped_smoking.feature

@@ -0,0 +1,42 @@
+@PeriodsWhenYouStoppedSmoking
+Feature: Periods when you stopped smoking page
+  Scenario: The page is accessible
+    Given I am logged in
+    When I go to "/periods-when-you-stopped-smoking"
+    # TODO: problem with aria expanded from nhsuk frontend
+    # Then there are no accessibility violations
+
+  Scenario: Form errors
+    Given I am logged in
+    When I go to "/periods-when-you-stopped-smoking"
+    When I submit the form
+    Then I am on "/periods-when-you-stopped-smoking"
+    And I see a form error "Select if you ever stopped smoking for periods of 1 year or longer"
+    # TODO: problem with aria expanded from nhsuk frontend
+    # And there are no accessibility violations
+
+  Scenario: Navigating backwards and forwards
+    Given I am logged in
+    When I go to "/periods-when-you-stopped-smoking"
+    Then I see a back link to "/family-history-lung-cancer"
+    When I check "Yes"
+    And I fill in "Enter the total number of years you stopped smoking for" with "10"
+    And I submit the form
+    Then I am on "/check-your-answers"
+
+  Scenario: Checking responses and changing them
+    Given I am logged in
+    When I go to "/periods-when-you-stopped-smoking"
+    And I check "Yes"
+    And I fill in "Enter the total number of years you stopped smoking for" with "10"
+    And I submit the form
+    When I go to "/check-your-answers"
+    Then I see "Yes (10 years)" as a response to "Have you ever stopped smoking for periods of 1 year or longer?" under "Smoking history"
+    And I see "/periods-when-you-stopped-smoking?change=True" as a link to change "Have you ever stopped smoking for periods of 1 year or longer?" under "Smoking history"
+    When I click the link to change "Have you ever stopped smoking for periods of 1 year or longer?" under "Smoking history"
+    Then I am on "/periods-when-you-stopped-smoking?change=True"
+    And I see "Yes" selected
+    When I check "No"
+    And I submit the form
+    Then I am on "/check-your-answers"
+    And I see "No" as a response to "Have you ever stopped smoking for periods of 1 year or longer?" under "Smoking history"

features/questionnaire.feature

@@ -56,12 +56,20 @@ Feature: Questionnaire
     Then I am on "/relatives-age-when-diagnosed"
     When I fill in and submit my relatives age when diagnosed with "Yes, they were younger than 60"
 
+    Then I am on "/periods-when-you-stopped-smoking"
+    When I check "Yes"
+    And I fill in "Enter the total number of years you stopped smoking for" with "10"
+    And I submit the form
+
     Then I am on "/check-your-answers"
-    And I see a back link to "/relatives-age-when-diagnosed"
+    And I see a back link to "/periods-when-you-stopped-smoking"
 
     And I see "Yes, I used to smoke" as a response to "Have you ever smoked tobacco?" under "Eligibility"
     And I see a date 55 years ago as a response to "Date of birth" under "Eligibility"
 
+    And I see "Yes, I used to smoke" as a response to "Have you ever smoked tobacco?" under "Smoking history"
+    And I see "Yes (10 years)" as a response to "Have you ever stopped smoking for periods of 1 year or longer?" under "Smoking history"
+
     And I see "5 feet 7 inches" as a response to "Height" under "About you"
     And I see "5 stone 10 pounds" as a response to "Weight" under "About you"
     And I see "Male" as a response to "Sex at birth" under "About you"

features/relatives_age_when_diagnosed.feature

@@ -24,7 +24,7 @@ Feature: Relatives age when diagnosed page
     Then I am on "/relatives-age-when-diagnosed"
     And I see a back link to "/family-history-lung-cancer"
     When I fill in and submit my relatives age when diagnosed with "Yes, they were younger than 60"
-    Then I am on "/check-your-answers"
+    Then I am on "/periods-when-you-stopped-smoking"
 
   Scenario: Redirecting if they have no family history of lung cancer
     Given I am logged in

features/steps/accessibility_steps.py

@@ -9,4 +9,5 @@ def then_there_are_no_accessibility_violations(context):
         f"Found the following accessibility violations: \n"
         f"{axe_results.generate_snapshot()}"
     )
+    print(axe_results)
     assert axe_results.violations_count == 0, violations_msg

features/steps/form_steps.py

@@ -105,6 +105,11 @@ def when_i_fill_in_and_submit_my_cancer_diagnosis(context, relatives_age_when_di
     context.page.get_by_label(relatives_age_when_diagnosed, exact=True).check()
     when_i_submit_the_form(context)
 
+@when(u'I fill in and submit my periods when you stopped smoking with "{periods_when_you_stopped_smoking}"')
+def when_i_fill_in_and_submit_my_periods_when_you_stopped_smoking(context, periods_when_you_stopped_smoking):
+    context.page.get_by_label(periods_when_you_stopped_smoking, exact=True).check()
+    when_i_submit_the_form(context)
+
 @when('I submit the form')
 def when_i_submit_the_form(context):
     when_i_click(context, "Continue")
@@ -129,3 +134,8 @@ def then_i_see_value_filled_in_for_label(context, value, label):
 @when('I check "{label}"')
 def when_i_check_label(context, label):
     context.page.get_by_label(label, exact=True).check()
+
+
+@when('I fill in "{label}" with "{value}"')
+def when_i_fill_in_label_with_value(context, label, value):
+    context.page.get_by_label(label, exact=True).fill(value)

lung_cancer_screening/assets/js/index.js

@@ -1 +1,6 @@
-import "nhsuk-frontend/packages/nhsuk.js";
+import { initAll } from "nhsuk-frontend";
+
+document.addEventListener("DOMContentLoaded", () => {
+  // Initialise NHS.UK frontend components
+  initAll();
+});

lung_cancer_screening/questions/forms/periods_when_you_stopped_smoking_form.py

@@ -0,0 +1,37 @@
+from django import forms
+
+from ...nhsuk_forms.integer_field import IntegerField
+from ...nhsuk_forms.typed_choice_field import TypedChoiceField
+from ..models.periods_when_you_stopped_smoking_response import PeriodsWhenYouStoppedSmokingResponse
+
+
+class PeriodsWhenYouStoppedSmokingForm(forms.ModelForm):
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        self.fields["value"] = TypedChoiceField(
+            choices=[(True, "Yes"), (False, "No")],
+            widget=forms.RadioSelect,
+            label="Have you ever stopped smoking for periods of 1 year or longer?",
+            label_classes="nhsuk-fieldset__legend--m",
+            coerce=lambda x: x == "True",
+            error_messages={
+                "required": "Select if you ever stopped smoking for periods of 1 year or longer"
+            },
+        )
+
+        self.fields["duration_years"] = IntegerField(
+            label="Enter the total number of years you stopped smoking for",
+            label_classes="nhsuk-fieldset__legend--s",
+            classes="nhsuk-input--width-4",
+            hint="Give an estimate if you are not sure",
+            required=False,
+            suffix="years",
+            error_messages={
+                "required": "Enter the total number of years you stopped smoking for"
+            },
+        )
+
+    class Meta:
+        model = PeriodsWhenYouStoppedSmokingResponse
+        fields = ["value", "duration_years"]

lung_cancer_screening/questions/jinja2/periods_when_you_stopped_smoking.jinja

@@ -0,0 +1,37 @@
+{% extends 'layout.jinja' %}
+
+{% from 'nhsuk/components/button/macro.jinja' import button %}
+
+{% if error %}
+  {% set error_message = { "text": error } %}
+{% endif %}
+
+{% block page_content %}
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-two-thirds">
+      <form action="{{ request.path }}" method="POST">
+        {{ csrf_input }}
+
+        {% if request.GET.get("change") == "True" %}
+          <input type="hidden" name="change" value="True">
+        {% endif %}
+
+        <h1 class="nhsuk-heading-l">Periods when you stopped smoking</h1>
+
+        <p>There may have been periods when you stopped or quit smoking.</p>
+        <p>If you stopped smoking for periods of 1 year or longer, tell us the total number of years you stopped smoking.</p>
+
+        {% do form.value.add_conditional_html(
+          True,
+          form.duration_years.as_field_group()
+        ) %}
+
+        {{ form.value.as_field_group() }}
+
+        {{ button({
+          "text": "Continue"
+        }) }}
+      </form>
+    </div>
+  </div>
+{% endblock %}

lung_cancer_screening/questions/jinja2/responses.jinja

@@ -17,6 +17,13 @@
         }) }}
       </section>
 
+      <section>
+        <h2 class="nhsuk-heading-m">Smoking history</h2>
+        {{ summaryList({
+          "rows": response_set.smoking_history_responses_items()
+        }) }}
+      </section>
+
       <section>
         <h2 class="nhsuk-heading-m">About you</h2>
         {{ summaryList({