hyrax 2.6.0 → 2.7.0

data/spec/search_builders/hyrax/dashboard/nested_collections_search_builder_spec.rb

@@ -1,85 +0,0 @@
-RSpec.describe Hyrax::Dashboard::NestedCollectionsSearchBuilder do
-  let(:collection) { double(id: 'Collection_123', collection_type_gid: 'gid/abc') }
-  let(:scope) { double(current_ability: ability, blacklight_config: CatalogController.blacklight_config) }
-  let(:access) { :read }
-  let(:user) { create(:user) }
-  let(:ability) { ::Ability.new(user) }
-  let(:nesting_attributes) { double(parents: ['Parent_1', 'Parent_2'], pathnames: ['Parent_1/Collection_123', 'Parent_2/Collection_123'], ancestors: ['Parent_1', 'Parent_2'], depth: 2) }
-  let(:test_nest_direction) { :as_parent }
-  let(:builder) { described_class.new(scope: scope, access: access, collection: collection, nesting_attributes: nesting_attributes, nest_direction: test_nest_direction) }
-
-  describe '#query' do
-    subject { builder.query }
-
-    it { is_expected.to be_a(Hash) }
-  end
-
-  describe '#default_processor_chain' do
-    subject { builder.default_processor_chain }
-
-    it { is_expected.to include(:with_pagination) }
-    it { is_expected.to include(:show_only_other_collections_of_the_same_collection_type) }
-  end
-
-  describe '#show_only_other_collections_of_the_same_collection_type' do
-    let(:solr_params) { {} }
-
-    subject { builder.show_only_other_collections_of_the_same_collection_type(solr_params) }
-
-    context 'when nesting :as_parent' do
-      let(:test_nest_direction) { :as_parent }
-
-      it 'will exclude the given collection and its parents' do
-        subject
-        expect(solr_params.fetch(:fq)).to contain_exactly(
-          "-{!terms f=id}#{collection.id},#{nesting_attributes.parents.first},#{nesting_attributes.parents.last}",
-          "_query_:\"{!field f=collection_type_gid_ssim}#{collection.collection_type_gid}\"",
-          "-_query_:\"{!lucene df=nesting_collection__pathnames_ssim}*#{collection.id}*\""
-        )
-      end
-    end
-
-    context 'when nesting :as_child' do
-      let(:test_nest_direction) { :as_child }
-
-      it 'will build the search for valid children' do
-        subject
-        # rubocop:disable Metrics/LineLength
-        expect(solr_params.fetch(:fq)).to contain_exactly(
-          "-{!terms f=id}#{collection.id}",
-          "_query_:\"{!field f=collection_type_gid_ssim}#{collection.collection_type_gid}\"",
-          "-_query_:\"{!lucene q.op=OR df=nesting_collection__pathnames_ssim}#{nesting_attributes.pathnames.first} #{nesting_attributes.pathnames.last} #{nesting_attributes.ancestors.first} #{nesting_attributes.ancestors.last}\"",
-          "-_query_:\"{!field f=nesting_collection__parent_ids_ssim}#{collection.id}\""
-        )
-        # rubocop:enable Metrics/LineLength
-      end
-    end
-  end
-
-  describe '#gated_discovery_filters' do
-    subject { builder.gated_discovery_filters(access, ability) }
-
-    context 'when access is :deposit' do
-      let(:access) { "deposit" }
-      let!(:collection) { create(:collection_lw, with_permission_template: attributes) }
-
-      context 'and user has access' do
-        let(:attributes) { { deposit_users: [user.user_key] } }
-
-        it { is_expected.to eq ["{!terms f=id}#{collection.id}"] }
-      end
-
-      context 'and group has access' do
-        let(:attributes) { { deposit_groups: ['registered'] } }
-
-        it { is_expected.to eq ["{!terms f=id}#{collection.id}"] }
-      end
-
-      context "and user has no access" do
-        let(:attributes) { true }
-
-        it { is_expected.to eq ["{!terms f=id}"] }
-      end
-    end
-  end
-end

data/spec/search_builders/hyrax/dashboard/works_search_builder_spec.rb

