hyrax 2.0.3 → 2.1.0.beta1

data/spec/models/hyrax/user_usage_stats_spec.rb

@@ -1,5 +1,5 @@
 RSpec.describe Hyrax::UserUsageStats do
-  subject { FactoryBot.create(:user) }
+  subject { create(:user) }
 
   describe 'with cached stats' do
     let!(:stat_1_day_ago) { UserStat.create!(user_id: subject.id, date: 1.day.ago, file_views: 3, file_downloads: 2, work_views: 5) }

data/spec/models/hyrax/work_behavior_spec.rb

@@ -18,8 +18,10 @@ RSpec.describe Hyrax::WorkBehavior do
                                                Hydra::WithDepositor,
                                                Hydra::AccessControls::Embargoable,
                                                Solrizer::Common,
-                                               Hyrax::Suppressible)
+                                               Hyrax::Suppressible,
+                                               Hyrax::CollectionNesting)
   end
+
   describe '#to_s' do
     it 'uses the provided titles' do
       subject.title = %w[Hello World]
@@ -32,7 +34,8 @@ RSpec.describe Hyrax::WorkBehavior do
     it 'has a default' do
       expect(subject.human_readable_type).to eq 'Essential Work'
     end
-    it 'is settable' do
+    it 'is settable (deprecated)' do
+      allow(Deprecation).to receive(:warn)
       EssentialWork.human_readable_type = 'Custom Type'
       expect(subject.human_readable_type).to eq 'Custom Type'
     end
@@ -51,4 +54,11 @@ RSpec.describe Hyrax::WorkBehavior do
       expect(EssentialWork.indexer).to eq klass
     end
   end
+
+  describe '#update_nested_collection_relationship_indices', :with_nested_reindexing do
+    it 'will be called after save' do
+      expect(Samvera::NestingIndexer).to receive(:reindex_relationships).with(id: kind_of(String))
+      subject.save!
+    end
+  end
 end

data/spec/models/sipity/agent_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe Agent, type: :model, no_clean: true do
+  RSpec.describe Agent, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include("proxy_for_id") }

data/spec/models/sipity/comment_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe Comment, type: :model, no_clean: true do
+  RSpec.describe Comment, type: :model do
     context 'database configuration' do
       subject { described_class }
 

data/spec/models/sipity/entity_spec.rb

@@ -1,6 +1,6 @@
 module Sipity
   RSpec.describe Entity, type: :model do
-    describe 'database configuration', no_clean: true do
+    describe 'database configuration' do
       subject { described_class }
 
       its(:column_names) { is_expected.to include("proxy_for_global_id") }
@@ -10,13 +10,13 @@ module Sipity
 
     subject { described_class.new }
 
-    describe 'delegations', no_clean: true do
+    describe 'delegations' do
       it { is_expected.to delegate_method(:workflow_state_name).to(:workflow_state).as(:name) }
       it { is_expected.to delegate_method(:workflow_name).to(:workflow).as(:name) }
     end
 
     describe '#proxy_for' do
-      let(:work) { FactoryBot.create(:generic_work) }
+      let(:work) { create(:generic_work) }
       let(:entity) { Sipity::Entity.new(proxy_for_global_id: work.to_global_id) }
 
       it 'will retrieve based on a GlobalID of the object' do

data/spec/models/sipity/entity_specific_responsibility_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe EntitySpecificResponsibility, type: :model, no_clean: true do
+  RSpec.describe EntitySpecificResponsibility, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include('workflow_role_id') }

data/spec/models/sipity/notifiable_context_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe NotifiableContext, type: :model, no_clean: true do
+  RSpec.describe NotifiableContext, type: :model do
     subject { described_class.new }
 
     it "should have tests"

data/spec/models/sipity/notification_recipient_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe NotificationRecipient, type: :model, no_clean: true do
+  RSpec.describe NotificationRecipient, type: :model do
     subject { described_class.new }
 
     it 'will raise an ArgumentError if you provide an invalid recipient_strategy' do

data/spec/models/sipity/notification_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe Notification, type: :model, no_clean: true do
+  RSpec.describe Notification, type: :model do
     subject { described_class.new }
 
     it 'will raise an ArgumentError if you provide an invalid #notification_type' do

data/spec/models/sipity/role_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe Role, type: :model, no_clean: true do
+  RSpec.describe Role, type: :model do
     context 'class methods' do
       subject { described_class }
 

