Add AppPatientSessionPsdComponent

Shows a PSD status card ("PSD added" / "PSD not added") on the patient session page, only rendered when the session has PSD enabled.

Author: murugapl

app/components/app_patient_session_psd_component.html.erb

@@ -0,0 +1,5 @@
+<%= render AppCardComponent.new(section: true) do |card| %>
+  <% card.with_heading(level: 2) { "Patient Specific Directions (PSD)" } %>
+
+  <p><%= psd_status %></p>
+<% end %>

app/components/app_patient_session_psd_component.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+class AppPatientSessionPsdComponent < ViewComponent::Base
+  def initialize(patient:, session:, programme:)
+    @patient = patient
+    @session = session
+    @programme = programme
+  end
+
+  def render?
+    session.psd_enabled?
+  end
+
+  private
+
+  attr_reader :patient, :session, :programme
+
+  delegate :academic_year, :team, to: :session
+
+  def psd_status
+    has_patient_specific_direction? ? "PSD added" : "PSD not added"
+  end
+
+  def has_patient_specific_direction?
+    patient
+      .patient_specific_directions
+      .not_invalidated
+      .for_programme(programme)
+      .where(team:, academic_year:)
+      .exists?
+  end
+end

app/views/patient_sessions/_header.html.erb

@@ -27,18 +27,6 @@
   <%= patient_year_group(@patient, academic_year: @academic_year) %>
 </p>
 
-<%= render AppActionListComponent.new do |action_list| %>
-  <% if @programme && @session.psd_enabled? %>
-    <% action_list.with_item do %>
-      <% if @patient.has_patient_specific_direction?(academic_year: @academic_year, programme_type: @programme.type, team: current_team) %>
-        <%= render AppStatusTagComponent.new(:added, context: :patient_specific_direction) %>
-      <% else %>
-        <%= render AppStatusTagComponent.new(:not_added, context: :patient_specific_direction) %>
-      <% end %>
-    <% end %>
-  <% end %>
-<% end %>
-
 <%= render AppStickyNavigationComponent.new do %>
   <%= render AppSecondaryNavigationComponent.new do |nav|
         @session.programmes_for(patient: @patient).each do |programme|

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 AppPatientSessionPsdComponent.new(patient: @patient, session: @session, programme: @programme) %>
+
     <%= render AppPatientSessionRegistrationComponent.new(patient: @patient, session: @session) %>
 
     <%= render AppPatientSessionRecordComponent.new(patient: @patient, session: @session, programme: @programme, current_user:, vaccinate_form: @vaccinate_form) %>

spec/components/app_patient_session_psd_component_spec.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+describe AppPatientSessionPsdComponent do
+  subject(:rendered) { render_inline(component) }
+
+  let(:component) { described_class.new(patient:, session:, programme:) }
+
+  let(:programme) { Programme.flu }
+  let(:session) { create(:session, :psd_enabled, programmes: [programme]) }
+  let(:patient) { create(:patient, session:) }
+
+  context "when the session does not use PSDs" do
+    let(:session) { create(:session, programmes: [programme]) }
+
+    it "does not render" do
+      expect(component.render?).to be false
+    end
+  end
+
+  context "when the session uses PSDs" do
+    it { should have_heading("Patient Specific Directions (PSD)") }
+
+    context "and the patient does not have a PSD" do
+      it { should have_text("PSD not added") }
+    end
+
+    context "and the patient has a PSD" do
+      before do
+        create(
+          :patient_specific_direction,
+          patient:,
+          programme:,
+          team: session.team,
+          academic_year: session.academic_year
+        )
+      end
+
+      it { should have_text("PSD added") }
+    end
+
+    context "and the patient has an invalidated PSD" do
+      before do
+        create(
+          :patient_specific_direction,
+          :invalidated,
+          patient:,
+          programme:,
+          team: session.team,
+          academic_year: session.academic_year
+        )
+      end
+
+      it { should have_text("PSD not added") }
+    end
+  end
+end

spec/features/triage_required_spec.rb

@@ -456,11 +456,11 @@ def and_the_vaccine_method_is_recorded_as_nasal
   end
 
   def then_i_should_see_the_patient_tagged_psd_added
-    within(".app-action-list") { expect(page).to have_content("PSD added") }
+    expect(page).to have_content("PSD added")
   end
 
   def then_i_should_see_the_patient_tagged_psd_not_added
-    within(".app-action-list") { expect(page).to have_content("PSD not added") }
+    expect(page).to have_content("PSD not added")
   end
 
   def then_i_should_see_the_patient_with_status_psd_added