hyrax 2.9.6 → 3.0.0.pre.beta1

data/spec/views/hyrax/dashboard/collections/_sort_and_per_page.html.erb_spec.rb

@@ -0,0 +1,60 @@
+RSpec.describe 'hyrax/dashboard/collections/_sort_and_per_page.html.erb', type: :view do
+  let(:subject) { 'hyrax/dashboard/collections/sort_and_per_page.html.erb' }
+  let(:collection) { instance_double(Collection) }
+
+  before do
+    allow(view).to receive(:show_sort_and_per_page?).and_return(true)
+    allow(view).to receive(:dashboard_collection_path).and_return("collection/path")
+    stub_template('_view_type_group.erb' => "")
+  end
+
+  context "when there are multiple sort fields" do
+    let(:active_sort_fields) do
+      {
+        "sort field value 1" => Blacklight::Configuration::SortField.new(label: "sort field label 1"),
+        "sort field value 2" => Blacklight::Configuration::SortField.new(label: "sort field label 2")
+      }
+    end
+
+    it "renders the sort options without any selected when no sort param given" do
+      render(subject, collection: collection, active_sort_fields: active_sort_fields)
+      expect(rendered).to have_select('sort', options: ["sort field label 1", "sort field label 2"], with_selected: [])
+    end
+
+    it "renders the sort options with the correct option selected when a valid sort param given" do
+      allow(view).to receive(:params).and_return(sort: "sort field value 1")
+      render(subject, collection: collection, active_sort_fields: active_sort_fields)
+      expect(rendered).to have_select('sort', options: ["sort field label 1", "sort field label 2"], with_selected: ["sort field label 1"])
+    end
+
+    it "renders the sort options without any selected when an invalid sort param given" do
+      allow(view).to receive(:params).and_return(sort: "sort field value DNE")
+      render(subject, collection: collection, active_sort_fields: active_sort_fields)
+      expect(rendered).to have_select('sort', options: ["sort field label 1", "sort field label 2"], with_selected: [])
+    end
+  end
+
+  context "when there is only one sort field" do
+    let(:active_sort_fields) do
+      {
+        "sort field value 1" => Blacklight::Configuration::SortField.new(label: "sort field label 1")
+      }
+    end
+
+    it "does not render sort options" do
+      render(subject, collection: collection, active_sort_fields: active_sort_fields)
+      expect(rendered).not_to have_select('sort')
+    end
+  end
+
+  context "when there are no sort fields" do
+    let(:active_sort_fields) do
+      {}
+    end
+
+    it "does not render sort options" do
+      render(subject, collection: collection, active_sort_fields: active_sort_fields)
+      expect(rendered).not_to have_select('sort')
+    end
+  end
+end

data/spec/views/hyrax/dashboard/collections/_subcollection_list.html.erb_spec.rb

