hyrax 2.9.5 → 3.0.0.pre.beta1

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

data/spec/routing/citations_route_spec.rb

@@ -0,0 +1,21 @@
+RSpec.describe "citations routes", type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  context "for works" do
+    it 'routes to the controller' do
+      expect(get: '/works/7/citation').to route_to(controller: 'hyrax/citations', action: 'work', id: '7')
+    end
+    it 'builds a url' do
+      expect(url_for(controller: 'hyrax/citations', action: 'work', id: '7', only_path: true)).to eql('/works/7/citation')
+    end
+  end
+
+  context "for files" do
+    it 'routes to the controller' do
+      expect(get: '/files/7/citation').to route_to(controller: 'hyrax/citations', action: 'file', id: '7')
+    end
+    it 'builds a url' do
+      expect(url_for(controller: 'hyrax/citations', action: 'file', id: '7', only_path: true)).to eql('/files/7/citation')
+    end
+  end
+end

data/spec/routing/collection_permission_templates_routes_spec.rb

@@ -0,0 +1,31 @@
+RSpec.describe 'Collection Permission Templates Routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'routes to #create' do
+    expect(post: '/dashboard/collections/1/permission_template').to route_to(controller: 'hyrax/admin/permission_templates', action: 'create', collection_id: '1')
+  end
+
+  it 'routes to #new' do
+    expect(get: '/dashboard/collections/permission_template/new').to route_to(controller: 'hyrax/admin/permission_templates', action: 'new')
+  end
+
+  it 'routes to #edit' do
+    expect(get: '/dashboard/collections/1/permission_template/edit').to route_to(controller: 'hyrax/admin/permission_templates', action: 'edit', collection_id: '1')
+  end
+
+  it 'routes to #show' do
+    expect(get: '/dashboard/collections/1/permission_template').to route_to(controller: 'hyrax/admin/permission_templates', action: 'show', collection_id: '1')
+  end
+
+  it 'routes to #update (patch)' do
+    expect(patch: '/dashboard/collections/1/permission_template').to route_to(controller: 'hyrax/admin/permission_templates', action: 'update', collection_id: '1')
+  end
+
+  it 'routes to #update (put)' do
+    expect(put: '/dashboard/collections/1/permission_template').to route_to(controller: 'hyrax/admin/permission_templates', action: 'update', collection_id: '1')
+  end
+
+  it 'routes to #destroy' do
+    expect(delete: '/dashboard/collections/1/permission_template').to route_to(controller: 'hyrax/admin/permission_templates', action: 'destroy', collection_id: '1')
+  end
+end

data/spec/routing/collection_types_routes.rb

@@ -0,0 +1,35 @@
+RSpec.describe 'Collection Types Routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'routes to #index' do
+    expect(get: '/admin/collection_types').to route_to(controller: 'hyrax/admin/collection_types', action: 'index')
+  end
+
+  it 'routes to #create' do
+    expect(post: '/admin/collection_types').to route_to(controller: 'hyrax/admin/collection_types', action: 'create')
+  end
+
+  it 'routes to #new' do
+    expect(get: '/admin/collection_types/new').to route_to(controller: 'hyrax/admin/collection_types', action: 'new')
+  end
+
+  it 'routes to #edit' do
+    expect(get: '/admin/collection_types/1/edit').to route_to(controller: 'hyrax/admin/collection_types', action: 'edit', id: '1')
+  end
+
+  it 'routes to #update (patch)' do
+    expect(patch: '/admin/collection_types/1').to route_to(controller: 'hyrax/admin/collection_types', action: 'update', id: '1')
+  end
+
+  it 'routes to #update (put)' do
+    expect(put: '/admin/collection_types/1').to route_to(controller: 'hyrax/admin/collection_types', action: 'update', id: '1')
+  end
+
+  it 'routes to #destroy' do
+    expect(delete: '/admin/collection_types/1').to route_to(controller: 'hyrax/admin/collection_types', action: 'destroy', id: '1')
+  end
+
+  it 'does not route to #show' do
+    expect(get: '/admin/collection_types/1').not_to route_to(controller: 'hyrax/admin/collection_types', action: 'show', id: '1')
+  end
+end

data/spec/routing/dashboard_routes_spec.rb

