hyrax 2.6.0 → 2.7.0

data/spec/models/single_use_link_spec.rb

@@ -1,37 +0,0 @@
-RSpec.describe SingleUseLink do
-  let(:file) { FileSet.new(id: 'abc123') }
-
-  describe "default attributes" do
-    let(:hash) { "sha2hash#{DateTime.current.to_f}" }
-    let(:path) { '/foo/file/99999' }
-
-    subject { described_class.new itemId: '99999', path: path }
-
-    it "creates link" do
-      expect(Digest::SHA2).to receive(:new).and_return(hash)
-      expect(subject.downloadKey).to eq hash
-      expect(subject.itemId).to eq '99999'
-      expect(subject.path).to eq path
-    end
-  end
-
-  describe "#expired?" do
-    let(:link) do
-      described_class.new(expires: expires)
-    end
-
-    subject { link.expired? }
-
-    context "when not expired" do
-      let(:expires) { DateTime.current.advance(hours: 1) }
-
-      it { is_expected.to be false }
-    end
-
-    context "when not expired" do
-      let(:expires) { DateTime.current.advance(hours: -1) }
-
-      it { is_expected.to be true }
-    end
-  end
-end

data/spec/models/sipity/agent_spec.rb

@@ -1,8 +0,0 @@
-module Sipity
-  RSpec.describe Agent, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include("proxy_for_id") }
-    its(:column_names) { is_expected.to include("proxy_for_type") }
-  end
-end

data/spec/models/sipity/comment_spec.rb

@@ -1,24 +0,0 @@
-module Sipity
-  RSpec.describe Comment, type: :model do
-    context 'database configuration' do
-      subject { described_class }
-
-      its(:column_names) { is_expected.to include('entity_id') }
-      its(:column_names) { is_expected.to include('agent_id') }
-      its(:column_names) { is_expected.to include('comment') }
-    end
-
-    describe '#name_of_commentor' do
-      let(:instance) { described_class.new }
-      let(:agent) { instance_double(Agent, proxy_for: user) }
-      let(:user) { instance_double(User, to_s: 'Hiya') }
-
-      subject { instance.name_of_commentor }
-
-      before do
-        allow(instance).to receive(:agent).and_return(agent)
-      end
-      it { is_expected.to eq('Hiya') }
-    end
-  end
-end

data/spec/models/sipity/entity_spec.rb

@@ -1,27 +0,0 @@
-module Sipity
-  RSpec.describe Entity, type: :model do
-    describe 'database configuration' do
-      subject { described_class }
-
-      its(:column_names) { is_expected.to include("proxy_for_global_id") }
-      its(:column_names) { is_expected.to include("workflow_id") }
-      its(:column_names) { is_expected.to include("workflow_state_id") }
-    end
-
-    subject { described_class.new }
-
-    describe 'delegations' do
-      it { is_expected.to delegate_method(:workflow_state_name).to(:workflow_state).as(:name) }
-      it { is_expected.to delegate_method(:workflow_name).to(:workflow).as(:name) }
-    end
-
-    describe '#proxy_for' do
-      let(:work) { create(:generic_work) }
-      let(:entity) { Sipity::Entity.new(proxy_for_global_id: work.to_global_id) }
-
-      it 'will retrieve based on a GlobalID of the object' do
-        expect(entity.proxy_for).to eq(work)
-      end
-    end
-  end
-end

data/spec/models/sipity/entity_specific_responsibility_spec.rb

@@ -1,9 +0,0 @@
-module Sipity
-  RSpec.describe EntitySpecificResponsibility, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include('workflow_role_id') }
-    its(:column_names) { is_expected.to include('entity_id') }
-    its(:column_names) { is_expected.to include('agent_id') }
-  end
-end

data/spec/models/sipity/notifiable_context_spec.rb

@@ -1,7 +0,0 @@
-module Sipity
-  RSpec.describe NotifiableContext, type: :model do
-    subject { described_class.new }
-
-    it "should have tests"
-  end
-end

data/spec/models/sipity/notification_recipient_spec.rb

@@ -1,9 +0,0 @@
-module Sipity
-  RSpec.describe NotificationRecipient, type: :model do
-    subject { described_class.new }
-
-    it 'will raise an ArgumentError if you provide an invalid recipient_strategy' do
-      expect { subject.recipient_strategy = '__incorrect_name__' }.to raise_error(ArgumentError)
-    end
-  end
-end

data/spec/models/sipity/notification_spec.rb