@@ -0,0 +1,75 @@
+RSpec.describe 'hyrax/dashboard/collections/_subcollection_list.html.erb', type: :view do
+  let(:user) { create :user }
+  let(:ability) { instance_double("Ability") }
+  let(:collection) { stub_model(Collection, id: '123') }
+  let(:subject) { render('subcollection_list.html.erb', id: collection.id, collection: subcollection) }
+
+  before do
+    allow(view).to receive(:id).and_return(collection.id)
+    allow(view).to receive(:current_ability).and_return(ability)
+    allow(controller).to receive(:can?).with(:edit, collection.id).and_return true
+  end
+
+  context 'when subcollection list is empty' do
+    let(:subcollection) { nil }
+
+    before do
+      assign(:subcollection_docs, subcollection)
+    end
+
+    it "posts a warning message" do
+      render('subcollection_list.html.erb', collection: subcollection)
+      expect(rendered).to have_text("There are no visible subcollections.")
+    end
+  end
+
+  context 'when subcollection list is not empty' do
+    let(:subcollection) { [collection] }
+
+    before do
+      assign(:subcollection_docs, subcollection)
+      assign(:document, collection)
+      allow(collection).to receive(:title_or_label).and_return(collection.title)
+      # make the collection "persisted" so the route returned is valid for show
+      allow(collection).to receive(:persisted?).and_return true
+      stub_template "hyrax/collections/_paginate" => "<div>paginate</div>"
+    end
+
+    context 'when user has edit access to the collection' do
+      before do
+        allow(controller).to receive(:can?).with(:edit, collection.id).and_return true
+      end
+
+      it "includes link to the collection and remove button" do
+        subject
+        expect(rendered).to have_link(collection.title.to_s)
+        expect(subject).to render_template('_modal_remove_sub_collection')
+      end
+
+      it "renders the proper data attributes on list element" do
+        expect(subject).to have_selector(:css, 'li[data-post-url="/dashboard/collections/123/remove_child/123"]')
+        expect(subject).to have_selector(:css, 'li[data-id="123"]')
+      end
+
+      it 'renders pagination' do
+        expect(subject).to render_template("hyrax/collections/_paginate")
+      end
+    end
+
+    context 'when user has no edit access to the collection' do
+      before do
+        allow(controller).to receive(:can?).with(:edit, collection.id).and_return false
+      end
+
+      it "includes link to the collection and no remove button" do
+        subject
+        expect(rendered).to have_link(collection.title.to_s)
+        expect(subject).not_to render_template('_modal_remove_sub_collection')
+      end
+
+      it 'renders pagination' do
+        expect(subject).to render_template("hyrax/collections/_paginate")
+      end
+    end
+  end
+end

data/spec/views/hyrax/dashboard/collections/_work_action_menu.html.erb_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+RSpec.describe 'hyrax/dashboard/collections/_work_action_menu.html.erb', type: :view do
+  let(:item) { stub_model(GenericWork) }
+  let(:collection) { stub_model(Collection) }
+
+  before do
+    allow(view).to receive(:display_trophy_link)
+    assign(:collection, collection)
+    render 'hyrax/dashboard/collections/work_action_menu', document: item
+  end
+
+  it "generates a form that can remove the item" do
+    expect(rendered).to have_selector("form[action=\"#{hyrax.dashboard_collection_path(collection)}\"]")
+    expect(rendered).to have_selector('input#collection_members[type="hidden"][value="remove"]', visible: false)
+    expect(rendered).to have_selector("input[type='hidden'][name='batch_document_ids[]'][value='#{item.id}']", visible: false)
+  end
+end

data/spec/views/hyrax/dashboard/collections/edit.html.erb_spec.rb

@@ -0,0 +1,20 @@
+RSpec.describe 'hyrax/dashboard/collections/edit.html.erb', type: :view do
+  let(:collection_type) { stub_model(Hyrax::CollectionType) }
+  let(:collection) { stub_model(Collection, id: 'xyz123z4', title: ["Make Collections Great Again"]) }
+  let(:form) { Hyrax::Forms::CollectionForm.new(collection, double, double) }
+
+  before do
+    assign(:collection, collection)
+    assign(:form, form)
+    allow(collection).to receive(:collection_type).and_return(collection_type)
+    stub_template '_form.html.erb' => 'my-edit-form partial'
+    stub_template '_flash_msg.html.erb' => 'flash_msg partial'
+
+    render
+  end
+
+  it 'displays the page' do
+    expect(rendered).to have_content 'my-edit-form partial'
+    expect(rendered).to have_content 'flash_msg partial'
+  end
+end

data/spec/views/hyrax/dashboard/collections/show.html.erb_spec.rb

