hyrax 2.9.6 → 3.0.0.pre.beta1

data/spec/presenters/hyrax/work_usage_spec.rb

@@ -0,0 +1,144 @@
+RSpec.describe Hyrax::WorkUsage, type: :model do
+  let!(:work) { create(:work, id: 'abc12345xy') }
+
+  let(:dates) do
+    ldates = []
+    4.downto(0) { |idx| ldates << (Time.zone.today - idx.day) }
+    ldates
+  end
+  let(:date_strs) do
+    ldate_strs = []
+    dates.each { |date| ldate_strs << date.strftime("%Y%m%d") }
+    ldate_strs
+  end
+
+  let(:view_output) do
+    [[statistic_date(dates[0]), 4], [statistic_date(dates[1]), 8], [statistic_date(dates[2]), 6], [statistic_date(dates[3]), 10], [statistic_date(dates[4]), 2]]
+  end
+
+  # This is what the data looks like that's returned from Google Analytics (GA) via the Legato gem
+  # Due to the nature of querying GA, testing this data in an automated fashion is problematc.
+  # Sample data structures were created by sending real events to GA from a test instance of
+  # Scholarsphere.  The data below are essentially a "cut and paste" from the output of query
+  # results from the Legato gem.
+
+  let(:sample_pageview_statistics) do
+    [
+      SpecStatistic.new(date: date_strs[0], pageviews: 4),
+      SpecStatistic.new(date: date_strs[1], pageviews: 8),
+      SpecStatistic.new(date: date_strs[2], pageviews: 6),
+      SpecStatistic.new(date: date_strs[3], pageviews: 10),
+      SpecStatistic.new(date: date_strs[4], pageviews: 2)
+    ]
+  end
+
+  let(:usage) do
+    allow_any_instance_of(GenericWork).to receive(:create_date).and_return((Time.zone.today - 4.days).to_s)
+    allow(WorkViewStat).to receive(:ga_statistics).and_return(sample_pageview_statistics)
+    described_class.new(work.id)
+  end
+
+  describe "#initialize" do
+    it "sets the model" do
+      expect(usage.model).to eq work
+    end
+
+    it "sets the created date" do
+      expect(usage.created).to eq(work.create_date)
+    end
+  end
+
+  describe "#to_s" do
+    let(:work) { create(:work, title: ['Butter sculpture']) }
+
+    subject { usage.to_s }
+
+    it { is_expected.to eq 'Butter sculpture' }
+  end
+
+  describe "#to_flot" do
+    let(:flots) { usage.to_flot }
+
+    it "returns an array of hashes for use with JQuery Flot" do
+      expect(flots[0][:label]).to eq("Pageviews")
+      expect(flots[0][:data]).to include(*view_output)
+    end
+  end
+
+  describe "#total_pageviews" do
+    it "counts the total number of pageviews" do
+      expect(usage.total_pageviews).to eq(30)
+    end
+  end
+
+  describe "#created" do
+    let!(:system_timezone) { ENV['TZ'] }
+    let(:create_date) { Time.zone.parse('2014-01-02 12:00:00').iso8601 }
+
+    before do
+      ENV['TZ'] = 'UTC'
+    end
+
+    after do
+      ENV['TZ'] = system_timezone
+    end
+
+    describe "analytics start date set" do
+      let(:earliest) { Time.zone.parse('2014-01-02 12:00:00').iso8601 }
+
+      before do
+        Hyrax.config.analytic_start_date = earliest
+      end
+
+      describe "create date before earliest date set" do
+        let(:usage) do
+          allow_any_instance_of(GenericWork).to receive(:create_date).and_return(create_date.to_s)
+          described_class.new(work.id)
+        end
+
+        it "sets the created date to the earliest date not the created date" do
+          expect(usage.created).to eq(earliest)
+        end
+      end
+
+      describe "create date after earliest" do
+        let(:usage) do
+          allow_any_instance_of(GenericWork).to receive(:create_date).and_return((Time.zone.today - 4.days).to_s)
+          Hyrax.config.analytic_start_date = earliest
+          described_class.new(work.id)
+        end
+
+        it "sets the created date to the earliest date not the created date" do
+          expect(usage.created).to eq(work.create_date)
+        end
+      end
+    end
+    describe "start date not set" do
+      before do
+        Hyrax.config.analytic_start_date = nil
+      end
+
+      let(:usage) do
+        allow_any_instance_of(GenericWork).to receive(:create_date).and_return(create_date.to_s)
+        described_class.new(work.id)
+      end
+
+      it "sets the created date to the earliest date not the created date" do
+        expect(usage.created).to eq(create_date)
+      end
+    end
+  end
+
+  describe "on a migrated work" do
+    let(:date_uploaded) { "2014-12-31" }
+    let(:work_migrated) { create(:work, id: '678901234', date_uploaded: date_uploaded) }
+
+    let(:usage) do
+      described_class.new(work_migrated.id)
+    end
+
+    it "uses the date_uploaded for analytics" do
+      expect(usage.created).to eq(date_uploaded)
+    end
+  end
+end

