blacklight-spotlight 0.23.0 → 0.24.0

data/spec/controllers/spotlight/confirmations_controller_spec.rb

@@ -25,7 +25,7 @@ describe Spotlight::ConfirmationsController, type: :controller do
     end
     describe 'when the token is valid' do
       it 'updates the user' do
-        get :show, confirmation_token: raw_token
+        get :show, params: { confirmation_token: raw_token }
         expect(contact_email.reload).to be_confirmed
         expect(response).to redirect_to main_app.new_user_session_path
       end

data/spec/controllers/spotlight/contact_forms_controller_spec.rb

@@ -16,15 +16,15 @@ describe Spotlight::ContactFormsController, type: :controller do
   describe 'POST create' do
     it 'sends an email' do
       expect do
-        post :create, exhibit_id: exhibit.id, contact_form: { name: 'Joe Doe', email: 'jdoe@example.com' }
+        post :create, params: { exhibit_id: exhibit.id, contact_form: { name: 'Joe Doe', email: 'jdoe@example.com' } }
       end.to change { ActionMailer::Base.deliveries.count }.by(1)
     end
     it 'redirects back' do
-      post :create, exhibit_id: exhibit.id, contact_form: { name: 'Joe Doe', email: 'jdoe@example.com' }
-      expect(response).to redirect_to :back
+      post :create, params: { exhibit_id: exhibit.id, contact_form: { name: 'Joe Doe', email: 'jdoe@example.com' } }
+      expect(response).to redirect_to 'http://example.com'
     end
     it 'sets a flash message' do
-      post :create, exhibit_id: exhibit.id, contact_form: { name: 'Joe Doe', email: 'jdoe@example.com' }
+      post :create, params: { exhibit_id: exhibit.id, contact_form: { name: 'Joe Doe', email: 'jdoe@example.com' } }
       expect(flash[:notice]).to eq 'Thanks. Your feedback has been sent.'
     end
   end

data/spec/controllers/spotlight/contacts_controller_spec.rb

@@ -4,7 +4,7 @@ describe Spotlight::ContactsController, type: :controller do
     describe 'GET edit' do
       let(:contact) { FactoryGirl.create(:contact) }
       it 'is successful' do
-        get :edit, id: contact, exhibit_id: contact.exhibit
+        get :edit, params: { id: contact, exhibit_id: contact.exhibit }
         expect(response).to redirect_to main_app.new_user_session_path
       end
     end
@@ -22,19 +22,19 @@ describe Spotlight::ContactsController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Curation', exhibit_dashboard_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('About Pages', exhibit_about_pages_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with(contact.name, edit_exhibit_contact_path(exhibit, contact))
-        get :edit, id: contact, exhibit_id: contact.exhibit
+        get :edit, params: { id: contact, exhibit_id: contact.exhibit }
         expect(response).to be_successful
       end
     end
     describe 'PATCH update' do
       it 'is successful' do
-        patch :update, id: contact, contact: { name: 'Chester' }, exhibit_id: contact.exhibit
+        patch :update, params: { id: contact, contact: { name: 'Chester' }, exhibit_id: contact.exhibit }
         expect(response).to redirect_to exhibit_about_pages_path(exhibit)
         expect(contact.reload.name).to eq 'Chester'
       end
       it 'fails by rendering edit' do
         expect_any_instance_of(Spotlight::Contact).to receive(:update).and_return(false)
-        patch :update, id: contact, contact: { name: 'Chester' }, exhibit_id: contact.exhibit
+        patch :update, params: { id: contact, contact: { name: 'Chester' }, exhibit_id: contact.exhibit }
         expect(response).to render_template 'edit'
       end
     end
@@ -42,7 +42,7 @@ describe Spotlight::ContactsController, type: :controller do
       it 'is successful' do
         contact # force contact to be created
         expect do
