hyrax 2.6.0 → 2.7.0

data/spec/helpers/hyrax/citations_behaviors/formatters/chicago_formatter_spec.rb

@@ -1,10 +0,0 @@
-RSpec.describe Hyrax::CitationsBehaviors::Formatters::ChicagoFormatter do
-  subject(:formatter) { described_class.new(:no_context) }
-
-  let(:presenter) { Hyrax::WorkShowPresenter.new(SolrDocument.new(work.to_solr), :no_ability) }
-  let(:work)      { build(:generic_work, title: ['<ScrIPt>prompt("Confirm Password")</sCRIpt>']) }
-
-  it 'sanitizes input' do
-    expect(formatter.format(presenter)).not_to include 'prompt'
-  end
-end

data/spec/helpers/hyrax/collections_helper_spec.rb

@@ -1,117 +0,0 @@
-RSpec.describe Hyrax::CollectionsHelper do
-  let(:user) { create(:user, groups: ['admin']) }
-  let(:ability) { Ability.new(user) }
-
-  before do
-    # Stub route because helper specs don't handle engine routes
-    # https://github.com/rspec/rspec-rails/issues/1250
-    allow(view).to receive(:collection_path) do |collection|
-      id = collection.respond_to?(:id) ? collection.id : collection
-      "/collections/#{id}"
-    end
-  end
-
-  describe '#render_collection_links' do
-    before do
-      allow(controller).to receive(:current_ability).and_return(ability)
-    end
-
-    context 'when a GenericWork does not belongs to any collections', :clean_repo do
-      let!(:work_doc) { SolrDocument.new(id: '123', title_tesim: ['My GenericWork']) }
-
-      it 'renders nothing' do
-        expect(helper.render_collection_links(work_doc)).to be_nil
-      end
-    end
-
-    context 'when a GenericWork belongs to collections' do
-      let(:coll_ids) { ['111', '222'] }
-      let(:coll_titles) { ['Collection 111', 'Collection 222'] }
-      let(:coll1_attrs) { { id: coll_ids[0], title_tesim: [coll_titles[0]] } }
-      let(:coll2_attrs) { { id: coll_ids[1], title_tesim: [coll_titles[1]] } }
-      let!(:work_doc) { SolrDocument.new(id: '123', title_tesim: ['My GenericWork'], member_of_collection_ids_ssim: coll_ids) }
-
-      before do
-        ActiveFedora::SolrService.add(coll1_attrs)
-        ActiveFedora::SolrService.add(coll2_attrs)
-        ActiveFedora::SolrService.commit
-      end
-
-      it 'renders a list of links to the collections' do
-        expect(helper.render_collection_links(work_doc)).to match(/Is part of/i)
-        expect(helper.render_collection_links(work_doc)).to match("href=\"/collections/#{coll_ids[0]}\"")
-        expect(helper.render_collection_links(work_doc)).to match("href=\"/collections/#{coll_ids[1]}\"")
-        expect(helper.render_collection_links(work_doc)).to match(coll_titles[0])
-        expect(helper.render_collection_links(work_doc)).to match(coll_titles[1])
-      end
-    end
-  end
-
-  describe "has_collection_search_parameters?" do
-    subject { helper }
-
-    context "when cq is set" do
-      before { allow(helper).to receive(:params).and_return(cq: 'foo') }
-      it { is_expected.to have_collection_search_parameters }
-    end
-
-    context "when cq is not set" do
-      before { allow(helper).to receive(:params).and_return(cq: '') }
-      it { is_expected.not_to have_collection_search_parameters }
-    end
-  end
-
-  describe "button_for_remove_selected_from_collection" do
-    let(:collection) { create(:collection) }
-
-    it "creates a button to the collections delete path" do
-      str = button_for_remove_selected_from_collection collection
-      doc = Nokogiri::HTML(str)
-      form = doc.xpath('//form').first
-      expect(form.attr('action')).to eq hyrax.dashboard_collection_path(collection)
-      i = form.xpath('.//input')[2]
-      expect(i.attr('value')).to eq("remove")
-      expect(i.attr('name')).to eq("collection[members]")
-    end
-
-    it "creates a button with my text" do
-      str = button_for_remove_selected_from_collection collection, "Remove My Button"
-      doc = Nokogiri::HTML(str)
-      form = doc.css('form').first
-      expect(form.attr('action')).to eq hyrax.dashboard_collection_path(collection)
-      expect(form.css('input[type="submit"]').attr('value').value).to eq "Remove My Button"
-    end
-  end
-
-  describe "collection_type_label" do
-    context "when the CollectionType is found" do
-      let(:test_collection_type) { FactoryBot.create(:collection_type) }
-
-      it "returns the CollectionType title" do
-        expect(collection_type_label(test_collection_type.gid)).to eq test_collection_type.title
-      end
-    end
-
-    context "when the CollectionType cannot be found" do
-      it "returns the input gid unchanged" do
-        expect(collection_type_label(nil)).to eq "User Collection"
-      end
-    end
-  end
-
-  describe '#append_collection_type_url' do
-    let(:url) { "http://example.com" }
-    context "when a provided url has no querystring" do
-      it 'returns the url with added collection_type_id' do
-        expect(append_collection_type_url(url, '1')).to eq "#{url}?collection_type_id=1"
-      end
-    end
-
-    context "when a provided url has an existing querystring" do
-      let(:url) { "http://example.com?bob=ross" }
-      it 'return the url with added collection_type_id' do
-        expect(append_collection_type_url(url, '1')).to eq "#{url}&collection_type_id=1"
-      end
-    end
-  end
-end

