Add AppPatientSessionRegistrationComponent

Shows a registration card on the patient session page for sessions that require registration. Displays a form to mark the child as attending or absent when not yet registered, a status summary with an update link when already registered, and a completion message when the session is done

Author: murugapl

app/components/app_patient_session_registration_component.html.erb

@@ -0,0 +1,44 @@
+<%= render AppCardComponent.new(section: true) do |card| %>
+  <% card.with_heading(level: 2) { "Register attendance" } %>
+
+  <% if registration_status.completed? %>
+    <p><%= patient.given_name %> has completed this session.</p>
+  <% elsif registration_status.unknown? %>
+    <p><%= patient.full_name %> is not registered yet.</p>
+    <% if can_edit? %>
+      <hr class="nhsuk-section-break nhsuk-section-break--visible nhsuk-section-break--m">
+      <%= form_with model: attendance_record,
+                    url: session_patient_attendance_path(session, patient),
+                    method: :put,
+                    builder: MavisFormBuilder do |f| %>
+        <%= f.govuk_radio_buttons_fieldset(
+              :attending,
+              legend: { size: "s", text: "Is #{patient.full_name} attending today’s session?" },
+            ) do %>
+          <%= f.govuk_radio_button(:attending, true,
+                                   label: { text: "Yes, they are attending today’s session" },
+                                   link_errors: true) %>
+          <%= f.govuk_radio_button(:attending, false,
+                                   label: { text: "No, they are absent from today’s session" }) %>
+        <% end %>
+        <%= f.govuk_submit "Update attendance", class: "nhsuk-u-margin-bottom-0" %>
+      <% end %>
+    <% end %>
+  <% elsif registration_status.attending? %>
+    <p><%= patient.full_name %> is attending today’s session.</p>
+    <% if can_edit? %>
+      <%= render AppActionLinkComponent.new(
+            text: "Update attendance",
+            href: edit_session_patient_attendance_path(session, patient),
+          ) %>
+    <% end %>
+  <% elsif registration_status.not_attending? %>
+    <p><%= patient.full_name %> is absent from today’s session.</p>
+    <% if can_edit? %>
+      <%= render AppActionLinkComponent.new(
+            text: "Update attendance",
+            href: edit_session_patient_attendance_path(session, patient),
+          ) %>
+    <% end %>
+  <% end %>
+<% end %>

app/components/app_patient_session_registration_component.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+class AppPatientSessionRegistrationComponent < ViewComponent::Base
+  def initialize(patient:, session:)
+    @patient = patient
+    @session = session
+  end
+
+  def render?
+    session.requires_registration? && session.today?
+  end
+
+  private
+
+  attr_reader :patient, :session
+
+  delegate :policy, to: :helpers
+
+  def registration_status
+    @registration_status ||= patient.registration_status(session:)
+  end
+
+  def attendance_record
+    @attendance_record ||=
+      patient
+        .attendance_records
+        .find_or_initialize_by(location: session.location, date: Date.current)
+        .tap { it.session = session }
+  end
+
+  def can_edit?
+    @can_edit ||= policy(attendance_record).edit?
+  end
+end

app/views/patient_sessions/_header.html.erb

@@ -37,22 +37,6 @@
       <% end %>
     <% end %>
   <% end %>
-
-  <% if @session.requires_registration? && (date = @session.dates.find(&:today?)) %>
-    <% attendance_record = @patient.attendance_records.find_or_initialize_by(location: @session.location, date:) %>
-
-    <% action_list.with_item do %>
-      <%= render AppStatusTagComponent.new(@patient.registration_status(session: @session)&.status || "unknown", context: :registration) %>
-    <% end %>
-
-    <% attendance_record.session = @session %>
-    <% if policy(attendance_record).edit? %>
-      <% action_list.with_item(
-           text: "Update attendance",
-           href: edit_session_patient_attendance_path(@session, @patient),
-         ) %>
-    <% end %>
-  <% end %>
 <% end %>
 
 <%= render AppStickyNavigationComponent.new do %>

app/views/patient_sessions/programmes/show.html.erb

@@ -12,6 +12,8 @@
 
     <%= render AppPatientSessionTriageComponent.new(patient: @patient, session: @session, programme: @programme, current_user:, triage_form: @triage_form) %>
 
+    <%= render AppPatientSessionRegistrationComponent.new(patient: @patient, session: @session) %>
+  
     <%= render AppPatientSessionRecordComponent.new(patient: @patient, session: @session, programme: @programme, current_user:, vaccinate_form: @vaccinate_form) %>
   </div>
 </div>

spec/components/app_patient_session_registration_component_spec.rb

