hyrax 2.6.0 → 2.9.0

data/spec/views/hyrax/admin/users/index.html.erb_spec.rb

@@ -1,45 +0,0 @@
-RSpec.describe 'hyrax/admin/users/index.html.erb', type: :view do
-  let(:presenter) { Hyrax::Admin::UsersPresenter.new }
-  let(:users) { [] }
-
-  before do
-    (1..4).each { |i| users << build(:user, display_name: "user#{i}", email: "email#{i}@example.com", created_at: Time.zone.now - 3.days) }
-    allow(presenter).to receive(:users).and_return(users)
-    assign(:presenter, presenter)
-    allow(presenter).to receive(:show_last_access?).and_return(show_last_access)
-  end
-
-  let(:page) do
-    render
-    Capybara::Node::Simple.new(rendered)
-  end
-
-  context 'when show_last_access? is true' do
-    let(:show_last_access) { true }
-    before do
-      allow(presenter).to receive(:last_accessed).and_return(Time.zone.now - 3.days)
-    end
-
-    it 'draws user list with all users' do
-      expect(page).to have_content("Username")
-      expect(page).to have_content("Roles")
-      expect(page).to have_content("Last access")
-      (1..4).each do |i|
-        expect(page).to have_content("email#{i}@example.com")
-      end
-    end
-  end
-
-  context 'when show_last_access? is false' do
-    let(:show_last_access) { false }
-
-    it 'draws user list with all users' do
-      expect(page).to have_content("Username")
-      expect(page).to have_content("Roles")
-      expect(page).not_to have_content("Last access")
-      (1..4).each do |i|
-        expect(page).to have_content("email#{i}@example.com")
-      end
-    end
-  end
-end

data/spec/views/hyrax/admin/workflow_roles/index.html.erb_spec.rb

@@ -1,40 +0,0 @@
-RSpec.describe 'hyrax/admin/workflow_roles/index.html.erb', type: :view do
-  let!(:user1) { build(:user) }
-  let!(:user2) { build(:user) }
-  let(:agent_presenter) do
-    instance_double(Hyrax::Admin::WorkflowRolesPresenter::AgentPresenter,
-                    responsibilities_present?: responsibilities_exist)
-  end
-
-  let(:presenter) do
-    instance_double(Hyrax::Admin::WorkflowRolesPresenter,
-                    users: [user1, user2],
-                    presenter_for: agent_presenter)
-  end
-
-  before do
-    assign(:presenter, presenter)
-    allow(view).to receive(:admin_workflow_roles_path).and_return('/admin/workflow_roles')
-  end
-
-  context 'with no users having workflow roles' do
-    let(:responsibilities_exist) { false }
-
-    before { render }
-    it 'displays "No Roles" for each user' do
-      expect(rendered).to have_content('No roles', count: 2)
-    end
-  end
-
-  context 'with some users having workflow roles' do
-    let(:responsibilities_exist) { true }
-
-    before do
-      allow(agent_presenter).to receive(:responsibilities).and_return(["stuuf"])
-      render
-    end
-    it 'displays roles for each user' do
-      expect(rendered.match(/<ul>\s+<\/ul>/m)).to be nil
-    end
-  end
-end

data/spec/views/hyrax/base/_attribute_rows.html.erb_spec.rb

@@ -1,30 +0,0 @@
-RSpec.describe 'hyrax/base/_attribute_rows.html.erb', type: :view do
-  let(:url) { "http://example.com" }
-  let(:rights_statement_uri) { 'http://rightsstatements.org/vocab/InC/1.0/' }
-  let(:ability) { double }
-  let(:work) do
-    stub_model(GenericWork,
-               related_url: [url],
-               rights_statement: [rights_statement_uri])
-  end
-  let(:solr_document) do
-    SolrDocument.new(has_model_ssim: 'GenericWork',
-                     rights_statement_tesim: [rights_statement_uri],
-                     related_url_tesim: [url])
-  end
-  let(:presenter) { Hyrax::WorkShowPresenter.new(solr_document, ability) }
-
-  let(:page) do
-    render 'hyrax/base/attribute_rows', presenter: presenter
-    Capybara::Node::Simple.new(rendered)
-  end
-
-  it 'shows external link with icon for related url field' do
-    expect(page).to have_selector '.glyphicon-new-window'
-    expect(page).to have_link(url)
-  end
-
-  it 'shows rights statement with link to statement URL' do
-    expect(page).to have_link("In Copyright", href: rights_statement_uri)
-  end
-end

