Merge pull request #4165 from nhsuk/show-notify-parents-on-vaccination

Show `notify_parents_on_vaccination` on confirmation

Author: thomasleese

app/components/app_consent_summary_component.rb

@@ -54,6 +54,22 @@ def call
         end
       end
 
+      unless consent.notify_parents_on_vaccination.nil?
+        summary_list.with_row do |row|
+          row.with_key { "Confirmation of vaccination sent to parent?" }
+          row.with_value do
+            consent.notify_parents_on_vaccination ? "Yes" : "No"
+          end
+          if (href = change_links[:notify_parents_on_vaccination])
+            row.with_action(
+              text: "Change",
+              visually_hidden_text: "decision",
+              href:
+            )
+          end
+        end
+      end
+
       if consent.reason_for_refusal.present?
         summary_list.with_row do |row|
           row.with_key { "Reason for refusal" }
@@ -63,7 +79,7 @@ def call
 
       unless consent.notify_parent_on_refusal.nil?
         summary_list.with_row do |row|
-          row.with_key { "Confirmation of decision sent to parent" }
+          row.with_key { "Confirmation of decision sent to parent?" }
           row.with_value { consent.notify_parent_on_refusal ? "Yes" : "No" }
         end
       end

app/views/draft_consents/confirm.html.erb

@@ -16,6 +16,7 @@
   <%= render AppConsentSummaryComponent.new(@draft_consent, change_links: {
                                                               response: wizard_path("agree"),
                                                               route: wizard_path(@draft_consent.via_self_consent? ? "who" : "route"),
+                                                              notify_parents_on_vaccination: wizard_path("notify-parents-on-vaccination"),
                                                             }) %>
 <% end %>
 

spec/components/app_consent_summary_component_spec.rb

@@ -35,6 +35,30 @@
     it { should have_content("Notes") }
   end
 
+  it { should_not have_content("Confirmation of vaccination sent to parent?") }
+
+  context "when the child doesn't want the parents to know about the vaccination" do
+    let(:consent) { create(:consent, :given, :self_consent) }
+
+    it do
+      expect(rendered).to have_content(
+        "Confirmation of vaccination sent to parent?\nNo"
+      )
+    end
+  end
+
+  context "when the child wants the parents to know about the vaccination" do
+    let(:consent) do
+      create(:consent, :given, :self_consent, :notify_parents_on_vaccination)
+    end
+
+    it do
+      expect(rendered).to have_content(
+        "Confirmation of vaccination sent to parent?\nYes"
+      )
+    end
+  end
+
   it { should_not have_content("Consent also given for injected vaccine?") }
 
   context "with the flu programme" do

spec/features/self_consent_spec.rb

@@ -246,6 +246,8 @@ def and_the_child_can_give_their_own_consent_that_the_nurse_records
     choose "Child (Gillick competent)"
     5.times { click_on "Continue" }
 
+    expect(page).to have_content("Confirmation of vaccination sent to parent")
+
     click_on "Confirm"
 
     expect(page).to have_content("Consent recorded for #{@patient.full_name}")

spec/features/verbal_consent_given_spec.rb

@@ -159,6 +159,9 @@ def record_that_verbal_consent_was_given(
   def then_i_see_the_check_and_confirm_page
     expect(page).to have_content("Check and confirm answers")
     expect(page).to have_content(["Method", "By phone"].join)
+    expect(page).not_to have_content(
+      "Confirmation of vaccination sent to parent"
+    )
   end
 
   def and_i_see_the_flu_injection_consent_given

spec/features/verbal_consent_refused_personal_choice_spec.rb

@@ -65,9 +65,11 @@ def when_i_record_the_consent_refusal_and_reason(notify_parent:)
     expect(page).to have_content(["Name", @parent.full_name].join)
 
     if notify_parent
-      expect(page).to have_content("Confirmation of decision sent to parentYes")
+      expect(page).to have_content(
+        "Confirmation of decision sent to parent?Yes"
+      )
     else
-      expect(page).to have_content("Confirmation of decision sent to parentNo")
+      expect(page).to have_content("Confirmation of decision sent to parent?No")
     end
 
     click_button "Confirm"