Merge pull request #559 from IU-Libraries-Joint-Development/essi-1435…

…_pdf_download_redirect

[ESSI-1435] fix failing unauthorized pdf download redirect

app/controllers/concerns/essi/pdf_download.rb

@@ -15,7 +15,7 @@ def pdf
                   type: 'application/pdf',
                   disposition: 'inline'
       else
-        redirect_to [main_app, curation_concern],
+        redirect_to [main_app, presenter],
                     alert: 'You do not have access to download this PDF.'
       end
     end

spec/controllers/hyrax/archival_materials_controller_spec.rb

@@ -8,4 +8,5 @@
                    Hyrax::ArchivalMaterialPresenter,
                    described_class)
   include_examples('update metadata remotely', :archival_material)
+  include_examples('pdf controller actions', :archival_material)
 end

spec/controllers/hyrax/paged_resources_controller_spec.rb

@@ -8,4 +8,5 @@
                    Hyrax::PagedResourcePresenter,
                    described_class)
   include_examples('update metadata remotely', :paged_resource)
+  include_examples('pdf controller actions', :paged_resource)
 end

spec/support/shared_examples/pdf_controller_actions.rb

@@ -0,0 +1,23 @@
+RSpec.shared_examples 'pdf controller actions' do |resource_symbol|
+
+  describe 'when logged in', :clean do
+    let(:main_app) { Rails.application.routes.url_helpers }
+    let(:user) { FactoryBot.create(:user) }
+    let(:resource) { FactoryBot.create(resource_symbol, user: user, title: ['Dummy Title'], pdf_state: 'NOT downloadable') }
+
+    describe '#pdf' do
+      context 'without authorization' do
+        before(:each) do
+          get :pdf, params: { id: resource.id }
+        end
+        it 'redirects to the resource' do
+          show_page = main_app.send("hyrax_#{resource_symbol}_path", resource, locale: 'en')
+          expect(response).to redirect_to show_page
+        end
+        it 'displays an alert' do
+          expect(flash[:alert]).to match /You do not have access to download this PDF/i
+        end
+      end
+    end
+  end
+end