test_hyrax 0.0.1.alpha

data/spec/fixtures/xml_fits.xml

@@ -0,0 +1,37 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<fits xmlns="http://hul.harvard.edu/ois/xml/ns/fits/fits_output" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://hul.harvard.edu/ois/xml/ns/fits/fits_output http://hul.harvard.edu/ois/xml/xsd/fits/fits_output.xsd" version="0.6.0" timestamp="4/24/12 9:03 PM">
+  <identification>
+    <identity format="Extensible Markup Language" mimetype="text/xml" toolname="FITS" toolversion="0.6.0">
+      <tool toolname="Jhove" toolversion="1.5" />
+      <tool toolname="file utility" toolversion="5.04" />
+      <tool toolname="Exiftool" toolversion="7.74" />
+      <tool toolname="Droid" toolversion="3.0" />
+      <tool toolname="NLNZ Metadata Extractor" toolversion="3.4GA" />
+      <tool toolname="OIS XML Metadata" toolversion="0.2" />
+      <tool toolname="ffident" toolversion="0.2" />
+      <version toolname="Jhove" toolversion="1.5">1.0</version>
+      <externalIdentifier toolname="Droid" toolversion="3.0" type="puid">fmt/101</externalIdentifier>
+    </identity>
+  </identification>
+  <fileinfo>
+    <size toolname="Jhove" toolversion="1.5">4000</size>
+    <lastmodified toolname="Exiftool" toolversion="7.74" status="SINGLE_RESULT">2012:04:24 20:07:24-04:00</lastmodified>
+    <filepath toolname="OIS File Information" toolversion="0.1" status="SINGLE_RESULT">/home/mjg/workspace/scholarsphere/spec/fixtures/scholarsphere/scholarsphere_test1.foxml.xml</filepath>
+    <filename toolname="OIS File Information" toolversion="0.1" status="SINGLE_RESULT">scholarsphere_test1.foxml.xml</filename>
+    <md5checksum toolname="OIS File Information" toolversion="0.1" status="SINGLE_RESULT">1b5cfeeca9ddecd1a4cf30256bac6bdd</md5checksum>
+    <fslastmodified toolname="OIS File Information" toolversion="0.1" status="SINGLE_RESULT">1335312444000</fslastmodified>
+  </fileinfo>
+  <filestatus>
+    <well-formed toolname="Jhove" toolversion="1.5" status="SINGLE_RESULT">true</well-formed>
+    <valid toolname="Jhove" toolversion="1.5" status="SINGLE_RESULT">true</valid>
+  </filestatus>
+  <metadata>
+    <text>
+      <charset toolname="Jhove" toolversion="1.5">UTF-8</charset>
+      <markupBasis toolname="Jhove" toolversion="1.5">XML</markupBasis>
+      <markupBasisVersion toolname="Jhove" toolversion="1.5">1.0</markupBasisVersion>
+      <markupLanguage toolname="OIS XML Metadata" toolversion="0.2" status="SINGLE_RESULT">http://www.fedora.info/definitions/1/0/foxml1-1.xsd</markupLanguage>
+    </text>
+  </metadata>
+</fits>
+

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

@@ -0,0 +1,20 @@
+require 'spec_helper'
+RSpec.describe Hyrax::Forms::AdminSetForm do
+  let(:form) { described_class.new(model) }
+
+  describe "[] accessors" do
+    let(:model) { AdminSet.new(description: ['one']) }
+    it "cast to scalars" do
+      expect(form[:description]).to eq 'one'
+    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
+end

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

@@ -0,0 +1,102 @@
+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'],
+           rights: ['rights1'],
+           subject: ['subject1'],
+           identifier: ['id1'],
+           based_near: ['based_near1'],
+           related_url: ['related_url1']
+  end
+
+  let(:work2) do
+    create :generic_work,
+           title: ["title 2"],
+           keyword: ["123"],
+           creator: ["Fred"],
+           publisher: ['Rand McNally'],
+           language: ['en'],
+           resource_type: ['bar'],
+           contributor: ['contributor2'],
+           description: ['description2'],
+           rights: ['rights2'],
+           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,
+                         :rights,
+                         :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.rights).to match_array ["rights1", "rights2"]
+      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: [] },
+                         { rights: [] },
+                         { publisher: [] },
+                         { date_created: [] },
+                         { subject: [] },
+                         { language: [] },
+                         { identifier: [] },
+                         { based_near: [] },
+                         { related_url: [] },
+                         { permissions_attributes: [:type, :name, :access, :id, :_destroy] },
+                         :on_behalf_of,
+                         :visibility_during_embargo,
+                         :embargo_release_date,
+                         :visibility_after_embargo,
+                         :visibility_during_lease,
+                         :lease_expiration_date,
+                         :visibility_after_lease,
+                         :visibility]
+    end
+  end
+end

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

