hyrax 3.0.0.pre.beta1 → 3.0.0.pre.beta2

data/spec/actors/hyrax/actors/create_with_remote_files_ordered_members_actor_spec.rb

@@ -1,4 +1,5 @@
 RSpec.describe Hyrax::Actors::CreateWithRemoteFilesOrderedMembersActor do
+  let(:null_operation) { class_double('Hyrax::Operation').as_null_object }
   let(:terminator) { Hyrax::Actors::Terminator.new }
   let(:actor) { stack.build(terminator) }
   let(:stack) do
@@ -9,41 +10,48 @@ RSpec.describe Hyrax::Actors::CreateWithRemoteFilesOrderedMembersActor do
   let(:user) { create(:user) }
   let(:ability) { Ability.new(user) }
   let(:work) { create(:generic_work, user: user) }
-  let(:url1) { "https://dl.dropbox.com/fake/blah-blah.filepicker-demo.txt.txt" }
-  let(:url2) { "https://dl.dropbox.com/fake/blah-blah.Getting%20Started.pdf" }
+  let(:urls) { ["https://dl.dropbox.com/fake/blah-blah.filepicker-demo.txt.txt", "https://dl.dropbox.com/fake/blah-blah.Getting%20Started.pdf"] }
+  let(:filenames) { ['filepicker-demo.txt.txt', 'Getting%20Started.pdf'] }
 
   let(:remote_files) do
-    [{ url: url1,
-       expires: "2014-03-31T20:37:36.214Z",
-       file_name: "filepicker-demo.txt.txt" },
-     { url: url2,
-       expires: "2014-03-31T20:37:36.731Z",
-       file_name: "Getting+Started.pdf" }]
+    urls.collect.with_index do |url, index|
+      { url: url, expires: "2014-03-31T20:37:36.214Z", file_name: filenames[index] }
+    end
   end
+
   let(:attributes) { { remote_files: remote_files } }
   let(:environment) { Hyrax::Actors::Environment.new(work, ability, attributes) }
 
   before do
+    allow(Hyrax::Operation).to receive(:create!).and_return(null_operation)
     allow(terminator).to receive(:create).and_return(true)
   end
 
   context "with two file_sets" do
-    let(:remote_files) do
-      [{ url: url1,
-         expires: "2014-03-31T20:37:36.214Z",
-         file_name: "filepicker-demo.txt.txt" },
-       { url: url2,
-         expires: "2014-03-31T20:37:36.731Z",
-         file_name: "Getting+Started.pdf" }]
-    end
-
     it "attaches files and passes ordered_members to OrderedMembersActor in correct order" do
-      expect(Hyrax::Actors::OrderedMembersActor).to receive(:new).with([FileSet, FileSet], user).and_return(Hyrax::Actors::OrderedMembersActor)
-      expect(Hyrax::Actors::OrderedMembersActor).to receive(:attach_ordered_members_to_work).with(work)
-      expect(ImportUrlJob).to receive(:perform_later).with(FileSet, Hyrax::Operation, {}).twice
+      expect(ImportUrlJob).to receive(:perform_later).with(FileSet, null_operation, {}).twice
       expect(actor.create(environment)).to be true
-      expect(actor.ordered_members.first.label).to eq('filepicker-demo.txt.txt')
-      expect(actor.ordered_members.last.label).to eq('Getting+Started.pdf')
+      expect(work.ordered_members.to_a.collect(&:label)).to eq(filenames)
+    end
+  end
+
+  context "with two environments processed by the same actor instance simultaneously" do
+    let(:work2) { create(:generic_work, user: user) }
+    let(:environments) { [environment, Hyrax::Actors::Environment.new(work2, ability, attributes)] }
+
+    # rubocop:disable RSpec/ExampleLength
+    it "attaches the correct FileSets to the correct works in the correct order" do
+      expect(ImportUrlJob).to receive(:perform_later).with(FileSet, null_operation, {}).exactly(4).times
+      threads = environments.collect do |env|
+        Thread.new do
+          Rails.application.reloader.wrap do
+            expect(actor.create(env)).to be true
+            expect(env.curation_concern.ordered_members.to_a.collect(&:label)).to eq(filenames)
+          end
+        end
+      end
+      threads.each(&:join)
     end
+    # rubocop:enable RSpec/ExampleLength
   end
 end

data/spec/actors/hyrax/actors/interpret_visibility_actor_spec.rb

