blacklight 8.11.0 → 9.0.0.beta2

data/spec/components/blacklight/facet_component_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 
 RSpec.describe Blacklight::FacetComponent, type: :component do
   subject(:rendered) do
-    render_inline_to_capybara_node(component)
+    render_inline(component)
   end
 
   let(:component) { described_class.new(**component_kwargs) }
@@ -18,13 +18,14 @@ RSpec.describe Blacklight::FacetComponent, type: :component do
   let(:facet_config) { Blacklight::Configuration::FacetField.new(key: 'field').normalize! }
 
   it 'delegates to the configured component to render something' do
-    expect(rendered).to have_css 'ul.facet-values'
+    rendered
+    expect(page).to have_css 'ul.facet-values'
   end
 
   context 'with a provided component' do
     let(:component_kwargs) { { field_config: facet_config, display_facet: display_facet, component: component_class } }
     let(:component_class) do
-      Class.new(Blacklight::FacetFieldListComponent) do
+      Class.new(Blacklight::Facets::ListComponent) do
         def self.name
           'CustomFacetComponent'
         end
@@ -35,8 +36,10 @@ RSpec.describe Blacklight::FacetComponent, type: :component do
       end
     end
 
+    before { rendered }
+
     it 'renders the provided component' do
-      expect(rendered).to have_content 'Custom facet rendering'
+      expect(page).to have_content 'Custom facet rendering'
     end
   end
 
@@ -53,14 +56,15 @@ RSpec.describe Blacklight::FacetComponent, type: :component do
       replace_hash = { 'catalog/_facet_partial.html.erb' => 'facet partial' }
 
       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
-        controller.view_context.view_paths.unshift(RSpec::Rails::ViewExampleGroup::StubResolverCache.resolver_for(replace_hash))
+        vc_test_controller.view_context.view_paths.unshift(RSpec::Rails::ViewExampleGroup::StubResolverCache.resolver_for(replace_hash))
       end
+      rendered
     end
 
     it 'renders the partial' do
-      expect(rendered).to have_content 'facet partial'
+      expect(page).to have_content 'facet partial'
     end
   end
 
@@ -113,7 +117,7 @@ RSpec.describe Blacklight::FacetComponent, type: :component do
       { display_facet_or_field_config: presenter }
     end
 
-    let(:presenter) { Blacklight::FacetFieldPresenter.new(facet_config, display_facet, controller.view_context) }
+    let(:presenter) { Blacklight::FacetFieldPresenter.new(facet_config, display_facet, vc_test_controller.view_context) }
 
     it 'renders the component with the provided presenter' do
       allow(facet_config.component).to receive(:new).and_call_original

data/spec/components/blacklight/facet_item_pivot_component_spec.rb

@@ -3,8 +3,8 @@
 require 'spec_helper'
 
 RSpec.describe Blacklight::FacetItemPivotComponent, type: :component do
-  subject(:rendered) do
-    render_inline_to_capybara_node(described_class.new(facet_item: facet_item))
+  before do
+    render_inline(described_class.new(facet_item: facet_item))
   end
 
   let(:blacklight_config) do
@@ -31,17 +31,17 @@ RSpec.describe Blacklight::FacetItemPivotComponent, type: :component do
     )
   end
 
-  let(:facet_config) { Blacklight::Configuration::NullField.new(key: 'z', item_component: Blacklight::FacetItemComponent, item_presenter: Blacklight::FacetItemPivotPresenter) }
+  let(:facet_config) { Blacklight::Configuration::NullField.new(key: 'z', item_component: Blacklight::Facets::ItemComponent, item_presenter: Blacklight::FacetItemPivotPresenter) }
 
   it 'links to the facet and shows the number of hits' do
-    expect(rendered).to have_css 'li'
-    expect(rendered).to have_link 'x', href: nokogiri_mediated_href(facet_item.href)
-    expect(rendered).to have_css '.facet-count', text: '10'
+    expect(page).to have_css 'li'
+    expect(page).to have_link 'x', href: nokogiri_mediated_href(facet_item.href)
+    expect(page).to have_css '.facet-count', text: '10'
   end
 
   it 'has the facet hierarchy' do