@@ -0,0 +1,68 @@
+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] }
+    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,
+                         :rights,
+                         :publisher,
+                         :date_created,
+                         :subject,
+                         :language,
+                         :identifier,
+                         :based_near,
+                         :related_url,
+                         :representative_id,
+                         :thumbnail_id,
+                         :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

@@ -0,0 +1,122 @@
+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,
+                         :rights,
+                         :publisher,
+                         :date_created,
+                         :subject,
+                         :language,
+                         :representative_id,
+                         :thumbnail_id,
+                         :identifier,
+                         :based_near,
+                         :related_url,
+                         :visibility]
+    end
+  end
+
+  let(:collection) { build(:collection) }
+  let(:form) { described_class.new(collection) }
+
+  describe "#primary_terms" do
+    subject { form.primary_terms }
+    it { is_expected.to eq([:title]) }
+  end
+
+  describe "#secondary_terms" do
+    subject { form.secondary_terms }
+
+    it do
+      is_expected.to eq [
+        :creator,
+        :contributor,
+        :description,
+        :keyword,
+        :rights,
+        :publisher,
+        :date_created,
+        :subject,
+        :language,
+        :identifier,
+        :based_near,
+        :related_url,
+        :resource_type
+      ]
+    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: [] },
+                         { rights: [] },
+                         { publisher: [] },
+                         { date_created: [] },
+                         { subject: [] },
+                         { language: [] },
+                         :representative_id,
+                         :thumbnail_id,
+                         { identifier: [] },
+                         { based_near: [] },
+                         { related_url: [] },
+                         :visibility]
+    end
+  end
+
+  describe '#select_files' do
+    context 'without any works/files attached' do
+      subject { form.select_files }
+      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(:collection_with_file) do
+        create(:collection, members: [work])
+      end
+
+      it 'returns a hash of with file title as key and file id as value' do
+        form_with_files = described_class.new(collection_with_file)
+        expect(form_with_files.select_files).to eq(title => file_id)
+      end
+    end
+  end
+end

data/spec/forms/hyrax/forms/file_set_edit_form_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe Hyrax::Forms::FileSetEditForm do
+  subject { described_class.new(FileSet.new) }
+
+  describe '#terms' do
+    it 'returns a list' do
+      expect(subject.terms).to eq(
+        [:resource_type, :title, :creator, :contributor, :description, :keyword,
+         :rights, :publisher, :date_created, :subject, :language, :identifier,
+         :based_near, :related_url,
+         :visibility_during_embargo, :visibility_after_embargo, :embargo_release_date,
+         :visibility_during_lease, :visibility_after_lease, :lease_expiration_date,
+         :visibility]
+      )
+    end
+
+    it "doesn't contain fields that users shouldn't be allowed to edit" do
+      # date_uploaded is reserved for the original creation date of the record.
+      expect(subject.terms).not_to include(:date_uploaded)
+    end
+  end
+
+  it 'initializes multivalued fields' do
+    expect(subject.title).to eq ['']
+  end
+
+  describe '.model_attributes' do
+    let(:params) do
+      ActionController::Parameters.new(
+        title: ['foo'],
+        "visibility" => "on-campus",
+        "visibility_during_embargo" => "restricted",
+        "embargo_release_date" => "2015-10-21",
+        "visibility_after_embargo" => "open",
+        "visibility_during_lease" => "open",
+        "lease_expiration_date" => "2015-10-21",
+        "visibility_after_lease" => "restricted"
+      )
+    end
+    subject { described_class.model_attributes(params) }
+
+    it 'changes only the title' do
+      expect(subject['title']).to eq ['foo']
+      expect(subject['visibility']).to eq('on-campus')
+      expect(subject['visibility_during_embargo']).to eq('restricted')
+      expect(subject['visibility_after_embargo']).to eq('open')
+      expect(subject['embargo_release_date']).to eq('2015-10-21')
+      expect(subject['visibility_during_lease']).to eq('open')
+      expect(subject['visibility_after_lease']).to eq('restricted')
+      expect(subject['lease_expiration_date']).to eq('2015-10-21')
+    end
+  end
+end

data/spec/forms/hyrax/forms/permission_template_form_spec.rb

