blacklight-spotlight 0.3.1 → 0.4.1

data/spec/features/main_navigation_spec.rb

@@ -39,4 +39,14 @@ describe "Main navigation labels are settable", :type => :feature do
     expect(page).to have_content("New Browse Label")
     expect(page).to have_css(".breadcrumb li", text: "New Browse Label")
   end
+  it 'should not display any main navigation menu items that are configured to not display' do
+    about_nav = exhibit.main_navigations.about
+    about_nav.display = false
+    about_nav.save
+    visit spotlight.exhibit_path(exhibit)
+    expect(page).to_not have_css(".navbar-nav li", text: "New About Label")
+    about_nav = exhibit.main_navigations.about
+    about_nav.display = true
+    about_nav.save
+  end
 end

data/spec/features/{site_mastehead_spec.rb → site_masthead_spec.rb}

@@ -14,8 +14,11 @@ describe "Add and update the site masthead", :type => :feature do
 
     click_link "Site masthead"
 
-    check "Show background image in masthead"
-    choose "Upload an image"
+    within "#site-masthead" do
+      check "Show background image in masthead"
+      choose "Upload an image"
+      attach_file('appearance_masthead_image', File.absolute_path(File.join(FIXTURES_PATH, 'avatar.png')));
+    end
 
     click_button "Save changes"
 
@@ -26,9 +29,11 @@ describe "Add and update the site masthead", :type => :feature do
     end
 
     click_link "Site masthead"
-
-    expect(field_labeled('Show background image in masthead')).to be_checked
-    expect(field_labeled('Upload an image')).to be_checked
+    
+    within "#site-masthead" do
+      expect(field_labeled('Show background image in masthead')).to be_checked
+      expect(field_labeled('Upload an image')).to be_checked
+    end
   end
   it 'should display a masthead image when one is uploaded and configured' do
     visit spotlight.exhibit_dashboard_path(exhibit)
@@ -38,10 +43,12 @@ describe "Add and update the site masthead", :type => :feature do
     end
 
     click_link "Site masthead"
+    
+    within "#site-masthead" do
+      check "Show background image in masthead"
 
-    check "Show background image in masthead"
-
-    attach_file('appearance_masthead_image', File.absolute_path(File.join(FIXTURES_PATH, 'avatar.png')));
+      attach_file('appearance_masthead_image', File.absolute_path(File.join(FIXTURES_PATH, 'avatar.png')));
+    end
 
     click_button "Save changes"
 
@@ -57,8 +64,10 @@ describe "Add and update the site masthead", :type => :feature do
     end
 
     click_link "Site masthead"
-
-    attach_file('appearance_masthead_image', File.absolute_path(File.join(FIXTURES_PATH, 'avatar.png')));
+    
+    within "#site-masthead" do
+      attach_file('appearance_masthead_image', File.absolute_path(File.join(FIXTURES_PATH, 'avatar.png')));
+    end
 
     click_button "Save changes"
 
@@ -75,10 +84,12 @@ describe "Add and update the site masthead", :type => :feature do
     end
 
     click_link "Site masthead"
+    
+    within "#site-masthead" do
+      check "Show background image in masthead"
 
-    check "Show background image in masthead"
-
-    fill_in_typeahead_field 'document_title', with: "Armenia"
+      fill_in_typeahead_field 'document_title', with: "Armenia"
+    end
 
     click_button "Save changes"
 

data/spec/features/upload_non_repository_item_spec.rb

@@ -6,7 +6,7 @@ describe "Uploading a non-repository item", :type => :feature do
   let!(:exhibit_curator) { FactoryGirl.create(:exhibit_curator, exhibit: exhibit) }
   before { login_as exhibit_curator }
 
-  describe "upload" do
+  describe "forms" do
     it "should display the single item upload form" do
       visit spotlight.new_exhibit_resources_upload_path(exhibit)
       expect(page).to have_css("h1", text: /Curation/)
@@ -32,4 +32,44 @@ describe "Uploading a non-repository item", :type => :feature do
       end
     end
   end