data/spec/models/sipity/workflow_action_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe WorkflowAction, type: :model, no_clean: true do
+  RSpec.describe WorkflowAction, type: :model do
     context 'database configuration' do
       subject { described_class }
 

data/spec/models/sipity/workflow_responsibility_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe WorkflowResponsibility, type: :model, no_clean: true do
+  RSpec.describe WorkflowResponsibility, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include('agent_id') }

data/spec/models/sipity/workflow_role_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe WorkflowRole, type: :model, no_clean: true do
+  RSpec.describe WorkflowRole, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include('workflow_id') }

data/spec/models/sipity/workflow_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe Workflow, type: :model, no_clean: true do
+  RSpec.describe Workflow, type: :model do
     context 'class configuration' do
       subject { described_class }
 
@@ -33,7 +33,7 @@ module Sipity
         end.to raise_error(ActiveRecord::RecordNotFound)
       end
       context 'with :workflow_id keyword' do
-        it 'toggles current active workflows for the permission_template' do
+        it 'activates the specified workflow and deactivates the unspecified workflow for the permission_template' do
           active_workflow_wrong_template = create(:workflow, active: true, permission_template_id: other_permission_template.id)
           inactive_workflow = create(:workflow, permission_template_id: permission_template.id)
 
@@ -44,7 +44,7 @@ module Sipity
         end
       end
       context 'with :workflow_name keyword' do
-        it 'toggles current active workflows for the permission_template' do
+        it 'activates the specified workflow and deactivates the unspecified workflow for the permission_template' do
           active_workflow_wrong_template = create(:workflow, active: true, permission_template_id: other_permission_template.id)
           inactive_workflow = create(:workflow, permission_template_id: permission_template.id)
 
@@ -57,18 +57,18 @@ module Sipity
     end
 
     context '.find_active_workflow_for' do
-      let(:admin_set_id) { 1234 }
-      let(:permission_template) { create(:permission_template, admin_set_id: admin_set_id) }
+      let(:source_id) { 1234 }
+      let(:permission_template) { create(:permission_template, source_id: source_id) }
 
       it 'returns the active workflow for the permission template' do
         active_workflow = create(:workflow, active: true, permission_template: permission_template)
         _inactive_workflow = create(:workflow, active: false, permission_template: permission_template)
-        expect(described_class.find_active_workflow_for(admin_set_id: admin_set_id)).to eq(active_workflow)
+        expect(described_class.find_active_workflow_for(admin_set_id: source_id)).to eq(active_workflow)
       end
 
       it 'raises an exception when none exists' do
         create(:workflow, active: false, permission_template: permission_template)
-        expect { described_class.find_active_workflow_for(admin_set_id: admin_set_id) }.to raise_error(ActiveRecord::RecordNotFound)
+        expect { described_class.find_active_workflow_for(admin_set_id: source_id) }.to raise_error(ActiveRecord::RecordNotFound)
       end
     end
   end

data/spec/models/sipity/workflow_state_action_permission_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe WorkflowStateActionPermission, type: :model, no_clean: true do
+  RSpec.describe WorkflowStateActionPermission, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include("workflow_role_id") }

data/spec/models/sipity/workflow_state_action_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe WorkflowStateAction, type: :model, no_clean: true do
+  RSpec.describe WorkflowStateAction, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include("originating_workflow_state_id") }

data/spec/models/sipity/workflow_state_spec.rb

@@ -1,5 +1,5 @@
 module Sipity
-  RSpec.describe WorkflowState, type: :model, no_clean: true do
+  RSpec.describe WorkflowState, type: :model do
     subject { described_class }
 
     its(:column_names) { is_expected.to include("workflow_id") }

data/spec/models/solr_document_spec.rb

@@ -178,4 +178,28 @@ RSpec.describe ::SolrDocument, type: :model do
 
     it { is_expected.to eq("1234.endnote") }
   end
+
+  describe "#admin_set?" do
+    let(:attributes) { { 'has_model_ssim' => 'AdminSet' } }
+
+    subject { document }
+
+    it { is_expected.to be_admin_set }
+  end
+
+  describe "#collection?" do
+    let(:attributes) { { 'has_model_ssim' => 'Collection' } }
+
+    subject { document }
+
+    it { is_expected.to be_collection }
+  end
+
+  describe "#collection_type_gid?" do
+    let(:attributes) { { 'collection_type_gid_ssim' => 'gid://internal/hyrax-collectiontype/5' } }
+
+    subject { document.collection_type_gid }
+
+    it { is_expected.to eq 'gid://internal/hyrax-collectiontype/5' }
+  end
 end

