blacklight-spotlight 3.4.0 → 3.4.1

data/spec/spec_helper.rb

@@ -1,146 +0,0 @@
-# frozen_string_literal: true
-
-ENV['RAILS_ENV'] ||= 'test'
-require 'factory_bot'
-require 'devise'
-require 'engine_cart'
-EngineCart.load_application!
-
-Internal::Application.config.active_job.queue_adapter = :inline
-
-require 'rails-controller-testing'
-require 'rspec/collection_matchers'
-require 'rspec/its'
-require 'rspec/rails'
-require 'rspec/active_model/mocks'
-require 'paper_trail/frameworks/rspec'
-
-require 'selenium-webdriver'
-require 'webdrivers'
-require 'webmock/rspec'
-
-Capybara.javascript_driver = :headless_chrome
-
-Capybara.register_driver :headless_chrome do |app|
-  Capybara::Selenium::Driver.load_selenium
-  browser_options = ::Selenium::WebDriver::Chrome::Options.new.tap do |opts|
-    opts.args << '--headless'
-    opts.args << '--disable-gpu'
-    opts.args << '--no-sandbox'
-    opts.args << '--window-size=1280,1696'
-  end
-  Capybara::Selenium::Driver.new(app, browser: :chrome, capabilities: [browser_options])
-end
-require 'webmock/rspec'
-allowed_sites = ['chromedriver.storage.googleapis.com']
-
-WebMock.disable_net_connect!(net_http_connect_on_start: true, allow_localhost: true, allow: allowed_sites)
-
-if ENV['COVERAGE'] || ENV['CI']
-  require 'simplecov'
-
-  SimpleCov.start do
-    add_filter '/spec/'
-  end
-end
-
-require 'spotlight'
-
-# configure spotlight with all the settings necessary to test functionality
-Spotlight::Engine.config.reindexing_batch_count = 1
-Spotlight::Engine.config.assign_default_roles_to_first_user = false
-
-Dir['./spec/support/**/*.rb'].sort.each { |f| require f }
-
-FactoryBot.definition_file_paths = [File.expand_path('factories', __dir__)]
-FactoryBot.find_definitions
-
-FIXTURES_PATH = File.expand_path('fixtures', __dir__)
-
-RSpec.configure do |config|
-  config.infer_spec_type_from_file_location!
-  config.filter_rails_from_backtrace!
-
-  config.use_transactional_fixtures = true
-
-  if defined? Devise::Test::ControllerHelpers
-    config.include Devise::Test::ControllerHelpers, type: :controller
-    config.include Devise::Test::ControllerHelpers, type: :view
-  else
-    config.include Devise::TestHelpers, type: :controller
-    config.include Devise::TestHelpers, type: :view
-  end
-
-  config.include Spotlight::TestViewHelpers, type: :view
-  config.include Warden::Test::Helpers, type: :feature
-
-  config.include(ControllerLevelHelpers, type: :helper)
-  config.before(:each, type: :helper) { initialize_controller_helpers(helper) }
-
-  config.include(ControllerLevelHelpers, type: :view)
-  config.before(:each, type: :view) { initialize_controller_helpers(view) }
-
-  config.after(:each, type: :feature) { Warden.test_reset! }
-  config.include Controllers::EngineHelpers, type: :controller
-  config.include Capybara::DSL
-  config.include ::Rails.application.routes.url_helpers
-  config.include ::Rails.application.routes.mounted_helpers
-  config.include Spotlight::TestFeaturesHelpers, type: :feature
-  config.include CapybaraDefaultMaxWaitMetadataHelper, type: :feature
-
-  config.expect_with :rspec do |expectations|
-    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
-  end
-
-  config.default_formatter = 'doc' if config.files_to_run.one?
-
-  config.shared_context_metadata_behavior = :apply_to_host_groups
-
-  # This allows you to limit a spec run to individual examples or groups
-  # you care about by tagging them with `:focus` metadata. When nothing
-  # is tagged with `:focus`, all examples get run. RSpec also provides
-  # aliases for `it`, `describe`, and `context` that include `:focus`
-  # metadata: `fit`, `fdescribe` and `fcontext`, respectively.
-  config.filter_run_when_matching :focus
-
-  config.example_status_persistence_file_path = 'spec/examples.txt'
-  # Many RSpec users commonly either run the entire suite or an individual
-  # file, and it's useful to allow more verbose output when running an
-  # individual spec file.
-  if config.files_to_run.one?
-    # Use the documentation formatter for detailed output,
-    # unless a formatter has already been configured
-    # (e.g. via a command-line flag).
-    config.default_formatter = 'doc'
-  end
-
-  # Print the 10 slowest examples and example groups at the
-  # end of the spec run, to help surface which specs are running
-  # particularly slow.
-  config.profile_examples = 10
-
-  # Run specs in random order to surface order dependencies. If you find an
-  # order dependency and want to debug it, you can fix the order by providing
-  # the seed, which is printed after each run.
-  #     --seed 1234
-  # config.order = :random
-
-  # Seed global randomization in this process using the `--seed` CLI option.
-  # Setting this allows you to use `--seed` to deterministically reproduce
-  # test failures related to randomization by passing the same `--seed` value
-  # as the one that triggered the failure.
-  Kernel.srand config.seed
-end
-
-def add_new_via_button(title = 'New Page')
-  add_link = find('[data-expanded-add-button]')
-  within(add_link) do
-    expect(page).to have_css("input[type='text']", visible: false)
-  end
-  add_link.hover
-  within(add_link) do
-    input = find("input[type='text']", visible: true)
-    input.set(title)
-    find("input[data-behavior='save']").click
-  end
-end