-          delete :destroy, id: contact, exhibit_id: contact.exhibit
+          delete :destroy, params: { id: contact, exhibit_id: contact.exhibit }
         end.to change { Spotlight::Contact.count }.by(-1)
         expect(response).to redirect_to exhibit_about_pages_path(exhibit)
       end
@@ -53,19 +53,19 @@ describe Spotlight::ContactsController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Curation', exhibit_dashboard_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('About Pages', exhibit_about_pages_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('Add contact', new_exhibit_contact_path(exhibit))
-        get :new, exhibit_id: exhibit
+        get :new, params: { exhibit_id: exhibit }
         expect(response).to be_successful
       end
     end
     describe 'POST create' do
       it 'fails by rendering new' do
         expect_any_instance_of(Spotlight::Contact).to receive(:update).and_return(false)
-        post :create, exhibit_id: exhibit, contact: { name: 'Chester' }
+        post :create, params: { exhibit_id: exhibit, contact: { name: 'Chester' } }
         expect(response).to render_template 'new'
       end
       it 'is successful' do
         expect do
-          post :create, exhibit_id: exhibit, contact: { name: 'Chester' }
+          post :create, params: { exhibit_id: exhibit, contact: { name: 'Chester' } }
         end.to change { Spotlight::Contact.count }.by(1)
         expect(response).to redirect_to exhibit_about_pages_path(exhibit)
         expect(Spotlight::Contact.last.show_in_sidebar).to be_truthy

data/spec/controllers/spotlight/custom_fields_controller_spec.rb

@@ -12,7 +12,7 @@ describe Spotlight::CustomFieldsController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Configuration', exhibit_dashboard_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('Metadata', edit_exhibit_metadata_configuration_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('Add new field', new_exhibit_custom_field_path(exhibit))
-        get :new, exhibit_id: exhibit
+        get :new, params: { exhibit_id: exhibit }
         expect(assigns(:custom_field)).to be_a_new(Spotlight::CustomField)
       end
     end
@@ -24,7 +24,7 @@ describe Spotlight::CustomFieldsController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Configuration', exhibit_dashboard_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('Metadata', edit_exhibit_metadata_configuration_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with(field.label, edit_exhibit_custom_field_path(exhibit, field))
-        get :edit, exhibit_id: exhibit, id: field
+        get :edit, params: { exhibit_id: exhibit, id: field }
         expect(assigns(:custom_field)).to eq field
         expect(assigns(:exhibit)).to eq exhibit
       end
@@ -34,12 +34,12 @@ describe Spotlight::CustomFieldsController, type: :controller do
       describe 'with valid params' do
         it 'creates a new Page' do
           expect do
-            post :create, custom_field: { label: 'MyString' }, exhibit_id: exhibit
+            post :create, params: { custom_field: { label: 'MyString' }, exhibit_id: exhibit }
           end.to change(Spotlight::CustomField, :count).by(1)
         end
 
         it 'redirects to the exhibit metadata page' do
-          post :create, custom_field: { label: 'MyString' }, exhibit_id: exhibit
+          post :create, params: { custom_field: { label: 'MyString' }, exhibit_id: exhibit }
           expect(response).to redirect_to(edit_exhibit_metadata_configuration_path(exhibit))
         end
       end
@@ -48,7 +48,7 @@ describe Spotlight::CustomFieldsController, type: :controller do
         it "re-renders the 'new' template" do
           # Trigger the behavior that occurs when invalid params are submitted
           allow_any_instance_of(Spotlight::CustomField).to receive(:save).and_return(false)
-          post :create, custom_field: { label: 'MyString' }, exhibit_id: exhibit
+          post :create, params: { custom_field: { label: 'MyString' }, exhibit_id: exhibit }
           expect(assigns(:custom_field)).to be_a_new(Spotlight::CustomField)
           expect(response).to render_template('new')
         end

data/spec/controllers/spotlight/dashboards_controller_spec.rb