data/spec/helpers/hyrax/content_block_helper_spec.rb

@@ -1,44 +0,0 @@
-RSpec.describe Hyrax::ContentBlockHelper, type: :helper do
-  let(:content_block) { create(:content_block, value: "<p>foo bar</p>") }
-
-  describe '#displayable_content_block' do
-    let(:options) { {} }
-
-    subject { helper.displayable_content_block(content_block, options) }
-
-    it 'is defined' do
-      expect(helper).to respond_to(:displayable_content_block)
-    end
-
-    context 'when a block is nil' do
-      let(:content_block) { nil }
-
-      it { is_expected.to be_nil }
-    end
-
-    context 'when a block has a nil value' do
-      let(:content_block) { double(value: nil) }
-
-      it { is_expected.to be_nil }
-    end
-
-    context 'when a block has an empty string value' do
-      let(:content_block) { double(value: '') }
-
-      it { is_expected.to be_nil }
-    end
-
-    context 'when a block has a non-empty string value' do
-      let(:content_block) { double(value: value) }
-      let(:value) { '<p>foobarbaz</p>' }
-
-      it { is_expected.to eq "<div>#{value}</div>" }
-
-      context 'with options' do
-        let(:options) { { id: 'my_id', class: 'huge' } }
-
-        it { is_expected.to eq "<div id=\"my_id\" class=\"huge\">#{value}</div>" }
-      end
-    end
-  end
-end

data/spec/helpers/hyrax/dashboard_helper_behavior_spec.rb

