Browse Source

test: put test within context block for clarity

pull/1524/head
Sam Seed 3 years ago
parent
commit
7a4b189c04
  1. 12
      spec/services/bulk_upload/lettings/year2023/row_parser_spec.rb

12
spec/services/bulk_upload/lettings/year2023/row_parser_spec.rb

@ -238,12 +238,14 @@ RSpec.describe BulkUpload::Lettings::Year2023::RowParser do
end end
describe "#validate_nulls" do describe "#validate_nulls" do
let(:attributes) { { bulk_upload:, field_18: "", field_19: "", field_21: "" } } context "for non-setup questions" do
let(:attributes) { { bulk_upload:, field_1: "", field_18: "", field_19: "", field_21: "" } }
it "fetches the question's check_answer_label if it exists, otherwise it get's the question's header" do it "fetches the question's check_answer_label if it exists, otherwise it get's the question's header" do
parser.valid? parser.valid?
expect(parser.errors[:field_19]).to eql(["You must answer q12 - address"]) expect(parser.errors[:field_19]).to eql(["You must answer q12 - address"])
expect(parser.errors[:field_21]).to eql(["You must answer town or city"]) expect(parser.errors[:field_21]).to eql(["You must answer town or city"])
end
end end
end end
end end

Loading…
Cancel
Save