Browse Source

correct linting errors and play a little code golf

pull/1101/head
Arthur Campbell 3 years ago
parent
commit
58f5658ed0
  1. 18
      spec/models/validations/sales/financial_validations_spec.rb

18
spec/models/validations/sales/financial_validations_spec.rb

@ -8,7 +8,7 @@ RSpec.describe Validations::Sales::FinancialValidations do
describe "income validations for shared ownership" do describe "income validations for shared ownership" do
let(:record) { FactoryBot.create(:sales_log, ownershipsch: 1) } let(:record) { FactoryBot.create(:sales_log, ownershipsch: 1) }
context "in a non london borough" do context "when buying in a non london borough" do
before do before do
record.update!(la: "E08000035") record.update!(la: "E08000035")
record.reload record.reload
@ -78,7 +78,7 @@ RSpec.describe Validations::Sales::FinancialValidations do
end end
end end
context "in a london borough" do context "when buying in a london borough" do
before do before do
record.update!(la: "E09000030") record.update!(la: "E09000030")
record.reload record.reload
@ -167,7 +167,7 @@ RSpec.describe Validations::Sales::FinancialValidations do
it "does not add an error if the cash discount is in the expected range" do it "does not add an error if the cash discount is in the expected range" do
record.cashdis = 10_000 record.cashdis = 10_000
financial_validator.validate_cash_discount(record) financial_validator.validate_cash_discount(record)
expect(record.errors["cashdis"]).to be_empty expect(record.errors).to be_empty
end end
end end
@ -178,16 +178,14 @@ RSpec.describe Validations::Sales::FinancialValidations do
record.stairbought = 20 record.stairbought = 20
record.stairowned = 40 record.stairowned = 40
financial_validator.validate_percentage_bought_not_greater_than_percentage_owned(record) financial_validator.validate_percentage_bought_not_greater_than_percentage_owned(record)
expect(record.errors["stairbought"]).to be_empty expect(record.errors).to be_empty
expect(record.errors["stairowned"]).to be_empty
end end
it "does not add an error if the percentage bought is equal to the percentage owned" do it "does not add an error if the percentage bought is equal to the percentage owned" do
record.stairbought = 30 record.stairbought = 30
record.stairowned = 30 record.stairowned = 30
financial_validator.validate_percentage_bought_not_greater_than_percentage_owned(record) financial_validator.validate_percentage_bought_not_greater_than_percentage_owned(record)
expect(record.errors["stairbought"]).to be_empty expect(record.errors).to be_empty
expect(record.errors["stairowned"]).to be_empty
end end
it "adds an error to stairowned and not stairbought if the percentage bought is more than the percentage owned" do it "adds an error to stairowned and not stairbought if the percentage bought is more than the percentage owned" do
@ -206,8 +204,7 @@ RSpec.describe Validations::Sales::FinancialValidations do
record.type = 2 record.type = 2
record.stairowned = 80 record.stairowned = 80
financial_validator.validate_percentage_owned_not_too_much_if_older_person(record) financial_validator.validate_percentage_owned_not_too_much_if_older_person(record)
expect(record.errors["stairowned"]).to be_empty expect(record.errors).to be_empty
expect(record.errors["type"]).to be_empty
end end
end end
@ -216,8 +213,7 @@ RSpec.describe Validations::Sales::FinancialValidations do
record.type = 24 record.type = 24
record.stairowned = 50 record.stairowned = 50
financial_validator.validate_percentage_owned_not_too_much_if_older_person(record) financial_validator.validate_percentage_owned_not_too_much_if_older_person(record)
expect(record.errors["stairowned"]).to be_empty expect(record.errors).to be_empty
expect(record.errors["type"]).to be_empty
end end
it "adds an error when percentage owned after staircasing transaction exceeds 75%" do it "adds an error when percentage owned after staircasing transaction exceeds 75%" do

Loading…
Cancel
Save