@@ -0,0 +1,116 @@
+RSpec.describe 'hyrax/dashboard/collections/show.html.erb', type: :view do
+  let(:document) do
+    SolrDocument.new(id: 'xyz123z4',
+                     'title_tesim' => ['Make Collections Great Again'],
+                     'rights_tesim' => ["http://creativecommons.org/licenses/by-sa/3.0/us/"])
+  end
+  let(:ability) { double }
+  let(:collection) { mock_model(Collection) }
+  let(:presenter) { Hyrax::CollectionPresenter.new(document, ability) }
+  let(:collection_type) { double }
+
+  before do
+    user = stub_model(User)
+    allow(document).to receive(:hydra_model).and_return(::Collection)
+    allow(view).to receive(:current_user).and_return(user)
+    allow(view).to receive(:can?).with(:edit, document).and_return(true)
+    allow(view).to receive(:can?).with(:destroy, document).and_return(true)
+    allow(view).to receive(:current_ability).and_return(ability)
+    allow(ability).to receive(:current_user).and_return(user)
+
+    allow(Collection).to receive(:find).with(document.id).and_return(collection)
+
+    allow(presenter).to receive(:total_items).and_return(0)
+    allow(presenter).to receive(:collection_type).and_return(collection_type)
+    allow(presenter).to receive(:subcollection_count).and_return(0)
+    assign(:parent_collection_count, 0)
+    assign(:members_count, 0)
+
+    allow(collection_type).to receive(:nestable?).and_return(true)
+    allow(collection_type).to receive(:title).and_return("User Collection")
+    allow(collection_type).to receive(:badge_color).and_return("#ffa510")
+
+    assign(:presenter, presenter)
+    # Stub route because view specs don't handle engine routes
+    allow(view).to receive(:edit_dashboard_collection_path).and_return("/dashboard/collection/123/edit")
+    allow(view).to receive(:dashboard_collection_path).and_return("/dashboard/collection/123")
+    allow(view).to receive(:collection_path).and_return("/collection/123")
+
+    stub_template '_search_form.html.erb' => 'search form'
+    stub_template 'hyrax/dashboard/collections/_sort_and_per_page.html.erb' => 'sort and per page'
+    stub_template '_document_list.html.erb' => 'document list'
+    # This is tested ./spec/views/hyrax/dashboard/collections/_show_actions.html.erb_spec.rb
+    stub_template '_show_actions.html.erb' => '<div class="stubbed-actions">THE COLLECTION ACTIONS</div>'
+    stub_template '_show_subcollection_actions.html.erb' => '<div class="stubbed-actions">THE SUBCOLLECTION ACTIONS</div>'
+    stub_template '_show_add_items_actions.html.erb' => '<div class="stubbed-actions">THE ADD ITEMS ACTIONS</div>'
+    stub_template '_show_parent_collections.html.erb' => '<div class="stubbed-actions">THE PARENT COLLECTIONS LIST</div>'
+    stub_template '_subcollection_list.html.erb' => '<div class="stubbed-actions">THE SUB-COLLECTIONS LIST</div>'
+    stub_template 'hyrax/collections/_paginate.html.erb' => 'paginate'
+    stub_template 'hyrax/collections/_media_display.html.erb' => '<span class="fa fa-cubes collection-icon-search"></span>'
+    stub_template 'hyrax/my/collections/_modal_add_to_collection.html.erb' => 'modal add as subcollection'
+    stub_template 'hyrax/my/collections/_modal_add_subcollection.html.erb' => 'modal add as parent'
+  end
+
+  it 'draws the page' do
+    render
+    # Making sure that we are verifying that the _show_actions.html.erb is rendering
+    expect(rendered).to have_css('.stubbed-actions', text: 'THE COLLECTION ACTIONS')
+    expect(rendered).to have_css('.stubbed-actions', text: 'THE SUBCOLLECTION ACTIONS')
+    expect(rendered).to have_css('.stubbed-actions', text: 'THE ADD ITEMS ACTIONS')
+    expect(rendered).to match '<span class="fa fa-cubes collection-icon-search"></span>'
+    expect(rendered).not_to have_text('Search Results within this Collection')
+  end
+
+  context 'with a not-nested collection_type' do
+    before do
+      allow(presenter).to receive(:subcollection_count).and_return(0)
+      render
+    end
+    it 'draws the page' do
+      allow(collection_type).to receive(:nestable?).and_return(false)
+      # Making sure that we are verifying that the _show_actions.html.erb is rendering
+      expect(rendered).to have_css('.stubbed-actions', text: 'THE COLLECTION ACTIONS')
+      expect(rendered).to have_css('.stubbed-actions', text: 'THE SUBCOLLECTION ACTIONS')
+      expect(rendered).to have_css('.stubbed-actions', text: 'THE ADD ITEMS ACTIONS')
+      expect(rendered).to match '<span class="fa fa-cubes collection-icon-search"></span>'
+      expect(rendered).not_to have_text('Search Results within this Collection')
+    end
+  end
+
+  context 'when search results exist' do
+    before do
+      # rubocop:disable RSpec/AnyInstance
+      allow_any_instance_of(Hyrax::CollectionsHelper).to receive(:has_collection_search_parameters?).and_return(true)
+      # rubocop:enable RSpec/AnyInstance
+    end
+
+    context 'and only works are in search results' do
+      before do
+        assign(:members_count, 1)
+        render
+      end
+
+      it 'shows results header' do
+        expect(rendered).to have_text('Search Results within this Collection')
+        expect(rendered).to have_css('.stubbed-actions', text: 'THE COLLECTION ACTIONS')
+        expect(rendered).not_to have_css('.stubbed-actions', text: 'THE SUBCOLLECTION ACTIONS')
+        expect(rendered).not_to have_css('.stubbed-actions', text: 'THE ADD ITEMS ACTIONS')
+      end
+    end
+
+    context ' and only subcollections are in search results' do
+      before do
+        allow(presenter).to receive(:subcollection_count).and_return(1)
+        assign(:members_count, 0)
+        render
+      end
+
+      it 'shows results header' do
+        expect(rendered).to have_text('Search Results within this Collection')
+        expect(rendered).to have_css('.stubbed-actions', text: 'THE COLLECTION ACTIONS')
+        expect(rendered).not_to have_css('.stubbed-actions', text: 'THE SUBCOLLECTION ACTIONS')
+        expect(rendered).not_to have_css('.stubbed-actions', text: 'THE ADD ITEMS ACTIONS')
+      end
+    end
+  end
+end