@@ -18,7 +18,7 @@ describe Spotlight::DashboardsController, type: :controller do
         expect(controller).to receive(:search_results).with(sort: 'timestamp_field desc').and_return([double(:response), [{ id: 1 }]])
         expect(controller).to receive(:add_breadcrumb).with('Home', exhibit)
         expect(controller).to receive(:add_breadcrumb).with('Dashboard', exhibit_dashboard_path(exhibit))
-        get :show, exhibit_id: exhibit.id
+        get :show, params: { exhibit_id: exhibit.id }
         expect(response).to render_template 'spotlight/dashboards/show'
         expect(assigns[:exhibit]).to eq exhibit
         expect(assigns[:pages].length).to eq exhibit.pages.length
@@ -30,7 +30,7 @@ describe Spotlight::DashboardsController, type: :controller do
       it 'loads the exhibit' do
         expect(controller).to receive(:add_breadcrumb).with('Home', exhibit)
         expect(controller).to receive(:add_breadcrumb).with('Analytics', analytics_exhibit_dashboard_path(exhibit))
-        get :analytics, exhibit_id: exhibit.id
+        get :analytics, params: { exhibit_id: exhibit.id }
         expect(response).to render_template 'spotlight/dashboards/analytics'
         expect(assigns[:exhibit]).to eq exhibit
       end
@@ -40,12 +40,12 @@ describe Spotlight::DashboardsController, type: :controller do
   describe 'when user does not have access' do
     before { sign_in FactoryGirl.create(:exhibit_visitor) }
     it 'does not allow show' do
-      get :show, exhibit_id: exhibit.id
+      get :show, params: { exhibit_id: exhibit.id }
       expect(response).to redirect_to main_app.root_path
     end
 
     it 'does not allow analytics' do
-      get :analytics, exhibit_id: exhibit.id
+      get :analytics, params: { exhibit_id: exhibit.id }
       expect(response).to redirect_to main_app.root_path
     end
   end
@@ -53,7 +53,7 @@ describe Spotlight::DashboardsController, type: :controller do
   describe 'when not logged in' do
     describe 'GET show' do
       it 'redirects to the sign in form' do
-        get :show, exhibit_id: exhibit.id
+        get :show, params: { exhibit_id: exhibit.id }
         expect(response).to redirect_to(main_app.new_user_session_path)
         expect(flash[:alert]).to be_present
         expect(flash[:alert]).to match(/You need to sign in/)

data/spec/controllers/spotlight/exhibits_controller_spec.rb

@@ -15,7 +15,7 @@ describe Spotlight::ExhibitsController, type: :controller do
 
     describe 'GET edit' do
       it 'denies access' do
-        get :edit, id: exhibit
+        get :edit, params: { id: exhibit }
         expect(response).to redirect_to main_app.root_path
         expect(flash[:alert]).to be_present
       end
@@ -32,35 +32,35 @@ describe Spotlight::ExhibitsController, type: :controller do
 
     describe 'GET new' do
       it 'is not allowed' do
-        get :new, id: exhibit
+        get :new, params: { id: exhibit }
         expect(response).to redirect_to main_app.new_user_session_path
       end
     end
 
     describe 'GET edit' do
       it 'is not allowed' do
-        get :edit, id: exhibit
+        get :edit, params: { id: exhibit }
         expect(response).to redirect_to main_app.new_user_session_path
       end
     end
 
     describe 'PATCH update' do
       it 'is not allowed' do
-        patch :update, id: exhibit
+        patch :update, params: { id: exhibit }
         expect(response).to redirect_to main_app.new_user_session_path
       end
     end
 
     describe 'PATCH process_import' do
       it 'is not allowed' do
-        patch :process_import, id: exhibit
+        patch :process_import, params: { id: exhibit }
         expect(response).to redirect_to main_app.new_user_session_path
       end
     end
 
     describe 'DELETE destroy' do
       it 'is not allowed' do