data/spec/views/hyrax/base/_attributes.html.erb_spec.rb

@@ -1,35 +0,0 @@
-RSpec.describe 'hyrax/base/_attributes.html.erb' do
-  let(:creator)     { 'Bilbo' }
-  let(:contributor) { 'Frodo' }
-  let(:subject)     { 'history' }
-  let(:description) { ['Lorem ipsum < lorem ipsum. http://my.link.com'] }
-
-  let(:solr_document) { SolrDocument.new(attributes) }
-  let(:attributes) do
-    {
-      Solrizer.solr_name('has_model', :symbol) => ["GenericWork"],
-      subject_tesim: subject,
-      contributor_tesim: contributor,
-      creator_tesim: creator,
-      description_tesim: description
-    }
-  end
-  let(:ability) { double(admin?: true) }
-  let(:presenter) do
-    Hyrax::WorkShowPresenter.new(solr_document, ability)
-  end
-  let(:doc) { Nokogiri::HTML(rendered) }
-
-  before do
-    allow(presenter).to receive(:member_of_collection_presenters).and_return([])
-    allow(view).to receive(:dom_class) { '' }
-
-    render 'hyrax/base/attributes', presenter: presenter
-  end
-
-  it 'has links to search for other objects with the same metadata' do
-    expect(rendered).to have_link(creator)
-    expect(rendered).to have_link(contributor)
-    expect(rendered).to have_link(subject)
-  end
-end

data/spec/views/hyrax/base/_citations.html.erb_spec.rb

@@ -1,43 +0,0 @@
-RSpec.describe 'hyrax/base/_citations.html.erb', type: :view do
-  let(:user) { create(:user) }
-  let(:object_profile) { ["{\"id\":\"999\"}"] }
-  let(:contributor) { ['Frodo'] }
-  let(:creator)     { ['Bilbo'] }
-  let(:solr_document) do
-    SolrDocument.new(
-      id: '999',
-      object_profile_ssm: object_profile,
-      has_model_ssim: ['GenericWork'],
-      human_readable_type_tesim: ['Generic Work'],
-      contributor_tesim: contributor,
-      creator_tesim: creator,
-      rights_tesim: ['http://creativecommons.org/licenses/by/3.0/us/']
-    )
-  end
-  let(:ability) { Ability.new(user) }
-  let(:presenter) do
-    Hyrax::WorkShowPresenter.new(solr_document, ability)
-  end
-  let(:page) { Capybara::Node::Simple.new(rendered) }
-
-  before do
-    Hyrax.config.citations = citations
-    allow(controller).to receive(:can?).with(:edit, presenter).and_return(false)
-    render 'hyrax/base/citations', presenter: presenter
-  end
-  context 'when enabled' do
-    let(:citations) { true }
-
-    it 'appears on page' do
-      expect(page).to have_selector('a#citations', count: 1)
-    end
-  end
-
-  context 'when disabled' do
-    let(:citations) { false }
-
-    it 'does not appear on page' do
-      expect(page).to have_no_selector('a#citations')
-    end
-  end
-end

data/spec/views/hyrax/base/_form.html.erb_spec.rb