@@ -1,15 +0,0 @@
-module Sipity
-  RSpec.describe Notification, type: :model do
-    subject { described_class.new }
-
-    it 'will raise an ArgumentError if you provide an invalid #notification_type' do
-      expect { subject.notification_type = '__incorrect_name__' }.to raise_error(ArgumentError)
-    end
-
-    describe '.valid_notification_types' do
-      subject { described_class.valid_notification_types }
-
-      it { is_expected.to eq([described_class::NOTIFICATION_TYPE_EMAIL]) }
-    end
-  end
-end

data/spec/models/sipity/role_spec.rb

@@ -1,40 +0,0 @@
-module Sipity
-  RSpec.describe Role, type: :model do
-    context 'class methods' do
-      subject { described_class }
-
-      its(:column_names) { is_expected.to include('name') }
-      its(:column_names) { is_expected.to include('description') }
-      context '.[]' do
-        let(:valid_name) { 'reviewing' }
-
-        it 'will find the named role' do
-          expected_object = described_class.create!(name: valid_name)
-          expect(described_class[valid_name]).to eq(expected_object)
-        end
-
-        it 'will created the named role' do
-          expect { described_class[valid_name].name }.to change { described_class.count }.by(1)
-        end
-      end
-    end
-
-    subject { described_class.new }
-
-    it 'will have a #to_s that is a name' do
-      subject.name = 'advising'
-      expect(subject.to_s).to eq(subject.name)
-    end
-
-    context '#destroy' do
-      it 'will not allow registered role names to be destroyed' do
-        role = Sipity::Role.create!(name: Hyrax::RoleRegistry::MANAGING)
-        expect { role.destroy! }.to raise_error(ActiveRecord::RecordNotDestroyed)
-      end
-      it 'will allow unregistered role names to be destroyed' do
-        role = Sipity::Role.create!(name: 'gong_farming')
-        expect { role.destroy! }.not_to raise_error
-      end
-    end
-  end
-end

data/spec/models/sipity/workflow_action_spec.rb

@@ -1,11 +0,0 @@
-module Sipity
-  RSpec.describe WorkflowAction, type: :model do
-    context 'database configuration' do
-      subject { described_class }
-
-      its(:column_names) { is_expected.to include('workflow_id') }
-      its(:column_names) { is_expected.to include('resulting_workflow_state_id') }
-      its(:column_names) { is_expected.to include('name') }
-    end
-  end
-end

data/spec/models/sipity/workflow_responsibility_spec.rb

@@ -1,8 +0,0 @@
-module Sipity
-  RSpec.describe WorkflowResponsibility, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include('agent_id') }
-    its(:column_names) { is_expected.to include('workflow_role_id') }
-  end
-end

data/spec/models/sipity/workflow_role_spec.rb

@@ -1,8 +0,0 @@
-module Sipity
-  RSpec.describe WorkflowRole, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include('workflow_id') }
-    its(:column_names) { is_expected.to include('role_id') }
-  end
-end

data/spec/models/sipity/workflow_spec.rb