data/spec/views/hyrax/dashboard/profiles/_trophy_edit.html.erb_spec.rb

@@ -0,0 +1,36 @@
+RSpec.describe 'hyrax/dashboard/profiles/_trophy_edit.html.erb', type: :view do
+  let(:user) { stub_model(User, user_key: 'mjg') }
+  let(:page) { Capybara::Node::Simple.new(rendered) }
+
+  before do
+    allow(view).to receive(:signed_in?).and_return(true)
+    allow(view).to receive(:current_user).and_return(user)
+    assign(:user, user)
+  end
+
+  context "when there are no highlighted works" do
+    before do
+      assign(:trophies, [])
+      render 'edit_primary'
+    end
+
+    it "has no trophy" do
+      expect(page).not_to have_css('a#remove_trophy_help')
+      expect(page).not_to have_content('Remove Highlight Designation')
+    end
+  end
+
+  context "when there are highlighted works" do
+    let(:solr_document) { SolrDocument.new(id: 'abc123', has_model_ssim: 'GenericWork', title_tesim: ['Title']) }
+
+    before do
+      assign(:trophies, [Hyrax::TrophyPresenter.new(solr_document)])
+      render 'edit_primary'
+    end
+
+    it "has trophy" do
+      expect(page).to have_css('a#remove_trophy_help')
+      expect(page).to have_selector("#remove_trophy_abc123")
+    end
+  end
+end

data/spec/views/hyrax/dashboard/profiles/edit.html.erb_spec.rb

