Browse Source

Add frequency to ecstat error (#2046)

pull/2050/head
kosiakkatrina 1 year ago committed by GitHub
parent
commit
e778771690
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 3
      app/models/validations/financial_validations.rb
  2. 2
      config/locales/en.yml
  3. 2
      spec/models/validations/financial_validations_spec.rb

3
app/models/validations/financial_validations.rb

@ -27,6 +27,7 @@ module Validations::FinancialValidations
if record.ecstat1 && record.weekly_net_income if record.ecstat1 && record.weekly_net_income
if record.weekly_net_income > record.applicable_income_range.hard_max if record.weekly_net_income > record.applicable_income_range.hard_max
hard_max = format_as_currency(record.applicable_income_range.hard_max) hard_max = format_as_currency(record.applicable_income_range.hard_max)
frequency = record.form.get_question("incfreq", record).label_from_value(record.incfreq).downcase
record.errors.add( record.errors.add(
:earnings, :earnings,
:over_hard_max, :over_hard_max,
@ -35,7 +36,7 @@ module Validations::FinancialValidations
record.errors.add( record.errors.add(
:ecstat1, :ecstat1,
:over_hard_max, :over_hard_max,
message: I18n.t("validations.financial.ecstat.over_hard_max", hard_max:), message: I18n.t("validations.financial.ecstat.over_hard_max", earnings: format_as_currency(record.earnings), frequency:),
) )
end end

2
config/locales/en.yml

@ -371,7 +371,7 @@ en:
less_than_shortfall: "Enter an amount that is more than the shortfall in basic rent" less_than_shortfall: "Enter an amount that is more than the shortfall in basic rent"
out_of_range: "Enter a value for the %{charge_name} between £0 and %{maximum_per_period} paid %{frequency}. %{maximum_per_period} is the max limit for rent and charges paid %{frequency} for %{letting_type} lettings owned by a %{provider_type}." out_of_range: "Enter a value for the %{charge_name} between £0 and %{maximum_per_period} paid %{frequency}. %{maximum_per_period} is the max limit for rent and charges paid %{frequency} for %{letting_type} lettings owned by a %{provider_type}."
ecstat: ecstat:
over_hard_max: "Net income of %{hard_max} per week is too high given the tenant’s working situation" over_hard_max: "Net income of %{earnings} %{frequency} is too high given the tenant’s working situation"
brent: brent:
below_hard_min: "Rent is below the absolute minimum expected for a property of this type. Please check the rent, rent period, local authority and (if general needs) number of bedrooms" below_hard_min: "Rent is below the absolute minimum expected for a property of this type. Please check the rent, rent period, local authority and (if general needs) number of bedrooms"
above_hard_max: "Rent is higher than the absolute maximum expected for a property of this type. Please check the rent, rent period, local authority and (if general needs) number of bedrooms" above_hard_max: "Rent is higher than the absolute maximum expected for a property of this type. Please check the rent, rent period, local authority and (if general needs) number of bedrooms"

2
spec/models/validations/financial_validations_spec.rb

@ -203,6 +203,8 @@ RSpec.describe Validations::FinancialValidations do
financial_validator.validate_net_income(record) financial_validator.validate_net_income(record)
expect(record.errors["earnings"]) expect(record.errors["earnings"])
.to eq(["Net income cannot be greater than £1,230.00 per week given the tenant’s working situation"]) .to eq(["Net income cannot be greater than £1,230.00 per week given the tenant’s working situation"])
expect(record.errors["ecstat1"])
.to eq(["Net income of £5,000.00 weekly is too high given the tenant’s working situation"])
end end
end end

Loading…
Cancel
Save