blacklight-spotlight 3.4.0 → 3.4.1

data/spec/features/javascript/locale_selector_spec.rb

@@ -1,66 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe 'Locale Selector', js: true do
-  let(:exhibit) { FactoryBot.create(:exhibit, published: true) }
-  let!(:language_es) { FactoryBot.create(:language, exhibit: exhibit, locale: 'es', public: true) }
-  let!(:language_zh) { FactoryBot.create(:language, exhibit: exhibit, locale: 'zh') }
-
-  before { login_as user }
-
-  before(:all) do
-    # mimics setting config.i18n.fallbacks = [I18n.default_locale] in the rails environment
-    I18n.fallbacks[:es] = [:es, I18n.default_locale]
-  end
-
-  context 'with an anonymous user' do
-    let(:user) { FactoryBot.create(:exhibit_visitor) }
-
-    it 'only sees public languages' do
-      visit spotlight.exhibit_path(exhibit)
-
-      expect(page).to have_css('li.dropdown', text: 'English')
-      click_link 'English'
-
-      within('.dropdown-menu', visible: true) do
-        expect(page).to have_css('li', count: 1)
-        expect(page).to have_css('li', text: 'Español')
-      end
-    end
-  end
-
-  context 'with an exhibit curator' do
-    let(:user) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-
-    it 'can see any saved languages' do
-      visit spotlight.exhibit_path(exhibit)
-
-      expect(page).to have_css('li.dropdown', text: 'English')
-      click_link 'English'
-
-      within('.dropdown-menu', visible: true) do
-        expect(page).to have_css('li', count: 2)
-        expect(page).to have_css('li', text: '中文')
-        expect(page).to have_css('li', text: 'Español')
-      end
-    end
-  end
-
-  describe 'switching locales' do
-    let(:user) { FactoryBot.create(:exhibit_visitor) }
-
-    it 'works' do
-      visit spotlight.exhibit_path(exhibit)
-
-      expect(page).to have_css('input[placeholder="Search..."]')
-
-      click_link 'English'
-
-      within('.dropdown-menu', visible: true) do
-        click_link 'Español'
-      end
-
-      expect(page).not_to have_css('input[placeholder="Search..."]')
-      expect(page).to have_css('input[placeholder="Buscar..."]')
-    end
-  end
-end

data/spec/features/javascript/metadata_admin_spec.rb

@@ -1,41 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Metadata Administration', js: true do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:admin) { FactoryBot.create(:exhibit_admin, exhibit: exhibit) }
-
-  before { login_as admin }
-
-  describe 'Select/Deselect all button' do
-    it 'deselects all checkboxes when all are selected' do
-      visit spotlight.edit_exhibit_metadata_configuration_path exhibit
-      # No checkboxes should be unchecked
-      expect(page).not_to have_css("tr td:nth-child(2) input[type='checkbox']:not(:checked)")
-      within('tr th:nth-child(2)') do
-        click_button 'Deselect all'
-        expect(page).to have_css('button', text: 'Select all', visible: true)
-      end
-      # No checkboxes should be checked
-      expect(page).not_to have_css("tr td:nth-child(2) input[type='checkbox']:checked")
-    end
-
-    it 'selects all checkboxes when any are unselected' do
-      visit spotlight.edit_exhibit_metadata_configuration_path exhibit
-      # No checkboxes should be unchecked
-      expect(page).not_to have_css("tr td:nth-child(2) input[type='checkbox']:not(:checked)")
-      first_button_area = find('tr th:nth-child(2)')
-      within first_button_area do
-        expect(page).to have_css('button', text: 'Deselect all')
-      end
-      # Uncheck first checkbox
-      find("tr:first-child td:nth-child(2) input[type='checkbox']").set(false)
-      # A checkbox should be checked
-      expect(page).to have_css("tr td:nth-child(2) input[type='checkbox']:checked")
-      within first_button_area do
-        click_button 'Select all'
-      end
-      # No checkboxes should be unchecked
-      expect(page).not_to have_css("tr td:nth-child(2) input[type='checkbox']:not(:checked)")
-    end
-  end
-end

data/spec/features/javascript/multi_image_select_spec.rb

