hyrax 2.9.5 → 3.0.0.pre.beta1

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

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

@@ -0,0 +1,62 @@
+RSpec.describe 'hyrax/file_sets/_show_actions.html.erb', type: :view do
+  let(:user) { create(:user) }
+  let(:object_profile) { ["{\"id\":\"999\"}"] }
+  let(:contributor) { ['Frodo'] }
+  let(:creator)     { ['Bilbo'] }
+  let(:solr_document) do
+    SolrDocument.new(
+      id: '999',
+      object_profile_ssm: object_profile,
+      has_model_ssim: ['FileSet'],
+      human_readable_type_tesim: ['File'],
+      contributor_tesim: contributor,
+      creator_tesim: creator,
+      rights_tesim: ['http://creativecommons.org/licenses/by/3.0/us/']
+    )
+  end
+  let(:ability) { Ability.new(user) }
+  let(:presenter) do
+    Hyrax::WorkShowPresenter.new(solr_document, ability)
+  end
+  let(:page) { Capybara::Node::Simple.new(rendered) }
+
+  describe 'citations' do
+    before do
+      Hyrax.config.citations = citations
+      allow(ability).to receive(:can?).with(:edit, solr_document).and_return(false)
+      assign(:presenter, presenter)
+      view.lookup_context.view_paths.push 'app/views/hyrax/base'
+      render
+    end
+
+    context 'when enabled' do
+      let(:citations) { true }
+
+      it 'does not appear on page' do
+        expect(page).to have_no_selector('a#citations')
+      end
+    end
+
+    context 'when disabled' do
+      let(:citations) { false }
+
+      it 'does not appear on page' do
+        expect(page).to have_no_selector('a#citations')
+      end
+    end
+  end
+
+  describe 'editor' do
+    before do
+      allow(presenter).to receive(:editor?).and_return(true)
+      assign(:presenter, presenter)
+      view.lookup_context.view_paths.push 'app/views/hyrax/base'
+      render
+    end
+
+    it 'renders actions for the user' do
+      expect(page).to have_link("Edit This File")
+      expect(page).to have_link("Delete This File")
+    end
+  end
+end

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

@@ -0,0 +1,29 @@
+RSpec.describe 'hyrax/file_sets/_show_characterization_details.html.erb', type: :view do
+  let(:solr_document) { SolrDocument.new({}) }
+  let(:ability) { double "Ability" }
+  let(:presenter) { Hyrax::FileSetPresenter.new(solr_document, ability) }
+  let(:mock_metadata) do
+    {
+      format: ["Tape"],
+      long_term: ["x" * 255],
+      multi_term: ["1", "2", "3", "4", "5", "6", "7", "8"],
+      string_term: 'oops, I used a string instead of an array'
+    }
+  end
+  let(:page) { Capybara::Node::Simple.new(rendered) }
+
+  before do
+    allow(presenter).to receive(:characterization_metadata).and_return(mock_metadata)
+    assign(:presenter, presenter)
+    render
+  end
+
+  it 'displays characterization terms' do
+    expect(page).to have_content("oops, I used a string instead of an array")
+    expect(page).to have_content("xxxxxxx...")
+    expect(page).to have_content("Format: Tape")
+    expect(page).to have_css("div.modal")
+    expect(page).to have_css("div.modal-body")
+    expect(page).to have_css("h2#extraFieldsModal_multi_term_Label")
+  end
+end

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

@@ -0,0 +1,34 @@
+RSpec.describe 'hyrax/file_sets/_single_use_links.html.erb', type: :view do
+  let(:user)          { create(:user) }
+  let(:solr_document) { SolrDocument.new(id: '1234', 'has_model_ssim' => 'FileSet') }
+  let(:ability)       { Ability.new(user) }
+  let(:presenter)     { Hyrax::FileSetPresenter.new(solr_document, ability) }
+
+  context "with no single-use links" do
+    before do
+      allow(presenter).to receive(:single_use_links).and_return([])
+      render 'hyrax/file_sets/single_use_links.html.erb', presenter: presenter
+    end
+    it "renders a table with no links" do
+      expect(rendered).to include("<tr><td>No links have been generated</td></tr>")
+    end
+  end
+
+  context "with single use links" do
+    let(:link)           { SingleUseLink.create(item_id: "1234", download_key: "sha2hashb") }
+    let(:link_presenter) { Hyrax::SingleUseLinkPresenter.new(link) }
+
+    before do
+      controller.params = { id: "1234" }
+      allow(presenter).to receive(:single_use_links).and_return([link_presenter])
+      render 'hyrax/file_sets/single_use_links.html.erb', presenter: presenter
+    end
+    it "renders a table with links" do
+      expect(rendered).to include("Link sha2ha expires in 23 hours")
+    end
+
+    it "renders the single use link button" do
+      expect(rendered).to have_link("Create Single-Use Link")
+    end
+  end
+end

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

@@ -0,0 +1,15 @@
+RSpec.describe 'hyrax/file_sets/_versioning.html.erb', type: :view do
+  let(:file_set) { stub_model(FileSet) }
+
+  before do
+    allow(view).to receive(:curation_concern).and_return(file_set)
+    assign(:version_list, [])
+    render
+  end
+
+  context "without additional users" do
+    it "draws the new version form without error" do
+      expect(rendered).to have_css("input[name='file_set[files][]']")
+    end
+  end
+end

data/spec/views/hyrax/file_sets/media_display/_audio.html.erb_spec.rb

@@ -0,0 +1,29 @@
+RSpec.describe 'hyrax/file_sets/media_display/_audio.html.erb', type: :view do
+  let(:file_set) { stub_model(FileSet) }
+  let(:config) { double }
+  let(:link) { true }
+
+  before do
+    allow(Hyrax.config).to receive(:display_media_download_link?).and_return(link)
+    render 'hyrax/file_sets/media_display/audio', file_set: file_set
+  end
+
+  it "draws the view with the link" do
+    expect(rendered).to have_selector("audio")
+    expect(rendered).to have_css('a', text: 'Download audio')
+  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
+
+  context "no download links" do
+    let(:link) { false }
+
+    it "draws the view without the link" do
+      expect(rendered).to have_selector("audio")
+      expect(rendered).not_to have_css('a', text: 'Download audio')
+    end
+  end
+end