hyrax 2.9.5 → 3.0.0.pre.beta1

data/spec/views/hyrax/base/_form_relationships.html.erb_spec.rb

@@ -0,0 +1,48 @@
+RSpec.describe 'hyrax/base/_form_relationships.html.erb', type: :view do
+  let(:ability) { double }
+  let(:work) { GenericWork.new }
+  let(:form) do
+    Hyrax::GenericWorkForm.new(work, ability, controller)
+  end
+  let(:service) { instance_double Hyrax::AdminSetService }
+  let(:presenter) { instance_double Hyrax::AdminSetOptionsPresenter, select_options: [] }
+  let(:form_template) do
+    %(
+      <%= simple_form_for [main_app, @form] do |f| %>
+        <%= render "hyrax/base/form_relationships", f: f %>
+      <% end %>
+    )
+  end
+
+  let(:page) do
+    assign(:form, form)
+    render inline: form_template
+    Capybara::Node::Simple.new(rendered)
+  end
+
+  before do
+    allow(view).to receive(:action_name).and_return('new')
+    allow(Hyrax::AdminSetService).to receive(:new).with(controller).and_return(service)
+    allow(Hyrax::AdminSetOptionsPresenter).to receive(:new).with(service).and_return(presenter)
+  end
+
+  context 'with assign_admin_set turned on' do
+    before do
+      allow(Flipflop).to receive(:assign_admin_set?).and_return(true)
+    end
+
+    it "draws the page" do
+      expect(page).to have_content('Administrative Set')
+      expect(page).to have_selector('select#generic_work_admin_set_id')
+    end
+  end
+
+  context 'with assign_admin_set disabled' do
+    before do
+      allow(Flipflop).to receive(:assign_admin_set?).and_return(false)
+    end
+    it 'draws the page, but not the admin set widget' do
+      expect(page).not_to have_content('administrative set')
+    end
+  end
+end

data/spec/views/hyrax/base/_form_rendering.html.erb_spec.rb

@@ -0,0 +1,22 @@
+RSpec.describe 'hyrax/base/_form_rendering.html.erb', type: :view do
+  let(:ability) { double }
+  let(:work) { stub_model(GenericWork, new_record?: false) }
+  let(:form) do
+    Hyrax::GenericWorkForm.new(work, ability, controller)
+  end
+
+  let(:page) do
+    view.simple_form_for form do |f|
+      render 'hyrax/base/form_rendering', f: f
+    end
+    Capybara::Node::Simple.new(rendered)
+  end
+
+  before do
+    allow(form).to receive(:select_files).and_return([{ '123' => 'File one' }])
+  end
+
+  it 'has a rendering_ids field' do
+    expect(page).to have_selector("select#generic_work_rendering_ids", count: 1)
+  end
+end

data/spec/views/hyrax/base/_items.html.erb_spec.rb

