blacklight 8.11.0 → 9.0.0.beta2

data/spec/components/blacklight/search_context/server_applied_params_component_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe Blacklight::SearchContext::ServerAppliedParamsComponent, type: :c
 
   let(:instance) { described_class.new }
   let(:current_search_session) { nil }
-  let(:view_context) { controller.view_context }
+  let(:view_context) { vc_test_controller.view_context }
 
   before do
     # Not sure why we need to re-implement rspec's stub_template, but
@@ -16,7 +16,7 @@ RSpec.describe Blacklight::SearchContext::ServerAppliedParamsComponent, type: :c
     # https://github.com/rspec/rspec-rails/issues/2696
     replace_hash = { 'application/_start_over.html.erb' => 'start over' }
     if Rails.version.to_f >= 7.1
-      controller.prepend_view_path(RSpec::Rails::ViewExampleGroup::StubResolverCache.resolver_for(replace_hash))
+      vc_test_controller.prepend_view_path(RSpec::Rails::ViewExampleGroup::StubResolverCache.resolver_for(replace_hash))
     else
       view_context.view_paths.unshift(RSpec::Rails::ViewExampleGroup::StubResolverCache.resolver_for(replace_hash))
     end

data/spec/components/blacklight/search_context/server_item_pagination_component_spec.rb

@@ -11,8 +11,8 @@ RSpec.describe Blacklight::SearchContext::ServerItemPaginationComponent, type: :
   let(:instance) { described_class.new(search_context: search_context, search_session: search_session, current_document: current_document) }
 
   before do
-    allow(controller).to receive(:current_search_session).and_return(double(id: current_document_id))
-    controller.class.helper_method :current_search_session
+    allow(vc_test_controller).to receive(:current_search_session).and_return(double(id: current_document_id))
+    vc_test_controller.class.helper_method :current_search_session
   end
 
   context 'when there is no next or previous' do
@@ -40,9 +40,7 @@ RSpec.describe Blacklight::SearchContext::ServerItemPaginationComponent, type: :
     let(:next_doc) { SolrDocument.new(id: '888') }
 
     before do
-      # allow(controller).to receive(:controller_tracking_method).and_return('track_catalog_path')
-
-      allow(controller).to receive_messages(controller_name: 'catalog', link_to_previous_document: '', link_to_next_document: '')
+      allow(vc_test_controller).to receive_messages(controller_name: 'catalog', link_to_previous_document: '', link_to_next_document: '')
     end
 
     it "renders content" do

data/spec/components/blacklight/skip_link_component_spec.rb

@@ -3,12 +3,9 @@
 require 'spec_helper'
 
 RSpec.describe Blacklight::SkipLinkComponent, type: :component do
-  subject(:rendered) do
-    render_inline_to_capybara_node(described_class.new)
-  end
-
   before do
-    allow(controller).to receive(:blacklight_config).and_return(blacklight_config)
+    allow(vc_test_controller).to receive(:blacklight_config).and_return(blacklight_config)
+    render_inline(described_class.new)
   end
 
   context 'with no search fields' do
@@ -19,8 +16,8 @@ RSpec.describe Blacklight::SkipLinkComponent, type: :component do
     end
 
     it 'renders skip links with correct link to search' do
-      expect(rendered).to have_link("Skip to main content", href: '#main-container')
-      expect(rendered).to have_link("Skip to search", href: "#q")
+      expect(page).to have_link("Skip to main content", href: '#main-container')
+      expect(page).to have_link("Skip to search", href: "#q")
     end
   end
 
@@ -32,8 +29,8 @@ RSpec.describe Blacklight::SkipLinkComponent, type: :component do
     end
 
     it 'renders skip links with correct link to search' do
-      expect(rendered).to have_link("Skip to main content", href: "#main-container")
-      expect(rendered).to have_link("Skip to search", href: "#q")
+      expect(page).to have_link("Skip to main content", href: "#main-container")
+      expect(page).to have_link("Skip to search", href: "#q")
     end
   end
 
@@ -45,8 +42,8 @@ RSpec.describe Blacklight::SkipLinkComponent, type: :component do
     end
 
     it 'renders skip links with correct link to search' do
-      expect(rendered).to have_link("Skip to main content", href: "#main-container")
-      expect(rendered).to have_link("Skip to search", href: "#search_field")
+      expect(page).to have_link("Skip to main content", href: "#main-container")
+      expect(page).to have_link("Skip to search", href: "#search_field")
     end
   end
 end

