hyrax 2.6.0 → 2.9.0

data/spec/presenters/hyrax/dashboard/user_presenter_spec.rb

@@ -1,65 +0,0 @@
-RSpec.describe Hyrax::Dashboard::UserPresenter do
-  let(:since) { nil }
-  let(:context) { ActionView::TestCase::TestController.new.view_context }
-  let(:user) { create(:user) }
-  let(:instance) { described_class.new(user, context, since) }
-
-  describe "#activity" do
-    subject { instance.activity }
-
-    let(:activity) { double }
-
-    before do
-      allow(user).to receive(:all_user_activity).and_return(activity)
-    end
-
-    it { is_expected.to eq activity }
-  end
-
-  describe "#notifications" do
-    subject { instance.notifications }
-
-    context "when the user has mail" do
-      let(:user) { create(:user_with_mail) }
-
-      it { is_expected.to be_truthy }
-    end
-
-    context "when the user doesn't have mail" do
-      it { is_expected.to be_empty }
-    end
-  end
-
-  describe "#transfers" do
-    subject { instance.transfers }
-
-    it { is_expected.to be_instance_of Hyrax::TransfersPresenter }
-  end
-
-  describe "#render_recent_activity" do
-    subject(:rendered) { instance.render_recent_activity }
-
-    context "when there is no activity" do
-      before do
-        allow(instance).to receive(:activity).and_return([])
-      end
-
-      it "returns a messages stating the user has no recent activity" do
-        expect(rendered).to eq "User has no recent activity"
-      end
-    end
-  end
-
-  describe "#render_recent_notifications" do
-    subject(:rendered) { instance.render_recent_notifications }
-
-    context "when there are no notifications" do
-      before do
-        allow(instance).to receive(:notifications).and_return([])
-      end
-      it "returns a messages stating the user has no recent notifications" do
-        expect(rendered).to eq "User has no notifications"
-      end
-    end
-  end
-end

data/spec/presenters/hyrax/embargo_presenter_spec.rb

@@ -1,51 +0,0 @@
-RSpec.describe Hyrax::EmbargoPresenter do
-  let(:document) { SolrDocument.new(attributes) }
-  let(:presenter) { described_class.new(document) }
-  let(:attributes) { {} }
-
-  describe "visibility" do
-    subject { presenter.visibility }
-
-    it { is_expected.to eq 'restricted' }
-  end
-
-  describe "to_s" do
-    let(:attributes) { { 'title_tesim' => ['Hey guys!'] } }
-
-    subject { presenter.to_s }
-
-    it { is_expected.to eq 'Hey guys!' }
-  end
-
-  describe "human_readable_type" do
-    let(:attributes) { { 'human_readable_type_tesim' => ['File'] } }
-
-    subject { presenter.human_readable_type }
-
-    it { is_expected.to eq 'File' }
-  end
-
-  describe "embargo_release_date" do
-    let(:attributes) { { 'embargo_release_date_dtsi' => '2015-10-15T00:00:00Z' } }
-
-    subject { presenter.embargo_release_date }
-
-    it { is_expected.to eq '15 Oct 2015' }
-  end
-
-  describe "visibility_after_embargo" do
-    let(:attributes) { { 'visibility_after_embargo_ssim' => ['restricted'] } }
-
-    subject { presenter.visibility_after_embargo }
-
-    it { is_expected.to eq 'restricted' }
-  end
-
-  describe "embargo_history" do
-    let(:attributes) { { 'embargo_history_ssim' => ['This is in the past'] } }
-
-    subject { presenter.embargo_history }
-
-    it { is_expected.to eq ['This is in the past'] }
-  end
-end

data/spec/presenters/hyrax/file_set_presenter_spec.rb