data/spec/support/controllers/engine_helpers.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-module Controllers
-  module EngineHelpers
-    def main_app
-      Rails.application.class.routes.url_helpers
-    end
-  end
-end

data/spec/support/disable_friendly_id_deprecation_warnings.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-if Rails::VERSION::MAJOR == 5 && Rails::VERSION::MINOR <= 1
-  default_deprecation_behaviours = ActiveSupport::Deprecation.behavior
-  ActiveSupport::Deprecation.behavior = lambda do |message, callstack|
-    unless callstack.find { |l| l.path =~ %r{gems/friendly_id} } &&
-           message =~ /The behavior of .* inside of after callbacks will be changing/
-      default_deprecation_behaviours.each { |b| b.call(message, callstack) }
-    end
-  end
-end

data/spec/support/features/capybara_default_max_wait_metadata_helper.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-module CapybaraDefaultMaxWaitMetadataHelper
-  extend ActiveSupport::Concern
-
-  included do
-    before do |example|
-      next unless example.metadata[:default_max_wait_time]
-
-      @previous_wait_time = Capybara.default_max_wait_time
-      Capybara.default_max_wait_time = example.metadata[:default_max_wait_time]
-    end
-
-    after do |example|
-      next unless example.metadata[:default_max_wait_time]
-
-      Capybara.default_max_wait_time = @previous_wait_time
-    end
-  end
-end

data/spec/support/features/test_features_helpers.rb