data/spec/presenters/hyrax/workflow_presenter_spec.rb

@@ -0,0 +1,75 @@
+RSpec.describe Hyrax::WorkflowPresenter do
+  let(:solr_document) { SolrDocument.new(attributes) }
+  let(:attributes) do
+    { "id" => '888888',
+      "has_model_ssim" => ["GenericWork"] }
+  end
+
+  let(:user) { create(:user) }
+  let(:ability) { Ability.new(user) }
+  let(:presenter) { described_class.new(solr_document, ability) }
+  let(:entity) { instance_double(Sipity::Entity) }
+
+  describe "#actions" do
+    let(:workflow) { create(:workflow, name: 'testing') }
+
+    subject { presenter.actions }
+
+    context 'with a Sipity::Entity' do
+      before do
+        allow(Hyrax::Workflow::PermissionQuery).to receive(:scope_permitted_workflow_actions_available_for_current_state).and_return([Sipity::WorkflowAction.new(name: "complete", workflow: workflow)])
+        allow(presenter).to receive(:sipity_entity).and_return(entity)
+      end
+      it "is an Array of Sipity::Action#name and translated names" do
+        allow(I18n).to receive(:t).with('hyrax.workflow.testing.complete', default: 'Complete').and_return("Approve")
+        is_expected.to eq [['complete', 'Approve']]
+      end
+    end
+    context 'without a Sipity::Entity' do
+      before do
+        allow(presenter).to receive(:sipity_entity).and_return(nil)
+      end
+      it { is_expected.to eq [] }
+    end
+  end
+
+  describe "#badge" do
+    let(:workflow) { create(:workflow, name: 'testing') }
+
+    subject { presenter.badge }
+
+    context 'with a Sipity::Entity' do
+      before do
+        allow(entity).to receive(:workflow_state_name).and_return('complete')
+        allow(presenter).to receive(:sipity_entity).and_return(entity)
+      end
+      it { is_expected.to eq '<span class="state state-complete label label-primary">Complete</span>' }
+    end
+    context 'without a Sipity::Entity' do
+      before do
+        allow(presenter).to receive(:sipity_entity).and_return(nil)
+      end
+      it { is_expected.to be nil }
+    end
+  end
+
+  describe "#comments" do
+    subject { presenter.comments }
+
+    context 'with a Sipity::Entity' do
+      let(:comment) { instance_double(Sipity::Comment) }
+
+      before do
+        allow(entity).to receive(:comments).and_return([comment])
+        allow(presenter).to receive(:sipity_entity).and_return(entity)
+      end
+      it { is_expected.to eq [comment] }
+    end
+    context 'without a Sipity::Entity' do
+      before do
+        allow(presenter).to receive(:sipity_entity).and_return(nil)
+      end
+      it { is_expected.to eq [] }
+    end
+  end
+end

data/spec/rails_helper.rb

@@ -0,0 +1,2 @@
+# This file is normaly copied to spec/ when you run 'rails generate rspec:install'
+# but this is a dummy file to allow hyrax specs to run.

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

