hyrax 2.9.6 → 3.0.0.pre.beta1

data/spec/factory_tests/collections_factory_spec.rb

@@ -0,0 +1,212 @@
+RSpec.describe 'Collections Factory' do # rubocop:disable RSpec/DescribeClass
+  let(:user) { build(:user, email: 'user@example.com') }
+  let(:user_mgr) { build(:user, email: 'user_mgr@example.com') }
+  let(:user_dep) { build(:user, email: 'user_dep@example.com') }
+  let(:user_vw) { build(:user, email: 'user_vw@example.com') }
+  let(:collection_type) { create(:collection_type) }
+
+  describe 'build' do
+    context 'with collection_type_settings and/or collection_type_gid' do
+      it 'will use the default User Collection type when neither is specified' do
+        col = build(:collection_lw)
+        expect(col.collection_type.title).to eq 'User Collection'
+        expect(col.collection_type.machine_id).to eq 'user_collection'
+      end
+
+      it 'uses collection type for passed in collection_type_gid when collection_type_settings is nil' do
+        col = build(:collection_lw, collection_type_gid: collection_type.gid)
+        expect(col.collection_type_gid).to eq collection_type.gid
+      end
+
+      it 'ignores collection_type_gid when collection_type_settings is set to attributes identifying settings' do
+        col = build(:collection_lw, collection_type_settings: [:not_discoverable, :not_sharable], collection_type_gid: collection_type.gid)
+        expect(col.collection_type_gid).not_to eq collection_type.gid
+      end
+
+      it 'will create a collection type when collection_type_settings is set to attributes identifying settings' do
+        expect { build(:collection_lw, collection_type_settings: [:discoverable]) }.to change { Hyrax::CollectionType.count }.by(1)
+        expect { build(:collection_lw, collection_type_settings: [:not_discoverable, :not_sharable]) }.to change { Hyrax::CollectionType.count }.by(1)
+      end
+
+      it 'will create a collection type with specified settings when collection_type_settings is set to attributes identifying settings' do
+        col = build(:collection_lw, collection_type_settings: [:not_discoverable, :not_sharable, :not_brandable, :nestable])
+        expect(col.collection_type.discoverable?).to be false
+        expect(col.collection_type.sharable?).to be false
+        expect(col.collection_type.brandable?).to be false
+        expect(col.collection_type.nestable?).to be true
+      end
+    end
+
+    context 'with_permission_template' do
+      it 'will not create a permission template or access when it is the default value of false' do
+        expect { build(:collection_lw) }.not_to change { Hyrax::PermissionTemplate.count }
+        expect { build(:collection_lw) }.not_to change { Hyrax::PermissionTemplateAccess.count }
+      end
+
+      it 'will create a permission template and one access for the creating user when set to true' do
+        expect { build(:collection_lw, with_permission_template: true) }.to change { Hyrax::PermissionTemplate.count }.by(1)
+        expect { build(:collection_lw, with_permission_template: true) }.to change { Hyrax::PermissionTemplateAccess.count }.by(1)
+      end
+
+      it 'will create a permission template and access for each user specified when it is set to attributes identifying access' do
+        expect { build(:collection_lw, with_permission_template: { manage_users: [user_mgr] }) }.to change { Hyrax::PermissionTemplate.count }.by(1)
+        expect { build(:collection_lw, with_permission_template: { manage_users: [user_mgr] }) }.to change { Hyrax::PermissionTemplateAccess.count }.by(2)
+        expect { build(:collection_lw, with_permission_template: { manage_users: [user_mgr], deposit_users: [user_dep], view_users: [user_vw] }) }
+          .to change { Hyrax::PermissionTemplate.count }.by(1)
+        expect { build(:collection_lw, with_permission_template: { manage_users: [user_mgr], deposit_users: [user_dep], view_users: [user_vw] }) }
+          .to change { Hyrax::PermissionTemplateAccess.count }.by(4)
+      end
+    end
+
+    context 'with_solr_document' do
+      it 'will not create a solr document by default' do
+        col = build(:collection_lw)
+        expect(col.id).to eq nil # no real way to confirm a solr document wasn't created if the collection doesn't have an id
+      end
+
+      context 'true' do
+        let(:col) { build(:collection_lw, with_solr_document: true) }
+
+        subject { ActiveFedora::SolrService.get("id:#{col.id}")["response"]["docs"].first }
+
+        it 'will create a solr document' do
+          expect(subject["id"]).to eq col.id
+          expect(subject["has_model_ssim"].first).to eq "Collection"
+          expect(subject["edit_access_person_ssim"]).not_to be_blank
+        end
+      end
+
+      context 'true and with_permission_template defines additional access' do
+        let(:col) do
+          build(:collection_lw, user: user,
+                                with_solr_document: true,
+                                with_permission_template: { manage_users: [user_mgr],
+                                                            deposit_users: [user_dep],
+                                                            view_users: [user_vw] })
+        end
+
+        subject { ActiveFedora::SolrService.get("id:#{col.id}")["response"]["docs"].first }
+
+        it 'will create a solr document' do
+          expect(subject["id"]).to eq col.id
+          expect(subject["has_model_ssim"].first).to eq "Collection"
+          expect(subject["edit_access_person_ssim"]).to include(user.user_key, user_mgr.user_key)
+          expect(subject["read_access_person_ssim"]).to include(user_dep.user_key, user_vw.user_key)
+        end
+      end
+    end
+
+    context 'with_nesting_attributes' do
+      let(:collection_type) { create(:collection_type) }
+      let(:blacklight_config) { CatalogController.blacklight_config }
+      let(:repository) { Blacklight::Solr::Repository.new(blacklight_config) }
+      let(:current_ability) { instance_double(Ability, admin?: true) }
+      let(:scope) { double('Scope', can?: true, current_ability: current_ability, repository: repository, blacklight_config: blacklight_config) }
+      let(:solr_doc) { ActiveFedora::SolrService.get("id:#{col.id}")["response"]["docs"].first }
+      let(:nesting_attributes) do
+        Hyrax::Collections::NestedCollectionQueryService::NestingAttributes.new(id: col.id, scope: scope)
+      end
+
+      context 'without additional permissions' do
+        let(:col) do
+          build(:collection_lw, id: 'Collection123',
+                                collection_type_gid: collection_type.gid,
+                                with_nesting_attributes: { ancestors: ['Parent_1'],
+                                                           parent_ids: ['Parent_1'],
+                                                           pathnames: ['Parent_1/Collection123'],
+                                                           depth: 2 })
+        end
+
+        it 'will persist a queryable solr document with the given attributes' do
+          expect(nesting_attributes.id).to eq('Collection123')
+          expect(nesting_attributes.parents).to eq(['Parent_1'])
+          expect(nesting_attributes.pathnames).to eq(['Parent_1/Collection123'])
+          expect(nesting_attributes.ancestors).to eq(['Parent_1'])
+          expect(nesting_attributes.depth).to eq(2)
+          expect(solr_doc["id"]).to eq col.id
+          expect(solr_doc["has_model_ssim"].first).to eq "Collection"
+          expect(solr_doc["edit_access_person_ssim"]).to include(col.depositor)
+        end
+      end
+
+      context ' and with_permission_template' do
+        let(:col) do
+          build(:collection_lw, id: 'Collection123',
+                                collection_type_gid: collection_type.gid,
+                                with_nesting_attributes: { ancestors: ['Parent_1'],
+                                                           parent_ids: ['Parent_1'],
+                                                           pathnames: ['Parent_1/Collection123'],
+                                                           depth: 2 },
+                                with_permission_template: { manage_users: [user_mgr],
+                                                            deposit_users: [user_dep],
+                                                            view_users: [user_vw] })
+        end
+
+        it 'will persist a queryable solr document with the given attributes' do
+          expect(nesting_attributes.id).to eq('Collection123')
+          expect(nesting_attributes.parents).to eq(['Parent_1'])
+          expect(nesting_attributes.pathnames).to eq(['Parent_1/Collection123'])
+          expect(nesting_attributes.ancestors).to eq(['Parent_1'])
+          expect(nesting_attributes.depth).to eq(2)
+          expect(solr_doc["id"]).to eq col.id
+          expect(solr_doc["has_model_ssim"].first).to eq "Collection"
+          expect(solr_doc["edit_access_person_ssim"]).to include(col.depositor, user_mgr.user_key)
+          expect(solr_doc["read_access_person_ssim"]).to include(user_dep.user_key, user_vw.user_key)
+        end
+      end
+
+      context 'and with_permission_template and with_solr_document' do
+        let(:col) do
+          build(:collection_lw, id: 'Collection123',
+                                collection_type_gid: collection_type.gid,
+                                with_nesting_attributes: { ancestors: ['Parent_1'],
+                                                           parent_ids: ['Parent_1'],
+                                                           pathnames: ['Parent_1/Collection123'],
+                                                           depth: 2 },
+                                with_permission_template: { manage_users: [user_mgr],
+                                                            deposit_users: [user_dep],
+                                                            view_users: [user_vw] },
+                                with_solr_document: true)
+        end
+
+        it 'will persist a queryable solr document with the given attributes' do
+          expect(nesting_attributes.id).to eq('Collection123')
+          expect(nesting_attributes.parents).to eq(['Parent_1'])
+          expect(nesting_attributes.pathnames).to eq(['Parent_1/Collection123'])
+          expect(nesting_attributes.ancestors).to eq(['Parent_1'])
+          expect(nesting_attributes.depth).to eq(2)
+          expect(solr_doc["id"]).to eq col.id
+          expect(solr_doc["has_model_ssim"].first).to eq "Collection"
+          expect(solr_doc["edit_access_person_ssim"]).to include(col.depositor, user_mgr.user_key)
+          expect(solr_doc["read_access_person_ssim"]).to include(user_dep.user_key, user_vw.user_key)
+        end
+      end
+    end
+  end
+
+  describe 'create' do
+    # collection_type_settings and collection_type_gid are tested by `build` and are the same for `build` and `create`
+    # with_solr_document is tested by build
+    # with_permission_template is tested by build except that the permission template is always created for `create`
+    # with_nested_attributes not supported for create
+
+    context 'with_permission_template' do
+      it 'will create a permission template and access even when it is the default value of false' do
+        expect { create(:collection_lw) }.to change { Hyrax::PermissionTemplate.count }.by(1)
+        expect { create(:collection_lw) }.to change { Hyrax::PermissionTemplateAccess.count }.by(1)
+      end
+    end
+
+    context 'when including nesting indexing', with_nested_reindexing: true do
+      # Nested indexing requires that the user's permissions be saved
+      # on the Fedora object... if simply in local memory, they are
+      # lost when the adapter pulls the object from Fedora to reindex.
+      let(:user) { create(:user) }
+      let(:collection) { create(:collection_lw, user: user) }
+
+      it 'will authorize the creating user' do
+        expect(user.can?(:edit, collection)).to be true
+      end
+    end
+  end
+end

