Merge pull request #3954 from nhsuk/ensure-consent-emails-send

Ensure consented_vaccine_methods_message is set for consents

Author: thomasleese

app/lib/govuk_notify_personalisation.rb

@@ -129,10 +129,15 @@ def consent_link
   end
 
   def consented_vaccine_methods_message
-    return nil if consent_form.nil?
-    return "" if consent_form.programmes.none?(&:flu?)
+    return nil if consent.nil? && consent_form.nil?
 
-    consent_form_programmes = consent_form.consent_form_programmes
+    if (consent && !consent.programme.flu?) ||
+         (consent_form && consent_form.programmes.none?(&:flu?))
+      return ""
+    end
+
+    consent_form_programmes =
+      consent ? [consent] : consent_form.consent_form_programmes
 
     consented_vaccine_methods =
       if consent_form_programmes.any?(&:vaccine_method_injection_and_nasal?)

spec/lib/govuk_notify_personalisation_spec.rb

@@ -194,11 +194,46 @@
     it do
       expect(to_h).to match(
         hash_including(
+          consented_vaccine_methods_message: "",
           reason_for_refusal: "of personal choice",
           survey_deadline_date: "8 January 2024"
         )
       )
     end
+
+    context "for the flu programme" do
+      let(:programmes) { [create(:programme, :flu)] }
+
+      it do
+        expect(to_h).to include(
+          consented_vaccine_methods_message:
+            "You’ve agreed that John can have the injected flu vaccine."
+        )
+      end
+
+      context "when consented to both nasal and injection" do
+        before { consent.update!(vaccine_methods: %w[nasal injection]) }
+
+        it do
+          expect(to_h).to include(
+            consented_vaccine_methods_message:
+              "You’ve agreed that John can have the nasal spray flu vaccine, " \
+                "or the injected flu vaccine if the nasal spray is not suitable."
+          )
+        end
+      end
+
+      context "when consented only to nasal" do
+        before { consent.update!(vaccine_methods: %w[nasal]) }
+
+        it do
+          expect(to_h).to include(
+            consented_vaccine_methods_message:
+              "You’ve agreed that John can have the nasal spray flu vaccine."
+          )
+        end
+      end
+    end
   end
 
   context "with a consent form" do