hyrax 2.1.0.beta1 → 2.1.0.beta2

data/spec/controllers/hyrax/downloads_controller_spec.rb

@@ -66,6 +66,48 @@ RSpec.describe Hyrax::DownloadsController do
             expect(ActiveFedora::Base).not_to receive(:find).with(file_set.id)
             get :show, params: { id: file_set, file: 'thumbnail' }
           end
+
+          context "stream" do
+            it "head request" do
+              request.env["HTTP_RANGE"] = 'bytes=0-15'
+              head :show, params: { id: file_set, file: 'thumbnail' }
+              expect(response.headers['Content-Length']).to eq '4218'
+              expect(response.headers['Accept-Ranges']).to eq 'bytes'
+              expect(response.headers['Content-Type']).to start_with 'image/png'
+            end
+
+            it "sends the whole thing" do
+              request.env["HTTP_RANGE"] = 'bytes=0-4217'
+              get :show, params: { id: file_set, file: 'thumbnail' }
+              expect(response.headers["Content-Range"]).to eq 'bytes 0-4217/4218'
+              expect(response.headers["Content-Length"]).to eq '4218'
+              expect(response.headers['Accept-Ranges']).to eq 'bytes'
+              expect(response.headers['Content-Type']).to start_with "image/png"
+              expect(response.headers["Content-Disposition"]).to eq "inline; filename=\"world.png\""
+              expect(response.body).to eq content
+              expect(response.status).to eq 206
+            end
+
+            it "sends the whole thing when the range is open ended" do
+              request.env["HTTP_RANGE"] = 'bytes=0-'
+              get :show, params: { id: file_set, file: 'thumbnail' }
+              expect(response.body).to eq content
+            end
+
+            it "gets a range not starting at the beginning" do
+              request.env["HTTP_RANGE"] = 'bytes=4200-4217'
+              get :show, params: { id: file_set, file: 'thumbnail' }
+              expect(response.headers["Content-Range"]).to eq 'bytes 4200-4217/4218'
+              expect(response.headers["Content-Length"]).to eq '18'
+            end
+
+            it "gets a range not ending at the end" do
+              request.env["HTTP_RANGE"] = 'bytes=4-11'
+              get :show, params: { id: file_set, file: 'thumbnail' }
+              expect(response.headers["Content-Range"]).to eq 'bytes 4-11/4218'
+              expect(response.headers["Content-Length"]).to eq '8'
+            end
+          end
         end
 
         context "that isn't persisted" do

data/spec/controllers/hyrax/embargoes_controller_spec.rb

@@ -2,7 +2,6 @@ RSpec.describe Hyrax::EmbargoesController do
   let(:user) { create(:user) }
   let(:a_work) { create(:generic_work, user: user) }
   let(:not_my_work) { create(:generic_work) }
-
   before { sign_in user }
 
   describe '#index' do
@@ -16,8 +15,12 @@ RSpec.describe Hyrax::EmbargoesController do
       let(:user) { create(:user, groups: ['admin']) }
 
       it 'shows me the page' do
+        expect(controller).to receive(:add_breadcrumb).with('Home', root_path)
+        expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path)
+        expect(controller).to receive(:add_breadcrumb).with('Manage Embargoes', embargoes_path)
         get :index
         expect(response).to be_success
+        expect(response).to render_template('dashboard')
       end
     end
   end
@@ -32,8 +35,13 @@ RSpec.describe Hyrax::EmbargoesController do
     end
     context 'when I have permission to edit the object' do
       it 'shows me the page' do
+        expect(controller).to receive(:add_breadcrumb).with('Home', root_path)
+        expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path)
+        expect(controller).to receive(:add_breadcrumb).with('Manage Embargoes', embargoes_path)
+        expect(controller).to receive(:add_breadcrumb).with('Update Embargo', '#')
         get :edit, params: { id: a_work }
         expect(response).to be_success
+        expect(response).to render_template('dashboard')
       end
     end
   end
@@ -101,6 +109,37 @@ RSpec.describe Hyrax::EmbargoesController do
           expect(response).to redirect_to embargoes_path
         end
       end
