hyrax 2.6.0 → 2.7.0

data/spec/forms/hyrax/forms/admin/collection_type_participant_form_spec.rb

@@ -1,10 +0,0 @@
-RSpec.describe Hyrax::Forms::Admin::CollectionTypeParticipantForm do
-  let(:collection_type_participant) { build(:collection_type_participant) }
-  let(:form) { described_class.new(collection_type_participant: collection_type_participant) }
-
-  subject { form }
-
-  it { is_expected.to delegate_method(:agent_id).to(:collection_type_participant) }
-  it { is_expected.to delegate_method(:agent_type).to(:collection_type_participant) }
-  it { is_expected.to delegate_method(:access).to(:collection_type_participant) }
-end

data/spec/forms/hyrax/forms/admin_set_form_spec.rb

@@ -1,88 +0,0 @@
-RSpec.describe Hyrax::Forms::AdminSetForm do
-  let(:ability) { Ability.new(create(:user)) }
-  let(:repository) { double }
-  let(:form) { described_class.new(model, ability, repository) }
-
-  describe "[] accessors" do
-    let(:model) { build(:admin_set, description: ['one']) }
-
-    it "cast to scalars" do
-      expect(form[:description]).to eq 'one'
-    end
-  end
-
-  describe "#thumbnail_title" do
-    subject { form.thumbnail_title }
-
-    context "when the admin_set has a thumbnail" do
-      let(:thumbnail) { stub_model(FileSet, title: ['Ulysses']) }
-      let(:model) { AdminSet.new(thumbnail: thumbnail) }
-
-      it { is_expected.to eq "Ulysses" }
-    end
-
-    context "when the admin_set has no thumbnail" do
-      let(:model) { AdminSet.new }
-
-      it { is_expected.to be nil }
-    end
-  end
-
-  describe "#permission_template" do
-    subject { form.permission_template }
-
-    context "when the PermissionTemplate doesn't exist" do
-      let(:model) { create(:admin_set) }
-
-      it "gets created" do
-        expect(subject).to be_instance_of Hyrax::Forms::PermissionTemplateForm
-        expect(subject.model).to be_instance_of Hyrax::PermissionTemplate
-      end
-    end
-
-    context "when the PermissionTemplate exists" do
-      let(:permission_template) { Hyrax::PermissionTemplate.find_by(source_id: model.id) }
-      let(:model) { create(:admin_set, with_permission_template: true) }
-
-      it "uses the existing template" do
-        expect(subject).to be_instance_of Hyrax::Forms::PermissionTemplateForm
-        expect(subject.model).to eq permission_template
-      end
-    end
-  end
-
-  describe "model_attributes" do
-    let(:raw_attrs) { ActionController::Parameters.new(title: 'test title') }
-
-    subject { described_class.model_attributes(raw_attrs) }
-
-    it "casts to enums" do
-      expect(subject[:title]).to eq ['test title']
-    end
-  end
-
-  describe '#select_files' do
-    subject { form.select_files }
-
-    let(:repository) { Hyrax::CollectionsController.new.repository }
-
-    context 'without any works/files attached' do
-      let(:model) { create(:admin_set) }
-
-      it { is_expected.to be_empty }
-    end
-
-    context 'with a work/file attached' do
-      let(:work) { create(:work_with_one_file) }
-      let(:title) { work.file_sets.first.title.first }
-      let(:file_id) { work.file_sets.first.id }
-      let(:model) do
-        create(:admin_set, members: [work])
-      end
-
-      it 'returns a hash of with file title as key and file id as value' do
-        expect(subject).to eq(title => file_id)
-      end
-    end
-  end
-end

data/spec/forms/hyrax/forms/batch_edit_form_spec.rb

