Create PDSSearchResult from daily job

For data consistency, and to enable flaggind PDS info in the patient activity log, it would help to store any NHS numbers retrieved from the daily lookup job as well. This is unlikely to happen given that the dialy job is more restrictive than the import pds jobs. But int he event it does, we should have the info stored.

Author: murugapl

app/jobs/patient_nhs_number_lookup_job.rb

@@ -26,9 +26,22 @@ def perform(patient)
        )
       PatientMerger.call(to_keep: existing_patient, to_destroy: patient)
       existing_patient.update_from_pds!(pds_patient)
+
+      PDSSearchResult.create!(
+        patient_id: existing_patient.id,
+        step: :no_fuzzy_with_history_daily,
+        result: :one_match,
+        nhs_number: pds_patient.nhs_number
+      )
     else
       patient.nhs_number = pds_patient.nhs_number
       patient.update_from_pds!(pds_patient)
+      PDSSearchResult.create!(
+        patient_id: patient.id,
+        step: :no_fuzzy_with_history_daily,
+        result: :one_match,
+        nhs_number: pds_patient.nhs_number
+      )
     end
   end
 end

app/models/pds_search_result.rb

@@ -31,12 +31,13 @@ class PDSSearchResult < ApplicationRecord
   enum :step,
        {
          no_fuzzy_with_history: 0,
-         no_fuzzy_without_history: 1,
-         no_fuzzy_with_wildcard_postcode: 2,
-         no_fuzzy_with_wildcard_given_name: 3,
-         no_fuzzy_with_wildcard_family_name: 4,
-         fuzzy_without_history: 5,
-         fuzzy_with_history: 6
+         no_fuzzy_with_history_daily: 1,
+         no_fuzzy_without_history: 2,
+         no_fuzzy_with_wildcard_postcode: 3,
+         no_fuzzy_with_wildcard_given_name: 4,
+         no_fuzzy_with_wildcard_family_name: 5,
+         fuzzy_without_history: 6,
+         fuzzy_with_history: 7
        },
        validate: true
 

spec/jobs/patient_nhs_number_lookup_job_spec.rb

@@ -60,6 +60,13 @@
         expect(patient).to receive(:update_from_pds!)
         perform_now
       end
+
+      it "creates a PDSSearchResult" do
+        expect { perform_now }.to change(PDSSearchResult, :count).by(1)
+        expect(PDSSearchResult.last.step).to eq("no_fuzzy_with_history_daily")
+        expect(PDSSearchResult.last.result).to eq("one_match")
+        expect(PDSSearchResult.last.nhs_number).to eq("9449306168")
+      end
     end
 
     context "with a match and the patient already exists" do