@@ -1,92 +0,0 @@
-# frozen_string_literal: true
-
-module Spotlight
-  module TestFeaturesHelpers
-    def fill_in_typeahead_field(opts = {})
-      type = opts[:type] || 'twitter'
-      # Poltergeist / Capybara doesn't fire the events typeahead.js
-      # is listening for, so we help it out a little:
-      page.execute_script <<-EOF
-        $("[data-#{type}-typeahead]:visible").val("#{opts[:with]}").trigger("input");
-        $("[data-#{type}-typeahead]:visible").typeahead("open");
-        $(".tt-suggestion").click();
-      EOF
-
-      find('.tt-suggestion', text: opts[:with], match: :first).click
-    end
-
-    ##
-    # For typeahead "prefetched" fields, we need to wait for a resolved selector
-    # before proceeding.
-    def fill_in_prefetched_typeahead_field(opts)
-      type = opts[:type] || 'twitter'
-      # Poltergeist / Capybara doesn't fire the events typeahead.js
-      # is listening for, so we help it out a little:
-      find(opts[:wait_for]) if opts[:wait_for]
-      page.execute_script <<-EOF
-        $("[data-#{type}-typeahead]:visible").val("#{opts[:with]}").trigger("input");
-        $("[data-#{type}-typeahead]:visible").typeahead("open");
-        $(".tt-suggestion").click();
-      EOF
-    end
-
-    # just like #fill_in_typeahead_field, but wait for the
-    # form fields to show up on the page too
-    def fill_in_solr_document_block_typeahead_field(opts)
-      fill_in_typeahead_field(opts)
-      expect(page).to have_css('li[data-resource-id="' + opts[:with] + '"]')
-    end
-
-    def add_widget(type)
-      click_add_widget
-
-      # click the item + image widget
-      expect(page).to have_css("button[data-type='#{type}']")
-      find("button[data-type='#{type}']").click
-    end
-
-    def click_add_widget
-      unless all('.st-block-replacer').present?
-        expect(page).to have_css('.st-block-addition')
-        first('.st-block-addition').click
-      end
-      expect(page).to have_css('.st-block-replacer')
-      first('.st-block-replacer').click
-    end
-
-    def save_page
-      page.execute_script <<-EOF
-        SirTrevor.getInstance().onFormSubmit();
-      EOF
-      click_button('Save changes')
-      # verify that the page was created
-      expect(page).not_to have_selector('.alert-danger')
-      expect(page).to have_selector('.alert-info', text: 'page was successfully updated')
-    end
-
-    RSpec::Matchers.define :have_breadcrumbs do |*expected|
-      match do |actual|
-        errors = []
-        errors << 'Unable to find breadcrumbs' unless actual.has_css? '.breadcrumb'
-
-        breadcrumbs = expected.dup
-
-        actual.within('.breadcrumb') do
-          last = breadcrumbs.pop
-          breadcrumbs.each do |e|
-            errors << "Unable to find breadcrumb #{e}" unless actual.has_link? e
-          end
-
-          errors << "Unable to find breadcrumb #{last}" unless actual.has_content? last
-          errors << "Expected #{last} not to be a link" if actual.has_link? last
-        end
-
-        errors.empty?
-      end
-
-      failure_message do |actual|
-        "expected that #{actual.all('.breadcrumb li').map(&:text).join(' / ')} would include #{expected.join(' / ')}"
-      end
-    end
-  end
-end

data/spec/support/helpers/controller_level_helpers.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-module ControllerLevelHelpers
-  def search_state
-    @search_state ||= Blacklight::SearchState.new(params, blacklight_config)
-  end
-
-  def current_site
-    Spotlight::Site.instance
-  end
-
-  def blacklight_configuration_context
-    @blacklight_configuration_context ||= Blacklight::Configuration::Context.new(controller)
-  end
-
-  def initialize_controller_helpers(helper)
-    helper.extend ControllerLevelHelpers
-    initialize_routing_helpers(helper)
-  end
-
-  def initialize_routing_helpers(helper)
-    helper.class.include ::Rails.application.routes.url_helpers
-    helper.class.include ::Rails.application.routes.mounted_helpers if ::Rails.application.routes.respond_to?(:mounted_helpers)
-  end
-end

data/spec/support/stub_iiif_response.rb