@@ -1,80 +0,0 @@
-RSpec.describe 'hyrax/base/_form.html.erb', type: :view do
-  let(:work) do
-    stub_model(GenericWork, id: '456')
-  end
-  let(:ability) { double }
-
-  let(:form) do
-    Hyrax::GenericWorkForm.new(work, ability, controller)
-  end
-  let(:options_presenter) { double(select_options: []) }
-
-  before do
-    allow(Hyrax::AdminSetOptionsPresenter).to receive(:new).and_return(options_presenter)
-    stub_template('hyrax/base/_form_progress.html.erb' => 'Progress')
-    # TODO: stub_model is not stubbing new_record? correctly on ActiveFedora models.
-    allow(work).to receive(:new_record?).and_return(true)
-    allow(work).to receive(:member_ids).and_return([1, 2])
-    allow(view).to receive(:curation_concern).and_return(work)
-    allow(controller).to receive(:current_user).and_return(stub_model(User))
-    assign(:form, form)
-    allow(controller).to receive(:controller_name).and_return('batch_uploads')
-    allow(controller).to receive(:action_name).and_return('new')
-    allow(controller).to receive(:repository).and_return(Hyrax::GenericWorksController.new.repository)
-    allow(controller).to receive(:blacklight_config).and_return(Hyrax::GenericWorksController.new.blacklight_config)
-
-    allow(form).to receive(:permissions).and_return([])
-    allow(form).to receive(:visibility).and_return('public')
-    stub_template 'hyrax/base/_form_files.html.erb' => 'files'
-  end
-
-  context "for a new object" do
-    let(:work) { GenericWork.new }
-
-    context 'with batch_upload on' do
-      before do
-        allow(Flipflop).to receive(:batch_upload?).and_return(true)
-        render
-      end
-      it 'shows batch uploads' do
-        expect(rendered).to have_link('Batch upload')
-        expect(rendered).to have_selector("form[action='/concern/generic_works'][data-param-key='generic_work']")
-        # Draws the "Share" tab, with data for the javascript.
-        expect(rendered).to have_selector('#share[data-param-key="generic_work"]')
-      end
-    end
-
-    context 'with batch_upload off' do
-      before do
-        allow(Flipflop).to receive(:batch_upload?).and_return(false)
-        render
-      end
-      it 'hides batch uploads' do
-        expect(rendered).not_to have_link('Batch upload')
-      end
-    end
-  end
-
-  context "for a persisted object" do
-    let(:work) { stub_model(GenericWork, id: '456') }
-
-    before do
-      # Add an error to the work
-      work.errors.add :base, 'broken'
-      work.errors.add :visibility, 'visibility_error'
-      allow(form).to receive(:select_files).and_return([])
-      render
-    end
-
-    it "draws the page" do
-      expect(rendered).to have_selector("form[action='/concern/generic_works/456']")
-      expect(rendered).to have_selector("select#generic_work_resource_type", count: 1)
-      expect(rendered).to have_selector("select#generic_work_thumbnail_id", count: 1)
-      expect(rendered).to have_selector("select#generic_work_representative_id", count: 1)
-
-      # It diplays form errors
-      expect(rendered).to have_content("broken")
-      expect(rendered).to have_content("visibility_error")
-    end
-  end
-end

data/spec/views/hyrax/base/_form_child_work_relationships.html.erb_spec.rb

@@ -1,68 +0,0 @@
-RSpec.describe "hyrax/base/_form_child_work_relationships.html.erb", type: :view do
-  let(:work) do
-    stub_model(GenericWork, id: '456', title: ["MyWork"])
-  end
-
-  let(:ability) { double }
-
-  let(:form) do
-    Hyrax::GenericWorkForm.new(work, ability, controller)
-  end
-
-  let(:f) do
-    view.simple_form_for(form, url: '/update') do |work_form|
-      return work_form
-    end
-  end
-
-  before do
-    # TODO: stub_model is not stubbing new_record? correctly on ActiveFedora models.
-    allow(work).to receive(:new_record?).and_return(false)
-
-    allow(view).to receive(:params).and_return(id: work.id)
-    allow(view).to receive(:curation_concern).and_return(work)
-    allow(view).to receive(:f).and_return(f)
-    allow(f).to receive(:object).and_return(form)
-    allow(controller).to receive(:current_user).and_return(stub_model(User))
-    assign(:form, form)
-  end
-
-  context "When editing a work" do
-    context "and no children works are present" do
-      before do
-        allow(form).to receive(:work_members).and_return([])
-        render
-      end
-
-      it "draws the page" do
-        # remove button is not present
-        expect(rendered).not_to have_selector("[data-behavior='remove-relationship']")
-
-        # input with add button
-        expect(rendered).to have_selector('input[data-autocomplete-url="/authorities/search/find_works"]')
-        expect(rendered).to have_selector("[data-behavior='add-relationship']")
-      end
-    end
-
-    context "and child works are present" do
-      before do
-        allow(form).to receive(:work_members_json).and_return('stub-data')
-        render
-      end
-
-      it "draws the page" do
-        # input with add button
-        expect(rendered).to have_selector('input[data-autocomplete-url="/authorities/search/find_works"]')
-        expect(rendered).to have_selector("[data-behavior='add-relationship']")
-
-        # generate the json to drive the script
-        expect(rendered).to have_selector('div[data-members="stub-data"]')
-
-        # a remove button
-        within "tr" do
-          expect(rendered).to have_selector("[data-behavior='remove-relationship']")
-        end
-      end
-    end
-  end
-end

