Merge pull request #43 from projectblacklight/use_kwargs

Use kwargs to create the search_builder

Author: mjgiarlo

lib/blacklight/access_controls/catalog.rb

@@ -18,7 +18,7 @@ def enforce_show_permissions(_opts = {})
 
       # This will work for BL 6, but will need to move to SearchService in BL 7
       def search_builder
-        Blacklight::AccessControls::SearchBuilder.new(self, current_ability)
+        Blacklight::AccessControls::SearchBuilder.new(self, ability: current_ability)
       end
     end
   end

spec/unit/catalog_spec.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-describe Blacklight::AccessControls::Catalog do
+RSpec.describe Blacklight::AccessControls::Catalog do
   let(:controller) { CatalogController.new }
 
   describe '#enforce_show_permissions' do
-    subject { controller.send(:enforce_show_permissions) }
+    subject { controller.enforce_show_permissions }
 
     let(:params) { { id: doc.id } }
 
@@ -38,4 +38,16 @@
       end
     end
   end
+
+  describe '#search_builder' do
+    subject { controller.search_builder }
+
+    before do
+      allow(controller).to receive(:current_ability).and_return(instance_double(Ability))
+    end
+
+    it 'creates a search_builder' do
+      expect(subject).to be_kind_of Blacklight::AccessControls::SearchBuilder
+    end
+  end
 end