-    expect(rendered).to have_css 'li ul.pivot-facet'
-    expect(rendered).to have_link 'x:1', href: nokogiri_mediated_href(facet_item.facet_item_presenters.first.href)
+    expect(page).to have_css 'li ul.pivot-facet'
+    expect(page).to have_link 'x:1', href: nokogiri_mediated_href(facet_item.facet_item_presenters.first.href)
   end
 
   context 'with a selected facet' do
@@ -60,10 +60,10 @@ RSpec.describe Blacklight::FacetItemPivotComponent, type: :component do
     end
 
     it 'links to the facet and shows the number of hits' do
-      expect(rendered).to have_css 'li'
-      expect(rendered).to have_css '.selected', text: 'x'
-      expect(rendered).to have_link '[remove]', href: '/catalog'
-      expect(rendered).to have_css '.selected.facet-count', text: '10'
+      expect(page).to have_css 'li'
+      expect(page).to have_css '.selected', text: 'x'
+      expect(page).to have_link '[remove]', href: '/catalog'
+      expect(page).to have_css '.selected.facet-count', text: '10'
     end
   end
 end

data/spec/components/blacklight/{facet_field_checkboxes_component_spec.rb → facets/checkboxes_component_spec.rb}

@@ -2,15 +2,15 @@
 
 require 'spec_helper'
 
