hyrax 3.0.0.pre.rc1 → 3.0.0.pre.rc2

data/spec/services/hyrax/embargo_manager_spec.rb

@@ -1,123 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe Hyrax::EmbargoManager do
-  subject(:manager) { described_class.new(resource: resource) }
-  let(:resource)    { Hyrax::Resource.new }
-
-  shared_context 'when under embargo' do
-    let(:resource) { FactoryBot.build(:hyrax_resource, :under_embargo) }
-  end
-
-  shared_context 'with expired embargo' do
-    let(:resource) { FactoryBot.build(:hyrax_resource, embargo: embargo) }
-    let(:embargo)  { FactoryBot.build(:hyrax_embargo, :expired) }
-  end
-
-  describe '#apply' do
-    context 'with no embargo' do
-      it 'is a no-op' do
-        expect { manager.apply }
-          .not_to change { resource.visibility }
-      end
-    end
-
-    context 'with expried embargo' do
-      include_context 'with expired embargo'
-
-      it 'is a no-op' do
-        expect { manager.apply }
-          .not_to change { resource.visibility }
-      end
-    end
-
-    context 'when under embargo' do
-      include_context 'when under embargo'
-
-      before { resource.visibility = 'open' }
-
-      it 'applies the active embargo visibility' do
-        expect { manager.apply }
-          .to change { resource.visibility }
-          .to 'authenticated'
-      end
-    end
-  end
-
-  describe '#copy_embargo_to' do
-    let(:other_resource) { Hyrax::Resource.new }
-
-    it 'does not assign an embargo when none is present' do
-      expect { manager.copy_embargo_to(target: other_resource) }
-        .not_to change { other_resource.embargo }
-        .from nil
-    end
-
-    context 'with expried embargo' do
-      include_context 'with expired embargo'
-
-      it 'does not copy the embargo' do
-        expect { manager.copy_embargo_to(target: other_resource) }
-          .not_to change { other_resource.embargo }
-          .from nil
-      end
-    end
-
-    context 'when under embargo' do
-      include_context 'when under embargo'
-
-      before { other_resource.visibility = 'open' }
-
-      it 'copies the embargo to the target' do
-        expect { manager.copy_embargo_to(target: other_resource) }
-          .to change { other_resource.embargo }
-          .from(nil)
-          .to have_attributes(embargo_release_date:
-                                manager.embargo.embargo_release_date)
-      end
-
-      it 'applies the active embargo visibility' do
-        expect { manager.copy_embargo_to(target: other_resource) }
-          .to change { other_resource.visibility }
-          .to 'authenticated'
-      end
-    end
-  end
-
-  describe '#embargo' do
-    it 'gives an inactive embargo' do
-      expect(manager.embargo).not_to be_active
-    end
-
-    context 'when under embargo' do
-      include_context 'when under embargo'
-
-      it 'gives an active embargo' do
-        expect(manager.embargo).to be_active
-      end
-
-      it 'has embargo attributes' do
-        expect(manager.embargo)
-          .to have_attributes visibility_after_embargo: 'open',
-                              visibility_during_embargo: 'authenticated',
-                              embargo_release_date: an_instance_of(Date),
-                              embargo_history: be_empty
-      end
-    end
-  end
-
-  describe '#under_embargo?' do
-    it { is_expected.not_to be_under_embargo }
-
-    context 'when under embargo' do
-      include_context 'when under embargo'
-
-      it { is_expected.to be_under_embargo }
-    end
-
-    context 'with expired embargo' do
-      include_context 'with expired embargo'
-
-      it { is_expected.not_to be_under_embargo }
-    end
-  end
-end

data/spec/services/hyrax/embargo_service_spec.rb