-        delete :destroy, id: exhibit
+        delete :destroy, params: { id: exhibit }
         expect(response).to redirect_to main_app.new_user_session_path
       end
     end
@@ -85,7 +85,7 @@ describe Spotlight::ExhibitsController, type: :controller do
 
       it 'is successful' do
         expect do
-          post :create, exhibit: { title: 'Some Title', slug: 'custom-slug', tag_list: '2014, R. Buckminster Fuller' }
+          post :create, params: { exhibit: { title: 'Some Title', slug: 'custom-slug', tag_list: '2014, R. Buckminster Fuller' } }
         end.to change { Spotlight::Exhibit.count }.by(1)
 
         exhibit = Spotlight::Exhibit.last
@@ -119,7 +119,7 @@ describe Spotlight::ExhibitsController, type: :controller do
           f.write '{ "title": "Foo", "subtitle": "Bar"}'
           f.rewind
           file = Rack::Test::UploadedFile.new(f.path, 'application/json')
-          patch :process_import, id: exhibit, file: file
+          patch :process_import, params: { id: exhibit, file: file }
         ensure
           f.close
           f.unlink
@@ -137,18 +137,21 @@ describe Spotlight::ExhibitsController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Home', exhibit)
         expect(controller).to receive(:add_breadcrumb).with('Configuration', exhibit_dashboard_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('General', edit_exhibit_path(exhibit))
-        get :edit, id: exhibit
+        get :edit, params: { id: exhibit }
         expect(response).to be_successful
       end
     end
 
     describe '#update' do
       it 'is successful' do
-        patch :update, id: exhibit, exhibit: {
-          title: 'Foo',
-          subtitle: 'Bar',
-          description: 'Baz',
-          contact_emails_attributes: { '0' => { email: 'bess@stanford.edu' }, '1' => { email: 'naomi@stanford.edu' } }
+        patch :update, params: {
+          id: exhibit,
+          exhibit: {
+            title: 'Foo',
+            subtitle: 'Bar',
+            description: 'Baz',
+            contact_emails_attributes: { '0' => { email: 'bess@stanford.edu' }, '1' => { email: 'naomi@stanford.edu' } }
+          }
         }
 
         expect(flash[:notice]).to eq 'The exhibit was successfully updated.'
@@ -162,11 +165,14 @@ describe Spotlight::ExhibitsController, type: :controller do
       end
 
       it 'shows errors and ignore blank emails' do
-        patch :update, id: exhibit, exhibit: {
-          title: 'Foo',
-          subtitle: 'Bar',
-          description: 'Baz',
-          contact_emails_attributes: { '0' => { email: 'bess@stanford.edu' }, '1' => { email: 'naomi@' }, '2' => { email: '' } }
+        patch :update, params: {
+          id: exhibit,
+          exhibit: {
+            title: 'Foo',
+            subtitle: 'Bar',
+            description: 'Baz',
+            contact_emails_attributes: { '0' => { email: 'bess@stanford.edu' }, '1' => { email: 'naomi@' }, '2' => { email: '' } }
+          }
         }
 
         expect(response).to be_successful
@@ -179,7 +185,7 @@ describe Spotlight::ExhibitsController, type: :controller do
 
     describe '#destroy' do
       it 'is successful' do
-        delete :destroy, id: exhibit
+        delete :destroy, params: { id: exhibit }
         expect(Spotlight::Exhibit.exists?(exhibit.id)).to be_falsey
         expect(flash[:notice]).to eq 'The exhibit was deleted.'
         expect(response).to redirect_to main_app.root_path

data/spec/controllers/spotlight/feature_pages_controller_spec.rb

@@ -16,7 +16,7 @@ describe Spotlight::FeaturePagesController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Home', exhibit_root_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('Curation', exhibit_dashboard_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with('Feature pages', exhibit_feature_pages_path(exhibit))
-        get :index, exhibit_id: exhibit
+        get :index, params: { exhibit_id: exhibit }
         expect(assigns(:pages)).to include page
         expect(assigns(:exhibit)).to eq exhibit
       end
@@ -28,7 +28,7 @@ describe Spotlight::FeaturePagesController, type: :controller do
         it 'assigns the requested page as @page' do
           expect(controller).to receive(:add_breadcrumb).with('Home', exhibit_root_path(exhibit))
           expect(controller).to receive(:add_breadcrumb).with(page.title, [exhibit, page])
-          get :show, exhibit_id: page.exhibit.id, id: page
+          get :show, params: { exhibit_id: page.exhibit.id, id: page }
           expect(assigns(:page)).to eq(page)
         end
       end
@@ -38,7 +38,7 @@ describe Spotlight::FeaturePagesController, type: :controller do
           expect(controller).to receive(:add_breadcrumb).with('Home', exhibit_root_path(exhibit))
           expect(controller).to receive(:add_breadcrumb).with(page.parent_page.title, [exhibit, page.parent_page])
           expect(controller).to receive(:add_breadcrumb).with(page.title, [exhibit, page])
-          get :show, exhibit_id: page.exhibit, id: page
+          get :show, params: { exhibit_id: page.exhibit, id: page }
           expect(assigns(:page)).to eq(page)
         end
       end
@@ -46,7 +46,7 @@ describe Spotlight::FeaturePagesController, type: :controller do
 
     describe 'GET new' do
       it 'assigns a new page as @page' do
-        get :new, exhibit_id: exhibit
+        get :new, params: { exhibit_id: exhibit }
         expect(assigns(:page)).to be_a_new(Spotlight::FeaturePage)
         expect(assigns(:page).exhibit).to eq exhibit
       end
@@ -59,7 +59,7 @@ describe Spotlight::FeaturePagesController, type: :controller do
         expect(controller).to receive(:add_breadcrumb).with('Feature pages', exhibit_feature_pages_path(exhibit))
         expect(controller).to receive(:add_breadcrumb).with(page.parent_page.title, [exhibit, page.parent_page])
         expect(controller).to receive(:add_breadcrumb).with(page.title, [:edit, exhibit, page])
-        get :edit, exhibit_id: page.exhibit.id, id: page.id
+        get :edit, params: { exhibit_id: page.exhibit.id, id: page.id }
         expect(assigns(:page)).to eq page
       end
     end
@@ -68,17 +68,17 @@ describe Spotlight::FeaturePagesController, type: :controller do
       describe 'with valid params' do
         it 'creates a new Page' do
           expect do
-            post :create, feature_page: { title: 'MyString' }, exhibit_id: exhibit
+            post :create, params: { feature_page: { title: 'MyString' }, exhibit_id: exhibit }
           end.to change(Spotlight::FeaturePage, :count).by(1)
         end
 
         it 'assigns a newly created page as @page' do
-          post :create, feature_page: { title: 'MyString' }, exhibit_id: exhibit
+          post :create, params: { feature_page: { title: 'MyString' }, exhibit_id: exhibit }
           expect(assigns(:page)).to be_a(Spotlight::FeaturePage)
           expect(assigns(:page)).to be_persisted
         end
         it 'redirects to the feature page index' do
-          post :create, feature_page: { title: 'MyString' }, exhibit_id: exhibit
+          post :create, params: { feature_page: { title: 'MyString' }, exhibit_id: exhibit }
           expect(response).to redirect_to(exhibit_feature_pages_path(Spotlight::FeaturePage.last.exhibit))
         end
       end
@@ -87,14 +87,14 @@ describe Spotlight::FeaturePagesController, type: :controller do
         it 'assigns a newly created but unsaved page as @page' do
           # Trigger the behavior that occurs when invalid params are submitted
           allow_any_instance_of(Spotlight::FeaturePage).to receive(:save).and_return(false)
-          post :create, feature_page: { 'title' => 'invalid value' }, exhibit_id: exhibit
+          post :create, params: { feature_page: { 'title' => 'invalid value' }, exhibit_id: exhibit }
           expect(assigns(:page)).to be_a_new(Spotlight::FeaturePage)
         end
 
         it "re-renders the 'new' template" do
           # Trigger the behavior that occurs when invalid params are submitted
           allow_any_instance_of(Spotlight::FeaturePage).to receive(:save).and_return(false)
-          post :create, feature_page: { 'title' => 'invalid value' }, exhibit_id: exhibit
+          post :create, params: { feature_page: { 'title' => 'invalid value' }, exhibit_id: exhibit }
           expect(response).to render_template('new')
         end
       end
@@ -109,16 +109,16 @@ describe Spotlight::FeaturePagesController, type: :controller do
           # receives the :update_attributes message with whatever params are
           # submitted in the request.
           expect_any_instance_of(Spotlight::FeaturePage).to receive(:update).with(hash_including(valid_attributes))
-          put :update, id: page, exhibit_id: page.exhibit.id, feature_page: valid_attributes
+          put :update, params: { id: page, exhibit_id: page.exhibit.id, feature_page: valid_attributes }
         end
 
         it 'assigns the requested page as @page' do
-          put :update, id: page, exhibit_id: page.exhibit.id, feature_page: valid_attributes
+          put :update, params: { id: page, exhibit_id: page.exhibit.id, feature_page: valid_attributes }
           expect(assigns(:page)).to eq(page)
         end
 
         it 'redirects to the feature page' do
-          put :update, id: page, exhibit_id: page.exhibit.id, feature_page: valid_attributes
+          put :update, params: { id: page, exhibit_id: page.exhibit.id, feature_page: valid_attributes }
           page.reload
           expect(response).to redirect_to(exhibit_feature_page_path(page.exhibit, page))
           expect(flash[:notice]).to have_link 'Undo changes'
@@ -129,14 +129,14 @@ describe Spotlight::FeaturePagesController, type: :controller do
         it 'assigns the page as @page' do
           # Trigger the behavior that occurs when invalid params are submitted
           allow_any_instance_of(Spotlight::FeaturePage).to receive(:save).and_return(false)
-          put :update, id: page, exhibit_id: page.exhibit.id, feature_page: { 'title' => 'invalid value' }
+          put :update, params: { id: page, exhibit_id: page.exhibit.id, feature_page: { 'title' => 'invalid value' } }
           expect(assigns(:page)).to eq(page)
         end
 
         it "re-renders the 'edit' template" do
           # Trigger the behavior that occurs when invalid params are submitted
           allow_any_instance_of(Spotlight::FeaturePage).to receive(:save).and_return(false)
-          put :update, id: page, exhibit_id: page.exhibit.id, feature_page: { 'title' => 'invalid value' }
+          put :update, params: { id: page, exhibit_id: page.exhibit.id, feature_page: { 'title' => 'invalid value' } }
           expect(response).to render_template('edit')
         end
       end
@@ -148,7 +148,7 @@ describe Spotlight::FeaturePagesController, type: :controller do
       let!(:page3) { FactoryGirl.create(:feature_page, exhibit: page1.exhibit, parent_page_id: page1.id) }
       before { request.env['HTTP_REFERER'] = 'http://example.com' }
       it 'updates the parent/child relationship' do
-        post :update_all, exhibit_id: page1.exhibit, exhibit: { feature_pages_attributes: [{ id: page2.id, parent_page_id: page1.id }] }
+        post :update_all, params: { exhibit_id: page1.exhibit, exhibit: { feature_pages_attributes: [{ id: page2.id, parent_page_id: page1.id }] } }
         expect(response).to redirect_to 'http://example.com'
         expect(flash[:notice]).to eq 'Feature pages were successfully updated.'
         expect(page1.parent_page).to be_nil
@@ -161,12 +161,12 @@ describe Spotlight::FeaturePagesController, type: :controller do
       let!(:page) { FactoryGirl.create(:feature_page, exhibit: exhibit) }
       it 'destroys the requested page' do
         expect do
-          delete :destroy, id: page, exhibit_id: page.exhibit.id
+          delete :destroy, params: { id: page, exhibit_id: page.exhibit.id }
         end.to change(Spotlight::FeaturePage, :count).by(-1)
       end
 
       it 'redirects to the pages list' do
-        delete :destroy, id: page, exhibit_id: page.exhibit.id
+        delete :destroy, params: { id: page, exhibit_id: page.exhibit.id }
         expect(response).to redirect_to(exhibit_feature_pages_path(page.exhibit))
       end
     end

data/spec/controllers/spotlight/filters_controller_spec.rb

@@ -11,7 +11,7 @@ describe Spotlight::FiltersController do
 
     context 'when not signed in' do
       it 'is not successful' do
-        post :create, exhibit_id: exhibit, filter: { field: 'foo_ssi', value: 'bar_ssi' }
+        post :create, params: { exhibit_id: exhibit, filter: { field: 'foo_ssi', value: 'bar_ssi' } }
         expect(:response).to redirect_to main_app.new_user_session_path
       end
     end
@@ -21,13 +21,13 @@ describe Spotlight::FiltersController do
       let(:user) { FactoryGirl.create(:site_admin) }
 
       it 'is successful' do
-        post :create, exhibit_id: exhibit, filter: { field: 'foo_ssi', value: 'bar' }
+        post :create, params: { exhibit_id: exhibit, filter: { field: 'foo_ssi', value: 'bar' } }
         expect(:response).to redirect_to edit_exhibit_path(exhibit, anchor: 'filter')
         expect(assigns[:exhibit].solr_data).to eq('foo_ssi' => 'bar')
       end
 
       it 'valids filter values' do
-        post :create, exhibit_id: exhibit, filter: { field: 'foo_ssi', value: '' }
+        post :create, params: { exhibit_id: exhibit, filter: { field: 'foo_ssi', value: '' } }
         expect(:response).to redirect_to edit_exhibit_path(exhibit, anchor: 'filter')
         expect(flash[:alert]).to include "Value can't be blank"
       end
@@ -44,7 +44,7 @@ describe Spotlight::FiltersController do
 
     context 'when not signed in' do
       it 'is not successful' do
-        patch :update, exhibit_id: exhibit, id: exhibit_filter, filter: { field: 'foo_ssi', value: 'bar_ssi' }
+        patch :update, params: { exhibit_id: exhibit, id: exhibit_filter, filter: { field: 'foo_ssi', value: 'bar_ssi' } }
         expect(:response).to redirect_to main_app.new_user_session_path
       end
     end
@@ -54,13 +54,13 @@ describe Spotlight::FiltersController do
       let(:user) { FactoryGirl.create(:site_admin) }
 
       it 'is successful' do
-        patch :update, exhibit_id: exhibit, id: exhibit_filter, filter: { field: 'foo_ssi', value: 'bar' }
+        patch :update, params: { exhibit_id: exhibit, id: exhibit_filter, filter: { field: 'foo_ssi', value: 'bar' } }
         expect(:response).to redirect_to edit_exhibit_path(exhibit, anchor: 'filter')
         expect(assigns[:exhibit].solr_data).to eq('foo_ssi' => 'bar')
       end
 
       it 'valids filter values' do
-        patch :update, exhibit_id: exhibit, id: exhibit_filter, filter: { field: 'foo_ssi', value: '' }
+        patch :update, params: { exhibit_id: exhibit, id: exhibit_filter, filter: { field: 'foo_ssi', value: '' } }
         expect(:response).to redirect_to edit_exhibit_path(exhibit, anchor: 'filter')
         expect(flash[:alert]).to include "Value can't be blank"
       end