@@ -114,6 +114,7 @@ RSpec.describe Hyrax::Actors::InterpretVisibilityActor do
           expect(curation_concern.visibility_during_embargo).to eq 'authenticated'
           expect(curation_concern.visibility_after_embargo).to eq 'open'
           expect(curation_concern.visibility).to eq 'authenticated'
+          expect(curation_concern.reload).to be_under_embargo
         end
       end
 
@@ -429,6 +430,7 @@ RSpec.describe Hyrax::Actors::InterpretVisibilityActor do
           expect(curation_concern.visibility_during_lease).to eq 'open'
           expect(curation_concern.visibility_after_lease).to eq 'restricted'
           expect(curation_concern.visibility).to eq 'open'
+          expect(curation_concern.reload.lease).not_to be_new_record
         end
       end
 

data/spec/controllers/catalog_controller_spec.rb

@@ -39,7 +39,7 @@ RSpec.describe CatalogController, type: :controller do
     end
 
     context 'with collections' do
-      let(:collection) { create(:public_collection, keyword: ['rocks']) }
+      let(:collection) { create(:public_collection_lw, keyword: ['rocks']) }
       let(:objects) { [collection, rocks, clouds] }
 
       it 'finds collections' do

data/spec/controllers/hyrax/admin/collection_types_controller_spec.rb

@@ -306,7 +306,7 @@ RSpec.describe Hyrax::Admin::CollectionTypesController, type: :controller, clean
       end
 
       context "when collections exist of this type" do
-        let!(:collection) { create(:collection, collection_type_gid: collection_type_to_destroy.gid) }
+        let!(:collection) { create(:collection_lw, collection_type_gid: collection_type_to_destroy.gid) }
 
         it "doesn't delete the collection type or collection" do
           delete :destroy, params: { id: collection_type_to_destroy }

data/spec/controllers/hyrax/batch_edits_controller_spec.rb

@@ -174,15 +174,15 @@ RSpec.describe Hyrax::BatchEditsController, type: :controller do
     let(:user) { create(:user) }
 
     let(:collection1) do
-      create(:public_collection, title: ["My First Collection"],
-                                 description: ["My incredibly detailed description of the collection"],
-                                 user: user)
+      create(:public_collection_lw, title: ["My First Collection"],
+                                    description: ["My incredibly detailed description of the collection"],
+                                    user: user)
     end
 
     let(:collection2) do
-      create(:public_collection, title: ["My Other Collection"],
-                                 description: ["My incredibly detailed description of the other collection"],
-                                 user: user)
+      create(:public_collection_lw, title: ["My Other Collection"],
+                                    description: ["My incredibly detailed description of the other collection"],
+                                    user: user)
     end
 
     let!(:work1) { create(:work, title: ["First of the Assets"], member_of_collections: [collection1], user: user) }
@@ -204,9 +204,9 @@ RSpec.describe Hyrax::BatchEditsController, type: :controller do
       let(:user2) { create(:user) }
 
       let(:collection3) do
-        create(:public_collection, title: ["User2's Collection"],
-                                   description: ["Collection created by user2"],
-                                   user: user2)
+        create(:public_collection_lw, title: ["User2's Collection"],
+                                      description: ["Collection created by user2"],
+                                      user: user2)
       end
 
       before do

data/spec/controllers/hyrax/collections_controller_spec.rb

@@ -4,16 +4,16 @@ RSpec.describe Hyrax::CollectionsController do
   let(:other) { build(:user) }
 
   let(:collection) do
-    create(:public_collection, title: ["My collection"],
-                               description: ["My incredibly detailed description of the collection"],
-                               user: user)
+    create(:public_collection_lw, title: ["My collection"],
+                                  description: ["My incredibly detailed description of the collection"],
+                                  user: user)
   end
 
   let(:asset1)         { create(:work, title: ["First of the Assets"], user: user) }
   let(:asset2)         { create(:work, title: ["Second of the Assets"], user: user) }
   let(:asset3)         { create(:work, title: ["Third of the Assets"], user: user) }
-  let(:asset4)         { create(:collection, title: ["First subcollection"], user: user) }
-  let(:asset5)         { create(:collection, title: ["Second subcollection"], user: user) }
+  let(:asset4)         { build(:collection_lw, title: ["First subcollection"], user: user) }
+  let(:asset5)         { build(:collection_lw, title: ["Second subcollection"], user: user) }
   let(:unowned_asset)  { create(:work, user: other) }
 
   let(:collection_attrs) do