@@ -1,111 +0,0 @@
-RSpec.describe Hyrax::Forms::BatchEditForm do
-  let(:model) { GenericWork.new }
-  let(:work1) do
-    create :generic_work,
-           title: ["title 1"],
-           keyword: ["abc"],
-           creator: ["Wilma"],
-           language: ['en'],
-           contributor: ['contributor1'],
-           description: ['description1'],
-           license: ['license1'],
-           subject: ['subject1'],
-           identifier: ['id1'],
-           based_near: ['based_near1'],
-           related_url: ['related_url1']
-  end
-
-  # Using a different work type in order to show that the form supports
-  # batches containing multiple types of works
-  let(:work2) do
-    NamespacedWorks::NestedWork.create!(
-      title: ["title 2"],
-      keyword: ["123"],
-      creator: ["Fred"],
-      publisher: ['Rand McNally'],
-      language: ['en'],
-      resource_type: ['bar'],
-      contributor: ['contributor2'],
-      description: ['description2'],
-      license: ['license2'],
-      subject: ['subject2'],
-      identifier: ['id2'],
-      based_near: ['based_near2'],
-      related_url: ['related_url2']
-    )
-  end
-
-  let(:batch) { [work1.id, work2.id] }
-  let(:form) { described_class.new(model, ability, batch) }
-  let(:ability) { Ability.new(user) }
-  let(:user) { build(:user, display_name: 'Jill Z. User') }
-
-  describe "#terms" do
-    subject { form.terms }
-
-    it do
-      is_expected.to eq [:creator,
-                         :contributor,
-                         :description,
-                         :keyword,
-                         :resource_type,
-                         :license,
-                         :publisher,
-                         :date_created,
-                         :subject,
-                         :language,
-                         :identifier,
-                         :based_near,
-                         :related_url]
-    end
-  end
-
-  describe "#model" do
-    it "combines the models in the batch" do
-      expect(form.model.creator).to match_array ["Wilma", "Fred"]
-      expect(form.model.contributor).to match_array ["contributor1", "contributor2"]
-      expect(form.model.description).to match_array ["description1", "description2"]
-      expect(form.model.keyword).to match_array ["abc", "123"]
-      expect(form.model.resource_type).to match_array ["bar"]
-      expect(form.model.license).to match_array ["license1", "license2"]
-      expect(form.model.publisher).to match_array ["Rand McNally"]
-      expect(form.model.subject).to match_array ["subject1", "subject2"]
-      expect(form.model.language).to match_array ["en"]
-      expect(form.model.identifier).to match_array ["id1", "id2"]
-      expect(form.model.based_near).to match_array ["based_near1", "based_near2"]
-      expect(form.model.related_url).to match_array ["related_url1", "related_url2"]
-    end
-  end
-
-  describe ".build_permitted_params" do
-    subject { described_class.build_permitted_params }
-
-    it do
-      is_expected.to eq [{ creator: [] },
-                         { contributor: [] },
-                         { description: [] },
-                         { keyword: [] },
-                         { resource_type: [] },
-                         { license: [] },
-                         { publisher: [] },
-                         { date_created: [] },
-                         { subject: [] },
-                         { language: [] },
-                         { identifier: [] },
-                         { based_near: [] },
-                         { related_url: [] },
-                         { permissions_attributes: [:type, :name, :access, :id, :_destroy] },
-                         :on_behalf_of,
-                         :version,
-                         :add_works_to_collection,
-                         :visibility_during_embargo,
-                         :embargo_release_date,
-                         :visibility_after_embargo,
-                         :visibility_during_lease,
-                         :lease_expiration_date,
-                         :visibility_after_lease,
-                         :visibility,
-                         { based_near_attributes: [:id, :_destroy] }]
-    end
-  end
-end

data/spec/forms/hyrax/forms/batch_upload_form_spec.rb

@@ -1,75 +0,0 @@
-RSpec.describe Hyrax::Forms::BatchUploadForm do
-  let(:model) { GenericWork.new }
-  let(:controller) { instance_double(Hyrax::BatchUploadsController) }
-  let(:form) { described_class.new(model, ability, controller) }
-  let(:ability) { Ability.new(user) }
-  let(:user) { build(:user, display_name: 'Jill Z. User') }
-
-  describe "#primary_terms" do
-    subject { form.primary_terms }
-
-    it { is_expected.to eq [:creator, :keyword, :rights_statement] }
-    it { is_expected.not_to include(:title) }
-  end
-
-  describe "#secondary_terms" do
-    subject { form.secondary_terms }
-
-    it { is_expected.not_to include(:title) } # title is per file, not per form
-  end
-
-  describe ".model_name" do
-    subject { described_class.model_name }
-
-    it "has a route_key" do
-      expect(subject.route_key).to eq 'batch_uploads'
-    end
-    it "has a param_key" do
-      expect(subject.param_key).to eq 'batch_upload_item'
-    end
-  end
-
-  describe "#to_model" do
-    subject { form.to_model }
-
-    it "returns itself" do
-      expect(subject.to_model).to be_kind_of described_class
-    end
-  end
-
-  describe "#terms" do
-    subject { form.terms }
-
-    it do
-      is_expected.to eq [:creator,
-                         :contributor,
-                         :description,
-                         :keyword,
-                         :license,
-                         :rights_statement,
-                         :publisher,
-                         :date_created,
-                         :subject,
-                         :language,
-                         :identifier,
-                         :based_near,
-                         :related_url,
-                         :representative_id,
-                         :thumbnail_id,
-                         :rendering_ids,
-                         :files,
-                         :visibility_during_embargo,
-                         :embargo_release_date,
-                         :visibility_after_embargo,
-                         :visibility_during_lease,
-                         :lease_expiration_date,
-                         :visibility_after_lease,
-                         :visibility,
-                         :ordered_member_ids,
-                         :source,
-                         :in_works_ids,
-                         :member_of_collection_ids,
-                         :admin_set_id]
-    end
-  end
-end

data/spec/forms/hyrax/forms/collection_form_spec.rb