@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Multi image selector', type: :feature, js: true, versioning: true, default_max_wait_time: 5 do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:exhibit_curator) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-  let(:feature_page) { FactoryBot.create(:feature_page, exhibit: exhibit) }
-
-  before { login_as exhibit_curator }
-
-  it 'allows the user to select which image in a multi image object to display' do
-    visit spotlight.edit_exhibit_feature_page_path(exhibit, feature_page)
-
-    add_widget 'solr_documents' # the "Item Row" widget
-
-    fill_in_typeahead_field with: 'xd327cm9378'
-
-    expect(page).to have_selector '.card'
-
-    within('.card') do
-      expect(page).to have_content(/Image \d of \d/)
-      expect(page).to have_link 'Change'
-    end
-
-    save_page
-
-    visit spotlight.exhibit_feature_page_path(exhibit, feature_page)
-    expect(page).to have_css("[data-id='xd327cm9378']")
-    expect(page).to have_css("img[src='https://stacks.stanford.edu/image/iiif/xd327cm9378%2Fxd327cm9378_05_0001/full/!400,400/0/default.jpg']")
-    expect(page).not_to have_css("img[src='https://stacks.stanford.edu/image/iiif/xd327cm9378%2Fxd327cm9378_05_0002/full/!400,400/0/default.jpg']")
-
-    click_link('Edit')
-
-    within('.card') do
-      expect(page).to have_content(/Image \d of \d/)
-      find('a', text: 'Change').click
-    end
-
-    expect(page).to have_css('.thumbs-list ul', visible: true)
-
-    within('.thumbs-list ul') do
-      all('li')[1].click
-    end
-
-    save_page
-
-    expect(page).to have_css("[data-id='xd327cm9378']")
-    expect(page).not_to have_css("img[src='https://stacks.stanford.edu/image/iiif/xd327cm9378%2Fxd327cm9378_05_0001/full/!400,400/0/default.jpg']")
-    expect(page).to have_css("img[src='https://stacks.stanford.edu/image/iiif/xd327cm9378%2Fxd327cm9378_05_0002/full/!400,400/0/default.jpg']")
-  end
-end

data/spec/features/javascript/reindex_monitor_spec.rb

@@ -1,23 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Reindex Monitor', js: true, default_max_wait_time: 10 do
-  let(:resources) do
-    [FactoryBot.create(:resource)]
-  end
-  let(:exhibit) { FactoryBot.create(:exhibit, resources: resources) }
-  let(:exhibit_curator) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-
-  before do
-    exhibit.job_trackers.create(resource: exhibit, status: 'completed', job_class: 'Spotlight::ReindexExhibitJob')
-    login_as exhibit_curator
-    visit spotlight.admin_exhibit_catalog_path(exhibit)
-  end
-
-  it 'is rendered on the item admin page' do
-    expect(page).to have_css('.card.index-status', visible: true)
-    within('.card.index-status') do
-      expect(page).to have_css('p', text: /Began reindexing a total of \d+ items/)
-      expect(page).to have_css('p', text: /Reindexed \d+ of \d+ items/)
-    end
-  end
-end

data/spec/features/javascript/roles_admin_spec.rb

@@ -1,38 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Roles Admin', type: :feature, js: true do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:exhibit_admin) { FactoryBot.create(:exhibit_admin, exhibit: exhibit) }
-
-  before do
-    login_as exhibit_admin
-
-    visit spotlight.exhibit_dashboard_path(exhibit)
-    click_link 'Users'
-  end
-
-  it 'has the appropriate status message when an existing user is added' do
-    second_user = FactoryBot.create(:site_admin)
-
-    click_link 'Add a new user'
-    fill_in 'User key', with: second_user.email
-
-    click_button 'Save changes'
-
-    expect(page).to have_css('.alert-info', text: 'User has been updated.')
-    expect(page).to have_css('.table.users td', text: second_user.email)
-  end
-
-  it 'persists invited users to the exhibits user list' do
-    expect(page).not_to have_css('.badge-warning pending-label', text: 'pending', visible: true)
-
-    click_link 'Add a new user'
-    fill_in 'User key', with: 'user@example.com'
-    click_button 'Save changes'
-
-    within('tr.invite-pending') do
-      expect(page).to have_css('td', text: 'user@example.com')
-      expect(page).to have_css('.badge-warning.pending-label', text: 'pending', visible: true)
-    end
-  end
-end

data/spec/features/javascript/search_config_admin_spec.rb

