blacklight-spotlight 3.4.0 → 3.4.1

data/spec/factories/bulk_updates.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :bulk_update, class: 'Spotlight::BulkUpdate' do
-    file { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'updated-bulk-update-template.csv'), __dir__)) }
-
-    exhibit
-  end
-
-  factory :bulk_update_no_cols, class: 'Spotlight::BulkUpdate' do
-    file { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'updated-bulk-update-template-no-cols.csv'), __dir__)) }
-
-    exhibit
-  end
-
-  factory :tagged_bulk_update, class: 'Spotlight::BulkUpdate' do
-    file { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'updated-bulk-update-template-w-tags.csv'), __dir__)) }
-
-    exhibit
-  end
-end

data/spec/factories/contact_emails.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :contact_email, class: 'Spotlight::ContactEmail' do
-    email { 'exhibit_contact@example.com' }
-    exhibit
-  end
-end

data/spec/factories/contact_images.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :contact_image, class: 'Spotlight::ContactImage' do
-    image { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'avatar.png'), __dir__)) }
-    iiif_tilesource { 'https://exhibits-stage.stanford.edu/images/78' }
-  end
-end

data/spec/factories/contacts.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :contact, class: 'Spotlight::Contact' do
-    exhibit
-
-    trait :with_avatar do
-      association :avatar, factory: :contact_image
-    end
-  end
-end

data/spec/factories/custom_fields.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :custom_field, class: 'Spotlight::CustomField' do
-    exhibit
-    field { 'field_name_tesim' }
-    configuration { { 'label' => 'Some Field' } }
-  end
-end

data/spec/factories/custom_search_fields.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :custom_search_field, class: 'Spotlight::CustomSearchField' do
-    exhibit
-    field { 'field_name_tesim^60' }
-    configuration { { 'label' => 'Some Field' } }
-  end
-end

data/spec/factories/exhibit_thumbnails.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :exhibit_thumbnail, class: 'Spotlight::ExhibitThumbnail' do
-    image { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'avatar.png'), __dir__)) }
-    iiif_tilesource { 'https://exhibits-stage.stanford.edu/images/78' }
-  end
-end

data/spec/factories/exhibits.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :exhibit, class: 'Spotlight::Exhibit' do
-    sequence(:title) { |n| "Exhibit Title #{n}" }
-    published { true }
-    after(:build) { |exhibit| exhibit.searches << FactoryBot.build(:default_search, exhibit: exhibit) }
-
-    trait :with_thumbnail do
-      association :thumbnail, factory: :exhibit_thumbnail
-    end
-  end
-  factory :skinny_exhibit, class: 'Spotlight::Exhibit' do
-    sequence(:title) { |n| "Exhibit Title #{n}" }
-    published { true }
-  end
-end

data/spec/factories/featured_images.rb

@@ -1,16 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :featured_image, class: 'Spotlight::FeaturedImage' do
-    image { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'avatar.png'), __dir__)) }
-    iiif_tilesource { 'https://exhibits-stage.stanford.edu/images/78' }
-  end
-
-  factory :temporary_image, class: 'Spotlight::TemporaryImage' do
-    image { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'avatar.png'), __dir__)) }
-  end
-
-  factory :masthead, class: 'Spotlight::Masthead' do
-    image { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'avatar.png'), __dir__)) }
-  end
-end

data/spec/factories/group.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :group, class: 'Spotlight::Group' do
-    exhibit
-
-    factory :group_with_searches do
-      transient do
-        searches_count { 5 }
-      end
-    end
-
-    after(:create) do |group, evaluator|
-      create_list(:search, evaluator.searches_count, groups: [group]) if evaluator.respond_to?(:searches_count)
-    end
-  end
-end

data/spec/factories/job_trackers.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :job_tracker, class: 'Spotlight::JobTracker' do
-    user
-    on factory: :exhibit
-    resource factory: :exhibit
-    created_at { Time.zone.parse('2017-01-05 23:00:00') }
-    updated_at { Time.zone.parse('2017-01-05 23:05:00') }
-  end
-end

data/spec/factories/language.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :language, class: 'Spotlight::Language' do
-    exhibit
-    locale { 'es' }
-  end
-end

data/spec/factories/main_navigation.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :main_navigation, class: 'Spotlight::MainNavigation' do
-    exhibit
-    nav_type { 'browse' }
-    display { true }
-  end
-end

data/spec/factories/pages.rb

