blacklight 7.32.0 → 8.0.0.beta1

data/spec/views/catalog/facet.html.erb_spec.rb

@@ -3,8 +3,13 @@
 RSpec.describe 'catalog/facet.html.erb' do
   let(:display_facet) { double }
   let(:blacklight_config) { Blacklight::Configuration.new }
+  let(:component) { instance_double(Blacklight::FacetComponent) }
 
   before do
+    allow(Blacklight::FacetComponent).to receive(:new).and_return(component)
+    allow(view).to receive(:render).and_call_original
+    allow(view).to receive(:render).with(component)
+
     blacklight_config.add_facet_field 'xyz', label: "Facet title"
     allow(view).to receive(:blacklight_config).and_return(blacklight_config)
     stub_template 'catalog/_facet_pagination.html.erb' => 'pagination'
@@ -13,19 +18,17 @@ RSpec.describe 'catalog/facet.html.erb' do
   end
 
   it "has the facet title" do
-    allow(view).to receive(:render_facet_limit)
     render
     expect(rendered).to have_selector 'h1', text: "Facet title"
   end
 
   it "renders facet pagination" do
-    allow(view).to receive(:render_facet_limit)
     render
     expect(rendered).to have_content 'pagination'
   end
 
   it "renders the facet limit" do
-    allow(view).to receive(:render_facet_limit).with(display_facet, layout: false)
     render
+    expect(view).to have_received(:render).with(component)
   end
 end

data/spec/views/catalog/index.atom.builder_spec.rb

@@ -1,14 +1,15 @@
 # frozen_string_literal: true
+
 require 'rexml/document'
 
 RSpec.describe "catalog/index" do
   let(:document_list) do
     10.times.map do |i|
-      doc = SolrDocument.new(id: i)
-      allow(doc).to receive(:export_as_some_format).and_return("")
-      allow(doc).to receive(:to_semantic_values).and_return(author: ['xyz']) if i == 0
-      doc.will_export_as(:some_format, "application/some-format") if i == 1
-      doc
+      SolrDocument.new(id: i, title_tsim: "Title #{i}").tap do |doc|
+        allow(doc).to receive(:export_as_some_format).and_return("")
+        allow(doc).to receive(:to_semantic_values).and_return(author: ['xyz']) if i.zero?
+        doc.will_export_as(:some_format, "application/some-format") if i == 1
+      end
     end
   end
 
@@ -21,7 +22,6 @@ RSpec.describe "catalog/index" do
     params['content_format'] = 'some_format'
     allow(view).to receive(:action_name).and_return('index')
     allow(view).to receive(:blacklight_config).and_return(blacklight_config)
-    allow(view).to receive(:search_field_options_for_select).and_return([])
   end
 
   # We need to use rexml to test certain things that have_tag wont' test
@@ -87,15 +87,22 @@ RSpec.describe "catalog/index" do
     end
 
     it "has a summary" do
-      stub_template "catalog/_index.html.erb" => "partial content"
       render template: 'catalog/index', formats: [:atom]
-      expect(rendered).to have_selector("entry > summary", text: 'partial content')
+      expect(rendered).to have_selector("entry > summary", text: 'Title 0')
     end
 
-    context 'with a custom HTML partial' do
+    context 'with a custom template' do
       before do
-        blacklight_config.view.atom.summary_partials = ['whatever']
-        stub_template 'catalog/_whatever_default.html.erb' => 'whatever content'
+        my_template = Class.new(ViewComponent::Base) do
+          def call
+            'whatever content'
+          end
+
+          def self.name
+            'TestComponent'
+          end
+        end
+        blacklight_config.view.atom.summary_component = my_template
       end
 
       it "has the customized summary" do

data/spec/views/catalog/index.html.erb_spec.rb

@@ -4,26 +4,25 @@ RSpec.describe "catalog/index.html.erb" do
   describe "with no search parameters" do
     before do
       allow(view).to receive(:has_search_parameters?).and_return(false)
-      allow(view).to receive(:blacklight_config).and_return(Blacklight::Configuration.new)
+      allow(view).to receive(:blacklight_config).and_return(CatalogController.blacklight_config)
+      @response = instance_double(Blacklight::Solr::Response, empty?: true, total: 11, start: 1, limit_value: 10, aggregations: {})
     end
 
     let(:sidebar) { view.content_for(:sidebar) }
 
