hyrax 2.6.0 → 2.9.0

data/spec/forms/hyrax/forms/workflow_responsibility_form_spec.rb

@@ -1,41 +0,0 @@
-RSpec.describe Hyrax::Forms::WorkflowResponsibilityForm do
-  let(:instance) { described_class.new }
-
-  describe "#initialize" do
-    let(:user) { create(:user) }
-    let(:instance) { described_class.new(user_id: user.id, workflow_role_id: 7) }
-
-    subject { instance.model_instance }
-
-    it "creates an agent and sets the workflow_role_id" do
-      expect(subject.agent).to be_kind_of Sipity::Agent
-      expect(subject.workflow_role_id).to eq 7
-    end
-  end
-
-  describe "#user_options" do
-    subject { instance.user_options }
-
-    it { is_expected.to eq User.all }
-  end
-
-  describe "#workflow_role_options" do
-    subject { instance.workflow_role_options }
-
-    let(:wf_role1) { instance_double(Sipity::WorkflowRole, id: 1) }
-    let(:wf_role2) { instance_double(Sipity::WorkflowRole, id: 2) }
-
-    before do
-      allow(Sipity::WorkflowRole).to receive(:all).and_return([wf_role1, wf_role2])
-      allow(Hyrax::Admin::WorkflowRolePresenter).to receive(:new)
-        .with(wf_role1)
-        .and_return(instance_double(Hyrax::Admin::WorkflowRolePresenter,
-                                    label: 'generic_work - foo'))
-      allow(Hyrax::Admin::WorkflowRolePresenter).to receive(:new)
-        .with(wf_role2)
-        .and_return(instance_double(Hyrax::Admin::WorkflowRolePresenter,
-                                    label: 'generic_work - bar'))
-    end
-    it { is_expected.to eq [['generic_work - bar', 2], ['generic_work - foo', 1]] }
-  end
-end

data/spec/forms/hyrax/generic_work_form_spec.rb

@@ -1,127 +0,0 @@
-# TODO: this should be merged with work_form_spec.rb
-RSpec.describe Hyrax::GenericWorkForm do
-  let(:work) { GenericWork.new }
-  let(:form) { described_class.new(work, nil, nil) }
-
-  describe "#required_fields" do
-    subject { form.required_fields }
-
-    it { is_expected.to eq [:title, :creator, :keyword, :rights_statement] }
-  end
-
-  describe "#primary_terms" do
-    subject { form.primary_terms }
-
-    it { is_expected.to eq [:title, :creator, :keyword, :rights_statement] }
-  end
-
-  describe "#secondary_terms" do
-    subject { form.secondary_terms }
-
-    it do
-      is_expected.not_to include(:title, :creator, :keyword,
-                                 :visibilty, :visibility_during_embargo,
-                                 :embargo_release_date, :visibility_after_embargo,
-                                 :visibility_during_lease, :lease_expiration_date,
-                                 :visibility_after_lease, :collection_ids)
-    end
-  end
-
-  describe "#[]" do
-    subject { form[term] }
-
-    context "for member_of_collection_ids" do
-      let(:term) { :member_of_collection_ids }
-
-      it { is_expected.to eq [] }
-
-      context "when the model has collection ids" do
-        before do
-          allow(work).to receive(:member_of_collection_ids).and_return(['col1', 'col2'])
-        end
-        # This allows the edit form to show collections the work is already a member of.
-        it { is_expected.to eq ['col1', 'col2'] }
-      end
-    end
-  end
-
-  describe '.model_attributes' do
-    let(:permission_template) { create(:permission_template, source_id: source_id) }
-    let!(:workflow) { create(:workflow, active: true, permission_template_id: permission_template.id) }
-    let(:source_id) { '123' }
-    let(:file_set) { create(:file_set) }
-    let(:params) do
-      ActionController::Parameters.new(
-        title: ['foo'],
-        description: [''],
-        visibility: 'open',
-        source_id: source_id,
-        representative_id: '456',
-        rendering_ids: [file_set.id],
-        thumbnail_id: '789',
-        keyword: ['penguin'],
-        license: ['http://creativecommons.org/licenses/by/3.0/us/'],
-        member_of_collection_ids: ['123456', 'abcdef']
-      )
-    end
-
-    subject { described_class.model_attributes(params) }
-
-    it 'permits parameters' do
-      expect(subject['title']).to eq ['foo']
-      expect(subject['description']).to be_empty
-      expect(subject['visibility']).to eq 'open'
-      expect(subject['license']).to eq ['http://creativecommons.org/licenses/by/3.0/us/']
-      expect(subject['keyword']).to eq ['penguin']
-      expect(subject['member_of_collection_ids']).to eq ['123456', 'abcdef']
-      expect(subject['rendering_ids']).to eq [file_set.id]
-    end
-
-    context '.model_attributes' do
-      let(:params) do
-        ActionController::Parameters.new(
-          title: [''],
-          description: [''],
-          keyword: [''],
-          license: [''],
-          member_of_collection_ids: [''],
-          on_behalf_of: 'Melissa'
-        )
-      end
-
-      it 'removes blank parameters' do
-        expect(subject['title']).to be_empty
-        expect(subject['description']).to be_empty
-        expect(subject['license']).to be_empty
-        expect(subject['keyword']).to be_empty
-        expect(subject['member_of_collection_ids']).to be_empty
-        expect(subject['on_behalf_of']).to eq 'Melissa'
-      end
-    end
-  end
-
-  describe "#visibility" do
-    subject { form.visibility }
-
-    it { is_expected.to eq 'restricted' }
-  end
-
-  subject { form }
-
-  it { is_expected.to delegate_method(:on_behalf_of).to(:model) }
-  it { is_expected.to delegate_method(:depositor).to(:model) }
-  it { is_expected.to delegate_method(:permissions).to(:model) }
-
-  describe "#agreement_accepted" do
-    subject { form.agreement_accepted }
-
-    it { is_expected.to eq false }
-  end
-
-  context "on a work already saved" do
-    before { allow(work).to receive(:new_record?).and_return(false) }
-    it "defaults deposit agreement to true" do
-      expect(form.agreement_accepted).to eq(true)
-    end
-  end
-end

