[CPDLP-3632] Overrides application ukprn from schools table when the …

…school ukprn pulled in from ECF is nil

app/services/migration/migrators/application.rb

@@ -72,8 +72,10 @@ def call
         if ecf_npq_application.school_urn.present?
           application.school_id = find_school_id!(urn: ecf_npq_application.school_urn)
 
-          if application.school.ukprn.to_s == ecf_npq_application.school_ukprn.to_s
+          if ecf_npq_application.school_ukprn && ecf_npq_application.school_ukprn.to_s == application.school.ukprn.to_s
             application.ukprn = ecf_npq_application.school_ukprn
+          elsif ecf_npq_application.school_ukprn.blank?
+            application.ukprn = application.school.ukprn
           else
             ecf_npq_application.errors.add(:base, "School UKPRN does not match")
             raise ActiveRecord::RecordInvalid, ecf_npq_application

spec/services/migration/migrators/application_spec.rb

@@ -161,6 +161,19 @@ def setup_failure_state
           expect(failure_manager).to have_received(:record_failure).once.with(ecf_resource1, /Validation failed: School UKPRN does not match/)
         end
       end
+
+      context "when the school ukprn pulled in from ECF is nil" do
+        before { ecf_resource1.update!(school_ukprn: nil) }
+
+        it "overrides ukprn from Schools table on the NPQ application" do
+          application = Application.find_by!(ecf_id: ecf_resource1.id)
+          expect(application.ukprn).to be_nil
+
+          instance.call
+
+          expect(application.reload.ukprn).to eq(application.school.ukprn)
+        end
+      end
     end
   end
 end