Refactor how consent forms are modelled per programme

This refactors how consents forms model the information related to
giving consent to multiple programmes in the same form. This change
isn't strictly related to Flu, but future work will build upon it.

The main change is to move whether a programme has been given consent
for or not out of the `ConsentForm` model and in to the individual
`ConsentFormProgramme` models. At the moment we only support up to two
programmes in a single consent form, but this would open the door to
allowing more than two programmes in a single consent form.

Where before the model looked something like:

- `ConsentForm` with many programmes
  - `response` one of `given`, `given_one` or `refused`
  - `chosen_vaccines` either `nil` or `hpv`, `menacwy`, `td_ipv`

The model now looks something like:

- `ConsentForm` with many programmes
  - Each programme
    - `response` one of either `given` or `refused`

I've made this change in preparation for Flu where we will also need to
store the preferred vaccine method or methods and the
`ConsentFormProgramme` seemed like the right place for it. Although this
could be done without this refactor, there was a mismatch between where
we were storing the consent response and where we were storing
preferences specific to the programme.

Jira-Issue: MAV-1230

Author: thomasleese

app/components/app_consent_confirmation_component.rb

@@ -14,24 +14,22 @@ def initialize(consent_form)
   end
 
   def title
-    case response
-    when "given"
-      "Consent confirmed"
-    when "given_one"
-      chosen_programme = chosen_programmes.first.name
-      "Consent for the #{chosen_programme} vaccination confirmed"
-    when "refused"
-      "Consent refused"
+    if response_given?
+      if refused_programmes.empty?
+        "Consent confirmed"
+      else
+        "Consent for the #{given_vaccinations} confirmed"
+      end
     else
-      raise "unrecognised consent response: #{response}"
+      "Consent refused"
     end
   end
 
   private
 
-  delegate :chosen_programmes,
-           :not_chosen_programmes,
-           :response,
+  delegate :given_programmes,
+           :refused_programmes,
+           :response_given?,
            :parent_email,
            to: :@consent_form
 
@@ -40,33 +38,26 @@ def full_name
   end
 
   def panel_text
-    case response
-    when "given", "given_one"
+    if response_given?
       if @consent_form.needs_triage?
         <<-END_OF_TEXT
           As you answered ‘yes’ to some of the health questions, we need to check
-          the #{chosen_vaccinations_are} suitable for #{full_name}. We’ll review
+          the #{given_vaccinations_are} suitable for #{full_name}. We’ll review
           your answers and get in touch again soon.
         END_OF_TEXT
       else
-        "#{full_name} is due to get the #{chosen_vaccinations} at school" +
+        "#{full_name} is due to get the #{given_vaccinations} at school" +
           (session_dates.present? ? " on #{session_dates}" : "")
       end
-    when "refused"
-      "You’ve told us that you do not want #{full_name} to get the" \
-        " #{not_chosen_vaccinations} at school"
     else
-      raise "unrecognised consent response: #{response}"
+      "You’ve told us that you do not want #{full_name} to get the" \
+        " #{refused_vaccinations} at school"
     end
   end
 
-  def chosen_vaccinations
-    vaccinations_text(chosen_programmes)
-  end
+  def given_vaccinations = vaccinations_text(given_programmes)
 
-  def not_chosen_vaccinations
-    vaccinations_text(not_chosen_programmes)
-  end
+  def refused_vaccinations = vaccinations_text(refused_programmes)
 
   def vaccinations_text(programmes)
     programme_names =
@@ -79,8 +70,8 @@ def vaccinations_text(programmes)
     )
   end
 
-  def chosen_vaccinations_are
-    "#{chosen_vaccinations} #{chosen_programmes.one? ? "is" : "are"}"
+  def given_vaccinations_are
+    "#{given_vaccinations} #{given_programmes.one? ? "is" : "are"}"
   end
 
   def session_dates

app/controllers/concerns/consent_form_mailer_concern.rb

@@ -4,11 +4,9 @@ module ConsentFormMailerConcern
   extend ActiveSupport::Concern
 
   def send_consent_form_confirmation(consent_form)
-    if consent_form.consent_refused?
-      send_consent_form_confirmation_refused(consent_form)
-    else
+    if consent_form.response_given?
       ProgrammeGrouper
-        .call(consent_form.chosen_programmes)
+        .call(consent_form.given_programmes)
         .each_value do |programmes|
           if consent_form.needs_triage?
             EmailDeliveryJob.perform_later(
@@ -41,10 +39,12 @@ def send_consent_form_confirmation(consent_form)
         end
 
       ProgrammeGrouper
-        .call(consent_form.not_chosen_programmes)
+        .call(consent_form.refused_programmes)
         .each_value do |programmes|
           send_consent_form_confirmation_refused(consent_form, programmes:)
         end
+    else
+      send_consent_form_confirmation_refused(consent_form)
     end
   end
 

app/controllers/parent_interface/consent_forms/edit_controller.rb

@@ -11,6 +11,8 @@ class ConsentForms::EditController < ConsentForms::BaseController
     HOME_EDUCATED_SCHOOL_ID = "home-educated"
 
     def show
+      set_response if current_step == :consent
+
       render_wizard
     end
 
@@ -47,6 +49,7 @@ def update
           jump_to("contact-method", skip_to_confirm: true)
         end
       elsif current_step == :consent
+        @consent_form.update_programme_responses
         @consent_form.seed_health_questions
       end
 
@@ -94,7 +97,7 @@ def update_params
           parent_contact_method_type
           parent_contact_method_other_details
         ],
-        consent: %i[response chosen_vaccine],
+        consent: %i[response chosen_programme],
         reason: %i[reason],
         reason_notes: %i[reason_notes],
         address: %i[address_line_1 address_line_2 address_town address_postcode]
@@ -122,6 +125,18 @@ def set_health_answer
       @health_answer = @consent_form.health_answers[@question_number]
     end
 
+    def set_response
+      if @consent_form.response_given? && @consent_form.response_refused?
+        @consent_form.response = "given_one"
+        @consent_form.chosen_programme =
+          @consent_form.given_programmes.first&.type
+      elsif @consent_form.response_given?
+        @consent_form.response = "given"
+      elsif @consent_form.response_refused?
+        @consent_form.response = "refused"
+      end
+    end
+
     def validate_params
       case current_step
       when :date_of_birth

app/models/consent.rb

@@ -159,7 +159,7 @@ def self.from_consent_form!(consent_form, patient:, current_user:)
         consent_form.find_or_create_parent_with_relationship_to!(patient:)
 
       consent_given =
-        consent_form.chosen_programmes.map do |programme|
+        consent_form.given_programmes.map do |programme|
           patient.consents.create!(
             consent_form:,
             organisation: consent_form.organisation,
@@ -175,7 +175,7 @@ def self.from_consent_form!(consent_form, patient:, current_user:)
         end
 
       consent_refused =
-        consent_form.not_chosen_programmes.map do |programme|
+        consent_form.refused_programmes.map do |programme|
           patient.consents.create!(
             consent_form:,
             organisation: consent_form.organisation,