@@ -0,0 +1,55 @@
+RSpec.describe 'hyrax/base/_items.html.erb', type: :view do
+  let(:ability) { double }
+  let(:request) { double "request", params: params }
+  let(:params) { ActionController::Parameters.new(rows: 10) }
+
+  context 'when children are not present' do
+    let(:member_list) { [] }
+    let(:presenter) { double(:presenter, list_of_items_to_display: member_list, member_presenters_for: member_list, id: 'the-id', human_readable_type: 'Thing') }
+
+    before do
+      expect(presenter).to receive(:list_of_item_ids_to_display).and_return(member_list)
+    end
+
+    context 'and the current user can edit the presenter' do
+      it 'renders an alert' do
+        expect(view).to receive(:can?).with(:edit, presenter.id).and_return(true)
+        render 'hyrax/base/items', presenter: presenter
+        expect(rendered).to have_css('.alert-warning[role=alert]', text: 'This Thing has no files associated with it. Click "edit" to add more files.')
+      end
+    end
+    context 'and the current user cannot edit the presenter' do
+      it 'renders an alert' do
+        expect(view).to receive(:can?).with(:edit, presenter.id).and_return(false)
+        render 'hyrax/base/items', presenter: presenter
+        expect(rendered).to have_css('.alert-warning[role=alert]', text: "There are no publicly available items in this Thing.")
+      end
+    end
+  end
+
+  context 'when children are present' do
+    let(:child1) { double('Thing1', id: 'Thing 1', title: 'Title 1') }
+    let(:child2) { double('Thing2', id: 'Thing 2', title: 'Title 2') }
+    let(:child3) { double('Thing3', id: 'Thing 3', title: 'Title 3') }
+    let(:member_list) { [child1, child2, child3] }
+    let(:solr_document) { double('Solr Doc', id: 'the-id') }
+    let(:presenter) { Hyrax::WorkShowPresenter.new(solr_document, ability, request) }
+
+    before do
+      stub_template 'hyrax/base/_member.html.erb' => '<%= member %>'
+      expect(Flipflop).to receive(:hide_private_items?).and_return(:flipflop)
+      allow(presenter).to receive(:list_of_item_ids_to_display).and_return(member_list)
+      allow(presenter).to receive(:member_presenters_for).with(member_list).and_return(member_list)
+      allow(presenter).to receive(:ordered_ids).and_return([])
+
+      allow(ability).to receive(:can?).with(:read, child1.id).and_return true
+      allow(ability).to receive(:can?).with(:read, child2.id).and_return false
+      allow(ability).to receive(:can?).with(:read, child3.id).and_return true
+    end
+
+    it "displays children" do
+      render 'hyrax/base/items', presenter: presenter
+      expect(rendered).to have_css('tbody')
+    end
+  end
+end

data/spec/views/hyrax/base/_member.html.erb_spec.rb

@@ -0,0 +1,43 @@
+RSpec.describe 'hyrax/base/_member.html.erb' do
+  let(:solr_document) do
+    SolrDocument.new(id: '999',
+                     has_model_ssim: ['FileSet'],
+                     active_fedora_model_ssi: 'FileSet',
+                     thumbnail_path_ss: '/downloads/999?file=thumbnail',
+                     representative_tesim: ["999"],
+                     title_tesim: ["My File"])
+  end
+
+  # Ability is checked in FileSetPresenter#link_name
+  let(:ability) { double(can?: true) }
+  let(:presenter) { Hyrax::FileSetPresenter.new(solr_document, ability) }
+
+  before do
+    assign(:presenter, presenter)
+    allow(view).to receive(:current_search_session).and_return nil
+    allow(view).to receive(:search_session).and_return({})
+    # abilities called in _actions.html.erb
+    allow(view).to receive(:can?).with(:download, kind_of(String)).and_return(true)
+    allow(view).to receive(:can?).with(:edit, kind_of(String)).and_return(true)
+    allow(view).to receive(:can?).with(:destroy, String).and_return(true)
+    allow(view).to receive(:contextual_path).with(anything, anything) do |x, y|
+      Hyrax::ContextualPath.new(x, y).show
+    end
+    render 'hyrax/base/member.html.erb', member: presenter
+  end
+
+  it 'checks the :download ability' do
+    expect(view).to have_received(:can?).with(:download, kind_of(String)).once
+  end
+
+  it 'renders the view' do
+    # A thumbnail
+    expect(rendered).to have_selector ".thumbnail img[src='#{hyrax.download_path(presenter, file: 'thumbnail')}']"
+
+    # Action buttons
+    expect(rendered).to have_selector "a[title=\"Edit My File\"][href='#{edit_polymorphic_path(presenter)}']", text: 'Edit'
+    expect(rendered).to have_selector "a[title=\"Delete My File\"][data-method='delete'][href='#{polymorphic_path(presenter)}']", text: 'Delete'
+    expect(rendered).to have_link('Download')
+    expect(rendered).to have_selector "a[title='Download My File'][href='#{hyrax.download_path(presenter)}']", text: 'Download'
+  end
+end

