Merge pull request #1460 from DFE-Digital/fix/161102-clear-support-gr…

…ant-type

(Fix) Clear the value of sponsored_support_grant_type if "Not applicable" ticked

CHANGELOG.md

@@ -6,6 +6,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## [Unreleased][unreleased]
 
+### Fixed
+
+- Clear any radio button values in the Sponsored support grant type task if the
+  task is later saved as "Not applicable"
+
 ## [Release-61][release-61]
 
 ### Added

app/forms/conversion/task/sponsored_support_grant_task_form.rb

@@ -8,4 +8,14 @@ class Conversion::Task::SponsoredSupportGrantTaskForm < BaseOptionalTaskForm
   def type_options
     Conversion::TasksData.sponsored_support_grant_types.values
   end
+
+  def save
+    if not_applicable
+      @tasks_data.sponsored_support_grant_type = nil
+      @tasks_data.assign_attributes prefixed_attributes.except("sponsored_support_grant_type")
+    else
+      @tasks_data.assign_attributes prefixed_attributes
+    end
+    @tasks_data.save!
+  end
 end

app/forms/transfer/task/sponsored_support_grant_task_form.rb

@@ -4,4 +4,14 @@ class Transfer::Task::SponsoredSupportGrantTaskForm < BaseOptionalTaskForm
   def type_options
     Transfer::TasksData.sponsored_support_grant_types.values
   end
+
+  def save
+    if not_applicable
+      @tasks_data.sponsored_support_grant_type = nil
+      @tasks_data.assign_attributes prefixed_attributes.except("sponsored_support_grant_type")
+    else
+      @tasks_data.assign_attributes prefixed_attributes
+    end
+    @tasks_data.save!
+  end
 end

spec/forms/conversion/tasks/sponsored_support_grant_task_form_spec.rb

@@ -0,0 +1,23 @@
+require "rails_helper"
+
+RSpec.describe Conversion::Task::SponsoredSupportGrantTaskForm do
+  let(:user) { create(:user) }
+  let(:project) { create(:conversion_project) }
+
+  describe "#save" do
+    before { mock_successful_api_response_to_create_any_project }
+
+    it "clears the value of 'type' if 'Not applicable' is ticked" do
+      form = described_class.new(project.tasks_data, user)
+      form.assign_attributes(type: "fast_track")
+      form.save
+
+      expect(project.tasks_data.sponsored_support_grant_type).to eq("fast_track")
+
+      form.assign_attributes(not_applicable: true)
+      form.save
+
+      expect(project.tasks_data.sponsored_support_grant_type).to be_nil
+    end
+  end
+end

spec/forms/transfer/tasks/sponsored_support_grant_task_form_spec.rb

@@ -0,0 +1,23 @@
+require "rails_helper"
+
+RSpec.describe Transfer::Task::SponsoredSupportGrantTaskForm do
+  let(:user) { create(:user) }
+  let(:project) { create(:transfer_project) }
+
+  describe "#save" do
+    before { mock_successful_api_response_to_create_any_project }
+
+    it "clears the value of 'type' if 'Not applicable' is ticked" do
+      form = described_class.new(project.tasks_data, user)
+      form.assign_attributes(type: "fast_track")
+      form.save
+
+      expect(project.tasks_data.sponsored_support_grant_type).to eq("fast_track")
+
+      form.assign_attributes(not_applicable: true)
+      form.save
+
+      expect(project.tasks_data.sponsored_support_grant_type).to be_nil
+    end
+  end
+end