hyrax 2.6.0 → 2.9.0

data/spec/presenters/hyrax/file_usage_spec.rb

@@ -1,162 +0,0 @@
-RSpec.describe Hyrax::FileUsage, type: :model do
-  let(:user) { build(:user) }
-  let(:file) do
-    create(:file_set, user: user)
-  end
-
-  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
-
-  let(:download_output) do
-    [[statistic_date(dates[0]), 1], [statistic_date(dates[1]), 1], [statistic_date(dates[2]), 2], [statistic_date(dates[3]), 3], [statistic_date(dates[4]), 5]]
-  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_download_statistics) do
-    [
-      SpecStatistic.new(eventCategory: "Files", eventAction: "Downloaded", eventLabel: "hyrax:x920fw85p", date: date_strs[0], totalEvents: "1"),
-      SpecStatistic.new(eventCategory: "Files", eventAction: "Downloaded", eventLabel: "hyrax:x920fw85p", date: date_strs[1], totalEvents: "1"),
-      SpecStatistic.new(eventCategory: "Files", eventAction: "Downloaded", eventLabel: "hyrax:x920fw85p", date: date_strs[2], totalEvents: "2"),
-      SpecStatistic.new(eventCategory: "Files", eventAction: "Downloaded", eventLabel: "hyrax:x920fw85p", date: date_strs[3], totalEvents: "3"),
-      SpecStatistic.new(eventCategory: "Files", eventAction: "Downloaded", eventLabel: "hyrax:x920fw85p", date: date_strs[4], totalEvents: "5")
-    ]
-  end
-
-  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(FileSet).to receive(:create_date).and_return((Time.zone.today - 4.days).to_s)
-    allow(FileDownloadStat).to receive(:ga_statistics).and_return(sample_download_statistics)
-    allow(FileViewStat).to receive(:ga_statistics).and_return(sample_pageview_statistics)
-    described_class.new(file.id)
-  end
-
-  describe "#initialize" do
-    it "sets the model" do
-      expect(usage.model).to eq file
-    end
-  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[1][:label]).to eq("Downloads")
-      expect(flots[0][:data]).to include(*view_output)
-      expect(flots[1][:data]).to include(*download_output)
-    end
-  end
-
-  describe "#created" do
-    let!(:system_timezone) { ENV['TZ'] }
-
-    before do
-      ENV['TZ'] = 'UTC'
-    end
-
-    after do
-      ENV['TZ'] = system_timezone
-    end
-
-    it "sets the created date" do
-      expect(usage.created).to eq(file.create_date)
-    end
-
-    it "counts the total numver of downloads" do
-      expect(usage.total_downloads).to eq(12)
-    end
-
-    it "counts the total numver of pageviews" do
-      expect(usage.total_pageviews).to eq(30)
-    end
-
-    context "when the analytics start date is set" do
-      let(:earliest) { DateTime.new(2014, 1, 2).iso8601 }
-      let(:create_date) { DateTime.new(2014, 1, 1).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(FileSet).to receive(:create_date).and_return(create_date.to_s)
-          described_class.new(file.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(FileSet).to receive(:create_date).and_return((Time.zone.today - 4.days).to_s)
-          Hyrax.config.analytic_start_date = earliest
-          described_class.new(file.id)
-        end
-
-        it "sets the created date to the earliest date not the created date" do
-          expect(usage.created).to eq(file.create_date)
-        end
-      end
-    end
-
-    context "when the start date is not set" do
-      before do
-        Hyrax.config.analytic_start_date = nil
-      end
-      let(:create_date) { DateTime.new(2014, 1, 1).iso8601 }
-
-      let(:usage) do
-        allow_any_instance_of(FileSet).to receive(:create_date).and_return(create_date.to_s)
-        described_class.new(file.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 file" do
-    let(:date_uploaded) { "2014-12-31" }
-    let(:file_migrated) do
-      create(:file_set, date_uploaded: date_uploaded, user: user)
-    end
-
-    let(:usage) do
-      described_class.new(file_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/fixity_status_presenter_spec.rb

@@ -1,55 +0,0 @@
-RSpec.describe Hyrax::FixityStatusPresenter do
-  let(:file_set_id) { "xw42n7888" }
-  let(:file_ids) do
-    [
-      "#{file_set_id}/files/3ec4c460-db36-49d4-914c-2d2036b0bfc6",
-      "#{file_set_id}/files/c3e8d90c-e47b-4c3b-9cff-50d20b5b0583"
-    ]
-  end
-  let(:presenter) { described_class.new(file_set_id) }
-
-  describe "#render_file_set_status" do
-    describe "no logs recorded" do
-      it "returns message" do
-        expect(presenter.render_file_set_status).to eq "Fixity checks have not yet been run on this object"
-      end
-    end
-
-    describe "success" do
-      before do
-        # Create some ChecksumAuditLots representing multiple files with multiple
-        # versions on a single fileset. BEWARE DANGER that this starts diverging from what
-        # the FileSetFixityCheckService actually creates, as specs have before.
-        file_ids.each do |file_id|
-          ChecksumAuditLog.create!(passed: true, file_set_id: file_set_id, file_id: file_id, checked_uri: "#{file_id}/fcr:versions/version1", created_at: 2.days.ago)
-          ChecksumAuditLog.create!(passed: true, file_set_id: file_set_id, file_id: file_id, checked_uri: "#{file_id}/fcr:versions/version2", created_at: 1.day.ago)
-        end
-      end
-      it "creates success message with details" do
-        result = presenter.render_file_set_status
-        expect(result).to be_html_safe
-        expect(result).to include("<span class=\"label label-success\">passed</span>")
-        expect(result).to match(/2 Files with 4 total versions checked between .* and .*/)
-      end
-    end
-    describe "failure" do
-      let(:file_id) { file_ids.second }
-      let(:failing_file_id) { file_ids.first }
-      let(:failing_checked_uri) { "#{failing_file_id}/fcr:versions/version1" }
-
-      before do
-        ChecksumAuditLog.create!(passed: true, file_set_id: file_set_id, file_id: file_id, checked_uri: "#{file_id}/fcr:versions/version1", created_at: 2.days.ago)
-        ChecksumAuditLog.create!(passed: true, file_set_id: file_set_id, file_id: file_id, checked_uri: "#{file_id}/fcr:versions/version2", created_at: 1.day.ago)
-        ChecksumAuditLog.create!(passed: false, file_set_id: file_set_id, file_id: failing_file_id, checked_uri: failing_checked_uri, created_at: Time.zone.now)
-      end
-      it "creates failure message with details" do
-        result = presenter.render_file_set_status
-        expect(result).to be_html_safe
-        expect(result).to include("<span class=\"label label-danger\">FAIL</span>")
-        expect(result).to match(/2 Files with 3 total versions checked between .* and .*/)
-        expect(result).to include failing_checked_uri
-        expect(result).to include failing_file_id
-      end
-    end
-  end
-end

data/spec/presenters/hyrax/homepage_presenter_spec.rb

@@ -1,42 +0,0 @@
-RSpec.describe Hyrax::HomepagePresenter do
-  let(:presenter) { described_class.new(ability, collections) }
-  let(:ability) { Ability.new(user) }
-  let(:collections) { double }
-  let(:user) { build(:user) }
-
-  describe "#collections" do
-    subject { presenter.collections }
-
-    it { is_expected.to eq collections }
-  end
-
-  describe "#display_share_button?" do
-    subject { presenter.display_share_button? }
-
-    context "when config is set to display_share_button_when_not_logged_in" do
-      context "and the user is registered" do
-        before do
-          allow(user).to receive(:new_record?).and_return(false)
-          allow(Hyrax.config).to receive(:display_share_button_when_not_logged_in?).and_return(true)
-          allow(ability).to receive(:can_create_any_work?).and_return(false)
-        end
-        it { is_expected.to be false }
-      end
-
-      context "and the user is a guest" do
-        before do
-          allow(Hyrax.config).to receive(:display_share_button_when_not_logged_in?).and_return(true)
-        end
-        it { is_expected.to be true }
-      end
-    end
-
-    context "when config is not set to display_share_button_when_not_logged_in" do
-      before do
-        allow(Hyrax.config).to receive(:display_share_button_when_not_logged_in?).and_return(false)
-        allow(ability).to receive(:can_create_any_work?).and_return(false)
-      end
-      it { is_expected.to be false }
-    end
-  end
-end

data/spec/presenters/hyrax/inspect_work_presenter_spec.rb

@@ -1,59 +0,0 @@
-RSpec.describe Hyrax::InspectWorkPresenter 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 "#workflow" do
-    let(:comments) do
-      { comment: 'comment', created_at: 'unknown' }
-    end
-    let(:roles) do
-      { id: '1', name: 'reviewing', users: ['user1', 'user2'] }
-    end
-
-    before do
-      allow(entity).to receive(:id).and_return('1')
-      allow(entity).to receive(:workflow_name).and_return('generic_workflow')
-      allow(entity).to receive(:workflow_id).and_return('1')
-      allow(entity).to receive(:proxy_for_global_id).and_return(attributes["id"])
-      allow(entity).to receive(:workflow_id).and_return('1')
-      allow(entity).to receive(:workflow_state_id).and_return('1')
-      allow(entity).to receive(:workflow_state_name).and_return('completed')
-      allow(presenter).to receive(:sipity_entity).and_return(entity)
-      allow(presenter).to receive(:workflow_comments).and_return(comments)
-      allow(presenter).to receive(:sipity_entity_roles).and_return(roles)
-    end
-
-    context "when a valid sipity_entity with workflow exists" do
-      subject { presenter.workflow }
-
-      it 'returns a hash of workflow related values for ispection' do
-        expect(subject[:entity_id]).to eq '1'
-        expect(subject[:workflow_name]).to eq 'generic_workflow'
-        expect(subject[:workflow_id]).to eq '1'
-        expect(subject[:proxy_for]).to eq attributes["id"]
-        expect(subject[:state_id]).to eq '1'
-        expect(subject[:state_name]).to eq 'completed'
-        expect(subject[:comments][:comment]).to eq 'comment'
-        expect(subject[:roles][:id]).to eq '1'
-        expect(subject[:roles][:name]).to eq 'reviewing'
-        expect(subject[:roles][:users][0]).to eq 'user1'
-      end
-    end
-
-    context "when no sipity_entity with workflow exists" do
-      let(:invalid) { described_class.new('no_solr_document', ability) }
-
-      it "raises PowerConverter::ConversionError" do
-        expect { invalid.workflow }.to raise_exception(PowerConverter::ConversionError)
-      end
-    end
-  end
-end

data/spec/presenters/hyrax/lease_presenter_spec.rb

@@ -1,51 +0,0 @@
-RSpec.describe Hyrax::LeasePresenter do
-  let(:document) { SolrDocument.new(attributes) }
-  let(:presenter) { described_class.new(document) }
-  let(:attributes) { {} }
-
-  describe "visibility" do
-    subject { presenter.visibility }
-
-    it { is_expected.to eq 'restricted' }
-  end
-
-  describe "to_s" do
-    let(:attributes) { { 'title_tesim' => ['Hey guys!'] } }
-
-    subject { presenter.to_s }
-
-    it { is_expected.to eq 'Hey guys!' }
-  end
-
-  describe "human_readable_type" do
-    let(:attributes) { { 'human_readable_type_tesim' => ['File'] } }
-
-    subject { presenter.human_readable_type }
-
-    it { is_expected.to eq 'File' }
-  end
-
-  describe "lease_expiration_date" do
-    let(:attributes) { { 'lease_expiration_date_dtsi' => '2015-10-15T00:00:00Z' } }
-
-    subject { presenter.lease_expiration_date }
-
-    it { is_expected.to eq '15 Oct 2015' }
-  end
-
-  describe "visibility_after_lease" do
-    let(:attributes) { { 'visibility_after_lease_ssim' => ['restricted'] } }
-
-    subject { presenter.visibility_after_lease }
-
-    it { is_expected.to eq 'restricted' }
-  end
-
-  describe "lease_history" do
-    let(:attributes) { { 'lease_history_ssim' => ['This is in the past'] } }
-
-    subject { presenter.lease_history }
-
-    it { is_expected.to eq ['This is in the past'] }
-  end
-end

data/spec/presenters/hyrax/member_presenter_factory_spec.rb

@@ -1,26 +0,0 @@
-RSpec.describe Hyrax::MemberPresenterFactory do
-  describe "#file_set_presenters" do
-    describe "getting presenters from factory" do
-      let(:solr_document) { SolrDocument.new(attributes) }
-      let(:attributes) { {} }
-      let(:ability) { double }
-      let(:request) { double }
-      let(:factory) { described_class.new(solr_document, ability, request) }
-      let(:presenter_class) { double }
-
-      before do
-        allow(factory).to receive(:composite_presenter_class).and_return(presenter_class)
-        allow(factory).to receive(:ordered_ids).and_return(['12', '33'])
-        allow(factory).to receive(:file_set_ids).and_return(['33', '12'])
-      end
-
-      it "uses the set class" do
-        expect(Hyrax::PresenterFactory).to receive(:build_for)
-          .with(ids: ['12', '33'],
-                presenter_class: presenter_class,
-                presenter_args: [ability, request])
-        factory.file_set_presenters
-      end
-    end
-  end
-end

data/spec/presenters/hyrax/menu_presenter_spec.rb

@@ -1,107 +0,0 @@
-RSpec.describe Hyrax::MenuPresenter do
-  let(:instance) { described_class.new(context) }
-  let(:context) { ActionView::TestCase::TestController.new.view_context }
-  let(:controller_name) { controller.controller_name }
-
-  describe "#settings_section?" do
-    before do
-      allow(context).to receive(:controller_name).and_return(controller_name)
-    end
-    subject { instance.settings_section? }
-
-    context "for the ContentBlocksController" do
-      let(:controller_name) { Hyrax::ContentBlocksController.controller_name }
-
-      it { is_expected.to be true }
-    end
-    context "for the PagesController" do
-      let(:controller_name) { Hyrax::PagesController.controller_name }
-
-      it { is_expected.to be true }
-    end
-    context "for the CollectionTypesController" do
-      let(:controller_name) { Hyrax::Admin::CollectionTypesController.controller_name }
-
-      it { is_expected.to be true }
-    end
-  end
-
-  describe "#collapsable_section" do
-    subject do
-      instance.collapsable_section('link title',
-                                   id: 'mySection',
-                                   icon_class: 'fa fa-cog',
-                                   open: open) do
-                                     "Some content"
-                                   end
-    end
-
-    let(:rendered) { Capybara::Node::Simple.new(subject) }
-
-    context "when collapsed" do
-      let(:open) { false }
-
-      it "draws a collapsable section" do
-        expect(rendered).to have_content "Some content"
-        expect(rendered).to have_selector "span.fa.fa-cog"
-        expect(rendered).to have_selector "a.collapsed.collapse-toggle[href='#mySection']"
-        expect(rendered).to have_selector "ul#mySection"
-      end
-    end
-
-    context "when open" do
-      let(:open) { true }
-
-      it "draws a collapsable section" do
-        expect(rendered).to have_content "Some content"
-        expect(rendered).to have_selector "span.fa.fa-cog"
-        expect(rendered).to have_selector "a.collapse-toggle[href='#mySection']"
-        expect(rendered).to have_selector "ul#mySection.in"
-      end
-    end
-  end
-
-  describe "#user_activity_section?" do
-    before do
-      allow(context).to receive(:controller_name).and_return(controller_name)
-      allow(context).to receive(:controller).and_return(controller)
-    end
-    subject { instance.user_activity_section? }
-
-    context "for the Hyrax::UsersController" do
-      let(:controller) { Hyrax::UsersController.new }
-
-      it { is_expected.to be true }
-    end
-
-    context "for the Admin::UsersController" do
-      let(:controller) { Hyrax::Admin::UsersController.new }
-
-      it { is_expected.to be false }
-    end
-
-    context "for the Hyrax::DepositorsController" do
-      let(:controller) { Hyrax::DepositorsController.new }
-
-      it { is_expected.to be true }
-    end
-  end
-
-  describe "#show_configuration?" do
-    subject { instance.show_configuration? }
-
-    context "for a regular user" do
-      before do
-        allow(instance.view_context).to receive(:can?).and_return(false)
-      end
-      it { is_expected.to be false }
-    end
-
-    context "for a user who can manage users" do
-      before do
-        allow(instance.view_context).to receive(:can?).and_return(true)
-      end
-      it { is_expected.to be true }
-    end
-  end
-end