Add filtering patients by date of birth

This makes it possible to filter patients by their date of birth when
looking at the patients page for a particular programme.

Author: thomasleese

app/components/app_session_patient_table_component.html.erb

@@ -25,6 +25,18 @@
                                           "data-turbo-permanent": "true" %>
       <% end %>
 
+      <% if @columns.include?(:dob) %>
+        <%= f.govuk_text_field :dob, label: { text: "Date of birth" },
+                                     hint: {
+                                       text: "e.g. 2005 or 01/03/2014",
+                                       class: "nhsuk-u-font-size-16",
+                                     },
+                                     value: params[:dob],
+                                     "data-autosubmit-target": "field",
+                                     "data-action": "autosubmit#submit",
+                                     "data-turbo-permanent": "true" %>
+      <% end %>
+
       <% if @columns.include?(:year_group) %>
         <%= f.govuk_check_boxes_fieldset :year_groups, legend: { text: "Year group", size: "s" } do %>
           <% year_groups.each do |year_group| %>

app/controllers/concerns/patient_sorting_concern.rb

@@ -52,6 +52,13 @@ def filter_patients!(patients_or_patient_sessions)
       end
     end
 
+    if (date_of_birth = params[:dob]).present?
+      patients_or_patient_sessions.select! do
+        value = _1.try(:date_of_birth) || _1.patient.date_of_birth
+        value.to_fs(:uk_short).include?(date_of_birth)
+      end
+    end
+
     if (year_groups = params[:year_groups]).present?
       patients_or_patient_sessions.select! do
         value = _1.try(:year_group) || _1.patient.year_group

spec/components/app_session_patient_table_component_spec.rb

@@ -156,5 +156,11 @@ def have_column(text)
 
       it { should have_column("Status") }
     end
+
+    context "includes date of birth" do
+      let(:columns) { %i[name dob] }
+
+      it { should have_column("Date of birth") }
+    end
   end
 end

spec/features/patient_sorting_filtering_spec.rb

@@ -50,6 +50,22 @@
     then_i_see_patients_ordered_by_name_desc
   end
 
+  scenario "Users can sort and filter date of birth" do
+    given_that_i_am_signed_in
+    when_i_visit_the_programme_patients_page
+    then_i_see_patients_ordered_by_name_asc # Initial server load is name asc
+
+    when_i_click_on_the_dob_header
+    then_i_see_patients_ordered_by_dob_asc
+
+    when_i_click_on_the_dob_header
+    then_i_see_patients_ordered_by_dob_desc
+
+    when_i_filter_by_dob
+    and_i_click_filter
+    then_i_see_patients_with_dob
+  end
+
   def given_that_i_am_signed_in
     @programme = create(:programme, :hpv)
     @organisation =
@@ -83,6 +99,10 @@ def when_i_visit_the_consents_page
     visit session_consents_path(@session)
   end
 
+  def when_i_visit_the_programme_patients_page
+    visit patients_programme_path(@programme)
+  end
+
   def when_i_click_on_the_name_header
     click_link "Full name"
     sleep 0.1
@@ -156,4 +176,27 @@ def when_i_reset_filters
     expect(page).to have_button "Reset filters", disabled: false
     click_button "Reset filters"
   end
+
+  def when_i_click_on_the_dob_header
+    click_link "Date of birth"
+  end
+
+  def then_i_see_patients_ordered_by_dob_asc
+    expect(page).to have_selector("tr:nth-child(4)", text: "Alex")
+    expect(page).to have_selector("tr:nth-child(3)", text: "Blair")
+  end
+
+  def then_i_see_patients_ordered_by_dob_desc
+    expect(page).to have_selector("tr:nth-child(1)", text: "Alex")
+    expect(page).to have_selector("tr:nth-child(2)", text: "Blair")
+  end
+
+  def when_i_filter_by_dob
+    fill_in "Date of birth", with: "2011"
+  end
+
+  def then_i_see_patients_with_dob
+    expect(page).not_to have_selector("tr:nth-child(2)")
+    expect(page).to have_selector("tr:nth-child(1)", text: "Alex")
+  end
 end