Merge pull request #240 from NHSDigital/PPHA-526-types-smoking-tobacco

PPHA-526: Add types of tobacco smoked

Author: jamiefalcus

features/date_of_birth_page.feature

@@ -43,7 +43,7 @@ Feature: Date of birth page
 
   Scenario: Checking responses and changing them
     Given I am logged in
-    And I have answered have you ever smoked with an eligible response
+    And I have answered questions showing I am eligible
     When I go to "/date-of-birth"
     And I fill in and submit my date of birth as 55 years ago
     When I go to "/check-your-answers"

features/have_you_ever_smoked_page.feature

@@ -30,6 +30,7 @@ Feature: Have you ever smoked page
 
   Scenario: Checking responses and changing them
     Given I am logged in
+    And I have answered questions showing I am eligible
     When I go to "/have-you-ever-smoked"
     And I fill in and submit my smoking status with "Yes, I used to smoke"
     When I go to "/check-your-answers"

features/periods_when_you_stopped_smoking.feature

@@ -26,7 +26,7 @@ Feature: Periods when you stopped smoking page
     When I check "Yes"
     And I fill in "Enter the total number of years you stopped smoking for" with "1"
     And I submit the form
-    Then I am on "/check-your-answers"
+    Then I am on "/types-tobacco-smoking"
 
   Scenario: Checking responses and changing them
     Given I am logged in

features/questionnaire.feature

@@ -65,9 +65,12 @@ Feature: Questionnaire
     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 "/periods-when-you-stopped-smoking"
+    Then I am on "/types-tobacco-smoking"
+    When I check "Cigarettes"
+    And I check "Pipe"
+    And I submit the form
 
+    Then I am on "/check-your-answers"
     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"
 

features/steps/debug_steps.py

@@ -0,0 +1,20 @@
+from behave import when
+from datetime import datetime
+
+@given("I take a screenshot")
+@given("I take a screenshot {value}")
+@when("I take a screenshot")
+@when("I take a screenshot {value}")
+@then("I take a screenshot")
+@then("I take a screenshot {value}")
+def screenshot(context, value=""):
+    context.page.screenshot(
+        full_page=True, path=f"screenshots/{datetime.now()}-{value}-screenshot.png"
+    )
+
+@given("I print eligibiity")
+@when("I print eligibiity")
+@then("I print eligibiity")
+def print_eligibiity(context):
+    print("Is user eligible?: ", context.current_user.responseset_set.last().is_eligible())
+

features/steps/form_steps.py

@@ -14,11 +14,6 @@ def when_i_enter_value_and_submit(context, field, value):
     context.page.get_by_label(field).fill(value)
     when_i_submit_the_form(context)
 
-@when("I take a screenshot")
-@when("I take a screenshot {value}")
-def screenshot(context, value=""):
-    context.page.screenshot(full_page=True, path=f"screenshots/{datetime.now()}-{value}-screenshot.png")
-
 @when('I fill in and submit my smoking status with "{smoking_status}"')
 def when_i_fill_in_and_submit_my_smoking_status(context, smoking_status):
     context.page.get_by_label(smoking_status).check()

features/types_tobacco_smoking.feature

