Merge pull request #11 from co-cddo/add-security-classification-step

Add security classification step

app/forms/security_classification_form.rb

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

app/models/security_classification.rb

@@ -0,0 +1,7 @@
+module SecurityClassification
+  VALUES = %w[OFFICIAL SECRET TOP_SECRET NOT_APPLICABLE].freeze
+
+  def self.values
+    VALUES
+  end
+end

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

@@ -0,0 +1,6 @@
+<%= form.govuk_select(:security_classification, label: { text: "Security Classification" }) do %>
+  <%= options_for_select(
+        SecurityClassification.values,
+        selected: form.object.metadata["securityClassification"]
+      ) %>
+<% end %>

app/workflows/item_workflow.rb

@@ -9,6 +9,7 @@ class ItemWorkflow
     contact_point: ContactPointForm,
     creator: CreatorForm,
     publisher: PublisherForm,
+    security_classification: SecurityClassificationForm,
   }.freeze
 
   def self.form_for(**options)

spec/forms/security_classification_form_spec.rb

@@ -0,0 +1,25 @@
+require "rails_helper"
+
+RSpec.describe SecurityClassificationForm, type: :form do
+  let(:item) { build :item }
+  let(:security_classification) { SecurityClassification.values.sample }
+  let(:params) do
+    { item: { security_classification: } }
+  end
+  let(:security_classification_form) { described_class.new(item:, params:) }
+
+  describe "#save" do
+    it "saves the security classification to metadata" do
+      security_classification_form.save
+      expect(item.reload.metadata["securityClassification"]).to eq(security_classification)
+    end
+
+    context "when security classification blank" do
+      let(:security_classification) { "" }
+
+      it "invalidates form" do
+        expect(security_classification_form).to be_invalid
+      end
+    end
+  end
+end

spec/requests/item_workflow_intergration_spec.rb

@@ -15,6 +15,7 @@
       },
       creator: Organisation.all.sample.id,
       publisher: Organisation.all.sample.id,
+      security_classification: SecurityClassification.values.sample,
     }
   end