@@ -1,34 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :feature_page, class: 'Spotlight::FeaturePage' do
-    exhibit
-    sequence(:title) { |n| "FeaturePage#{n}" }
-    published { true }
-    content { '[]' }
-  end
-  factory :feature_page_static_title, class: 'Spotlight::FeaturePage' do
-    exhibit
-    title { 'FeaturePage' }
-    published { true }
-    content { '[]' }
-  end
-  factory :feature_subpage, parent: :feature_page do
-    transient do
-      exhibit
-    end
-    title { 'SubPage1' }
-    content { '[]' }
-    after(:build) { |subpage, evaluator| subpage.parent_page = FactoryBot.create(:feature_page, exhibit: evaluator.exhibit) }
-  end
-  factory :about_page, class: 'Spotlight::AboutPage' do
-    exhibit
-    sequence(:title) { |n| "AboutPage#{n}" }
-    content { '[]' }
-    published { true }
-  end
-
-  factory :home_page, class: 'Spotlight::HomePage' do
-    exhibit
-  end
-end

data/spec/factories/resources.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :resource, class: 'Spotlight::Resource' do
-    exhibit
-    type { 'Spotlight::Resource' }
-    url { 'some url' }
-  end
-  factory :uploaded_resource, class: 'Spotlight::Resources::Upload', parent: :resource do
-    type { 'Spotlight::Resources::Upload' }
-    association :upload, factory: :featured_image
-    # url { Rack::Test::UploadedFile.new(File.expand_path(File.join('..', 'fixtures', 'avatar.png'), __dir__)) }
-  end
-end

data/spec/factories/roles.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :role, class: 'Spotlight::Role' do
-    resource { FactoryBot.build(:exhibit) }
-    role { 'curator' }
-    user
-  end
-end

data/spec/factories/searches.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :search, class: 'Spotlight::Search' do
-    exhibit
-    sequence(:title) { |n| "Exhibit Search #{n}" }
-    sequence(:slug) { |n| "Search#{n}" }
-
-    after(:build) { |search| search.thumbnail = FactoryBot.create(:featured_image) }
-
-    factory :search_with_groups do
-      transient do
-        groups_count { 2 }
-      end
-    end
-
-    after(:create) do |search, evaluator|
-      create_list(:group, evaluator.groups_count, searches: [search]) if evaluator.respond_to?(:groups_count)
-    end
-  end
-
-  factory :published_search, parent: :search do
-    published { true }
-  end
-
-  factory :default_search, class: 'Spotlight::Search' do
-    exhibit
-    title { 'All exhibit items' }
-    long_description { 'All items in this exhibit.' }
-
-    after(:build) { |search| search.thumbnail = FactoryBot.create(:featured_image) }
-  end
-end

data/spec/factories/tags.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :tag, class: 'ActsAsTaggableOn::Tag' do
-    sequence(:name) { |n| "tag#{n}" }
-  end
-
-  factory :tagging, class: 'ActsAsTaggableOn::Tagging' do
-    sequence(:tag) { |n| FactoryBot.create(:tag, name: "tagging#{n}") }
-    tagger { FactoryBot.create(:exhibit) }
-    taggable { FactoryBot.create(:exhibit) }
-    context { :tags }
-  end
-end

data/spec/factories/translation.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :translation do
-    locale { 'en' }
-    exhibit
-  end
-end

data/spec/factories/users.rb

@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-FactoryBot.define do
-  factory :user do
-    sequence(:email) { |n| "user#{n}@example.com" }
-    password { 'insecure' }
-    factory :site_admin do
-      after(:create) do |user, _evaluator|
-        user.roles.create role: 'admin', resource: Spotlight::Site.instance
-      end
-    end
-
-    factory :named_exhibit_roles do
-      transient do
-        exhibit { FactoryBot.create(:exhibit) }
-        role { nil }
-      end
-
-      after(:create) do |user, evaluator|
-        user.roles.create role: evaluator.role, resource: evaluator.exhibit if evaluator.role
-      end
-
-      factory :exhibit_admin do
-        transient do
-          role { 'admin' }
-        end
-      end
-
-      factory :exhibit_curator do
-        transient do
-          role { 'curator' }
-        end
-      end
-    end
-
-    trait :with_exhibit_role do
-      transient do
-        exhibit { FactoryBot.create(:exhibit) }
-        role { nil }
-      end
-
-      after(:create) do |user, evaluator|
-        user.roles.create role: evaluator.role, resource: evaluator.exhibit if evaluator.role
-      end
-    end
-
-    factory :exhibit_visitor do
-    end
-  end
-end

data/spec/features/about_page_spec.rb