data/spec/controllers/hyrax/dashboard/collection_members_controller_spec.rb

@@ -10,20 +10,23 @@ RSpec.describe Hyrax::Dashboard::CollectionMembersController, :clean_repo do
   let(:work_5_read) { create(:work, id: 'work-5-read', title: ["Other's work with read access"], user: other, read_users: [user]) }
   let(:work_6_noaccess) { create(:work, id: 'work-6-no_access', title: ["Other's work with no access"], user: other) }
 
-  let(:coll_1_own) { create(:private_collection, id: 'col-1-own', title: ['User created'], user: user, create_access: true) }
+  let(:coll_1_own) { create(:private_collection_lw, id: 'col-1-own', title: ['User created'], user: user, with_permission_template: true) }
   let(:coll_2_mgr) do
-    create(:private_collection, id: 'col-2-mgr', title: ['User has manage access'], user: other,
-                                with_permission_template: { manage_users: [user] }, create_access: true)
+    create(:private_collection_lw, id: 'col-2-mgr', title: ['User has manage access'], user: other,
+                                   with_permission_template: { manage_users: [user] })
   end
   let(:coll_3_dep) do
-    create(:private_collection, id: 'col-3-dep', title: ['User has deposit access'], user: other,
-                                with_permission_template: { deposit_users: [user] }, create_access: true)
+    create(:private_collection_lw, id: 'col-3-dep', title: ['User has deposit access'], user: other,
+                                   with_permission_template: { deposit_users: [user] })
   end
   let(:coll_4_view) do
-    create(:private_collection, id: 'col-4-dep', title: ['User has view access'], user: other,
-                                with_permission_template: { view_users: [user] }, create_access: true)
+    create(:private_collection_lw, id: 'col-4-dep', title: ['User has view access'], user: other,
+                                   with_permission_template: { view_users: [user] })
+  end
+  let(:coll_5_noaccess) do
+    create(:private_collection_lw, id: 'col-5-no_access', title: ['Other user created'],
+                                   user: other, with_permission_template: true)
   end
-  let(:coll_5_noaccess) { create(:private_collection, id: 'col-5-no_access', title: ['Other user created'], user: other, create_access: true) }
 
   describe '#update_members' do
     context 'when user created the collection' do
@@ -95,7 +98,7 @@ RSpec.describe Hyrax::Dashboard::CollectionMembersController, :clean_repo do
       end
 
       context 'and user adds a subcollection' do
-        let(:parent_collection) { create(:private_collection, id: 'pcol', title: ['User created another'], user: user, create_access: true) }
+        let(:parent_collection) { create(:private_collection_lw, id: 'pcol', title: ['User created another'], user: user, with_permission_template: true) }
 
         it 'adds collection user created' do
           expect do
@@ -209,8 +212,8 @@ RSpec.describe Hyrax::Dashboard::CollectionMembersController, :clean_repo do
 
       context 'and user adds a subcollection' do
         let(:parent_collection) do
-          create(:private_collection, id: 'pcol-mgr', title: ['User has manage access to another'], user: other,
-                                      with_permission_template: { manage_users: [user] }, create_access: true)
+          create(:private_collection_lw, id: 'pcol-mgr', title: ['User has manage access to another'], user: other,
+                                         with_permission_template: { manage_users: [user] })
         end
 
         it 'adds collection user created' do
@@ -325,8 +328,8 @@ RSpec.describe Hyrax::Dashboard::CollectionMembersController, :clean_repo do
 
       context 'and user adds a subcollection' do
         let(:parent_collection) do
-          create(:private_collection, id: 'pcol-dep', title: ['User has deposit access to another'], user: other,
-                                      with_permission_template: { deposit_users: [user] }, create_access: true)
+          create(:private_collection_lw, id: 'pcol-dep', title: ['User has deposit access to another'], user: other,
+                                         with_permission_template: { deposit_users: [user] })
         end
 
         it 'adds collection user created' do

data/spec/controllers/hyrax/dashboard/collections_controller_spec.rb

@@ -5,16 +5,16 @@ RSpec.describe Hyrax::Dashboard::CollectionsController, :clean_repo do
   let(:collection_type_gid) { create(:user_collection_type).gid }
 
   let(:collection) do