data/spec/components/blacklight/start_over_button_component_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 
 RSpec.describe Blacklight::StartOverButtonComponent, type: :component do
-  subject(:render) { render_inline(instance) }
+  subject(:render) { render_inline instance }
 
   let(:instance) { described_class.new }
   let(:blacklight_config) do
@@ -13,12 +13,12 @@ RSpec.describe Blacklight::StartOverButtonComponent, type: :component do
   end
 
   before do
-    allow(controller).to receive(:blacklight_config).and_return(blacklight_config)
+    allow(vc_test_controller).to receive(:blacklight_config).and_return(blacklight_config)
   end
 
   context 'with the current view type' do
     before do
-      controller.params[:view] = 'abc'
+      vc_test_controller.params[:view] = 'abc'
     end
 
     it 'is the catalog path' do
@@ -28,7 +28,7 @@ RSpec.describe Blacklight::StartOverButtonComponent, type: :component do
 
   context 'when the current view type is the default' do
     before do
-      controller.params[:view] = 'list'
+      vc_test_controller.params[:view] = 'list'
     end
 
     it 'does not include the current view type' do

data/spec/components/blacklight/system/flash_message_component_spec.rb

@@ -5,26 +5,22 @@ require 'spec_helper'
 RSpec.describe Blacklight::System::FlashMessageComponent, type: :component do
   subject(:component) { described_class.new(message: message, type: type) }
 
-  let(:view_context) { controller.view_context }
-  let(:render) do
-    component.render_in(view_context)
+  before do
+    render_inline(component)
   end
 
-  let(:rendered) do
-    Capybara::Node::Simple.new(render)
-  end
   let(:message) { 'This is an important message' }
   let(:type) { 'whatever' }
 
   it 'renders a message inside an alert' do
-    expect(rendered).to have_css 'div.alert.alert-whatever', text: message
+    expect(page).to have_css 'div.alert.alert-whatever', text: message
   end
 
   context 'with a success message' do
     let(:type) { 'success' }
 
     it 'adds some styling' do
-      expect(rendered).to have_css 'div.alert-success'
+      expect(page).to have_css 'div.alert-success'
     end
   end
 
@@ -32,7 +28,7 @@ RSpec.describe Blacklight::System::FlashMessageComponent, type: :component do
     let(:type) { 'notice' }
 
     it 'adds some styling' do
-      expect(rendered).to have_css 'div.alert-info'
+      expect(page).to have_css 'div.alert-info'
     end
   end
 
@@ -40,7 +36,7 @@ RSpec.describe Blacklight::System::FlashMessageComponent, type: :component do
     let(:type) { 'alert' }
 
     it 'adds some styling' do
-      expect(rendered).to have_css 'div.alert-warning'
+      expect(page).to have_css 'div.alert-warning'
     end
   end
 
@@ -48,7 +44,7 @@ RSpec.describe Blacklight::System::FlashMessageComponent, type: :component do
     let(:type) { 'error' }
 
     it 'adds some styling' do
-      expect(rendered).to have_css 'div.alert-danger'
+      expect(page).to have_css 'div.alert-danger'
     end
   end
 end

data/spec/controllers/catalog_controller_spec.rb

@@ -499,13 +499,14 @@ RSpec.describe CatalogController, :api do
         expect(request.flash[:error]).to be_nil
       end
 
-      it "redirects back to the record upon success" do
-        allow(RecordMailer).to receive(:email_record)
-          .with(anything, { to: 'test_email@projectblacklight.org', message: 'xyz', config: config }, hash_including(host: 'test.host'))
-          .and_return double(deliver: nil)
-        post :email, params: { id: doc_id, to: 'test_email@projectblacklight.org', message: 'xyz' }
-        expect(request.flash[:error]).to be_nil
-        expect(request).to redirect_to(solr_document_path(doc_id))
+      context 'when message is provided' do
+        it "sends email and redirects back to the record" do
+          expect do
+            post :email, params: { id: doc_id, to: 'test_email@projectblacklight.org', message: 'xyz' }
+          end.to send_email(to: 'test_email@projectblacklight.org')
+          expect(request.flash[:error]).to be_nil
+          expect(request).to redirect_to(solr_document_path(doc_id))
+        end
       end
 
       it "renders email_success for XHR requests" do
@@ -549,11 +550,9 @@ RSpec.describe CatalogController, :api do
       end
 
       it "sends to the appropriate carrier email address" do
