hyrax 2.0.3 → 2.1.0.beta1

data/spec/features/admin_spec.rb

@@ -17,7 +17,7 @@ RSpec.describe "The admin dashboard", :clean_repo do
     create(:work_with_two_children, title: ["Work C"], admin_set_id: admin_set_2.id, edit_users: [user])
   end
 
-  scenario do
+  it do
     login_as(user, scope: :user)
     visit '/dashboard'
 

data/spec/features/batch_create_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-RSpec.feature 'Batch creation of works', type: :feature do
+RSpec.describe 'Batch creation of works', type: :feature do
   let(:user) { create(:user) }
 
   before do

data/spec/features/batch_edit_spec.rb

@@ -65,7 +65,6 @@ RSpec.describe 'batch', type: :feature, clean_repo: true, js: true do
   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

data/spec/features/browse_catalog_spec.rb

@@ -1,6 +1,4 @@
-# coding: utf-8
-
-RSpec.describe "Browse catalog:", type: :feature do
+RSpec.describe "Browse catalog:", type: :feature, clean_repo: true do
   let!(:jills_work) do
     GenericWork.new do |work|
       work.title = ["Jill's Research"]

data/spec/features/browse_dashboard_works_spec.rb

@@ -50,7 +50,6 @@ RSpec.describe "Browse Dashboard", type: :feature 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/collection_multi_membership_spec.rb