+
+      context 'with an expired embargo and filesets in batch_document_ids and in embargoes' do
+        let(:file_set2) { create(:file_set, id: 'fileset_2', visibility: Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED) }
+        let(:file_set3) { create(:file_set, id: 'fileset_3', visibility: Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED) }
+        let(:release_date) { Time.zone.today - 2 }
+        let(:batch) { [file_set2.id, a_work.id] }
+        before do
+          file_set2.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+          file_set2.visibility_during_embargo = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+          file_set2.visibility_after_embargo = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+          file_set2.embargo_release_date = release_date.to_s
+          file_set2.embargo.save(validate: false)
+          file_set2.save(validate: false)
+          file_set3.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+          file_set3.visibility_during_embargo = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+          file_set3.visibility_after_embargo = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+          file_set3.embargo_release_date = release_date.to_s
+          file_set3.embargo.save(validate: false)
+          file_set3.save(validate: false)
+        end
+
+        it 'deactivates embargo, updates the visibility and redirects' do
+          allow(controller).to receive(:filter_docs_with_edit_access!).and_return(true)
+          patch :update, params: { batch_document_ids: batch, embargoes: { '0' => { copy_visibility: file_set2.id }, '1' => { copy_visibility: a_work.id } } }
+          expect(a_work.reload.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+          expect(file_set.reload.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+          expect(file_set2.reload.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
+          expect(file_set3.reload.visibility).to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED
+          expect(response).to redirect_to embargoes_path
+        end
+      end
     end
   end
 end

data/spec/controllers/hyrax/file_sets_controller_spec.rb

@@ -56,6 +56,7 @@ RSpec.describe Hyrax::FileSetsController do
       end
 
       it "sets the breadcrumbs and versions presenter" do
+        expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.title'), Hyrax::Engine.routes.url_helpers.dashboard_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.my.works'), Hyrax::Engine.routes.url_helpers.my_works_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.file_set.browse_view'), Rails.application.routes.url_helpers.hyrax_file_set_path(file_set, locale: 'en'))
@@ -66,6 +67,7 @@ RSpec.describe Hyrax::FileSetsController do
         expect(assigns[:version_list]).to be_kind_of Hyrax::VersionListPresenter
         expect(assigns[:parent]).to eq parent
         expect(response).to render_template(:edit)
+        expect(response).to render_template('dashboard')
       end
     end
 
@@ -150,6 +152,7 @@ RSpec.describe Hyrax::FileSetsController do
               post :update, params: { id: file_set, revision: version1 }
               expect(response.code).to eq '401'
               expect(response).to render_template 'unauthorized'
+              expect(response).to render_template('dashboard')
             end
           end
         end
@@ -196,6 +199,7 @@ RSpec.describe Hyrax::FileSetsController do
           post :update, params: { id: file_set, file_set: { keyword: [''] } }
           expect(response.code).to eq '422'
           expect(response).to render_template('edit')
+          expect(response).to render_template('dashboard')
           expect(assigns[:file_set]).to eq file_set
         end
       end
@@ -220,6 +224,7 @@ RSpec.describe Hyrax::FileSetsController do
           get :edit, params: { id: file_set }
           expect(response.code).to eq '401'
           expect(response).to render_template('unauthorized')
+          expect(response).to render_template('dashboard')
         end
       end
     end
@@ -231,6 +236,7 @@ RSpec.describe Hyrax::FileSetsController do
 
       context "without a referer" do
         it "shows me the file and set breadcrumbs" do
+          expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
           expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.title'), Hyrax::Engine.routes.url_helpers.dashboard_path(locale: 'en'))
           get :show, params: { id: file_set }
           expect(response).to be_successful
@@ -257,8 +263,9 @@ RSpec.describe Hyrax::FileSetsController do
         end
 
         it "shows me the breadcrumbs" do
-          expect(controller).to receive(:add_breadcrumb).with('My Dashboard', Hyrax::Engine.routes.url_helpers.dashboard_path(locale: 'en'))
-          expect(controller).to receive(:add_breadcrumb).with('Your Works', Hyrax::Engine.routes.url_helpers.my_works_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Dashboard', Hyrax::Engine.routes.url_helpers.dashboard_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Works', Hyrax::Engine.routes.url_helpers.my_works_path(locale: 'en'))
           expect(controller).to receive(:add_breadcrumb).with('test title', main_app.hyrax_generic_work_path(work.id, locale: 'en'))
           expect(controller).to receive(:add_breadcrumb).with('test file', main_app.hyrax_file_set_path(file_set, locale: 'en'))
           get :show, params: { id: file_set }
@@ -278,6 +285,7 @@ RSpec.describe Hyrax::FileSetsController do
           get :edit, params: { id: public_file_set }
           expect(response.code).to eq '401'
           expect(response).to render_template(:unauthorized)
+          expect(response).to render_template('dashboard')
         end
       end
 

data/spec/controllers/hyrax/generic_works_controller_spec.rb

@@ -48,6 +48,36 @@ RSpec.describe Hyrax::GenericWorksController do
     before do
       create(:sipity_entity, proxy_for_global_id: work.to_global_id.to_s)
     end
+
+    context 'while logged out' do
+      let(:work) { create(:public_generic_work, user: user, title: ['public thing']) }
+
+      before { sign_out user }
+
+      context "without a referer" do
+        it "sets the default breadcrumbs" do
+          expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
+          get :show, params: { id: work }
+          expect(response).to be_successful
+        end
+      end
+
+      context "with a referer" do
+        before do
+          request.env['HTTP_REFERER'] = 'http://test.host/foo'
+        end
+
+        it "sets breadcrumbs to authorized pages" do
+          expect(controller).not_to receive(:add_breadcrumb).with('Dashboard', hyrax.dashboard_path(locale: 'en'))
+          expect(controller).not_to receive(:add_breadcrumb).with('Your Works', hyrax.my_works_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('public thing', main_app.hyrax_generic_work_path(work.id, locale: 'en'))
+          get :show, params: { id: work }
+          expect(response).to be_successful
+          expect(response).to render_template("layouts/hyrax/1_column")
+        end
+      end
+    end
+
     context 'my own private work' do
       let(:work) { create(:private_generic_work, user: user, title: ['test title']) }
 
@@ -59,7 +89,8 @@ RSpec.describe Hyrax::GenericWorksController do
 
       context "without a referer" do
         it "sets breadcrumbs" do
-          expect(controller).to receive(:add_breadcrumb).with("My Dashboard", hyrax.dashboard_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with("Dashboard", hyrax.dashboard_path(locale: 'en'))
           get :show, params: { id: work }
           expect(response).to be_successful
         end
@@ -71,8 +102,9 @@ RSpec.describe Hyrax::GenericWorksController do
         end
 
         it "sets breadcrumbs" do
-          expect(controller).to receive(:add_breadcrumb).with('My Dashboard', hyrax.dashboard_path(locale: 'en'))
-          expect(controller).to receive(:add_breadcrumb).with('Your Works', hyrax.my_works_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Dashboard', hyrax.dashboard_path(locale: 'en'))
+          expect(controller).to receive(:add_breadcrumb).with('Works', hyrax.my_works_path(locale: 'en'))
           expect(controller).to receive(:add_breadcrumb).with('test title', main_app.hyrax_generic_work_path(work.id, locale: 'en'))
           get :show, params: { id: work }
           expect(response).to be_successful
@@ -120,7 +152,7 @@ RSpec.describe Hyrax::GenericWorksController do
       end
     end
 
-    context 'someone elses public work' do
+    context 'someone else\'s public work' do
       let(:work) { create(:public_generic_work) }
 
       context "html" do
@@ -352,8 +384,8 @@ RSpec.describe Hyrax::GenericWorksController do
 
       it 'shows me the page and sets breadcrumbs' do
         expect(controller).to receive(:add_breadcrumb).with("Home", root_path(locale: 'en'))
-        expect(controller).to receive(:add_breadcrumb).with("Administration", hyrax.dashboard_path(locale: 'en'))
-        expect(controller).to receive(:add_breadcrumb).with("Your Works", hyrax.my_works_path(locale: 'en'))
+        expect(controller).to receive(:add_breadcrumb).with("Dashboard", hyrax.dashboard_path(locale: 'en'))
+        expect(controller).to receive(:add_breadcrumb).with("Works", hyrax.my_works_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(work.title.first, main_app.hyrax_generic_work_path(work.id, locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with('Edit', main_app.edit_hyrax_generic_work_path(work.id))
 

data/spec/controllers/hyrax/leases_controller_spec.rb

@@ -16,8 +16,13 @@ RSpec.describe Hyrax::LeasesController do
       let(:user) { create(:user, groups: ['admin']) }
 
       it 'shows me the page' do
+        expect(controller).to receive(:add_breadcrumb).with('Home', root_path)
+        expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path)
+        expect(controller).to receive(:add_breadcrumb).with('Manage Leases', leases_path)
+
         get :index
         expect(response).to be_success
+        expect(response).to render_template('dashboard')
       end
     end
   end
@@ -32,8 +37,14 @@ RSpec.describe Hyrax::LeasesController do
     end
     context 'when I have permission to edit the object' do
       it 'shows me the page' do
+        expect(controller).to receive(:add_breadcrumb).with('Home', root_path)
+        expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path)
+        expect(controller).to receive(:add_breadcrumb).with('Manage Leases', leases_path)
+        expect(controller).to receive(:add_breadcrumb).with('Update Lease', '#')
+
         get :edit, params: { id: a_work }
         expect(response).to be_success
+        expect(response).to render_template('dashboard')
       end
     end
   end

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

@@ -13,7 +13,7 @@ RSpec.describe Hyrax::My::CollectionsController, type: :controller do
         expect(controller).to receive(:search_results).with(ActionController::Parameters).and_return([response, doc_list])
 
         expect(controller).to receive(:add_breadcrumb).with('Home', root_path(locale: 'en'))
-        expect(controller).to receive(:add_breadcrumb).with('Administration', dashboard_path(locale: 'en'))
+        expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with('Collections', my_collections_path(locale: 'en'))
 
         get :index, params: { per_page: 2 }

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

@@ -17,7 +17,7 @@ RSpec.describe Hyrax::My::WorksController, type: :controller do
     it "shows search results and breadcrumbs" do
       expect(controller).to receive(:search_results).with(ActionController::Parameters).and_return([response, doc_list])
       expect(controller).to receive(:add_breadcrumb).with('Home', root_path(locale: 'en'))
-      expect(controller).to receive(:add_breadcrumb).with('Administration', dashboard_path(locale: 'en'))
+      expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path(locale: 'en'))
       expect(controller).to receive(:add_breadcrumb).with('Works', my_works_path(locale: 'en'))
       expect(collection_service).to receive(:search_results).with(:deposit).and_return([my_collection])
       get :index, params: { per_page: 2 }

data/spec/controllers/hyrax/pages_controller_spec.rb

@@ -68,7 +68,7 @@ RSpec.describe Hyrax::PagesController, type: :controller do
       describe "GET #edit" do
         it "renders breadcrumbs and dashboard layout" do
           expect(controller).to receive(:add_breadcrumb).with('Home', root_path)
-          expect(controller).to receive(:add_breadcrumb).with('Administration', dashboard_path)
+          expect(controller).to receive(:add_breadcrumb).with('Dashboard', dashboard_path)
           expect(controller).to receive(:add_breadcrumb).with('Configuration', '#')
           expect(controller).to receive(:add_breadcrumb).with('Pages', edit_pages_path)
           get :edit

data/spec/controllers/hyrax/stats_controller_spec.rb

@@ -17,6 +17,7 @@ RSpec.describe Hyrax::StatsController do
 
       it 'renders the stats view' do
         expect(Hyrax::FileUsage).to receive(:new).with(file_set.id).and_return(usage)
+        expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.title'), Hyrax::Engine.routes.url_helpers.dashboard_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.my.works'), Hyrax::Engine.routes.url_helpers.my_works_path(locale: 'en'))
         expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.file_set.browse_view'), Rails.application.routes.url_helpers.hyrax_file_set_path(file_set, locale: 'en'))
@@ -60,6 +61,7 @@ RSpec.describe Hyrax::StatsController do
 
     it 'renders the stats view' do
       expect(Hyrax::WorkUsage).to receive(:new).with(work.id).and_return(usage)
+      expect(controller).to receive(:add_breadcrumb).with('Home', Hyrax::Engine.routes.url_helpers.root_path(locale: 'en'))
       expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.my.works'), Hyrax::Engine.routes.url_helpers.my_works_path(locale: 'en'))
       expect(controller).to receive(:add_breadcrumb).with(I18n.t('hyrax.dashboard.title'), Hyrax::Engine.routes.url_helpers.dashboard_path(locale: 'en'))
       expect(controller).to receive(:add_breadcrumb).with('Test title', main_app.hyrax_generic_work_path(work, locale: 'en'))

data/spec/controllers/hyrax/transfers_controller_spec.rb

@@ -64,7 +64,7 @@ RSpec.describe Hyrax::TransfersController, type: :controller do
         # AND A NOTIFICATION SHOULD HAVE BEEN CREATED
         notification = another_user.reload.mailbox.inbox[0].messages[0]
         expect(notification.subject).to eq("Ownership Change Request")
-        expect(notification.body).to eq("<a href=\"/users/#{user.user_key}\">#{user.name}</a> " \
+        expect(notification.body).to eq("<a href=\"#{routes.url_helpers.user_path(user)}\">#{user.name}</a> " \
                                         "wants to transfer a work to you. Review all " \
                                         "<a href=\"#{routes.url_helpers.transfers_path}\">transfer requests</a>")
       end

data/spec/factories/collection_types.rb

@@ -1,12 +1,13 @@
 FactoryBot.define do
   factory :collection_type, class: Hyrax::CollectionType do
-    sequence(:title) { |n| "Title #{n}" }
+    sequence(:title) { |n| "Collection Type #{n}" }
     sequence(:machine_id) { |n| "title_#{n}" }
 
     description 'Collection type with all options'
     nestable true
     discoverable true
     sharable true
+    brandable true
     share_applies_to_new_works true
     allow_multiple_membership true
     require_membership false
@@ -70,6 +71,14 @@ FactoryBot.define do
       discoverable false
     end
 
+    trait :brandable do
+      brandable true
+    end
+
+    trait :not_brandable do
+      brandable false
+    end
+
     trait :sharable do
       sharable true
       share_applies_to_new_works true

data/spec/factories/collections.rb

@@ -52,6 +52,7 @@ FactoryBot.define do
   #   let(:settings) { [
   #                      :nestable,                  # OR :not_nestable,
   #                      :discoverable,              # OR :not_discoverable
+  #                      :brandable,                 # OR :not_brandable
   #                      :sharable,                  # OR :not_sharable OR :sharable_no_work_permissions
   #                      :allow_multiple_membership, # OR :not_allow_multiple_membership
   #                    ] }
@@ -63,6 +64,7 @@ FactoryBot.define do
   #   let(:settings) { [
   #                      :nestable,                  # OR :not_nestable,
   #                      :discoverable,              # OR :not_discoverable
+  #                      :brandable,                 # OR :not_brandable
   #                      :sharable,                  # OR :not_sharable OR :sharable_no_work_permissions
   #                      :allow_multiple_membership, # OR :not_allow_multiple_membership
   #                    ] }
@@ -105,7 +107,7 @@ FactoryBot.define do
       with_nesting_attributes nil
       with_solr_document false
     end
-    sequence(:title) { |n| ["Title #{n}"] }
+    sequence(:title) { |n| ["Collection Title #{n}"] }
 
     after(:build) do |collection, evaluator|
       collection.apply_depositor_metadata(evaluator.user.user_key)
@@ -162,7 +164,7 @@ FactoryBot.define do
       user { create(:user) }
     end
 
-    sequence(:title) { |n| ["Title #{n}"] }
+    sequence(:title) { |n| ["User Collection Title #{n}"] }
 
     after(:build) do |collection, evaluator|
       collection.apply_depositor_metadata(evaluator.user.user_key)
@@ -181,7 +183,7 @@ FactoryBot.define do
       do_save false
     end
 
-    sequence(:title) { |n| ["Title #{n}"] }
+    sequence(:title) { |n| ["Typeless Collection Title #{n}"] }
 
     after(:build) do |collection, evaluator|
       collection.apply_depositor_metadata(evaluator.user.user_key)

data/spec/factories/collections_factory.rb

@@ -25,7 +25,7 @@ FactoryBot.define do
       create_access false
       with_nesting_attributes nil
     end
-    sequence(:title) { |n| ["Title #{n}"] }
+    sequence(:title) { |n| ["Collection Title #{n}"] }
 
     after(:build) do |collection, evaluator|
       collection.apply_depositor_metadata(evaluator.user.user_key)
@@ -88,7 +88,7 @@ FactoryBot.define do
       user { create(:user) }
     end
 
-    sequence(:title) { |n| ["Title #{n}"] }
+    sequence(:title) { |n| ["User Collection Title #{n}"] }
 
     after(:build) do |collection, evaluator|
       collection.apply_depositor_metadata(evaluator.user.user_key)
@@ -108,7 +108,7 @@ FactoryBot.define do
       do_save false
     end
 
-    sequence(:title) { |n| ["Title #{n}"] }
+    sequence(:title) { |n| ["Typeless Collection Title #{n}"] }
 
     after(:build) do |collection, evaluator|
       collection.apply_depositor_metadata(evaluator.user.user_key)