-    create(:public_collection, title: ["My collection"],
-                               description: ["My incredibly detailed description of the collection"],
-                               user: user)
+    create(:public_collection_lw, title: ["My collection"],
+                                  description: ["My incredibly detailed description of the collection"],
+                                  user: user)
   end
 
   let(:asset1)         { create(:work, title: ["First of the Assets"], user: user) }
   let(:asset2)         { create(:work, title: ["Second of the Assets"], user: user) }
   let(:asset3)         { create(:work, title: ["Third of the Assets"], user: user) }
-  let(:asset4)         { create(:collection, title: ["First subcollection"], user: user) }
-  let(:asset5)         { create(:collection, title: ["Second subcollection"], user: user) }
+  let(:asset4)         { build(:collection_lw, title: ["First subcollection"], user: user) }
+  let(:asset5)         { build(:collection_lw, title: ["Second subcollection"], user: user) }
   let(:unowned_asset)  { create(:work, user: other) }
 
   let(:collection_attrs) do
@@ -82,7 +82,7 @@ RSpec.describe Hyrax::Dashboard::CollectionsController, :clean_repo do
         }
 
         expect(assigns[:collection].member_objects).to eq [asset1]
-        asset_results = ActiveFedora::SolrService.instance.conn.get "select", params: { fq: ["id:\"#{asset1.id}\""], fl: ['id', Solrizer.solr_name(:collection)] }
+        asset_results = ActiveFedora::SolrService.instance.conn.get "select", params: { fq: ["id:\"#{asset1.id}\""], fl: ['id', ActiveFedora.index_field_mapper.solr_name(:collection)] }
         expect(asset_results["response"]["numFound"]).to eq 1
         doc = asset_results["response"]["docs"].first
         expect(doc["id"]).to eq asset1.id
@@ -112,7 +112,7 @@ RSpec.describe Hyrax::Dashboard::CollectionsController, :clean_repo do
     end
 
     context "when params includes parent_id" do
-      let(:parent_collection) { create(:collection, title: ['Parent']) }
+      let(:parent_collection) { create(:collection_lw, title: ['Parent']) }
 
       it "creates a collection as a subcollection of parent" do
         parent_collection
@@ -189,7 +189,7 @@ RSpec.describe Hyrax::Dashboard::CollectionsController, :clean_repo do
       let(:asset1) { create(:generic_work, user: user) }
       let(:asset2) { create(:generic_work, user: user) }
       let(:asset3) { create(:generic_work, user: user) }
-      let(:collection2) { create(:collection, title: ['Some Collection'], user: user) }
+      let(:collection2) { create(:collection_lw, title: ['Some Collection'], user: user) }
 
       before do
         [asset1, asset2, asset3].each do |asset|
@@ -233,7 +233,7 @@ RSpec.describe Hyrax::Dashboard::CollectionsController, :clean_repo do
     end
 
     context "when update fails" do
-      let(:collection) { create(:collection, id: '12345') }
+      let(:collection) { build(:collection_lw, id: '12345') }
       let(:repository) { instance_double(Blacklight::Solr::Repository, search: result) }
       let(:result) { double(documents: [], total: 0) }
 

data/spec/controllers/hyrax/dashboard/nest_collections_controller_spec.rb

@@ -2,7 +2,7 @@ RSpec.describe Hyrax::Dashboard::NestCollectionsController do
   routes { Hyrax::Engine.routes }
   let(:child_id) { 'child1' }
   let(:child) { instance_double(Collection, title: ["Awesome Child"]) }
-  let(:parent) { create(:collection, id: 'parent1', collection_type_settings: :nestable, title: ["Uncool Parent"]) }
+  let(:parent) { create(:collection_lw, id: 'parent1', collection_type_settings: :nestable, title: ["Uncool Parent"]) }
 
   describe '#blacklight_config' do
     subject { controller.blacklight_config }

data/spec/controllers/hyrax/generic_works_controller_spec.rb

@@ -534,7 +534,7 @@ RSpec.describe Hyrax::GenericWorksController do
 
   describe '#destroy' do
     let(:work_to_be_deleted) { create(:private_generic_work, user: user) }
-    let(:parent_collection) { create(:collection) }
+    let(:parent_collection) { build(:collection_lw) }
 
     it 'deletes the work' do
       delete :destroy, params: { id: work_to_be_deleted }
@@ -618,5 +618,14 @@ RSpec.describe Hyrax::GenericWorksController do
       get :manifest, params: { id: work, format: :html }
       expect(response.body).to eq "{\"test\":\"manifest\"}"
     end