@@ -1,84 +0,0 @@
-module Sipity
-  RSpec.describe Workflow, type: :model do
-    context 'class configuration' do
-      subject { described_class }
-
-      its(:column_names) { is_expected.to include('name') }
-    end
-
-    subject { create(:workflow, name: 'ETD Workflow') }
-
-    it { is_expected.to belong_to(:permission_template) }
-
-    context '#initial_workflow_state' do
-      it 'will create a state if one does not exist' do
-        expect { subject.initial_workflow_state }
-          .to change { subject.workflow_states.count }.by(1)
-      end
-    end
-
-    context '.activate!' do
-      let!(:permission_template) { create(:permission_template) }
-      let!(:other_permission_template) { create(:permission_template) }
-      let!(:active_workflow) { create(:workflow, active: true, permission_template_id: permission_template.id) }
-
-      context 'selecting already active workflow' do
-        it 'makes no change' do
-          inactive_workflow = create(:workflow, permission_template_id: permission_template.id)
-          described_class.activate!(permission_template: permission_template, workflow_id: active_workflow.id)
-          expect(active_workflow.reload).to be_active
-          expect(inactive_workflow.reload).not_to be_active
-        end
-      end
-
-      it 'raises an exception if you do not pass a workflow_id nor workflow_name' do
-        expect do
-          described_class.activate!(permission_template: other_permission_template)
-        end.to raise_error(RuntimeError)
-      end
-      it 'raises an exception on a mismatch' do
-        expect do
-          described_class.activate!(permission_template: other_permission_template, workflow_id: active_workflow.id)
-        end.to raise_error(ActiveRecord::RecordNotFound)
-      end
-      context 'with :workflow_id keyword' do
-        it 'activates the specified workflow and deactivates the unspecified workflow for the permission_template' do
-          active_workflow_wrong_template = create(:workflow, active: true, permission_template_id: other_permission_template.id)
-          inactive_workflow = create(:workflow, permission_template_id: permission_template.id)
-
-          described_class.activate!(permission_template: permission_template, workflow_id: inactive_workflow)
-          expect(inactive_workflow.reload).to be_active
-          expect(active_workflow.reload).not_to be_active
-          expect(active_workflow_wrong_template.reload).to be_active
-        end
-      end
-      context 'with :workflow_name keyword' do
-        it 'activates the specified workflow and deactivates the unspecified workflow for the permission_template' do
-          active_workflow_wrong_template = create(:workflow, active: true, permission_template_id: other_permission_template.id)
-          inactive_workflow = create(:workflow, permission_template_id: permission_template.id)
-
-          described_class.activate!(permission_template: permission_template, workflow_name: inactive_workflow.name)
-          expect(inactive_workflow.reload).to be_active
-          expect(active_workflow.reload).not_to be_active
-          expect(active_workflow_wrong_template.reload).to be_active
-        end
-      end
-    end
-
-    context '.find_active_workflow_for' do
-      let(:source_id) { 1234 }
-      let(:permission_template) { create(:permission_template, source_id: source_id) }
-
-      it 'returns the active workflow for the permission template' do
-        active_workflow = create(:workflow, active: true, permission_template: permission_template)
-        _inactive_workflow = create(:workflow, active: false, permission_template: permission_template)
-        expect(described_class.find_active_workflow_for(admin_set_id: source_id)).to eq(active_workflow)
-      end
-
-      it 'raises an exception when none exists' do
-        create(:workflow, active: false, permission_template: permission_template)
-        expect { described_class.find_active_workflow_for(admin_set_id: source_id) }.to raise_error(ActiveRecord::RecordNotFound)
-      end
-    end
-  end
-end

data/spec/models/sipity/workflow_state_action_permission_spec.rb

@@ -1,8 +0,0 @@
-module Sipity
-  RSpec.describe WorkflowStateActionPermission, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include("workflow_role_id") }
-    its(:column_names) { is_expected.to include("workflow_state_action_id") }
-  end
-end

data/spec/models/sipity/workflow_state_action_spec.rb

@@ -1,8 +0,0 @@
-module Sipity
-  RSpec.describe WorkflowStateAction, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include("originating_workflow_state_id") }
-    its(:column_names) { is_expected.to include("workflow_action_id") }
-  end
-end

data/spec/models/sipity/workflow_state_spec.rb

@@ -1,17 +0,0 @@
-module Sipity
-  RSpec.describe WorkflowState, type: :model do
-    subject { described_class }
-
-    its(:column_names) { is_expected.to include("workflow_id") }
-    its(:column_names) { is_expected.to include("name") }
-
-    describe "#entities" do
-      let(:workflow_state) { create(:workflow_state, name: 'test') }
-      let!(:entity)        { Sipity::Entity.create!(workflow_state: workflow_state, proxy_for_global_id: "abc123", workflow_id: workflow_state.workflow_id) }
-
-      it "has entites" do
-        expect(workflow_state.entities).to match_array([entity])
-      end
-    end
-  end
-end

data/spec/models/solr_document_spec.rb