@@ -1,26 +0,0 @@
-# frozen_string_literal: true
-
-require 'fixtures/iiif_responses'
-module StubIiifResponse
-  def stub_iiif_response_for_url(url, response)
-    allow(Spotlight::Resources::IiifService).to receive(:iiif_response).with(url).and_return(response)
-  end
-
-  def stub_default_collection
-    allow_any_instance_of(Spotlight::Resources::IiifHarvester).to receive_messages(url_is_iiif?: true)
-    stub_iiif_response_for_url('uri://for-top-level-collection', complex_collection)
-    stub_iiif_response_for_url('uri://for-child-collection1', child_collection1)
-    stub_iiif_response_for_url('uri://for-child-collection2', child_collection2)
-    stub_iiif_response_for_url('uri://for-child-collection3', child_collection3)
-
-    stub_iiif_response_for_url('uri://for-manifest1', test_manifest1)
-    stub_iiif_response_for_url('uri://for-manifest2', test_manifest2)
-    stub_iiif_response_for_url('uri://for-manifest3', test_manifest3)
-    stub_iiif_response_for_url('uri://for-manifest4', test_manifest4)
-  end
-end
-
-RSpec.configure do |config|
-  config.include IiifResponses
-  config.include StubIiifResponse
-end

data/spec/support/views/test_view_helpers.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-module Spotlight
-  module TestViewHelpers
-    extend ActiveSupport::Concern
-
-    included do
-      before do
-        view.send(:extend, Spotlight::RenderingHelper)
-        view.send(:extend, Spotlight::MainAppHelpers)
-        view.send(:extend, Spotlight::CrudLinkHelpers)
-        view.send(:extend, Spotlight::TitleHelper)
-        view.send(:extend, Spotlight::NavbarHelper)
-        view.send(:extend, Spotlight::CropHelper)
-        view.send(:extend, Spotlight::PagesHelper)
-        view.send(:extend, Blacklight::ComponentHelperBehavior)
-        view.send(:extend, BreadcrumbsOnRails::ActionController::HelperMethods)
-      end
-    end
-  end
-end

data/spec/test_app_templates/Gemfile.extra

@@ -1,2 +0,0 @@
-gem 'sprockets', '< 4' if ENV['RAILS_VERSION'] && ENV['RAILS_VERSION'] < '6'
-gem 'view_component', '!= 2.26.0'

data/spec/test_app_templates/carrierwave.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-if Rails.env.test? || Rails.env.cucumber?
-  CarrierWave.configure do |config|
-    config.storage = :file
-    config.enable_processing = false
-  end
-end

data/spec/test_app_templates/catalog_controller.rb

