Merge pull request #530 from sul-dlss/deprecate-reindex_pid_list

Deprecate reindex_pid_list
sul-dlss-deprecated · Apr 9, 2019 · 8df263d · 8df263d
2 parents 4a4b00b + 6e7b0d1
commit 8df263d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/lib/dor/services/indexing_service.rb b/lib/dor/services/indexing_service.rb
@@ -119,6 +119,7 @@ def self.reindex_pid(pid, *args)
 
     # given a list of pids, retrieve those objects from fedora, index each to solr, optionally commit
     def self.reindex_pid_list(pid_list, should_commit = false)
+      Deprecation.warn(self, 'reindex_pid_list is deprecated and will be removed in dor-services 7')
       pid_list.each { |pid| reindex_pid pid, raise_errors: false } # use the default logger, don't let individual errors nuke the rest of the batch
       ActiveFedora.solr.conn.commit if should_commit
     end

diff --git a/spec/services/indexing_service_spec.rb b/spec/services/indexing_service_spec.rb
@@ -94,6 +94,7 @@
 
   describe '#reindex_pid_list' do
     before do
+      expect(Deprecation).to receive(:warn)
       @mock_solr_conn = double(ActiveFedora.solr.conn)
     end