@@ -1,59 +0,0 @@
-RSpec.describe Hyrax::EmbargoService do
-  subject { described_class }
-
-  let(:future_date) { 2.days.from_now }
-  let(:past_date) { 2.days.ago }
-
-  let!(:work_with_expired_embargo1) do
-    build(:work, embargo_release_date: past_date.to_s).tap do |work|
-      work.save(validate: false)
-    end
-  end
-
-  let!(:work_with_expired_embargo2) do
-    build(:work, embargo_release_date: past_date.to_s).tap do |work|
-      work.save(validate: false)
-    end
-  end
-
-  let!(:work_with_embargo_in_effect) { create(:work, embargo_release_date: future_date.to_s) }
-  let!(:work_without_embargo) { create(:generic_work) }
-
-  describe '#assets_with_expired_embargoes' do
-    it 'returns an array of assets with expired embargoes' do
-      returned_pids = subject.assets_with_expired_embargoes.map(&:id)
-      expect(returned_pids).to include work_with_expired_embargo1.id, work_with_expired_embargo2.id
-      expect(returned_pids).not_to include work_with_embargo_in_effect.id, work_without_embargo.id
-    end
-  end
-
-  describe '#assets_under_embargo' do
-    it 'returns all assets with embargo release date set' do
-      returned_pids = subject.assets_under_embargo.map(&:id)
-      expect(returned_pids).to include work_with_expired_embargo1.id, work_with_expired_embargo2.id, work_with_embargo_in_effect.id
-      expect(returned_pids).not_to include work_without_embargo.id
-    end
-  end
-
-  describe '#assets_with_deactivated_embargoes' do
-    let(:id) { Noid::Rails::Service.new.mint }
-    let(:attributes) do
-      { 'embargo_history_ssim' => ['This is in the past'],
-        'id' => id }
-    end
-
-    before do
-      Hyrax::SolrService.add(attributes)
-      Hyrax::SolrService.commit
-    end
-
-    it 'returns all assets with embargo history set' do
-      returned_pids = subject.assets_with_deactivated_embargoes.map(&:id)
-      expect(returned_pids).to include id
-      expect(returned_pids).not_to include(work_without_embargo.id,
-                                           work_with_expired_embargo1.id,
-                                           work_with_expired_embargo2.id,
-                                           work_with_embargo_in_effect.id)
-    end
-  end
-end

data/spec/services/hyrax/file_set_csv_service_spec.rb

@@ -1,83 +0,0 @@
-RSpec.describe Hyrax::FileSetCSVService do
-  let(:mock_file) do
-    Hydra::PCDM::File.new
-  end
-  let(:file) do
-    FileSet.new(id: '123abc', title: ['My Title'], creator: ['Von, Creator'],
-                resource_type: ['Book', 'Other'], license: ['Mine']) do |f|
-      f.apply_depositor_metadata('jilluser@example.com')
-    end
-  end
-  let(:solr_document) { SolrDocument.new(file.to_solr) }
-
-  before do
-    allow(mock_file).to receive(:mime_type).and_return('application/pdf')
-    allow(file).to receive(:resource_type).and_return(['Book', 'Other'])
-    allow(file).to receive(:original_file).and_return(mock_file)
-  end
-
-  context "when using the defaults" do
-    let(:csv_service) { described_class.new(solr_document) }
-
-    describe "csv" do
-      subject { csv_service.csv }
-
-      it { is_expected.to eq "123abc,My Title,jilluser@example.com,\"Von, Creator\",restricted,Book|Other,Mine,pdf\n" }
-      it "parses as valid csv" do
-        expect(CSV.parse(subject)).to eq([["123abc", "My Title", "jilluser@example.com", "Von, Creator", "restricted", "Book|Other", "Mine", "pdf"]])
-      end
-    end
-
-    describe "csv_header" do
-      subject { csv_service.csv_header }
-
-      it { is_expected.to eq "id,title,depositor,creator,visibility,resource_type,license,file_format\n" }
-    end
-  end
-
-  context "when specifying terms" do
-    let(:csv_service) { described_class.new(file, [:id, :title, :resource_type]) }
-
-    describe "csv" do
-      subject { csv_service.csv }
-
-      it { is_expected.to eq "123abc,My Title,Book|Other\n" }
-    end
-    describe "csv_header" do
-      subject { csv_service.csv_header }
-
-      it { is_expected.to eq "id,title,resource_type\n" }
-    end
-  end
-
-  context "when specifying separator" do
-    let(:csv_service) { described_class.new(solr_document, nil, '&&') }
-
-    describe "csv" do
-      subject { csv_service.csv }
-
-      it { is_expected.to eq "123abc,My Title,jilluser@example.com,\"Von, Creator\",restricted,Book&&Other,Mine,pdf\n" }
-    end
-
-    describe "csv_header" do
-      subject { csv_service.csv_header }
-
-      it { is_expected.to eq "id,title,depositor,creator,visibility,resource_type,license,file_format\n" }
-    end
-  end
-
-  context "when specifying terms and separator" do
-    let(:csv_service) { described_class.new(file, [:id, :title, :resource_type], '*$*') }
-
-    describe "csv" do
-      subject { csv_service.csv }
-
-      it { is_expected.to eq "123abc,My Title,Book*$*Other\n" }
-    end
-    describe "csv_header" do
-      subject { csv_service.csv_header }
-
-      it { is_expected.to eq "id,title,resource_type\n" }
-    end
-  end
-end