+
+  describe "upload" do
+    it "should create a new non-repository item" do
+      visit spotlight.new_exhibit_resources_upload_path(exhibit)
+      attach_file("resources_upload_url", File.join(FIXTURES_PATH, "800x600.png"))
+      fill_in "Title", with: "800x600"
+
+      within "#new_resources_upload" do
+        click_button "Add item"
+      end
+      expect(page).to have_content "Object uploaded successfully."
+
+      expect(Spotlight::Resource.last.url.file.path).to end_with "800x600.png"
+      Blacklight.default_index.connection.delete_by_id Spotlight::Resource.last.send(:compound_id)
+      Blacklight.default_index.connection.commit
+    end
+
+    it "should be editable" do
+      visit spotlight.new_exhibit_resources_upload_path(exhibit)
+      attach_file("resources_upload_url", File.join(FIXTURES_PATH, "800x600.png"))
+      fill_in "Title", with: "800x600"
+
+      within "#new_resources_upload" do
+        click_button "Add item"
+      end
+
+      click_link "800x600"
+      click_link "Edit"
+      fill_in "Title", with: "This is a now an avatar"
+
+      attach_file("File", File.join(FIXTURES_PATH, "avatar.png"))
+
+      click_button "Save"
+
+      expect(page).to have_content "This is a now an avatar"
+      expect(Spotlight::Resource.last.url.file.path).to end_with "avatar.png"
+      Blacklight.default_index.connection.delete_by_id Spotlight::Resource.last.send(:compound_id)
+      Blacklight.default_index.connection.commit
+    end
+  end
 end

data/spec/helpers/spotlight/application_helper_spec.rb

