blacklight-spotlight 3.4.0 → 3.4.1

data/spec/views/spotlight/searches/_search.html.erb_spec.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/searches/_search.html.erb', type: :view do
-  let(:search) do
-    FactoryBot.build_stubbed(:search, exhibit: FactoryBot.create(:exhibit),
-                                      id: 99,
-                                      title: 'Title1',
-                                      query_params: {
-                                        f: {
-                                          genre_ssim: ['xyz']
-                                        }
-                                      })
-  end
-  let(:thumbnail) { double(id: '7777', iiif_url: '/images/7777/full/400,300/0/default.jpg') }
-
-  before do
-    allow(view).to receive(:edit_search_path).and_return('/edit')
-    allow(view).to receive(:search_path).and_return('/search')
-    allow(search).to receive(:thumbnail).and_return(thumbnail)
-    allow(search).to receive(:count).and_return(15)
-    allow(search).to receive(:params).and_return({})
-
-    form_for(search, url: '/update') do |f|
-      @f = f
-    end
-  end
-
-  it 'renders a list of pages' do
-    # rubocop:disable RSpec/InstanceVariable
-    render partial: 'spotlight/searches/search', locals: { f: @f }
-    # rubocop:enable RSpec/InstanceVariable
-    expect(rendered).to have_selector "li[data-id='99']"
-    expect(rendered).to have_selector '.card-title', text: 'Title1'
-    expect(rendered).to have_selector 'img[src="/images/7777/full/400,300/0/default.jpg"]'
-    expect(rendered).to have_selector 'input[type=hidden][data-property=weight]', visible: false
-  end
-end

data/spec/views/spotlight/searches/edit.html.erb_spec.rb

@@ -1,40 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/searches/edit.html.erb', type: :view do
-  let(:blacklight_config) do
-    Blacklight::Configuration.new do |config|
-      config.add_facet_field :some_field
-    end
-  end
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:search) do
-    stub_model(Spotlight::Search, exhibit: exhibit,
-                                  id: 99, title: 'Title1', query_params: { f: { 'some_field' => ['xyz'] } })
-  end
-
-  before do
-    allow(view).to receive(:search_action_path).and_return('/search')
-    allow(view).to receive(:mastheads_path).and_return('/mastheads')
-    allow(view).to receive(:featured_images_path).and_return('/featured_images')
-    allow(view).to receive(:exhibit_search_path).and_return('/search')
-    allow(view).to receive(:exhibit_searches_path).and_return('/searches')
-    allow(view).to receive(:blacklight_config).and_return(blacklight_config)
-    allow(controller).to receive(:search_state_class).and_return(Blacklight::SearchState)
-
-    assign(:exhibit, exhibit)
-    assign(:search, search)
-    allow(view).to receive(:current_exhibit).and_return(exhibit)
-  end
-
-  it 'renders a form w/ the appropriate autocomplete data attribute' do
-    render
-    expect(rendered).to have_selector 'form[data-autocomplete-exhibit-catalog-path]'
-  end
-
-  it 'renders active search constraints' do
-    render
-    expect(rendered).to have_selector '.appliedParams .constraint-value'
-    expect(rendered).to have_selector '.appliedParams .constraint-value .filter-name', text: 'Some Field'
-    expect(rendered).to have_selector '.appliedParams .constraint-value .filter-value', text: 'xyz'
-  end
-end