@@ -1,140 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Search Configuration Administration', js: true do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:user) { FactoryBot.create(:exhibit_admin, exhibit: exhibit) }
-
-  before { login_as user }
-
-  describe 'search fields', default_max_wait_time: 5 do
-    it 'allows the curator to disable all search fields' do
-      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
-      expect(page).to have_css 'select#search_field'
-
-      visit spotlight.edit_exhibit_search_configuration_path(exhibit)
-      expect(page).to have_css 'input#enable_feature', visible: true
-      uncheck 'Display search box'
-
-      click_button 'Save changes'
-
-      expect(page).to have_css('.alert', text: 'The exhibit was successfully updated.', visible: true)
-
-      expect(page).not_to have_css 'select#search_field'
-    end
-
-    it 'allows the curator to update search field options' do
-      visit spotlight.edit_exhibit_search_configuration_path(exhibit)
-
-      click_link 'Options'
-
-      within('#nested-search-fields') do
-        expect(page).to have_css("#blacklight_configuration_search_fields_title_label[type='hidden']", visible: false)
-        expect(page).not_to have_css("#blacklight_configuration_search_fields_title_label[type='text']")
-        click_link('Title')
-        expect(page).not_to have_css("#blacklight_configuration_search_fields_title_label[type='hidden']")
-        expect(page).to have_css("#blacklight_configuration_search_fields_title_label[type='text']")
-        fill_in 'blacklight_configuration_search_fields_title_label', with: 'My Title Label'
-      end
-
-      click_button 'Save changes'
-
-      expect(page).to have_css('.alert', text: 'The exhibit was successfully updated.', visible: true)
-      expect(page).to have_select 'Search in', with_options: ['My Title Label']
-    end
-  end
-
-  describe 'facets' do
-    it 'allows us to update the label with edit-in-place' do
-      input_id = 'blacklight_configuration_facet_fields_genre_ssim_label'
-      visit spotlight.edit_exhibit_search_configuration_path(exhibit)
-
-      click_link 'Facets'
-
-      facet = find('.edit-in-place', text: 'Genre')
-      expect(page).not_to have_content('Topic')
-      expect(page).to have_css("input##{input_id}", visible: false)
-
-      facet.click
-
-      expect(page).to have_css("input##{input_id}", visible: true)
-
-      fill_in(input_id, with: 'Topic')
-
-      click_button 'Save changes'
-      click_link 'Facets'
-
-      expect(page).to have_css('.alert', text: 'The exhibit was successfully updated.', visible: true)
-
-      expect(page).not_to have_content('Genre')
-      expect(page).to have_content('Topic')
-    end
-
-    it 'allows the curator to select a different facet sort order' do
-      visit spotlight.edit_exhibit_search_configuration_path(exhibit)
-      click_link 'Facets'
-
-      within '.facet-config-genre_ssim' do
-        click_button 'Options'
-        expect(find(:css, '#blacklight_configuration_facet_fields_genre_ssim_sort_count')).to be_checked
-
-        choose 'Value'
-      end
-
-      click_button 'Save changes'
-
-      expect(page).to have_css('.alert', text: 'The exhibit was successfully updated.', visible: true)
-
-      exhibit.reload
-      expect(exhibit.blacklight_config.facet_fields['genre_ssim'].sort).to eq 'index'
-    end
-  end
-
-  describe 'results' do
-    it 'updates search result options' do
-      visit spotlight.edit_exhibit_search_configuration_path(exhibit)
-
-      click_link 'Results'
-
-      uncheck 'List'
-
-      choose '20'
-
-      click_button 'Save changes'
-
-      expect(page).to have_css('.alert', text: 'The exhibit was successfully updated.', visible: true)
-
-      click_link 'Results'
-
-      expect(page).to have_unchecked_field 'List'
-      expect(page).to have_checked_field 'Gallery'
-
-      expect(page).to have_checked_field '20'
-      expect(page).to have_unchecked_field '10'
-    end
-
-    it 'updates Sort field result options' do
-      visit spotlight.edit_exhibit_search_configuration_path(exhibit)
-
-      click_link 'Results'
-
-      within('#nested-sort-fields') do
-        expect(page).to have_css("#blacklight_configuration_sort_fields_title_label[type='hidden']", visible: false)
-        expect(page).not_to have_css("#blacklight_configuration_sort_fields_title_label[type='text']")
-        click_link('title')
-        expect(page).not_to have_css("#blacklight_configuration_sort_fields_title_label[type='hidden']")
-        expect(page).to have_css("#blacklight_configuration_sort_fields_title_label[type='text']")
-        fill_in 'blacklight_configuration_sort_fields_title_label', with: 'My Title Label'
-      end
-
-      click_button 'Save changes'
-
-      expect(page).to have_css('.alert', text: 'The exhibit was successfully updated.', visible: true)
-
-      click_link 'Results'
-
-      within('#nested-sort-fields') do
-        expect(page).to have_css('h3', text: 'My Title Label')
-      end
-    end # Sort field
-  end # results tab
-end

data/spec/features/javascript/search_context_spec.rb

