hyrax 2.6.0 → 2.7.0

data/spec/lib/hyrax/arkivo/metadata_munger_spec.rb

@@ -1,46 +0,0 @@
-RSpec.describe Hyrax::Arkivo::MetadataMunger do
-  subject { described_class.new(metadata) }
-
-  let(:metadata) { JSON.parse(FactoryBot.json(:post_item))['metadata'] }
-
-  it 'makes camelCase symbols into underscored strings' do
-    expect(metadata).to include('resourceType', 'dateCreated', 'basedNear')
-    munged = subject.call
-    expect(munged).not_to include('resourceType', 'dateCreated', 'basedNear')
-    expect(munged).to include('resource_type', 'date_created', 'based_near')
-  end
-
-  it 'replaces url with related_url' do
-    expect(metadata).to include('url')
-    munged = subject.call
-    expect(munged).not_to include('url')
-    expect(munged).to include('related_url')
-  end
-
-  it 'replaces tags with keyword05' do
-    expect(metadata).to include('tags')
-    munged = subject.call
-    expect(munged).not_to include('tags')
-    expect(munged).to include('keyword')
-  end
-
-  it 'replaces firstName and lastName with name' do
-    expect(name_parts(metadata['creators']).count).to eq 4
-    subject.call
-    expect(name_parts(metadata['creators']).count).to eq 0
-    expect(metadata['creators'].map { |c| c['name'] }.compact.count).to eq 4
-  end
-
-  it 'segregates creators and contributors' do
-  end
-
-  it 'deletes the original creators array' do
-    expect(metadata['creators']).not_to be_nil
-    munged = subject.call
-    expect(munged['creators']).to be_nil
-  end
-
-  def name_parts(creators)
-    creators.map { |c| c['firstName'] }.compact + creators.map { |c| c['lastName'] }.compact
-  end
-end

data/spec/lib/hyrax/arkivo/schema_validator_spec.rb

@@ -1,63 +0,0 @@
-RSpec.describe Hyrax::Arkivo::SchemaValidator do
-  let(:item) { JSON.parse(FactoryBot.json(:post_item)) }
-
-  it 'ensures a token is included' do
-    expect do
-      described_class.new(item.except('token')).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'token'/)
-  end
-
-  it 'ensures a metadata section is included' do
-    expect do
-      described_class.new(item.except('metadata')).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'metadata'/)
-  end
-
-  it 'ensures a file section is included' do
-    expect do
-      described_class.new(item.except('file')).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'file'/)
-  end
-
-  it 'ensures the metadata has a title' do
-    item['metadata'].delete('title')
-    expect do
-      described_class.new(item).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'title'/)
-  end
-
-  it 'ensures the metadata has license' do
-    item['metadata'].delete('license')
-    expect do
-      described_class.new(item).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'license'/)
-  end
-
-  it 'ensures the file has a b64-encoded content' do
-    item['file'].delete('base64')
-    expect do
-      described_class.new(item).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'base64'/)
-  end
-
-  it 'ensures the file has a checksum' do
-    item['file'].delete('md5')
-    expect do
-      described_class.new(item).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'md5'/)
-  end
-
-  it 'ensures the file has a filename' do
-    item['file'].delete('filename')
-    expect do
-      described_class.new(item).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'filename'/)
-  end
-
-  it 'ensures the file has a content type' do
-    item['file'].delete('contentType')
-    expect do
-      described_class.new(item).call
-    end.to raise_error(Hyrax::Arkivo::InvalidItem, /required property of 'contentType'/)
-  end
-end

data/spec/lib/hyrax/arkivo_spec.rb

@@ -1,12 +0,0 @@
-RSpec.describe Hyrax::Arkivo do
-  describe '.config' do
-    it 'returns a hash with the :url key' do
-      expect(described_class.config).to have_key(:url)
-    end
-  end
-  describe '.new_subscription_url' do
-    it 'returns a string' do
-      expect(described_class.new_subscription_url).to eq '/api/subscription'
-    end
-  end
-end

data/spec/lib/hyrax/collections/search_service_spec.rb