-    it "renders the search_sidebar partial" do
-      stub_template "catalog/_search_sidebar.html.erb" => "sidebar_content"
+    it "renders the Search::SidebarComponent component" do
       render
-      expect(sidebar).to match /sidebar_content/
+
+      expect(sidebar).to match /Limit your search/
     end
   end
 
   describe "with search parameters" do
     before do
-      allow(controller).to receive(:search_state_class).and_return(Blacklight::SearchState)
       allow(view).to receive(:has_search_parameters?).and_return(true)
       stub_template "catalog/_results_pagination.html.erb" => ""
       stub_template "catalog/_search_header.html.erb" => "header_content"
       allow(view).to receive(:blacklight_config).and_return(Blacklight::Configuration.new)
-      allow(view).to receive(:render_opensearch_response_metadata).and_return("")
       @response = instance_double(Blacklight::Solr::Response, empty?: true, total: 11, start: 1, limit_value: 10)
     end
 

data/spec/views/catalog/index.json.jbuilder_spec.rb

@@ -10,7 +10,7 @@ RSpec.describe "catalog/index.json", api: true do
   end
   let(:config) do
     Blacklight::Configuration.new do |config|
-      config.add_index_field 'title', label: 'Title', field: 'title_tsim'
+      config.index.title_field = 'title_tsim'
       config.add_facet_field :format
     end
   end
@@ -66,14 +66,7 @@ RSpec.describe "catalog/index.json", api: true do
                                 id: '123',
                                 type: 'Book',
                                 attributes: {
-                                  'title': {
-                                    id: 'http://test.host/catalog/123#title',
-                                    type: 'document_value',
-                                    attributes: {
-                                      value: 'Book1',
-                                      label: 'Title'
-                                    }
-                                  }
+                                  title: 'Book1'
                                 },
                                 links: { self: 'http://test.host/catalog/123' }
                               },
@@ -81,14 +74,7 @@ RSpec.describe "catalog/index.json", api: true do
                                 id: '456',
                                 type: 'Article',
                                 attributes: {
-                                  'title': {
-                                    id: 'http://test.host/catalog/456#title',
-                                    type: 'document_value',
-                                    attributes: {
-                                      value: 'Article1',
-                                      label: 'Title'
-                                    }
-                                  }
+                                  title: 'Article1'
                                 },
                                 links: { self: 'http://test.host/catalog/456' }
                               }
@@ -99,12 +85,12 @@ RSpec.describe "catalog/index.json", api: true do
     let(:facets) { hash[:included].select { |x| x['type'] == 'facet' } }
     let(:format) { facets.find { |x| x['id'] == 'format' } }
     let(:format_items) { format['attributes']['items'] }
-    let(:format_item_attributes) { format_items.map { |x| x['attributes'] } }
+    let(:format_item_attributes) { format_items.pluck('attributes') }
 
     context 'when no facets have been selected' do
       it 'has facet information and links' do
         expect(facets).to be_present
-        expect(facets.map { |x| x['id'] }).to include 'format'
+        expect(facets.pluck('id')).to include 'format'
         expect(format['links']).to include self: 'http://test.host/some/facet/url'
         expect(format['attributes']['label']).to eq 'Format'
         expect(format_item_attributes).to match_array [{ value: 'Book', hits: 30, label: 'Book' }]

data/spec/views/catalog/show.html.erb_spec.rb

@@ -3,8 +3,11 @@
 RSpec.describe "catalog/show.html.erb" do
   let(:document) { SolrDocument.new id: 'xyz', format: 'a' }
   let(:blacklight_config) { Blacklight::Configuration.new }
+  let(:presenter) { Blacklight::ShowPresenter.new(document, view, blacklight_config) }
 
   before do
+    allow(presenter).to receive(:html_title).and_return('Heading')
+    allow(view).to receive(:document_presenter).and_return(presenter)
     allow(view).to receive(:action_name).and_return('show')
     allow(view).to receive_messages(has_user_authentication_provider?: false)
     allow(view).to receive_messages(render_document_sidebar_partial: "Sidebar")