@@ -0,0 +1,188 @@
+RSpec.describe 'Adding a work to multiple collections', type: :feature, clean_repo: true, js: true, with_nested_reindexing: true do
+  let(:admin_user) { create(:admin, email: 'admin@example.com') }
+  let(:single_membership_type_1) { create(:collection_type, :not_allow_multiple_membership, title: 'Single-membership 1') }
+  let(:single_membership_type_2) { create(:collection_type, :not_allow_multiple_membership, title: 'Single-membership 2') }
+  let(:multi_membership_type_1) { create(:collection_type, :allow_multiple_membership, title: 'Multi-membership 1') }
+  let(:multi_membership_type_2) { create(:collection_type, :allow_multiple_membership, title: 'Multi-membership 2') }
+
+  before do
+    sign_in admin_user
+  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) }
+    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
+      let!(:new_collection) { create(:collection_lw, user: admin_user, collection_type_gid: multi_membership_type_2.gid) }
+
+      it 'then the work is added to both collections' do
+        # Add to second multi-membership collection of a different type
+        visit '/dashboard/my/works'
+        check 'check_all'
+        click_button 'Add to collection' # opens the modal
+        within('div#collection-list-container') do
+          choose new_collection.title.first # selects the collection
+          click_button 'Save changes'
+        end
+
+        # forwards to collection show page
+        expect(page).to have_content new_collection.title.first
+        expect(page).to have_content 'Works (1)'
+        expect(page).to have_content work.title.first
+        expect(page).to have_selector '.alert-success', text: 'Collection was successfully updated.'
+      end
+    end
+
+    context 'and are of the same type' do
+      let!(:new_collection) { create(:collection_lw, user: admin_user, collection_type_gid: multi_membership_type_1.gid) }
+
+      it 'then the work is added to both collections' do
+        # Add to second multi-membership collection of a different type
+        visit '/dashboard/my/works'
+        check 'check_all'
+        click_button 'Add to collection' # opens the modal
+        within('div#collection-list-container') do
+          choose new_collection.title.first # selects the collection
+          click_button 'Save changes'
+        end
+
+        # forwards to collection show page
+        expect(page).to have_content new_collection.title.first
+        expect(page).to have_content 'Works (1)'
+        expect(page).to have_content work.title.first
+        expect(page).to have_selector '.alert-success', text: 'Collection was successfully updated.'
+      end
+    end
+  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) }
+    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
+      let!(:new_collection) { create(:collection_lw, user: admin_user, collection_type_gid: single_membership_type_2.gid) }
+
+      it 'then the work is added to both collections' do
+        # Add to second single-membership collection of a different type
+        visit '/dashboard/my/works'
+        check 'check_all'
+        click_button 'Add to collection' # opens the modal
+        within('div#collection-list-container') do
+          choose new_collection.title.first # selects the collection
+          click_button 'Save changes'
+        end
+        # forwards to collection show page
+        expect(page).to have_content new_collection.title.first
+        expect(page).to have_content 'Works (1)'
+        expect(page).to have_content work.title.first
+        expect(page).to have_selector '.alert-success', text: 'Collection was successfully updated.'
+      end
+    end
+
+    context 'and are of the same type' do
+      let!(:new_collection) { create(:collection_lw, user: admin_user, collection_type_gid: single_membership_type_1.gid) }
+
+      context 'then the work fails to add to the second collection' do
+        it 'from the dashboard->works batch add to collection' do
+          # Attempt to add to second single-membership collection of the same type
+          visit '/dashboard/my/works'
+          check 'check_all'
+          click_button 'Add to collection' # opens the modal
+          within('div#collection-list-container') do
+            choose new_collection.title.first # selects the collection
+            click_button 'Save changes'
+          end
+          # forwards to collections index page and shows flash message
+          expect(page).to have_link 'All Collections'
+          expect(page).to have_link 'Your Collections'
+
+          err_message = "Error: You have specified more than one of the same single-membership collection types: " \
+                        "Single-membership 1 (#{new_collection.title.first} and #{old_collection.title.first})"
+          expect(page).to have_selector '.alert', text: err_message
+        end
+
+        it "from the work's edit form Relationships tab" do
+          skip 'Needs additional work to find and select the second collection'
+          # Attempt to add to second single-membership collection of the same type
+          visit edit_hyrax_generic_work_path(work)
+          click_link "Relationships"
+          # TODO: not sure how to find and select the correct collection
+
+          # forwards to work index page and shows flash message
+          expect(page).to have_link 'All Works'
+          expect(page).to have_link 'Your Works'
+
+          err_message = "Error: You have specified more than one of the same single-membership collection types: " \
+                        "Single-membership 1 (Collection 1 for SM1)"
+          expect(page).to have_selector '.alert', text: err_message
+        end
+
+        it "from the collection's show page Add to collection" do
+          # Attempt to add to second single-membership collection of the same type
+          visit "/dashboard/collections/#{new_collection.id}"
+          click_link 'Add existing works'
+          check 'check_all'
+          click_button 'Add to collection' # opens the modal
+          within('div#collection-list-container') do
+            choose new_collection.title.first # selects the collection
+            click_button 'Save changes'
+          end
+          # forwards to collections index page and shows flash message
+          expect(page).to have_link 'All Collections'
+          expect(page).to have_link 'Your Collections'
+
+          err_message = "Error: You have specified more than one of the same single-membership collection types: " \
+                        "Single-membership 1 (#{new_collection.title.first} and #{old_collection.title.first})"
+          expect(page).to have_selector '.alert', text: err_message
+        end
+      end
+    end
+  end
+
+  describe 'when adding a work already in a collection' do
+    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 'allowing multi-membership' do
+      let(:old_collection) { create(:collection_lw, user: admin_user, collection_type_gid: multi_membership_type_1.gid) }
+      let!(:new_collection) { old_collection }
+
+      it 'then the add is treated as a success' do
+        # Re-add to same multi-membership collection
+        visit '/dashboard/my/works'
+        check 'check_all'
+        click_button 'Add to collection' # opens the modal
+        within('div#collection-list-container') do
+          choose new_collection.title.first # selects the collection
+          click_button 'Save changes'
+        end
+        # forwards to collection show page
+        expect(page).to have_content new_collection.title.first
+        expect(page).to have_content 'Works (1)'
+        expect(page).to have_content work.title.first
+        expect(page).to have_selector '.alert-success', text: 'Collection was successfully updated.'
+      end
+    end
+
+    context 'requiring single-membership' do
+      let(:old_collection) { create(:collection_lw, user: admin_user, collection_type_gid: single_membership_type_1.gid) }
+      let!(:new_collection) { old_collection }
+
+      it 'then the add is treated as a success' do
+        # Re-add to same single-membership collection
+        visit '/dashboard/my/works'
+        check 'check_all'
+        click_button 'Add to collection' # opens the modal
+        within('div#collection-list-container') do
+          choose new_collection.title.first # selects the collection
+          click_button 'Save changes'
+        end
+        # forwards to collection show page
+        expect(page).to have_content new_collection.title.first
+        expect(page).to have_content 'Works (1)'
+        expect(page).to have_content work.title.first
+        expect(page).to have_selector '.alert-success', text: 'Collection was successfully updated.'
+      end
+    end
+  end
+end

data/spec/features/collection_spec.rb

@@ -1,4 +1,4 @@
-RSpec.describe 'collection', type: :feature do
+RSpec.describe 'collection', type: :feature, with_nested_reindexing: true do
   let(:user) { create(:user) }
 
   let(:collection1) { create(:public_collection, user: user) }