data/spec/features/admin_spec.rb

@@ -0,0 +1,30 @@
+RSpec.describe "The admin dashboard", :clean_repo do
+  let(:user) { create :admin }
+  let(:admin_set_1) do
+    create(:admin_set, title: ["First Admin Set"],
+                       description: ["A description"],
+                       edit_users: [user.user_key])
+  end
+  let(:admin_set_2) do
+    create(:admin_set, title: ["Second Admin Set"],
+                       description: ["A description"],
+                       edit_users: [user.user_key])
+  end
+
+  before do
+    create(:work_with_files, title: ["Work A"], admin_set_id: admin_set_1.id, edit_users: [user])
+    create(:work_with_one_file, title: ["Work B"], admin_set_id: admin_set_2.id, edit_users: [user])
+    create(:work_with_file_and_work, title: ["Work C"], admin_set_id: admin_set_2.id, edit_users: [user])
+  end
+
+  it 'renders the counts of Works and Files in all AdminSets' do
+    login_as(user, scope: :user)
+    visit '/dashboard'
+
+    expect(find('tr', text: 'First Admin Set').find('td:eq(2)')).to have_content(1)
+    expect(find('tr', text: 'First Admin Set').find('td:eq(3)')).to have_content(2)
+
+    expect(find('tr', text: 'Second Admin Set').find('td:eq(2)')).to have_content(2)
+    expect(find('tr', text: 'Second Admin Set').find('td:eq(3)')).to have_content(2)
+  end
+end

