hyrax 2.6.0 → 2.7.0

data/spec/presenters/hyrax/admin/dashboard_presenter_spec.rb

@@ -1,33 +0,0 @@
-RSpec.describe Hyrax::Admin::DashboardPresenter do
-  let(:instance) { described_class.new }
-
-  describe "#user_count" do
-    before do
-      create(:user)
-      create(:user)
-      create(:user, :guest)
-    end
-
-    subject { instance.user_count }
-
-    it { is_expected.to eq 2 }
-  end
-
-  describe "#repository_objects" do
-    subject { instance.repository_objects }
-
-    it { is_expected.to be_kind_of Hyrax::Admin::RepositoryObjectPresenter }
-  end
-
-  describe "#repository_growth" do
-    subject { instance.repository_growth }
-
-    it { is_expected.to be_kind_of Hyrax::Admin::RepositoryGrowthPresenter }
-  end
-
-  describe "#user_activity" do
-    subject { instance.user_activity }
-
-    it { is_expected.to be_kind_of Hyrax::Admin::UserActivityPresenter }
-  end
-end

data/spec/presenters/hyrax/admin/repository_growth_presenter_spec.rb

@@ -1,25 +0,0 @@
-RSpec.describe Hyrax::Admin::RepositoryGrowthPresenter do
-  let(:instance) { described_class.new }
-
-  describe "#to_json" do
-    subject { instance.to_json }
-
-    let(:works) do
-      instance_double(Hyrax::Statistics::Works::OverTime,
-                      points: [['2017-02-16', '12']])
-    end
-    let(:collections) do
-      instance_double(Hyrax::Statistics::Collections::OverTime,
-                      points: [['2017-02-16', '3']])
-    end
-
-    before do
-      allow(Hyrax::Statistics::Works::OverTime).to receive(:new).and_return(works)
-      allow(Hyrax::Statistics::Collections::OverTime).to receive(:new).and_return(collections)
-    end
-
-    it "returns points" do
-      expect(subject).to eq "[{\"y\":\"2017-02-16\",\"a\":\"12\",\"b\":\"3\"}]"
-    end
-  end
-end

data/spec/presenters/hyrax/admin/repository_object_presenter_spec.rb

@@ -1,24 +0,0 @@
-RSpec.describe Hyrax::Admin::RepositoryObjectPresenter do
-  let(:instance) { described_class.new }
-
-  describe "#as_json" do
-    subject { instance.as_json }
-
-    let(:stub_repo) { double(search: response) }
-    let(:response) { Blacklight::Solr::Response.new(solr_data, {}) }
-    let(:solr_data) do
-      { "facet_counts" => {
-        "facet_fields" => { "suppressed_bsi" => ["false", 1, "true", 2, nil, 3] }
-      } }
-    end
-
-    before do
-      allow(instance).to receive(:repository).and_return(stub_repo)
-    end
-    it do
-      is_expected.to eq [{ label: "Published", value: 1 },
-                         { label: "Unpublished", value: 2 },
-                         { label: "Unknown", value: 3 }]
-    end
-  end
-end

data/spec/presenters/hyrax/admin/user_activity_presenter_spec.rb

@@ -1,20 +0,0 @@
-RSpec.describe Hyrax::Admin::UserActivityPresenter do
-  let(:instance) { described_class.new }
-
-  describe "#to_json" do
-    subject { instance.to_json }
-
-    let(:users) do
-      instance_double(Hyrax::Statistics::Users::OverTime,
-                      points: [['2017-02-16', '12']])
-    end
-
-    before do
-      allow(Hyrax::Statistics::Users::OverTime).to receive(:new).and_return(users)
-    end
-
-    it "returns points" do
-      expect(subject).to eq "[{\"y\":\"2017-02-16\",\"a\":\"12\",\"b\":null}]"
-    end
-  end
-end

data/spec/presenters/hyrax/admin/users_presenter_spec.rb

