hyrax 2.0.3 → 2.1.0.beta1

data/spec/presenters/hyrax/file_set_presenter_spec.rb

@@ -1,3 +1,5 @@
+require 'iiif_manifest'
+
 RSpec.describe Hyrax::FileSetPresenter do
   let(:solr_document) { SolrDocument.new(attributes) }
   let(:ability) { double "Ability" }
@@ -115,19 +117,31 @@ RSpec.describe Hyrax::FileSetPresenter do
     it { is_expected.to eq 'FileSet' }
   end
 
-  describe "#events" do
+  describe '#events' do
     subject(:events) { presenter.events }
 
-    let(:store) { double }
+    let(:event_stream) { double }
     let(:response) { double }
 
-    it "calls the event store" do
-      expect(Hyrax::RedisEventStore).to receive(:for).with('FileSet:123abc:event').and_return(store)
-      allow(store).to receive(:fetch).with(100).and_return(response)
+    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') }
 
@@ -265,4 +279,109 @@ RSpec.describe Hyrax::FileSetPresenter do
       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, nil, request) }
+    let(:id) { CGI.escape(file_set.original_file.id) }
+
+    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') }
+
+          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
+        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

data/spec/presenters/hyrax/inspect_work_presenter_spec.rb

@@ -1,4 +1,4 @@
-RSpec.describe Hyrax::InspectWorkPresenter, no_clean: true do
+RSpec.describe Hyrax::InspectWorkPresenter do
   let(:solr_document) { SolrDocument.new(attributes) }
   let(:attributes) do
     { "id" => '888888',

data/spec/presenters/hyrax/menu_presenter_spec.rb

@@ -17,6 +17,11 @@ RSpec.describe Hyrax::MenuPresenter do
     context "for the PagesController" do
       let(:controller_name) { Hyrax::PagesController.controller_name }
 
+      it { is_expected.to be true }
+    end
+    context "for the CollectionTypesController" do
+      let(:controller_name) { Hyrax::Admin::CollectionTypesController.controller_name }
+
       it { is_expected.to be true }
     end
   end

data/spec/presenters/hyrax/select_collection_type_list_presenter_spec.rb

@@ -0,0 +1,59 @@
+RSpec.describe Hyrax::SelectCollectionTypeListPresenter, :clean_repo do
+  let(:user) { create(:user) }
+  let(:instance) { described_class.new(user) }
+  let(:collection_type) { create(:collection_type, creator_user: user) }
+  let(:user_collection_type) { create(:user_collection_type) }
+
+  describe "#many?" do
+    let(:subject) { instance.many? }
+
+    it "finds only one" do
+      expect(subject).to be false
+    end
+
+    it "finds many" do
+      collection_type
+      user_collection_type
+      expect(subject).to be true
+    end
+  end
+
+  describe "#any?" do
+    let(:subject) { instance.any? }
+
+    it 'finds no collection types' do
+      expect(subject).to be false
+    end
+
+    it 'finds a collection type' do
+      collection_type
+      expect(subject).to be true
+    end
+  end
+
+  describe "authorized_collection_types" do
+    let(:subject) { instance.authorized_collection_types }
+
+    it 'returns an array of Hyrax::CollectionTypes' do
+      expect(Hyrax::CollectionTypes::PermissionsService).to receive(:can_create_collection_types).with(user: user).and_return(:the_response)
+      expect(subject).to equal(:the_response)
+    end
+  end
+
+  describe "#first_collection_type" do
+    let(:subject) { instance.first_collection_type }
+
+    it 'returns the first collection_type' do
+      collection_type
+      user_collection_type
+      expect(subject).to eq(Hyrax::CollectionType.first)
+    end
+  end
+
+  describe "#each" do
+    it 'iterates through all collection types' do
+      expect(Hyrax::CollectionTypes::PermissionsService).to receive(:can_create_collection_types).with(user: user).and_return([:the_response])
+      expect { |block| instance.each(&block) }.to yield_successive_args(kind_of(Hyrax::SelectCollectionTypePresenter))
+    end
+  end
+end

data/spec/presenters/hyrax/select_collection_type_presenter_spec.rb

@@ -0,0 +1,10 @@
+RSpec.describe Hyrax::SelectCollectionTypePresenter do
+  subject { described_class.new(collection_type) }
+
+  let(:collection_type) { create(:collection_type) }
+
+  it { is_expected.to delegate_method(:title).to(:collection_type) }
+  it { is_expected.to delegate_method(:description).to(:collection_type) }
+  it { is_expected.to delegate_method(:admin_set?).to(:collection_type) }
+  it { is_expected.to delegate_method(:id).to(:collection_type) }
+end

data/spec/presenters/hyrax/select_type_presenter_spec.rb

@@ -34,6 +34,12 @@ RSpec.describe Hyrax::SelectTypePresenter do
       it { is_expected.to eq "/concern/#{model.to_s.tableize}/new?add_works_to_collection=#{collection_id}" }
     end
 
+    context 'with id and controller params' do
+      let(:params) { { id: collection_id, controller: 'hyrax/dashboard/collections' } }
+
+      it { is_expected.to eq "/concern/#{model.to_s.tableize}/new?add_works_to_collection=#{collection_id}" }
+    end
+
     context 'with no params' do
       let(:params) { {} }
 
@@ -53,6 +59,12 @@ RSpec.describe Hyrax::SelectTypePresenter do
       it { is_expected.to eq "/batch_uploads/new?add_works_to_collection=#{collection_id}&payload_concern=#{model}" }
     end
 
+    context 'with id and controller params' do
+      let(:params) { { id: collection_id, controller: 'hyrax/dashboard/collections' } }
+
+      it { is_expected.to eq "/batch_uploads/new?add_works_to_collection=#{collection_id}&payload_concern=#{model}" }
+    end
+
     context 'with no params' do
       let(:params) { {} }
 

data/spec/presenters/hyrax/work_show_presenter_spec.rb

@@ -1,6 +1,6 @@
 RSpec.describe Hyrax::WorkShowPresenter do
   let(:solr_document) { SolrDocument.new(attributes) }
-  let(:request) { double(host: 'example.org') }
+  let(:request) { double(host: 'example.org', base_url: 'http://example.org') }
   let(:user_key) { 'a_user_key' }
 
   let(:attributes) do
@@ -37,6 +37,64 @@ RSpec.describe Hyrax::WorkShowPresenter do
     it { is_expected.to be_kind_of ActiveModel::Name }
   end
 
+  describe '#manifest_url' do
+    subject { presenter.manifest_url }
+
+    it { is_expected.to eq 'http://example.org/concern/generic_works/888888/manifest' }
+  end
+
+  describe '#universal_viewer?' do
+    let(:id_present) { false }
+    let(:representative_presenter) { double('representative', present?: false) }
+    let(:image_boolean) { false }
+    let(:iiif_enabled) { false }
+
+    before do
+      allow(presenter).to receive(:representative_id).and_return(id_present)
+      allow(presenter).to receive(:representative_presenter).and_return(representative_presenter)
+      allow(representative_presenter).to receive(:image?).and_return(image_boolean)
+      allow(Hyrax.config).to receive(:iiif_image_server?).and_return(iiif_enabled)
+    end
+
+    subject { presenter.universal_viewer? }
+
+    context 'with no representative_id' do
+      it { is_expected.to be false }
+    end
+
+    context 'with no representative_presenter' do
+      let(:id_present) { true }
+
+      it { is_expected.to be false }
+    end
+
+    context 'with non-image representative_presenter' do
+      let(:id_present) { true }
+      let(:representative_presenter) { double('representative', present?: true) }
+      let(:image_boolean) { true }
+
+      it { is_expected.to be false }
+    end
+
+    context 'with IIIF image server turned off' do
+      let(:id_present) { true }
+      let(:representative_presenter) { double('representative', present?: true) }
+      let(:image_boolean) { true }
+      let(:iiif_enabled) { false }
+
+      it { is_expected.to be false }
+    end
+
+    context 'with representative image and IIIF turned on' do
+      let(:id_present) { true }
+      let(:representative_presenter) { double('representative', present?: true) }
+      let(:image_boolean) { true }
+      let(:iiif_enabled) { true }
+
+      it { is_expected.to be true }
+    end
+  end
+
   describe '#stats_path' do
     let(:user) { 'sarah' }
     let(:ability) { double "Ability" }
@@ -198,6 +256,27 @@ RSpec.describe Hyrax::WorkShowPresenter do
         .and_return ["abc"]
       expect(presenter.representative_presenter).to eq("abc")
     end
+
+    context 'without a representative' do
+      let(:obj) { create(:work) }
+
+      it 'has a nil presenter' do
+        expect(presenter.representative_presenter).to be_nil
+      end
+    end
+
+    context 'when it is its own representative' do
+      let(:obj) { create(:work) }
+
+      before do
+        obj.representative_id = obj.id
+        obj.save
+      end
+
+      it 'has a nil presenter; avoids infinite loop' do
+        expect(presenter.representative_presenter).to be_nil
+      end
+    end
   end
 
   describe "#download_url" do
@@ -321,4 +400,40 @@ RSpec.describe Hyrax::WorkShowPresenter do
       end
     end
   end
+
+  describe "#manifest" do
+    let(:work) { create(:work_with_one_file) }
+    let(:solr_document) { SolrDocument.new(work.to_solr) }
+
+    describe "#sequence_rendering" do
+      subject do
+        presenter.sequence_rendering
+      end
+
+      before do
+        Hydra::Works::AddFileToFileSet.call(work.file_sets.first,
+                                            File.open(fixture_path + '/world.png'), :original_file)
+      end
+
+      it "returns a hash containing the rendering information" do
+        work.rendering_ids = [work.file_sets.first.id]
+        expect(subject).to be_an Array
+      end
+    end
+
+    describe "#manifest_metadata" do
+      subject do
+        presenter.manifest_metadata
+      end
+
+      before do
+        work.title = ['Test title', 'Another test title']
+      end
+
+      it "returns an array of metadata values" do
+        expect(subject[0]['label']).to eq('Title')
+        expect(subject[0]['value']).to include('Test title', 'Another test title')
+      end
+    end
+  end
 end

data/spec/presenters/hyrax/workflow_presenter_spec.rb

@@ -1,4 +1,4 @@
-RSpec.describe Hyrax::WorkflowPresenter, no_clean: true do
+RSpec.describe Hyrax::WorkflowPresenter do
   let(:solr_document) { SolrDocument.new(attributes) }
   let(:attributes) do
     { "id" => '888888',

data/spec/renderers/hyrax/renderers/attribute_renderer_spec.rb

@@ -23,8 +23,8 @@ RSpec.describe Hyrax::Renderers::AttributeRenderer do
       end
       let(:tr_content) do
         "<tr><th>Name</th>\n" \
-         "<td><ul class='tabular'><li class=\"attribute name\">Bob</li>\n" \
-         "<li class=\"attribute name\">Jessica</li>\n" \
+         "<td><ul class='tabular'><li class=\"attribute attribute-name\">Bob</li>\n" \
+         "<li class=\"attribute attribute-name\">Jessica</li>\n" \
          "</ul></td></tr>"
       end
 
@@ -37,7 +37,7 @@ RSpec.describe Hyrax::Renderers::AttributeRenderer do
       let(:renderer) { described_class.new(field, [567]) }
       let(:tr_content) do
         "<tr><th>Height</th>\n" \
-         "<td><ul class='tabular'><li class=\"attribute height\">567</li>\n" \
+         "<td><ul class='tabular'><li class=\"attribute attribute-height\">567</li>\n" \
          "</ul></td></tr>"
       end
 
@@ -47,9 +47,9 @@ RSpec.describe Hyrax::Renderers::AttributeRenderer do
     context 'with microdata enabled' do
       let(:tr_content) do
         "<tr><th>Name</th>\n" \
-         "<td><ul class='tabular'><li class=\"attribute name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
+         "<td><ul class='tabular'><li class=\"attribute attribute-name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
          "<span itemprop=\"firstName\">Bob</span></li>\n" \
-         "<li class=\"attribute name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
+         "<li class=\"attribute attribute-name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
          "<span itemprop=\"firstName\">Jessica</span></li>\n" \
          "</ul></td></tr>"
       end
@@ -63,7 +63,7 @@ RSpec.describe Hyrax::Renderers::AttributeRenderer do
       let(:renderer) { described_class.new(field, ['Foo < Bar http://www.example.com. & More Text']) }
       let(:tr_content) do
         "<tr><th>Description</th>\n" \
-         "<td><ul class='tabular'><li class=\"attribute description\">" \
+         "<td><ul class='tabular'><li class=\"attribute attribute-description\">" \
          "<span itemprop=\"description\">Foo &lt; Bar " \
          "<a href=\"http://www.example.com\">http://www.example.com</a>. &amp; More Text</span></li>\n" \
          "</ul></td></tr>"