@@ -1,44 +0,0 @@
-RSpec.describe Hyrax::Dashboard::WorksSearchBuilder do
-  let(:context) do
-    double(blacklight_config: CatalogController.blacklight_config,
-           current_ability: ability,
-           current_user: user)
-  end
-  let(:ability) do
-    ::Ability.new(user)
-  end
-  let(:user) { create(:user, groups: 'registered') }
-  let(:builder) { described_class.new(context) }
-
-  describe ".default_processor_chain" do
-    subject { described_class.default_processor_chain }
-
-    it { is_expected.to include :show_only_managed_works_for_non_admins }
-  end
-
-  describe '#discovery_permissions' do
-    subject { builder.discovery_permissions }
-
-    it { is_expected.to eq %w[edit read] }
-  end
-
-  describe "#show_only_managed_works_for_non_admins" do
-    subject { builder.show_only_managed_works_for_non_admins(solr_params) }
-
-    let(:solr_params) { Blacklight::Solr::Request.new }
-
-    it "has filter that excludes depositor" do
-      subject
-      expect(solr_params[:fq]).to eq ["-_query_:\"{!raw f=depositor_ssim}#{user.user_key}\""]
-    end
-
-    context "as admin" do
-      let(:user) { create(:user, groups: 'admin') }
-
-      it "does nothing" do
-        subject
-        expect(solr_params[:fq]).to eq []
-      end
-    end
-  end
-end

data/spec/search_builders/hyrax/embargo_search_builder_spec.rb

@@ -1,17 +0,0 @@
-RSpec.describe Hyrax::EmbargoSearchBuilder do
-  let(:context) { double }
-  let(:search_builder) { described_class.new(context) }
-
-  describe "#processor_chain" do
-    subject { search_builder.processor_chain }
-
-    it { is_expected.to eq [:with_pagination, :with_sorting, :only_active_embargoes] }
-  end
-
-  describe "#with_sorting" do
-    subject { {} }
-
-    before { search_builder.with_sorting(subject) }
-    it { is_expected.to eq(sort: 'embargo_release_date_dtsi desc') }
-  end
-end

data/spec/search_builders/hyrax/file_set_search_builder_spec.rb

@@ -1,28 +0,0 @@
-RSpec.describe Hyrax::FileSetSearchBuilder do
-  let(:processor_chain) { [:filter_models] }
-  let(:ability) { double('ability') }
-  let(:context) { double('context') }
-  let(:user) { double('user') }
-  let(:solr_params) { { fq: [] } }
-
-  subject { described_class.new(context) }
-
-  describe '#filter_models' do
-    before { subject.filter_models(solr_params) }
-
-    it 'adds FileSet to query' do
-      expect(solr_params[:fq].first).to include('{!terms f=has_model_ssim}FileSet')
-    end
-  end
-
-  describe '#find_one' do
-    before do
-      allow(subject).to receive(:blacklight_params).and_return(id: '12345')
-      subject.find_one(solr_params)
-    end
-
-    it 'adds id to query' do
-      expect(solr_params[:fq].first).to include('{!raw f=id}12345')
-    end
-  end
-end

data/spec/search_builders/hyrax/lease_search_builder_spec.rb

@@ -1,17 +0,0 @@
-RSpec.describe Hyrax::LeaseSearchBuilder do
-  let(:context) { double }
-  let(:search_builder) { described_class.new(context) }
-
-  describe "#processor_chain" do
-    subject { search_builder.processor_chain }
-
-    it { is_expected.to eq [:with_pagination, :with_sorting, :only_active_leases] }
-  end
-
-  describe "#with_sorting" do
-    subject { {} }
-
-    before { search_builder.with_sorting(subject) }
-    it { is_expected.to eq(sort: 'lease_expiration_date_dtsi desc') }
-  end
-end

data/spec/search_builders/hyrax/my/collections_search_builder_spec.rb

@@ -1,35 +0,0 @@
-RSpec.describe Hyrax::My::CollectionsSearchBuilder do
-  let(:context) do
-    double(blacklight_config: CatalogController.blacklight_config,
-           current_ability: ability,
-           current_user: user)
-  end
-  let(:ability) do
-    ::Ability.new(user)
-  end
-  let(:user) { create(:user) }
-  let(:builder) { described_class.new(context) }
-
-  describe '#models' do
-    subject { builder.models }
-
-    it { is_expected.to eq([AdminSet, Collection]) }
-  end
-
-  describe ".default_processor_chain" do
-    subject { described_class.default_processor_chain }
-
-    it { is_expected.to include :show_only_collections_deposited_by_current_user }
-  end
-
-  describe "#show_only_collections_deposited_by_current_user" do
-    subject { builder.show_only_collections_deposited_by_current_user(solr_params) }
-
-    let(:solr_params) { Blacklight::Solr::Request.new }
-
-    it "has filter that excludes depositor" do
-      subject
-      expect(solr_params[:fq]).to eq ["(_query_:\"{!raw f=depositor_ssim}#{user.user_key}\" OR (_query_:\"{!raw f=has_model_ssim}AdminSet\" AND _query_:\"{!raw f=creator_ssim}#{user.user_key}\"))"]
-    end
-  end
-end