@@ -1,35 +0,0 @@
-RSpec.describe Hyrax::Collections::SearchService do
-  let(:login) { 'vanessa' }
-  let(:session) { { history: [17, 14, 12, 9] } }
-  let(:service) { described_class.new(session, login) }
-
-  it "gets the documents for the first history entry" do
-    expect(Search).to receive(:find).with(17).and_return(Search.new(query_params: { q: "World Peace" }))
-    expect(service).to receive(:get_search_results).and_return([:one, [:doc1, :doc2]])
-    expect(service.last_search_documents).to eq([:doc1, :doc2])
-  end
-
-  describe 'apply_gated_search' do
-    before do
-      allow(::User.group_service).to receive(:roles).with(login).and_return(['umg/test.group.1'])
-    end
-
-    let(:params) { service.apply_gated_search({}, {}) }
-    let(:group_query) { params[:fq].first.split(' OR ')[1] }
-
-    it "escapes slashes in groups" do
-      expect(group_query).to eq('edit_access_group_ssim:umg\/test.group.1')
-    end
-
-    context "when Solr's access control suffix is overridden" do
-      let(:service) { described_class.new({}, '') }
-
-      it "uses the overriden value" do
-        allow(service).to receive(:solr_access_control_suffix).and_return("edit_group_customfield")
-        params = service.apply_gated_search({}, {})
-        public_query = params[:fq].first.split(' OR ')[0]
-        expect(public_query).to eq('edit_group_customfield:public')
-      end
-    end
-  end
-end

data/spec/lib/hyrax/configuration_spec.rb

@@ -1,91 +0,0 @@
-RSpec.describe Hyrax::Configuration do
-  subject(:configuration) { described_class.new }
-
-  describe '#register_roles' do
-    it 'yields a RoleRegistry' do
-      expect { |b| subject.register_roles(&b) }.to yield_with_args(kind_of(Hyrax::RoleRegistry))
-    end
-  end
-  it { is_expected.to delegate_method(:registered_role?).to(:role_registry) }
-  it { is_expected.to delegate_method(:persist_registered_roles!).to(:role_registry) }
-
-  describe '#default_active_workflow_name' do
-    subject { described_class.new.default_active_workflow_name }
-
-    it { is_expected.to eq('default') }
-  end
-
-  it { is_expected.to respond_to(:active_deposit_agreement_acceptance?) }
-  it { is_expected.to respond_to(:active_deposit_agreement_acceptance=) }
-  it { is_expected.to respond_to(:activity_to_show_default_seconds_since_now) }
-  it { is_expected.to respond_to(:admin_set_predicate) }
-  it { is_expected.to respond_to(:admin_set_predicate=) }
-  it { is_expected.to respond_to(:analytic_start_date) }
-  it { is_expected.to respond_to(:analytics?) }
-  it { is_expected.to respond_to(:analytics) }
-  it { is_expected.to respond_to(:arkivo_api?) }
-  it { is_expected.to respond_to(:arkivo_api=) }
-  it { is_expected.to respond_to(:audit_user_key) }
-  it { is_expected.to respond_to(:batch_user_key) }
-  it { is_expected.to respond_to(:browse_everything?) }
-  it { is_expected.to respond_to(:browse_everything=) }
-  it { is_expected.to respond_to(:cache_path) }
-  it { is_expected.to respond_to(:citations?) }
-  it { is_expected.to respond_to(:citations=) }
-  it { is_expected.to respond_to(:contact_email) }
-  it { is_expected.to respond_to(:display_media_download_link?) }
-  it { is_expected.to respond_to(:display_media_download_link=) }
-  it { is_expected.to respond_to(:display_microdata?) }
-  it { is_expected.to respond_to(:display_share_button_when_not_logged_in?) }
-  it { is_expected.to respond_to(:display_share_button_when_not_logged_in=) }
-  it { is_expected.to respond_to(:enable_noids?) }
-  it { is_expected.to respond_to(:feature_config_path) }
-  it { is_expected.to respond_to(:google_analytics_id?) }
-  it { is_expected.to respond_to(:google_analytics_id) }
-  it { is_expected.to respond_to(:iiif_image_server?) }
-  it { is_expected.to respond_to(:iiif_image_server=) }
-  it { is_expected.to respond_to(:iiif_image_url_builder) }
-  it { is_expected.to respond_to(:iiif_image_url_builder=) }
-  it { is_expected.to respond_to(:iiif_info_url_builder) }
-  it { is_expected.to respond_to(:iiif_info_url_builder=) }
-  it { is_expected.to respond_to(:iiif_image_compliance_level_uri) }
-  it { is_expected.to respond_to(:iiif_image_compliance_level_uri=) }
-  it { is_expected.to respond_to(:iiif_image_size_default) }
-  it { is_expected.to respond_to(:iiif_image_size_default=) }
-  it { is_expected.to respond_to(:iiif_metadata_fields) }
-  it { is_expected.to respond_to(:iiif_metadata_fields=) }
-  it { is_expected.to respond_to(:libreoffice_path) }
-  it { is_expected.to respond_to(:license_service_class) }
-  it { is_expected.to respond_to(:license_service_class=) }
-  it { is_expected.to respond_to(:max_days_between_fixity_checks=) }
-  it { is_expected.to respond_to(:max_days_between_fixity_checks) }
-  it { is_expected.to respond_to(:max_notifications_for_dashboard) }
-  it { is_expected.to respond_to(:owner_permission_levels) }
-  it { is_expected.to respond_to(:permission_levels) }
-  it { is_expected.to respond_to(:permission_options) }
-  it { is_expected.to respond_to(:persistent_hostpath) }
-  it { is_expected.to respond_to(:realtime_notifications?) }
-  it { is_expected.to respond_to(:realtime_notifications=) }
-  it { is_expected.to respond_to(:rendering_predicate) }
-  it { is_expected.to respond_to(:rendering_predicate=) }
-  it { is_expected.to respond_to(:rights_statement_service_class) }
-  it { is_expected.to respond_to(:rights_statement_service_class=) }
-  it { is_expected.to respond_to(:redis_namespace) }
-  it { is_expected.to respond_to(:subject_prefix) }
-  it { is_expected.to respond_to(:show_work_item_rows) }
-  it { is_expected.to respond_to(:translate_id_to_uri) }
-  it { is_expected.to respond_to(:translate_uri_to_id) }
-  it { is_expected.to respond_to(:upload_path) }
-  it { is_expected.to respond_to(:work_requires_files?) }
-  it { is_expected.to respond_to(:extract_full_text?) }
-  it { is_expected.to respond_to(:nested_relationship_reindexer) }
-  it { is_expected.to respond_to(:default_nested_relationship_reindexer) }
-  it { is_expected.to respond_to(:whitelisted_ingest_dirs) }
-  it { is_expected.to respond_to(:whitelisted_ingest_dirs=) }
-
-  describe "#whitelisted_ingest_dirs" do
-    it "provides the Rails tmp directory for temporary downloads for cloud files" do
-      expect(configuration.whitelisted_ingest_dirs).to include(Rails.root.join('tmp').to_s)
-    end
-  end
-end