data/spec/features/batch_create_spec.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+RSpec.describe 'Batch creation of works', type: :feature do
+  let(:user) { create(:user) }
+  let!(:default_admin_set) do
+    build(:admin_set,
+          id: AdminSet::DEFAULT_ID,
+          title: ["Default Admin Set"],
+          description: ["A description"],
+          with_permission_template: { deposit_groups: [::Ability.registered_group_name] })
+  end
+
+  before { sign_in user }
+
+  it "renders the batch create form" do
+    visit hyrax.new_batch_upload_path
+    expect(page).to have_content "Add New Works by Batch"
+    within("li.active") do
+      expect(page).to have_content("Files")
+    end
+    expect(page).to have_content("Each file will be uploaded to a separate new work resulting in one work per uploaded file.")
+  end
+
+  context 'when the user is a proxy', :js, :workflow, :perform_enqueued do
+    let(:second_user) { create(:user) }
+
+    before do
+      allow(CharacterizeJob).to receive(:perform_later).and_return(true)
+      allow(CreateDerivativesJob).to receive(:perform_later).and_return(true)
+
+      ProxyDepositRights.create!(grantor: second_user, grantee: user)
+      sign_in user
+      click_link 'Works'
+      click_link "Create batch of works"
+      choose "payload_concern", option: "GenericWork"
+      click_button 'Create work'
+    end
+
+    it "allows on-behalf-of batch deposit", :js do
+      click_link "Files" # switch tab
+      expect(page).to have_content "Add files"
+      within('span#addfiles') do
+        # two arbitrary files that aren't actually related, but should be
+        # small enough to require minimal processessing.
+        attach_file("files[]", "#{Hyrax::Engine.root}/spec/fixtures/small_file.txt", visible: false)
+        attach_file("files[]", "#{Hyrax::Engine.root}/spec/fixtures/png_fits.xml", visible: false)
+      end
+      click_link "Descriptions" # switch tab
+      fill_in('Creator', with: 'Doe, Jane')
+      fill_in('Keyword', with: 'testing')
+      select('In Copyright', from: 'Rights statement')
+      # With selenium and the chrome driver, focus remains on the
+      # select box. Click outside the box so the next line can't find
+      # its element
+      find('body').click
+
+      choose('batch_upload_item_visibility_open')
+      expect(page).to have_content('Please note, making something visible to the world (i.e. marking this as Public) may be viewed as publishing which could impact your ability to')
+      select(second_user.user_key, from: 'On behalf of')
+      check('agreement')
+      click_on('Save')
+
+      # Expect the proxy depositor (grantor) to be able to see both uploaded files.
+      expect(page).to have_content 'small_file.txt'
+      expect(page).to have_content 'png_fits.xml'
+
+      # Sign in with the grantee user, and expect to see the works deposited
+      # on their behalf.
+      sign_in second_user
+      click_link 'Works'
+      expect(page).to have_content 'small_file.txt'
+      expect(page).to have_content 'png_fits.xml'
+    end
+  end
+end