+
+    describe "when there are html tags in the labels or description" do
+      let(:manifest_factory) { double(to_h: { label: "The title<img src=xx:x onerror=eval('\x61ler\x74(1)') />", description: ["Some description <script>something</script> here..."] }) }
+
+      it "sanitizes the labels and description" do
+        get :manifest, params: { id: work, format: :json }
+        expect(response.body).to include "{\"label\":\"The title\\u003cimg\\u003e\",\"description\":[\"Some description  here...\"]}"
+      end
+    end
   end
 end

data/spec/controllers/hyrax/homepage_controller_spec.rb

@@ -48,7 +48,7 @@ RSpec.describe Hyrax::HomepageController, type: :controller do
     it "includes only GenericWork objects in recent documents" do
       get :index
       assigns(:recent_documents).each do |doc|
-        expect(doc[Solrizer.solr_name("has_model", :symbol)]).to eql ["GenericWork"]
+        expect(doc[ActiveFedora.index_field_mapper.solr_name("has_model", :symbol)]).to eql ["GenericWork"]
       end
     end
 
@@ -60,8 +60,8 @@ RSpec.describe Hyrax::HomepageController, type: :controller do
         old_to_solr = gw3.method(:to_solr)
         allow(gw3).to receive(:to_solr) do
           old_to_solr.call.merge(
-            Solrizer.solr_name('system_create', :stored_sortable, type: :date) => 1.day.ago.iso8601,
-            Solrizer.solr_name('date_uploaded', :stored_sortable, type: :date) => 1.day.ago.iso8601
+            ActiveFedora.index_field_mapper.solr_name('system_create', :stored_sortable, type: :date) => 1.day.ago.iso8601,
+            ActiveFedora.index_field_mapper.solr_name('date_uploaded', :stored_sortable, type: :date) => 1.day.ago.iso8601
           )
         end
         gw3.save

data/spec/controllers/hyrax/my/shares_controller_spec.rb

@@ -15,7 +15,7 @@ RSpec.describe Hyrax::My::SharesController, type: :controller do
       let!(:shared_with_me)           { create(:work, user: other_user, edit_users: [user, other_user]) }
       let!(:read_shared_with_me)      { create(:work, user: other_user, read_users: [user, other_user]) }
       let!(:shared_with_someone_else) { create(:work, user: other_user, edit_users: [someone_else, other_user]) }
-      let!(:my_collection)            { create(:public_collection, user: user) }
+      let!(:my_collection)            { create(:public_collection_lw, user: user) }
 
       it "responds with success" do
         get :index

data/spec/features/actor_stack_spec.rb

@@ -0,0 +1,48 @@
+require 'rails_helper'
+
+# Integration tests for the full midddleware stack
+RSpec.describe Hyrax::DefaultMiddlewareStack, :clean_repo do
+  subject(:actor)  { stack.build(Hyrax::Actors::Terminator.new) }
+  let(:ability)    { ::Ability.new(user) }
+  let(:attributes) { {} }
+  let(:stack)      { described_class.build_stack }
+  let(:terminator) { Hyrax::Actors::Terminator.new }
+  let(:user)       { FactoryBot.create(:user) }
+  let(:work)       { FactoryBot.build(:work) }
+  let(:env)        { Hyrax::Actors::Environment.new(work, ability, attributes) }
+
+  let(:delayed_failure_actor) do
+    Class.new(Hyrax::Actors::AbstractActor) do
+      def create(env)
+        next_actor.create(env) && raise('ALWAYS RAISE')
+      end
+    end
+  end
+
+  describe '#create' do
+    it 'persists the work' do
+      expect { actor.create(env) }
+        .to change { work.persisted? }
+        .to true
+    end
+
+    context 'when failing on the way back up the actor stack' do
+      before { stack.insert_before(Hyrax::Actors::ModelActor, delayed_failure_actor) }
+
+      before(:context) do
+        Hyrax.config.enable_noids = true
+        # we need to mint once to set the `rand` database column and
+        # make minter behavior predictable
+        ::Noid::Rails.config.minter_class.new.mint
+      end
+
+      after(:context) { Hyrax.config.enable_noids = false }
+
+      it 'leaves a valid minter state', :aggregate_failures do
+        expect { actor.create(env) }.to raise_error 'ALWAYS RAISE'
+
+        expect(GenericWork.new.assign_id).not_to eq work.id
+      end
+    end
+  end
+end

data/spec/features/catalog_search_spec.rb