data/spec/views/hyrax/base/_form_files.html.erb_spec.rb

@@ -1,38 +0,0 @@
-RSpec.describe 'hyrax/base/_form_files.html.erb', type: :view do
-  let(:model) { stub_model(GenericWork) }
-  let(:form) { Hyrax::GenericWorkForm.new(model, double, controller) }
-  let(:f) { double(object: form) }
-
-  before do
-    stub_template 'hyrax/uploads/_js_templates.html.erb' => 'templates'
-    # TODO: stub_model is not stubbing new_record? correctly on ActiveFedora models.
-    allow(model).to receive(:new_record?).and_return(false)
-  end
-
-  context "without browse_everything" do
-    before do
-      allow(Hyrax.config).to receive(:browse_everything?).and_return(false)
-      render 'hyrax/base/form_files', f: f
-    end
-
-    it 'shows a message and buttons' do
-      expect(rendered).to have_content 'You can add one or more files to associate with this work.'
-      expect(rendered).to have_content('Add folder...')
-
-      expect(rendered).not_to have_content 'cloud provider'
-      expect(rendered).not_to have_selector('button#browse-btn')
-    end
-  end
-
-  context "with browse_everything" do
-    before do
-      allow(Hyrax.config).to receive(:browse_everything?).and_return(true)
-      render 'hyrax/base/form_files', f: f
-    end
-
-    it 'shows user timing warning' do
-      expect(rendered).to have_content 'Note that if you use a cloud provider to upload a large number of files'
-      expect(rendered).to have_selector("button[id='browse-btn'][data-target='#edit_generic_work_#{form.model.id}']")
-    end
-  end
-end

data/spec/views/hyrax/base/_form_metadata.html.erb_spec.rb

@@ -1,40 +0,0 @@
-RSpec.describe 'hyrax/base/_form_metadata.html.erb', type: :view do
-  before do
-    allow(form).to receive(:display_additional_fields?).and_return(additional_fields)
-  end
-  let(:ability) { double }
-  let(:work) { GenericWork.new }
-  let(:form) do
-    Hyrax::GenericWorkForm.new(work, ability, controller)
-  end
-
-  let(:form_template) do
-    %(
-      <%= simple_form_for [main_app, @form] do |f| %>
-        <%= render "hyrax/base/form_metadata", f: f %>
-      <% end %>
-     )
-  end
-
-  let(:page) do
-    assign(:form, form)
-    render inline: form_template
-    Capybara::Node::Simple.new(rendered)
-  end
-
-  context 'with secondary terms' do
-    let(:additional_fields) { true }
-
-    it "renders the additional fields button" do
-      expect(page).to have_content('Additional fields')
-    end
-  end
-
-  context 'without secondary terms' do
-    let(:additional_fields) { false }
-
-    it 'does not render the addtional fields button' do
-      expect(page).not_to have_content('Additional fields')
-    end
-  end
-end

data/spec/views/hyrax/base/_form_progress.html.erb_spec.rb