@@ -1,46 +0,0 @@
-# frozen_string_literal: true
-
-describe 'About page', type: :feature, versioning: true do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:exhibit_curator) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-  let!(:about_page1) { FactoryBot.create(:about_page, title: 'First Page', exhibit: exhibit) }
-  let!(:about_page2) { FactoryBot.create(:about_page, title: 'Second Page', exhibit: exhibit) }
-  let(:unpublished_page) { FactoryBot.create(:about_page, title: 'Unpublished Page', published: false, exhibit: exhibit) }
-
-  describe 'sidebar' do
-    it 'displays' do
-      visit spotlight.exhibit_about_page_path(about_page1.exhibit, about_page1)
-      # the sidebar should display
-      within('#sidebar') do
-        # within the sidebar navigation
-        within('ul.sidenav') do
-          # the current page should be active
-          expect(page).to have_css('li.active', text: about_page1.title)
-          # the other page should be linked
-          expect(page).to have_css('li a', text: about_page2.title)
-        end
-      end
-    end
-  end
-
-  describe 'page options' do
-    before { login_as exhibit_curator }
-
-    describe 'publish' do
-      it 'is updatable from the edit page' do
-        expect(unpublished_page).not_to be_published
-
-        visit spotlight.edit_exhibit_about_page_path(unpublished_page.exhibit, unpublished_page)
-        expect(find('#about_page_published')).not_to be_checked
-
-        check 'Publish'
-        click_button 'Save changes'
-
-        expect(unpublished_page.reload).to be_published
-
-        visit spotlight.edit_exhibit_about_page_path(unpublished_page.exhibit, unpublished_page)
-        expect(find('#about_page_published')).to be_checked
-      end
-    end
-  end
-end

data/spec/features/add_contacts_spec.rb

@@ -1,63 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Add a contact to an exhibit', type: :feature do
-  let(:curator) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let!(:about_page) { FactoryBot.create(:about_page, exhibit: exhibit) }
-
-  before { login_as curator }
-
-  it 'displays a newly added contact in the sidebar' do
-    visit spotlight.exhibit_about_pages_path(exhibit)
-    click_link 'Add contact'
-    within '#new_contact' do
-      fill_in 'Name', with: 'Marcus Aurelius'
-      fill_in 'Email', with: 'marcus@rome.gov'
-      fill_in 'Title', with: 'Emperor'
-      fill_in 'Location', with: 'Rome'
-      fill_in 'Telephone', with: '(555) 555-5555 ext. 12345 (mobile)'
-
-      find('input[name="commit"]').click
-    end
-    expect(page).to have_content 'The contact was created.'
-
-    within '.contacts_admin' do
-      check 'exhibit_contacts_attributes_0_show_in_sidebar'
-    end
-    within '.exhibit-contacts' do
-      click_button 'Save changes'
-    end
-
-    expect(page).to have_content 'Contacts were successfully updated.'
-
-    within '#nested-pages' do
-      click_link 'View'
-    end
-
-    within '#sidebar .contacts' do
-      expect(page).to have_selector '.name', text: 'Marcus Aurelius'
-      expect(page).to have_selector 'div', text: 'marcus@rome.gov'
-      expect(page).to have_selector 'div', text: 'Emperor'
-      expect(page).to have_selector 'div', text: 'Rome'
-      expect(page).to have_selector 'div', text: '(555) 555-5555 ext. 12345 (mobile)'
-      expect(page).not_to have_selector 'img.contact-photo'
-    end
-  end
-
-  it "allows the curator to crop the contact's avatar", js: true do
-    skip "Capybara doesn't play well with the image cropper..."
-
-    visit spotlight.exhibit_about_pages_path(exhibit)
-    click_link 'Add contact'
-
-    within '#new_contact' do
-      fill_in 'Name', with: 'Pictured User'
-      fill_in 'Email', with: 'marcus@rome.gov'
-      attach_file('contact_avatar_attributes_file', File.absolute_path(File.join(FIXTURES_PATH, 'avatar.png')))
-      click_button 'Save'
-    end
-    expect(page).to have_content 'The contact was created.'
-    expect(page).to have_selector 'img.contact-photo'
-    expect(Spotlight::Contact.last.avatar.iiif_url).to be_present
-  end
-end

data/spec/features/add_custom_field_metadata_spec.rb