@@ -0,0 +1,118 @@
+RSpec.describe Hyrax::Renderers::AttributeRenderer do
+  let(:field) { :name }
+  let(:renderer) { described_class.new(field, ['Bob', 'Jessica']) }
+  let(:yml_path) { File.join(fixture_path, 'config', 'schema_org.{yml}') }
+
+  before do
+    Hyrax::Microdata.load_paths += Dir[yml_path]
+    Hyrax::Microdata.reload!
+  end
+  after do
+    Hyrax::Microdata.load_paths -= Dir[yml_path]
+    Hyrax::Microdata.reload!
+  end
+
+  describe "#attribute_to_html" do
+    subject { Nokogiri::HTML(renderer.render) }
+
+    let(:expected) { Nokogiri::HTML(tr_content) }
+
+    context 'without microdata enabled' do
+      before do
+        allow(Hyrax.config).to receive(:display_microdata?).and_return(false)
+      end
+      let(:tr_content) do
+        "<tr><th>Name</th>\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
+
+      it { expect(renderer).not_to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+
+    context 'with an integer attribute' do
+      let(:field) { :height }
+      let(:renderer) { described_class.new(field, [567]) }
+      let(:tr_content) do
+        "<tr><th>Height</th>\n" \
+         "<td><ul class='tabular'><li class=\"attribute attribute-height\">567</li>\n" \
+         "</ul></td></tr>"
+      end
+
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+
+    context 'with microdata enabled' do
+      let(:tr_content) do
+        "<tr><th>Name</th>\n" \
+         "<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 attribute-name\" itemscope itemtype=\"http://schema.org/Person\" itemprop=\"name\">" \
+         "<span itemprop=\"firstName\">Jessica</span></li>\n" \
+         "</ul></td></tr>"
+      end
+
+      it { expect(renderer).to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+
+    context 'with links and < characters' do
+      let(:field) { :description }
+      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 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>"
+      end
+
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+  end
+
+  describe "#render_dl_row" do
+    subject { Nokogiri::HTML(renderer.render_dl_row) }
+
+    let(:expected) { Nokogiri::HTML(dl_content) }
+    let(:dl_content) do
+      "<dt>Name</dt>\n" \
+       "<dd><ul class='tabular'><li class=\"attribute attribute-name\">Bob</li>\n" \
+       "<li class=\"attribute attribute-name\">Jessica</li>\n" \
+       "</ul></dd>"
+    end
+
+    before do
+      allow(Hyrax.config).to receive(:display_microdata?).and_return(false)
+    end
+
+    it { expect(renderer).not_to be_microdata(field) }
+    it { expect(subject).to be_equivalent_to(expected) }
+  end
+
+  describe "#label" do
+    subject { renderer }
+
+    context 'with work type option' do
+      let(:work_type) { "GenericWork".underscore }
+      let(:renderer) { described_class.new(field, ['Bob', 'Jessica'], work_type: work_type) }
+
+      context 'no work type specific label' do
+        it { expect(subject.label).to eq(field.to_s.humanize) }
+      end
+      context 'work type specific label' do
+        let(:work_type_name_label) { "Appellation" }
+
+        before do
+          allow(I18n).to receive(:translate).and_call_original
+          allow(I18n).to receive(:translate).with(:"blacklight.search.fields.#{work_type}.show.#{field}", Hash) do
+            work_type_name_label
+          end
+        end
+        it { expect(subject.label).to eq(work_type_name_label) }
+      end
+    end
+  end
+end

data/spec/renderers/hyrax/renderers/date_attribute_renderer_spec.rb

@@ -0,0 +1,39 @@
+RSpec.describe Hyrax::Renderers::DateAttributeRenderer do
+  subject { Nokogiri::HTML(renderer.render) }
+
+  let(:expected) { Nokogiri::HTML(tr_content) }
+
+  describe "#attribute_to_html" do
+    context 'with embargo release date' do
+      let(:field) { :embargo_release_date }
+      let(:renderer) { described_class.new(field, ['2013-03-14T00:00:00Z']) }
+      let(:tr_content) do
+        %(
+      <tr><th>Embargo release date</th>
+      <td><ul class="tabular">
+      <li class="attribute attribute-embargo_release_date">03/14/2013</li>
+      </ul></td></tr>
+      )
+      end
+
+      it { expect(renderer).not_to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+
+    context 'with lease expiration date' do
+      let(:field) { :lease_expiration_date }
+      let(:renderer) { described_class.new(field, ['2013-03-14T00:00:00Z']) }
+      let(:tr_content) do
+        %(
+      <tr><th>Lease expiration date</th>
+      <td><ul class="tabular">
+      <li class="attribute attribute-lease_expiration_date">03/14/2013</li>
+      </ul></td></tr>
+      )
+      end
+
+      it { expect(renderer).not_to be_microdata(field) }
+      it { expect(subject).to be_equivalent_to(expected) }
+    end
+  end
+end

data/spec/renderers/hyrax/renderers/external_link_attribute_renderer_spec.rb

@@ -0,0 +1,22 @@
+RSpec.describe Hyrax::Renderers::ExternalLinkAttributeRenderer do
+  let(:field) { :name }
+  let(:renderer) { described_class.new(field, ['http://example.com']) }
+
+  describe "#attribute_to_html" do
+    subject { Nokogiri::HTML(renderer.render) }
+
+    let(:expected) { Nokogiri::HTML(tr_content) }
+
+    let(:tr_content) do
+      "<tr><th>Name</th>\n" \
+       "<td><ul class='tabular'>" \
+       "<li class=\"attribute attribute-name\">"\
+       "<a href=\"http://example.com\">"\
+       "<span class='glyphicon glyphicon-new-window'></span>&nbsp;"\
+       "http://example.com</a></li>\n" \
+       "</ul></td></tr>"
+    end
+
+    it { expect(subject).to be_equivalent_to(expected) }
+  end
+end

data/spec/renderers/hyrax/renderers/faceted_attribute_renderer_spec.rb

@@ -0,0 +1,35 @@
+RSpec.describe Hyrax::Renderers::FacetedAttributeRenderer do
+  let(:field) { :name }
+  let(:renderer) { described_class.new(field, ['Bob', 'Jessica']) }
+
+  describe "#attribute_to_html" do
+    subject { Nokogiri::HTML(renderer.render) }
+
+    let(:expected) { Nokogiri::HTML(tr_content) }
+
+    let(:tr_content) do
+      %(
+      <tr><th>Name</th>
+      <td><ul class='tabular'>
+      <li class="attribute attribute-name"><a href="/catalog?f%5Bname_sim%5D%5B%5D=Bob&locale=en">Bob</a></li>
+      <li class="attribute attribute-name"><a href="/catalog?f%5Bname_sim%5D%5B%5D=Jessica&locale=en">Jessica</a></li>
+      </ul></td></tr>
+    )
+    end
+
+    it { expect(renderer).not_to be_microdata(field) }
+    it { expect(subject).to be_equivalent_to(expected) }
+  end
+
+  describe "href generated" do
+    describe "escaping" do
+      let(:renderer) { described_class.new(field, ['John & Bob']) }
+      let(:rendered_link) { Nokogiri::HTML(renderer.render).at_css("a") }
+      let(:rendered_link_query) { URI.parse(rendered_link['href']).query }
+
+      it "escapes content properly" do
+        expect(rendered_link_query).to eq "#{CGI.escape('f[name_sim][]')}=#{CGI.escape('John & Bob')}&locale=en"
+      end
+    end
+  end
+end

data/spec/renderers/hyrax/renderers/license_attribute_renderer_spec.rb

@@ -0,0 +1,21 @@
+RSpec.describe Hyrax::Renderers::LicenseAttributeRenderer do
+  let(:field) { :license }
+  let(:renderer) { described_class.new(field, ['http://creativecommons.org/licenses/by/3.0/us/', 'http://creativecommons.org/licenses/by-nd/3.0/us/']) }
+
+  describe "#attribute_to_html" do
+    subject { Nokogiri::HTML(renderer.render) }
+
+    let(:expected) { Nokogiri::HTML(tr_content) }
+
+    let(:tr_content) do
+      "<tr><th>License</th>\n" \
+       "<td><ul class='tabular'>\n" \
+       "<li class=\"attribute attribute-license\"><a href=\"http://creativecommons.org/licenses/by/3.0/us/\" target=\"_blank\">Attribution 3.0 United States</a></li>\n" \
+       "<li class=\"attribute attribute-license\"><a href=\"http://creativecommons.org/licenses/by-nd/3.0/us/\" target=\"_blank\">Attribution-NoDerivs 3.0 United States</a></li>\n" \
+       "</ul></td></tr>"
+    end
+
+    it { expect(renderer).not_to be_microdata(field) }
+    it { expect(subject).to be_equivalent_to(expected) }
+  end
+end

data/spec/renderers/hyrax/renderers/linked_attribute_renderer_spec.rb

@@ -0,0 +1,21 @@
+RSpec.describe Hyrax::Renderers::LinkedAttributeRenderer do
+  let(:field) { :name }
+  let(:renderer) { described_class.new(field, ['Bob', 'Jessica']) }
+
+  describe "#attribute_to_html" do
+    subject { Nokogiri::HTML(renderer.render) }
+
+    let(:expected) { Nokogiri::HTML(tr_content) }
+
+    let(:tr_content) do
+      "<tr><th>Name</th>\n" \
+       "<td><ul class='tabular'>" \
+       "<li class=\"attribute attribute-name\"><a href=\"/catalog?locale=en&q=Bob&amp;search_field=name\">Bob</a></li>\n" \
+       "<li class=\"attribute attribute-name\"><a href=\"/catalog?locale=en&q=Jessica&amp;search_field=name\">Jessica</a></li>\n" \
+       "</ul></td></tr>"
+    end
+
+    it { expect(renderer).not_to be_microdata(field) }
+    it { expect(subject).to be_equivalent_to(expected) }
+  end
+end

data/spec/renderers/hyrax/renderers/rights_statement_attribute_renderer_spec.rb

@@ -0,0 +1,29 @@
+RSpec.describe Hyrax::Renderers::RightsStatementAttributeRenderer do
+  let(:field) { :rights_statement }
+  let(:renderer) { described_class.new(field, ['http://rightsstatements.org/vocab/InC/1.0/']) }
+
+  describe "#attribute_to_html" do
+    subject { Nokogiri::HTML(renderer.render) }
+
+    let(:expected) { Nokogiri::HTML(tr_content) }
+
+    let(:tr_content) do
+      "<tr><th>Rights statement</th>\n" \
+       "<td><ul class='tabular'>" \
+       "<li class=\"attribute attribute-rights_statement\"><a href=\"http://rightsstatements.org/vocab/InC/1.0/\" target=\"_blank\">In Copyright</a></li>" \
+       "</ul></td></tr>"
+    end
+
+    it { expect(renderer).not_to be_microdata(field) }
+    it { expect(subject).to be_equivalent_to(expected) }
+
+    context 'with off-authority term' do
+      let(:renderer) { described_class.new(field, [value]) }
+      let(:value)    { 'moomin' }
+
+      it 'renders a value' do
+        expect(subject.to_s).to include value
+      end
+    end
+  end
+end

data/spec/requests/legacy_routing_spec.rb

@@ -0,0 +1,7 @@
+RSpec.describe 'Legacy GenericFile routes', type: :request do
+  it 'redirects to the work' do
+    get '/files/gm80hv36p'
+    expect(response).to redirect_to("/concern/generic_works/gm80hv36p")
+    expect(response.code).to eq '301' # Moved Permanently
+  end
+end

data/spec/requests/riiif_spec.rb

@@ -0,0 +1,32 @@
+RSpec.describe 'IIIF image API', type: :request do
+  let(:user) { create(:user) }
+  let(:work) { create(:work_with_one_file, user: user) }
+  let(:file_set) { work.ordered_members.to_a.first }
+  let(:file) { file_set.original_file }
+  let(:size) { '300,' }
+
+  before do
+    Hydra::Works::AddFileToFileSet.call(file_set,
+                                        File.open(fixture_path + '/world.png'),
+                                        :original_file)
+  end
+
+  describe 'GET /images/:id' do
+    context "when the user is authorized" do
+      it "returns an image" do
+        login_as user
+        get Riiif::Engine.routes.url_helpers.image_path(file.id, size: size)
+        expect(response).to have_http_status(:ok)
+        expect(response.content_type).to eq 'image/jpeg'
+      end
+    end
+
+    context "when the user is not authorized" do
+      it "returns an image" do
+        get Riiif::Engine.routes.url_helpers.image_path(file.id, size: size)
+        expect(response).to have_http_status(:unauthorized)
+        expect(response.content_type).to eq 'image/jpeg'
+      end
+    end
+  end
+end

data/spec/routing/admin_routes_spec.rb

@@ -0,0 +1,29 @@
+RSpec.describe 'Admin Routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'routes the appearance' do
+    expect(get: '/admin/appearance').to route_to(controller: 'hyrax/admin/appearances', action: 'show')
+  end
+
+  it 'routes the statistics page' do
+    expect(get: '/admin/stats').to route_to(controller: 'hyrax/admin/stats', action: 'show')
+  end
+
+  it 'routes the workflows' do
+    expect(get: '/admin/workflows').to route_to(controller: 'hyrax/admin/workflows', action: 'index')
+  end
+
+  it 'routes the workflow roles' do
+    expect(get: '/admin/workflow_roles').to route_to(controller: 'hyrax/admin/workflow_roles', action: 'index')
+  end
+
+  describe "Features" do
+    it "routes to the features controller" do
+      expect(get: '/admin/features').to route_to(controller: 'hyrax/admin/features', action: 'index')
+    end
+
+    it "routes to the strategies controller" do
+      expect(patch: '/admin/features/foo/strategies/bar').to route_to(controller: 'hyrax/admin/strategies', action: 'update', id: 'bar', feature_id: 'foo')
+    end
+  end
+end

data/spec/routing/api_route_spec.rb

@@ -0,0 +1,89 @@
+RSpec.describe 'routing and paths', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  let(:item_id) { '123' }
+
+  describe 'Arkivo' do
+    context 'with a constraint defined' do
+      before do
+        allow(Hyrax::ArkivoConstraint).to receive(:matches?) { false }
+      end
+
+      it 'does not recognize routes' do
+        expect(post: '/api/items').not_to be_routable
+      end
+    end
+
+    context 'without a constraint defined' do
+      before do
+        allow(Hyrax::ArkivoConstraint).to receive(:matches?) { true }
+      end
+
+      it 'routes POSTs to the items resource' do
+        expect(post: '/api/items').to route_to(controller: 'hyrax/api/items', action: 'create', format: :json)
+      end
+
+      it 'does not route GETs to the items resource' do
+        expect(get: '/api/items').not_to route_to(controller: 'hyrax/api/items', action: 'index', format: :json)
+      end
+
+      it 'does not route DELETEs to the items resource' do
+        expect(delete: '/api/items').not_to route_to(controller: 'hyrax/api/items', action: 'destroy', format: :json)
+      end
+
+      it 'does not route PUTs to the items resource' do
+        expect(put: '/api/items').not_to route_to(controller: 'hyrax/api/items', action: 'update', format: :json)
+      end
+
+      it 'does not route PATCHes to the items resource' do
+        expect(patch: '/api/items').not_to route_to(controller: 'hyrax/api/items', action: 'update', format: :json)
+      end
+
+      context 'with a member resource' do
+        subject { "/api/items/#{item_id}" }
+
+        it 'routes GETs to an item resource' do
+          expect(get: subject).to route_to(controller: 'hyrax/api/items', action: 'show', id: item_id, format: :json)
+        end
+
+        it 'routes PUTs to an item resource' do
+          expect(put: subject).to route_to(controller: 'hyrax/api/items', action: 'update', id: item_id, format: :json)
+        end
+
+        it 'routes DELETEs to an item resource' do
+          expect(delete: subject).to route_to(controller: 'hyrax/api/items', action: 'destroy', id: item_id, format: :json)
+        end
+      end
+    end
+  end
+
+  describe 'Zotero' do
+    it 'routes GETs to the zotero resource' do
+      expect(get: '/api/zotero').to route_to(controller: 'hyrax/api/zotero', action: 'initiate')
+    end
+  end
+
+  describe 'Zotero callbacks' do
+    it 'routes GETs to the callback resource' do
+      expect(get: '/api/zotero/callback').to route_to(controller: 'hyrax/api/zotero', action: 'callback')
+    end
+  end
+
+  describe 'path helpers' do
+    it 'has a path for creating new items' do
+      expect(api_items_path).to eq '/api/items'
+    end
+
+    it 'has a path for operating on existing items' do
+      expect(api_item_path(item_id)).to eq "/api/items/#{item_id}"
+    end
+
+    it 'has a path for initiating Zotero authentication' do
+      expect(api_zotero_initiate_path).to eq '/api/zotero'
+    end
+
+    it 'has a path for Zotero authN callbacks' do
+      expect(api_zotero_callback_path).to eq '/api/zotero/callback'
+    end
+  end
+end