-RSpec.describe Blacklight::FacetFieldCheckboxesComponent, type: :component do
-  subject(:rendered) do
-    render_inline_to_capybara_node(described_class.new(facet_field: facet_field))
+RSpec.describe Blacklight::Facets::CheckboxesComponent, type: :component do
+  before do
+    render_inline(described_class.new(facet_field: facet_field))
   end
 
   let(:facet_field) do
     instance_double(
       Blacklight::FacetFieldPresenter,
-      facet_field: Blacklight::Configuration::NullField.new(key: 'field', item_component: Blacklight::FacetItemComponent, item_presenter: Blacklight::FacetItemPresenter),
+      facet_field: Blacklight::Configuration::NullField.new(key: 'field', item_component: Blacklight::Facets::ItemComponent, item_presenter: Blacklight::FacetItemPresenter),
       paginator: paginator,
       key: 'field',
       label: 'Field',
@@ -32,19 +32,19 @@ RSpec.describe Blacklight::FacetFieldCheckboxesComponent, type: :component do
   let(:search_state) { Blacklight::SearchState.new(params.with_indifferent_access, Blacklight::Configuration.new) }
   let(:params) { { f: { field: ['a'] } } }
 
-  it 'renders a collapsible card' do
-    expect(rendered).to have_css '.card'
-    expect(rendered).to have_button 'Field'
-    expect(rendered).to have_css 'button[data-bs-target="#facet-field"]'
-    expect(rendered).to have_css '#facet-field.collapse.show'
+  it 'renders an accordion item' do
+    expect(page).to have_css '.accordion-item'
+    expect(page).to have_button 'Field'
+    expect(page).to have_css 'button[data-bs-target="#facet-field"]'
+    expect(page).to have_css '#facet-field.collapse.show'
   end
 
   it 'renders the facet items' do
-    expect(rendered).to have_css 'ul.facet-values'
-    expect(rendered).to have_css 'li', count: 3
+    expect(page).to have_css 'ul.facet-values'
+    expect(page).to have_css 'li', count: 3
 
-    expect(rendered).to have_field 'f_inclusive[field][]', with: 'a'
-    expect(rendered).to have_field 'f_inclusive[field][]', with: 'b'
-    expect(rendered).to have_field 'f_inclusive[field][]', with: 'c'
+    expect(page).to have_field 'f_inclusive[field][]', with: 'a'
+    expect(page).to have_field 'f_inclusive[field][]', with: 'b'
+    expect(page).to have_field 'f_inclusive[field][]', with: 'c'
   end
 end

data/spec/components/blacklight/facets/filters_component_spec.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Blacklight::Facets::FiltersComponent, type: :component do
+  let(:facet_field) { Blacklight::Configuration::FacetField.new key: 'language_facet', suggest: true }
+  let(:presenter) do
+    instance_double(Blacklight::FacetFieldPresenter, facet_field: facet_field, label: 'Lang',
+                                                     view_context: view_context, suggest: true, key: 'lang')
+  end
+  let(:view_context) { vc_test_controller.view_context }
+
+  before do
+    allow(view_context).to receive(:search_facet_path).and_return('/catalog/facet/language_facet')
+
+    with_request_url '/catalog?q=foo' do
+      render_inline(component)
+    end
+  end
+
+  context 'with default classes' do
+    let(:component) { described_class.new(presenter: presenter) }
+
+    it 'draws default classes' do
+      expect(page).to have_css(".facet-filters.card.card-body.bg-light.p-3.mb-3.border-0")
+    end
+  end
+
+  context 'with custom classes' do
+    let(:component) { described_class.new(presenter: presenter, classes: 'foo') }
+
+    it 'draws default classes' do
+      expect(page).to have_css(".foo")
+    end
+  end
+end

data/spec/components/blacklight/facets/index_navigation_component_spec.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+RSpec.describe Blacklight::Facets::IndexNavigationComponent, type: :component do
+  let(:pagination) { Blacklight::Solr::FacetPaginator.new([]) }
+  let(:facet) { Blacklight::Configuration::FacetField.new(index_range: '0'..'9', presenter: Blacklight::FacetFieldPresenter) }
+  let(:display_facet) { instance_double(Blacklight::Solr::Response::Facets::FacetField, items: [], offset: 0, prefix: '', sort: 'index', index?: true) }
+  let(:blacklight_config) { Blacklight::Configuration.new }
+
+  let(:presenter) { facet.presenter.new(facet, display_facet, vc_test_controller.view_context) }
+
+  before do
+    with_request_url "/catalog/facet/language" do
+      render_inline(described_class.new(presenter: presenter))
+    end
+  end
+
+  it 'renders the facet index navigation range' do
+    expect(page).to have_css '.pagination'
+    expect(page).to have_link '0', href: '/catalog/facet/language.html?facet.prefix=0&facet.sort=index'
+    expect(page).to have_link '1'
+    expect(page).to have_link '8'
+    expect(page).to have_link '9'
+  end
+
+  it 'renders an "all" button' do
+    expect(page).to have_css '.page-link', text: 'All'
+  end
+
+  context 'with a selected index' do
+    let(:display_facet) { instance_double(Blacklight::Solr::Response::Facets::FacetField, items: [], offset: 0, prefix: '5', sort: 'index', index?: true) }
+
+    it 'highlights the selected index' do
+      expect(page).to have_css '.active', text: '5'
+    end
+
+    it 'enables the clear facets button' do
+      expect(page).to have_link 'All'
+    end
+  end
+end

data/spec/components/blacklight/{facet_item_component_spec.rb → facets/item_component_spec.rb}

@@ -2,9 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Blacklight::FacetItemComponent, type: :component do
-  subject(:rendered) do
-    render_inline_to_capybara_node(described_class.new(facet_item: facet_item))
+RSpec.describe Blacklight::Facets::ItemComponent, type: :component do
+  before do
+    render_inline(described_class.new(facet_item: facet_item))
   end
 
   let(:facet_item) do
@@ -19,11 +19,11 @@ RSpec.describe Blacklight::FacetItemComponent, type: :component do
   end
 
   it 'links to the facet and shows the number of hits' do
-    expect(rendered).to have_css 'li'
-    expect(rendered).to have_link 'x', href: '/catalog?f=x' do |link|
+    expect(page).to have_css 'li'
+    expect(page).to have_link 'x', href: '/catalog?f=x' do |link|
       link['rel'] == 'nofollow'
     end
-    expect(rendered).to have_css '.facet-count', text: '10'
+    expect(page).to have_css '.facet-count', text: '10'
   end
 
   context 'with a selected facet' do
@@ -39,12 +39,12 @@ RSpec.describe Blacklight::FacetItemComponent, type: :component do
     end
 
     it 'links to the facet and shows the number of hits' do
-      expect(rendered).to have_css 'li'
-      expect(rendered).to have_css '.selected', text: 'x'
-      expect(rendered).to have_link '[remove]', href: '/catalog' do |link|
+      expect(page).to have_css 'li'
+      expect(page).to have_css '.selected', text: 'x'
+      expect(page).to have_link '[remove]', href: '/catalog' do |link|
         link['rel'] == 'nofollow'
       end
-      expect(rendered).to have_css '.selected.facet-count', text: '10'
+      expect(page).to have_css '.selected.facet-count', text: '10'
     end
   end
 end

data/spec/components/blacklight/{facet_field_list_component_spec.rb → facets/list_component_spec.rb}

@@ -2,9 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
-  subject(:rendered) do
-    render_inline_to_capybara_node(described_class.new(facet_field: facet_field))
+RSpec.describe Blacklight::Facets::ListComponent, type: :component do
+  before do
+    render_inline(described_class.new(facet_field: facet_field))
   end
 
   let(:facet_field) do
@@ -21,7 +21,7 @@ RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
     )
   end
 
