🧹 Make DeleteJob work wth new class method .find

The DeleteJob previously was not working with the
old factory#find method because when it is doing a
delete action, the parsed_metadata does not get
generated like during a regular import.  Because
of this, the #search_by_identifier method fails to
find anything because we don't have a
`work_identifier` field which would have came from
the parsed_metadata.  So instead, we are using the
new class method .find which will take an id
(which we find on the raw_metadata) to find the
object.  We make sure to reindex and publish the
action to any relevant listeners.

Author: Kirk Wang

app/jobs/bulkrax/delete_job.rb

@@ -5,8 +5,16 @@ class DeleteJob < ApplicationJob
     queue_as :import
 
     def perform(entry, importer_run)
-      obj = entry.factory.find
-      obj&.delete
+      obj = entry.factory.class.find(entry.raw_metadata["id"])
+
+      if defined?(Valkyrie) && obj.class < Valkyrie::Resource
+        Hyrax.persister.delete(resource: obj)
+        Hyrax.index_adapter.delete(resource: obj)
+        Hyrax.publisher.publish('object.deleted', object: obj, user: importer_run.importer.user)
+      else
+        obj&.delete
+      end
+
       # rubocop:disable Rails/SkipsModelValidations
       ImporterRun.increment_counter(:deleted_records, importer_run.id)
       ImporterRun.decrement_counter(:enqueued_records, importer_run.id)

spec/jobs/bulkrax/delete_work_job_spec.rb

@@ -7,14 +7,19 @@ module Bulkrax
     subject(:delete_work_job) { described_class.new }
     let(:entry) { create(:bulkrax_entry) }
     let(:importer_run) { create(:bulkrax_importer_run) }
+    let(:factory) do
+      Bulkrax::ObjectFactory.new(attributes: {},
+                                 source_identifier_value: '123',
+                                 work_identifier: :source,
+                                 work_identifier_search_field: :source_identifier)
+    end
 
     describe 'successful job object removed' do
       before do
         work = instance_double("Work")
-        factory = instance_double("Bulkrax::ObjectFactory")
-        expect(work).to receive(:delete).and_return true
-        expect(factory).to receive(:find).and_return(work)
-        expect(entry).to receive(:factory).and_return(factory)
+        allow(work).to receive(:delete).and_return true
+        allow(factory.class).to receive(:find).and_return(work)
+        allow(entry).to receive(:factory).and_return(factory)
       end
 
       it 'increments :deleted_records' do
@@ -31,9 +36,8 @@ module Bulkrax
 
     describe 'successful job object not found' do
       before do
-        factory = instance_double("Bulkrax::ObjectFactory")
-        expect(factory).to receive(:find).and_return(nil)
-        expect(entry).to receive(:factory).and_return(factory)
+        allow(factory.class).to receive(:find).and_return(nil)
+        allow(entry).to receive(:factory).and_return(factory)
       end
 
       it 'increments :deleted_records' do