@@ -0,0 +1,117 @@
+# frozen_string_literal: true
+
+describe AppPatientSessionRegistrationComponent do
+  subject(:rendered) { render_inline(component) }
+
+  let(:component) { described_class.new(patient:, session:) }
+  let(:programme) { Programme.flu }
+  let(:session) { create(:session, :today, programmes: [programme]) }
+  let(:patient) { create(:patient, session:) }
+
+  before { stub_authorization(allowed: true) }
+
+  context "when the session does not require registration" do
+    let(:session) do
+      create(
+        :session,
+        :today,
+        :requires_no_registration,
+        programmes: [programme]
+      )
+    end
+
+    it "does not render" do
+      expect(component.render?).to be false
+    end
+  end
+
+  context "when the session is not active today" do
+    let(:session) { create(:session, :scheduled, programmes: [programme]) }
+
+    it "does not render" do
+      expect(component.render?).to be false
+    end
+  end
+
+  context "when the child has not been registered yet" do
+    it { should have_heading("Register attendance") }
+    it { should have_css("form") }
+    it { should have_text("Yes, they are attending today’s session") }
+    it { should have_text("No, they are absent from today’s session") }
+    it { should have_button("Update attendance") }
+    it { should_not have_link("Update attendance") }
+
+    context "when the user cannot edit" do
+      before { stub_authorization(allowed: false) }
+
+      it { should have_text(patient.full_name) }
+      it { should_not have_css("form") }
+    end
+  end
+
+  context "when the child is attending" do
+    before do
+      create(:patient_registration_status, :attending, patient:, session:)
+    end
+
+    it { should have_heading("Register attendance") }
+
+    it do
+      expect(rendered).to have_text(
+        "#{patient.full_name} is attending today’s session."
+      )
+    end
+
+    it { should have_link("Update attendance") }
+    it { should_not have_css("form") }
+
+    context "when the user cannot edit" do
+      before { stub_authorization(allowed: false) }
+
+      it do
+        expect(rendered).to have_text(
+          "#{patient.full_name} is attending today’s session."
+        )
+      end
+
+      it { should_not have_link("Update attendance") }
+    end
+  end
+
+  context "when the child is absent" do
+    before do
+      create(:patient_registration_status, :not_attending, patient:, session:)
+    end
+
+    it do
+      expect(rendered).to have_text(
+        "#{patient.full_name} is absent from today’s session."
+      )
+    end
+
+    it { should have_link("Update attendance") }
+    it { should_not have_css("form") }
+
+    context "when the user cannot edit" do
+      before { stub_authorization(allowed: false) }
+
+      it do
+        expect(rendered).to have_text(
+          "#{patient.full_name} is absent from today’s session."
+        )
+      end
+
+      it { should_not have_link("Update attendance") }
+    end
+  end
+
+  context "when the child has completed the session" do
+    before do
+      create(:patient_registration_status, :completed, patient:, session:)
+    end
+
+    it { should have_text("#{patient.given_name} has completed this session.") }
+    it { should_not have_link("Update registration") }
+    it { should_not have_css("form") }
+  end
+end

spec/features/manage_attendance_spec.rb

@@ -22,14 +22,14 @@
     then_the_patient_is_not_registered_yet
     and_i_am_not_able_to_vaccinate
 
-    when_i_choose_the_patient_has_not_been_registered_yet
-    then_the_patient_is_not_registered_yet
-    and_i_see_the_not_registered_flash
-
     when_i_choose_the_patient_is_absent
     then_the_patient_is_absent
     and_i_see_the_absent_flash
 
+    when_i_choose_the_patient_has_not_been_registered_yet
+    then_the_patient_is_not_registered_yet
+    and_i_see_the_not_registered_flash
+
     when_i_choose_the_patient_is_attending
     then_the_patient_is_attending
     and_i_see_the_attending_flash
@@ -192,7 +192,7 @@ def and_i_go_to_a_patient
   end
 
   def then_the_patient_is_not_registered_yet
-    expect(page).to have_content("Not registered yet")
+    expect(page).to have_content("registered yet")
   end
 
   def and_i_am_not_able_to_vaccinate
@@ -210,13 +210,12 @@ def and_i_see_the_not_registered_flash
   end
 
   def when_i_choose_the_patient_is_absent
-    click_link "Update attendance"
-    choose "No, they are absent"
-    click_button "Save changes"
+    choose "No, they are absent from today’s session"
+    click_button "Update attendance"
   end
 
   def then_the_patient_is_absent
-    expect(page).to have_content("Absent from session")
+    expect(page).to have_content("is absent from today’s session")
   end
 
   def and_i_see_the_absent_flash
@@ -225,13 +224,12 @@ def and_i_see_the_absent_flash
   alias_method :then_i_see_the_absent_flash, :and_i_see_the_absent_flash
 
   def when_i_choose_the_patient_is_attending
-    click_link "Update attendance"
     choose "Yes, they are attending"
-    click_button "Save changes"
+    click_button "Update attendance"
   end
 
   def then_the_patient_is_attending
-    expect(page).to have_content("Attending session")
+    expect(page).to have_content("is attending today’s session")
   end
 
   def and_i_see_the_attending_flash