data/spec/views/hyrax/base/_relationships.html.erb_spec.rb

@@ -0,0 +1,92 @@
+RSpec.describe 'hyrax/base/relationships', type: :view do
+  let(:user) { create(:user, groups: 'admin') }
+  let(:ability) { Ability.new(user) }
+  let(:solr_doc) { instance_double(SolrDocument, id: '123', human_readable_type: 'Work', admin_set: nil) }
+  let(:presenter) { Hyrax::WorkShowPresenter.new(solr_doc, ability) }
+  let(:generic_work) do
+    Hyrax::WorkShowPresenter.new(
+      SolrDocument.new(
+        id: '456',
+        has_model_ssim: ['GenericWork'],
+        title_tesim: ['Containing work']
+      ),
+      ability
+    )
+  end
+
+  let(:collection) do
+    Hyrax::CollectionPresenter.new(
+      SolrDocument.new(
+        id: '345',
+        has_model_ssim: ['Collection'],
+        title_tesim: ['Containing collection']
+      ),
+      ability
+    )
+  end
+  let(:page) { Capybara::Node::Simple.new(rendered) }
+
+  context "when collections are present and no parents are present" do
+    let(:member_of_collection_presenters) { [collection] }
+
+    before do
+      allow(controller).to receive(:current_user).and_return user
+      allow(view).to receive(:contextual_path).and_return("/collections/456")
+      allow(presenter).to receive(:member_of_collection_presenters).and_return(member_of_collection_presenters)
+      render 'hyrax/base/relationships', presenter: presenter
+    end
+    it "links to collections" do
+      expect(page).to have_text 'In Collection'
+      expect(page).to have_link 'Containing collection'
+      expect(page).not_to have_text 'In Generic work'
+    end
+  end
+
+  context "when parents are present and no collections are present" do
+    let(:member_of_collection_presenters) { [generic_work] }
+
+    before do
+      allow(controller).to receive(:current_user).and_return user
+      allow(view).to receive(:contextual_path).and_return("/concern/generic_works/456")
+      allow(presenter).to receive(:member_of_collection_presenters).and_return(member_of_collection_presenters)
+      render 'hyrax/base/relationships', presenter: presenter
+    end
+    it "links to work" do
+      expect(page).to have_text 'In Generic work'
+      expect(page).to have_link 'Containing work'
+      expect(page).not_to have_text 'In Collection'
+    end
+  end
+
+  context "when parents are present and collections are present" do
+    let(:member_of_collection_presenters) { [generic_work, collection] }
+
+    before do
+      allow(controller).to receive(:current_user).and_return user
+      allow(view).to receive(:contextual_path).and_return("/concern/generic_works/456")
+      allow(presenter).to receive(:member_of_collection_presenters).and_return(member_of_collection_presenters)
+      render 'hyrax/base/relationships', presenter: presenter
+    end
+    it "links to work and collection" do
+      expect(page).to have_link 'Containing work'
+      expect(page).to have_link 'Containing collection'
+    end
+  end
+
+  context "with admin sets" do
+    it "renders using attribute_to_html" do
+      allow(controller).to receive(:current_user).and_return(user)
+      allow(solr_doc).to receive(:member_of_collection_ids).and_return([])
+      allow(presenter).to receive(:grouped_presenters).and_return({})
+      expect(presenter).to receive(:attribute_to_html).with(:admin_set, render_as: :faceted, html_dl: true)
+      render 'hyrax/base/relationships', presenter: presenter
+    end
+
+    it "skips admin sets if user not logged in" do
+      allow(controller).to receive(:current_user).and_return(nil)
+      allow(presenter).to receive(:member_of_collection_presenters).and_return([])
+      expect(presenter).not_to receive(:attribute_to_html).with(:admin_set, render_as: :faceted)
+      render 'hyrax/base/relationships', presenter: presenter
+    end
+  end
+end