@@ -37,7 +37,7 @@ describe Spotlight::ApplicationHelper, :type => :helper do
     end
     describe "blacklight_view_config_for_search_block" do
       let(:sir_trevor_block) { 
-        SirTrevorRails::Block.new({type: 'xyz', data: {"list" => "on", "gallery" => "on", "slideshow" => "null"}}, 'parent')
+        SirTrevorRails::Block.new({type: 'xyz', data: {view: ["list", "gallery"]}}, 'parent')
       }
 
       let(:config) { Blacklight::Configuration.new do |config|
@@ -84,7 +84,8 @@ describe Spotlight::ApplicationHelper, :type => :helper do
 
     it "should generate a twitter card for the exhibit" do
       allow(helper).to receive(:exhibit_root_url).and_return("some/url")
-      allow(current_exhibit).to receive(:featured_image).and_return(double(url: "http://some.host/image"))
+      allow(current_exhibit).to receive(:thumbnail).and_return(double)
+      allow(current_exhibit).to receive_message_chain(:thumbnail, :image, :thumb, url: "/image")
 
       helper.add_exhibit_twitter_card_content
 
@@ -93,7 +94,7 @@ describe Spotlight::ApplicationHelper, :type => :helper do
       expect(card).to have_css "meta[name='twitter:url'][value='some/url']", visible: false
       expect(card).to have_css "meta[name='twitter:title'][value='#{current_exhibit.title}']", visible: false
       expect(card).to have_css "meta[name='twitter:description'][value='#{current_exhibit.subtitle}']", visible: false
-      expect(card).to have_css "meta[name='twitter:image'][value='http://some.host/image']", visible: false
+      expect(card).to have_css "meta[name='twitter:image'][value='http://test.host/image']", visible: false
     end
   end
   
@@ -153,4 +154,22 @@ describe Spotlight::ApplicationHelper, :type => :helper do
       expect(helper.uploaded_field_label(field)).to eq 'solr x'
     end
   end
+
+  describe "#available_view_fields" do    
+    let :blacklight_config do
+      Blacklight::Configuration.new
+    end
+
+    before do
+      allow(helper).to receive_message_chain(:current_exhibit, :blacklight_configuration, default_blacklight_config: blacklight_config)
+    end
+
+    it "should exclude view fields that are never visible (e.g. atom, rss)" do
+      blacklight_config.view.a.if = true
+      blacklight_config.view.b.if = false
+
+      expect(helper.available_view_fields).to include :a
+      expect(helper.available_view_fields).to_not include :b
+    end
+  end
 end

data/spec/helpers/spotlight/navbar_helper_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+module Spotlight
+  describe NavbarHelper, :type => :helper do
+    describe '#should_render_search_bar?' do
+      before do
+        allow(helper).to receive_messages(current_exhibit: nil)
+        allow(helper).to receive_messages(exhibit_masthead?: true)
+      end
+      it 'should return true when there is no exhibit context' do
+        expect(helper.should_render_spotlight_search_bar?).to be_truthy
+      end
+      it 'should return true if searchable' do
+        allow(helper).to receive_messages(current_exhibit: double(searchable?: true))
+        expect(helper.should_render_spotlight_search_bar?).to be_truthy
+      end
+      it 'should return false if currently under an "Exhibity" browse category' do
+        allow(helper).to receive_messages(exhibit_masthead?: false)
+        expect(helper.should_render_spotlight_search_bar?).to be_falsey
+      end
+    end
+  end
+end

data/spec/helpers/spotlight/pages_helper_spec.rb

@@ -7,7 +7,7 @@ module Spotlight
     let(:untitled_document) { ::SolrDocument.new( :id  => "1234"  ) }
     let!(:current_exhibit) { FactoryGirl.create(:exhibit) }
     let!(:home_page) { current_exhibit.home_page }
-    let!(:search) { FactoryGirl.create(:search, exhibit: current_exhibit, query_params: { "q" => "query" }) }
+    let!(:search) { FactoryGirl.create(:search, exhibit: current_exhibit, query_params: { "q" => "query" }, on_landing_page: true ) }
 
     before(:each) do
       allow(helper).to receive_messages(:blacklight_config => blacklight_config)
@@ -39,8 +39,8 @@ module Spotlight
       end
     end
     describe "get_search_widget_search_results" do
-      let(:good) { SirTrevorRails::Blocks::SearchResultsBlock.new({type: 'xyz', data: {'slug' => search.slug}}, home_page) }
-      let(:bad) { SirTrevorRails::Blocks::SearchResultsBlock.new({type: 'xyz', data: { 'slug' => 'missing'}}, home_page) }
+      let(:good) { SirTrevorRails::Blocks::SearchResultsBlock.new({type: 'xyz', data: {'item' => { search.slug => { id: search.slug, display: "true" }}}}, home_page) }
+      let(:bad) { SirTrevorRails::Blocks::SearchResultsBlock.new({type: 'xyz', data: { 'item' => { 'garbage' => { id: 'missing', display: "true" }}}}, home_page) }
       let(:search_result) { [double('response'), double('documents')] }
       it "should return the results for a given search browse category" do
         expect(helper).to receive(:get_search_results).with({"q" => "query"}).and_return(search_result)
@@ -50,68 +50,7 @@ module Spotlight
         expect(helper.get_search_widget_search_results( bad )).to be_empty
       end
     end
-    describe "item grid helpers" do
-      describe "block objects" do
-        let(:block1) { {'item-grid-id_0' => "abc", 'item-grid-id_1' => "cba", 'item-grid-display_0' => false, 'item-grid-display_1' => false} }
-        let(:block_with_hidden) { {'item-grid-id_0' => "abc", 'item-grid-id_1' => "cba", 'item-grid-display_0' => false, 'item-grid-display_1' => true, 'item-grid-thumbnail_1' => 'image-url'} }
-        let(:block_with_blank) { {'item-grid-id_0' => "abc", 'item-grid-id_1' => "", 'item-grid-id_2' => "", 'item-grid-display_0' => true, 'item-grid-display_1' => false} }
-        let(:bad_keys) { {'another-key' => "something"} }
-        describe "item_grid_block_objects" do
-          it "should not return items that have display set to false" do
-            expect(helper.item_grid_block_objects(block1)).to be_blank
-          end
-          it "should get set the display attribute to true if a corresponding display field is set to 'true'" do
-            objects = helper.item_grid_block_objects(block_with_hidden)
-            expect(objects).to include({:id => "cba", :display => true, thumbnail: 'image-url'})
-          end
-          it "should omit any blank values" do
-            objects = helper.item_grid_block_objects(block_with_blank)
-            expect(objects).to eq([{id: "abc", display: true, thumbnail: nil}])
-          end
-          it "should omit any unnecessary keys" do
-            expect(helper.item_grid_block_objects(bad_keys)).to be_blank
-          end
-        end
-        describe "item_grid_block_ids" do
-          it "should get all of the displayable document IDs from the block" do
-            expect(helper.item_grid_block_ids(block_with_hidden)).to eq ["cba"]
-          end
-          it "should omit blank keys" do
-            expect(helper.item_grid_block_ids(block_with_blank)).to eq ["abc"]
-          end
-          it "should omit any unnecessary keys" do
-            expect(helper.item_grid_block_ids(bad_keys)).to be_blank
-          end
-        end
-        describe 'item_grid_block_with_documents' do
-          it 'should return hashes representing documents to display from the block, including the solr_document object itself' do
-            allow(helper).to receive_messages(get_solr_response_for_document_ids: [nil, [{id: 'cba'}]])
-            objects = helper.item_grid_block_with_documents(block_with_hidden)
-            expect(objects.length).to eq 1
-            expect(objects.first[:id]).to eq 'cba'
-            expect(objects.first[:thumbnail]).to eq 'image-url'
-            expect(objects.first[:solr_document][:id]).to eq 'cba'
-          end
-        end
-      end
-      describe "captions" do
-        let(:solr_document) { ::SolrDocument.new(:id => "123", 'a_field' => "A field value", 'b_field' => "Another field value") }
-        it "should return the document_heading when the special title field is selected" do
-          expect(helper).to receive(:document_heading).with(solr_document).and_return("A title")
-          expect(helper.multi_up_item_grid_caption({'item-grid-primary-caption-field' => 'spotlight_title_field'}, solr_document)).to eq "A title"
-        end
-        it "should render the field value when any other field is selected" do
-          expect(helper.multi_up_item_grid_caption({'item-grid-primary-caption-field' => 'a_field'}, solr_document)).to eq "A field value"
-        end
-        it "should handle secondary caption fields" do
-          expect(helper.multi_up_item_grid_caption({'item-grid-secondary-caption-field' => 'b_field'}, solr_document, 'secondary')).to eq "Another field value"
-        end
-        it "should do nothing if the item-grid-caption-field is blank or nil" do
-          expect(helper.multi_up_item_grid_caption({'item-grid-primary-caption-field' => ''}, solr_document)).to be_blank
-          expect(helper.multi_up_item_grid_caption({}, solr_document)).to be_blank
-        end
-      end
-    end
+
     describe 'nestable helpers' do
       describe 'nestable data attributes' do
         it 'should return the appropriate attributes for feature pages' do

data/spec/jobs/spotlight/rename_sidecar_field_job_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Spotlight::RenameSidecarFieldJob do
+  let(:exhibit) { FactoryGirl.create(:exhibit) }
+  let(:sidecar) { SolrDocument.new(id: 'test').sidecar(exhibit).tap { |x| x.save! } }
+
+  it "should update the sidecar data and reindex affected documents" do
+    expect_any_instance_of(::SolrDocument).to receive(:reindex)
+
+    sidecar.data['old_field'] = 'some value'
+    sidecar.save!
+
+    described_class.perform_later(exhibit, "old_field", "new_field")
+    
+    sidecar.reload
+    expect(sidecar.data['new_field']).to eq 'some value'
+  end
+
+  it "shouldn't touch unaffected documents" do
+    expect_any_instance_of(::SolrDocument).not_to receive(:reindex)
+
+    sidecar.data['other_field'] = 'some value'
+    sidecar.save!
+
+    described_class.perform_later(exhibit, "old_field", "new_field")
+  end
+end

data/spec/lib/spotlight/catalog/access_controls_enforcement_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe Spotlight::Catalog::AccessControlsEnforcement do
   class MockCatalogController
-    include Blacklight::SolrHelper
+    include Blacklight::SearchHelper
     include Spotlight::Catalog::AccessControlsEnforcement
 
     def blacklight_config
@@ -20,7 +20,7 @@ describe Spotlight::Catalog::AccessControlsEnforcement do
 
   describe "#apply_permissive_visibility_filter" do
     it "should add the filter to the params logic" do
-      expect(subject.solr_search_params_logic).to include :apply_permissive_visibility_filter 
+      expect(subject.search_params_logic).to include :apply_permissive_visibility_filter 
     end
 
     it "should allow curators to view everything" do

data/spec/lib/spotlight/controller_spec.rb

@@ -7,9 +7,54 @@ describe Spotlight::Controller do
 
   subject { MockController.new }
 
+  before do
+    allow(subject).to receive_messages(params: {action: 'show'})
+  end
+
   describe "#current_exhibit" do
     it "should be nil by default" do
       expect(subject.current_exhibit).to be_nil
     end
   end
+
+  describe '#exhibit_masthead?' do
+    let(:masthead) { double('masthead', display?: true) }
+
+    before do
+      allow(subject).to receive_messages(current_exhibit: nil, current_masthead: nil)
+    end
+
+    it "should be true if there is no exhibit" do
+      expect(subject.exhibit_masthead?).to eq true
+    end
+
+    it "should be true if there is no custom exhibit masthead" do
+      allow(subject).to receive_messages(current_exhibit: double, current_masthead: nil)
+      expect(subject.exhibit_masthead?).to eq true
+    end
+
+    it "should be false if the current masthead is not the exhibit masthead" do
+      allow(subject).to receive_messages(current_exhibit: double(masthead: double), current_masthead: masthead)
+      expect(subject.exhibit_masthead?).to eq false
+    end
+  end
+  describe '#current_masthead' do
+    let(:search_masthead) { double('search-masthead', display?: true) }
+    let(:no_display_search_masthead) { double('no-display-search-masthead', display?: false) }
+    let(:exhibit_masthead) { double('exhibit-masthead', display?: true) }
+    let(:exhibit) { FactoryGirl.create(:exhibit) }
+    let(:search) { FactoryGirl.create(:search) }
+    it 'should be nil by default' do
+      expect(subject.current_masthead).to be_nil
+    end
+    it 'should return the exhibit masthead if available' do
+      allow(exhibit).to receive_messages(masthead: exhibit_masthead)
+      subject.instance_variable_set(:@exhibit, exhibit)
+      expect(subject.current_masthead).to eq exhibit_masthead
+    end
+    it 'should allow the masthead to be set' do
+      subject.current_masthead = search_masthead
+      expect(subject.current_masthead).to eq search_masthead
+    end
+  end
 end

data/spec/models/sir_trevor_rails/blocks/solr_documents_block_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe SirTrevorRails::Blocks::SolrDocumentsBlock do
+  let(:page) { FactoryGirl.create(:feature_page) }
+  let(:block_data) { {} }
+  subject { described_class.new({type: "", data: block_data}, page) }
+
+  describe "#text" do
+    it "should be the block's text data" do
+      block_data[:text] = "abc"
+
+      expect(subject.text).to eq "abc"
+    end
+
+    it "should squelch sir-trevor's placeholder values" do
+      block_data[:text] = "<p><br></p>"
+      expect(subject.text).to be_blank
+    end
+  end
+end

data/spec/models/solr_document_spec.rb

@@ -133,7 +133,6 @@ describe SolrDocument, :type => :model do
   describe 'uploaded resources' do
     let(:uploaded_resource) {
       SolrDocument.new(
-        Spotlight::Engine.config.full_image_field => "ImageData",
         spotlight_resource_type_ssim: "spotlight/resources/uploads"
       )
     }

data/spec/models/spotlight/blacklight_configuration_spec.rb

@@ -22,6 +22,10 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
     subject.save!
   end
 
+  it "should add a default thumbnail field" do
+    expect(subject.blacklight_config.index.thumbnail_field).to eq Spotlight::Engine.config.thumbnail_field
+  end
+
   describe "facet fields" do
     it "should have facet fields" do
       expect(subject.facet_fields).to be_empty
@@ -64,6 +68,30 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
 
       expect(subject.blacklight_config.facet_fields.keys).to eq ['c', 'a', 'b']
     end
+
+    context "custom fields" do
+      it "should include any custom fields" do
+        allow(subject).to receive_messages(custom_facet_fields: { 'a' => double(if: nil, :if= => true, merge!: true, validate!: true, normalize!: true) })
+        expect(subject.blacklight_config.facet_fields).to include('a')
+      end
+
+      it "should default to not showing a custom field in the facets" do
+        allow(subject).to receive_messages(custom_facet_fields: { 'a' => Blacklight::Configuration::IndexField.new(field: 'a') })
+        expect(subject.blacklight_config.facet_fields).to include('a')
+        expect(subject.blacklight_config.facet_fields['a'].show).to be_falsey
+      end
+    end
+
+    context "exhibit fields" do
+      before do
+        # undo the stubbing we've used elsewhere..
+        allow(subject).to receive(:default_blacklight_config).and_call_original
+        allow(Spotlight::Engine).to receive_messages blacklight_config: blacklight_config
+      end
+      it "should inject a tags facet" do
+        expect(subject.blacklight_config.facet_fields).to include "exhibit_tags"
+      end
+    end
   end
 
   describe "index fields" do
@@ -129,6 +157,13 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
       expect(subject.blacklight_config.index_fields).not_to have_key 'a'
     end
 
+    it "should include persisted configurations for show fields" do
+      subject.index_fields['a'] = { enabled: true, list: true }
+      blacklight_config.add_show_field 'a'
+      expect(subject.blacklight_config.index_fields).to have_key 'a'
+      expect(subject.blacklight_config.index_fields['a'].enabled).to be_truthy
+      expect(subject.blacklight_config.index_fields['a'].list).to be_truthy
+    end
 
     context "custom fields" do
       it "should include any custom fields" do
@@ -329,7 +364,7 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
       blacklight_config.view.list
       blacklight_config.view.gallery
       blacklight_config.view.something
-      expect(subject.blacklight_config.view.keys).to eq blacklight_config.view.keys
+      expect(subject.blacklight_config.view.keys).to include *blacklight_config.view.keys
     end
   end
 
@@ -348,18 +383,6 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
     end
   end
 
-  describe "default_autocomplete_solr_params" do
-    it "should include all the fields we need to render the autocomplete widget" do
-      blacklight_config.show.title_field = "some_field"
-      expect(subject.blacklight_config.default_autocomplete_solr_params).to have_key :fl
-      expect(subject.blacklight_config.default_autocomplete_solr_params[:fl]).to include "id"
-      expect(subject.blacklight_config.default_autocomplete_solr_params[:fl]).to include "some_field"
-      expect(subject.blacklight_config.default_autocomplete_solr_params[:fl]).to include "full_image_url_ssm"
-      expect(subject.blacklight_config.default_autocomplete_solr_params[:fl]).to include "thumbnail_url_ssm"
-      expect(subject.blacklight_config.default_autocomplete_solr_params[:fl]).to include "thumbnail_square_url_ssm"
-    end
-  end
-
   describe "show" do
     it "should have show view configuration" do
       expect(subject.show).to be_empty
@@ -372,11 +395,6 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
       subject.show[:title_field] = 'abc'
       expect(subject.blacklight_config.show.title_field).to eq 'abc'
     end
-
-    it "should inject partials" do
-      partials = subject.blacklight_config.show.partials
-      expect(partials).to include "spotlight/catalog/tags"
-    end
   end
 
   describe "index" do
@@ -401,8 +419,41 @@ describe Spotlight::BlacklightConfiguration, :type => :model do
       ]))
 
       expect(subject.custom_index_fields).to include 'abc', 'xyz'
