Merge pull request #409 from sul-dlss/deprecate_get_preserved_file

Add deprecation warning to get_preserved_file

lib/dor/models/concerns/contentable.rb

@@ -71,6 +71,7 @@ def replace_file(file, file_name)
     def get_preserved_file(file, version)
       Sdr::Client.get_preserved_file_content(pid, file, version)
     end
+    deprecation_deprecate get_preserved_file: 'use dor-services-app/v1/sdr/objects/:druid/content/:filename instead'
 
     def get_file(file)
       druid_tools = DruidTools::Druid.new(pid, Config.content.content_base_dir)

spec/models/concerns/contentable_spec.rb

@@ -142,10 +142,12 @@ def internal_uri
     let(:preserved_file_content) { 'expected content' }
     it 'should get the file content' do
       expect(Sdr::Client).to receive(:get_preserved_file_content).with(@item.id, filename, item_version).and_return(preserved_file_content)
+      expect(Deprecation).to receive(:warn)
       returned_content = @item.get_preserved_file(filename, item_version)
       expect(returned_content).to eq(preserved_file_content)
     end
   end
+
   describe 'get_file' do
     it 'should fetch the file' do
       data_file = File.new(File.join(fixture_dir, 'ab123cd4567_descMetadata.xml'))