Validate BU headers

app/services/bulk_upload/lettings/validator.rb

@@ -7,6 +7,7 @@ class BulkUpload::Lettings::Validator
   validate :validate_field_numbers_count
   validate :validate_max_columns_count_if_no_headers
   validate :validate_correct_template
+  validate :validate_missing_required_headers
 
   def initialize(bulk_upload:, path:)
     @bulk_upload = bulk_upload
@@ -175,6 +176,12 @@ private
     errors.add(:base, :wrong_template) if csv_parser.wrong_template_for_year?
   end
 
+  def validate_missing_required_headers
+    return if halt_validations?
+
+    errors.add :base, I18n.t("bulk_upload.lettings.validator.attributes.base.no_headers", guidance_link: Forms::BulkUploadLettings::Guidance.new(year: bulk_upload.year).view_path) if csv_parser.missing_required_headers?
+  end
+
   def halt_validations!
     @halt_validations = true
   end

app/services/bulk_upload/lettings/year2023/csv_parser.rb

@@ -70,6 +70,10 @@ class BulkUpload::Lettings::Year2023::CsvParser
     false
   end
 
+  def missing_required_headers?
+    false
+  end
+
 private
 
   def default_field_numbers

app/services/bulk_upload/lettings/year2024/csv_parser.rb

@@ -70,6 +70,10 @@ class BulkUpload::Lettings::Year2024::CsvParser
     false
   end
 
+  def missing_required_headers?
+    !with_headers?
+  end
+
 private
 
   def default_field_numbers

app/services/bulk_upload/sales/validator.rb

@@ -6,6 +6,7 @@ class BulkUpload::Sales::Validator
   validate :validate_file_not_empty
   validate :validate_max_columns
   validate :validate_correct_template
+  validate :validate_missing_required_headers
 
   def initialize(bulk_upload:, path:)
     @bulk_upload = bulk_upload
@@ -159,6 +160,12 @@ private
     errors.add(:base, :wrong_template) if csv_parser.wrong_template_for_year?
   end
 
+  def validate_missing_required_headers
+    return if halt_validations?
+
+    errors.add :base, I18n.t("bulk_upload.sales.validator.attributes.base.no_headers", guidance_link: Forms::BulkUploadSales::Guidance.new(year: bulk_upload.year).view_path) if csv_parser.missing_required_headers?
+  end
+
   def halt_validations!
     @halt_validations = true
   end

app/services/bulk_upload/sales/year2023/csv_parser.rb

@@ -55,6 +55,10 @@ class BulkUpload::Sales::Year2023::CsvParser
     false
   end
 
+  def missing_required_headers?
+    false
+  end
+
 private
 
   def default_field_numbers

app/services/bulk_upload/sales/year2024/csv_parser.rb

@@ -55,6 +55,10 @@ class BulkUpload::Sales::Year2024::CsvParser
     false
   end
 
+  def missing_required_headers?
+    !with_headers?
+  end
+
 private
 
   def default_field_numbers

config/locales/en.yml

@@ -59,6 +59,7 @@ en:
               wrong_field_numbers_count: "Incorrect number of fields, please ensure you have used the correct template"
               over_max_column_count: "Too many columns, please ensure you have used the correct template"
               wrong_template: "Incorrect start dates, please ensure you have used the correct template"
+              no_headers: "Your file does not contain the required header rows. Add or check the header rows and upload your file again. [Read more about using the template headers](%{guidance_link})."
         bulk_upload/sales/validator:
           attributes:
             base:
@@ -66,6 +67,7 @@ en:
               wrong_field_numbers_count: "Incorrect number of fields, please ensure you have used the correct template"
               over_max_column_count: "Too many columns, please ensure you have used the correct template"
               wrong_template: "Incorrect sale dates, please ensure you have used the correct template"
+              no_headers: "Your file does not contain the required header rows. Add or check the header rows and upload your file again. [Read more about using the template headers](%{guidance_link})."
         forms/bulk_upload_lettings/year:
           attributes:
             year:

spec/services/bulk_upload/sales/validator_spec.rb

@@ -88,6 +88,22 @@ RSpec.describe BulkUpload::Sales::Validator do
         end
       end
     end
+
+    context "when file is missing required headers" do
+      let(:bulk_upload) { create(:bulk_upload, user:, year: 2024) }
+      let(:log) { build(:sales_log, :completed, saledate: Time.zone.local(2024, 5, 5)) }
+      let(:file) { Tempfile.new }
+      let(:path) { file.path }
+
+      before do
+        file.write(BulkUpload::SalesLogToCsv.new(log:, line_ending: "\r\n", col_offset: 0).to_2024_csv_row)
+        file.close
+      end
+
+      it "is not valid" do
+        expect(validator).not_to be_valid
+      end
+    end
   end
 
   describe "#call" do