@@ -1,48 +0,0 @@
-RSpec.describe Hyrax::Admin::UsersPresenter do
-  let(:instance) { described_class.new }
-  let!(:user) { create(:user) }
-  let!(:admin_user) { create(:user, groups: 'admin') }
-  let!(:audit_user) { User.audit_user }
-  let!(:batch_user) { User.batch_user }
-
-  describe "#users" do
-    it "includes all users except batch and audit users" do
-      expect(instance.users).to match_array [admin_user, user]
-    end
-  end
-
-  describe "#user_count" do
-    it "counts users excluding batch_user and audit_user" do
-      expect(instance.user_count).to eq 2
-    end
-  end
-
-  describe "#user_roles" do
-    describe "for an admin user" do
-      it "finds the admin role" do
-        expect(instance.user_roles(admin_user)).to eq(['admin'])
-      end
-    end
-    describe "for a generic user with no user roles" do
-      it "returns blank" do
-        expect(instance.user_roles(user)).to eq([])
-      end
-    end
-  end
-
-  describe 'show_last_access?' do
-    before do
-      allow(User).to receive(:devise_modules).and_return(modules)
-    end
-    subject { instance.show_last_access? }
-    context 'when devise trackable is installed' do
-      let(:modules) { [:trackable] }
-      it { is_expected.to be true }
-    end
-
-    context 'when devise trackable is not installed' do
-      let(:modules) { [:rememberable] }
-      it { is_expected.to be false }
-    end
-  end
-end

data/spec/presenters/hyrax/admin/workflow_role_presenter_spec.rb

@@ -1,12 +0,0 @@
-RSpec.describe Hyrax::Admin::WorkflowRolePresenter do
-  let(:presenter) { described_class.new(workflow_role) }
-  let(:role) { Sipity::Role[:depositor] }
-  let(:workflow) { create(:workflow) }
-  let(:workflow_role) { Sipity::WorkflowRole.new(role: role, workflow: workflow) }
-
-  describe '#label' do
-    subject { presenter.label }
-
-    it { is_expected.to be_a(String) }
-  end
-end

data/spec/presenters/hyrax/admin/workflow_roles_presenter_spec.rb

@@ -1,16 +0,0 @@
-RSpec.describe Hyrax::Admin::WorkflowRolesPresenter do
-  let(:presenter) { described_class.new }
-
-  describe "#users" do
-    subject { presenter.users }
-
-    let!(:user) { create(:user) }
-
-    before do
-      create(:user, :guest)
-    end
-    it "doesn't include guests" do
-      expect(subject).to eq [user]
-    end
-  end
-end

data/spec/presenters/hyrax/admin_set_options_presenter_spec.rb