@@ -14,7 +14,7 @@ RSpec.describe 'catalog searching', type: :feature do
       create(:public_work, title: ["Jack's Research"], keyword: ['jacks_keyword', 'shared_keyword'])
     end
 
-    let!(:collection) { create(:public_collection, keyword: ['collection_keyword', 'shared_keyword']) }
+    let!(:collection) { create(:public_collection_lw, keyword: ['collection_keyword', 'shared_keyword']) }
 
     it 'performing a search' do
       within('#search-form-header') do
@@ -30,7 +30,7 @@ RSpec.describe 'catalog searching', type: :feature do
   end
 
   context 'with public works and private collections', clean_repo: true do
-    let!(:collection) { create(:private_collection) }
+    let!(:collection) { build(:private_collection_lw) }
 
     let!(:jills_work) do
       create(:public_work, title: ["Jill's Research"], keyword: ['jills_keyword'], member_of_collections: [collection])

data/spec/features/collection_multi_membership_spec.rb

@@ -11,7 +11,7 @@ RSpec.describe 'Adding a work to multiple collections', type: :feature, clean_re
   end
 
   describe 'when both collections support multiple membership' do
-    let(:old_collection) { create(:collection_lw, user: admin_user, collection_type_gid: multi_membership_type_1.gid, title: ['OldCollectionTitle']) }
+    let(:old_collection) { build(:collection_lw, user: admin_user, collection_type_gid: multi_membership_type_1.gid, title: ['OldCollectionTitle']) }
     let!(:work) { create(:generic_work, user: admin_user, member_of_collections: [old_collection], title: ['The highly valued work that everyone wants in their collection']) }
 
     context 'and are of different types' do
@@ -56,7 +56,7 @@ RSpec.describe 'Adding a work to multiple collections', type: :feature, clean_re
   end
 
   describe 'when both collections require single membership' do
-    let(:old_collection) { create(:collection_lw, user: admin_user, collection_type_gid: single_membership_type_1.gid, title: ['OldCollectionTitle']) }
+    let(:old_collection) { build(:collection_lw, user: admin_user, collection_type_gid: single_membership_type_1.gid, title: ['OldCollectionTitle']) }
     let!(:work) do
       create(:generic_work,
              user: admin_user,

data/spec/features/collection_spec.rb

@@ -1,17 +1,17 @@
 RSpec.describe 'collection', type: :feature, clean_repo: true do
   let(:user) { create(:user) }
 
-  let(:collection1) { create(:public_collection, user: user) }
-  let(:collection2) { create(:public_collection, user: user) }
+  let(:collection1) { create(:public_collection_lw, user: user) }
+  let(:collection2) { create(:public_collection_lw, user: user) }
 
   describe 'collection show page' do
     let(:collection) do
-      create(:public_collection, user: user, description: ['collection description'], collection_type_settings: :nestable)
+      create(:public_collection_lw, user: user, description: ['collection description'], collection_type_settings: :nestable)
     end
     let!(:work1) { create(:work, title: ["King Louie"], member_of_collections: [collection], user: user) }
     let!(:work2) { create(:work, title: ["King Kong"], member_of_collections: [collection], user: user) }
-    let!(:col1) { create(:public_collection, title: ["Sub-collection 1"], member_of_collections: [collection], user: user) }
-    let!(:col2) { create(:public_collection, title: ["Sub-collection 2"], member_of_collections: [collection], user: user) }
+    let!(:col1) { create(:public_collection_lw, title: ["Sub-collection 1"], member_of_collections: [collection], user: user) }
+    let!(:col2) { create(:public_collection_lw, title: ["Sub-collection 2"], member_of_collections: [collection], user: user) }
 
     before do
       sign_in user
@@ -90,7 +90,7 @@ RSpec.describe 'collection', type: :feature, clean_repo: true do
 
       sign_in user
     end
-    let(:collection) { create(:named_collection, user: user) }
+    let(:collection) { create(:named_collection_lw, user: user) }
 
     it "shows a collection with a listing of Descriptive Metadata and catalog-style search results" do
       visit "/collections/#{collection.id}"
@@ -112,7 +112,7 @@ RSpec.describe 'collection', type: :feature, clean_repo: true do
 
       sign_in user
     end
-    let(:collection) { create(:named_collection, user: user) }
+    let(:collection) { create(:named_collection_lw, user: user) }
 
     it "shows a collection with a listing of Descriptive Metadata and catalog-style search results" do
       visit "/collections/#{collection.id}"