@@ -1,88 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Search contexts' do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:exhibit_curator) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-  let(:feature_page) do
-    FactoryBot.create(
-      :feature_page,
-      title: 'FeaturePage1',
-      exhibit: exhibit
-    )
-  end
-
-  before { login_as exhibit_curator }
-
-  it 'adds context breadcrumbs back to the home page when navigating to an item from the home page', js: true do
-    exhibit.home_page.content = [
-      {
-        type: 'solr_documents',
-        data: {
-          item: {
-            dq287tq6352: {
-              id: 'dq287tq6352',
-              display: 'true'
-            }
-          }
-        }
-      }
-    ].to_json
-    exhibit.home_page.save
-
-    visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
-
-    # verify that the item + image widget is displaying an image from the document.
-    within(:css, '.items-block') do
-      expect(page).to have_css('.img-thumbnail')
-      expect(page).not_to have_css('.title')
-    end
-
-    find('.items-block a').click
-
-    expect(page).to have_selector '.breadcrumb a', text: 'Home'
-  end
-
-  it 'adds context breadcrumb back to the feature page when navigating to an item from a feature page', js: true do
-    feature_page.content = [
-      {
-        type: 'solr_documents',
-        data: {
-          item: {
-            dq287tq6352: {
-              id: 'dq287tq6352',
-              display: 'true'
-            }
-          }
-        }
-      }
-    ].to_json
-    feature_page.save
-
-    visit spotlight.exhibit_feature_page_path(exhibit, feature_page)
-
-    # verify that the item + image widget is displaying an image from the document.
-    within(:css, '.items-block') do
-      expect(page).to have_css('.img-thumbnail')
-      expect(page).not_to have_css('.title')
-    end
-
-    find('.items-block a').click
-
-    expect(page).to have_selector '.breadcrumb a', text: 'Home'
-    expect(page).to have_link 'FeaturePage1', href: spotlight.exhibit_feature_page_path(exhibit, feature_page)
-  end
-
-  context 'from a browse page' do
-    let!(:search) { FactoryBot.create(:search, title: 'Some Saved Search', exhibit: exhibit, published: true) }
-
-    it 'adds context breadcrumbs back to the browse page when navigating to an item', js: true do
-      visit spotlight.exhibit_home_page_path(exhibit, exhibit.home_page)
-      click_link 'Browse'
-      click_link 'Some Saved Search'
-      click_link 'A MAP of AMERICA from the latest and best Observations'
-      expect(page).to have_link 'Home'
-      expect(page).to have_link 'Browse'
-      expect(page).to have_link 'Some Saved Search', href: spotlight.exhibit_browse_path(exhibit, search)
-    end
-  end
-end

data/spec/features/main_navigation_spec.rb

@@ -1,96 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Main navigation labels are settable', type: :feature do
-  let!(:exhibit) { FactoryBot.create(:exhibit) }
-  let!(:about) { FactoryBot.create(:about_page, exhibit: exhibit, published: true) }
-
-  before do
-    about_nav = exhibit.main_navigations.about
-    about_nav.label = 'New About Label'
-    about_nav.save
-    browse_nav = exhibit.main_navigations.browse
-    browse_nav.label = 'New Browse Label'
-    browse_nav.save
-    search = exhibit.searches.first
-    search.published = true
-    search.save
-    exhibit.reload
-  end
-
-  it 'has the configured about and browse navigation labels' do
-    visit spotlight.exhibit_path(exhibit)
-    expect(page).to have_css('.navbar-nav li', text: 'New About Label')
-    expect(page).to have_css('.navbar-nav li', text: 'New Browse Label')
-  end
-
-  it 'has the configured about page label in the sidebar' do
-    visit spotlight.exhibit_about_page_path(exhibit, about)
-    expect(page).to have_css('#sidebar h2', text: 'New About Label')
-  end
-
-  it 'has the configured about page label visible in the breadcrumb' do
-    visit spotlight.exhibit_about_page_path(exhibit, about)
-    expect(page).to have_css('.breadcrumb li', text: 'New About Label')
-  end
-
-  it 'has the configured browse page label visible in the breadcrumb of the browse index page' do
-    visit spotlight.exhibit_browse_index_path(exhibit, exhibit.searches.first)
-    expect(page).to have_content('New Browse Label')
-    expect(page).to have_css('.breadcrumb li', text: 'New Browse Label')
-  end
-
-  it 'has the configured browse page label visible in the breadcrumb of the browse show page' do
-    visit spotlight.exhibit_browse_path(exhibit, exhibit.searches.first)
-    expect(page).to have_content('New Browse Label')
-    expect(page).to have_css('.breadcrumb li', text: 'New Browse Label')
-  end
-
-  it 'does 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).not_to have_css('.navbar-nav li', text: 'New About Label')
-    about_nav = exhibit.main_navigations.about
-    about_nav.display = true
-    about_nav.save
-  end
-
-  describe 'Restore default button functionality', js: true do
-    let(:user) { FactoryBot.create(:exhibit_admin, exhibit: exhibit) }
-
-    before { login_as user }
-
-    it 'is present when the navigation label is not the default value' do
-      visit spotlight.edit_exhibit_appearance_path(exhibit)
-
-      click_link 'Main menu'
-
-      within '.main_navigation_admin' do
-        within all('li').first do
-          expect(page).not_to have_css('button.restore-default', visible: true)
-        end
-
-        within all('li').last do
-          expect(page).to have_css('button.restore-default', visible: true)
-        end
-      end
-    end
-
-    context 'when the navigation label is not the default value' do
-      it 'restores the default value' do
-        visit spotlight.edit_exhibit_appearance_path(exhibit)
-
-        click_link 'Main menu'
-
-        within '.main_navigation_admin' do
-          within all('li').last do
-            expect(page).to have_css('a', text: 'New About Label')
-            click_button 'Restore default'
-            expect(page).to have_css('a', text: 'About')
-          end
-        end
-      end
-    end
-  end
-end