data/spec/lib/hyrax/controlled_vocabulary/importer/downloader_spec.rb

@@ -1,33 +0,0 @@
-require 'hyrax/controlled_vocabulary/importer/downloader'
-
-RSpec.describe Hyrax::ControlledVocabulary::Importer::Downloader do
-  describe '.fetch' do
-    let(:url) { 'http://example.org/' }
-    let(:output) { double('output') }
-
-    context 'when connection is successful' do
-      let(:io) { double('io') }
-      let(:stream) { 'foo' }
-
-      before do
-        allow(IO).to receive(:copy_stream).with(io, output).and_return(stream)
-        allow(described_class).to receive(:open).with(url).and_yield(io)
-      end
-      it 'returns an IO stream' do
-        expect(described_class.fetch(url, output)).to eq stream
-      end
-    end
-    context 'when connection is unsuccessful' do
-      let(:exception_io) { double('io', read: '') }
-
-      before do
-        allow(described_class).to receive(:open).with(url) do
-          raise OpenURI::HTTPError.new('', exception_io)
-        end
-      end
-      it 'raises an exception' do
-        expect { described_class.fetch(url, output) }.to raise_error(RuntimeError, "Unable to download from #{url}\n: ")
-      end
-    end
-  end
-end

data/spec/lib/hyrax/controlled_vocabulary/importer/language_spec.rb

