fixup! CLDC-4162: Update tests for existing years

properly mark year specific tests

spec/services/bulk_upload/lettings/validator_spec.rb

@@ -159,18 +159,40 @@ RSpec.describe BulkUpload::Lettings::Validator do
         expect { validator.call }.to change(BulkUploadError, :count)
       end
 
-      it "create validation error with correct values" do
+      context "and in 2025", metadata: { year: 2025 } do
-        validator.call
+        let(:year) { 2025 }
 
-        error = BulkUploadError.find_by(field: "field_4")
+        it "create validation error with correct values" do
+          validator.call
 
-        expect(error.field).to eql("field_4")
+          error = BulkUploadError.find_by(field: "field_4")
-        expect(error.error).to eql("You must answer needs type.")
+
-        expect(error.tenant_code).to eql(log.tenancycode)
+          expect(error.field).to eql("field_4")
-        expect(error.property_ref).to eql(log.propcode)
+          expect(error.error).to eql("You must answer needs type.")
-        expect(error.row).to eql("2")
+          expect(error.tenant_code).to eql(log.tenancycode)
-        expect(error.cell).to eql("CX2")
+          expect(error.property_ref).to eql(log.propcode)
-        expect(error.col).to eql("CX")
+          expect(error.row).to eql("2")
+          expect(error.cell).to eql("CX2")
+          expect(error.col).to eql("CX")
+        end
+      end
+
+      context "and in 2026", metadata: { year: 2026 } do
+        let(:year) { 2026 }
+
+        it "create validation error with correct values" do
+          validator.call
+
+          error = BulkUploadError.find_by(field: "field_4")
+
+          expect(error.field).to eql("field_4")
+          expect(error.error).to eql("You must answer needs type.")
+          expect(error.tenant_code).to eql(log.tenancycode)
+          expect(error.property_ref).to eql(log.propcode)
+          expect(error.row).to eql("2")
+          expect(error.cell).to eql("CX2")
+          expect(error.col).to eql("CX")
+        end
       end
     end
 

spec/services/bulk_upload/sales/validator_spec.rb

@@ -136,17 +136,38 @@ RSpec.describe BulkUpload::Sales::Validator do
         expect { validator.call }.to change(BulkUploadError, :count)
       end
 
-      it "create validation error with correct values" do
+      context "and in 2025", metadata: { year: 2025 } do
-        validator.call
+        let(:year) { 2025 }
+
+        it "create validation error with correct values" do
+          validator.call
+
+          error = BulkUploadError.find_by(row: "2", field: "field_4", category: "setup")
 
-        error = BulkUploadError.find_by(row: "2", field: "field_4", category: "setup")
+          expect(error.field).to eql("field_4")
+          expect(error.error).to eql("You must answer owning organisation.")
+          expect(error.purchaser_code).to eql(log.purchaser_code)
+          expect(error.row).to eql("2")
+          expect(error.cell).to eql("E2")
+          expect(error.col).to eql("E")
+        end
+      end
 
-        expect(error.field).to eql("field_4")
+      context "and in 2026", metadata: { year: 2026 } do
-        expect(error.error).to eql("You must answer owning organisation.")
+        let(:year) { 2026 }
-        expect(error.purchaser_code).to eql(log.purchaser_code)
+
-        expect(error.row).to eql("2")
+        it "create validation error with correct values" do
-        expect(error.cell).to eql("E2")
+          validator.call
-        expect(error.col).to eql("E")
+
+          error = BulkUploadError.find_by(row: "2", field: "field_4", category: "setup")
+
+          expect(error.field).to eql("field_4")
+          expect(error.error).to eql("You must answer owning organisation.")
+          expect(error.purchaser_code).to eql(log.purchaser_code)
+          expect(error.row).to eql("2")
+          expect(error.cell).to eql("E2")
+          expect(error.col).to eql("E")
+        end
       end
     end