-        expect(RecordMailer)
-          .to receive(:sms_record)
-          .with(anything, { to: '5555555555@txt.att.net', config: config }, hash_including(host: 'test.host'))
-          .and_return double(deliver: nil)
-        post :sms, params: { id: doc_id, to: '5555555555', carrier: 'txt.att.net' }
+        expect do
+          post :sms, params: { id: doc_id, to: '5555555555', carrier: 'txt.att.net' }
+        end.to send_email(to: '5555555555@txt.att.net')
       end
 
       it "redirects back to the record upon success" do
@@ -786,13 +785,6 @@ RSpec.describe CatalogController, :api do
     end
   end
 
-  describe "search_facet_path" do
-    it "is the same as the catalog path" do
-      get :index, params: { page: 1 }
-      expect(controller.send(:search_facet_path, id: "some_facet", page: 5)).to eq facet_catalog_path(id: "some_facet")
-    end
-  end
-
   describe "page_links" do
     it "has prev/next docs and result set data for non-empty result sets", :integration do
       get :page_links, params: { f: { "format" => 'Book' }, counter: 2 }

data/spec/features/axe_spec.rb

@@ -3,7 +3,7 @@
 RSpec.describe 'Accessibility testing', :js, api: false do
   it 'validates the home page' do
     visit root_path
-    expect(page).to be_accessible
+    expect(page).to be_axe_clean
   end
 
   it 'validates the catalog page' do
@@ -11,28 +11,23 @@ RSpec.describe 'Accessibility testing', :js, api: false do
     fill_in "q", with: 'history'
     click_on 'search'
 
-    expect(page).to be_accessible
+    expect(page).to be_axe_clean
 
-    within '.card.blacklight-language_ssim' do
+    within '.accordion-item.blacklight-language_ssim' do
       click_on 'Language'
       click_on "Tibetan"
     end
 
-    expect(page).to be_accessible
+    expect(page).to be_axe_clean
   end
 
   it 'validates the advanced search form' do
     visit advanced_search_catalog_path
-    expect(page).to be_accessible
+    expect(page).to be_axe_clean
   end
 
   it 'validates the single results page' do
     visit solr_document_path('2007020969')
-    expect(page).to be_accessible
-  end
-
-  def be_accessible(skipping: [])
-    # typeahead does funny things with the search bar
-    be_axe_clean.excluding('.tt-hint').skipping(skipping + [('color-contrast' if defined?(Bootstrap) && Bootstrap::VERSION < '5')].compact)
+    expect(page).to be_axe_clean
   end
 end

data/spec/features/bookmarks_spec.rb

@@ -1,6 +1,11 @@
 # frozen_string_literal: true
 
 RSpec.describe "Bookmarks" do
+  before do
+    # Prevent any existing bookmarks from interfering
+    Bookmark.delete_all
+  end
+
   describe "navigating from the homepage" do
     it "has a link to the history page" do
       sign_in 'user1'
@@ -40,13 +45,25 @@ RSpec.describe "Bookmarks" do
     expect(page).to have_content 'Successfully removed bookmark.'
   end
 
-  it 'shows bookmarks as checkboxes', :js do
-    visit solr_document_path('2007020969')
-    check 'Bookmark'
+  context 'when bookmark_icon_component is set to nil' do
+    let!(:default_bookmark_icon_component) { CatalogController.blacklight_config.bookmark_icon_component }
 
-    visit solr_document_path('2007020969')
-    expect(page).to have_css('input[type="checkbox"][checked]')
-    uncheck 'In Bookmarks'
+    before do
+      CatalogController.blacklight_config.bookmark_icon_component = nil
+    end
+
+    after do
+      CatalogController.blacklight_config.bookmark_icon_component = default_bookmark_icon_component
+    end
+
+    it 'shows bookmarks as checkboxes', :js do
+      visit solr_document_path('2007020969')
+      check 'Bookmark'
+
+      visit solr_document_path('2007020969')
+      expect(page).to have_css('input[type="checkbox"][checked]')
+      uncheck 'In Bookmarks'
+    end
   end
 
   it "adds bookmarks after a user logs in" do
@@ -82,17 +99,37 @@ RSpec.describe "Bookmarks" do
     expect(page).to have_content 'Ci an zhou bian'
   end
 