@@ -1,205 +0,0 @@
-RSpec.describe ::SolrDocument, type: :model do
-  let(:document) { described_class.new(attributes) }
-  let(:attributes) { {} }
-
-  describe "#itemtype" do
-    let(:attributes) { { resource_type_tesim: ['Article'] } }
-
-    it "delegates to the Hyrax::ResourceTypesService" do
-      expect(Hyrax::ResourceTypesService).to receive(:microdata_type).with('Article')
-      subject
-    end
-    subject { document.itemtype }
-
-    it { is_expected.to eq 'http://schema.org/Article' }
-
-    context 'with no resource_type' do
-      let(:attributes) { {} }
-
-      it { is_expected.to eq 'http://schema.org/CreativeWork' }
-    end
-  end
-
-  describe "date_uploaded" do
-    let(:attributes) { { 'date_uploaded_dtsi' => '2013-03-14T00:00:00Z' } }
-
-    subject { document.date_uploaded }
-
-    it { is_expected.to eq Date.parse('2013-03-14') }
-
-    context "when an invalid type is provided" do
-      let(:attributes) { { 'date_uploaded_dtsi' => 'Test' } }
-
-      it "logs parse errors" do
-        expect(Hyrax.logger).to receive(:info).with(/Unable to parse date.*/)
-        subject
-      end
-    end
-  end
-
-  describe "rights_statement" do
-    let(:attributes) { { 'rights_statement_tesim' => ['A rights statement'] } }
-
-    it "responds to rights_statement" do
-      expect(document).to respond_to(:rights_statement)
-    end
-    it "returns the proper data" do
-      expect(document.rights_statement).to eq ['A rights statement']
-    end
-  end
-
-  describe "create_date" do
-    let(:attributes) { { 'system_create_dtsi' => '2013-03-14T00:00:00Z' } }
-
-    subject { document.create_date }
-
-    it { is_expected.to eq Date.parse('2013-03-14') }
-
-    context "when an invalid type is provided" do
-      let(:attributes) { { 'system_create_dtsi' => 'Test' } }
-
-      it "logs parse errors" do
-        expect(Hyrax.logger).to receive(:info).with(/Unable to parse date.*/)
-        subject
-      end
-    end
-  end
-
-  describe "resource_type" do
-    let(:attributes) { { 'resource_type_tesim' => ['Image'] } }
-
-    subject { document.resource_type }
-
-    it { is_expected.to eq ['Image'] }
-  end
-
-  describe "thumbnail_path" do
-    let(:attributes) { { 'thumbnail_path_ss' => ['/foo/bar'] } }
-
-    subject { document.thumbnail_path }
-
-    it { is_expected.to eq '/foo/bar' }
-  end
-
-  describe '#to_param' do
-    let(:id) { '1v53kn56d' }
-    let(:attributes) { { id: id } }
-
-    subject { document.to_param }
-
-    it { is_expected.to eq id }
-  end
-
-  describe "#suppressed?" do
-    let(:attributes) { { 'suppressed_bsi' => suppressed_value } }
-
-    subject { document }
-
-    context 'when true' do
-      let(:suppressed_value) { true }
-
-      it { is_expected.to be_suppressed }
-    end
-    context 'when false' do
-      let(:suppressed_value) { false }
-
-      it { is_expected.not_to be_suppressed }
-    end
-  end
-  describe "document types" do
-    class Mimes
-      include Hydra::Works::MimeTypes
-    end
-
-    Mimes.office_document_mime_types.each do |type|
-      context "when mime-type is #{type}" do
-        let(:attributes) { { 'mime_type_ssi' => type } }
-
-        subject { document }
-
-        it { is_expected.to be_office_document }
-      end
-    end
-
-    Mimes.video_mime_types.each do |type|
-      context "when mime-type is #{type}" do
-        let(:attributes) { { 'mime_type_ssi' => type } }
-
-        subject { document }
-
-        it { is_expected.to be_video }
-      end
-    end
-  end
-
-  describe '#collection_ids' do
-    subject { document.collection_ids }
-
-    context 'when the object belongs to collections' do
-      let(:attributes) do
-        { id: '123',
-          title_tesim: ['A generic work'],
-          collection_ids_tesim: ['123', '456', '789'] }
-      end
-
-      it { is_expected.to eq ['123', '456', '789'] }
-    end
-
-    context 'when the object does not belong to any collections' do
-      let(:attributes) do
-        { id: '123',
-          title_tesim: ['A generic work'] }
-      end
-
-      it { is_expected.to eq [] }
-    end
-  end
-
-  describe "#height" do
-    let(:attributes) { { height_is: '444' } }
-
-    subject { document.height }
-
-    it { is_expected.to eq '444' }
-  end
-
-  describe "#width" do
-    let(:attributes) { { width_is: '555' } }
-
-    subject { document.width }
-
-    it { is_expected.to eq '555' }
-  end
-
-  context "when exporting in endnote format" do
-    let(:attributes) { { id: "1234" } }
-
-    subject { document.endnote_filename }
-
-    it { is_expected.to eq("1234.endnote") }
-  end
-
-  describe "#admin_set?" do
-    let(:attributes) { { 'has_model_ssim' => 'AdminSet' } }
-
-    subject { document }
-
-    it { is_expected.to be_admin_set }
-  end
-
-  describe "#collection?" do
-    let(:attributes) { { 'has_model_ssim' => 'Collection' } }
-
-    subject { document }
-
-    it { is_expected.to be_collection }
-  end
-
-  describe "#collection_type_gid?" do
-    let(:attributes) { { 'collection_type_gid_ssim' => 'gid://internal/hyrax-collectiontype/5' } }
-
-    subject { document.collection_type_gid }
-
-    it { is_expected.to eq 'gid://internal/hyrax-collectiontype/5' }
-  end
-end