data/spec/views/spotlight/searches/index.html.erb_spec.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/searches/index.html.erb', type: :view do
-  let(:exhibit) { stub_model(Spotlight::Exhibit) }
-
-  before do
-    allow(view).to receive_messages(update_all_exhibit_searches_path: '/')
-    allow(view).to receive(:current_exhibit).and_return(exhibit)
-    assign(:exhibit, exhibit)
-  end
-
-  describe 'Without searches' do
-    it 'disables the update button' do
-      assign(:searches, [])
-      expect(exhibit).to receive(:searchable?).and_return(true)
-      render
-      expect(rendered).to have_content 'You can save search results'
-    end
-  end
-
-  describe 'When the exhibit is not searchable' do
-    it 'displays a warning' do
-      assign(:searches, [])
-      expect(exhibit).to receive(:searchable?).and_return(false)
-      render
-      expect(rendered).to have_css '.alert-warning', text: %(\
-This exhibit is not currently searchable. To perform searches that can \
-be saved as additional browse categories, \
-temporarily turn on the Display search box option in the Options section \
-of the Configuration > Search page.)
-    end
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_browse_block.html.erb_spec.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_browse_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/browse_block' }
-  let(:page) { double('Page', display_sidebar?: true) }
-  let(:search) { FactoryBot.create(:search) }
-  let(:block) do
-    assign(:page, page)
-    SirTrevorRails::Blocks::BrowseBlock.new({ type: 'block', data: {} }, page)
-  end
-
-  before do
-    allow(block).to receive(:searches).and_return([search])
-  end
-
-  it 'links to the search' do
-    render partial: p, locals: { browse_block: block }
-    expect(rendered).to have_link search.title, href: spotlight.exhibit_browse_path(search.exhibit, search)
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_iframe_block.html.erb_spec.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_iframe_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/iframe_block' }
-  let(:block) do
-    OpenStruct.new
-  end
-
-  it 'renders iframes' do
-    block.code = "<iframe src='xyz'></iframe>"
-    render partial: p, locals: { iframe_block: block }
-    expect(rendered).to have_selector 'iframe[src="xyz"]'
-  end
-
-  it 'strips extra markup from the code' do
-    block.code = '<a><b></b></a>'
-    render partial: p, locals: { iframe_block: block }
-    expect(rendered).to be_blank
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_link_to_search_block.html.erb_spec.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_link_to_search_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/link_to_search_block' }
-  let(:page) { double('Page', display_sidebar?: true) }
-  let(:search) { FactoryBot.create(:search, query_params: { a: 1 }) }
-  let(:block) do
-    assign(:page, page)
-    SirTrevorRails::Blocks::LinkToSearchBlock.new({ type: 'block', data: {} }, page)
-  end
-
-  before do
-    allow(block).to receive(:searches).and_return([search])
-  end
-
-  it 'links to the search' do
-    render partial: p, locals: { link_to_search_block: block }
-    expect(rendered).to have_link search.title, href: spotlight.search_exhibit_catalog_path(search.exhibit, a: 1)
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_rule_block.html.erb_spec.rb

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_rule_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/rule_block' }
-
-  it 'has an hr' do
-    render partial: p
-    expect(rendered).to have_selector 'hr'
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_block.html.erb_spec.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_solr_documents_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/solr_documents_block' }
-  let(:page) { double('Page') }
-  let(:block) do
-    SirTrevorRails::Blocks::SolrDocumentsBlock.new({ type: 'block', data: { title: 'Some title', text: 'Some text', 'text-align' => 'right' } }, page)
-  end
-  let(:doc) { blacklight_config.document_model.new(id: 1) }
-  let(:blacklight_config) do
-    Blacklight::Configuration.new do |config|
-      config.view.embed(partials: %w[a b c], locals: { a: 1 })
-    end
-  end
-  let(:stub_presenter) do
-    instance_double(Blacklight::DocumentPresenter, heading: 'blah', thumbnail: thumbnail_presenter)
-  end
-
-  let(:thumbnail_presenter) { instance_double(Blacklight::ThumbnailPresenter, exists?: true, thumbnail_tag: 'thumb') }
-
-  before do
-    allow(block).to receive(:each_document).and_yield({}, doc)
-    allow(block).to receive(:documents?).and_return(true)
-  end
-
-  before do
-    allow(view).to receive(:blacklight_config).and_return(blacklight_config)
-    allow(view).to receive_messages(document_presenter: stub_presenter, document_link_params: {})
-  end
-
-  context 'with a multi-image object' do
-    let(:block_options) { { thumbnail_image_url: 'http://example.com' } }
-
-    before do
-      allow(block).to receive(:each_document).and_yield(block_options, doc)
-    end
-
-    it 'uses the provided thumbnail url' do
-      render partial: p, locals: { solr_documents_block: block }
-      expect(rendered).to have_selector 'img[src="http://example.com"]'
-    end
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_carousel_block.html.erb_spec.rb