data/spec/models/user_mailbox_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 RSpec.describe UserMailbox do
   let(:user)         { create(:user) }
   let(:another_user) { create(:user) }

data/spec/models/user_spec.rb

@@ -1,10 +1,10 @@
 RSpec.describe User, type: :model do
-  let(:user) { FactoryBot.build(:user) }
-  let(:another_user) { FactoryBot.build(:user) }
+  let(:user) { build(:user) }
+  let(:another_user) { build(:user) }
 
   describe 'verifying factories' do
     describe ':user' do
-      let(:user) { FactoryBot.build(:user) }
+      let(:user) { build(:user) }
 
       it 'will, by default, have no groups' do
         expect(user.groups).to eq([])
@@ -13,7 +13,7 @@ RSpec.describe User, type: :model do
         expect(user.class.find(user.id).groups).to eq([])
       end
       it 'will allow for override of groups' do
-        user = FactoryBot.build(:user, groups: 'chicken')
+        user = build(:user, groups: 'chicken')
         expect(user.groups).to eq(['chicken'])
         user.save!
         # Ensuring that we can refind it and have the correct groups
@@ -21,7 +21,7 @@ RSpec.describe User, type: :model do
       end
     end
     describe ':admin' do
-      let(:admin_user) { FactoryBot.create(:admin) }
+      let(:admin_user) { create(:admin) }
 
       it 'will have an "admin" group' do
         expect(admin_user.groups).to eq(['admin'])
@@ -236,8 +236,8 @@ RSpec.describe User, type: :model do
     end
   end
   describe "scope Users" do
-    let!(:basic_user) { FactoryBot.create(:user) }
-    let!(:guest_user) { FactoryBot.create(:user, :guest) }
+    let!(:basic_user) { create(:user) }
+    let!(:guest_user) { create(:user, :guest) }
     let!(:audit_user) { User.audit_user }
     let!(:batch_user) { User.batch_user }
 

data/spec/presenters/hyrax/admin/users_presenter_spec.rb

@@ -1,7 +1,7 @@
 RSpec.describe Hyrax::Admin::UsersPresenter do
   let(:instance) { described_class.new }
-  let!(:user) { FactoryBot.create(:user) }
-  let!(:admin_user) { FactoryBot.create(:user, groups: 'admin') }
+  let!(:user) { create(:user) }
+  let!(:admin_user) { create(:user, groups: 'admin') }
   let!(:audit_user) { User.audit_user }
   let!(:batch_user) { User.batch_user }
 

data/spec/presenters/hyrax/admin_set_options_presenter_spec.rb

@@ -18,8 +18,8 @@ RSpec.describe Hyrax::AdminSetOptionsPresenter do
 
       before do
         allow(presenter).to receive(:workflow) { nil }
-        create(:permission_template, admin_set_id: '123', visibility: 'open')
-        create(:permission_template, admin_set_id: '345', visibility: 'restricted')
+        create(:permission_template, source_id: '123', visibility: 'open')
+        create(:permission_template, source_id: '345', visibility: 'restricted')
       end
 
       it do
@@ -38,10 +38,10 @@ RSpec.describe Hyrax::AdminSetOptionsPresenter do
 
       before do
         allow(presenter).to receive(:workflow) { nil }
-        create(:permission_template, admin_set_id: '123', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED, release_date: today + 2.days)
-        create(:permission_template, admin_set_id: '345', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY)
-        create(:permission_template, admin_set_id: '567', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR)
-        create(:permission_template, admin_set_id: '789', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE, release_date: today + 1.month)
+        create(:permission_template, source_id: '123', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED, release_date: today + 2.days)
+        create(:permission_template, source_id: '345', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY)
+        create(:permission_template, source_id: '567', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR)
+        create(:permission_template, source_id: '789', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE, release_date: today + 1.month)
       end
 
       it do
@@ -57,7 +57,7 @@ RSpec.describe Hyrax::AdminSetOptionsPresenter do
       let(:results) { [solr_doc1] }
 
       before do
-        create(:permission_template, admin_set_id: '567')
+        create(:permission_template, source_id: '567')
       end
 
       it { is_expected.to eq [['Empty Template Set', '123', {}]] }

data/spec/presenters/hyrax/admin_set_presenter_spec.rb

@@ -50,4 +50,24 @@ RSpec.describe Hyrax::AdminSetPresenter do
       it { is_expected.to be true }
     end
   end