-  let(:facet_config) { Blacklight::Configuration::NullField.new(key: 'field', item_component: Blacklight::FacetItemComponent, item_presenter: Blacklight::FacetItemPresenter) }
+  let(:facet_config) { Blacklight::Configuration::NullField.new(key: 'field', item_component: Blacklight::Facets::ItemComponent, item_presenter: Blacklight::FacetItemPresenter) }
 
   let(:paginator) do
     instance_double(Blacklight::FacetPaginator, items: [
@@ -30,16 +30,16 @@ RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
                     ])
   end
 
-  it 'renders a collapsible card' do
-    expect(rendered).to have_css '.card'
-    expect(rendered).to have_button 'Field'
-    expect(rendered).to have_css 'button[data-bs-target="#facet-field"]'
-    expect(rendered).to have_css '#facet-field.collapse.show'
+  it 'renders an accordion item' do
+    expect(page).to have_css '.accordion-item'
+    expect(page).to have_button 'Field'
+    expect(page).to have_css 'button[data-bs-target="#facet-field"]'
+    expect(page).to have_css '#facet-field.collapse.show'
   end
 
   it 'renders the facet items' do
-    expect(rendered).to have_css 'ul.facet-values'
-    expect(rendered).to have_css 'li', count: 2
+    expect(page).to have_css 'ul.facet-values'
+    expect(page).to have_css 'li', count: 2
   end
 
   context 'with an active facet' do
@@ -58,7 +58,7 @@ RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
     end
 
     it 'adds the facet-limit-active class' do
-      expect(rendered).to have_css 'div.facet-limit-active'
+      expect(page).to have_css 'div.facet-limit-active'
     end
   end
 
@@ -78,13 +78,13 @@ RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
     end
 
     it 'renders a collapsed facet' do
-      expect(rendered).to have_css '.facet-content.collapse'
-      expect(rendered).to have_no_css '.facet-content.collapse.show'
+      expect(page).to have_css '.facet-content.collapse'
+      expect(page).to have_no_css '.facet-content.collapse.show'
     end
 
     it 'renders the toggle button in the collapsed state' do
-      expect(rendered).to have_css '.btn.collapsed'
-      expect(rendered).to have_css '.btn[aria-expanded="false"]'
+      expect(page).to have_css '.btn.collapsed'
+      expect(page).to have_css '.btn[aria-expanded="false"]'
     end
   end
 
@@ -104,7 +104,7 @@ RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
     end
 
     it 'renders a link to the modal' do
-      expect(rendered).to have_link 'more Field', href: '/catalog/facet/modal'
+      expect(page).to have_link 'more Field', href: '/catalog/facet/modal'
     end
   end
 
@@ -133,13 +133,13 @@ RSpec.describe Blacklight::FacetFieldListComponent, type: :component do
     let(:params) { { f_inclusive: { field: %w[a b c] } } }
 
     it 'displays the constraint above the list' do