@@ -1,88 +0,0 @@
-# frozen_string_literal: true
-
-describe 'Adding custom metadata field data', type: :feature do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:admin) { FactoryBot.create(:exhibit_admin, exhibit: exhibit) }
-  let(:custom_field) { FactoryBot.create(:custom_field, exhibit: exhibit) }
-  let(:config) { exhibit.blacklight_configuration }
-
-  before do
-    login_as(admin)
-    config.index_fields[custom_field.field] = { enabled: true, show: true, 'label' => 'Some Field' }
-    config.save!
-  end
-
-  it 'works' do
-    visit spotlight.exhibit_solr_document_path(exhibit, 'dq287tq6352')
-
-    expect(page).to have_link 'Edit'
-
-    click_on 'Edit'
-
-    fill_in 'Some Field', with: 'My new custom field value'
-
-    click_on 'Save changes'
-
-    expect(::SolrDocument.new(id: 'dq287tq6352').sidecar(exhibit).data).to include 'some-field' => 'My new custom field value'
-    sleep(1) # The data isn't commited to solr immediately.
-
-    visit spotlight.exhibit_solr_document_path(exhibit, 'dq287tq6352')
-    expect(page).to have_content 'Some Field'
-    expect(page).to have_content 'My new custom field value'
-  end
-
-  context 'when given a read-only field' do
-    let(:custom_field) { FactoryBot.create(:custom_field, exhibit: exhibit, readonly_field: true) }
-
-    it 'can not be edited' do
-      visit spotlight.exhibit_solr_document_path(exhibit, 'dq287tq6352')
-
-      expect(page).to have_link 'Edit'
-
-      click_on 'Edit'
-
-      expect(page).to have_selector 'textarea[readonly]'
-    end
-  end
-
-  context 'with a multivalued field', js: true do
-    let(:custom_field) { FactoryBot.create(:custom_field, exhibit: exhibit, is_multiple: true) }
-
-    it 'can add multiple values' do
-      visit spotlight.exhibit_solr_document_path(exhibit, 'dq287tq6352')
-
-      expect(page).to have_link 'Edit'
-
-      click_on 'Edit'
-      fill_in 'Some Field', with: 'value 1'
-      click_on 'Add another'
-      fill_in 'solr_document_sidecar_data_some-field_2', with: 'value 2'
-
-      click_on 'Save changes'
-
-      expect(::SolrDocument.new(id: 'dq287tq6352').sidecar(exhibit).data).to include 'some-field' => ['value 1', 'value 2']
-    end
-  end
-
-  it 'has a public toggle' do
-    visit spotlight.exhibit_solr_document_path(exhibit, 'dq287tq6352')
-
-    expect(page).not_to have_selector '.blacklight-private'
-
-    click_on 'Edit'
-
-    uncheck 'Public'
-
-    click_on 'Save changes'
-
-    expect(page).to have_selector '.blacklight-private'
-
-    click_on 'Edit'
-
-    check 'Public'
-
-    click_on 'Save changes'
-
-    expect(page).not_to have_selector '.blacklight-private'
-  end
-end

data/spec/features/add_iiif_manifest_spec.rb

@@ -1,46 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe 'adding IIIF Manifest', type: :feature do
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:curator) { FactoryBot.create(:exhibit_curator, exhibit: exhibit) }
-
-  before { login_as curator }
-
-  it 'has form to add IIIF Manifests' do
-    visit spotlight.admin_exhibit_catalog_path(exhibit)
-    click_link 'Add items'
-
-    expect(page).to have_link('IIIF URL') # tab name
-    expect(page).to have_css("input[id='resource_url'][type='text']")
-    expect(page).to have_content 'Add the URL of a IIIF manifest or collection'
-    expect(page).to have_button 'Add IIIF items'
-  end
-
-  it 'submits the form to create a new item' do
-    expect_any_instance_of(Spotlight::Resource).to receive(:reindex_later).and_return(true)
-    url = 'https://purl.stanford.edu/vw754mr2281/iiif/manifest'
-    stub_request(:head, url).to_return(status: 200, headers: { 'Content-Type' => 'application/json' })
-    visit spotlight.admin_exhibit_catalog_path(exhibit)
-
-    click_link 'Add items'
-    fill_in 'URL', with: url
-
-    click_button 'Add IIIF items'
-
-    expect(Spotlight::Resource.last.url).to eq url
-  end
-
-  it 'returns an error message if the URL returned in not a IIIF endpoint' do
-    visit spotlight.admin_exhibit_catalog_path(exhibit)
-    stub_request(:head, 'http://example.com').to_return(status: 200, headers: { 'Content-Type' => 'text/html' })
-
-    click_link 'Add items'
-    fill_in 'URL', with: 'http://example.com'
-
-    click_button 'Add IIIF items'
-
-    expect(page).to have_css('.alert', text: 'Invalid IIIF URL')
-  end
-end