data/spec/helpers/blacklight_helper_spec.rb

@@ -1,137 +0,0 @@
-RSpec.describe BlacklightHelper, type: :helper do
-  let(:blacklight_config) { CatalogController.blacklight_config }
-  let(:attributes) do
-    { 'creator_tesim'          => ['Justin', 'Joe'],
-      'depositor_tesim'        => ['jcoyne@justincoyne.com'],
-      'proxy_depositor_ssim'   => ['atz@stanford.edu'],
-      'description_tesim'      => ['This links to http://example.com/ What about that?'],
-      'date_uploaded_dtsi'     => '2013-03-14T00:00:00Z',
-      'license_tesim' => ["http://creativecommons.org/publicdomain/zero/1.0/",
-                          "http://creativecommons.org/publicdomain/mark/1.0/",
-                          "http://www.europeana.eu/portal/rights/rr-r.html"],
-      'rights_statement_tesim' => ['http://rightsstatements.org/vocab/InC/1.0/'],
-      'identifier_tesim'       => ['65434567654345654'],
-      'keyword_tesim'          => ['taco', 'mustache'],
-      'subject_tesim'          => ['Awesome'],
-      'contributor_tesim'      => ['Bird, Big'],
-      'publisher_tesim'        => ['Penguin Random House'],
-      'based_near_label_tesim' => ['Pennsylvania'],
-      'language_tesim'         => ['English'],
-      'resource_type_tesim'    => ['Capstone Project'] }
-  end
-
-  let(:document) { SolrDocument.new(attributes) }
-
-  before do
-    allow(helper).to receive(:blacklight_config).and_return(blacklight_config)
-    allow(controller).to receive(:action_name).and_return('index')
-  end
-
-  describe "render_index_field_value" do
-    include HyraxHelper # FIXME: isolate testing BlacklightHelper, not HyraxHelper. Also, this method is odd.
-    def search_action_path(stuff)
-      search_catalog_path(stuff)
-    end
-
-    subject { index_presenter(document).field_value field_name }
-
-    context "description_tesim" do
-      let(:field_name) { 'description_tesim' }
-
-      it do
-        pending 'need a different way to test description'
-        is_expected.to eq '<span itemprop="description">This links to ' \
-                          '<a href="http://example.com/"><span class="glyphicon glyphicon-new-window"></span>' \
-                          ' http://example.com/</a> What about that?</span>'
-      end
-    end
-
-    context "license_tesim" do
-      let(:field_name) { 'license_tesim' }
-
-      it do
-        is_expected.to eq "<a href=\"http://creativecommons.org/publicdomain/zero/1.0/\">Creative Commons CC0 1.0 Universal</a>, " \
-                             "<a href=\"http://creativecommons.org/publicdomain/mark/1.0/\">Creative Commons Public Domain Mark 1.0</a>, " \
-                             "and <a href=\"http://www.europeana.eu/portal/rights/rr-r.html\">All rights reserved</a>"
-      end
-    end
-
-    context "rights_statement_tesim" do
-      let(:field_name) { 'rights_statement_tesim' }
-
-      it do
-        is_expected.to eq "<a href=\"http://rightsstatements.org/vocab/InC/1.0/\">In Copyright</a>"
-      end
-    end
-
-    context "metadata index links" do
-      let(:search_state) { Hyrax::SearchState.new(params, blacklight_config, controller) }
-
-      before do
-        allow(controller).to receive(:search_state).and_return(search_state)
-      end
-
-      context "keyword_tesim" do
-        let(:field_name) { 'keyword_tesim' }
-
-        it do
-          is_expected.to eq '<span itemprop="keywords">' \
-                               '<a href="/catalog?f%5Bkeyword_sim%5D%5B%5D=taco">taco</a></span> and ' \
-                               '<span itemprop="keywords"><a href="/catalog?f%5Bkeyword_sim%5D%5B%5D=mustache">mustache</a></span>'
-        end
-      end
-
-      context "subject_tesim" do
-        let(:field_name) { 'subject_tesim' }
-
-        it { is_expected.to eq '<span itemprop="about"><a href="/catalog?f%5Bsubject_sim%5D%5B%5D=Awesome">Awesome</a></span>' }
-      end
-
-      context "creator_tesim" do
-        let(:field_name) { 'creator_tesim' }
-
-        it do
-          is_expected.to eq '<span itemprop="creator">' \
-                               '<a href="/catalog?f%5Bcreator_sim%5D%5B%5D=Justin">Justin</a></span> and ' \
-                               '<span itemprop="creator"><a href="/catalog?f%5Bcreator_sim%5D%5B%5D=Joe">Joe</a></span>'
-        end
-      end
-
-      context "contributor_tesim" do
-        let(:field_name) { 'contributor_tesim' }
-
-        it { is_expected.to eq '<span itemprop="contributor"><a href="/catalog?f%5Bcontributor_sim%5D%5B%5D=Bird%2C+Big">Bird, Big</a></span>' }
-      end
-
-      context "publisher_tesim" do
-        let(:field_name) { 'publisher_tesim' }
-
-        it { is_expected.to eq '<span itemprop="publisher"><a href="/catalog?f%5Bpublisher_sim%5D%5B%5D=Penguin+Random+House">Penguin Random House</a></span>' }
-      end
-
-      context "based_near_label_tesim" do
-        let(:field_name) { 'based_near_label_tesim' }
-
-        it { is_expected.to eq '<span itemprop="contentLocation"><a href="/catalog?f%5Bbased_near_label_sim%5D%5B%5D=Pennsylvania">Pennsylvania</a></span>' }
-      end
-
-      context "language_tesim" do
-        let(:field_name) { 'language_tesim' }
-
-        it { is_expected.to eq '<span itemprop="inLanguage"><a href="/catalog?f%5Blanguage_sim%5D%5B%5D=English">English</a></span>' }
-      end
-
-      context "resource_type_tesim" do
-        let(:field_name) { 'resource_type_tesim' }
-
-        it { is_expected.to eq '<a href="/catalog?f%5Bresource_type_sim%5D%5B%5D=Capstone+Project">Capstone Project</a>' }
-      end
-
-      context "identifier_tesim" do
-        let(:field_name) { 'identifier_tesim' }
-
-        it { is_expected.to eq '<a href="/catalog?q=%2265434567654345654%22&amp;search_field=identifier">65434567654345654</a>' }
-      end
-    end
-  end
-end

