From e374687bea14ae84c7802714a321d355e46035e1 Mon Sep 17 00:00:00 2001
From: Manny Dinssa <44172848+Dinssa@users.noreply.github.com>
Date: Thu, 20 Feb 2025 16:30:17 +0000
Subject: [PATCH] Add test

---
 .../lettings/year2024/row_parser_spec.rb       | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/spec/services/bulk_upload/lettings/year2024/row_parser_spec.rb b/spec/services/bulk_upload/lettings/year2024/row_parser_spec.rb
index 33ab390f4..e07d42453 100644
--- a/spec/services/bulk_upload/lettings/year2024/row_parser_spec.rb
+++ b/spec/services/bulk_upload/lettings/year2024/row_parser_spec.rb
@@ -1247,7 +1247,7 @@ RSpec.describe BulkUpload::Lettings::Year2024::RowParser do
         end
       end
 
-      context "when some reasonable preference options are seleceted" do
+      context "when some reasonable preference options are selected" do
         let(:attributes) { setup_section_params.merge({ bulk_upload:, field_106: "1", field_107: "1", field_108: nil, field_109: "1", field_110: nil, field_111: nil }) }
 
         it "sets the rest of the options to 0" do
@@ -1260,7 +1260,7 @@ RSpec.describe BulkUpload::Lettings::Year2024::RowParser do
         end
       end
 
-      context "when some reasonable preference options are seleceted but reasonpref is No" do
+      context "when some reasonable preference options are selected but reasonpref is No" do
         let(:attributes) { setup_section_params.merge({ bulk_upload:, field_106: "2", field_107: "1", field_108: nil, field_109: "1", field_110: nil, field_111: nil }) }
 
         it "sets the options to nil" do
@@ -1272,6 +1272,16 @@ RSpec.describe BulkUpload::Lettings::Year2024::RowParser do
           expect(parser.log.rp_dontknow).to be_nil
         end
       end
+
+      context "when reasonpref is Yes, some reasonable preferences are selected but also so is 'Don't know'" do
+        let(:attributes) { setup_section_params.merge({ bulk_upload:, field_106: "1", field_107: "1", field_108: "1", field_109: nil, field_110: nil, field_111: "1" }) }
+
+        it "is not permitted" do
+          parser.valid?
+          expect(parser.errors[:field_111]).to be_present
+          expect(parser.errors[:field_111]).to include(I18n.t("validations.lettings.2024.bulk_upload.reasonpref.conflict"))
+        end
+      end
     end
 
     describe "#field_116" do # referral
@@ -2511,7 +2521,7 @@ RSpec.describe BulkUpload::Lettings::Year2024::RowParser do
         end
       end
 
-      context "when some illness type values are seleceted" do
+      context "when some illness type values are selected" do
         let(:attributes) { setup_section_params.merge({ bulk_upload:, field_85: "1", field_94: "1", field_87: "1" }) }
 
         it "sets the rest of the values to 0" do
@@ -2529,7 +2539,7 @@ RSpec.describe BulkUpload::Lettings::Year2024::RowParser do
         end
       end
 
-      context "when none of the illness type values are seleceted" do
+      context "when none of the illness type values are selected" do
         let(:attributes) { setup_section_params.merge({ bulk_upload:, field_85: "1" }) }
 
         it "sets the values to nil" do