blacklight-spotlight 3.4.0 → 3.4.2.1

data/spec/models/spotlight/access_controls_enforcement_search_builder_spec.rb

@@ -1,82 +0,0 @@
-# frozen_string_literal: true
-
-describe Spotlight::AccessControlsEnforcementSearchBuilder do
-  class MockSearchBuilder < Blacklight::SearchBuilder
-    attr_reader :blacklight_params, :scope
-
-    def initialize(blacklight_params, scope)
-      @blacklight_params = blacklight_params
-      @scope = scope
-    end
-    include Spotlight::AccessControlsEnforcementSearchBuilder
-    include Blacklight::Solr::SearchBuilderBehavior
-
-    def blacklight_config
-      scope.current_exhibit.blacklight_config
-    end
-  end
-
-  subject { MockSearchBuilder.new blacklight_params, scope }
-
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let(:scope) { double(current_exhibit: exhibit, context: { current_ability: current_ability }) }
-  let(:current_ability) { instance_double(Ability) }
-  let(:solr_request) { Blacklight::Solr::Request.new }
-  let(:blacklight_params) { {} }
-
-  describe '#apply_permissive_visibility_filter' do
-    it 'allows curators to view everything' do
-      allow(current_ability).to receive(:can?).and_return(true)
-      subject.apply_permissive_visibility_filter(solr_request)
-      expect(solr_request.to_hash).to be_empty
-    end
-
-    it 'restricts searches to public items' do
-      allow(current_ability).to receive(:can?).and_return(false)
-
-      subject.apply_permissive_visibility_filter(solr_request)
-      expect(solr_request).to include :fq
-      expect(solr_request[:fq]).to include "-exhibit_#{exhibit.slug}_public_bsi:false"
-    end
-
-    it 'does not filter resources to just those created by the exhibit' do
-      allow(current_ability).to receive(:can?).and_return(true)
-      subject.apply_permissive_visibility_filter(solr_request)
-      expect(solr_request[:fq]).to be_blank
-    end
-  end
-
-  describe '#apply_exhibit_resources_filter' do
-    context 'with filter_resources_by_exhibit' do
-      before do
-        allow(Spotlight::Engine.config).to receive(:filter_resources_by_exhibit).and_return(true)
-      end
-
-      it 'filters resources to just those created by the exhibit' do
-        allow(current_ability).to receive(:can?).and_return(true)
-
-        subject.apply_exhibit_resources_filter(solr_request)
-
-        expect(solr_request).to include :fq
-        expect(solr_request[:fq]).to include "{!term f=spotlight_exhibit_slug_#{exhibit.slug}_bsi}true"
-      end
-    end
-
-    context 'with a custom exhibit filter' do
-      let(:filter) { exhibit.filters.first_or_initialize }
-
-      before do
-        filter.update(field: 'author_ssim', value: 'Coyne, Justin')
-      end
-
-      it 'filters resources to just those identified by the exhibit filter' do
-        allow(current_ability).to receive(:can?).and_return(true)
-
-        subject.apply_exhibit_resources_filter(solr_request)
-
-        expect(solr_request).to include :fq
-        expect(solr_request[:fq]).to include '{!term f=author_ssim}Coyne, Justin'
-      end
-    end
-  end
-end

data/spec/models/spotlight/analytics/ga_spec.rb

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-describe Spotlight::Analytics::Ga do
-  it 'does not be enabled without configuration' do
-    expect(subject).not_to be_enabled
-  end
-
-  describe '#site' do
-    it 'selects the correct profile based on the web property id' do
-      allow(Spotlight::Engine.config).to receive_messages(ga_web_property_id: 'bar')
-      allow(subject).to receive_message_chain(
-        :user,
-        :accounts,
-        :first,
-        profiles: [
-          double('profile1', web_property_id: 'foo'), double('profile2', web_property_id: 'bar')
-        ]
-      )
-      expect(subject.site.web_property_id).to eq 'bar'
-    end
-  end
-end

data/spec/models/spotlight/background_job_progress_spec.rb