@@ -1,24 +0,0 @@
-require 'hyrax/controlled_vocabulary/importer/language'
-
-RSpec.describe Hyrax::ControlledVocabulary::Importer::Language do
-  before do
-    allow(Rails.logger).to receive(:extend)
-    allow(Hyrax::ControlledVocabulary::Importer::Downloader).to receive(:fetch)
-    allow(instance).to receive(:system) do
-      allow($CHILD_STATUS).to receive(:success?).and_return(true)
-    end
-  end
-
-  let(:instance) { described_class.new }
-  let(:rdf_path) { Gem.loaded_specs['hyrax'].full_gem_path + "/.internal_test_app/tmp/lexvo_2013-02-09.rdf" }
-
-  it "imports stuff" do
-    expect(Qa::Services::RDFAuthorityParser).to receive(:import_rdf).with(
-      'languages',
-      [rdf_path],
-      format: 'rdfxml',
-      predicate: RDF::URI('http://www.w3.org/2008/05/skos#prefLabel')
-    )
-    instance.import
-  end
-end

data/spec/lib/hyrax/move_all_works_to_admin_set_spec.rb

@@ -1,13 +0,0 @@
-require 'hyrax/move_all_works_to_admin_set'
-
-RSpec.describe MoveAllWorksToAdminSet, :clean_repo do
-  subject { described_class.run(admin_set) }
-
-  let(:admin_set) { create(:admin_set) }
-  let!(:work) { create(:work) }
-
-  it "moves the work into the admin set" do
-    subject
-    expect(work.reload.admin_set).to eq admin_set
-  end
-end

data/spec/lib/hyrax/name_spec.rb

@@ -1,21 +0,0 @@
-RSpec.describe Hyrax::Name do
-  let(:name) { described_class.new(GenericWork) }
-
-  describe "route_key" do
-    subject { name.route_key }
-
-    it { is_expected.to eq 'hyrax_generic_works' }
-  end
-
-  describe "singular_route_key" do
-    subject { name.singular_route_key }
-
-    it { is_expected.to eq 'hyrax_generic_work' }
-  end
-
-  describe "param_key" do
-    subject { name.param_key }
-
-    it { is_expected.to eq 'generic_work' }
-  end
-end

data/spec/lib/hyrax/redis_event_store_spec.rb

@@ -1,54 +0,0 @@
-RSpec.describe Hyrax::RedisEventStore do
-  let(:redis_instance) { described_class.instance }
-
-  before do
-    redis_instance.keys('events:*').each { |key| redis_instance.del key }
-  end
-
-  describe "::create" do
-    subject { described_class.create("some action", "1234") }
-
-    context "when it is successful" do
-      it { is_expected.to eq(1) }
-    end
-    context "when the Redis command fails" do
-      before { allow(Redis).to receive(:current).and_raise(Redis::CommandError) }
-      context "without a logger" do
-        before { allow(Rails).to receive(:logger).and_return(false) }
-        it { is_expected.to be_nil }
-      end
-      context "with a logger" do
-        it "logs the error" do
-          expect(Rails.logger).to receive(:error).exactly(:once).with("unable to create event: Redis::CommandError")
-          expect(subject).to be_nil
-        end
-      end
-    end
-  end
-
-  describe "#fetch" do
-    subject { described_class.new("key").fetch("size") }
-
-    context "when the Redis command fails" do
-      before { allow(Redis).to receive(:current).and_raise(Redis::CommandError) }
-      it { is_expected.to eq([]) }
-    end
-    context "when the Redis is unavailable" do
-      before { allow(Redis).to receive(:current).and_raise(Redis::CannotConnectError) }
-      it { is_expected.to eq([]) }
-    end
-  end
-
-  describe "#push" do
-    subject { described_class.new("key").push("some value") }
-
-    context "when the Redis command fails" do
-      before { allow(Redis).to receive(:current).and_raise(Redis::CommandError) }
-      it { is_expected.to be_nil }
-    end
-    context "when the Redis is unavailable" do
-      before { allow(Redis).to receive(:current).and_raise(Redis::CannotConnectError) }
-      it { is_expected.to be_nil }
-    end
-  end
-end

data/spec/lib/hyrax/resource_sync/capability_list_writer_spec.rb