-      expect(rendered).to have_content 'Any of:'
-      expect(rendered).to have_css '.inclusive_or .facet-label', text: 'a'
-      expect(rendered).to have_link '[remove]', href: 'http://test.host/catalog?f_inclusive%5Bfield%5D%5B%5D=b&f_inclusive%5Bfield%5D%5B%5D=c'
-      expect(rendered).to have_css '.inclusive_or .facet-label', text: 'b'
-      expect(rendered).to have_link '[remove]', href: 'http://test.host/catalog?f_inclusive%5Bfield%5D%5B%5D=a&f_inclusive%5Bfield%5D%5B%5D=c'
-      expect(rendered).to have_css '.inclusive_or .facet-label', text: 'c'
-      expect(rendered).to have_link '[remove]', href: 'http://test.host/catalog?f_inclusive%5Bfield%5D%5B%5D=a&f_inclusive%5Bfield%5D%5B%5D=b'
+      expect(page).to have_content 'Any of:'
+      expect(page).to have_css '.inclusive_or .facet-label', text: 'a'
+      expect(page).to have_link '[remove]', href: 'http://test.host/catalog?f_inclusive%5Bfield%5D%5B%5D=b&f_inclusive%5Bfield%5D%5B%5D=c'
+      expect(page).to have_css '.inclusive_or .facet-label', text: 'b'
+      expect(page).to have_link '[remove]', href: 'http://test.host/catalog?f_inclusive%5Bfield%5D%5B%5D=a&f_inclusive%5Bfield%5D%5B%5D=c'
+      expect(page).to have_css '.inclusive_or .facet-label', text: 'c'
+      expect(page).to have_link '[remove]', href: 'http://test.host/catalog?f_inclusive%5Bfield%5D%5B%5D=a&f_inclusive%5Bfield%5D%5B%5D=b'
     end
   end
 end

data/spec/components/blacklight/facets/suggest_component_spec.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Blacklight::Facets::SuggestComponent, type: :component do
+  let(:facet) { Blacklight::Configuration::FacetField.new key: 'language_facet', suggest: true }
+  let(:presenter) { Blacklight::FacetFieldPresenter.new(facet, nil, nil, nil) }
+  let(:component) { described_class.new(presenter: presenter) }
+
+  before do
+    allow(presenter).to receive_messages(label: 'Language')
+  end
+
+  it 'has an input with the facet-suggest class, which the javascript needs to find it' do
+    with_request_url "/catalog/facet/language_facet" do
+      rendered = render_inline component
+      expect(rendered.css("input.facet-suggest").count).to eq 1
+    end
+  end
+
+  it 'has an input with the data-facet-field attribute, which the javascript needs to determine the correct query' do
+    with_request_url "/catalog/facet/language_facet" do
+      rendered = render_inline component
+      expect(rendered.css('input[data-facet-field="language_facet"]').count).to eq 1
+    end
+  end
+
+  it 'has an input with the data-facet-search-context attribute, which the javascript needs to determine the current search context' do
+    with_request_url "/catalog/facet/language_facet?f%5Bformat%5D%5B%5D=Book&facet.prefix=R&facet.sort=index&q=tibet&search_field=all_fields" do
+      rendered = render_inline component
+      expect(rendered.css('input[data-facet-search-context="/catalog/facet/language_facet.html?f%5Bformat%5D%5B%5D=Book&facet.prefix=R&facet.sort=index&q=tibet&search_field=all_fields"]').count).to eq 1
+    end
+  end
+
+  it 'has a visible label that is associated with the input' do
+    with_request_url "/catalog/facet/language_facet" do
+      rendered = render_inline component
+      label = rendered.css('label').first
+      expect(label.text.strip).to eq 'Filter Language'
+
+      id_in_label_for = label.attribute('for').text
+      expect(id_in_label_for).to eq('facet_suggest_language_facet')
+
+      expect(rendered.css('input').first.attribute('id').text).to eq id_in_label_for
+    end
+  end
+
+  context 'when the facet is explicitly configured to suggest: false' do
+    let(:facet) { Blacklight::Configuration::FacetField.new key: 'language_facet', suggest: false }
+
+    it 'does not display' do
+      with_request_url "/catalog/facet/language_facet" do
+        expect(render_inline(component).to_s).to eq ''
+      end
+    end
+  end
+
+  context 'when the facet is not explicitly configured with a suggest key' do
+    let(:facet) { Blacklight::Configuration::FacetField.new key: 'language_facet' }
+
+    it 'displays' do
+      with_request_url "/catalog/facet/language_facet" do
+        rendered = render_inline component
+        expect(rendered.css("input.facet-suggest").count).to eq 1
+      end
+    end
+  end
+end