@@ -1,74 +0,0 @@
-RSpec.describe Hyrax::DashboardHelperBehavior, type: :helper do
-  describe "#on_the_dashboard?" do
-    it "returns false for controllers that aren't a part of the dashboard" do
-      allow(helper).to receive(:params).and_return(controller: "foo")
-      expect(helper).not_to be_on_the_dashboard
-    end
-
-    it "returns true for controllers that are part of the dashboard" do
-      allow(helper).to receive(:params).and_return(controller: "hyrax/my/works")
-      expect(helper).to be_on_the_dashboard
-      allow(helper).to receive(:params).and_return(controller: "hyrax/my/collections")
-      expect(helper).to be_on_the_dashboard
-      allow(helper).to receive(:params).and_return(controller: "hyrax/my/highlights")
-      expect(helper).to be_on_the_dashboard
-      allow(helper).to receive(:params).and_return(controller: "hyrax/my/shares")
-      expect(helper).to be_on_the_dashboard
-    end
-  end
-
-  describe "#number_of_works" do
-    let(:conn) { ActiveFedora::SolrService.instance.conn }
-    let(:user1) { User.new(email: "abc@test") }
-    let(:user2) { User.new(email: "abc@test.123") }
-
-    before do
-      create_models("GenericWork", user1, user2)
-    end
-
-    it "finds 3 works" do
-      expect(helper.number_of_works(user1)).to eq(3)
-    end
-  end
-
-  describe "#number_of_files" do
-    let(:conn) { ActiveFedora::SolrService.instance.conn }
-    let(:user1) { User.new(email: "abc@test") }
-    let(:user2) { User.new(email: "abc@test.123") }
-
-    before do
-      create_models("FileSet", user1, user2)
-    end
-
-    it "finds only 3 files" do
-      expect(helper.number_of_files(user1)).to eq(3)
-    end
-  end
-
-  describe "#number_of_collections" do
-    let(:conn) { ActiveFedora::SolrService.instance.conn }
-    let(:user1) { User.new(email: "abc@test") }
-    let(:user2) { User.new(email: "abc@test.123") }
-
-    before do
-      create_models("Collection", user1, user2)
-    end
-
-    it "finds only 3 files" do
-      expect(helper.number_of_collections(user1)).to eq(3)
-    end
-  end
-
-  def create_models(model, user1, user2)
-    # deposited by the first user
-    3.times do |t|
-      conn.add id: "199#{t}", Solrizer.solr_name('depositor', :stored_searchable) => user1.user_key, "has_model_ssim" => [model],
-               Solrizer.solr_name('depositor', :symbol) => user1.user_key
-    end
-
-    # deposited by the second user, but editable by the first
-    conn.add id: "1994", Solrizer.solr_name('depositor', :stored_searchable) => user2.user_key, "has_model_ssim" => [model],
-             Solrizer.solr_name('depositor', :symbol) => user2.user_key, "edit_access_person_ssim" => user1.user_key
-    conn.commit
-  end
-end

data/spec/helpers/hyrax/file_set_helper_spec.rb

@@ -1,65 +0,0 @@
-RSpec.describe Hyrax::FileSetHelper do
-  describe '#media_display' do
-    let(:file_set) { SolrDocument.new(mime_type_ssi: mime_type) }
-    let(:mime_type) { 'image/tiff' }
-
-    before do
-      allow(helper).to receive(:media_display_partial).with(file_set)
-                                                      .and_return('hyrax/file_sets/media_display/image')
-    end
-
-    it "renders a partial" do
-      expect(helper).to receive(:render)
-        .with('hyrax/file_sets/media_display/image', file_set: file_set)
-      helper.media_display(file_set)
-    end
-
-    it "takes options" do
-      expect(helper).to receive(:render)
-        .with('hyrax/file_sets/media_display/image', file_set: file_set, transcript_id: '123')
-      helper.media_display(file_set, transcript_id: '123')
-    end
-  end
-
-  describe '#media_display_partial' do
-    subject { helper.media_display_partial(file_set) }
-
-    let(:file_set) { SolrDocument.new(mime_type_ssi: mime_type) }
-
-    context "with an image" do
-      let(:mime_type) { 'image/tiff' }
-
-      it { is_expected.to eq 'hyrax/file_sets/media_display/image' }
-    end
-
-    context "with a video" do
-      let(:mime_type) { 'video/webm' }
-
-      it { is_expected.to eq 'hyrax/file_sets/media_display/video' }
-    end
-
-    context "with an audio" do
-      let(:mime_type) { 'audio/wav' }
-
-      it { is_expected.to eq 'hyrax/file_sets/media_display/audio' }
-    end
-
-    context "with a pdf" do
-      let(:mime_type) { 'application/pdf' }
-
-      it { is_expected.to eq 'hyrax/file_sets/media_display/pdf' }
-    end
-
-    context "with a word document" do
-      let(:mime_type) { 'application/vnd.openxmlformats-officedocument.wordprocessingml.document' }
-
-      it { is_expected.to eq 'hyrax/file_sets/media_display/office_document' }
-    end
-
-    context "with anything else" do
-      let(:mime_type) { 'application/binary' }
-
-      it { is_expected.to eq 'hyrax/file_sets/media_display/default' }
-    end
-  end
-end