@@ -1,196 +0,0 @@
-RSpec.describe Hyrax::Forms::CollectionForm do
-  describe "#terms" do
-    subject { described_class.terms }
-
-    it do
-      is_expected.to eq [:resource_type,
-                         :title,
-                         :creator,
-                         :contributor,
-                         :description,
-                         :keyword,
-                         :license,
-                         :publisher,
-                         :date_created,
-                         :subject,
-                         :language,
-                         :representative_id,
-                         :thumbnail_id,
-                         :identifier,
-                         :based_near,
-                         :related_url,
-                         :visibility,
-                         :collection_type_gid]
-    end
-  end
-
-  let(:collection) { build(:collection_lw) }
-  let(:ability) { Ability.new(create(:user)) }
-  let(:repository) { double }
-  let(:form) { described_class.new(collection, ability, repository) }
-
-  describe "#primary_terms" do
-    subject { form.primary_terms }
-
-    it { is_expected.to eq([:title, :description]) }
-  end
-
-  describe "#secondary_terms" do
-    subject { form.secondary_terms }
-
-    it do
-      is_expected.to eq [
-        :creator,
-        :contributor,
-        :keyword,
-        :license,
-        :publisher,
-        :date_created,
-        :subject,
-        :language,
-        :identifier,
-        :based_near,
-        :related_url,
-        :resource_type
-      ]
-    end
-  end
-
-  describe '#display_additional_fields?' do
-    subject { form.display_additional_fields? }
-
-    context 'with no secondary terms' do
-      before do
-        allow(form).to receive(:secondary_terms).and_return([])
-      end
-      it { is_expected.to be false }
-    end
-    context 'with secondary terms' do
-      before do
-        allow(form).to receive(:secondary_terms).and_return([:foo, :bar])
-      end
-      it { is_expected.to be true }
-    end
-  end
-
-  context "nested relationships" do
-    let(:child_collection) { build(:collection_lw) }
-    let(:parent_collection) { build(:collection_lw) }
-    let(:service_object) { double(available_member_subcollections: double(documents: [child_collection])) }
-
-    before do
-      allow(collection).to receive(:member_of_collections).and_return([parent_collection])
-      allow(form).to receive(:collection_member_service).and_return(service_object)
-    end
-
-    describe "#list_parent_collections" do
-      subject { form.list_parent_collections }
-
-      it { is_expected.to eq([parent_collection]) }
-    end
-
-    describe "#list_child_collections" do
-      subject { form.list_child_collections }
-
-      it { is_expected.to eq([child_collection]) }
-    end
-  end
-
-  describe "#id" do
-    subject { form.id }
-
-    it { is_expected.to be_nil }
-  end
-
-  describe "#required?" do
-    subject { form.required?(:title) }
-
-    it { is_expected.to be true }
-  end
-
-  describe "#human_readable_type" do
-    subject { form.human_readable_type }
-
-    it { is_expected.to eq 'Collection' }
-  end
-
-  describe "#member_ids" do
-    before do
-      allow(collection).to receive(:member_ids).and_return(['9999'])
-    end
-    subject { form.member_ids }
-
-    it { is_expected.to eq ['9999'] }
-  end
-
-  describe ".build_permitted_params" do
-    subject { described_class.build_permitted_params }
-
-    it do
-      is_expected.to eq [{ resource_type: [] },
-                         { title: [] },
-                         { creator: [] },
-                         { contributor: [] },
-                         { description: [] },
-                         { keyword: [] },
-                         { license: [] },
-                         { publisher: [] },
-                         { date_created: [] },
-                         { subject: [] },
-                         { language: [] },
-                         :representative_id,
-                         :thumbnail_id,
-                         { identifier: [] },
-                         { based_near: [] },
-                         { related_url: [] },
-                         :visibility,
-                         :collection_type_gid,
-                         { permissions_attributes: [:type, :name, :access, :id, :_destroy] }]
-    end
-  end
-
-  describe '#select_files' do
-    subject { form.select_files }
-
-    let(:collection) { create(:collection) }
-    let(:repository) { Hyrax::CollectionsController.new.repository }
-
-    context 'without any works/files attached' do
-      it { is_expected.to be_empty }
-    end
-
-    context 'with a work/file attached' do
-      let!(:work) { create(:work_with_one_file, :public, member_of_collections: [collection]) }
-      let(:title) { work.file_sets.first.title.first }
-      let(:file_id) { work.file_sets.first.id }
-
-      it 'returns a hash of with file title as key and file id as value' do
-        expect(subject).to eq(title => file_id)
-      end
-    end
-  end
-
-  describe "#permission_template" do
-    subject { form.permission_template }
-
-    context "when the PermissionTemplate doesn't exist" do
-      let(:model) { build(:collection_lw) }
-
-      it "gets created" do
-        expect(subject).to be_instance_of Hyrax::Forms::PermissionTemplateForm
-        expect(subject.model).to be_instance_of Hyrax::PermissionTemplate
-      end
-    end
-
-    context "when the PermissionTemplate exists" do
-      let(:form) { described_class.new(model, ability, repository) }
-      let(:permission_template) { Hyrax::PermissionTemplate.find_by(source_id: model.id) }
-      let(:model) { build(:collection_lw, with_permission_template: true) }
-
-      it "uses the existing template" do
-        expect(subject).to be_instance_of Hyrax::Forms::PermissionTemplateForm
-        expect(subject.model).to eq permission_template
-      end
-    end
-  end
-end