@@ -1,33 +0,0 @@
-RSpec.describe Hyrax::ResourceSync::CapabilityListWriter do
-  let(:sitemap) { 'http://www.sitemaps.org/schemas/sitemap/0.9' }
-  let(:rs) { 'http://www.openarchives.org/rs/terms/' }
-
-  let(:resource_list) { 'http://example.com/resourcelist.xml' }
-  let(:change_list) { 'http://example.com/changelist.xml' }
-  let(:description) { 'http://example.com/resourcesync_description.xml' }
-  let(:xml) { Nokogiri::XML.parse(subject) }
-
-  subject do
-    described_class.new(resource_list_url: resource_list,
-                        change_list_url: change_list,
-                        description_url: description).write
-  end
-
-  it "has url to the capability list" do
-    description_href = xml.xpath('/x:urlset/rs:ln[@rel="up"]/@href', 'x' => sitemap, 'rs' => rs).map(&:value)
-    expect(description_href).to eq [description]
-
-    capability = xml.xpath('/x:urlset/rs:md/@capability', 'x' => sitemap, 'rs' => rs).map(&:value)
-    expect(capability).to eq ["capabilitylist"]
-
-    url = xml.xpath('//x:url[1]/x:loc', 'x' => sitemap).text
-    expect(url).to eq resource_list
-    capability = xml.xpath('//x:url[1]/rs:md/@capability', 'x' => sitemap, 'rs' => rs).map(&:value)
-    expect(capability).to eq ["resourcelist"]
-
-    url = xml.xpath('//x:url[2]/x:loc', 'x' => sitemap).text
-    expect(url).to eq change_list
-    capability = xml.xpath('//x:url[2]/rs:md/@capability', 'x' => sitemap, 'rs' => rs).map(&:value)
-    expect(capability).to eq ["changelist"]
-  end
-end

data/spec/lib/hyrax/resource_sync/change_list_writer_spec.rb

@@ -1,78 +0,0 @@
-RSpec.describe Hyrax::ResourceSync::ChangeListWriter, :clean_repo do
-  let(:sitemap) { 'http://www.sitemaps.org/schemas/sitemap/0.9' }
-  let(:public_collection) { create(:public_collection) }
-  let(:public_work) { create(:public_generic_work) }
-  let(:file_set) { create(:file_set, :public) }
-  let(:capability_list) { 'http://example.com/capabilityList.xml' }
-  let(:xml) { Nokogiri::XML.parse(subject) }
-  let(:url_list) { xml.xpath('//x:url', 'x' => sitemap) }
-  let(:instance) do
-    described_class.new(resource_host: 'example.com',
-                        capability_list_url: capability_list)
-  end
-  let(:namespaces) do
-    {
-      'rs' => "http://www.openarchives.org/rs/terms/",
-      'x' => sitemap
-    }
-  end
-  let(:capability_element) { xml.xpath('//rs:ln/@href', 'rs' => "http://www.openarchives.org/rs/terms/") }
-  # The creation and modified dates are used in order to determine whether or
-  # not the status is "created" or "updated"
-  # This avoids any potential delays/conflicts when testing against Solr and
-  # Fedora within the testing environment
-  # @see Hyrax::ResourceSync::ChangeListWriter#build_resource
-  let(:file_set_status) do
-    file_set.create_date.to_i == file_set.modified_date.to_i ? 'created' : 'updated'
-  end
-  let(:public_work_status) do
-    public_work.create_date.to_i == public_work.modified_date.to_i ? 'created' : 'updated'
-  end
-  let(:public_collection_status) do
-    public_collection.create_date.to_i == public_collection.modified_date.to_i ? 'created' : 'updated'
-  end
-
-  subject { instance.write }
-
-  context "without resources" do
-    it "has a list of resources" do
-      expect(url_list).to be_empty
-    end
-  end
-
-  context "when resources exist" do
-    before do
-      # These private items should not show up.
-      build(:private_collection_lw)
-      create(:work)
-
-      public_collection
-      public_work
-      file_set
-    end
-
-    it "has a list of resources" do
-      expect(capability_element.text).to eq capability_list
-      locations = location_elements(namespaces).map(&:text)
-      expect(locations).to include "http://example.com/concern/file_sets/#{file_set.id}"
-      expect(locations).to include "http://example.com/concern/generic_works/#{public_work.id}"
-      expect(locations).to include "http://example.com/collections/#{public_collection.id}"
-      changed = changed_elements(namespaces).map(&:value)
-      expect(changed).to match_array([public_collection_status, public_work_status, file_set_status])
-
-      expect(url_list.count).to eq 3
-    end
-  end
-
-  def changed_elements(namespaces = {})
-    query("rs:md/@change", namespaces)
-  end
-
-  def location_elements(namespaces = {})
-    query("x:loc", namespaces)
-  end
-
-  def query(part, ns = {})
-    xml.xpath("//x:url/#{part}", ns)
-  end
-end