@@ -0,0 +1,37 @@
+RSpec.describe 'Dashboard Routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'routes GET /dashboard/collections/:parent_id/under' do
+    expect(get: '/dashboard/collections/parent1/under').to route_to(controller: 'hyrax/dashboard/nest_collections', action: 'create_collection_under', parent_id: 'parent1')
+  end
+
+  it 'routes POST /dashboard/collections/:parent_id/under' do
+    expect(post: '/dashboard/collections/parent1/under').to route_to(controller: 'hyrax/dashboard/nest_collections', action: 'create_relationship_under', parent_id: 'parent1')
+  end
+
+  it 'routes POST /dashboard/collections/:child_id/within' do
+    expect(post: '/dashboard/collections/child1/within').to route_to(controller: 'hyrax/dashboard/nest_collections', action: 'create_relationship_within', child_id: 'child1')
+  end
+
+  it 'routes POST /dashboard/collections/:child_id/remove_parent/:parent_id' do
+    expect(post: '/dashboard/collections/child1/remove_parent/parent1').to route_to(
+      controller: 'hyrax/dashboard/nest_collections',
+      action: 'remove_relationship_above',
+      child_id: 'child1',
+      parent_id: 'parent1'
+    )
+  end
+
+  it 'routes POST /dashboard/collections/:parent_id/remove_child/:child_id' do
+    expect(post: '/dashboard/collections/parent1/remove_child/child1').to route_to(
+      controller: 'hyrax/dashboard/nest_collections',
+      action: 'remove_relationship_under',
+      child_id: 'child1',
+      parent_id: 'parent1'
+    )
+  end
+
+  it 'routes POST /dashboard/collections/:id' do
+    expect(post: '/dashboard/collections/id').to route_to(controller: 'hyrax/dashboard/collection_members', action: 'update_members', id: 'id')
+  end
+end

data/spec/routing/featured_works_route_spec.rb

@@ -0,0 +1,15 @@
+RSpec.describe "file routes", type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'creates a featured_work' do
+    expect(post: '/works/7/featured_work').to route_to(controller: 'hyrax/featured_works', action: 'create', id: '7')
+  end
+  it 'removes a featured_work' do
+    expect(delete: '/works/7/featured_work').to route_to(controller: 'hyrax/featured_works', action: 'destroy', id: '7')
+  end
+
+  it 'updates a collection of featured works' do
+    expect(featured_work_lists_path).to eq '/featured_works'
+    expect(post: '/featured_works').to route_to(controller: 'hyrax/featured_work_lists', action: 'create')
+  end
+end

data/spec/routing/operations_spec.rb

@@ -0,0 +1,9 @@
+RSpec.describe 'Operations routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'routes the list view' do
+    expect(get: '/users/77/operations').to route_to(controller: 'hyrax/operations',
+                                                    action: 'index',
+                                                    user_id: "77")
+  end
+end

data/spec/routing/ownership_transfers_route_spec.rb

@@ -0,0 +1,43 @@
+RSpec.describe "proxy deposit and transfers routing", type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it "lists transfers" do
+    expect(transfers_path).to eq '/dashboard/transfers'
+    expect(get: '/dashboard/transfers').to route_to(controller: 'hyrax/transfers', action: 'index')
+  end
+
+  it "creates a transfer" do
+    expect(work_transfers_path('7')).to eq '/works/7/transfers'
+    expect(post: '/works/7/transfers').to route_to(controller: 'hyrax/transfers', action: 'create', id: '7')
+  end
+
+  it "shows a form for a new transfer" do
+    expect(new_work_transfer_path('7')).to eq '/works/7/transfers/new'
+    expect(get: '/works/7/transfers/new').to route_to(controller: 'hyrax/transfers', action: 'new', id: '7')
+  end
+
+  it "cancels a transfer" do
+    expect(transfer_path('7')).to eq '/dashboard/transfers/7'
+    expect(delete: '/dashboard/transfers/7').to route_to(controller: 'hyrax/transfers', action: 'destroy', id: '7')
+  end
+
+  it "accepts a transfers" do
+    expect(accept_transfer_path('7')).to eq '/dashboard/transfers/7/accept'
+    expect(put: '/dashboard/transfers/7/accept').to route_to(controller: 'hyrax/transfers', action: 'accept', id: '7')
+  end
+
+  it "rejects a transfer" do
+    expect(reject_transfer_path('7')).to eq '/dashboard/transfers/7/reject'
+    expect(put: '/dashboard/transfers/7/reject').to route_to(controller: 'hyrax/transfers', action: 'reject', id: '7')
+  end
+
+  it "adds a proxy depositor" do
+    expect(user_depositors_path('xxx666@example-dot-org')).to eq '/users/xxx666@example-dot-org/depositors'
+    expect(post: '/users/xxx666@example-dot-org/depositors').to route_to(controller: 'hyrax/depositors', action: 'create', user_id: 'xxx666@example-dot-org')
+  end
+
+  it "removes a proxy depositor" do
+    expect(user_depositor_path('xxx666', '33')).to eq '/users/xxx666/depositors/33'
+    expect(delete: '/users/xxx666/depositors/33').to route_to(controller: 'hyrax/depositors', action: 'destroy', user_id: 'xxx666', id: '33')
+  end
+end