@@ -1,108 +0,0 @@
-# frozen_string_literal: true
-
-class CatalogController < ApplicationController
-  include Blacklight::Catalog
-  helper Openseadragon::OpenseadragonHelper
-  before_action :set_paper_trail_whodunnit
-
-  configure_blacklight do |config|
-    config.view.gallery(document_component: Blacklight::Gallery::DocumentComponent)
-    # config.view.gallery.classes = 'row-cols-2 row-cols-md-3'
-    config.view.masonry(document_component: Blacklight::Gallery::DocumentComponent)
-    config.view.slideshow(document_component: Blacklight::Gallery::SlideshowComponent)
-    config.show.tile_source_field = :content_metadata_image_iiif_info_ssm
-    config.show.partials.insert(1, :openseadragon)
-    ## Default parameters to send to solr for all search-like requests. See also SolrHelper#solr_search_params
-    config.default_solr_params = {
-      qt: 'search',
-      rows: 10,
-      fl: '*'
-    }
-
-    ## Default parameters to send on single-document requests to Solr. These settings are the Blackligt defaults (see SolrHelper#solr_doc_params) or
-    ## parameters included in the Blacklight document requestHandler.
-    #
-    # config.default_document_solr_params = {
-    #  :qt => 'document',
-    #  ## These are hard-coded in the blacklight 'document' requestHandler
-    #  # :fl => '*',
-    #  # :rows => 1
-    #  # :q => '{!raw f=id v=$id}'
-    # }
-
-    # solr field configuration for search results/index views
-    config.index.title_field = 'full_title_tesim'
-    config.index.display_type_field = 'content_metadata_type_ssm'
-    config.index.thumbnail_field = Spotlight::Engine.config.thumbnail_field
-
-    config.show.tile_source_field = :content_metadata_image_iiif_info_ssm
-
-    config.add_results_collection_tool(:sort_widget)
-    config.add_results_collection_tool(:per_page_widget)
-    config.add_results_collection_tool(:view_type_group)
-
-    # solr fields that will be treated as facets by the blacklight application
-    #   The ordering of the field names is the order of the display
-    #
-    # Setting a limit will trigger Blacklight's 'more' facet values link.
-    # * If left unset, then all facet values returned by solr will be displayed.
-    # * If set to an integer, then "f.somefield.facet.limit" will be added to
-    # solr request, with actual solr request being +1 your configured limit --
-    # you configure the number of items you actually want _displayed_ in a page.
-    # * If set to 'true', then no additional parameters will be sent to solr,
-    # but any 'sniffed' request limit parameters will be used for paging, with
-    # paging at requested limit -1. Can sniff from facet.limit or
-    # f.specific_field.facet.limit solr request params. This 'true' config
-    # can be used if you set limits in :default_solr_params, or as defaults
-    # on the solr side in the request handler itself. Request handler defaults
-    # sniffing requires solr requests to be made with "echoParams=all", for
-    # app code to actually have it echo'd back to see it.
-    #
-    # :show may be set to false if you don't want the facet to be drawn in the
-    # facet bar
-    config.add_facet_field 'genre_ssim', label: I18n.t('spotlight.search.fields.facet.genre_ssim'), limit: true
-    config.add_facet_field 'personal_name_ssm', label: I18n.t('spotlight.search.fields.facet.personal_name_ssm'), limit: true
-    config.add_facet_field 'corporate_name_ssm', label: I18n.t('spotlight.search.fields.facet.corporate_name_ssm'), limit: true
-    config.add_facet_field 'subject_geographic_ssim', label: I18n.t('spotlight.search.fields.facet.subject_geographic_ssim')
-    config.add_facet_field 'subject_temporal_ssim', label: I18n.t('spotlight.search.fields.facet.subject_temporal_ssim')
-    config.add_facet_field 'language_ssim', label: I18n.t('spotlight.search.fields.facet.language_ssim')
-
-    # Have BL send all facet field names to Solr, which has been the default
-    # previously. Simply remove these lines if you'd rather use Solr request
-    # handler defaults, or have no facets.
-    config.add_facet_fields_to_solr_request!
-
-    # solr fields to be displayed in the index (search results) view
-    #   The ordering of the field names is the order of the display
-    config.add_index_field 'language_ssm', label: I18n.t('spotlight.search.fields.language_ssm')
-    config.add_index_field 'abstract_tesim', label: I18n.t('spotlight.search.fields.abstract_tesim')
-    config.add_index_field 'note_mapuse_tesim', label: I18n.t('spotlight.search.fields.note_mapuse_tesim')
-    config.add_index_field 'note_source_tesim', label: I18n.t('spotlight.search.fields.note_source_tesim')
-    config.add_index_field 'subject_geographic_tesim', label: I18n.t('spotlight.search.fields.subject_geographic_tesim')
-    config.add_index_field 'subject_temporal_tesim', label: I18n.t('spotlight.search.fields.subject_temporal_tesim')
-
-    # solr fields to be displayed in the show (single result) view
-    #   The ordering of the field names is the order of the display
-    config.add_show_field 'note_phys_desc_tesim', label: I18n.t('spotlight.search.fields.note_phys_desc_tesim')
-    config.add_show_field 'note_source_tesim', label: I18n.t('spotlight.search.fields.note_source_tesim')
-    config.add_show_field 'note_desc_note_tesim', label: I18n.t('spotlight.search.fields.note_desc_note_tesim')
-    config.add_show_field 'note_references_tesim', label: I18n.t('spotlight.search.fields.note_references_tesim')
-    config.add_show_field 'note_provenance_tesim', label: I18n.t('spotlight.search.fields.note_provenance_tesim')
-    config.add_show_field 'note_page_num_tesim', label: I18n.t('spotlight.search.fields.note_page_num_tesim')
-    config.add_show_field 'subject_geographic_tesim', label: I18n.t('spotlight.search.fields.subject_geographic_tesim')
-    config.add_show_field 'subject_temporal_tesim', label: I18n.t('spotlight.search.fields.subject_temporal_tesim')
-    config.add_show_field 'personal_name_ssm', label: I18n.t('spotlight.search.fields.personal_name_ssm')
-    config.add_show_field 'corporate_name_ssm', label: I18n.t('spotlight.search.fields.corporate_name_ssm')
-
-    config.add_search_field 'all_fields', label: I18n.t('spotlight.search.fields.search.all_fields')
-    config.add_search_field 'title', label: I18n.t('spotlight.search.fields.search.title'), solr_parameters: { qf: 'full_title_tesim', pf: 'full_title_tesim' }
-    config.add_search_field 'author', label: I18n.t('spotlight.search.fields.search.author'), solr_parameters: { qf: '${qf_author}', pf: '${pf_author}' }
-
-    config.add_sort_field 'relevance', sort: 'score desc, sort_title_ssi asc', label: I18n.t('spotlight.search.fields.sort.relevance')
-    config.add_sort_field 'title', sort: 'sort_title_ssi asc', label: I18n.t('spotlight.search.fields.sort.title')
-    config.add_sort_field 'type', sort: 'sort_type_ssi asc', label: I18n.t('spotlight.search.fields.sort.type')
-    config.add_sort_field 'source', sort: 'sort_source_ssi asc', label: I18n.t('spotlight.search.fields.sort.source')
-    config.add_sort_field 'identifier', sort: 'id asc', label: I18n.t('spotlight.search.fields.sort.identifier')
-    config.add_sort_field 'date', sort: 'sort_date_dtsi desc', label: I18n.t('spotlight.search.fields.sort.date')
-  end
-end