data/spec/features/batch_edit_spec.rb

@@ -0,0 +1,119 @@
+# frozen_string_literal: true
+
+RSpec.describe 'batch', type: :feature, clean_repo: true, js: true do
+  let(:current_user) { create(:user) }
+  let(:admin_set) { create(:admin_set) }
+  let(:permission_template) { create(:permission_template, source_id: admin_set.id) }
+  let!(:workflow) { create(:workflow, allows_access_grant: true, active: true, permission_template_id: permission_template.id) }
+
+  let!(:work1)       { create(:public_work, admin_set_id: admin_set.id, user: current_user, members: [file_set]) }
+  let!(:work2)       { create(:public_work, admin_set_id: admin_set.id, user: current_user) }
+  let!(:file_set)    { create(:file_set) }
+
+  before do
+    RoleMapper.byname[current_user.user_key] << 'donor'
+    sign_in current_user
+    visit '/dashboard/my/works'
+    check 'check_all'
+  end
+
+  describe 'editing' do
+    it 'changes the value of each field for all selected works' do
+      click_on 'batch-edit'
+      fill_in_batch_edit_fields_and_verify!
+      work1.reload
+      work2.reload
+      batch_edit_fields.each do |field|
+        expect(work1.send(field)).to match_array("NEW #{field}")
+        expect(work2.send(field)).to match_array("NEW #{field}")
+      end
+
+      # Reload the form and verify
+      visit '/dashboard/my/works'
+      check 'check_all'
+      find('#batch-edit').click
+      expect(page).to have_content('Batch Edit Descriptions')
+      batch_edit_expand("creator") do
+        page.find("input#generic_work_creator[value='NEW creator']")
+      end
+      batch_edit_expand("contributor") do
+        page.find("input#generic_work_contributor[value='NEW contributor']")
+      end
+      batch_edit_expand("description") do
+        page.find("textarea#generic_work_description", text: 'NEW description')
+      end
+      batch_edit_expand("keyword") do
+        page.find("input#generic_work_keyword[value='NEW keyword']")
+      end
+      batch_edit_expand("publisher") do
+        page.find "input#generic_work_publisher[value='NEW publisher']"
+      end
+      batch_edit_expand("date_created") do
+        page.find("input#generic_work_date_created[value='NEW date_created']")
+      end
+      batch_edit_expand("subject") do
+        page.find("input#generic_work_subject[value='NEW subject']")
+      end
+      batch_edit_expand("language") do
+        page.find("input#generic_work_language[value='NEW language']")
+      end
+      batch_edit_expand("identifier") do
+        page.find("input#generic_work_identifier[value='NEW identifier']")
+      end
+      # batch_edit_expand("based_near")
+      # expect(page).to have_css "input#generic_work_based_near[value*='NEW based_near']"
+      batch_edit_expand("related_url") do
+        page.find("input#generic_work_related_url[value='NEW related_url']")
+      end
+    end
+
+    it 'updates permissions and roles' do
+      click_on 'batch-edit'
+      find('#edit_permissions_link').click
+      expect(page).to have_content('Batch Edit Descriptions')
+
+      # Set visibility to private
+      within "#form_permissions_visibility" do
+        batch_edit_expand('permissions_visibility')
+        find('#generic_work_visibility_authenticated').click
+        find('#permissions_visibility_save').click
+        # This was `expect(page).to have_content 'Changes Saved'`, however in debugging,
+        # the `have_content` check was ignoring the `within` scoping and finding
+        # "Changes Saved" for other field areas
+        find('.status', text: 'Changes Saved', wait: 5)
+      end
+
+      within "#form_permissions" do
+        batch_edit_expand('permissions_sharing')
+        page.select('donor', from: 'new_group_name_skel')
+        page.select('View/Download', from: 'new_group_permission_skel')
+        page.find('#add_new_group_skel').click
+        find('#collapse_permissions_sharing table', text: 'View/Download')
+        find('#collapse_permissions_sharing table', text: 'donor')
+        find('#permissions_sharing_save').click
+        # This was `expect(page).to have_content 'Changes Saved'`, however in debugging,
+        # the `have_content` check was ignoring the `within` scoping and finding
+        # "Changes Saved" for other field areas
+        find('.status', text: 'Changes Saved', wait: 5)
+      end
+
+      # Visit work permissions and verify
+      visit "/concern/generic_works/#{work1.id}/edit#share"
+      page.find('#generic_work_visibility_authenticated:checked')
+      page.find('#share table', text: 'donor')
+
+      # Visit file permissions and verify
+      visit "concern/file_sets/#{work1.file_sets.first.id}/edit#permissions_display"
+      page.find('#file_set_visibility_authenticated:checked')
+      page.find('#permissions_display table', text: 'donor')
+    end
+  end
+
+  describe 'deleting' do
+    it 'destroys the selected works' do
+      accept_confirm { click_button 'Delete Selected' }
+      expect(page).to have_content('Batch delete complete')
+      expect(GenericWork.count).to be_zero
+    end
+  end
+end