data/spec/search_builders/hyrax/my/find_works_search_builder_spec.rb

@@ -1,63 +0,0 @@
-RSpec.describe Hyrax::My::FindWorksSearchBuilder do
-  let(:controller) { Qa::TermsController.new }
-  let(:user) { create(:user) }
-  let(:ability) { instance_double(Ability, admin?: false, user_groups: [], current_user: user) }
-  let(:q) { "foo" }
-  let(:params) { ActionController::Parameters.new(q: q, id: work.id, user: user.email, controller: "qa/terms", action: "search", vocab: "find_works") }
-
-  let(:context) do
-    double(current_ability: ability,
-           current_user: user,
-           params: params)
-  end
-  let!(:work) { create(:generic_work, :public, title: ['foo'], user: user) }
-
-  let(:builder) { described_class.new(context) }
-  let(:solr_params) { Blacklight::Solr::Request.new }
-
-  describe "#filter_on_title" do
-    subject { builder.filter_on_title(solr_params) }
-
-    it "is successful" do
-      subject
-      expect(solr_params[:fq]).to eq [ActiveFedora::SolrQueryBuilder.construct_query(title_tesim: q)]
-    end
-  end
-
-  describe "#show_only_other_works" do
-    subject { builder.show_only_other_works(solr_params) }
-
-    it "is successful" do
-      subject
-      expect(solr_params[:fq]).to eq ["-" + ActiveFedora::SolrQueryBuilder.construct_query_for_ids([work.id])]
-    end
-  end
-
-  describe "#show_only_works_not_child" do
-    subject { builder.show_only_works_not_child(solr_params) }
-
-    it "is successful" do
-      subject
-      ids = ActiveFedora::SolrService.query("{!field f=id}#{work.id}", fl: "member_ids_ssim").flat_map { |x| x.fetch("member_ids_ssim", []) }
-      expect(solr_params[:fq]).to eq ["-" + ActiveFedora::SolrQueryBuilder.construct_query_for_ids([ids])]
-    end
-  end
-
-  describe "#show_only_works_not_parent" do
-    subject { builder.show_only_works_not_parent(solr_params) }
-
-    it "is successful" do
-      subject
-      expect(solr_params[:fq]).to eq ["-" + ActiveFedora::SolrQueryBuilder.construct_query(member_ids_ssim: work.id)]
-    end
-  end
-
-  describe "#only_works?" do
-    subject { builder.only_works? }
-
-    it "is successful" do
-      subject
-      expect(subject). to eq true
-    end
-  end
-end

data/spec/search_builders/hyrax/my/shares_search_builder_spec.rb

@@ -1,32 +0,0 @@
-RSpec.describe Hyrax::My::SharesSearchBuilder do
-  let(:me) { create(:user) }
-  let(:config) { CatalogController.blacklight_config }
-  let(:scope) do
-    double('The scope',
-           blacklight_config: config,
-           params: {},
-           current_ability: Ability.new(me),
-           current_user: me)
-  end
-  let(:builder) { described_class.new(scope) }
-
-  before do
-    allow(builder).to receive(:gated_discovery_filters).and_return(["access_filter1", "access_filter2"])
-
-    # This prevents any generated classes from interfering with this test:
-    allow(builder).to receive(:work_classes).and_return([GenericWork])
-
-    allow(ActiveFedora::SolrQueryBuilder).to receive(:construct_query_for_rel)
-      .with(depositor: me.user_key)
-      .and_return("depositor")
-  end
-
-  subject { builder.to_hash['fq'] }
-
-  it "filters things we have access to in which we are not the depositor" do
-    expect(subject).to eq ["access_filter1 OR access_filter2",
-                           "{!terms f=has_model_ssim}GenericWork,Collection",
-                           "-suppressed_bsi:true",
-                           "-depositor"]
-  end
-end