@@ -0,0 +1,67 @@
+RSpec.describe 'hyrax/dashboard/profiles/edit.html.erb', type: :view do
+  let(:user) { stub_model(User, user_key: 'mjg') }
+
+  before do
+    allow(view).to receive(:signed_in?).and_return(true)
+    allow(view).to receive(:current_user).and_return(user)
+    assign(:user, user)
+    assign(:trophies, [])
+  end
+
+  it "shows an ORCID field" do
+    render
+    expect(rendered).to match(/ORCID Profile/)
+  end
+
+  context 'with Zotero integration enabled' do
+    before do
+      allow(Hyrax.config).to receive(:arkivo_api?) { true }
+    end
+
+    it 'shows a Zotero label' do
+      render
+      expect(rendered).to match(/Zotero Profile/)
+    end
+
+    context 'with a userID already set on the user instance' do
+      before do
+        allow(user).to receive(:zotero_userid) { '12345' }
+        render
+      end
+
+      it 'shows a link to the Zotero profile' do
+        expect(rendered).to have_link("Connected!", href: "https://www.zotero.org/users/12345")
+      end
+    end
+
+    context 'with no existing token' do
+      before { render }
+
+      it 'shows a Zotero OAuth button' do
+        expect(rendered).to have_css('a#zotero')
+      end
+    end
+
+    context 'with an existing token, in the production env' do
+      before do
+        allow(Rails.env).to receive(:production?) { true }
+        render
+      end
+
+      it 'shows a Zotero OAuth button' do
+        expect(rendered).to have_css('a#zotero')
+      end
+    end
+  end
+
+  context 'with Zotero integration disabled' do
+    before do
+      allow(Hyrax.config).to receive(:arkivo_api?) { false }
+    end
+
+    it 'hides a Zotero OAuth button' do
+      render
+      expect(subject).not_to have_css('a#zotero')
+    end
+  end
+end

data/spec/views/hyrax/dashboard/profiles/show.html.erb_spec.rb

@@ -0,0 +1,36 @@
+RSpec.describe 'hyrax/dashboard/profiles/show.html.erb', type: :view do
+  let(:join_date) { 1.day.ago }
+  let(:user) { stub_model(User, user_key: 'mjg', created_at: join_date) }
+  let(:ability) { double(current_user: stub_model(User, user_key: 'mjg')) }
+  let(:presenter) { Hyrax::UserProfilePresenter.new(user, ability) }
+
+  before do
+    allow(view).to receive(:signed_in?).and_return(true)
+    allow(view).to receive(:current_user).and_return(user)
+    allow(view).to receive(:can?).and_return(true)
+    allow(view).to receive(:number_of_collections).and_return(3)
+    allow(view).to receive(:number_of_works).and_return(5)
+    assign(:user, user)
+    assign(:presenter, presenter)
+  end
+
+  context "with trophy" do
+    let(:trophy_presenter) { Hyrax::TrophyPresenter.new(solr_document) }
+    let(:solr_document) do
+      SolrDocument.new(id: 'trophy_abc123',
+                       has_model_ssim: 'GenericWork',
+                       thumbnail_path_ss: '/foo/bar.png')
+    end
+
+    before do
+      allow(presenter).to receive(:trophies).and_return([trophy_presenter])
+    end
+
+    it "has trophy" do
+      render
+      page = Capybara::Node::Simple.new(rendered)
+      expect(page).to have_selector(".list-group-item > div#contributions.tab-pane")
+      expect(page).to have_selector("#trophyrow_#{solr_document.id}")
+    end
+  end
+end

data/spec/views/hyrax/dashboard/show_user_spec.rb

