hyrax 2.9.6 → 3.0.0.pre.beta1

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

@@ -0,0 +1,63 @@
+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

@@ -0,0 +1,12 @@
+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

@@ -0,0 +1,35 @@
+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

@@ -0,0 +1,85 @@
+RSpec.describe Hyrax::Configuration do
+  subject { 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=) }
+end

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

@@ -0,0 +1,33 @@
+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

@@ -0,0 +1,24 @@
+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

@@ -0,0 +1,13 @@
+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

@@ -0,0 +1,21 @@
+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

@@ -0,0 +1,54 @@
+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

@@ -0,0 +1,33 @@
+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

@@ -0,0 +1,64 @@
+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
+
+  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.
+      create(:private_collection)
+      create(:work)
+
+      # Sleep in between to ensure modified dates are different
+      public_collection
+      sleep(1)
+      public_work
+      sleep(1)
+      file_set
+    end
+
+    it "has a list of resources" do
+      capability = xml.xpath('//rs:ln/@href', 'rs' => "http://www.openarchives.org/rs/terms/").text
+      expect(capability).to eq capability_list
+
+      expect(location(1)).to eq "http://example.com/concern/file_sets/#{file_set.id}"
+      expect(change(1)).to eq "created"
+
+      expect(location(2)).to eq "http://example.com/concern/generic_works/#{public_work.id}"
+      expect(change(2)).to eq "created"
+
+      expect(location(3)).to eq "http://example.com/collections/#{public_collection.id}"
+      expect(change(3)).to eq "created"
+
+      expect(url_list.count).to eq 3
+    end
+  end
+
+  def change(n)
+    query(n, 'rs:md/@change', 'rs' => "http://www.openarchives.org/rs/terms/")
+  end
+
+  def location(n)
+    query(n, 'x:loc')
+  end
+
+  def query(n, part, ns = {})
+    xml.xpath("//x:url[#{n}]/#{part}", ns.merge('x' => sitemap)).text
+  end
+end

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

@@ -0,0 +1,25 @@
+RSpec.describe Hyrax::ResourceSync::ResourceListWriter, :clean_repo do
+  let(:sitemap) { 'http://www.sitemaps.org/schemas/sitemap/0.9' }
+  let!(:private_collection) { create(:private_collection) }
+  let!(:public_collection) { create(:public_collection) }
+  let!(:public_work) { create(:public_generic_work) }
+  let!(:private_work) { create(:work) }
+  let!(:file_set) { create(:file_set, :public) }
+  let(:capability_list) { 'http://example.com/capabilityList.xml' }
+  let(:xml) { Nokogiri::XML.parse(subject) }
+
+  subject { described_class.new(resource_host: 'example.com', capability_list_url: capability_list).write }
+
+  it "has a list of resources" do
+    capability = xml.xpath('//rs:ln/@href', 'rs' => "http://www.openarchives.org/rs/terms/").text
+    expect(capability).to eq capability_list
+    expect(query(1)).to eq "http://example.com/collections/#{public_collection.id}"
+    expect(query(2)).to eq "http://example.com/concern/generic_works/#{public_work.id}"
+    expect(query(3)).to eq "http://example.com/concern/file_sets/#{file_set.id}"
+    expect(xml.xpath('//x:url', 'x' => sitemap).count).to eq 3
+  end
+
+  def query(n)
+    xml.xpath("//x:url[#{n}]/x:loc", 'x' => sitemap).text
+  end
+end

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

@@ -0,0 +1,18 @@
+RSpec.describe Hyrax::ResourceSync::SourceDescriptionWriter do
+  let(:sitemap) { 'http://www.sitemaps.org/schemas/sitemap/0.9' }
+  let(:rs) { 'http://www.openarchives.org/rs/terms/' }
+  let(:capability_list) { 'http://example.com/capabilityList.xml' }
+  let(:xml) { Nokogiri::XML.parse(subject) }
+
+  subject { described_class.new(capability_list_url: capability_list).write }
+
+  it "has url to the capability list" do
+    capability = xml.xpath('/x:urlset/rs:md/@capability', 'x' => sitemap, 'rs' => rs).map(&:value)
+    expect(capability).to eq ["description"]
+
+    url = xml.xpath('//x:url[1]/x:loc', 'x' => sitemap).text
+    expect(url).to eq capability_list
+    capability = xml.xpath('//x:url[1]/rs:md/@capability', 'x' => sitemap, 'rs' => rs).map(&:value)
+    expect(capability).to eq ["capabilitylist"]
+  end
+end