data/spec/features/browse_catalog_spec.rb

@@ -0,0 +1,59 @@
+RSpec.describe "Browse catalog:", type: :feature, clean_repo: true do
+  let!(:jills_work) do
+    GenericWork.new do |work|
+      work.title = ["Jill's Research"]
+      (1..25).each do |i|
+        work.keyword << ["keyword#{format('%02d', i)}"]
+      end
+      work.apply_depositor_metadata('jilluser')
+      work.read_groups = ['public']
+      work.save!
+    end
+  end
+
+  let!(:jacks_work) do
+    GenericWork.new do |work|
+      work.title = ["Jack's Research"]
+      work.keyword = ['jacks_keyword']
+      work.apply_depositor_metadata('jackuser')
+      work.read_groups = ['public']
+      work.save!
+    end
+  end
+
+  before do
+    visit '/'
+  end
+
+  describe 'when not logged in' do
+    it 'using facet pagination to browse by keywords' do
+      click_button "search-submit-header"
+
+      expect(page).to have_content 'Search Results'
+      expect(page).to have_content jills_work.title.first
+      expect(page).to have_content jacks_work.title.first
+
+      click_link "Keyword"
+      click_link "more Keywords »"
+      within('.bottom') do
+        click_link 'Next »'
+      end
+
+      within(".modal-body") do
+        expect(page).not_to have_content 'keyword05'
+        expect(page).to have_content 'keyword21'
+
+        click_link 'keyword21'
+      end
+
+      expect(page).to have_content jills_work.title.first
+      expect(page).not_to have_content jacks_work.title.first
+
+      # TODO:  After the _generic_work.html.erb view is finished
+      #
+      #      click_link jills_work.title.first
+      #      expect(page).to     have_content "Download"
+      #      expect(page).not_to have_content "Edit"
+    end
+  end
+end