@@ -6,10 +6,12 @@ RSpec.describe 'collection', type: :feature do
 
   describe 'collection show page' do
     let(:collection) do
-      create(:public_collection, user: user, description: ['collection description'])
+      create(:public_collection, 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) }
 
     before do
       sign_in user
@@ -19,15 +21,16 @@ RSpec.describe 'collection', type: :feature do
     it "shows a collection with a listing of Descriptive Metadata and catalog-style search results" do
       expect(page).to have_content(collection.title.first)
       expect(page).to have_content(collection.description.first)
+      expect(page).to have_content("Collection Details")
       # Should not show title and description a second time
       expect(page).not_to have_css('.metadata-collections', text: collection.title.first)
       expect(page).not_to have_css('.metadata-collections', text: collection.description.first)
-      # Should not have Collection Descriptive metadata table
-      expect(page).to have_content("Descriptions")
       # Should have search results / contents listing
       expect(page).to have_content(work1.title.first)
       expect(page).to have_content(work2.title.first)
-      expect(page).not_to have_css(".pager")
+      expect(page).to have_content(col1.title.first)
+      expect(page).to have_content(col2.title.first)
+      expect(page).not_to have_css(".pagination")
 
       click_link "Gallery"
       expect(page).to have_content(work1.title.first)
@@ -50,16 +53,51 @@ RSpec.describe 'collection', type: :feature do
       expect(page).to have_content(work1.title.first)
       expect(page).not_to have_content(work2.title.first)
     end
+
+    context "with a non-nestable collection type" do
+      let(:collection) do
+        build(:public_collection_lw, user: user, description: ['collection description'], collection_type_settings: :not_nestable, with_solr_document: true, with_permission_template: true)
+      end
+
+      it "displays basic information on its show page" do
+        expect(page).to have_content(collection.title.first)
+        expect(page).to have_content(collection.description.first)
+        expect(page).to have_content("Collection Details")
+      end
+    end
   end
 
   # TODO: this is just like the block above. Merge them.
-  describe 'show pages of a collection' do
+  describe 'show work pages of a collection' do
     before do
       docs = (0..12).map do |n|
         { "has_model_ssim" => ["GenericWork"], :id => "zs25x871q#{n}",
           "depositor_ssim" => [user.user_key],
           "suppressed_bsi" => false,
           "member_of_collection_ids_ssim" => [collection.id],
+          "nesting_collection__parent_ids_ssim" => [collection.id],
+          "edit_access_person_ssim" => [user.user_key] }
+      end
+      ActiveFedora::SolrService.add(docs, commit: true)
+
+      sign_in user
+    end
+    let(:collection) { create(:named_collection, user: user) }
+
+    it "shows a collection with a listing of Descriptive Metadata and catalog-style search results" do
+      visit "/collections/#{collection.id}"
+      expect(page).to have_css(".pagination")
+    end
+  end
+
+  describe 'show subcollection pages of a collection' do
+    before do
+      docs = (0..12).map do |n|
+        { "has_model_ssim" => ["Collection"], :id => "zs25x871q#{n}",
+          "depositor_ssim" => [user.user_key],
+          "suppressed_bsi" => false,
+          "member_of_collection_ids_ssim" => [collection.id],
+          "nesting_collection__parent_ids_ssim" => [collection.id],
           "edit_access_person_ssim" => [user.user_key] }
       end
       ActiveFedora::SolrService.add(docs, commit: true)
@@ -70,7 +108,7 @@ RSpec.describe 'collection', type: :feature do
 
     it "shows a collection with a listing of Descriptive Metadata and catalog-style search results" do
       visit "/collections/#{collection.id}"
-      expect(page).to have_css(".pager")
+      expect(page).to have_css(".pagination")
     end
   end
 end

data/spec/features/collection_type_spec.rb