+
+  describe '#collection_type' do
+    let(:admin_set) do
+      build(:admin_set, id: AdminSet::DEFAULT_ID)
+    end
+
+    subject { presenter.collection_type }
+
+    it { is_expected.to eq(create(:admin_set_collection_type)) }
+  end
+
+  describe '#show_path' do
+    let(:admin_set) do
+      build(:admin_set, id: 'sb397824g')
+    end
+
+    subject { presenter.show_path }
+
+    it { is_expected.to eq "/admin/admin_sets/#{admin_set.id}" }
+  end
 end

data/spec/presenters/hyrax/collection_options_presenter_spec.rb

@@ -1,4 +1,6 @@
 RSpec.describe Hyrax::CollectionOptionsPresenter do
+  before { allow(Deprecation).to receive(:warn) }
+
   let(:instance) { described_class.new(service) }
   let(:doc1) { instance_double(SolrDocument, id: 4, to_s: 'Title 1') }
   let(:doc2) { instance_double(SolrDocument, id: 2, to_s: 'Other Title 1') }

data/spec/presenters/hyrax/collection_presenter_spec.rb

@@ -28,6 +28,47 @@ RSpec.describe Hyrax::CollectionPresenter do
   # Mock bytes so collection does not have to be saved.
   before { allow(collection).to receive(:bytes).and_return(0) }
 
+  describe "collection type methods" do
+    subject { presenter }
+
+    it { is_expected.to delegate_method(:collection_type_is_nestable?).to(:collection_type).as(:nestable?) }
+    it { is_expected.to delegate_method(:collection_type_is_discoverable?).to(:collection_type).as(:discoverable?) }
+    it { is_expected.to delegate_method(:collection_type_is_sharable?).to(:collection_type).as(:sharable?) }
+    it { is_expected.to delegate_method(:collection_type_is_share_applies_to_new_works?).to(:collection_type).as(:share_applies_to_new_works?) }
+    it { is_expected.to delegate_method(:collection_type_is_allow_multiple_membership?).to(:collection_type).as(:allow_multiple_membership?) }
+    it { is_expected.to delegate_method(:collection_type_is_require_membership?).to(:collection_type).as(:require_membership?) }
+    it { is_expected.to delegate_method(:collection_type_is_assigns_workflow?).to(:collection_type).as(:assigns_workflow?) }
+    it { is_expected.to delegate_method(:collection_type_is_assigns_visibility?).to(:collection_type).as(:assigns_visibility?) }
+
+    it "sets a default value on subcollection_counts" do
+      expect(subject.subcollection_count).to eq(0)
+    end
+
+    it { is_expected.to respond_to(:subcollection_count=).with(1).argument }
+
+    it "provides the amount of subcollections when there are none" do
+      subject.subcollection_count = nil
+      expect(subject.subcollection_count).to eq(0)
+    end
+
+    it "provides the amount of subcollections when they exist" do
+      expect(subject.subcollection_count = 5).to eq(5)
+    end
+  end
+
+  describe '#collection_type' do
+    let(:collection_type) { create(:collection_type) }
+
+    describe 'when solr_document#collection_type_gid exists' do
+      let(:collection) { build(:collection, collection_type_gid: collection_type.gid) }
+      let(:solr_doc) { SolrDocument.new(collection.to_solr) }
+
+      it 'finds the collection type based on the solr_document#collection_type_gid if one exists' do
+        expect(presenter.collection_type).to eq(collection_type)
+      end
+    end
+  end
+
   describe "#resource_type" do
     subject { presenter.resource_type }
 
@@ -98,12 +139,7 @@ RSpec.describe Hyrax::CollectionPresenter do
     end
 
     context "collection with work" do
-      let(:work) { create(:work, title: ['unimaginitive title']) }
-
-      before do
-        work.member_of_collections << collection
-        work.save!
-      end
+      let!(:work) { create(:work, member_of_collections: [collection]) }
 
       it { is_expected.to eq 1 }
     end
@@ -115,6 +151,339 @@ RSpec.describe Hyrax::CollectionPresenter do
     end
   end
 