-      expect(subject.custom_index_fields['abc']).to be_a_kind_of Blacklight::Configuration::SolrField
+      expect(subject.custom_index_fields['abc']).to be_a_kind_of Blacklight::Configuration::Field
       expect(subject.custom_index_fields['abc'].a).to eq 1
     end
   end
+
+  describe "autocomplete configuration" do
+    before do
+      # undo the stubbing we've used elsewhere..
+      allow(subject).to receive(:default_blacklight_config).and_call_original
+      blacklight_config.show.title_field = "x"
+      allow(Spotlight::Engine).to receive_messages blacklight_config: blacklight_config.deep_copy
+    end
+
+    context "with the default search field" do
+      let(:search_field) do
+        subject.blacklight_config.search_fields[Spotlight::Engine.config.autocomplete_search_field]
+      end
+
+      it "should be hidden from the search field selector" do
+        expect(search_field.if).to eq false
+      end
+
+      it "should use the engine's autocomplete parameters" do
+        expect(search_field.solr_parameters).to include Spotlight::Engine.config.default_autocomplete_params
+      end
+
+      it "should include the relevant fields" do
+        Spotlight::Engine.blacklight_config.show.title_field = "some_field"
+        expect(search_field.solr_parameters).to have_key :fl
+        expect(search_field.solr_parameters[:fl]).to include "id"
+        expect(search_field.solr_parameters[:fl]).to include "some_field"
+        expect(search_field.solr_parameters[:fl]).to include "full_image_url_ssm"
+        expect(search_field.solr_parameters[:fl]).to include "thumbnail_url_ssm"
+        expect(search_field.solr_parameters[:fl]).to include "thumbnail_square_url_ssm"
+      end
+    end
+  end
 end