[WIP] Increasing coverage for ReindexJob

app/jobs/spotlight/reindex_job.rb

@@ -46,6 +46,9 @@ def resource_list(exhibit_or_resources)
         end
       end
 
+      # If it has been passed to the Job, retrieve the log entry for the reindexing
+      # @param job [Spotlight::ReindexJob]
+      # @return [Spotlight::ReindexingLogEntry]
       def log_entry(job)
         job.arguments.second if job.arguments.second.is_a?(Spotlight::ReindexingLogEntry)
       end

spec/jobs/spotlight/reindex_job_spec.rb

@@ -29,4 +29,36 @@
     expect(resources.first).to have_received(:reindex)
     expect(resources.last).to have_received(:reindex)
   end
+
+  context 'with an existing log entry' do
+    let(:log_entry) { Spotlight::ReindexingLogEntry.new }
+    let(:resource1) { instance_double(IIIFResource, reindex: true) }
+    let(:resource2) { instance_double(IIIFResource, reindex: true) }
+    let(:resources) { [resource1, resource2] }
+    let(:builder) { instance_double(Spotlight::SolrDocumentBuilder) }
+
+    before do
+      allow(builder).to receive(:documents_to_index).and_return([0])
+      allow(resource1).to receive(:document_builder).and_return(builder)
+      allow(resource2).to receive(:document_builder).and_return(builder)
+      allow(log_entry).to receive(:update)
+    end
+
+    it 'estimates the number of items being reindexed' do
+      described_class.perform_now(resources, log_entry)
+      expect(log_entry).to have_received(:update).with(items_reindexed_estimate: 2)
+    end
+
+    context 'when the job fails' do
+      before do
+        allow(resource1).to receive(:reindex).and_raise(StandardError)
+        allow(log_entry).to receive(:failed!)
+      end
+
+      it 'sets the state of the job to failure within the log entry' do
+        expect { described_class.perform_now(resources, log_entry) }.to raise_error(StandardError)
+        expect(log_entry).to have_received(:failed!)
+      end
+    end
+  end
 end