data/spec/components/blacklight/header_component_spec.rb

@@ -3,14 +3,12 @@
 RSpec.describe Blacklight::HeaderComponent, type: :component do
   before do
     with_controller_class(CatalogController) do
-      allow(controller).to receive_messages(current_user: nil, search_action_url: '/search')
-      render
+      allow(vc_test_controller).to receive_messages(current_user: nil, search_action_url: '/search')
+      render_inline described_class.new(blacklight_config: CatalogController.blacklight_config)
     end
   end
 
   context 'with no slots' do
-    let(:render) { render_inline(described_class.new(blacklight_config: CatalogController.blacklight_config)) }
-
     it 'draws the topbar' do
       expect(page).to have_css 'nav.topbar'
       expect(page).to have_link 'Blacklight', href: '/'

data/spec/components/blacklight/hidden_search_state_component_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 RSpec.describe Blacklight::HiddenSearchStateComponent, type: :component do
-  subject(:rendered) { render_inline_to_capybara_node(instance) }
+  before { render_inline(instance) }
 
   let(:params) do
     { q: "query",
@@ -13,11 +13,11 @@ RSpec.describe Blacklight::HiddenSearchStateComponent, type: :component do
   let(:instance) { described_class.new(params: params) }
 
   it "converts a hash with nested complex data to Rails-style hidden form fields" do
-    expect(rendered).to have_css("input[type='hidden'][name='q'][value='query']", visible: :hidden)
-    expect(rendered).to have_css("input[type='hidden'][name='per_page'][value='10']", visible: :hidden)
-    expect(rendered).to have_css("input[type='hidden'][name='extra_arbitrary_key'][value='arbitrary_value']", visible: :hidden)
-    expect(rendered).to have_css("input[type='hidden'][name='f[field2][]'][value='z']", visible: :hidden)
-    expect(rendered).to have_css("input[type='hidden'][name='f[field1][]'][value='a']", visible: :hidden)
-    expect(rendered).to have_css("input[type='hidden'][name='f[field1][]'][value='b']", visible: :hidden)
+    expect(page).to have_css("input[type='hidden'][name='q'][value='query']", visible: :hidden)
+    expect(page).to have_css("input[type='hidden'][name='per_page'][value='10']", visible: :hidden)
+    expect(page).to have_css("input[type='hidden'][name='extra_arbitrary_key'][value='arbitrary_value']", visible: :hidden)
+    expect(page).to have_css("input[type='hidden'][name='f[field2][]'][value='z']", visible: :hidden)
+    expect(page).to have_css("input[type='hidden'][name='f[field1][]'][value='a']", visible: :hidden)
+    expect(page).to have_css("input[type='hidden'][name='f[field1][]'][value='b']", visible: :hidden)
   end
 end

data/spec/components/blacklight/metadata_field_component_spec.rb

@@ -3,11 +3,7 @@
 require 'spec_helper'
 
 RSpec.describe Blacklight::MetadataFieldComponent, type: :component do
-  subject(:rendered) do
-    render_inline_to_capybara_node(described_class.new(field: field))
-  end
-
-  let(:view_context) { controller.view_context }
+  let(:view_context) { vc_test_controller.view_context }
   let(:document) { SolrDocument.new('field' => ['Value']) }
   let(:field_config) { Blacklight::Configuration::Field.new(key: 'field', field: 'field', label: 'Field label') }
 
@@ -15,23 +11,29 @@ RSpec.describe Blacklight::MetadataFieldComponent, type: :component do
     Blacklight::FieldPresenter.new(view_context, document, field_config)
   end
 
-  it 'renders the field label' do
-    expect(rendered).to have_css 'dt.blacklight-field', text: 'Field label'
-  end
+  context "from index view" do
+    before do
+      render_inline(described_class.new(field: field))
+    end
+
+    it 'renders the field label' do
+      expect(page).to have_css 'dt.blacklight-field', text: 'Field label'
+    end
 
-  it 'renders the field value' do
-    expect(rendered).to have_css 'dd.blacklight-field', text: 'Value'
+    it 'renders the field value' do
+      expect(page).to have_css 'dd.blacklight-field', text: 'Value'
+    end
   end
 
   context 'from a show view' do
-    subject(:rendered) do
-      render_inline_to_capybara_node(described_class.new(field: field, show: true))
+    before do
+      allow(field).to receive(:label).with('show').and_return('custom label')
+
+      render_inline(described_class.new(field: field, show: true))
     end
 
     it 'renders the right field label' do
-      allow(field).to receive(:label).with('show').and_return('custom label')
-
-      expect(rendered).to have_css 'dt.blacklight-field', text: 'custom label'
+      expect(page).to have_css 'dt.blacklight-field', text: 'custom label'
     end
   end
 end

data/spec/components/blacklight/response/facet_group_component_spec.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Blacklight::Response::FacetGroupComponent, type: :component do
+  before do
+    render_inline(instance)
+  end
+
+  context 'when classes are passed in' do
+    let(:instance) do
+      described_class.new(id: 'foo', title: 'bar', body_classes: 'custom-class').tap do |component|
+        component.with_body { 'body' }
+      end
+    end
+
+    it "uses them" do
+      within '.facets' do
+        expect(page).to have_css 'div.custom-class'
+      end
+    end
+  end
+
+  context 'when classes are not passed in' do
+    let(:instance) do
+      described_class.new(id: 'foo', title: 'bar').tap do |component|
+        component.with_body { 'body' }
+      end
+    end
+
+    it "uses default classes them" do
+      within '.facets' do
+        expect(page).to have_css 'div.facets-collapse.d-lg-block.collapse.accordion'
+      end
+    end
+  end
+end

data/spec/components/blacklight/response/pagination_component_spec.rb

@@ -38,7 +38,7 @@ RSpec.describe Blacklight::Response::PaginationComponent, type: :component do
 
     context 'when a different configuration that removes deep links is configured in the controller' do
       before do
-        allow(controller.blacklight_config.index)
+        allow(vc_test_controller.blacklight_config.index)
           .to receive(:pagination_options)
           .and_return(theme: 'blacklight', left: 5, right: 0)
         render

data/spec/components/blacklight/response/spellcheck_component_spec.rb

@@ -14,7 +14,7 @@ RSpec.describe Blacklight::Response::SpellcheckComponent, type: :component do
   end
 
   before do
-    allow(controller).to receive(:blacklight_config).and_return(config)
+    allow(vc_test_controller).to receive(:blacklight_config).and_return(config)
   end
 
   context 'when there are many results' do

data/spec/components/blacklight/search_bar_component_spec.rb

@@ -15,7 +15,7 @@ RSpec.describe Blacklight::SearchBarComponent, 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 default button' do
@@ -31,7 +31,7 @@ RSpec.describe Blacklight::SearchBarComponent, type: :component do
     subject(:render) do
       render_inline(instance) do |c|
         c.with_search_button do
-          controller.view_context.tag.button "hello", id: 'custom_search'
+          vc_test_controller.view_context.tag.button "hello", id: 'custom_search'
         end
       end
     end
@@ -70,8 +70,8 @@ RSpec.describe Blacklight::SearchBarComponent, type: :component do
   context 'with extra inputs' do
     subject(:render) do
       render_inline(instance) do |c|
-        c.with_before_input_group { controller.view_context.tag.input name: 'foo' }
-        c.with_before_input_group { controller.view_context.tag.input name: 'bar' }
+        c.with_before_input_group { vc_test_controller.view_context.tag.input name: 'foo' }
+        c.with_before_input_group { vc_test_controller.view_context.tag.input name: 'bar' }
       end
     end
 
@@ -85,7 +85,7 @@ RSpec.describe Blacklight::SearchBarComponent, type: :component do
     subject(:render) { render_inline(instance) }
 
     it 'sets the rounded border class' do
-      expect(render.css('.rounded-left.rounded-start')).to be_present
+      expect(render.css('.rounded-start')).to be_present
     end
   end