-  context "has bookmark icon" do
+  context "when the bookmark icon is configured (default)" do
+    it 'shows bookmark icon', :js do
+      visit solr_document_path('2007020969')
+      expect(page).to have_css('.blacklight-icons-bookmark')
+      find('.blacklight-icons-bookmark').click
+
+      expect(find('.toggle-bookmark-input', visible: false)).to be_checked
+      find('.blacklight-icons-bookmark').click
+    end
+  end
+
+  context "without bookmark navbar element" do
+    let!(:default_nav_partials) { CatalogController.blacklight_config.navbar.partials }
+
     before do
-      CatalogController.blacklight_config.bookmark_icon_component = Blacklight::Icons::BookmarkIconComponent
+      CatalogController.blacklight_config.navbar.partials = CatalogController.blacklight_config.navbar.partials.except(:bookmark)
+    end
+
+    after do
+      CatalogController.blacklight_config.navbar.partials = default_nav_partials
     end
 
-    it 'shows bookmark icon instead of checkbox', :js do
+    it 'adds and removes bookmarks', :js do
       visit solr_document_path('2007020969')
-      expect(page).to have_css('.blacklight-icons-bookmark')
+      expect(page).to have_no_css('#bookmarks_nav')
       find('.blacklight-icons-bookmark').click
 
-      expect(find('.toggle-bookmark[type="checkbox"]', visible: false)).to be_checked
+      expect(page).to have_content 'In Bookmarks'
+
+      visit solr_document_path('2007020969')
+      expect(find('.toggle-bookmark-input', visible: false)).to be_checked
       find('.blacklight-icons-bookmark').click
     end
   end

data/spec/features/facets_spec.rb

@@ -100,91 +100,86 @@ RSpec.describe "Facets" do
   end
 
   describe 'Facet modal content' do
-    context 'when facet is configured with suggest: true' do
-      # BL8: turn on the suggest feature for testing (off by default)
-      before do
-        CatalogController.blacklight_config.facet_fields[:subject_ssim].suggest = true
-      end
+    it 'allows the user to filter a long list of facet values', :js do
+      visit '/catalog/facet/subject_ssim'
+      expect(page).to have_no_link 'Old age' # This is on the second page of facet values
+      expect(page).to have_css 'a.facet-select', count: 20
 
-      after do
-        CatalogController.blacklight_config.facet_fields[:subject_ssim].delete(:suggest)
-      end
-
-      it 'allows the user to filter a long list of facet values', :js do
-        visit '/catalog/facet/subject_ssim'
-        expect(page).to have_no_link 'Old age' # This is on the second page of facet values
-        expect(page).to have_css 'a.facet-select', count: 20
+      fill_in 'facet_suggest_subject_ssim', with: "ag"
 
-        fill_in 'facet_suggest_subject_ssim', with: "ag"
+      expect(page).to have_css '.facet-suggestions'
+      expect(page).to have_link 'Old age'
+      expect(page).to have_css 'a.facet-select', count: 2
+    end
 
-        expect(page).to have_css '.facet-suggestions'
-        expect(page).to have_link 'Old age'
-        expect(page).to have_css 'a.facet-select', count: 2
-      end
+    it 'shows the user facet suggestions that are relevant to their q param', :js do
+      visit '/catalog/facet/subject_ssim?q=tibet&search_field=all_fields'
+      fill_in 'facet_suggest_subject_ssim', with: 'la'
 
-      it 'shows the user facet suggestions that are relevant to their q param', :js do
-        visit '/catalog/facet/subject_ssim?q=tibet&search_field=all_fields'
-        fill_in 'facet_suggest_subject_ssim', with: 'la'
+      expect(page).to have_css '.facet-suggestions'
+      expect(page).to have_link 'Tibetan language'
+      expect(page).to have_no_link 'Law'
+      expect(page).to have_css 'a.facet-select', count: 1
+    end
 
-        expect(page).to have_css '.facet-suggestions'
-        expect(page).to have_link 'Tibetan language'
-        expect(page).to have_no_link 'Law'
-        expect(page).to have_css 'a.facet-select', count: 1
-      end
+    it 'allows the user to toggle the sort, then filter', :js do
+      visit '/catalog/facet/subject_ssim'
 
-      it 'allows the user to toggle the sort, then filter', :js do
-        visit '/catalog/facet/subject_ssim'
+      fill_in 'facet_suggest_subject_ssim', with: 'po'
 
-        fill_in 'facet_suggest_subject_ssim', with: 'po'
+      expect(page).to have_css '.facet-suggestions'
+      expect(page).to have_css('.facet-values li:nth-child(1)', text: 'Political plays, Japanese')
+      expect(page).to have_css('.facet-values li:nth-child(2)', text: 'Military weapons')
+      expect(page).to have_css('.facet-values li:nth-child(3)', text: 'Political science')
 