@@ -0,0 +1,338 @@
+RSpec.describe 'collection_type', type: :feature, clean_repo: true do
+  let(:admin_user) { create(:admin) }
+  let(:exhibit_title) { 'Exhibit' }
+  let(:exhibit_description) { 'Description for exhibit collection type.' }
+  let(:exhibit_collection_type) { create(:collection_type, title: exhibit_title, description: exhibit_description, creator_user: admin_user) }
+  let(:user_collection_type) { create(:user_collection_type) }
+  let(:admin_set_type) { create(:admin_set_collection_type) }
+  let(:solr_gid) { Collection.collection_type_gid_document_field_name }
+
+  describe 'index' do
+    before do
+      exhibit_collection_type
+      user_collection_type
+      admin_set_type
+      sign_in admin_user
+      visit '/admin/collection_types'
+    end
+
+    it 'has page title and lists collection types' do
+      expect(page).to have_content 'Collection Types'
+
+      expect(page).to have_content 'Admin Set'
+      expect(page).to have_content 'User Collection'
+      expect(page).to have_content 'Collection Type'
+
+      expect(page).to have_link('Edit', count: 3)
+      expect(page).to have_link('Edit', href: hyrax.edit_admin_collection_type_path(admin_set_type.id, locale: 'en'))
+      expect(page).to have_link('Edit', href: hyrax.edit_admin_collection_type_path(user_collection_type.id, locale: 'en'))
+      expect(page).to have_link('Edit', href: hyrax.edit_admin_collection_type_path(exhibit_collection_type.id, locale: 'en'))
+      expect(page).to have_button('Delete', count: 2) # 1: Collection Type, 2: delete modal
+    end
+  end
+
+  describe 'create collection type' do
+    let(:title) { 'Test Type' }
+    let(:description) { 'Description for collection type we are testing.' }
+
+    before do
+      sign_in admin_user
+      visit '/admin/collection_types'
+    end
+
+    it 'makes a new collection type', :js do
+      click_link 'Create new collection type'
+
+      expect(page).to have_content 'Create New Collection Type'
+
+      # confirm only Description tab is visible
+      expect(page).to have_link('Description', href: '#metadata')
+      expect(page).not_to have_link('Settings', href: '#settings')
+      expect(page).not_to have_link('Participants', href: '#participants')
+
+      # confirm metadata fields exist
+      expect(page).to have_selector 'input#collection_type_title'
+      expect(page).to have_selector 'textarea#collection_type_description'
+
+      # set values and save
+      fill_in('Type name', with: title)
+      fill_in('Type description', with: description)
+
+      click_button('Save')
+
+      # confirm values were set
+      expect(page).to have_selector "input#collection_type_title[value='#{title}']"
+      expect(page).to have_selector 'textarea#collection_type_description', text: description
+
+      # confirm all edit tabs are now visible
+      expect(page).to have_link('Description', href: '#metadata')
+      expect(page).to have_link('Settings', href: '#settings')
+      expect(page).to have_link('Participants', href: '#participants')
+    end
+  end
+
+  describe 'edit collection type' do
+    context 'when there are no collections of this type' do
+      let(:title_old) { exhibit_title }
+      let(:description_old) { exhibit_description }
+      let(:title_new) { 'Exhibit modified' }
+      let(:description_new) { 'Change in description for exhibit collection type.' }
+
+      before do
+        exhibit_collection_type
+        sign_in admin_user
+        visit "/admin/collection_types/#{exhibit_collection_type.id}/edit"
+      end
+
+      it 'modifies metadata values of a collection type', :js do
+        expect(page).to have_content "Edit Collection Type: #{title_old}"
+
+        # confirm all tabs are visible
+        expect(page).to have_link('Description', href: '#metadata')
+        expect(page).to have_link('Settings', href: '#settings')
+        expect(page).to have_link('Participants', href: '#participants')
+
+        # confirm metadata fields have original values
+        expect(page).to have_selector "input#collection_type_title[value='#{title_old}']"
+        expect(page).to have_selector 'textarea#collection_type_description', text: description_old
+
+        # set values and save
+        fill_in('Type name', with: title_new)
+        fill_in('Type description', with: description_new)
+
+        click_button('Save changes')
+
+        expect(page).to have_content "Edit Collection Type: #{title_new}"
+
+        # confirm values were set
+        expect(page).to have_selector "input#collection_type_title[value='#{title_new}']"
+        expect(page).to have_selector 'textarea#collection_type_description', text: description_new
+
+        click_link('Settings', href: '#settings')
+
+        # confirm all non-admin_set checkboxes are on
+        expect(page).to have_checked_field('collection_type_nestable')
+        expect(page).to have_checked_field('collection_type_discoverable')
+        expect(page).to have_checked_field('collection_type_sharable')
+        expect(page).to have_checked_field('collection_type_share_applies_to_new_works')
+        expect(page).to have_checked_field('collection_type_allow_multiple_membership')
+
+        # confirm all admin_set only checkboxes are off and disabled
+        expect(page).to have_unchecked_field('collection_type_require_membership', disabled: true)
+        expect(page).to have_unchecked_field('collection_type_assigns_workflow', disabled: true)
+        expect(page).to have_unchecked_field('collection_type_assigns_visibility', disabled: true)
+
+        # change settings
+        page.uncheck('NESTABLE')
+        page.uncheck('DISCOVERY')
+        page.check('APPLY TO NEW WORKS')
+        page.uncheck('MULTIPLE MEMBERSHIP')
+
+        # confirm all non-admin_set checkboxes are now off
+        expect(page).to have_unchecked_field('collection_type_nestable')
+        expect(page).to have_unchecked_field('collection_type_discoverable')
+        expect(page).to have_checked_field('collection_type_sharable')
+        expect(page).to have_checked_field('collection_type_share_applies_to_new_works')
+        expect(page).to have_unchecked_field('collection_type_allow_multiple_membership')
+
+        # uncheck sharable should disable sharable options
+        page.uncheck('SHARING')
+
+        expect(page).to have_unchecked_field('collection_type_sharable')
+        expect(page).to have_unchecked_field('collection_type_share_applies_to_new_works', disabled: true)
+
+        # check sharable should enable sharable options
+        page.check('SHARING')
+
+        expect(page).to have_checked_field('collection_type_sharable')
+        expect(page).to have_unchecked_field('collection_type_share_applies_to_new_works', disabled: false)
+
+        click_link('Participants')
+
+        # TODO: Test adding participants
+      end
+
+      context 'when editing default user collection type' do
+        let(:title_old) { user_collection_type.title }
+        let(:description_old) { user_collection_type.description }
+        let(:title_new) { 'User Collection modified' }
+        let(:description_new) { 'Change in description for user collection type.' }
+
+        before do
+          user_collection_type
+          sign_in admin_user
+          visit "/admin/collection_types/#{user_collection_type.id}/edit"
+        end
+
+        it 'allows editing of metadata, but not settings', :js do
+          expect(page).to have_content "Edit Collection Type: #{title_old}"
+
+          # confirm metadata fields have original values
+          expect(page).to have_selector "input#collection_type_title[value='#{title_old}']"
+          expect(page).to have_selector 'textarea#collection_type_description', text: description_old
+
+          # set values and save
+          fill_in('Type name', with: title_new)
+          fill_in('Type description', with: description_new)
+
+          click_button('Save changes')
+
+          expect(page).to have_content "Edit Collection Type: #{title_new}"
+
+          # confirm values were set
+          expect(page).to have_selector "input#collection_type_title[value='#{title_new}']"
+          expect(page).to have_selector 'textarea#collection_type_description', text: description_new
+
+          click_link('Settings', href: '#settings')
+
+          # confirm default user collection checkboxes are set to appropriate values
+          expect(page).to have_checked_field('collection_type_nestable', disabled: true)
+          expect(page).to have_checked_field('collection_type_discoverable', disabled: true)
+          expect(page).to have_checked_field('collection_type_sharable', disabled: true)
+          expect(page).to have_unchecked_field('collection_type_share_applies_to_new_works', disabled: true)
+          expect(page).to have_checked_field('collection_type_allow_multiple_membership', disabled: true)
+
+          # confirm all admin_set only checkboxes are off and disabled
+          expect(page).to have_unchecked_field('collection_type_require_membership', disabled: true)
+          expect(page).to have_unchecked_field('collection_type_assigns_workflow', disabled: true)
+          expect(page).to have_unchecked_field('collection_type_assigns_visibility', disabled: true)
+        end
+      end
+
+      context 'when editing admin set collection type' do
+        let(:title_old) { admin_set_type.title }
+        let(:description_old) { admin_set_type.description }
+        let(:description_new) { 'Change in description for admin set collection type.' }
+
+        before do
+          admin_set_type
+          sign_in admin_user
+          visit "/admin/collection_types/#{admin_set_type.id}/edit"
+        end
+
+        it 'allows editing of metadata except title, but not settings', :js do
+          expect(page).to have_content "Edit Collection Type: #{title_old}"
+
+          # confirm metadata fields have original values
+          expect(page).to have_field("collection_type_title", disabled: true)
+          expect(page).to have_selector 'textarea#collection_type_description', text: description_old
+
+          # set values and save
+          fill_in('Type description', with: description_new)
+
+          click_button('Save changes')
+
+          expect(page).to have_content "Edit Collection Type: #{title_old}"
+
+          # confirm values were set
+          expect(page).to have_selector 'textarea#collection_type_description', text: description_new
+
+          click_link('Settings', href: '#settings')
+
+          # confirm default user collection checkboxes are set to appropriate values
+          expect(page).to have_unchecked_field('collection_type_nestable', disabled: true)
+          expect(page).to have_unchecked_field('collection_type_discoverable', disabled: true)
+          expect(page).to have_checked_field('collection_type_sharable', disabled: true)
+          expect(page).to have_checked_field('collection_type_share_applies_to_new_works', disabled: true)
+          expect(page).to have_unchecked_field('collection_type_allow_multiple_membership', disabled: true)
+
+          # confirm all admin_set only checkboxes are off and disabled
+          expect(page).to have_checked_field('collection_type_require_membership', disabled: true)
+          expect(page).to have_checked_field('collection_type_assigns_workflow', disabled: true)
+          expect(page).to have_checked_field('collection_type_assigns_visibility', disabled: true)
+        end
+      end
+    end
+
+    context 'when collections exist of this type' do
+      let!(:collection1) { create(:public_collection, user: build(:user), collection_type_gid: exhibit_collection_type.gid) }
+
+      before do
+        exhibit_collection_type
+        sign_in admin_user
+        visit "/admin/collection_types/#{exhibit_collection_type.id}/edit"
+      end
+
+      it 'all settings are disabled', :js do
+        expect(exhibit_collection_type.collections?).to be true
+
+        click_link('Settings', href: '#settings')
+
+        # confirm all checkboxes are disabled
+        expect(page).to have_field('collection_type_nestable', disabled: true)
+        expect(page).to have_field('collection_type_discoverable', disabled: true)
+        expect(page).to have_field('collection_type_sharable', disabled: true)
+        expect(page).to have_field('collection_type_share_applies_to_new_works', disabled: true)
+        expect(page).to have_field('collection_type_allow_multiple_membership', disabled: true)
+        expect(page).to have_field('collection_type_require_membership', disabled: true)
+        expect(page).to have_field('collection_type_assigns_workflow', disabled: true)
+        expect(page).to have_field('collection_type_assigns_visibility', disabled: true)
+      end
+    end
+  end
+
+  describe 'delete collection type' do
+    context 'when there are no collections of this type' do
+      let!(:empty_collection_type) { create(:collection_type, title: 'Empty Type', creator_user: admin_user) }
+      let!(:delete_modal_text) { 'Deleting this collection type will permanently remove the type and its settings from the repository.  Are you sure you want to delete this collection type?' }
+
+      before do
+        sign_in admin_user
+        visit '/admin/collection_types'
+      end
+
+      it 'shows warning and deletes collection type', :js do
+        expect(page).to have_content(empty_collection_type.title)
+
+        find(:xpath, "//tr[td[contains(.,'#{empty_collection_type.title}')]]/td/button", text: 'Delete').click
+
+        within('div#deleteModal') do
+          expect(page).to have_content(delete_modal_text)
+          click_button('Delete')
+        end
+
+        expect(page).not_to have_content(empty_collection_type.title)
+      end
+    end
+
+    context 'when collections exist of this type' do
+      let!(:not_empty_collection_type) { create(:collection_type, title: 'Not Empty Type', creator_user: admin_user) }
+      let!(:collection1) { create(:public_collection, user: admin_user, collection_type_gid: not_empty_collection_type.gid) }
+      let(:deny_delete_modal_text) do
+        'You cannot delete this collection type because one or more collections of this type have already been created. ' \
+        'To delete this collection type, first ensure that all collections of this type have been deleted.'
+      end
+
+      before do
+        sign_in admin_user
+        visit '/admin/collection_types'
+      end
+
+      it 'shows unable to delete dialog and forwards to All Collections with filter applied', :js do
+        expect(page).to have_content(not_empty_collection_type.title)
+
+        find(:xpath, "//tr[td[contains(.,'#{not_empty_collection_type.title}')]]/td/button", text: 'Delete').click
+
+        within('div#deleteDenyModal') do
+          expect(page).to have_content(deny_delete_modal_text)
+          click_link('View collections of this type')
+        end
+
+        # forwards to Dashboard -> Collections -> All Collections
+        within('li.active') do
+          expect(page).to have_link('All Collections')
+        end
+
+        # filter is applied
+        within('div#appliedParams') do
+          expect(page).to have_content('Filtering by:')
+          expect(page).to have_content('Type')
+          expect(page).to have_content(not_empty_collection_type.title)
+        end
+
+        # collection of this type is in the list of collections
+        expect(page).to have_content(collection1.title.first)
+      end
+    end
+  end
+end