@@ -1,34 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_solr_documents_carousel_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/solr_documents_carousel_block' }
-  let(:block) do
-    SirTrevorRails::Blocks::SolrDocumentsCarouselBlock.new({ type: 'block', data: {} }, page)
-  end
-  let(:stub_presenter) do
-    instance_double(Blacklight::DocumentPresenter, heading: 'blah', thumbnail: thumbnail_presenter)
-  end
-
-  let(:thumbnail_presenter) { instance_double(Blacklight::ThumbnailPresenter, exists?: true, thumbnail_tag: 'thumb') }
-
-  before do
-    allow(block).to receive(:each_document).and_return([
-                                                         [{}, SolrDocument.new(id: 1)],
-                                                         [{}, SolrDocument.new(id: 2)],
-                                                         [{}, SolrDocument.new(id: 3)]
-                                                       ])
-    allow(block).to receive_messages(documents?: true)
-    allow(view).to receive_messages(solr_documents_carousel_block: block)
-    allow(view).to receive_messages(document_presenter: stub_presenter, blacklight_config: Blacklight::Configuration.new)
-  end
-
-  it 'has a slideshow block' do
-    render partial: p, locals: { item_carousel_block: block }
-    expect(rendered).to have_selector '.carousel-block'
-    expect(rendered).to have_selector '.carousel-control-prev.left'
-    expect(rendered).to have_selector '.carousel-control-next.right'
-    expect(rendered).to have_selector '.carousel-item', text: 'thumb', count: 3
-    expect(rendered).to have_selector '.carousel-indicators'
-    expect(rendered).to have_selector '.carousel-indicators li', count: 3
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_embed_block.html.erb_spec.rb

@@ -1,53 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_solr_documents_embed_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/solr_documents_embed_block' }
-  let(:page) { double('Page') }
-  let(:block) do
-    SirTrevorRails::Blocks::SolrDocumentsEmbedBlock.new({ type: 'block', data: { title: 'Some title', text: 'Some text', 'text-align' => 'right' } }, page)
-  end
-  let(:doc) { blacklight_config.document_model.new(id: 1) }
-  let(:blacklight_config) do
-    Blacklight::Configuration.new do |config|
-      config.view.embed(partials: %w[a b c], locals: { a: 1 })
-    end
-  end
-  let(:stub_presenter) do
-    instance_double(Blacklight::DocumentPresenter, heading: 'blah', thumbnail: thumbnail_presenter)
-  end
-
-  let(:thumbnail_presenter) { instance_double(Blacklight::ThumbnailPresenter, exists?: true, thumbnail_tag: 'thumb') }
-
-  before do
-    allow(block).to receive(:each_document).and_yield({}, doc)
-    allow(block).to receive(:documents?).and_return(true)
-  end
-
-  before do
-    allow(view).to receive_messages(blacklight_config: blacklight_config, document_presenter: stub_presenter)
-  end
-
-  it 'has a embed block' do
-    expect(view).to receive(:render_document_partials).with(doc, %w[a b c], hash_including(a: 1, block: block)).and_return('OSD')
-    render partial: p, locals: { solr_documents_embed_block: block }
-    expect(rendered).to have_selector 'h3', text: 'Some title'
-    expect(rendered).to have_content 'Some text'
-    expect(rendered).to have_selector '.box', text: 'OSD'
-    expect(rendered).to have_selector '.items-col'
-    expect(rendered).to have_selector '.text-col'
-    expect(rendered).not_to have_selector '.col-md-12'
-  end
-
-  context 'with a block with no text' do
-    let(:block) do
-      SirTrevorRails::Blocks::SolrDocumentsEmbedBlock.new({ type: 'block', data: { title: 'Some title', 'text-align' => 'right' } }, page)
-    end
-
-    it 'does not have a two column layout' do
-      expect(view).to receive(:render_document_partials).with(doc, %w[a b c], hash_including(a: 1, block: block)).and_return('OSD')
-      render partial: p, locals: { solr_documents_embed_block: block }
-      expect(rendered).to have_selector '.col-md-12'
-      expect(rendered).to have_selector '.items-col h3', text: 'Some title'
-    end
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_features_block.html.erb_spec.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_solr_documents_features_block.html.erb', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/solr_documents_features_block' }
-  let(:block) do
-    SirTrevorRails::Blocks::SolrDocumentsFeaturesBlock.new({ type: 'block', data: { 'show-primary-caption' => true, 'primary-caption-field' => 'x' } }, page)
-  end
-
-  let(:blacklight_config) do
-    Blacklight::Configuration.new do |config|
-      config.index.title_field = 'full_title_tesim'
-    end
-  end
-
-  before do
-    doc1 = [{}, SolrDocument.new(id: 1, x: 'a' * 100)]
-    doc2 = [{}, SolrDocument.new(id: 2, full_title_tesim: 'full_title')]
-    doc3 = [{}, SolrDocument.new(id: 3)]
-    allow(block).to receive(:each_document).and_return([doc1, doc2, doc3])
-    allow(block).to receive_messages(documents?: true)
-    allow(view).to receive_messages(solr_documents_features_block: block)
-    allow(view).to receive_messages(blacklight_config: blacklight_config)
-    allow_any_instance_of(Blacklight::ThumbnailPresenter).to receive_messages(exists?: true, thumbnail_tag: 'thumb')
-  end
-
-  it 'has a slideshow block' do
-    render partial: p, locals: { item_carousel_block: block }
-    expect(rendered).to have_selector '.item-features'
-    expect(rendered).to have_selector '.carousel-item', text: 'thumb', count: 3
-    expect(rendered).to have_selector '.carousel-indicators'
-    expect(rendered).to have_selector '.carousel-indicators li', count: 3
-  end
-
-  it 'truncates long titles' do
-    render partial: p, locals: { item_carousel_block: block }
-    expect(rendered).to have_selector '.item-features'
-    expect(rendered).to have_selector '.carousel-item', text: 'thumb', count: 3
-    expect(rendered).to have_selector '.carousel-indicators'
-    expect(rendered).to have_selector '.carousel-indicators li', count: 3
-    expect(rendered).to have_selector '.carousel-indicators li', text: 'a' * 92 + '...'
-  end
-
-  describe 'without a primary caption' do
-    let(:block) do
-      SirTrevorRails::Blocks::SolrDocumentsFeaturesBlock.new({ type: 'block', data: { 'show-primary-caption' => false } }, page)
-    end
-
-    it 'falls back to the regular document title for the caption' do
-      render partial: p, locals: { item_carousel_block: block }
-      expect(rendered).to have_selector '.item-features'
-      expect(rendered).to have_selector '.carousel-indicators li', text: 'full_title'
-    end
-  end
-end