@@ -0,0 +1,45 @@
+@TypesTobaccoSmoking
+Feature: Types tobacco smoking page
+  Scenario: The page is accessible
+    Given I am logged in
+    And I have answered questions showing I am eligible
+    When I go to "/types-tobacco-smoking"
+    Then there are no accessibility violations
+
+  Scenario: Form errors
+    Given I am logged in
+    And I have answered questions showing I am eligible
+    When I go to "/types-tobacco-smoking"
+    And I click "Continue"
+    Then I am on "/types-tobacco-smoking"
+    And I see a form error "Select the type of tobacco you smoke or have smoked"
+    And there are no accessibility violations
+
+  Scenario: Navigating backwards and forwards
+    Given I am logged in
+    And I have answered questions showing I am eligible
+    When I go to "/types-tobacco-smoking"
+    Then I see a back link to "/periods-when-you-stopped-smoking"
+    When I check "Cigarettes"
+    And I submit the form
+    Then I am on "/check-your-answers"
+
+  Scenario: Checking responses and changing them
+    Given I am logged in
+    And I have answered questions showing I am eligible
+    When I go to "/types-tobacco-smoking"
+    And I check "Cigarettes"
+    And I check "Cigars"
+    And I submit the form
+    When I go to "/check-your-answers"
+    Then I take a screenshot
+    Then I see "Cigarettes and Cigars" as a response to "Types of tobacco smoked" under "Smoking history"
+    And I see "/types-tobacco-smoking?change=True" as a link to change "Types of tobacco smoked" under "Smoking history"
+    When I click the link to change "Types of tobacco smoked" under "Smoking history"
+    Then I am on "/types-tobacco-smoking?change=True"
+    And I see "Cigarettes" selected
+    And I see "Cigars" selected
+    When I check "Pipe"
+    And I click "Continue"
+    Then I am on "/check-your-answers"
+    And I see "Cigarettes, Pipe, and Cigars" as a response to "Types of tobacco smoked" under "Smoking history"

lung_cancer_screening/questions/forms/types_tobacco_smoking_form.py

@@ -0,0 +1,59 @@
+from django import forms
+
+from ...nhsuk_forms.choice_field import MultipleChoiceField
+from ..models.tobacco_smoking_history import TobaccoSmokingHistoryTypes, TobaccoSmokingHistory
+
+
+
+class TypesTobaccoSmokingForm(forms.Form):
+
+    value = MultipleChoiceField(
+        choices=TobaccoSmokingHistoryTypes.choices,
+        widget=forms.CheckboxSelectMultiple,
+        label="What do you or have you smoked?",
+        label_classes="nhsuk-fieldset__legend--m",
+        hint="Select all that apply",
+        error_messages={
+            "required": "Select the type of tobacco you smoke or have smoked"
+        },
+    )
+
+    def __init__(self, *args, response_set, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.response_set = response_set
+        self.existing_types = list(
+            response_set.tobacco_smoking_history.values_list('type', flat=True)
+        )
+        self.fields["value"].initial = self.existing_types
+
+        value_field=self['value']
+        value_field.add_hint_for_choice(
+            TobaccoSmokingHistoryTypes.CIGARILLOS,
+            "Cafe Creme or Signature cigars, roughly the size of a cigarette"
+        )
+
+
+    def save(self, commit=True):
+        if not self.is_valid():
+            return None
+
+        self._delete_types_not_selected()
+        return self._create_types_selected()
+
+
+    def _delete_types_not_selected(self):
+        for kind in self.existing_types:
+            if kind not in self.cleaned_data["value"]:
+                TobaccoSmokingHistory.objects.filter(response_set=self.response_set, type=kind).delete()
+
+
+    def _create_types_selected(self):
+        instances = [
+            TobaccoSmokingHistory(response_set=self.response_set, type=kind)
+            for kind in self.cleaned_data["value"]
+            if kind not in self.existing_types
+        ]
+
+        TobaccoSmokingHistory.objects.bulk_create(instances)
+
+        return instances

lung_cancer_screening/questions/jinja2/responses.jinja

@@ -40,7 +40,7 @@
       </section>
 
       <section>
-      <h2 class="nhsuk-heading-m">Smoking history</h2>
+        <h2 class="nhsuk-heading-m">Smoking history</h2>
 
         {{ summaryList({
           "rows": response_set.smoking_history_responses_items()

lung_cancer_screening/questions/jinja2/types_tobacco_smoking.jinja

@@ -0,0 +1,11 @@
+{% extends 'question_form.jinja' %}
+
+{% block prelude %}
+  <h1 class="nhsuk-heading-l">The type of tobacco you smoke or used to smoke</h1>
+
+  <p>Smoking affects your health in different ways depending on what you smoke.</p>
+
+  <p>We need to know what you smoke, or used to smoke, on a regular basis. For example, if you have smoked a pipe on a weekly basis for a year or longer.</p>
+
+  <p>You do not need to tell us about less frequent forms of smoking. For example, a cigar on special occasions.</p>
+{% endblock prelude %}