@@ -1,73 +0,0 @@
-RSpec.describe Hyrax::AdminSetOptionsPresenter do
-  let(:service) { instance_double(Hyrax::AdminSetService) }
-  let(:presenter) { described_class.new(service) }
-
-  describe "#select_options" do
-    subject { presenter.select_options }
-
-    before do
-      allow(service).to receive(:search_results)
-        .with(:deposit)
-        .and_return(results)
-    end
-
-    context "with permission_template visibility" do
-      let(:solr_doc1) { instance_double(SolrDocument, id: '123', to_s: 'Public Set') }
-      let(:solr_doc2) { instance_double(SolrDocument, id: '345', to_s: 'Private Set') }
-      let(:results) { [solr_doc1, solr_doc2] }
-
-      before do
-        allow(presenter).to receive(:workflow) { nil }
-        create(:permission_template, source_id: '123', visibility: 'open')
-        create(:permission_template, source_id: '345', visibility: 'restricted')
-      end
-
-      it do
-        is_expected.to eq [['Public Set', '123', { 'data-sharing' => false, 'data-visibility' => 'open' }],
-                           ['Private Set', '345', { 'data-sharing' => false, 'data-visibility' => 'restricted' }]]
-      end
-    end
-
-    context "with permission_template release_date" do
-      let(:today) { Time.zone.today }
-      let(:solr_doc1) { instance_double(SolrDocument, id: '123', to_s: 'Fixed Release Date Set') }
-      let(:solr_doc2) { instance_double(SolrDocument, id: '345', to_s: 'No Delay Set') }
-      let(:solr_doc3) { instance_double(SolrDocument, id: '567', to_s: 'One Year Max Embargo Set') }
-      let(:solr_doc4) { instance_double(SolrDocument, id: '789', to_s: 'Release Before Date Set') }
-      let(:results) { [solr_doc1, solr_doc2, solr_doc3, solr_doc4] }
-
-      before do
-        allow(presenter).to receive(:workflow) { nil }
-        create(:permission_template, source_id: '123', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED, release_date: today + 2.days)
-        create(:permission_template, source_id: '345', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY)
-        create(:permission_template, source_id: '567', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR)
-        create(:permission_template, source_id: '789', release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE, release_date: today + 1.month)
-      end
-
-      it do
-        is_expected.to eq [['Fixed Release Date Set', '123', { 'data-sharing' => false, 'data-release-date' => today + 2.days }],
-                           ['No Delay Set', '345', { 'data-sharing' => false, 'data-release-no-delay' => true }],
-                           ['One Year Max Embargo Set', '567', { 'data-sharing' => false, 'data-release-date' => today + 1.year, 'data-release-before-date' => true }],
-                           ['Release Before Date Set', '789', { 'data-sharing' => false, 'data-release-date' => today + 1.month, 'data-release-before-date' => true }]]
-      end
-    end
-
-    context "with empty permission_template" do
-      let(:solr_doc1) { instance_double(SolrDocument, id: '123', to_s: 'Empty Template Set') }
-      let(:results) { [solr_doc1] }
-
-      before do
-        create(:permission_template, source_id: '567')
-      end
-
-      it { is_expected.to eq [['Empty Template Set', '123', {}]] }
-    end
-
-    context "with no permission_template" do
-      let(:solr_doc1) { instance_double(SolrDocument, id: '123', to_s: 'No Template Set') }
-      let(:results) { [solr_doc1] }
-
-      it { is_expected.to eq [['No Template Set', '123', {}]] }
-    end
-  end
-end

data/spec/presenters/hyrax/admin_set_presenter_spec.rb