+  describe "#total_viewable_items", :clean_repo do
+    subject { presenter.total_viewable_items }
+
+    let(:user) { create(:user) }
+
+    before do
+      allow(ability).to receive(:user_groups).and_return(['public'])
+      allow(ability).to receive(:current_user).and_return(user)
+    end
+
+    context "empty collection" do
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with private work" do
+      let!(:work) { create(:private_work, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with private collection" do
+      let!(:work) { create(:private_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with public work" do
+      let!(:work) { create(:public_work, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "collection with public collection" do
+      let!(:subcollection) { create(:public_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "collection with public work and sub-collection" do
+      let!(:work) { create(:public_work, member_of_collections: [collection]) }
+      let!(:subcollection) { create(:public_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 2 }
+    end
+
+    context "null members" do
+      let(:presenter) { described_class.new(SolrDocument.new(id: '123'), ability) }
+
+      it { is_expected.to eq 0 }
+    end
+  end
+
+  describe "#total_viewable_works", :clean_repo do
+    subject { presenter.total_viewable_works }
+
+    let(:user) { create(:user) }
+
+    before do
+      allow(ability).to receive(:user_groups).and_return(['public'])
+      allow(ability).to receive(:current_user).and_return(user)
+    end
+
+    context "empty collection" do
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with private work" do
+      let!(:work) { create(:private_work, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with public work" do
+      let!(:work) { create(:public_work, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "collection with public work and sub-collection" do
+      let!(:work) { create(:public_work, member_of_collections: [collection]) }
+      let!(:subcollection) { create(:public_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "null members" do
+      let(:presenter) { described_class.new(SolrDocument.new(id: '123'), ability) }
+
+      it { is_expected.to eq 0 }
+    end
+  end
+
+  describe "#total_viewable_collections", :clean_repo do
+    subject { presenter.total_viewable_collections }
+
+    let(:user) { create(:user) }
+
+    before do
+      allow(ability).to receive(:user_groups).and_return(['public'])
+      allow(ability).to receive(:current_user).and_return(user)
+    end
+
+    context "empty collection" do
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with private collection" do
+      let!(:subcollection) { create(:private_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 0 }
+    end
+
+    context "collection with public collection" do
+      let!(:subcollection) { create(:public_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "collection with public work and sub-collection" do
+      let!(:work) { create(:public_work, member_of_collections: [collection]) }
+      let!(:subcollection) { create(:public_collection, member_of_collections: [collection]) }
+
+      it { is_expected.to eq 1 }
+    end
+
+    context "null members" do
+      let(:presenter) { described_class.new(SolrDocument.new(id: '123'), ability) }
+
+      it { is_expected.to eq 0 }
+    end
+  end
+
+  describe "#size", :clean_repo do
+    subject { presenter.size }
+
+    it { is_expected.to eq '0 Bytes' }
+  end
+
+  describe "#parent_collection_count" do
+    subject { presenter.parent_collection_count }
+
+    let(:parent_collections) { double(Object, documents: parent_docs, response: { "numFound" => parent_docs.size }, total_pages: 1) }
+
+    context('when parent_collections is nil') do
+      before do
+        allow(presenter).to receive(:parent_collections).and_return(nil)
+      end
+
+      it { is_expected.to eq 0 }
+    end
+
+    context('when parent_collections is has no collections') do
+      let(:parent_docs) { [] }
+
+      it { is_expected.to eq 0 }
+    end
+
+    context('when parent_collections is has collections') do
+      let(:collection1) { build(:collection, title: ['col1']) }
+      let(:collection2) { build(:collection, title: ['col2']) }
+      let!(:parent_docs) { [collection1, collection2] }
+
+      before do
+        presenter.parent_collections = parent_collections
+      end
+
+      it { is_expected.to eq 2 }
+    end
+  end
+
+  describe "#more_parent_collections?" do
+    subject { presenter.more_parent_collections? }
+
+    context('when parent_collections is has no collections') do
+      before do
+        allow(presenter).to receive(:parent_collection_count).and_return(0)
+      end
+
+      it { is_expected.to eq false }
+    end
+
+    context('when parent_collections has less than or equal to show limit') do
+      before do
+        allow(presenter).to receive(:parent_collection_count).and_return(3)
+      end
+
+      it { is_expected.to eq false }
+    end
+
+    context('when parent_collections has more than show limit') do
+      before do
+        allow(presenter).to receive(:parent_collection_count).and_return(4)
+      end
+
+      it { is_expected.to eq true }
+    end
+  end
+
+  describe "#visible_parent_collections" do
+    subject { presenter.visible_parent_collections }
+
+    let(:collection1) { build(:collection, title: ['col1']) }
+    let(:collection2) { build(:collection, title: ['col2']) }
+    let(:collection3) { build(:collection, title: ['col3']) }
+    let(:collection4) { build(:collection, title: ['col4']) }
+    let(:collection5) { build(:collection, title: ['col5']) }
+    let(:parent_collections) { double(Object, documents: parent_docs, response: { "numFound" => parent_docs.size }, total_pages: 1) }
+
+    before do
+      presenter.parent_collections = parent_collections
+    end
+
+    context('when parent_collections is nil') do
+      let(:parent_collections) { nil }
+
+      it { is_expected.to eq [] }
+    end
+
+    context('when parent_collections has less than or equal to show limit') do
+      let(:parent_docs) { [collection1, collection2, collection3] }
+
+      it { is_expected.to include(collection1, collection2, collection3) }
+    end
+
+    context('when parent_collections has more than show limit') do
+      let(:parent_docs) { [collection1, collection2, collection3, collection4, collection5] }
+
+      it { is_expected.to include(collection1, collection2, collection3) }
+    end
+  end
+
+  describe "#more_parent_collections" do
+    subject { presenter.more_parent_collections }
+
+    let(:collection1) { build(:collection, title: ['col1']) }
+    let(:collection2) { build(:collection, title: ['col2']) }
+    let(:collection3) { build(:collection, title: ['col3']) }
+    let(:collection4) { build(:collection, title: ['col4']) }
+    let(:collection5) { build(:collection, title: ['col5']) }
+    let(:parent_collections) { double(Object, documents: parent_docs, response: { "numFound" => parent_docs.size }, total_pages: 1) }
+
+    before do
+      presenter.parent_collections = parent_collections
+    end
+
+    context('when parent_collections is nil') do
+      let(:parent_collections) { nil }
+
+      it { is_expected.to eq [] }
+    end
+
+    context('when parent_collections is empty') do
+      let(:parent_docs) { [] }
+
+      it { is_expected.to eq [] }
+    end
+
+    context('when parent_collections has less than or equal to show limit') do
+      let(:parent_docs) { [collection1, collection2, collection3] }
+
+      it { is_expected.to eq [] }
+    end
+
+    context('when parent_collections has more than show limit') do
+      let(:parent_docs) { [collection1, collection2, collection3, collection4, collection5] }
+
+      it { is_expected.to include(collection4, collection5) }
+    end
+  end
+
+  describe "#user_can_nest_collection?" do
+    before do
+      allow(ability).to receive(:can?).with(:deposit, solr_doc).and_return(true)
+    end
+
+    subject { presenter.user_can_nest_collection? }
+
+    it { is_expected.to eq true }
+  end
+
+  describe "#user_can_create_new_nest_collection?" do
+    before do
+      allow(ability).to receive(:can?).with(:create_collection_of_type, collection.collection_type).and_return(true)
+    end
+
+    subject { presenter.user_can_create_new_nest_collection? }
+
+    it { is_expected.to eq true }
+  end
+
+  describe '#show_path' do
+    subject { presenter.show_path }
+
+    it { is_expected.to eq "/dashboard/collections/#{solr_doc.id}" }
+  end
+
+  describe "banner_file" do
+    let(:banner_info) do
+      CollectionBrandingInfo.new(
+        collection_id: "123",
+        filename: "banner.gif",
+        role: "banner",
+        target_url: ""
+      )
+    end
+
+    let(:logo_info) do
+      CollectionBrandingInfo.new(
+        collection_id: "123",
+        filename: "logo.gif",
+        role: "logo",
+        alt_txt: "This is the logo",
+        target_url: "http://logo.com"
+      )
+    end
+
+    before do
+      allow(presenter).to receive(:id).and_return('123')
+      allow(CollectionBrandingInfo).to receive(:where).with(collection_id: '123', role: 'banner').and_return([banner_info])
+      allow(banner_info).to receive(:local_path).and_return("/temp/public/branding/123/banner/banner.gif")
+      allow(CollectionBrandingInfo).to receive(:where).with(collection_id: '123', role: 'logo').and_return([logo_info])
+      allow(logo_info).to receive(:local_path).and_return("/temp/public/branding/123/logo/logo.gif")
+    end
+
+    it "banner check" do
+      expect(presenter.banner_file).to eq("/branding/123/banner/banner.gif")
+    end
+
+    it "logo check" do
+      expect(presenter.logo_record).to eq([{ file: "logo.gif", file_location: "/branding/123/logo/logo.gif", alttext: "This is the logo", linkurl: "http://logo.com" }])
+    end
+  end
+
   subject { presenter }
 
   it { is_expected.to delegate_method(:resource_type).to(:solr_document) }