data/spec/views/spotlight/sir_trevor/blocks/_solr_documents_grid_block.html.erb_spec.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sir_trevor/blocks/_solr_documents_grid_block', type: :view do
-  let(:p) { 'spotlight/sir_trevor/blocks/solr_documents_grid_block' }
-  let(:page) { double('Page') }
-  let(:block) do
-    SirTrevorRails::Blocks::SolrDocumentsGridBlock.new({ type: 'block', data: { title: 'Some title', text: 'Some text', 'text-align' => 'right' } }, page)
-  end
-  let(:blacklight_config) do
-    Blacklight::Configuration.new
-  end
-
-  let(:stub_presenter) do
-    instance_double(Blacklight::DocumentPresenter, heading: 'blah', thumbnail: thumbnail_presenter)
-  end
-
-  let(:thumbnail_presenter) { instance_double(Blacklight::ThumbnailPresenter, exists?: true, thumbnail_tag: 'thumb') }
-
-  before do
-    allow(block).to receive(:each_document).and_return([
-                                                         [{}, SolrDocument.new(id: 1)],
-                                                         [{}, SolrDocument.new(id: 2)],
-                                                         [{}, SolrDocument.new(id: 3)]
-                                                       ])
-  end
-
-  before do
-    allow(view).to receive_messages(
-      blacklight_config: blacklight_config,
-      document_presenter: stub_presenter
-    )
-  end
-
-  it 'has a slideshow block' do
-    render partial: p, locals: { solr_documents_grid_block: block }
-    expect(rendered).to have_selector 'h3', text: 'Some title'
-    expect(rendered).to have_content 'Some text'
-    expect(rendered).to have_selector '.box', text: 'thumb', count: 3
-    expect(rendered).to have_selector '.items-col'
-    expect(rendered).to have_selector '.text-col'
-  end
-end

data/spec/views/spotlight/sites/edit_exhibits.html.erb_spec.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/sites/edit_exhibits', type: :view do
-  let!(:exhibit_a) { FactoryBot.create(:exhibit) }
-  let!(:exhibit_b) { FactoryBot.create(:exhibit) }
-
-  before do
-    assign(:site, Spotlight::Site.instance)
-    allow(view).to receive_messages(exhibit_path: nil)
-  end
-
-  it 'has columns for the exhibit data' do
-    render
-    expect(rendered).to have_selector 'th', text: 'Title'
-    expect(rendered).to have_selector 'th', text: 'Published?'
-    expect(rendered).to have_selector 'th', text: 'Requested by'
-    expect(rendered).to have_selector 'th', text: 'Created at'
-    expect(rendered).to have_selector 'th', text: 'Updated at'
-  end
-
-  it 'has draggable rows for each exhibit' do
-    render
-    expect(rendered).to have_selector 'tr .dd-handle', count: 2
-  end
-end