data/spec/lib/hyrax/role_registry_spec.rb

@@ -0,0 +1,43 @@
+require 'rails_helper'
+
+RSpec.describe Hyrax::RoleRegistry do
+  let(:role_registry) { described_class.new }
+
+  describe '#role_names (without adding roles)' do
+    subject { role_registry.role_names }
+
+    it { is_expected.to eq(['approving', 'depositing', 'managing']) }
+  end
+
+  describe '#registered_role?' do
+    subject { role_registry.registered_role?(name: name) }
+
+    describe 'for already registered name' do
+      let(:name) { described_class::MANAGING }
+
+      it { is_expected.to be_truthy }
+    end
+    describe 'for non-registered name' do
+      let(:name) { 'gong_farming' }
+
+      it { is_expected.to be_falsey }
+    end
+  end
+
+  describe '#add' do
+    subject { role_registry.add(name: 'captaining', description: 'Grants captain duties') }
+
+    it 'includes those added via #add' do
+      expect { subject }.to change { role_registry.role_names }
+        .from(['approving', 'depositing', 'managing']).to(['approving', 'captaining', 'depositing', 'managing'])
+    end
+  end
+
+  describe '#persist_registered_roles!' do
+    subject { role_registry.persist_registered_roles! }
+
+    it 'creates Sipity::Role records for each role_name' do
+      expect { subject }.to change { Sipity::Role.count }.by(role_registry.role_names.count)
+    end
+  end
+end

data/spec/lib/hyrax/search_state_spec.rb

@@ -0,0 +1,26 @@
+RSpec.describe Hyrax::SearchState do
+  let(:controller) { CatalogController.new }
+  let(:config) { Blacklight::Configuration.new }
+  let(:state) { described_class.new({}, config, controller) }
+
+  describe "url_for_document" do
+    subject { state.url_for_document(document) }
+
+    context "with a collection" do
+      let(:document) { SolrDocument.new(id: '9999', has_model_ssim: ['Collection']) }
+
+      it "returns an array with the route set and doc" do
+        expect(subject.first).to be_kind_of ActionDispatch::Routing::RoutesProxy
+        expect(subject.last).to eq document
+      end
+    end
+    context "with a work" do
+      let(:document) { SolrDocument.new(id: '9999', has_model_ssim: ['GenericWork']) }
+
+      it "returns an array with the route set and doc" do
+        expect(subject.first).to be_kind_of ActionDispatch::Routing::RoutesProxy
+        expect(subject.last).to eq document
+      end
+    end
+  end
+end

data/spec/lib/hyrax/zotero/config_spec.rb

@@ -0,0 +1,26 @@
+RSpec.describe Hyrax::Zotero do
+  it { is_expected.to respond_to(:config) }
+
+  describe 'configuration' do
+    subject { described_class.reload_config! }
+
+    let(:client_key) { 'abc123' }
+    let(:client_secret) { '789xyz' }
+
+    before do
+      stub_const('ENV',           'ZOTERO_CLIENT_KEY' => client_key,
+                                  'ZOTERO_CLIENT_SECRET' => client_secret)
+    end
+
+    # Reload the config so other tests don't see the stub_const values
+    after { described_class.reload_config! }
+
+    it 'has a client key' do
+      expect(subject['client_key']).to eq(client_key)
+    end
+
+    it 'has a client secret' do
+      expect(subject['client_secret']).to eq(client_secret)
+    end
+  end
+end

data/spec/lib/hyrax/zotero_spec.rb

@@ -0,0 +1,9 @@
+RSpec.describe Hyrax::Zotero do
+  describe '.publications_url' do
+    let(:user_id) { 'foobar' }
+
+    it 'returns a string' do
+      expect(described_class.publications_url(user_id)).to eq "/users/#{user_id}/publications/items"
+    end
+  end
+end

data/spec/lib/hyrax_spec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+RSpec.describe Hyrax do
+  describe '.logger' do
+    it 'is a Logger' do
+      expect(described_class.logger).to respond_to :log
+    end
+  end
+end