data/spec/views/hyrax/base/_show_actions.html.erb_spec.rb

@@ -0,0 +1,149 @@
+RSpec.describe 'hyrax/base/_show_actions.html.erb', type: :view do
+  let(:presenter) { Hyrax::WorkShowPresenter.new(solr_document, ability) }
+  let(:solr_document) { SolrDocument.new(attributes) }
+  let(:attributes) { { "has_model_ssim" => ["GenericWork"], :id => "0r967372b" } }
+  let(:ability) { double }
+  let(:member) { Hyrax::WorkShowPresenter.new(member_document, ability) }
+  let(:member_document) { SolrDocument.new(member_attributes) }
+  let(:member_attributes) { { "has_model_ssim" => ["GenericWork"], :id => "8336h190k" } }
+
+  before do
+    allow(ability).to receive(:can?).with(:create, FeaturedWork).and_return(false)
+  end
+
+  context "as an unregistered user" do
+    before do
+      allow(presenter).to receive(:show_deposit_for?).with(anything).and_return(false)
+      allow(presenter).to receive(:editor?).and_return(false)
+      render 'hyrax/base/show_actions.html.erb', presenter: presenter
+    end
+    it "doesn't show edit / delete / Add to collection links" do
+      expect(rendered).not_to have_link 'Edit'
+      expect(rendered).not_to have_link 'Delete'
+      expect(rendered).not_to have_link 'Add to collection'
+    end
+  end
+
+  context "as an editor" do
+    before do
+      allow(presenter).to receive(:show_deposit_for?).with(anything).and_return(true)
+      allow(presenter).to receive(:editor?).and_return(true)
+    end
+    context "when the work does not contain children" do
+      before do
+        allow(presenter).to receive(:member_presenters).and_return([])
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+      it "does not show file manager link" do
+        expect(rendered).not_to have_link 'File Manager'
+      end
+      it "shows edit / delete / Add to collection links" do
+        expect(rendered).to have_link 'Edit'
+        expect(rendered).to have_link 'Delete'
+        expect(rendered).to have_button 'Add to collection'
+      end
+    end
+    context "when the work contains 1 child" do
+      before do
+        allow(presenter).to receive(:member_presenters).and_return([member])
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+      it "does not show file manager link" do
+        expect(rendered).not_to have_link 'File Manager'
+      end
+    end
+
+    context "when the work contains 2 children" do
+      let(:file_member) { Hyrax::FileSetPresenter.new(file_document, ability) }
+      let(:file_document) { SolrDocument.new(file_attributes) }
+      let(:file_attributes) { { id: '1234' } }
+
+      before do
+        allow(presenter).to receive(:member_presenters).and_return([member, file_member])
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+      it "shows file manager link" do
+        expect(rendered).to have_link 'File Manager'
+      end
+    end
+
+    context "when there are valid_child_concerns" do
+      before do
+        allow(presenter).to receive(:member_presenters).and_return([])
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+      it "creates a link to add child work" do
+        expect(rendered).to have_link 'Attach Generic Work', href: "/concern/parent/#{presenter.id}/generic_works/new"
+      end
+    end
+  end
+
+  context "when user CAN deposit to at least one collection" do
+    before do
+      allow(presenter).to receive(:show_deposit_for?).with(anything).and_return(true)
+      allow(presenter).to receive(:member_presenters).and_return([])
+    end
+
+    context "and user is editor" do
+      before do
+        allow(presenter).to receive(:editor?).and_return(true)
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+
+      it "shows editor related buttons" do
+        expect(rendered).not_to have_link 'File Manager'
+        expect(rendered).to have_link 'Edit'
+        expect(rendered).to have_link 'Delete'
+        expect(rendered).to have_button 'Add to collection'
+      end
+    end
+
+    context "and user is viewer" do
+      before do
+        allow(presenter).to receive(:editor?).and_return(false)
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+      it "shows only Add to collection link" do
+        expect(rendered).not_to have_link 'File Manager'
+        expect(rendered).not_to have_link 'Edit'
+        expect(rendered).not_to have_link 'Delete'
+        expect(rendered).to have_button 'Add to collection'
+      end
+    end
+  end
+
+  context "when user can NOT deposit to any collections" do
+    before do
+      allow(presenter).to receive(:show_deposit_for?).with(anything).and_return(false)
+      allow(presenter).to receive(:member_presenters).and_return([])
+    end
+
+    context "and user is editor" do
+      before do
+        allow(presenter).to receive(:editor?).and_return(true)
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+
+      it "shows editor related buttons" do
+        expect(rendered).not_to have_link 'File Manager'
+        expect(rendered).to have_link 'Edit'
+        expect(rendered).to have_link 'Delete'
+        expect(rendered).not_to have_button 'Add to collection'
+      end
+    end
+
+    context "and user is viewer" do
+      before do
+        allow(presenter).to receive(:editor?).and_return(false)
+        render 'hyrax/base/show_actions.html.erb', presenter: presenter
+      end
+
+      it "shows only Add to collection link" do
+        expect(rendered).not_to have_link 'File Manager'
+        expect(rendered).not_to have_link 'Edit'
+        expect(rendered).not_to have_link 'Delete'
+        expect(rendered).not_to have_button 'Add to collection'
+      end
+    end
+  end
+end