@@ -1,143 +0,0 @@
-RSpec.describe Hyrax::AdminSetPresenter do
-  let(:admin_set) do
-    mock_model(AdminSet,
-               id: '123',
-               description: ['An example admin set.'],
-               title: ['Example Admin Set Title'])
-  end
-
-  let(:work) { build(:work, title: ['Example Work Title']) }
-  let(:solr_document) { SolrDocument.new(admin_set.to_solr) }
-  let(:ability) { double }
-  let(:presenter) { described_class.new(solr_document, ability) }
-
-  describe "total_items" do
-    subject { presenter.total_items }
-
-    let(:admin_set) { build(:admin_set) }
-
-    context "empty admin set" do
-      it { is_expected.to eq 0 }
-    end
-
-    context "admin set with work" do
-      let(:admin_set) { create(:admin_set, members: [work]) }
-
-      it { is_expected.to eq 1 }
-    end
-  end
-
-  describe "disable_delete?" do
-    subject { presenter.disable_delete? }
-
-    context "empty admin set" do
-      let(:admin_set) { create(:admin_set) }
-
-      it { is_expected.to be false }
-    end
-
-    context "non-empty admin set" do
-      let(:admin_set) { create(:admin_set, members: [work]) }
-
-      it { is_expected.to be true }
-    end
-
-    context "default admin set" do
-      let(:admin_set) do
-        build(:admin_set, id: AdminSet::DEFAULT_ID)
-      end
-
-      it { is_expected.to be true }
-    end
-  end
-
-  describe '#collection_type' do
-    let(:admin_set) do
-      build(:admin_set, id: AdminSet::DEFAULT_ID)
-    end
-
-    subject { presenter.collection_type }
-
-    it { is_expected.to eq(create(:admin_set_collection_type)) }
-  end
-
-  describe '#show_path' do
-    let(:admin_set) do
-      build(:admin_set, id: 'sb397824g')
-    end
-
-    subject { presenter.show_path }
-
-    it { is_expected.to eq "/admin/admin_sets/#{admin_set.id}?locale=en" }
-  end
-
-  describe '#managed_access' do
-    let(:admin_set) { create(:admin_set, members: [work]) }
-
-    context 'when manager' do
-      before do
-        allow(ability).to receive(:can?).with(:edit, solr_document).and_return(true)
-      end
-      it 'returns Manage label' do
-        expect(presenter.managed_access).to eq 'Manage'
-      end
-    end
-
-    context 'when depositor' do
-      before do
-        allow(ability).to receive(:can?).with(:edit, solr_document).and_return(false)
-        allow(ability).to receive(:can?).with(:deposit, solr_document).and_return(true)
-      end
-      it 'returns Deposit label' do
-        expect(presenter.managed_access).to eq 'Deposit'
-      end
-    end
-
-    context 'when viewer' do
-      before do
-        allow(ability).to receive(:can?).with(:edit, solr_document).and_return(false)
-        allow(ability).to receive(:can?).with(:deposit, solr_document).and_return(false)
-        allow(ability).to receive(:can?).with(:read, solr_document).and_return(true)
-      end
-      it 'returns View label' do
-        expect(presenter.managed_access).to eq 'View'
-      end
-    end
-  end
-
-  describe '#allow_batch?' do
-    let(:admin_set) { create(:admin_set, members: [work]) }
-
-    context 'when user cannot edit' do
-      before do
-        allow(ability).to receive(:can?).with(:edit, solr_document).and_return(false)
-      end
-
-      it 'returns false' do
-        expect(presenter.allow_batch?).to be false
-      end
-    end
-
-    context 'when user can edit' do
-      before do
-        allow(ability).to receive(:can?).with(:edit, solr_document).and_return(true)
-      end
-
-      context 'and there are works in the admin set' do
-        it 'returns false' do
-          expect(presenter.allow_batch?).to be false
-        end
-      end
-
-      context 'and there are no works in the admin set' do
-        before do
-          allow(presenter).to receive(:total_items).and_return(0)
-        end
-
-        it 'returns true' do
-          expect(presenter.allow_batch?).to be true
-        end
-      end
-    end
-  end
-end

data/spec/presenters/hyrax/admin_stats_presenter_spec.rb