data/spec/services/hyrax/file_set_derivatives_service_spec.rb

@@ -1,13 +0,0 @@
-require 'hyrax/specs/shared_specs'
-
-RSpec.describe Hyrax::FileSetDerivativesService do
-  let(:valid_file_set) do
-    FileSet.new.tap do |f|
-      allow(f).to receive(:mime_type).and_return(FileSet.image_mime_types.first)
-    end
-  end
-
-  subject { described_class.new(file_set) }
-
-  it_behaves_like "a Hyrax::DerivativeService"
-end

data/spec/services/hyrax/file_set_fixity_check_service_spec.rb

@@ -1,77 +0,0 @@
-RSpec.describe Hyrax::FileSetFixityCheckService do
-  let(:f)                 { create(:file_set, content: File.open(fixture_path + '/world.png')) }
-  let(:service_by_object) { described_class.new(f) }
-  let(:service_by_id)     { described_class.new(f.id) }
-
-  describe "async_jobs: false" do
-    let(:service_by_object) { described_class.new(f, async_jobs: false) }
-    let(:service_by_id)     { described_class.new(f.id, async_jobs: false) }
-
-    describe '#fixity_check' do
-      subject { service_by_object.fixity_check }
-
-      context 'when a file has two versions' do
-        before do
-          Hyrax::VersioningService.create(f.original_file) # create a second version -- the factory creates the first version when it attaches +content+
-        end
-        specify 'returns two log results' do
-          expect(subject.values.flatten.length).to eq(2)
-        end
-
-        context "with latest_version_only" do
-          let(:service_by_object) { described_class.new(f, async_jobs: false, latest_version_only: true) }
-
-          specify "returns one log result" do
-            expect(subject.values.length).to eq(1)
-          end
-        end
-      end
-
-      context "existing check and disabled max_days_between_fixity_checks" do
-        let(:service_by_object) { described_class.new(f, async_jobs: false, max_days_between_fixity_checks: -1) }
-        let(:service_by_id)     { described_class.new(f.id, async_jobs: false, max_days_between_fixity_checks: -1) }
-        let!(:existing_record) do
-          ChecksumAuditLog.create!(passed: true, file_set_id: f.id, checked_uri: f.original_file.versions.first.label, file_id: f.original_file.id)
-        end
-
-        it "re-checks" do
-          existing_record
-          expect(subject.length).to eq 1
-          expect(subject.values.flatten.first.id).not_to eq(existing_record.id)
-          expect(subject.values.flatten.first.created_at).to be > existing_record.created_at
-        end
-      end
-    end
-
-    describe '#fixity_check_file' do
-      subject { service_by_object.send(:fixity_check_file, f.original_file) }
-
-      specify 'returns a single result' do
-        expect(subject.length).to eq(1)
-      end
-      describe 'non-versioned file with latest version only' do
-        let(:service_by_object) { described_class.new(f, async_jobs: false, latest_version_only: true) }
-
-        before do
-          allow(f.original_file).to receive(:has_versions?).and_return(false)
-        end
-
-        subject { service_by_object.send(:fixity_check_file, f.original_file) }
-
-        specify 'returns a single result' do
-          expect(subject.length).to eq(1)
-        end
-      end
-    end
-
-    describe '#fixity_check_file_version' do
-      subject { service_by_object.send(:fixity_check_file_version, f.original_file.id, f.original_file.uri.to_s) }
-
-      specify 'returns a single ChecksumAuditLog for the given file' do
-        expect(subject).to be_kind_of ChecksumAuditLog
-        expect(subject.file_set_id).to eq(f.id)
-        expect(subject.checked_uri).to eq(f.original_file.uri)
-      end
-    end
-  end
-end