data/spec/views/hyrax/base/_social_media.html.erb_spec.rb

@@ -0,0 +1,16 @@
+RSpec.describe 'hyrax/base/_social_media.html.erb', type: :view do
+  let(:url) { 'http://example.com/' }
+  let(:title) { 'Example' }
+  let(:page) do
+    render partial: 'hyrax/base/social_media', locals: { share_url: url, page_title: title }
+    Capybara::Node::Simple.new(rendered)
+  end
+
+  it 'renders various social media share links' do
+    expect(page).to have_selector '.resp-sharing-button__link'
+    expect(page).to have_link '', href: 'https://facebook.com/sharer/sharer.php?u=http%3A%2F%2Fexample.com%2F'
+    expect(page).to have_link '', href: 'https://twitter.com/intent/tweet/?text=Example&url=http%3A%2F%2Fexample.com%2F'
+    expect(page).to have_link '', href: 'https://plus.google.com/share?url=http%3A%2F%2Fexample.com%2F'
+    expect(page).to have_link '', href: 'https://www.tumblr.com/widgets/share/tool?canonicalUrl=http%3A%2F%2Fexample.com%2F&posttype=link&shareSource=tumblr_share_button'
+  end
+end

data/spec/views/hyrax/base/edit.html.erb_spec.rb

@@ -0,0 +1,26 @@
+RSpec.describe 'hyrax/base/edit.html.erb', type: :view do
+  let(:work) { stub_model(GenericWork, id: '456', title: ["A nice work"]) }
+  let(:ability) { double }
+
+  let(:form) do
+    Hyrax::GenericWorkForm.new(work, ability, controller)
+  end
+
+  before do
+    # TODO: stub_model is not stubbing new_record? correctly on ActiveFedora models.
+    allow(work).to receive(:new_record?).and_return(false)
+    allow(view).to receive(:curation_concern).and_return(work)
+    allow(controller).to receive(:current_user).and_return(stub_model(User))
+    assign(:form, form)
+    view.controller = Hyrax::GenericWorksController.new
+    view.controller.action_name = 'edit'
+    stub_template "hyrax/base/_form.html.erb" => 'a form'
+  end
+
+  it "sets a header and draws the form" do
+    expect(view).to receive(:provide).with(:page_title, 'A nice work // Generic Work [456] // Hyrax')
+    expect(view).to receive(:provide).with(:page_header).and_yield
+    render
+    expect(rendered).to eq "  <h1><span class=\"fa fa-edit\" aria-hidden=\"true\"></span>Edit Work</h1>\n\na form\n"
+  end
+end