Merge pull request #12 from co-cddo/add-identifier-and-related-resour…

…ce-steps

Add identifier and related resource forms

app/forms/identifier_form.rb

@@ -0,0 +1,11 @@
+class IdentifierForm < BaseForm
+  validates :identifier, presence: true
+
+  def update_item
+    item.metadata["identifier"] = identifier
+  end
+
+  def identifier
+    @identifier ||= from_params(:identifier)
+  end
+end

app/forms/related_resource_form.rb

@@ -0,0 +1,5 @@
+class RelatedResourceForm < BaseForm
+  def update_item
+    item.metadata["relatedResource"] = from_params(:related_resource)
+  end
+end

app/views/items/forms/_identifier.html.erb

@@ -0,0 +1 @@
+<%= form.govuk_text_field :identifier, value: form.object.metadata['identifier'] %>

app/views/items/forms/_related_resource.html.erb

@@ -0,0 +1,5 @@
+<%= form.govuk_text_field(
+      :related_resource,
+      value: form.object.metadata['related_resource'],
+      label: { text: 'Related resource' }
+    ) %>

app/workflows/item_workflow.rb

@@ -10,6 +10,8 @@ class ItemWorkflow
     creator: CreatorForm,
     publisher: PublisherForm,
     security_classification: SecurityClassificationForm,
+    identifier: IdentifierForm,
+    related_resource: RelatedResourceForm,
   }.freeze
 
   def self.form_for(**options)

spec/forms/identifier_form_spec.rb

@@ -0,0 +1,25 @@
+require "rails_helper"
+
+RSpec.describe IdentifierForm, type: :form do
+  let(:item) { build :item, metadata: {} }
+  let(:identifier) { Faker::Lorem.word }
+  let(:params) do
+    { item: { identifier: } }
+  end
+  let(:identifier_form) { described_class.new(item:, params:) }
+
+  describe "#save" do
+    it "saves the identifier to metadata" do
+      identifier_form.save
+      expect(item.reload.metadata["identifier"]).to eq(identifier)
+    end
+
+    context "when title blank" do
+      let(:identifier) { "" }
+
+      it "invalidates form" do
+        expect(identifier_form).to be_invalid
+      end
+    end
+  end
+end

spec/forms/related_resource_form_spec.rb

@@ -0,0 +1,25 @@
+require "rails_helper"
+
+RSpec.describe RelatedResourceForm, type: :form do
+  let(:item) { build :item, metadata: {} }
+  let(:related_resource) { Faker::Lorem.word }
+  let(:params) do
+    { item: { related_resource: } }
+  end
+  let(:related_resource_form) { described_class.new(item:, params:) }
+
+  describe "#save" do
+    it "saves the related resource to metadata" do
+      related_resource_form.save
+      expect(item.reload.metadata["relatedResource"]).to eq(related_resource)
+    end
+
+    context "when title blank" do
+      let(:relatedResource) { "" }
+
+      it "does not invalidates form" do
+        expect(related_resource_form).to be_valid
+      end
+    end
+  end
+end

spec/requests/item_workflow_intergration_spec.rb

@@ -16,6 +16,8 @@
       creator: Organisation.all.sample.id,
       publisher: Organisation.all.sample.id,
       security_classification: SecurityClassification.values.sample,
+      identifier: Faker::Lorem.word,
+      related_resource: Faker::Lorem.word,
     }
   end