data/spec/helpers/hyrax/iiif_helper_spec.rb

@@ -1,44 +0,0 @@
-RSpec.describe Hyrax::IiifHelper, type: :helper do
-  let(:solr_document) { SolrDocument.new }
-  let(:request) { double }
-  let(:ability) { nil }
-  let(:presenter) { Hyrax::WorkShowPresenter.new(solr_document, ability, request) }
-  let(:uv_partial_path) { 'hyrax/base/iiif_viewers/universal_viewer' }
-
-  describe '#iiif_viewer_display' do
-    before do
-      allow(helper).to receive(:iiif_viewer_display_partial).with(presenter)
-                                                            .and_return(uv_partial_path)
-    end
-
-    it "renders a partial" do
-      expect(helper).to receive(:render)
-        .with(uv_partial_path, presenter: presenter)
-      helper.iiif_viewer_display(presenter)
-    end
-
-    it "takes options" do
-      expect(helper).to receive(:render)
-        .with(uv_partial_path, presenter: presenter, transcript_id: '123')
-      helper.iiif_viewer_display(presenter, transcript_id: '123')
-    end
-  end
-
-  describe '#iiif_viewer_display_partial' do
-    subject { helper.iiif_viewer_display_partial(presenter) }
-
-    it 'defaults to universal viewer' do
-      expect(subject).to eq uv_partial_path
-    end
-
-    context "with #iiif_viewer override" do
-      let(:iiif_viewer) { :mirador }
-
-      before do
-        allow(presenter).to receive(:iiif_viewer).and_return(iiif_viewer)
-      end
-
-      it { is_expected.to eq 'hyrax/base/iiif_viewers/mirador' }
-    end
-  end
-end

data/spec/helpers/hyrax/trophy_helper_spec.rb

@@ -1,37 +0,0 @@
-RSpec.describe Hyrax::TrophyHelper, type: :helper do
-  describe "#display_trophy_link" do
-    let(:user) { create(:user) }
-    let(:id) { '9999' }
-
-    let(:text_attributes) { '[data-add-text="Highlight Work on Profile"][data-remove-text="Unhighlight Work"]' }
-    let(:url_attribute) { "[data-url=\"/works/#{id}/trophy\"]" }
-
-    context "when there is no trophy" do
-      it "has a link for highlighting" do
-        out = helper.display_trophy_link(user, id) { |text| "foo #{text} bar" }
-        node = Capybara::Node::Simple.new(out)
-        expect(node).to have_selector("a.trophy-class.trophy-off#{text_attributes}#{url_attribute}")
-        expect(node).to have_link 'foo Highlight Work on Profile bar', href: '#'
-      end
-    end
-
-    context "when there is a trophy" do
-      before do
-        user.trophies.create(work_id: id)
-      end
-
-      it "has a link for highlighting" do
-        out = helper.display_trophy_link(user, id) { |text| "foo #{text} bar" }
-        node = Capybara::Node::Simple.new(out)
-        expect(node).to have_selector("a.trophy-class.trophy-on#{text_attributes}#{url_attribute}")
-        expect(node).to have_link 'foo Unhighlight Work bar', href: '#'
-      end
-
-      it "allows removerow to be passed" do
-        out = helper.display_trophy_link(user, id, data: { removerow: true }) { |text| "foo #{text} bar" }
-        node = Capybara::Node::Simple.new(out)
-        expect(node).to have_selector("a.trophy-class.trophy-on[data-removerow=\"true\"]#{text_attributes}#{url_attribute}")
-      end
-    end
-  end
-end