data/spec/views/spotlight/tags/index.html.erb_spec.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/tags/index.html.erb', type: :view do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let!(:tag1) { FactoryBot.create(:tagging, tagger: exhibit, taggable: exhibit) }
-  let!(:tag2) { FactoryBot.create(:tagging, tagger: exhibit, taggable: exhibit) }
-
-  before do
-    assign(:exhibit, exhibit)
-    assign(:tags, exhibit.owned_tags)
-    allow(view).to receive_messages(exhibit_tag_path: '/tags')
-    allow(view).to receive(:current_exhibit).and_return(exhibit)
-    allow(view).to receive(:url_to_tag_facet, &:first)
-    allow(view).to receive_messages(update_all_exhibit_tags_path: '/update_all')
-    allow(view).to receive(:exhibit_path)
-  end
-
-  describe 'Tags' do
-    it 'is displayed' do
-      render
-      [tag1.tag.name, tag2.tag.name].each do |name|
-        expect(rendered).to have_css('h4', text: name)
-        expect(rendered).to have_link(name, href: '#edit-in-place')
-      end
-    end
-  end
-end

data/spec/views/spotlight/translations/_import.html.erb_spec.rb

@@ -1,24 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/translations/_import.html.erb', type: :view do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-
-  before do
-    allow(view).to receive(:can?).and_return(true)
-    allow(view).to receive(:current_exhibit).and_return(exhibit)
-    assign(:language, :es)
-    I18n.default_locale = 'en'
-  end
-
-  it 'has a link to export the translation data' do
-    render
-    expect(rendered).to have_link 'Export English', href: spotlight.exhibit_translations_path(exhibit_id: exhibit, format: 'yaml', locale: 'en')
-    expect(rendered).to have_link 'Export Spanish', href: spotlight.exhibit_translations_path(exhibit_id: exhibit, format: 'yaml', locale: 'es')
-  end
-
-  it 'has a form to import the translation data' do
-    render
-    expect(rendered).to have_selector "form[action='#{spotlight.import_exhibit_translations_path(exhibit_id: exhibit)}']"
-    expect(rendered).to have_selector 'input[name="file"]'
-  end
-end

data/spec/views/spotlight/translations/_page.html.erb_spec.rb

@@ -1,76 +0,0 @@
-# frozen_string_literal: true
-
-describe 'spotlight/translations/_page.html.erb', type: :view do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:page) { FactoryBot.create(:feature_page, exhibit: exhibit) }
-
-  before do
-    allow(view).to receive_messages(
-      clone_exhibit_feature_page_path: '/',
-      clone_exhibit_home_page_path: '/',
-      current_exhibit: exhibit,
-      edit_exhibit_feature_page_path: '/',
-      edit_exhibit_home_page_path: '/',
-      exhibit_feature_page_path: '/',
-      f: instance_double('form', fields_for: {}), # mockform builder
-      page: page
-    )
-    assign(:language, 'es')
-  end
-
-  it 'includes a check icon when the page is published' do
-    page.published = true
-    render
-
-    expect(rendered).to have_css('.translation-complete')
-  end
-
-  context 'when there is a translated page' do
-    let!(:page_es) { FactoryBot.create(:feature_page, exhibit: exhibit, locale: 'es', default_locale_page: page) }
-
-    it 'links to the translated page' do
-      render
-      expect(rendered).to have_link(page_es.title)
-    end
-
-    it 'includes the data attribute used by the progress tracker' do
-      render
-      expect(rendered).to have_css('[data-translation-present="true"]')
-    end
-
-    context 'when the default locale page has been updated more recently than the translation' do
-      before { page_es.update(updated_at: 10.seconds.ago) }
-
-      it 'includes an alert icon' do
-        render
-
-        expect(rendered).to have_css('.translation-warning')
-      end
-    end
-
-    context 'when the page is a home page' do
-      let(:page) { exhibit.home_page }
-
-      before { page_es.update(type: 'Spotlight::HomePage') }
-
-      it 'does not render a delete link' do
-        render
-
-        expect(rendered).not_to have_link('Delete')
-      end
-    end
-  end
-
-  context 'when there is no translated page' do
-    it 'links to create a new one' do
-      render
-      expect(rendered).to have_content 'No translated page.'
-      expect(rendered).to have_link 'Create one now.'
-    end
-
-    it 'does not include the data attribute used by the progress tracker' do
-      render
-      expect(rendered).not_to have_css('[data-translation-present="true"]')
-    end
-  end
-end