@@ -1,426 +0,0 @@
-require 'iiif_manifest'
-
-RSpec.describe Hyrax::FileSetPresenter do
-  let(:solr_document) { SolrDocument.new(attributes) }
-  let(:ability) { double "Ability" }
-  let(:presenter) { described_class.new(solr_document, ability) }
-  let(:attributes) { file.to_solr }
-  let(:file) do
-    build(:file_set,
-          id: '123abc',
-          user: user,
-          title: ["File title"],
-          depositor: user.user_key,
-          label: "filename.tif")
-  end
-  let(:user) { double(user_key: 'sarah') }
-
-  describe 'stats_path' do
-    before do
-      # https://github.com/samvera/active_fedora/issues/1251
-      allow(file).to receive(:persisted?).and_return(true)
-    end
-    it { expect(presenter.stats_path).to eq Hyrax::Engine.routes.url_helpers.stats_file_path(id: file, locale: 'en') }
-  end
-
-  subject { presenter }
-
-  describe "#to_s" do
-    subject { presenter.to_s }
-
-    it { is_expected.to eq 'File title' }
-  end
-
-  describe "#human_readable_type" do
-    subject { presenter.human_readable_type }
-
-    it { is_expected.to eq 'File' }
-  end
-
-  describe "#model_name" do
-    subject { presenter.model_name }
-
-    it { is_expected.to be_kind_of ActiveModel::Name }
-  end
-
-  describe "#to_partial_path" do
-    subject { presenter.to_partial_path }
-
-    it { is_expected.to eq 'file_sets/file_set' }
-  end
-
-  describe "office_document?" do
-    subject { presenter.office_document? }
-
-    it { is_expected.to be false }
-  end
-
-  describe "#user_can_perform_any_action?" do
-    subject { presenter.user_can_perform_any_action? }
-    let(:current_ability) { ability }
-
-    context 'when user can perform at least 1 action' do
-      before do
-        expect(current_ability).to receive(:can?).with(:edit, presenter.id).and_return false
-        expect(current_ability).to receive(:can?).with(:destroy, presenter.id).and_return false
-        expect(current_ability).to receive(:can?).with(:download, presenter.id).and_return true
-      end
-
-      it { is_expected.to be true }
-    end
-    context 'when user cannot perform any action' do
-      before do
-        expect(current_ability).to receive(:can?).with(:edit, presenter.id).and_return false
-        expect(current_ability).to receive(:can?).with(:destroy, presenter.id).and_return false
-        expect(current_ability).to receive(:can?).with(:download, presenter.id).and_return false
-      end
-
-      it { is_expected.to be false }
-    end
-  end
-
-  describe "properties delegated to solr_document" do
-    let(:solr_properties) do
-      ["date_uploaded", "title_or_label",
-       "contributor", "creator", "title", "description", "publisher",
-       "subject", "language", "license", "format_label", "file_size",
-       "height", "width", "filename", "well_formed", "page_count",
-       "file_title", "last_modified", "original_checksum", "mime_type",
-       "duration", "sample_rate"]
-    end
-
-    it "delegates to the solr_document" do
-      solr_properties.each do |property|
-        expect(solr_document).to receive(property.to_sym)
-        presenter.send(property)
-      end
-    end
-    it { is_expected.to delegate_method(:depositor).to(:solr_document) }
-    it { is_expected.to delegate_method(:keyword).to(:solr_document) }
-    it { is_expected.to delegate_method(:date_created).to(:solr_document) }
-    it { is_expected.to delegate_method(:date_modified).to(:solr_document) }
-    it { is_expected.to delegate_method(:itemtype).to(:solr_document) }
-    it { is_expected.to delegate_method(:fetch).to(:solr_document) }
-    it { is_expected.to delegate_method(:first).to(:solr_document) }
-    it { is_expected.to delegate_method(:has?).to(:solr_document) }
-  end
-
-  describe '#link_name' do
-    context "with a user who can view the file" do
-      before do
-        allow(ability).to receive(:can?).with(:read, "123abc").and_return(true)
-      end
-      it "shows the title" do
-        expect(presenter.link_name).to eq 'File title'
-        expect(presenter.link_name).not_to eq 'filename.tif'
-      end
-    end
-
-    context "with a user who cannot view the file" do
-      before do
-        allow(ability).to receive(:can?).with(:read, "123abc").and_return(false)
-      end
-      it "hides the title" do
-        expect(presenter.link_name).to eq 'File'
-      end
-    end
-  end
-
-  describe '#tweeter' do
-    subject { presenter.tweeter }
-
-    it 'delegates the depositor as the user_key to TwitterPresenter.call' do
-      expect(Hyrax::TwitterPresenter).to receive(:twitter_handle_for).with(user_key: solr_document.depositor)
-      subject
-    end
-  end
-
-  describe "#event_class" do
-    subject { presenter.event_class }
-
-    it { is_expected.to eq 'FileSet' }
-  end
-
-  describe '#events' do
-    subject(:events) { presenter.events }
-
-    let(:event_stream) { double }
-    let(:response) { double }
-
-    before do
-      allow(presenter).to receive(:event_stream).and_return(event_stream)
-    end
-
-    it 'calls the event store' do
-      allow(event_stream).to receive(:fetch).with(100).and_return(response)
-      expect(events).to eq response
-    end
-  end
-
-  describe '#event_stream' do
-    let(:object_stream) { double }
-
-    it 'returns a Nest stream' do
-      expect(Hyrax::RedisEventStore).to receive(:for).with(Nest).and_return(object_stream)
-      presenter.send(:event_stream)
-    end
-  end
-
-  describe "characterization" do
-    let(:user) { double(user_key: 'user') }
-
-    describe "#characterization_metadata" do
-      subject { presenter.characterization_metadata }
-
-      it "only has set attributes are in the metadata" do
-        expect(subject[:height]).to be_blank
-        expect(subject[:page_count]).to be_blank
-      end
-
-      context "when height is set" do
-        let(:attributes) { { height_is: '444' } }
-
-        it "only has set attributes are in the metadata" do
-          expect(subject[:height]).not_to be_blank
-          expect(subject[:page_count]).to be_blank
-        end
-      end
-    end
-
-    describe "#characterized?" do
-      subject { presenter }
-
-      it { is_expected.not_to be_characterized }
-
-      context "when height is set" do
-        let(:attributes) { { height_is: '444' } }
-
-        it { is_expected.to be_characterized }
-      end
-
-      context "when file_format is set" do
-        let(:attributes) { { file_format_tesim: ['format'] } }
-
-        it { is_expected.to be_characterized }
-      end
-    end
-
-    describe "#label_for_term" do
-      subject { presenter.label_for_term(:titleized_key) }
-
-      it { is_expected.to eq("Titleized Key") }
-    end
-
-    describe "with additional characterization metadata" do
-      let(:additional_metadata) do
-        {
-          foo: ["bar"],
-          fud: ["bars", "cars"]
-        }
-      end
-
-      before { allow(presenter).to receive(:additional_characterization_metadata).and_return(additional_metadata) }
-      subject { presenter }
-
-      specify do
-        expect(subject).to be_characterized
-        expect(subject.characterization_metadata[:foo]).to contain_exactly("bar")
-        expect(subject.characterization_metadata[:fud]).to contain_exactly("bars", "cars")
-      end
-    end
-
-    describe "characterization values" do
-      before { allow(presenter).to receive(:characterization_metadata).and_return(mock_metadata) }
-
-      context "with a limited set of short values" do
-        let(:mock_metadata) { { term: ["asdf", "qwer"] } }
-
-        describe "#primary_characterization_values" do
-          subject { presenter.primary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly("asdf", "qwer") }
-        end
-        describe "#secondary_characterization_values" do
-          subject { presenter.secondary_characterization_values(:term) }
-
-          it { is_expected.to be_empty }
-        end
-      end
-
-      context "with a value set exceeding the configured amount" do
-        let(:mock_metadata) { { term: ["1", "2", "3", "4", "5", "6", "7", "8"] } }
-
-        describe "#primary_characterization_values" do
-          subject { presenter.primary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly("1", "2", "3", "4", "5") }
-        end
-        describe "#secondary_characterization_values" do
-          subject { presenter.secondary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly("6", "7", "8") }
-        end
-      end
-
-      context "with values exceeding 250 characters" do
-        let(:mock_metadata) { { term: [("a" * 251), "2", "3", "4", "5", "6", ("b" * 251)] } }
-
-        describe "#primary_characterization_values" do
-          subject { presenter.primary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly(("a" * 247) + "...", "2", "3", "4", "5") }
-        end
-        describe "#secondary_characterization_values" do
-          subject { presenter.secondary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly("6", (("b" * 247) + "...")) }
-        end
-      end
-
-      context "with a string as a value" do
-        let(:mock_metadata) { { term: "string" } }
-
-        describe "#primary_characterization_values" do
-          subject { presenter.primary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly("string") }
-        end
-        describe "#secondary_characterization_values" do
-          subject { presenter.secondary_characterization_values(:term) }
-
-          it { is_expected.to be_empty }
-        end
-      end
-
-      context "with an integer as a value" do
-        let(:mock_metadata) { { term: 1440 } }
-
-        describe "#primary_characterization_values" do
-          subject { presenter.primary_characterization_values(:term) }
-
-          it { is_expected.to contain_exactly("1440") }
-        end
-      end
-    end
-  end
-
-  describe 'IIIF integration' do
-    let(:file_set) { create(:file_set) }
-    let(:solr_document) { SolrDocument.new(file_set.to_solr) }
-    let(:request) { double(base_url: 'http://test.host') }
-    let(:presenter) { described_class.new(solr_document, ability, request) }
-    let(:id) { CGI.escape(file_set.original_file.id) }
-    let(:read_permission) { true }
-
-    before do
-      allow(ability).to receive(:can?).with(:read, solr_document.id).and_return(read_permission)
-    end
-
-    describe "#display_image" do
-      subject { presenter.display_image }
-
-      context 'without a file' do
-        let(:id) { 'bogus' }
-
-        it { is_expected.to be_nil }
-      end
-
-      context 'with a file' do
-        before do
-          Hydra::Works::AddFileToFileSet.call(file_set,
-                                              file_path, :original_file)
-        end
-
-        context "when the file is not an image" do
-          let(:file_path) { File.open(fixture_path + '/hyrax_generic_stub.txt') }
-
-          it { is_expected.to be_nil }
-        end
-
-        context "when the file is an image" do
-          let(:file_path) { File.open(fixture_path + '/world.png') }
-
-          before do
-            allow(solr_document).to receive(:image?).and_return(true)
-          end
-
-          it { is_expected.to be_instance_of IIIFManifest::DisplayImage }
-          its(:url) { is_expected.to eq "http://test.host/images/#{id}/full/600,/0/default.jpg" }
-
-          context 'with custom image size default' do
-            let(:custom_image_size) { '666,' }
-
-            around do |example|
-              default_image_size = Hyrax.config.iiif_image_size_default
-              Hyrax.config.iiif_image_size_default = custom_image_size
-              example.run
-              Hyrax.config.iiif_image_size_default = default_image_size
-            end
-
-            it { is_expected.to be_instance_of IIIFManifest::DisplayImage }
-            its(:url) { is_expected.to eq "http://test.host/images/#{id}/full/#{custom_image_size}/0/default.jpg" }
-          end
-
-          context 'with custom image url builder' do
-            let(:id) { file_set.original_file.id }
-            let(:custom_builder) do
-              ->(file_id, base_url, _size) { "#{base_url}/downloads/#{file_id.split('/').first}" }
-            end
-
-            around do |example|
-              default_builder = Hyrax.config.iiif_image_url_builder
-              Hyrax.config.iiif_image_url_builder = custom_builder
-              example.run
-              Hyrax.config.iiif_image_url_builder = default_builder
-            end
-
-            it { is_expected.to be_instance_of IIIFManifest::DisplayImage }
-            its(:url) { is_expected.to eq "http://test.host/downloads/#{id.split('/').first}" }
-          end
-
-          context "when the user doesn't have permission to view the image" do
-            let(:read_permission) { false }
-
-            it { is_expected.to be_nil }
-          end
-        end
-      end
-    end
-
-    describe "#iiif_endpoint" do
-      subject { presenter.send(:iiif_endpoint, file_set.original_file.id) }
-
-      before do
-        allow(Hyrax.config).to receive(:iiif_image_server?).and_return(riiif_enabled)
-        Hydra::Works::AddFileToFileSet.call(file_set,
-                                            File.open(fixture_path + '/world.png'), :original_file)
-      end
-
-      context 'with iiif_image_server enabled' do
-        let(:riiif_enabled) { true }
-
-        its(:url) { is_expected.to eq "http://test.host/images/#{id}" }
-        its(:profile) { is_expected.to eq 'http://iiif.io/api/image/2/level2.json' }
-
-        context 'with a custom iiif image profile' do
-          let(:custom_profile) { 'http://iiif.io/api/image/2/level1.json' }
-
-          around do |example|
-            default_profile = Hyrax.config.iiif_image_compliance_level_uri
-            Hyrax.config.iiif_image_compliance_level_uri = custom_profile
-            example.run
-            Hyrax.config.iiif_image_compliance_level_uri = default_profile
-          end
-
-          its(:profile) { is_expected.to eq custom_profile }
-        end
-      end
-
-      context 'with iiif_image_server disabled' do
-        let(:riiif_enabled) { false }
-
-        it { is_expected.to be nil }
-      end
-    end
-  end
-end