data/spec/services/hyrax/file_set_visibility_propagator_spec.rb

@@ -1,59 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe Hyrax::FileSetVisibilityPropagator do
-  subject(:propagator) { described_class.new(source: work) }
-  let(:work)           { FactoryBot.create(:work_with_files) }
-
-  context 'a public work' do
-    before do
-      work.visibility = Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
-    end
-
-    it 'copies visibility to its contained files' do
-      # files are private at the outset
-      expect(work.file_sets.first.visibility)
-        .to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
-
-      propagator.propagate
-
-      work.reload.file_sets.each do |file|
-        expect(file.visibility)
-          .to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
-      end
-    end
-  end
-
-  context 'when work is under embargo' do
-    let(:work) { create(:embargoed_work_with_files) }
-    let(:file) { work.file_sets.first }
-
-    it 'copies visibility to its contained files and apply a copy of the embargo to the files' do
-      expect(work.visibility)
-        .to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE
-      expect(work).to be_under_embargo
-      expect(work.file_sets.first).not_to be_under_embargo
-
-      propagator.propagate
-
-      expect(file).to be_under_embargo
-      expect(file.embargo.id).not_to eq work.embargo.id
-    end
-  end
-
-  context 'when work is under lease' do
-    let(:work) { create(:leased_work_with_files) }
-    let(:file) { work.file_sets.first }
-
-    it 'copies visibility to its contained files and apply a copy of the lease to the files' do
-      expect(work.visibility)
-        .to eq Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC
-      expect(work).to be_active_lease
-      expect(work.file_sets.first).not_to be_active_lease
-
-      propagator.propagate
-
-      expect(file).to be_active_lease
-      expect(file.lease.id).not_to eq work.lease.id
-    end
-  end
-end

data/spec/services/hyrax/fixity_check_failure_service_spec.rb

@@ -1,30 +0,0 @@
-RSpec.describe Hyrax::FixityCheckFailureService do
-  let!(:depositor) { create(:user) }
-  let!(:log_date) { '2015-07-15 03:06:59' }
-  let(:inbox) { depositor.mailbox.inbox }
-  let(:file) { Hydra::PCDM::File.new }
-  let(:version_uri) { "#{file.uri}/fcr:versions/version1" }
-  let(:file_set) do
-    create(:file_set, user: depositor, title: ["World Icon"]).tap { |fs| fs.original_file = file }
-  end
-
-  let(:checksum_audit_log) do
-    ChecksumAuditLog.new(file_set_id: file_set.id,
-                         file_id: file_set.original_file.id,
-                         checked_uri: version_uri,
-                         created_at: log_date,
-                         updated_at: log_date,
-                         passed: false)
-  end
-
-  describe "#call" do
-    subject { described_class.new(file_set, checksum_audit_log: checksum_audit_log) }
-
-    it "sends failing mail" do
-      subject.call
-      expect(inbox.count).to eq(1)
-      inbox.each { |msg| expect(msg.last_message.subject).to eq('Failing Fixity Check') }
-      inbox.each { |msg| expect(msg.last_message.body).to eq('The fixity check run at ' + checksum_audit_log.created_at.to_s + ' for ' + file_set.to_s + ' (' + file.uri + ') failed.') }
-    end
-  end
-end