Merge pull request #5765 from nhsuk/mike/breadcrumb-for-national-reporting-vaccs-record

Never show session-based breadcrumb in national reporting vaccination records

Author: mikejamesthompson

app/controllers/vaccination_records_controller.rb

@@ -67,7 +67,7 @@ def set_breadcrumb_items
       { text: t("dashboard.index.title"), href: dashboard_path }
     ]
 
-    if @session
+    if @session && policy(@session).show?
       @breadcrumb_items << {
         text: t("sessions.index.title"),
         href: sessions_path

spec/features/edit_vaccination_record_spec.rb

@@ -263,6 +263,15 @@
       then_i_should_see_the_vaccination_record
       and_i_should_see_parent_details
     end
+
+    scenario "Breadcrumb shows session-specific path for POC team" do
+      given_i_am_signed_in
+      and_an_administered_vaccination_record_exists
+
+      when_i_visit_the_vaccination_record_directly
+      then_i_should_see_the_vaccination_record
+      and_i_should_see_the_session_specific_breadcrumb
+    end
   end
 
   context "in bulk upload Mavis" do
@@ -296,6 +305,16 @@
       then_i_should_see_the_vaccination_record
       and_i_should_not_see_parent_details
     end
+
+    scenario "Breadcrumb shows patient-based path when viewing session-based vaccination record" do
+      given_i_am_signed_in
+      and_a_vaccination_record_with_a_session_exists
+      and_the_patient_is_accessible_to_the_upload_only_team
+
+      when_i_visit_the_vaccination_record_directly
+      then_i_should_see_the_vaccination_record
+      and_i_should_see_the_patient_based_breadcrumb
+    end
   end
 
   def given_an_hpv_programme_is_underway
@@ -709,4 +728,43 @@ def and_i_should_not_see_parent_details
     expect(page).not_to have_content("First parent or guardian")
     expect(page).not_to have_content("Second parent or guardian")
   end
+
+  def when_i_visit_the_vaccination_record_directly
+    visit vaccination_record_path(@vaccination_record)
+  end
+
+  def and_i_should_see_the_session_specific_breadcrumb
+    breadcrumb = page.find(".nhsuk-breadcrumb")
+    expect(breadcrumb).to have_content("Sessions")
+    expect(breadcrumb).to have_content(@session.location.name)
+  end
+
+  def and_a_vaccination_record_with_a_session_exists
+    location = create(:school, urn: 100_001)
+
+    @session = create(:session, :completed, programmes: [@programme], location:)
+
+    @vaccination_record =
+      create(
+        :vaccination_record,
+        batch: @batch,
+        patient: @patient,
+        session: @session,
+        programme: @programme
+      )
+  end
+
+  def and_the_patient_is_accessible_to_the_upload_only_team
+    # Patient is already part of the upload-only team from the before block
+    # Just ensure patient_team association exists
+    PatientTeam.find_or_create_by!(patient: @patient, team: @team)
+  end
+
+  def and_i_should_see_the_patient_based_breadcrumb
+    # Patient-based breadcrumb should include: Home → Children → Patient name
+    # Should NOT include session-specific links (Sessions, Location name)
+    breadcrumb = page.find(".nhsuk-breadcrumb")
+    expect(breadcrumb).to have_content("Children")
+    expect(breadcrumb).not_to have_content("Sessions")
+  end
 end