data/spec/test_app_templates/lib/generators/test_app_generator.rb

@@ -1,71 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails/generators'
-
-class TestAppGenerator < Rails::Generators::Base
-  source_root '../spec/test_app_templates'
-
-  def use_capybara3
-    gsub_file 'Gemfile', /gem 'capybara'/, '# gem \'capybara\''
-  end
-
-  def add_gems
-    gem 'blacklight', '~> 7.17'
-    gem 'blacklight-gallery', '~> 4.0'
-    Bundler.with_clean_env do
-      run 'bundle install'
-    end
-  end
-
-  def run_blacklight_generator
-    say_status('warning', 'GENERATING BL', :yellow)
-
-    generate 'blacklight:install', '--devise'
-  end
-
-  def run_spotlight_migrations
-    rake 'spotlight:install:migrations'
-    rake 'db:migrate'
-  end
-
-  def add_spotlight_routes_and_assets
-    generate 'spotlight:install', '-f --mailer_default_url_host=localhost:3000'
-  end
-
-  def install_test_catalog_controller
-    copy_file 'catalog_controller.rb', 'app/controllers/catalog_controller.rb', force: true
-  end
-
-  def add_rake_tasks_to_app
-    rakefile 'spotlight_test.rake', File.read(find_in_source_paths('spotlight_test.rake'))
-  end
-
-  def disable_carrierwave_processing
-    copy_file 'carrierwave.rb', 'config/initializers/carrierwave.rb'
-  end
-
-  def add_theme_assets
-    copy_file 'fixture.png', 'app/assets/images/spotlight/themes/default_preview.png'
-    copy_file 'fixture.png', 'app/assets/images/spotlight/themes/modern_preview.png'
-
-    copy_file 'fixture.css', 'app/assets/stylesheets/application_modern.css'
-    append_to_file 'config/initializers/assets.rb', 'Rails.application.config.assets.precompile += %w( application_modern.css )'
-
-    append_to_file 'config/initializers/spotlight_initializer.rb', "\nSpotlight::Engine.config.exhibit_themes = %w[default modern]"
-  end
-
-  def disable_filter_resources_by_exhibit
-    initializer 'disable_filter_resources_by_exhibit.rb' do
-      <<-EOF
-      # Setting this to false when running tests so that we don't have to set up
-      # exhibit specific solr documents for tests that don't use the default exhibit.
-      Spotlight::Engine.config.filter_resources_by_exhibit = false
-      EOF
-    end
-  end
-
-  def raise_on_missing_translation
-    uncomment_lines 'config/environments/development.rb', /config.action_view.raise_on_missing_translations/
-    uncomment_lines 'config/environments/test.rb', /config.action_view.raise_on_missing_translations/
-  end
-end