data/spec/helpers/hyrax/ability_helper_spec.rb

@@ -1,61 +0,0 @@
-RSpec.describe Hyrax::AbilityHelper do
-  describe "#visibility_badge" do
-    subject { helper.visibility_badge visibility }
-
-    {
-      Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC =>
-        "<span class=\"label label-success\">Public</span>",
-      Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED =>
-        "<span class=\"label label-info\">%<name>s</span>",
-      Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE =>
-        "<span class=\"label label-danger\">Private</span>",
-      Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_EMBARGO =>
-        "<span class=\"label label-warning\">Embargo</span>",
-      Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_LEASE =>
-        "<span class=\"label label-warning\">Lease</span>"
-    }.each do |value, output|
-      context value do
-        let(:visibility) { value }
-
-        it { expect(subject).to eql(format(output, name: t('hyrax.institution_name'))) }
-      end
-    end
-  end
-
-  describe "#visibility_options" do
-    let(:public_opt) { ['Public', Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PUBLIC] }
-    let(:authenticated_opt) { [t('hyrax.institution_name'), Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_AUTHENTICATED] }
-    let(:private_opt) { ['Private', Hydra::AccessControls::AccessRight::VISIBILITY_TEXT_VALUE_PRIVATE] }
-
-    subject { helper.visibility_options(option) }
-
-    context 'all options' do
-      let(:options) { [public_opt, authenticated_opt, private_opt] }
-      let(:option) { nil }
-
-      it { is_expected.to eql(options) }
-    end
-    context 'restricting options' do
-      let(:options) { [private_opt, authenticated_opt] }
-      let(:option) { :restrict }
-
-      it { is_expected.to eql(options) }
-    end
-    context 'loosening options' do
-      let(:options) { [public_opt, authenticated_opt] }
-      let(:option) { :loosen }
-
-      it { is_expected.to eql(options) }
-    end
-  end
-
-  describe "#render_visibility_link" do
-    subject { helper.render_visibility_link(document) }
-
-    context 'admin set' do
-      let(:document) { double(admin_set?: true) }
-
-      it { is_expected.to be_nil }
-    end
-  end
-end