-        expect(page).to have_css '.facet-suggestions'
-        expect(page).to have_css('.facet-values li:nth-child(1)', text: 'Political plays, Japanese')
-        expect(page).to have_css('.facet-values li:nth-child(2)', text: 'Military weapons')
-        expect(page).to have_css('.facet-values li:nth-child(3)', text: 'Political science')
+      first(:link, 'A-Z Sort').click
 
-        first(:link, 'A-Z Sort').click
+      expect(page).to have_css('.facet-suggest[data-facet-search-context*="facet.sort=index"]')
 
-        expect(page).to have_css('.facet-suggest[data-facet-search-context*="facet.sort=index"]')
+      fill_in 'facet_suggest_subject_ssim', with: 'po'
 
-        fill_in 'facet_suggest_subject_ssim', with: 'po'
+      expect(page).to have_css '.facet-suggestions'
+      expect(page).to have_css('.facet-values li:nth-child(1)', text: 'Military weapons')
+      expect(page).to have_css('.facet-values li:nth-child(2)', text: 'Political plays, Japanese')
+      expect(page).to have_css('.facet-values li:nth-child(3)', text: 'Political science')
+    end
 
-        expect(page).to have_css '.facet-suggestions'
-        expect(page).to have_css('.facet-values li:nth-child(1)', text: 'Military weapons')
-        expect(page).to have_css('.facet-values li:nth-child(2)', text: 'Political plays, Japanese')
-        expect(page).to have_css('.facet-values li:nth-child(3)', text: 'Political science')
-      end
+    it 'allows the user to choose a starting letter, then filter', :js do
+      visit '/catalog/facet/subject_ssim'
 
-      it 'allows the user to choose a starting letter, then filter', :js do
-        visit '/catalog/facet/subject_ssim'
+      first(:link, 'A-Z Sort').click
+      expect(page).to have_css('.facet-suggest[data-facet-search-context*="facet.sort=index"]')
 
-        first(:link, 'A-Z Sort').click
-        expect(page).to have_css('.facet-suggest[data-facet-search-context*="facet.sort=index"]')
+      click_on 'M'
+      expect(page).to have_css('.facet-suggest[data-facet-search-context*="facet.prefix=M"]')
 
-        click_on 'M'
-        expect(page).to have_css('.facet-suggest[data-facet-search-context*="facet.prefix=M"]')
+      fill_in 'facet_suggest_subject_ssim', with: 'te'
 
-        fill_in 'facet_suggest_subject_ssim', with: 'te'
+      expect(page).to have_css '.facet-suggestions'
+      expect(page).to have_link 'Maternity insurance'
+      expect(page).to have_no_link 'Teaching'
+    end
 
-        expect(page).to have_css '.facet-suggestions'
-        expect(page).to have_link 'Maternity insurance'
-        expect(page).to have_no_link 'Teaching'
-      end
+    it 'hides previous/next links when filtering', :js do
+      visit '/catalog/facet/subject_ssim'
+      expect(page).to have_link 'Next »'
 
-      it 'hides previous/next links when filtering', :js do
-        visit '/catalog/facet/subject_ssim'
-        expect(page).to have_link 'Next »'
+      fill_in 'facet_suggest_subject_ssim', with: 'te'
+      expect(page).to have_css '.facet-suggestions'
+      expect(page).to have_no_link 'Next »'
 
-        fill_in 'facet_suggest_subject_ssim', with: 'te'
-        expect(page).to have_css '.facet-suggestions'
-        expect(page).to have_no_link 'Next »'
+      fill_in 'facet_suggest_subject_ssim', with: ''
+      expect(page).to have_no_css '.facet-suggestions'
+      expect(page).to have_link 'Next »'
+    end
 
-        fill_in 'facet_suggest_subject_ssim', with: ''
-        expect(page).to have_no_css '.facet-suggestions'
-        expect(page).to have_link 'Next »'
+    context 'when facet is configured with suggest: false' do
+      before do
+        enabled = CatalogController.blacklight_config.dup
+        enabled.facet_fields[:subject_ssim].merge!({ suggest: false })
+        allow(CatalogController).to receive(:blacklight_config).and_return enabled
       end
-    end
 
-    context 'when facet is NOT configured with suggest: true' do
       it 'does not offer the user a way to filter the list of facet values' do
         visit '/catalog/facet/subject_ssim'
         expect(page).to have_no_field 'facet_suggest_subject_ssim'