data/spec/routing/resource_sync_spec.rb

@@ -0,0 +1,19 @@
+RSpec.describe 'ResourceSync Routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  it 'routes the well-known uri' do
+    expect(get: '/.well-known/resourcesync').to route_to(controller: 'hyrax/resource_sync', action: 'source_description')
+  end
+
+  it 'routes the capability list' do
+    expect(get: '/capabilitylist').to route_to(controller: 'hyrax/resource_sync', action: 'capability_list')
+  end
+
+  it 'routes the resource list' do
+    expect(get: '/resourcelist').to route_to(controller: 'hyrax/resource_sync', action: 'resource_list')
+  end
+
+  it 'routes the change list' do
+    expect(get: '/changelist').to route_to(controller: 'hyrax/resource_sync', action: 'change_list')
+  end
+end

data/spec/routing/route_spec.rb

@@ -0,0 +1,205 @@
+RSpec.describe 'Routes', type: :routing do
+  routes { Hyrax::Engine.routes }
+
+  describe 'Homepage' do
+    it 'routes the root url to the homepage controller' do
+      expect(get: '/').to route_to(controller: 'hyrax/homepage', action: 'index')
+    end
+  end
+
+  describe "Fixity check" do
+    it 'creates a fixity check' do
+      expect(post: '/concern/file_sets/7/fixity_checks').to route_to(controller: 'hyrax/fixity_checks', action: 'create', file_set_id: '7')
+    end
+  end
+
+  describe "BatchUpload" do
+    context "without a batch" do
+      routes { Hyrax::Engine.routes }
+      it 'routes to create' do
+        expect(post: '/batch_uploads').to route_to(controller: 'hyrax/batch_uploads', action: 'create')
+      end
+
+      it "routes to new" do
+        expect(get: '/batch_uploads/new').to route_to(controller: 'hyrax/batch_uploads', action: 'new')
+      end
+    end
+  end
+
+  describe 'FileSet' do
+    context "main app routes" do
+      routes { Rails.application.routes }
+
+      it 'routes to edit' do
+        expect(get: '/concern/file_sets/3/edit').to route_to(controller: 'hyrax/file_sets', action: 'edit', id: '3')
+      end
+
+      it "routes to show" do
+        expect(get: '/concern/file_sets/4').to route_to(controller: 'hyrax/file_sets', action: 'show', id: '4')
+      end
+
+      it "routes to update" do
+        expect(put: '/concern/file_sets/5').to route_to(controller: 'hyrax/file_sets', action: 'update', id: '5')
+      end
+
+      it "routes to destroy" do
+        expect(delete: '/concern/file_sets/6').to route_to(controller: 'hyrax/file_sets', action: 'destroy', id: '6')
+      end
+
+      it "doesn't route to index" do
+        expect(get: '/concern/file_sets').not_to route_to(controller: 'hyrax/file_sets', action: 'index')
+      end
+    end
+  end
+
+  describe 'Download' do
+    it "routes to show" do
+      expect(get: '/downloads/9').to route_to(controller: 'hyrax/downloads', action: 'show', id: '9')
+    end
+  end
+
+  describe 'Dashboard' do
+    it "routes to dashboard" do
+      expect(get: '/dashboard').to route_to(controller: 'hyrax/dashboard', action: 'show')
+    end
+
+    it "routes to dashboard activity" do
+      expect(get: '/dashboard/activity').to route_to(controller: 'hyrax/dashboard', action: 'activity')
+    end
+
+    it "routes to all works" do
+      expect(get: '/dashboard/works').to route_to(controller: 'hyrax/dashboard/works', action: 'index')
+    end
+
+    it "routes to all collections" do
+      expect(get: '/dashboard/collections').to route_to(controller: 'hyrax/dashboard/collections', action: 'index')
+    end
+
+    it "routes to my works" do
+      expect(get: '/dashboard/my/works').to route_to(controller: 'hyrax/my/works', action: 'index')
+    end
+
+    it "routes to my collections" do
+      expect(get: '/dashboard/my/collections').to route_to(controller: 'hyrax/my/collections', action: 'index')
+    end
+
+    it "routes to my highlighted tab" do
+      expect(get: '/dashboard/highlights').to route_to(controller: 'hyrax/my/highlights', action: 'index')
+    end
+
+    it "routes to my shared tab" do
+      expect(get: '/dashboard/shares').to route_to(controller: 'hyrax/my/shares', action: 'index')
+    end
+  end
+
+  describe 'Trophies' do
+    it 'routes to user trophies' do
+      expect(post: '/works/1234abc/trophy').to route_to(controller: 'hyrax/trophies', action: 'toggle_trophy', id: '1234abc')
+    end
+  end
+
+  describe 'Users' do
+    it 'routes to user profile' do
+      expect(get: '/users/bob135').to route_to(controller: 'hyrax/users', action: 'show', id: 'bob135')
+    end
+  end
+
+  describe 'Profile' do
+    it "routes to edit profile" do
+      expect(get: '/dashboard/profiles/bob135/edit').to route_to(controller: 'hyrax/dashboard/profiles', action: 'edit', id: 'bob135')
+    end
+
+    it "routes to update profile" do
+      expect(put: '/dashboard/profiles/bob135').to route_to(controller: 'hyrax/dashboard/profiles', action: 'update', id: 'bob135')
+    end
+  end
+
+  describe "Notifications" do
+    it "has index" do
+      expect(get: '/notifications').to route_to(controller: 'hyrax/notifications', action: 'index')
+      expect(notifications_path).to eq '/notifications'
+    end
+    it "allows deleting" do
+      expect(delete: '/notifications/123').to route_to(controller: 'hyrax/notifications', action: 'destroy', id: '123')
+      expect(notification_path(123)).to eq '/notifications/123'
+    end
+    it "allows deleting all of them" do
+      expect(delete: '/notifications/delete_all').to route_to(controller: 'hyrax/notifications', action: 'delete_all')
+      expect(delete_all_notifications_path).to eq '/notifications/delete_all'
+    end
+  end
+
+  describe "Contact Form" do
+    it "routes to new" do
+      expect(get: '/contact').to route_to(controller: 'hyrax/contact_form', action: 'new')
+    end
+
+    it "routes to create" do
+      expect(post: '/contact').to route_to(controller: 'hyrax/contact_form', action: 'create')
+    end
+  end
+
+  describe 'Content Blocks' do
+    it 'routes to update' do
+      expect(patch: '/content_blocks/1').to route_to(controller: 'hyrax/content_blocks', action: 'update', id: '1')
+    end
+    it 'routes to edit' do
+      expect(get: '/content_blocks/edit').to route_to(controller: 'hyrax/content_blocks', action: 'edit')
+    end
+  end
+
+  describe "Dynamically edited pages" do
+    it "routes to about" do
+      expect(get: '/about').to route_to(controller: 'hyrax/pages', action: 'show', key: 'about')
+    end
+    it "routes to help" do
+      expect(get: '/help').to route_to(controller: 'hyrax/pages', action: 'show', key: 'help')
+    end
+    it "routes to terms" do
+      expect(get: '/terms').to route_to(controller: 'hyrax/pages', action: 'show', key: 'terms')
+    end
+    it "routes to agreement" do
+      expect(get: '/agreement').to route_to(controller: 'hyrax/pages', action: 'show', key: 'agreement')
+    end
+    it 'routes to update' do
+      expect(patch: '/pages/foo').to route_to(controller: 'hyrax/pages', action: 'update', id: 'foo')
+    end
+    it 'routes to edit' do
+      expect(get: '/pages/edit').to route_to(controller: 'hyrax/pages', action: 'edit')
+    end
+  end
+
+  describe "Static Pages" do
+    it "routes to zotero" do
+      expect(get: '/zotero').to route_to(controller: 'hyrax/static', action: 'zotero')
+    end
+
+    it "routes to mendeley" do
+      expect(get: '/mendeley').to route_to(controller: 'hyrax/static', action: 'mendeley')
+    end
+  end
+
+  describe 'Collections' do
+    it 'routes to files' do
+      expect(get: '/collections/6/files').to route_to(controller: 'hyrax/collections', action: 'files', id: '6')
+    end
+  end
+
+  describe 'main app routes' do
+    routes { Rails.application.routes }
+
+    describe 'GenericWork' do
+      it "routes to show" do
+        expect(get: '/concern/generic_works/4').to route_to(controller: 'hyrax/generic_works', action: 'show', id: '4')
+      end
+
+      it 'routes to inspect_work' do
+        expect(get: 'concern/generic_works/6/inspect_work').to route_to(controller: 'hyrax/generic_works', action: 'inspect_work', id: '6')
+      end
+
+      it 'routes to manifest' do
+        expect(get: 'concern/generic_works/6/manifest').to route_to(controller: 'hyrax/generic_works', action: 'manifest', id: '6')
+      end
+    end
+  end
+end