@@ -1,143 +0,0 @@
-RSpec.describe Hyrax::AdminStatsPresenter do
-  let(:user1) { create(:user) }
-  let(:user2) { create(:user) }
-
-  let(:one_day_ago_date)  { Time.zone.now - 1.day }
-  let(:two_days_ago_date) { Time.zone.now - 2.days }
-  let(:start_date) { '' }
-  let(:end_date) { '' }
-
-  let(:filters) { { start_date: start_date, end_date: end_date } }
-  let(:limit) { 5 }
-  let(:service) { described_class.new(filters, limit) }
-
-  describe '#active_users' do
-    it 'delegates to Hyrax::Statistics::Works::ByDepositor.query' do
-      expect(Hyrax::Statistics::Works::ByDepositor).to receive(:query).with(limit: limit).and_return(:query_response)
-      expect(service.active_users).to eq(:query_response)
-    end
-
-    context 'with alternatate class' do
-      subject(:service) do
-        described_class.new(filters, limit, by_depositor: by_depositor_class)
-      end
-
-      let(:by_depositor_class) { spy('ByDepositor') }
-
-      it 'retrieves active_users from the class' do
-        service.active_users
-
-        expect(by_depositor_class).to have_received(:query).with(limit: limit)
-      end
-    end
-  end
-
-  describe "#top_formats" do
-    it 'delegates to Hyrax::Statistics::FileSets::ByFormat.query' do
-      expect(Hyrax::Statistics::FileSets::ByFormat).to receive(:query).with(limit: limit).and_return(:query_response)
-      expect(service.top_formats).to eq(:query_response)
-    end
-
-    context 'with alternatate class' do
-      subject(:service) do
-        described_class.new(filters, limit, by_format: by_format)
-      end
-
-      let(:by_format) { spy('ByFormat') }
-
-      it 'retrieves formats from the class' do
-        service.top_formats
-
-        expect(by_format).to have_received(:query).with(limit: limit)
-      end
-    end
-  end
-
-  describe "#works_count" do
-    it 'delegates to Hyrax::Statistics::Works::Count.by_permission' do
-      expect(Hyrax::Statistics::Works::Count).to receive(:by_permission).with(start_date: service.start_date, end_date: service.end_date).and_return(:query_response)
-      expect(service.works_count).to eq(:query_response)
-    end
-
-    context 'with alternatate class' do
-      subject(:service)   { described_class.new(filters, limit, works_counter: works_counter) }
-      let(:works_counter) { spy('Works::Count') }
-
-      it 'retrieves count from the class' do
-        service.works_count
-
-        expect(works_counter)
-          .to have_received(:by_permission)
-          .with(start_date: service.start_date, end_date: service.end_date)
-      end
-    end
-  end
-
-  describe "#depositors" do
-    it 'delegates to Hyrax::Statistics::Depositors::Summary.depositors' do
-      expect(Hyrax::Statistics::Depositors::Summary).to receive(:depositors).with(start_date: service.start_date, end_date: service.end_date).and_return(:query_response)
-      expect(service.depositors).to eq(:query_response)
-    end
-
-    context 'with alternatate class' do
-      subject(:service) do
-        described_class.new(filters, limit, depositor_summary: summary_class)
-      end
-
-      let(:summary_class) { spy('depositor summary class') }
-
-      it 'retrieves depositors from the class' do
-        service.depositors
-
-        expect(summary_class)
-          .to have_received(:depositors)
-          .with(start_date: service.start_date, end_date: service.end_date)
-      end
-    end
-  end
-
-  describe "#recent_users" do
-    it 'delegates to Hyrax::Statistics::SystemStats.recent_users' do
-      expect(Hyrax::Statistics::SystemStats).to receive(:recent_users).with(limit: limit, start_date: service.start_date, end_date: service.end_date).and_return(:query_response)
-      expect(service.recent_users).to eq(:query_response)
-    end
-
-    context 'with alternatate class' do
-      subject(:service) do
-        described_class.new(filters, limit, system_stats: system_stats)
-      end
-
-      let(:system_stats) { spy('SystemStats') }
-
-      it 'retrieves users from the class' do
-        service.recent_users
-
-        expect(system_stats)
-          .to have_received(:recent_users)
-          .with(limit: limit, start_date: service.start_date, end_date: service.end_date)
-      end
-    end
-  end
-
-  describe '#date_filter_string' do
-    subject { service.date_filter_string }
-
-    context "default range" do
-      it { is_expected.to eq 'unfiltered' }
-    end
-
-    context "with a start and no end date" do
-      let(:start_date) { '2015-12-14' }
-      let(:today) { Time.zone.today.to_date.to_s(:standard) }
-
-      it { is_expected.to eq "12/14/2015 to #{today}" }
-    end
-
-    context 'with start and end dates' do
-      let(:start_date) { '2015-12-14' }
-      let(:end_date) { '2016-05-12' }
-
-      it { is_expected.to eq '12/14/2015 to 05/12/2016' }
-    end
-  end
-end