data/spec/helpers/hyrax/batch_edits_helper_spec.rb

@@ -1,59 +0,0 @@
-RSpec.describe Hyrax::BatchEditsHelper, type: :helper do
-  describe "#render_check_all" do
-    subject { helper.render_check_all }
-
-    before do
-      view.lookup_context.prefixes = ['hyrax/my']
-      allow(helper).to receive(:params).and_return(controller: controller_path)
-    end
-
-    context "with my works" do
-      let(:controller_path) { "hyrax/my/works" }
-
-      it "shows the check all dropdown" do
-        expect(subject).to have_css("span.caret")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_all")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_none")
-      end
-    end
-
-    context "with my shares" do
-      let(:controller_path) { "hyrax/my/shares" }
-
-      it "shows the check all dropdown" do
-        expect(subject).to have_css("span.caret")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_all")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_none")
-      end
-    end
-
-    context "with my highlights" do
-      let(:controller_path) { "hyrax/my/highlights" }
-
-      it "shows the check all dropdown" do
-        expect(subject).to have_css("span.caret")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_all")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_none")
-      end
-    end
-
-    context "with my collections" do
-      let(:controller_path) { "hyrax/my/collections" }
-
-      it "show the check all dropdown" do
-        expect(subject).to have_css("span.caret")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_all")
-        expect(subject).to have_content t("hyrax.dashboard.my.action.select_none")
-      end
-    end
-
-    context "with select all disabled" do
-      let(:controller_path) { "foo" }
-
-      it "does not show the check all dropdown" do
-        assign(:disable_select_all, true)
-        expect(subject).to have_css("input[disabled=disabled]")
-      end
-    end
-  end
-end

data/spec/helpers/hyrax/charts_helper_spec.rb

@@ -1,63 +0,0 @@
-RSpec.describe Hyrax::ChartsHelper do
-  describe '#hash_to_chart' do
-    subject { helper.hash_to_chart(data) }
-
-    let(:data) do
-      {
-        'Foo' => 5,
-        'Bar' => 10
-      }
-    end
-
-    it do
-      is_expected.to eq(
-        drilldown: {
-          series: []
-        },
-        series:
-        [
-          {
-            name: "Foo",
-            y: 5
-          },
-          {
-            name: "Bar",
-            y: 10
-          }
-        ]
-      )
-    end
-    context "given a drilldown" do
-      let(:data) do
-        {
-          "Foo" => {
-            "Bar" => 1,
-            "Baz" => 2
-          }
-        }
-      end
-
-      it do
-        is_expected.to eq(
-          drilldown: {
-            series: [
-              {
-                name: "Foo",
-                id: "Foo",
-                data: [["Bar", 1], ["Baz", 2]]
-              }
-            ]
-          },
-          series:
-          [
-            {
-              name: "Foo",
-              y: 3,
-              drilldown: "Foo"
-            }
-          ]
-        )
-      end
-    end
-  end
-end