@@ -14,7 +17,6 @@ RSpec.describe "catalog/show.html.erb" do
   end
 
   it "sets the @page_title" do
-    allow(view).to receive(:document_show_html_title).and_return("Heading")
     render
     page_title = view.instance_variable_get(:@page_title)
     expect(page_title).to eq "Heading - Blacklight"
@@ -22,22 +24,12 @@ RSpec.describe "catalog/show.html.erb" do
   end
 
   it "includes schema.org itemscope/type properties" do
-    allow(view).to receive(:document_show_html_title).and_return("Heading")
     allow(document).to receive_messages(itemtype: 'some-item-type-uri')
     render
     expect(rendered).to have_selector('div#document[@itemscope]')
     expect(rendered).to have_selector('div#document[@itemtype="some-item-type-uri"]')
   end
 
-  it "renders the show_header and show partials by default" do
-    allow(view).to receive(:render_grouped_response?).and_return(false)
-    stub_template "catalog/_show_header_default.html.erb" => "document_header"
-    stub_template "catalog/_show_default.html.erb" => "show_default"
-    render
-    expect(rendered).to match /document_header/
-    expect(rendered).to match /show_default/
-  end
-
   it "uses the show.partials parameter to determine the partials to render" do
     allow(view).to receive(:render_grouped_response?).and_return(false)
     blacklight_config.show.partials = %w[a b c]
@@ -49,18 +41,4 @@ RSpec.describe "catalog/show.html.erb" do
     expect(rendered).to match /b_partial/
     expect(rendered).to match /c_partial/
   end
-
-  it 'provides the rendered partials to an explicitly configured component but does not render them by default' do
-    blacklight_config.show.partials = %w[a]
-    stub_template "catalog/_a_default.html.erb" => "partial"
-    blacklight_config.show.document_component = Blacklight::DocumentComponent
-    allow(view).to receive(:search_session).and_return({})
-    allow(view).to receive(:current_search_session).and_return(nil)
-    allow(view.main_app).to receive(:track_test_path).and_return('/track')
-
-    render
-
-    expect(rendered).to have_selector 'div.document header h1', text: 'xyz'
-    expect(rendered).not_to match(/partial/)
-  end
 end

data/spec/views/catalog/show.json.jbuilder_spec.rb

@@ -6,7 +6,7 @@ RSpec.describe "catalog/show.json" do
   end
   let(:config) do
     Blacklight::Configuration.new do |config|
-      config.add_show_field 'title', label: 'Title', field: 'title_tsim'
+      config.show.title_field = 'title_tsim'
     end
   end
 
@@ -27,14 +27,7 @@ RSpec.describe "catalog/show.json" do
         id: '123',
         type: 'Book',
         attributes: {
-          'title' => {
-            id: 'http://test.host/catalog/123#title',
-            type: 'document_value',
-            attributes: {
-              value: 'Book1',
-              label: 'Title'
-            }
-          }
+          'title' => 'Book1'
         }
       })
   end

data/spec/views/catalog/sms_success.html.erb_spec.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 RSpec.describe "catalog/sms_success.html.erb" do
-  it "includes the data-blacklight-modal properties" do
+  it "includes updates to the main flash messages" do
     render
-    expect(rendered).to have_selector "div[data-blacklight-modal=container]"
+    expect(rendered).to have_selector 'turbo-stream[action="append"][target="main-flashes"]'
   end
 end

data/tasks/blacklight.rake

@@ -20,7 +20,7 @@ def system_with_error_handling(*args)
   end
 end
 
-def with_solr
+def with_solr(&block)
   # We're being invoked by the app entrypoint script and solr is already up via docker-compose
   if ENV['SOLR_ENV'] == 'docker-compose'
     yield
@@ -36,9 +36,7 @@ def with_solr
     end
   else
     SolrWrapper.wrap do |solr|
-      solr.with_collection do
-        yield
-      end
+      solr.with_collection(&block)
     end
   end
 end
@@ -48,6 +46,10 @@ desc "Run test suite"
 task :ci do
   with_solr do
     Rake::Task['blacklight:internal:seed'].invoke
+    within_test_app do
+      # Precompiles the javascript
+      system "bin/rake spec:prepare"
+    end
     Rake::Task['blacklight:coverage'].invoke
   end
 end