@@ -0,0 +1,290 @@
+require 'spec_helper'
+
+RSpec.describe Hyrax::Forms::PermissionTemplateForm do
+  describe "#update" do
+    let(:grant_attributes) { [] }
+    let(:input_params) do
+      ActionController::Parameters.new(access_grants_attributes: grant_attributes).permit!
+    end
+    let(:admin_set) { create(:admin_set) }
+    let(:permission_template) { create(:permission_template, admin_set_id: admin_set.id) }
+    let(:form) { described_class.new(permission_template) }
+    subject { form.update(input_params) }
+    let(:today) { Time.zone.today }
+
+    it "calls grant_workflow_roles" do
+      expect(form).to receive(:grant_workflow_roles)
+      subject
+    end
+
+    context "with a user manager" do
+      let(:grant_attributes) do
+        [ActionController::Parameters.new(agent_type: "user",
+                                          agent_id: "bob",
+                                          access: "manage").permit!]
+      end
+      it "also adds edit_access to the AdminSet itself" do
+        expect { subject }.to change { permission_template.access_grants.count }.by(1)
+        expect(admin_set.reload.edit_users).to include 'bob'
+      end
+    end
+
+    context "with a group manager" do
+      let(:grant_attributes) do
+        [ActionController::Parameters.new(agent_type: "group",
+                                          agent_id: "bob",
+                                          access: "manage").permit!]
+      end
+      it "also adds edit_access to the AdminSet itself" do
+        expect { subject }.to change { permission_template.access_grants.count }.by(1)
+        expect(admin_set.reload.edit_groups).to include 'bob'
+      end
+    end
+
+    context "without a manager" do
+      let(:grant_attributes) do
+        [ActionController::Parameters.new(agent_type: "user",
+                                          agent_id: "bob",
+                                          access: "view").permit!]
+      end
+      it "doesn't adds edit_access to the AdminSet itself" do
+        expect { subject }.to change { permission_template.access_grants.count }.by(1)
+        expect(admin_set.reload.edit_users).to be_empty
+      end
+    end
+
+    context "with visibility only" do
+      let(:input_params) do
+        ActionController::Parameters.new(visibility: "open").permit!
+      end
+      it "updates the visibility" do
+        expect { subject }.to change { permission_template.reload.visibility }.from(nil).to('open')
+      end
+    end
+
+    context "with release 'no delay'" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY).permit!
+      end
+      it "sets release_period=now, release_date=today" do
+        expect { subject }.to change { permission_template.reload.release_period }.from(nil).to(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY)
+        expect(permission_template.release_date).to eq(today)
+      end
+    end
+
+    context "with release 'varies', date specified" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: "", release_varies: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE, release_date: today + 1.year).permit!
+      end
+      it "sets release_period=before and release_date" do
+        expect { subject }.to change { permission_template.reload.release_period }.from(nil).to(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE)
+        expect(permission_template.release_date).to eq(today + 1.year)
+      end
+    end
+
+    context "with release 'varies', embargo specified" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: "",
+                                         release_varies: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_EMBARGO,
+                                         release_embargo: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_2_YEARS).permit!
+      end
+      it "sets release_period to embargo period and release_date to 2 years from now" do
+        expect { subject }.to change { permission_template.reload.release_period }.from(nil).to(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_2_YEARS)
+        expect(permission_template.release_date).to eq(today + 2.years)
+      end
+    end
+
+    context "with release 'fixed', date specified" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED,
+                                         release_date: today + 1.month).permit!
+      end
+      it "sets release_period=fixed and release_date" do
+        expect { subject }.to change { permission_template.reload.release_period }
+          .from(nil)
+          .to(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED)
+        expect(permission_template.release_date).to eq(today + 1.month)
+      end
+    end
+
+    context "with modifying release_period from 'fixed' to 'no_delay'" do
+      let(:permission_template) { create(:permission_template, admin_set_id: admin_set.id, release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED, release_date: today + 1.month) }
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY).permit!
+      end
+      it "sets release_period=now, release_date=today" do
+        expect { subject }.to change { permission_template.reload.release_period }
+          .from(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_FIXED)
+          .to(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY)
+        expect(permission_template.release_date).to eq(today)
+      end
+    end
+
+    context "with modifying release 'varies' from date specified to embargo specified" do
+      let(:permission_template) do
+        create(:permission_template,
+               admin_set_id: admin_set.id,
+               release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE,
+               release_date: today + 1.month)
+      end
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: "",
+                                         release_varies: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_EMBARGO,
+                                         release_embargo: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_2_YEARS).permit!
+      end
+      it "sets release_period to embargo period, release_date to 2 years from now" do
+        expect { subject }.to change { permission_template.reload.release_period }
+          .from(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE)
+          .to(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_2_YEARS)
+        expect(permission_template.release_date).to eq(today + 2.years)
+      end
+    end
+  end
+
+  describe "#grant_workflow_roles" do
+    subject { form.send(:grant_workflow_roles) }
+    let(:workflow) { create(:workflow) }
+    let(:user) { create(:user) }
+    let(:role1) { Sipity::Role.create!(name: 'hello') }
+    let(:role2) { Sipity::Role.create!(name: 'goodbye') }
+
+    let(:permission_template) do
+      create(:permission_template,
+             workflow_name: workflow.name,
+             access_grants_attributes:
+               [{ agent_type: 'user',
+                  agent_id: user.user_key,
+                  access: 'manage' },
+                { agent_type: 'group',
+                  agent_id: 'librarians',
+                  access: 'manage' }])
+    end
+    let(:form) { described_class.new(permission_template) }
+    before do
+      permission_template.clear_changes_information
+      workflow.workflow_roles.create!([{ role: role1 }, { role: role2 }])
+    end
+
+    context "when a new workflow has been chosen" do
+      before do
+        allow(permission_template).to receive(:previous_changes).and_return("workflow_name" => [nil, workflow.name])
+      end
+
+      it "gives the managers workflow roles" do
+        expect { subject }.to change { Sipity::WorkflowResponsibility.count }.by(4)
+      end
+    end
+
+    context "when a new workflow is not changed" do
+      it "does nothing" do
+        expect { subject }.not_to change { Sipity::WorkflowResponsibility.count }
+      end
+    end
+  end
+
+  describe "#select_release_varies_option" do
+    let(:admin_set) { create(:admin_set) }
+    let(:form) { described_class.new(permission_template) }
+    subject { form.send(:select_release_varies_option, permission_template) }
+    let(:today) { Time.zone.today }
+
+    context "with release before date specified" do
+      let(:permission_template) do
+        create(:permission_template,
+               admin_set_id: admin_set.id,
+               release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE,
+               release_date: today + 1.month)
+      end
+      it "selects release_varies='before'" do
+        expect(form.release_varies).to eq(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE)
+        expect(form.release_embargo).to be_nil
+      end
+    end
+
+    context "with release embargo specified" do
+      let(:permission_template) do
+        create(:permission_template,
+               admin_set_id: admin_set.id,
+               release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR)
+      end
+      it "selects release_varies='embargo' and value in embargo selectbox" do
+        expect(form.release_varies).to eq(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_EMBARGO)
+        expect(form.release_embargo).to eq(Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR)
+      end
+    end
+
+    context "with release no-delay (now) selected" do
+      let(:permission_template) do
+        create(:permission_template,
+               admin_set_id: admin_set.id,
+               release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY)
+      end
+      it "selects neither varies option, nor embargo" do
+        expect(form.release_varies).to be_nil
+        expect(form.release_embargo).to be_nil
+      end
+    end
+  end
+
+  describe "#permission_template_update_params" do
+    let(:admin_set) { create(:admin_set) }
+    let(:permission_template) { create(:permission_template, admin_set_id: admin_set.id) }
+    let(:form) { described_class.new(permission_template) }
+    subject { form.send(:permission_template_update_params, input_params) }
+    let(:today) { Time.zone.today }
+
+    context "with release varies by date selected" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: "",
+                                         release_varies: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE,
+                                         release_date: today + 1.month).permit!
+      end
+      it "updates params to release_period=before and keeps date" do
+        expect(subject).to eq ActionController::Parameters.new(
+          release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_BEFORE_DATE,
+          release_date: today + 1.month
+        ).permit!
+      end
+    end
+
+    context "with release varies by embargo selected" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: "",
+                                         release_varies: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_EMBARGO,
+                                         release_embargo: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR).permit!
+      end
+      it "updates params to release_period=1yr" do
+        expect(subject).to eq ActionController::Parameters.new(
+          release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_1_YEAR,
+          release_date: nil
+        ).permit!
+      end
+    end
+
+    context "with release no delay (now) selected, after filling out release_date" do
+      let(:input_params) do
+        ActionController::Parameters.new(release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY,
+                                         release_varies: "",
+                                         release_embargo: "",
+                                         release_date: today + 1.month).permit!
+      end
+      it "updates params to release_period=1yr" do
+        expect(subject).to eq ActionController::Parameters.new(
+          release_period: Hyrax::PermissionTemplate::RELEASE_TEXT_VALUE_NO_DELAY,
+          release_date: nil
+        ).permit!
+      end
+    end
+  end
+
+  describe "#workflows" do
+    let(:admin_set) { create(:admin_set) }
+    let(:permission_template) { create(:permission_template, admin_set_id: admin_set.id) }
+    let(:form) { described_class.new(permission_template) }
+
+    it "returns all of the workflows" do
+      expect(Sipity::Workflow).to receive(:all).and_return(:the_workflows)
+      expect(form.workflows).to eq(:the_workflows)
+    end
+  end
+end