Hacks to pass more tests

projectblacklight · Jul 13, 2016 · 47c344e · 47c344e
1 parent 92c9255
commit 47c344e
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 9 deletions.
diff --git a/app/controllers/concerns/spotlight/controller.rb b/app/controllers/concerns/spotlight/controller.rb
@@ -82,8 +82,7 @@ def exhibit_search_action_url(*args)
 
     def exhibit_search_facet_url(*args)
       options = args.extract_options!
-      options = params.merge(options).except(:exhibit_id, :only_path)
-
+      options = params.to_unsafe_h.merge(options).except(:exhibit_id, :only_path)
       spotlight.facet_exhibit_catalog_url(current_exhibit, *args, options)
     end
   end

diff --git a/app/controllers/spotlight/searches_controller.rb b/app/controllers/spotlight/searches_controller.rb
@@ -12,8 +12,8 @@ class SearchesController < Spotlight::ApplicationController
     include Spotlight::Base
 
     def create
-      @search.attributes = search_params
-      @search.query_params = params.except(:exhibit_id, :search, *blacklisted_search_session_params).reject { |_k, v| v.blank? }
+      @search.attributes = search_params.
+      @search.query_params = params.to_unsafe_h.except(:exhibit_id, :search, *blacklisted_search_session_params).reject { |_k, v| v.blank? }
 
       if @search.save
         redirect_to :back, notice: t(:'helpers.submit.search.created', model: @search.class.model_name.human.downcase)

diff --git a/spec/controllers/spotlight/appearances_controller_spec.rb b/spec/controllers/spotlight/appearances_controller_spec.rb
@@ -45,10 +45,10 @@
         first_nav = exhibit.main_navigations.first
         last_nav = exhibit.main_navigations.last
         patch :update, exhibit_id: exhibit, exhibit: {
-          main_navigations_attributes: [
-            { id: first_nav.id, label: 'Some Label', weight: 500 },
-            { id: last_nav.id, display: false }
-          ]
+          main_navigations_attributes: {
+            0 => { id: first_nav.id, label: 'Some Label', weight: 500 },
+            1 => { id: last_nav.id, display: false }
+          }
         }
         expect(flash[:notice]).to eq 'The exhibit was successfully updated.'
         expect(response).to redirect_to edit_exhibit_appearance_path(exhibit)

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,5 +1,4 @@
 ENV['RAILS_ENV'] ||= 'test'
-
 require 'factory_girl'
 require 'database_cleaner'
 require 'devise'
@@ -48,6 +47,7 @@
 
 RSpec.configure do |config|
   config.infer_spec_type_from_file_location!
+  config.filter_rails_from_backtrace!
 
   config.use_transactional_fixtures = false