data/spec/features/search_context_spec.rb

@@ -47,10 +47,9 @@ RSpec.describe "Search Results context" do
       search_for ''
       expect(page).to have_content "1 - 10"
       find_all('.index_title a').last.click
-      expect(page).to have_css "h1", text: "Pluvial nectar"
       click_on "Next »"
 
-      expect(page).to have_css "h1", text: "Naqdī barā-yi tamām-i"
+      expect(page).to have_content "Naqdī barā-yi tamām-i"
       click_on "Back to Search"
       expect(page).to have_content "11 - 20"
     end

data/spec/features/search_filters_spec.rb

@@ -171,7 +171,7 @@ RSpec.describe "Facets" do
     visit root_path
 
     within(".blacklight-subject_ssim") do
-      expect(page).to have_no_css(".card-body", visible: true)
+      expect(page).to have_no_css(".accordion-body", visible: true)
     end
   end
 
@@ -179,9 +179,9 @@ RSpec.describe "Facets" do
     visit root_path
 
     within(".blacklight-subject_ssim") do
-      expect(page).to have_no_css(".card-body", visible: true)
+      expect(page).to have_no_css(".accordion-body", visible: true)
       click_on 'Topic'
-      expect(page).to have_css(".card-body", visible: true)
+      expect(page).to have_css(".accordion-body", visible: true)
     end
   end
 
@@ -189,9 +189,9 @@ RSpec.describe "Facets" do
     visit root_path
 
     within(".blacklight-subject_ssim") do
-      expect(page).to have_no_css(".card-body", visible: true)
-      find(".card-header").click
-      expect(page).to have_css(".card-body", visible: true)
+      expect(page).to have_no_css(".accordion-body", visible: true)
+      find(".accordion-header").click
+      expect(page).to have_css(".accordion-body", visible: true)
     end
   end
 

data/spec/helpers/blacklight/configuration_helper_behavior_spec.rb

@@ -93,15 +93,6 @@ RSpec.describe Blacklight::ConfigurationHelperBehavior do
     end
   end
 
-  describe "#per_page_options_for_select" do
-    it "is the per-page values formatted as options_for_select" do
-      allow(helper).to receive_messages(blacklight_config: double(per_page: [11, 22, 33]))
-      expect(helper.per_page_options_for_select).to include ["11<span class=\"sr-only visually-hidden\"> per page</span>", 11]
-      expect(helper.per_page_options_for_select).to include ["22<span class=\"sr-only visually-hidden\"> per page</span>", 22]
-      expect(helper.per_page_options_for_select).to include ["33<span class=\"sr-only visually-hidden\"> per page</span>", 33]
-    end
-  end
-
   describe "#should_render_field?" do
     let(:field_config) { double('field config', if: true, unless: false) }
 

data/spec/helpers/blacklight/facets_helper_behavior_spec.rb

@@ -26,4 +26,14 @@ RSpec.describe Blacklight::FacetsHelperBehavior do
       expect(presenter).to be_a SomePresenter
     end
   end
+
+  describe "#search_facet_path" do
+    before do
+      params[:controller] = 'catalog'
+    end
+
+    it "is the same as the catalog path" do
+      expect(helper.search_facet_path(id: "some_facet", page: 5)).to eq facet_catalog_path(id: "some_facet")
+    end
+  end
 end

data/spec/helpers/blacklight/render_partials_helper_behavior_spec.rb

@@ -31,7 +31,7 @@ RSpec.describe Blacklight::RenderPartialsHelperBehavior do
     end
   end
 
-  describe "#render_document_partials" do
+  describe "#render_document_partial" do
     let(:doc) { double }
 
     before do

data/spec/lib/blacklight/configuration/facet_field_spec.rb

@@ -23,7 +23,7 @@ RSpec.describe Blacklight::Configuration::FacetField do
       subject.normalize!
 
       expect(subject.presenter).to eq Blacklight::FacetFieldPresenter
-      expect(subject.component).to eq Blacklight::FacetFieldListComponent
+      expect(subject.component).to eq Blacklight::Facets::ListComponent
     end
 
     context 'when component is set to true' do
@@ -34,7 +34,7 @@ RSpec.describe Blacklight::Configuration::FacetField do
       it 'casts to the default component' do
         subject.normalize!
 
-        expect(subject.component).to eq Blacklight::FacetFieldListComponent
+        expect(subject.component).to eq Blacklight::Facets::ListComponent
       end
     end
   end