data/spec/helpers/hyrax_helper_spec.rb

@@ -1,399 +0,0 @@
-def new_state
-  Blacklight::SearchState.new({}, CatalogController.blacklight_config)
-end
-
-RSpec.describe HyraxHelper, type: :helper do
-  describe "show_transfer_request_title" do
-    let(:sender) { create(:user) }
-    let(:user) { create(:user) }
-
-    context "when work is canceled" do
-      let(:request) do
-        instance_double(ProxyDepositRequest,
-                        deleted_work?: false,
-                        canceled?: true,
-                        to_s: 'Test work')
-      end
-
-      subject { helper.show_transfer_request_title request }
-
-      it { expect(subject).to eq 'Test work' }
-    end
-  end
-
-  describe '#render_notifications' do
-    let(:mailbox) { double('mailbox', unread_count: unread_count, label: 'Foobar') }
-
-    subject { helper.render_notifications(user: nil) }
-
-    before do
-      allow(UserMailbox).to receive(:new).and_return(mailbox)
-    end
-
-    context 'with unread messages' do
-      let(:unread_count) { 10 }
-
-      it 'renders with label-danger and is visible' do
-        expect(subject).to eq '<a aria-label="Foobar" class="notify-number" href="/notifications"><span class="fa fa-bell"></span>' \
-                              "\n" \
-                              '<span class="count label label-danger">10</span></a>'
-      end
-    end
-
-    context 'with no unread messages' do
-      let(:unread_count) { 0 }
-
-      it 'renders with label-default and is invisible' do
-        expect(subject).to eq '<a aria-label="Foobar" class="notify-number" href="/notifications"><span class="fa fa-bell"></span>' \
-                              "\n" \
-                              '<span class="count label invisible label-default">0</span></a>'
-      end
-    end
-  end
-
-  describe '#available_translations' do
-    subject { helper.available_translations }
-
-    it do
-      is_expected.to eq('de' => 'Deutsch',
-                        'en' => 'English',
-                        'es' => 'Español',
-                        'fr' => 'Français',
-                        'it' => 'Italiano',
-                        'pt-BR' => 'Português do Brasil',
-                        'zh' => '中文')
-    end
-  end
-
-  context 'link helpers' do
-    before do
-      allow(helper).to receive(:search_action_path) do |*args|
-        search_catalog_path(*args)
-      end
-      allow(helper).to receive(:search_state).and_return(new_state)
-    end
-
-    describe "#link_to_facet_list" do
-      context "with values" do
-        subject { helper.link_to_facet_list(['car', 'truck'], 'vehicle_type') }
-
-        it "joins the values" do
-          expect(helper).to receive(:search_state).exactly(2).times
-          car_link   = search_catalog_path(f: { 'vehicle_type_sim' => ['car'] })
-          truck_link = search_catalog_path(f: { 'vehicle_type_sim' => ['truck'] })
-          expect(subject).to eq "<a href=\"#{car_link}\">car</a>, <a href=\"#{truck_link}\">truck</a>"
-          expect(subject).to be_html_safe
-        end
-      end
-
-      context "without values" do
-        subject { helper.link_to_facet_list([], 'vehicle_type') }
-
-        it "shows the default text" do
-          expect(subject).to eq "No value entered"
-        end
-      end
-    end
-
-    describe '#index_field_link' do
-      let(:args) do
-        {
-          config: { field_name: 'contributor' },
-          value: ['Fritz Lang', 'Mel Brooks']
-        }
-      end
-
-      subject { helper.index_field_link(args) }
-
-      it 'returns a link' do
-        expect(subject).to be_html_safe
-        expect(subject).to eq '<a href="/catalog?q=%22Fritz+Lang%22&amp;search_field=contributor">Fritz Lang</a>, ' \
-                            + '<a href="/catalog?q=%22Mel+Brooks%22&amp;search_field=contributor">Mel Brooks</a>'
-      end
-    end
-  end
-
-  describe "#link_to_each_facet_field" do
-    subject { helper.link_to_each_facet_field(options) }
-
-    context "with helper_facet and default separator" do
-      let(:options) { { config: { helper_facet: Solrizer.solr_name("document_types", :facetable).to_sym }, value: ["Imaging > Object Photography"] } }
-
-      it do
-        is_expected.to eq("<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Imaging\">" \
-                             "Imaging</a> &gt; " \
-                             "<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Object+Photography\">Object Photography</a>")
-      end
-    end
-
-    context "with helper_facet and optional separator" do
-      let(:options) { { config: { helper_facet: Solrizer.solr_name("document_types", :facetable).to_sym, separator: " : " }, value: ["Imaging : Object Photography"] } }
-
-      it do
-        is_expected.to eq("<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Imaging\">" \
-                             "Imaging</a> : " \
-                             "<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Object+Photography\">Object Photography</a>")
-      end
-    end
-
-    context "with :output_separator" do
-      let(:options) { { config: { helper_facet: Solrizer.solr_name("document_types", :facetable).to_sym, output_separator: ' ~ ', separator: ":" }, value: ["Imaging : Object Photography"] } }
-
-      it do
-        is_expected.to eq("<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Imaging\">" \
-                             "Imaging</a> ~ " \
-                             "<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Object+Photography\">Object Photography</a>")
-      end
-    end
-
-    context "with :no_spaces_around_separator" do
-      let(:options) { { config: { helper_facet: Solrizer.solr_name("document_types", :facetable).to_sym, output_separator: '~', separator: ":" }, value: ["Imaging : Object Photography"] } }
-
-      it do
-        is_expected.to eq("<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Imaging\">" \
-                             "Imaging</a>~" \
-                             "<a href=\"/catalog?f%5Bdocument_types_sim%5D%5B%5D=Object+Photography\">Object Photography</a>")
-      end
-    end
-  end
-
-  describe "has_collection_search_parameters?" do
-    subject { helper }
-
-    context "when cq is set" do
-      before { allow(helper).to receive(:params).and_return(cq: 'foo') }
-      it { is_expected.to have_collection_search_parameters }
-    end
-    context "when cq is not set" do
-      before { allow(helper).to receive(:params).and_return(cq: '') }
-      it { is_expected.not_to have_collection_search_parameters }
-    end
-  end
-
-  describe "#collection_thumbnail" do
-    let(:document) { SolrDocument.new(has_model_ssim: ['Collection']) }
-
-    subject { helper.collection_thumbnail(document) }
-
-    it { is_expected.to eq '<span class="fa fa-cubes collection-icon-search"></span>' }
-  end
-
-  describe "#link_to_telephone" do
-    subject { helper.link_to_telephone(user) }
-
-    context "when user is set" do
-      let(:user) { mock_model(User, telephone: '867-5309') }
-
-      it { is_expected.to eq "<a href=\"wtai://wp/mc;867-5309\">867-5309</a>" }
-    end
-    context "when user is not set" do
-      let(:user) { nil }
-
-      it { is_expected.to be_nil }
-    end
-  end
-
-  describe "#current_search_parameters" do
-    context "when the user is not in the dashboard" do
-      it "is whatever q is" do
-        allow(helper).to receive(:params).and_return(controller: "catalog", q: "foo")
-        expect(helper.current_search_parameters).to eq("foo")
-      end
-    end
-
-    context "when the user is on any dashboard page" do
-      it "is ignored on dashboard" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/dashboard", q: "foo")
-        expect(helper.current_search_parameters).to be_nil
-      end
-
-      it "is ignored on dashboard works, collections, highlights and shares" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/works", q: "foo")
-        expect(helper.current_search_parameters).to be_nil
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/collections", q: "foo")
-        expect(helper.current_search_parameters).to be_nil
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/highlights", q: "foo")
-        expect(helper.current_search_parameters).to be_nil
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/shares", q: "foo")
-        expect(helper.current_search_parameters).to be_nil
-      end
-    end
-  end
-
-  describe "#search_form_action" do
-    context "when the user is not in the dashboard" do
-      it "returns the catalog index path" do
-        allow(helper).to receive(:params).and_return(controller: "foo")
-        expect(helper.search_form_action).to eq(search_catalog_path)
-      end
-    end
-
-    context "when the user is on the dashboard page" do
-      it "defaults to My Works" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/dashboard")
-        expect(helper.search_form_action).to eq(hyrax.my_works_path)
-      end
-    end
-
-    context "when the user is on the 'all works' tab on the dashboard page" do
-      it "returns the dashboard works path" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/dashboard/works")
-        expect(helper.search_form_action).to eq(hyrax.dashboard_works_path)
-      end
-    end
-
-    context "when the user is on the my works page" do
-      it "returns the my dashboard works path" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/works")
-        expect(helper.search_form_action).to eq(hyrax.my_works_path)
-      end
-    end
-
-    context "when the user is on the my collections page" do
-      it "returns the my dashboard collections path" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/collections")
-        expect(helper.search_form_action).to eq(hyrax.my_collections_path)
-      end
-    end
-
-    context "when the user is on the all collections page" do
-      it "returns the all dashboard collections path" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/dashboard/collections")
-        expect(helper.search_form_action).to eq(hyrax.dashboard_collections_path)
-      end
-    end
-
-    context "when the user is on the my highlights page" do
-      it "returns the my dashboard highlights path" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/highlights")
-        expect(helper.search_form_action).to eq(hyrax.dashboard_highlights_path)
-      end
-    end
-
-    context "when the user is on the my shares page" do
-      it "returns the my dashboard shares path" do
-        allow(helper).to receive(:params).and_return(controller: "hyrax/my/shares")
-        expect(helper.search_form_action).to eq(hyrax.dashboard_shares_path)
-      end
-    end
-  end
-
-  describe '#zotero_label' do
-    subject { helper }
-
-    it { is_expected.to respond_to(:zotero_label) }
-  end
-
-  describe "#iconify_auto_link" do
-    let(:text)              { 'Foo < http://www.example.com. & More text' }
-    let(:linked_text)       { 'Foo &lt; <a href="http://www.example.com"><span class="glyphicon glyphicon-new-window"></span> http://www.example.com</a>. &amp; More text' }
-    let(:document)          { SolrDocument.new(has_model_ssim: ['GenericWork'], id: 512, title_tesim: text, description_tesim: text) }
-    let(:blacklight_config) { CatalogController.blacklight_config }
-
-    before do
-      allow(controller).to receive(:action_name).and_return('index')
-      allow(helper).to receive(:blacklight_config).and_return(blacklight_config)
-    end
-    it "boring String argument" do
-      expect(helper.iconify_auto_link('no escapes or links necessary')).to eq 'no escapes or links necessary'
-      expect(helper.iconify_auto_link('no escapes or links necessary', false)).to eq 'no escapes or links necessary'
-    end
-    context "interesting String argument" do
-      subject { helper.iconify_auto_link(text) }
-
-      it "escapes input" do
-        expect(subject).to start_with('Foo &lt;').and end_with('. &amp; More text')
-      end
-      it "adds links" do
-        expect(subject).to include('<a href="http://www.example.com">')
-      end
-      it "adds icons" do
-        expect(subject).to include('class="glyphicon glyphicon-new-window"')
-      end
-    end
-
-    context "when using a hash argument" do
-      subject { helper.iconify_auto_link(arg) }
-
-      describe "auto-linking in the title" do
-        let(:arg) { { document: document, value: [text], config: blacklight_config.index_fields['title_tesim'], field: 'title_tesim' } }
-
-        it { is_expected.to eq(linked_text) }
-      end
-
-      describe "auto-linking in the description" do
-        let(:arg) { { document: document, value: [text], config: blacklight_config.index_fields['description_tesim'], field: 'description_tesim' } }
-
-        it { is_expected.to eq(linked_text) }
-      end
-    end
-  end
-
-  describe "#license_links" do
-    it "maps the url to a link with a label" do
-      expect(helper.license_links(
-               value: ["http://creativecommons.org/publicdomain/zero/1.0/"]
-      )).to eq("<a href=\"http://creativecommons.org/publicdomain/zero/1.0/\">Creative Commons CC0 1.0 Universal</a>")
-    end
-
-    it "converts multiple licenses to a sentence" do
-      expect(helper.license_links(
-               value: ["http://creativecommons.org/publicdomain/zero/1.0/",
-                       "http://creativecommons.org/publicdomain/mark/1.0/",
-                       "http://www.europeana.eu/portal/rights/rr-r.html"]
-      )).to eq("<a href=\"http://creativecommons.org/publicdomain/zero/1.0/\">Creative Commons CC0 1.0 Universal</a>, " \
-               "<a href=\"http://creativecommons.org/publicdomain/mark/1.0/\">Creative Commons Public Domain Mark 1.0</a>, " \
-               "and <a href=\"http://www.europeana.eu/portal/rights/rr-r.html\">All rights reserved</a>")
-    end
-  end
-
-  describe "#rights_statment_links" do
-    it "maps the url to a link with a label" do
-      expect(helper.rights_statement_links(
-               value: ["http://rightsstatements.org/vocab/InC/1.0/"]
-      )).to eq("<a href=\"http://rightsstatements.org/vocab/InC/1.0/\">In Copyright</a>")
-    end
-  end
-
-  describe "#human_readable_date" do
-    it "ensures that the display of the date is human-readable" do
-      expect(helper.human_readable_date(value: ["2016-08-15T00:00:00Z"])).to eq("08/15/2016")
-    end
-  end
-
-  describe "#banner_image" do
-    it "returns the configured Hyrax banner image" do
-      expect(helper.banner_image).to eq(Hyrax.config.banner_image)
-    end
-  end
-
-  describe "#collection_title_by_id" do
-    let(:solr_doc) { double(id: "abcd12345") }
-    let(:bad_solr_doc) { double(id: "efgh67890") }
-    let(:solr_response) { double(docs: [solr_doc]) }
-    let(:bad_solr_response) { double(docs: [bad_solr_doc]) }
-    let(:empty_solr_response) { double(docs: []) }
-    let(:repository) { double }
-
-    before do
-      allow(controller).to receive(:repository).and_return(repository)
-      allow(solr_doc).to receive(:[]).with("title_tesim").and_return(["Collection of Awesomeness"])
-      allow(bad_solr_doc).to receive(:[]).with("title_tesim").and_return(nil)
-      allow(repository).to receive(:find).with("abcd12345").and_return(solr_response)
-      allow(repository).to receive(:find).with("efgh67890").and_return(bad_solr_response)
-      allow(repository).to receive(:find).with("bad-id").and_return(empty_solr_response)
-    end
-
-    it "returns the first title of the collection" do
-      expect(helper.collection_title_by_id("abcd12345")).to eq "Collection of Awesomeness"
-    end
-
-    it "returns nil if collection doesn't have title_tesim field" do
-      expect(helper.collection_title_by_id("efgh67890")).to eq nil
-    end
-
-    it "returns nil if collection not found" do
-      expect(helper.collection_title_by_id("bad-id")).to eq nil
-    end
-  end
-end