@@ -0,0 +1,61 @@
+RSpec.describe "hyrax/dashboard/show_user.html.erb", type: :view do
+  before do
+    allow(controller).to receive(:current_user).and_return(user)
+    assign(:presenter, presenter)
+  end
+  let(:user) { build(:user) }
+  let(:ability) { instance_double("Ability") }
+  let(:presenter) do
+    instance_double(Hyrax::Dashboard::UserPresenter,
+                    render_recent_activity: 'recent activity',
+                    render_recent_notifications: 'recent notifications',
+                    link_to_additional_notifications: 'additional notifications',
+                    link_to_manage_proxies: 'Manage Proxies',
+                    transfers: transfer_presenter)
+  end
+  let(:transfer_presenter) do
+    instance_double(Hyrax::TransfersPresenter,
+                    render_sent_transfers: 'sent transfers',
+                    render_received_transfers: 'received transfers')
+  end
+
+  it "includes recent activities and notifications" do
+    render
+    expect(rendered).to have_content "User Activity"
+    expect(rendered).to have_content "recent activity"
+
+    expect(rendered).to have_content "User Notifications"
+    expect(rendered).to have_content "recent notifications"
+    expect(rendered).to have_content "additional notifications"
+
+    expect(rendered).to have_content "Select works to transfer"
+    expect(rendered).to have_content "sent transfers"
+    expect(rendered).to have_content "received transfers"
+  end
+
+  describe 'proxy deposit' do
+    context 'when enabled' do
+      before do
+        allow(Flipflop).to receive(:proxy_deposit?).and_return(true)
+      end
+
+      it 'renders proxy partial' do
+        render
+        expect(rendered).to match(/Current Proxies/)
+        expect(rendered).to match(/Manage Proxies/)
+      end
+    end
+
+    context 'when disabled' do
+      before do
+        allow(Flipflop).to receive(:proxy_deposit?).and_return(false)
+      end
+
+      it 'does not render proxy partial' do
+        render
+        expect(rendered).not_to match(/Current Proxies/)
+        expect(rendered).not_to match(/Manage Proxies/)
+      end
+    end
+  end
+end

data/spec/views/hyrax/depositors/index.html.erb_spec.rb

@@ -0,0 +1,13 @@
+RSpec.describe "hyrax/depositors/index.html.erb", type: :view do
+  before do
+    allow(controller).to receive(:current_user).and_return(some_user)
+    assign :user, some_user
+  end
+
+  let(:some_user) { build(:user) }
+
+  it 'renders proxy partial' do
+    render
+    expect(rendered).to match(/Authorize Proxies/)
+  end
+end

data/spec/views/hyrax/embargoes/index.html.erb_spec.rb

@@ -0,0 +1,14 @@
+RSpec.describe "hyrax/embargoes/index.html.erb", type: :view do
+  before do
+    stub_template 'hyrax/embargoes/_list_deactivated_embargoes' => 'rendered list_deactivated_embargoes'
+    stub_template 'hyrax/embargoes/_list_expired_active_embargoes' => 'rendered list_expired_active_embargoes'
+    stub_template 'hyrax/embargoes/_list_active_embargoes' => 'rendered list_active_embargoes'
+  end
+
+  it "displays the page and renderes deactivated, expired, and active embargoes" do
+    render template: 'hyrax/embargoes/index'
+    expect(rendered).to have_css('.tab-pane#active', text: 'rendered list_active_embargoes')
+    expect(rendered).to have_css('.tab-pane#expired', text: 'rendered list_expired_active_embargoes')
+    expect(rendered).to have_css('.tab-pane#deactivated', text: 'rendered list_deactivated_embargoes')
+  end
+end

data/spec/views/hyrax/file_sets/_actions.html.erb_spec.rb