@@ -1,137 +0,0 @@
-# frozen_string_literal: true
-
-describe Spotlight::BackgroundJobProgress, type: :model do
-  subject(:progress) { described_class.new(exhibit, job_class: Spotlight::ReindexExhibitJob) }
-
-  let(:exhibit) { FactoryBot.create(:exhibit) }
-  let!(:job_tracker) do
-    FactoryBot.create(
-      :job_tracker,
-      resource: exhibit,
-      on: exhibit,
-      job_class: 'Spotlight::ReindexExhibitJob',
-      updated_at: Time.zone.now,
-      **job_tracker_args
-    )
-  end
-  let(:job_tracker_args) { {} }
-
-  context 'with a completed job' do
-    let(:job_tracker_args) { { status: 'completed', data: { progress: 50, total: 50 } } }
-
-    it 'reports on reindexing progress' do
-      expect(progress.as_json).to include(
-        completed: 50,
-        total: 50,
-        finished: true,
-        errored: false,
-        recently_in_progress: true
-      )
-    end
-  end
-
-  context 'with a failed job' do
-    let(:job_tracker_args) { { status: 'failed', data: { progress: 32, total: 50 } } }
-
-    it 'reports on reindexing progress' do
-      expect(progress.as_json).to include(
-        completed: 32,
-        total: 50,
-        finished: true,
-        errored: true
-      )
-    end
-  end
-
-  context 'with an in-progress job' do
-    let(:job_tracker_args) { { status: 'in_progress', data: { progress: 32, total: 50 } } }
-
-    it 'reports on reindexing progress' do
-      expect(progress.as_json).to include(
-        completed: 32,
-        total: 50,
-        finished: false,
-        errored: false
-      )
-    end
-  end
-
-  context 'with an in-progress job with subtasks' do
-    before do
-      FactoryBot.create(:job_tracker, on: job_tracker, resource: exhibit, data: { progress: 32, total: 32 }, status: 'completed')
-      FactoryBot.create(:job_tracker, on: job_tracker, resource: exhibit, data: { progress: 16, total: 50 }, status: 'in_progress')
-    end
-
-    let(:job_tracker_args) { { status: 'in_progress' } }
-
-    it 'harvests total and completed data from the child jobs' do
-      expect(progress.as_json).to include(
-        completed: 48,
-        total: 82
-      )
-    end
-
-    it 'is not completed until all the child jobs are finished' do
-      job_tracker.update(status: 'failed')
-
-      expect(progress.as_json).to include(
-        finished: false,
-        errored: true
-      )
-    end
-  end
-
-  context 'with no job' do
-    before do
-      job_tracker.delete
-    end
-
-    it 'reports on reindexing progress' do
-      expect(progress.as_json).to include(
-        completed: 0,
-        total: 0,
-        finished: false,
-        errored: false,
-        recently_in_progress: false
-      )
-    end
-  end
-
-  context 'with multiple job trackers' do
-    let(:job_tracker_args) { { status: 'in_progress' } }
-
-    before do
-      FactoryBot.create(
-        :job_tracker,
-        resource: exhibit,
-        on: exhibit,
-        job_class: 'Spotlight::ReindexExhibitJob',
-        updated_at: Time.zone.now,
-        status: 'completed'
-      )
-    end
-
-    it 'uses the currently in-progress tracker' do
-      expect(progress.as_json).to include(finished: false)
-    end
-  end
-
-  context 'with multiple in-progress trackers' do
-    let(:job_tracker_args) { { status: 'in_progress' } }
-
-    before do
-      FactoryBot.create(
-        :job_tracker,
-        resource: exhibit,
-        on: exhibit,
-        job_class: 'Spotlight::ReindexExhibitJob',
-        updated_at: Time.zone.now - 5.years,
-        status: 'in_progress'
-      )
-    end
-
-    it 'uses the most recently updated tracker' do
-      expect(Time.zone.parse(progress.as_json[:updated_at]).year).to eq job_tracker.updated_at.year
-    end
-  end
-end