data/spec/features/browse_dashboard_works_spec.rb

@@ -0,0 +1,56 @@
+RSpec.describe "Browse Dashboard", type: :feature do
+  let(:user) { create(:user) }
+  let!(:dissertation) do
+    create(:public_work, user: user, title: ["Fake PDF Title"], subject: %w[lorem ipsum dolor sit amet])
+  end
+  let!(:mp3_work) do
+    create(:public_work, user: user, title: ["Test Document MP3"], subject: %w[consectetur adipisicing elit])
+  end
+
+  before do
+    # Grant the user access to deposit into an admin set.
+    create(:permission_template_access,
+           :deposit,
+           permission_template: create(:permission_template, with_admin_set: true),
+           agent_type: 'user',
+           agent_id: user.user_key)
+
+    sign_in user
+    create(:public_work, user: user, title: ["Fake Wav Files"], subject: %w[sed do eiusmod tempor incididunt ut labore])
+    visit "/dashboard/my/works"
+  end
+
+  it "lets the user search and display their files" do
+    fill_in "q", with: "PDF"
+    click_button "search-submit-header"
+    expect(page).to have_content("Fake PDF Title")
+    within(".constraints-container") do
+      expect(page).to have_content("Filtering by:")
+      expect(page).to have_css("span.glyphicon-remove")
+      find(".dropdown-toggle").click
+    end
+    expect(page).to have_content("Fake Wav File")
+
+    # Browse facets
+    click_button "Status"
+    click_link "Published"
+    within("#document_#{mp3_work.id}") do
+      expect(page).to have_link("Display all details of Test Document MP3",
+                                href: hyrax_generic_work_path(mp3_work, locale: 'en'))
+    end
+    click_link("Remove constraint Status: Published")
+
+    within("#document_#{dissertation.id}") do
+      click_button("Select")
+      expect(page).to have_content("Edit Work")
+    end
+  end
+
+  it "allows me to delete works in upload_sets", js: true do
+    first('input#check_all').click
+    expect do
+      accept_confirm { click_button('Delete Selected') }
+      expect(page).to have_content('Batch delete complete')
+    end.to change { GenericWork.count }.by(-3)
+  end
+end