@@ -0,0 +1,36 @@
+RSpec.describe 'hyrax/file_sets/_actions.html.erb', type: :view do
+  let(:solr_document) { double("Solr Doc", id: 'file_set_id') }
+  let(:user) { build(:user) }
+  let(:ability) { Ability.new(user) }
+  let(:file_set) { Hyrax::FileSetPresenter.new(solr_document, ability) }
+
+  context 'with download permission' do
+    before do
+      allow(file_set).to receive(:user_can_perform_any_action?).and_return(true)
+      allow(view).to receive(:can?).with(:edit, file_set.id).and_return(false)
+      allow(view).to receive(:can?).with(:destroy, file_set.id).and_return(false)
+      allow(view).to receive(:can?).with(:download, file_set.id).and_return(true)
+      render 'hyrax/file_sets/actions', file_set: file_set
+    end
+
+    it "includes google analytics data in the download link" do
+      expect(rendered).to have_css('a#file_download')
+      expect(rendered).to have_selector("a[data-label=\"#{file_set.id}\"]")
+    end
+  end
+
+  context 'with no permission' do
+    let(:current_ability) { ability }
+
+    before do
+      allow(current_ability).to receive(:can?).with(:edit, file_set.id).and_return(false)
+      allow(current_ability).to receive(:can?).with(:destroy, file_set.id).and_return(false)
+      allow(current_ability).to receive(:can?).with(:download, file_set.id).and_return(false)
+      render 'hyrax/file_sets/actions', file_set: file_set
+    end
+
+    it "renders nothing" do
+      expect(rendered).to eq('')
+    end
+  end
+end

data/spec/views/hyrax/file_sets/_permission.html.erb_spec.rb

@@ -0,0 +1,14 @@
+RSpec.describe 'hyrax/file_sets/_permission.html.erb', type: :view do
+  let(:file_set) { stub_model(FileSet) }
+
+  before do
+    stub_template "hyrax/file_sets/_permission_form.html.erb" => 'a form'
+    render 'hyrax/file_sets/permission', file_set: file_set
+  end
+
+  context "without additional users" do
+    it "draws the permissions form without error" do
+      expect(rendered).to have_css('form#permission[data-param-key="file_set"]')
+    end
+  end
+end

data/spec/views/hyrax/file_sets/_permission_form.html.erb_spec.rb

@@ -0,0 +1,44 @@
+RSpec.describe 'hyrax/file_sets/_permission_form.html.erb', type: :view do
+  let(:file_set) do
+    stub_model(FileSet, id: '123',
+                        depositor: 'bob',
+                        resource_type: ['Dataset'])
+  end
+
+  let(:form) do
+    view.simple_form_for(file_set, url: '/update') do |fs_form|
+      return fs_form
+    end
+  end
+
+  before do
+    allow(controller).to receive(:current_user).and_return(stub_model(User))
+    allow(file_set).to receive(:permissions).and_return(permissions)
+    allow(view).to receive(:f).and_return(form)
+    view.lookup_context.prefixes.push 'hyrax/base'
+    view.extend Hyrax::PermissionsHelper
+    render
+  end
+
+  context "without additional users" do
+    let(:permissions) { [] }
+
+    it "draws the permissions form without error" do
+      expect(rendered).to have_css("input#new_user_name_skel")
+      expect(rendered).not_to have_css("button.remove_perm")
+    end
+  end
+
+  context "with additional users" do
+    let(:depositor_permission) { Hydra::AccessControls::Permission.new(id: '123', name: 'bob', type: 'person', access: 'edit') }
+    let(:public_permission) { Hydra::AccessControls::Permission.new(id: '124', name: 'public', type: 'group', access: 'read') }
+    let(:other_permission) { Hydra::AccessControls::Permission.new(id: '125', name: 'joe@example.com', type: 'person', access: 'edit') }
+    let(:permissions) { [depositor_permission, public_permission, other_permission] }
+
+    it "draws the permissions form without error" do
+      expect(rendered).to have_css("input#new_user_name_skel")
+      expect(rendered).to have_css("button.remove_perm", count: 1) # depositor and public should be filtered out
+      expect(rendered).to have_css("button.remove_perm[data-index='2']")
+    end
+  end
+end