data/spec/features/metadata_admin_spec.rb

@@ -1,18 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Metadata Administration', type: :feature do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:user) { FactoryBot.create(:exhibit_admin, exhibit: exhibit) }
-
-  before { login_as user }
-
-  describe 'edit' do
-    it 'displays the metadata edit page' do
-      visit spotlight.edit_exhibit_metadata_configuration_path(exhibit)
-      expect(page).to have_css('h1 small', text: 'Metadata')
-      within("[data-id='language_ssm']") do
-        expect(page).to have_css('td', text: 'Language')
-      end
-    end
-  end
-end

data/spec/features/report_a_problem_spec.rb

@@ -1,62 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Report a Problem', type: :feature do
-  let!(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:honeypot_field_name) { Spotlight::Engine.config.spambot_honeypot_email_field }
-
-  it 'does not have a header link' do
-    visit root_path
-    expect(page).not_to have_content 'Feedback'
-  end
-
-  describe 'when emails are setup' do
-    before do
-      exhibit.contact_emails_attributes = [{ 'email' => 'test@example.com' }, { 'email' => 'test2@example.com' }]
-      exhibit.save!
-      exhibit.contact_emails.first.tap do |e|
-        if e.respond_to? :confirm
-          e.confirm
-        else
-          e.confirm!
-        end
-      end
-    end
-
-    it 'allows the link to be opened w/o javascript (or in a new tab/window)' do
-      visit spotlight.exhibit_solr_document_path(exhibit, id: 'dq287tq6352')
-
-      click_on 'Feedback'
-
-      expect(page).to have_css('h2', text: 'Contact us', visible: true)
-      expect(page).to have_css('#contact_form_name', count: 1)
-    end
-
-    it 'accepts a problem report', js: true do
-      visit spotlight.exhibit_solr_document_path(exhibit, id: 'dq287tq6352')
-      click_on 'Feedback'
-      expect(page).to have_css('.alert-primary', text: '/dq287tq6352')
-      expect(find('#contact_form_current_url', visible: false).value).to end_with spotlight.exhibit_solr_document_path(exhibit, id: 'dq287tq6352')
-      fill_in 'Your name', with: 'Some Body'
-      fill_in 'Your email', with: 'test@example.com'
-      fill_in 'Message', with: 'This is my problem report'
-
-      expect do
-        click_on 'Send'
-      end.to change { ActionMailer::Base.deliveries.count }.by(1)
-    end
-
-    it 'rejects a spammy looking problem report', js: true do
-      visit spotlight.exhibit_solr_document_path(exhibit, id: 'dq287tq6352')
-      click_on 'Feedback'
-      expect(find('#contact_form_current_url', visible: false).value).to end_with spotlight.exhibit_solr_document_path(exhibit, id: 'dq287tq6352')
-      fill_in 'Your name', with: 'Some Body'
-      fill_in 'Your email', with: 'test@example.com'
-      page.evaluate_script("$('#contact_form_#{honeypot_field_name}').val('possible_spam@spam.com')")
-      fill_in 'Message', with: 'This is my problem report'
-
-      expect do
-        click_on 'Send'
-      end.not_to change { ActionMailer::Base.deliveries.count }
-    end
-  end
-end