data/spec/features/catalog_search_spec.rb

@@ -0,0 +1,72 @@
+RSpec.describe 'catalog searching', type: :feature do
+  before do
+    allow(User).to receive(:find_by_user_key).and_return(stub_model(User, twitter_handle: 'bob'))
+    sign_in :user
+    visit '/'
+  end
+
+  context 'with works and collections' do
+    let!(:jills_work) do
+      create(:public_work, title: ["Jill's Research"], keyword: ['jills_keyword', 'shared_keyword'])
+    end
+
+    let!(:jacks_work) do
+      create(:public_work, title: ["Jack's Research"], keyword: ['jacks_keyword', 'shared_keyword'])
+    end
+
+    let!(:collection) { create(:public_collection, keyword: ['collection_keyword', 'shared_keyword']) }
+
+    it 'performing a search' do
+      within('#search-form-header') do
+        fill_in('search-field-header', with: 'shared_keyword')
+        click_button('Go')
+      end
+
+      expect(page).to have_content('Search Results')
+      expect(page).to have_content(jills_work.title.first)
+      expect(page).to have_content(jacks_work.title.first)
+      expect(page).to have_content(collection.title.first)
+    end
+  end
+
+  context 'with public works and private collections', clean_repo: true do
+    let!(:collection) { create(:private_collection) }
+
+    let!(:jills_work) do
+      create(:public_work, title: ["Jill's Research"], keyword: ['jills_keyword'], member_of_collections: [collection])
+    end
+
+    it "hides collection facet values the user doesn't have access to view when performing a search" do
+      within('#search-form-header') do
+        fill_in('search-field-header', with: 'jills_keyword')
+        click_button('Go')
+      end
+
+      expect(page).to have_content('Search Results')
+      expect(page).to have_content(jills_work.title.first)
+      expect(page).not_to have_content(collection.title.first)
+      expect(page).not_to have_css('.blacklight-member_of_collection_ids_ssim')
+    end
+
+    context 'as an admin' do
+      let(:admin_user) { create :admin }
+
+      before do
+        sign_in admin_user
+        visit '/'
+      end
+
+      it "shows collection facet values the user has access to view when performing a search" do
+        within('#search-form-header') do
+          fill_in('search-field-header', with: 'jills_keyword')
+          click_button('Go')
+        end
+
+        expect(page).to have_content('Search Results')
+        expect(page).to have_content(jills_work.title.first)
+        find('.blacklight-member_of_collection_ids_ssim').click
+        expect(page).to have_content(collection.title.first)
+      end
+    end
+  end
+end