data/spec/search_builders/hyrax/my/works_search_builder_spec.rb

@@ -1,52 +0,0 @@
-RSpec.describe Hyrax::My::WorksSearchBuilder do
-  let(:me) { create(:user) }
-  let(:config) { CatalogController.blacklight_config }
-  let(:scope) do
-    double('The scope',
-           blacklight_config: config,
-           params: {},
-           current_ability: Ability.new(me),
-           current_user: me)
-  end
-  let(:builder) { described_class.new(scope) }
-
-  describe "#to_hash" do
-    before do
-      # This prevents any generated classes from interfering with this test:
-      allow(builder).to receive(:work_classes).and_return([GenericWork])
-
-      allow(ActiveFedora::SolrQueryBuilder).to receive(:construct_query_for_rel)
-        .with(depositor: me.user_key)
-        .and_return("depositor")
-    end
-
-    subject { builder.to_hash['fq'] }
-
-    it "filters works that we are the depositor of" do
-      expect(subject).to eq ["{!terms f=has_model_ssim}GenericWork",
-                             "depositor"]
-    end
-  end
-
-  describe ".default_processor_chain" do
-    subject { described_class.default_processor_chain }
-
-    let(:expected_filters) do
-      [
-        :default_solr_parameters,
-        :add_query_to_solr,
-        :add_facet_fq_to_solr,
-        :add_facetting_to_solr,
-        :add_solr_fields_to_query,
-        :add_paging_to_solr,
-        :add_sorting_to_solr,
-        :add_group_config_to_solr,
-        :add_facet_paging_to_solr,
-        :filter_models,
-        :show_only_resources_deposited_by_current_user
-      ]
-    end
-
-    it { is_expected.to eq expected_filters }
-  end
-end

data/spec/search_builders/hyrax/parent_collection_search_builder_spec.rb

@@ -1,15 +0,0 @@
-RSpec.describe Hyrax::ParentCollectionSearchBuilder do
-  let(:solr_params) { { fq: [] } }
-  let(:item) { double(id: '12345', member_of_collection_ids: ['col1']) }
-  let(:builder) { described_class.new(solr_params, context) }
-  let(:context) { double("context", blacklight_config: CatalogController.blacklight_config, item: item) }
-
-  describe '#include_item_ids' do
-    let(:subject) { builder.include_item_ids(solr_params) }
-
-    it 'updates solr_parameters[:fq]' do
-      subject
-      expect(solr_params[:fq]).to include("{!terms f=id}col1")
-    end
-  end
-end

data/spec/search_builders/hyrax/single_admin_set_search_builder_spec.rb

@@ -1,20 +0,0 @@
-RSpec.describe Hyrax::SingleAdminSetSearchBuilder do
-  let(:ability) { instance_double(Ability, admin?: true) }
-  let(:context) do
-    double(blacklight_config: CatalogController.blacklight_config,
-           current_ability: ability)
-  end
-  let(:builder) { described_class.new(context) }
-
-  describe "#query" do
-    before do
-      expect(builder).to receive(:find_one)
-    end
-    subject { builder.with(id: '123').query.fetch('fq') }
-
-    it do
-      is_expected.to match_array ["",
-                                  "{!terms f=has_model_ssim}AdminSet"]
-    end
-  end
-end

data/spec/search_builders/hyrax/stats/work_status_search_builder_spec.rb

@@ -1,32 +0,0 @@
-RSpec.describe Hyrax::Stats::WorkStatusSearchBuilder do
-  let(:context) do
-    double(blacklight_config: CatalogController.blacklight_config,
-           current_ability: ability)
-  end
-  let(:ability) do
-    instance_double(Ability, admin?: true)
-  end
-  let(:instance) { described_class.new(context) }
-
-  describe "#query" do
-    subject { instance.query }
-
-    it "sets required parameters" do
-      expect(subject['facet.field']).to eq ["suppressed_bsi"]
-      expect(subject['facet.missing']).to eq true
-      expect(subject['rows']).to eq 0
-    end
-  end
-
-  describe "#only_works?" do
-    subject { instance.send(:only_works?) }
-
-    it { is_expected.to be true }
-  end
-
-  describe "::default_processor_chain" do
-    subject { described_class.default_processor_chain }
-
-    it { is_expected.to include(:filter_models) }
-  end
-end