@@ -1,131 +0,0 @@
-RSpec.describe 'hyrax/base/_form_progress.html.erb', type: :view do
-  let(:ability) { double }
-  let(:user) { stub_model(User) }
-  let(:form) do
-    Hyrax::GenericWorkForm.new(work, ability, controller)
-  end
-  let(:page) do
-    view.simple_form_for form do |f|
-      render 'hyrax/base/form_progress', f: f
-    end
-    Capybara::Node::Simple.new(rendered)
-  end
-
-  before do
-    allow(controller).to receive(:current_user).and_return(user)
-    # Stub visibility, or it will hit fedora
-    allow(work).to receive(:visibility).and_return('open')
-  end
-
-  context "for a new object" do
-    before { assign(:form, form) }
-
-    let(:work) { GenericWork.new }
-
-    context "with options for proxy" do
-      let(:proxies) { [stub_model(User, email: 'bob@example.com')] }
-
-      before do
-        allow(Flipflop).to receive(:proxy_deposit?).and_return(true)
-        allow(user).to receive(:can_make_deposits_for).and_return(proxies)
-      end
-
-      it "shows options for proxy" do
-        expect(page).to have_content 'On behalf of'
-        expect(page).to have_selector("select#generic_work_on_behalf_of option[value=\"\"]", text: 'Yourself')
-        expect(page).to have_selector("select#generic_work_on_behalf_of option[value=\"bob@example.com\"]")
-      end
-
-      context 'when feature disabled' do
-        before do
-          allow(Flipflop).to receive(:proxy_deposit?).and_return(false)
-        end
-
-        it "does not show options for proxy" do
-          expect(page).not_to have_content 'On behalf of'
-          expect(page).not_to have_selector("select#generic_work_on_behalf_of option[value=\"\"]", text: 'Yourself')
-          expect(page).not_to have_selector("select#generic_work_on_behalf_of option[value=\"bob@example.com\"]")
-        end
-      end
-    end
-
-    context "without options for proxy" do
-      let(:proxies) { [] }
-
-      before do
-        allow(user).to receive(:can_make_deposits_for).and_return(proxies)
-      end
-      it "doesn't show options for proxy" do
-        expect(page).not_to have_content 'On behalf of'
-        expect(page).not_to have_selector 'select#generic_work_on_behalf_of'
-      end
-    end
-
-    context "with active deposit agreement" do
-      before do
-        allow(Flipflop).to receive(:active_deposit_agreement_acceptance?)
-          .and_return(true)
-      end
-      it "shows accept text" do
-        expect(page).to have_content 'Check deposit agreement'
-        expect(page).to have_content 'I have read and agree to the'
-        expect(page).to have_link 'Deposit Agreement', href: '/agreement'
-        expect(page).not_to have_selector("#agreement[checked]")
-      end
-    end
-
-    context "with passive deposit agreement" do
-      before do
-        allow(Flipflop).to receive(:active_deposit_agreement_acceptance?)
-          .and_return(false)
-      end
-      it "shows accept text" do
-        expect(page).not_to have_content 'Check deposit agreement'
-        expect(page).to have_content 'By saving this work I agree to the'
-        expect(page).to have_link 'Deposit Agreement', href: '/agreement'
-      end
-    end
-
-    context "with no deposit agreement" do
-      before do
-        allow(Flipflop).to receive(:show_deposit_agreement?).and_return(false)
-      end
-      it "does not display active accept text" do
-        expect(page).not_to have_content 'I have read and agree to the'
-        expect(page).not_to have_selector("#agreement[checked]")
-      end
-      it "does not display passive accept text" do
-        expect(page).not_to have_content 'By saving this work I agree to the'
-        expect(page).not_to have_link 'Deposit Agreement', href: '/agreement'
-      end
-    end
-
-    context "with active deposit acceptance but no show deposit agreement" do
-      before do
-        allow(Flipflop).to receive(:show_deposit_agreement?).and_return(false)
-        allow(Flipflop).to receive(:active_deposit_agreement_acceptance?)
-          .and_return(true)
-      end
-      it "does not display the deposit agreement in the requirements" do
-        expect(page).not_to have_selector("#required-agreement")
-      end
-    end
-  end
-
-  context "when the work has been saved before" do
-    before do
-      # TODO: stub_model is not stubbing new_record? correctly on ActiveFedora models.
-      allow(work).to receive(:new_record?).and_return(false)
-      assign(:form, form)
-      allow(Hyrax.config).to receive(:active_deposit_agreement_acceptance)
-        .and_return(true)
-    end
-
-    let(:work) { stub_model(GenericWork, id: '456', etag: '123456') }
-
-    it "renders the deposit agreement already checked and the version" do
-      expect(page).to have_selector("#agreement[checked]")
-      expect(page).to have_selector("input#generic_work_version[value=\"123456\"]", visible: false)
-    end
-  end
-end