data/spec/test_app_templates/spotlight_test.rake

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-require 'rake'
-
-namespace :spotlight_test do
-  namespace :solr do
-    desc 'Index test data into solr; must be run from within an app (see spotlight:fixtures)'
-    task :seed do
-      docs = YAML.safe_load(File.open(File.expand_path(File.join('..', 'spec', 'fixtures', 'sample_solr_documents.yml'), Rails.root)))
-      Blacklight.default_index.connection.add docs
-      Blacklight.default_index.connection.commit
-    end
-  end
-end

data/spec/uploaders/spotlight/attachment_uploader_spec.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-describe Spotlight::AttachmentUploader do
-  subject(:attachment_uploader) { described_class.new(mounter, 'mounted_as') }
-
-  let(:mounter) { Spotlight::Attachment.new(id: '5') }
-
-  describe '#store_dir' do
-    let(:store_dir) { attachment_uploader.store_dir }
-
-    it 'is prefixed with "uploads/spotlight"' do
-      expect(store_dir).to start_with 'uploads/spotlight/'
-    end
-
-    it "includes the mounter's class name" do
-      expect(store_dir).to match '/attachment/'
-    end
-
-    it 'includes the mounted_as option' do
-      expect(store_dir).to match '/mounted_as/'
-    end
-
-    it "ends with the mounter's id" do
-      expect(store_dir).to end_with "/#{mounter.id}"
-    end
-  end
-end

data/spec/uploaders/spotlight/featured_image_uploader_spec.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-describe Spotlight::FeaturedImageUploader do
-  subject(:featured_image_uploader) { described_class.new(mounter, 'mounted_as') }
-
-  let(:mounter) { FactoryBot.create(:featured_image) }
-
-  describe '#extension_allowlist' do
-    it 'is the configured array of approved extension to be uploaded' do
-      expect(featured_image_uploader.extension_allowlist).to eq Spotlight::Engine.config.allowed_upload_extensions
-    end
-  end
-
-  describe '#store_dir' do
-    let(:store_dir) { featured_image_uploader.store_dir }
-
-    it 'is prefixed with "uploads/spotlight"' do
-      expect(store_dir).to start_with 'uploads/spotlight/'
-    end
-
-    it "includes the mounter's class name" do
-      expect(store_dir).to match '/featured_image/'
-    end
-
-    it 'includes the mounted_as option' do
-      expect(store_dir).to match '/mounted_as/'
-    end
-
-    it "ends with the mounter's id" do
-      expect(store_dir).to end_with "/#{mounter.id}"
-    end
-  end
-end

data/spec/views/shared/_analytics.html.erb_spec.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-describe 'shared/_analytics', type: :view do
-  it 'is empty without Google Analytics configured' do
-    render
-    expect(rendered).to be_empty
-  end
-
-  it 'renders the GA script tag if the web property id is configured' do
-    allow(Spotlight::Engine.config).to receive(:ga_web_property_id).and_return('XYZ-234')
-    allow(Spotlight::Engine.config).to receive(:ga_anonymize_ip).and_return(true)
-    render
-    expect(rendered).to have_selector 'script', visible: false
-    expect(rendered).to have_content 'XYZ-234'
-    expect(rendered).to have_content "ga('set', 'anonymizeIp', true)"
-  end
-end