baarkerlounger
/
submit-social-housing-lettings-and-sales-data
mirror of https://github.com/communitiesuk/submit-social-housing-lettings-and-sales-data.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity
Browse Source

feat: update ansewrs, hint text and test

pull/2192/head
natdeanlewissoftwire 2 years ago
parent
18f0c1dff9
commit
010396e49d
4 changed files with 610 additions and 244 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Unified View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 328
      app/models/form/lettings/questions/reason.rb
  2. 64
      app/models/form/lettings/questions/reason_renewal.rb
  3. 111
      spec/models/form/lettings/questions/reason_renewal_spec.rb
  4. 351
      spec/models/form/lettings/questions/reason_spec.rb

328
app/models/form/lettings/questions/reason.rb
Unescape View File

@ -6,8 +6,7 @@ class Form::Lettings::Questions::Reason < ::Form::Question
@header = "What is the tenant’s main reason for the household leaving their last settled home?" @header = "What is the tenant’s main reason for the household leaving their last settled home?"
@type = "radio" @type = "radio"
@check_answers_card_number = 0 @check_answers_card_number = 0
@hint_text = "The tenant’s ‘last settled home’ is their last long-standing home. For tenants who were in temporary accommodation or sleeping rough, their last settled home is where they were living previously." @hint_text = form.start_year_after_2024? ? "The tenant’s ‘last settled home’ is their last long-standing home. For tenants who were in temporary accommodation, sleeping rough or otherwise homeless, their last settled home is where they were living previously." : "The tenant’s ‘last settled home’ is their last long-standing home. For tenants who were in temporary accommodation or sleeping rough, their last settled home is where they were living previously."
@answer_options = ANSWER_OPTIONS
@conditional_for = { @conditional_for = {
"reasonother" => [ "reasonother" => [
20, 20,
@ -16,114 +15,219 @@ class Form::Lettings::Questions::Reason < ::Form::Question
@question_number = 77 @question_number = 77
end end
ANSWER_OPTIONS = { def answer_options
"40" => { if form.start_year_after_2024?
"value" => "End of assured shorthold tenancy (no fault)", {
}, "50" => {
"41" => { "value" => "End of social housing tenancy - no fault",
"value" => "End of assured shorthold tenancy (eviction or tenant at fault)", },
}, "51" => {
"42" => { "value" => "End of social housing tenancy - evicted due to anti-social behaviour (ASB)",
"value" => "End of fixed term tenancy (no fault)", },
}, "52" => {
"43" => { "value" => "End of social housing tenancy - evicted due to rent arrears",
"value" => "End of fixed term tenancy (eviction or tenant at fault)", },
}, "53" => {
"1" => { "value" => "End of social housing tenancy - evicted for any other reason",
"value" => "Permanently decanted from another property owned by this landlord", },
}, "1" => {
"46" => { "value" => "Permanently decanted from another property owned by this landlord",
"value" => "Discharged from long-stay hospital or similar institution", },
}, "2" => {
"45" => { "value" => "Left home country as a refugee",
"value" => "Discharged from prison", },
}, "45" => {
"2" => { "value" => "Discharged from prison",
"value" => "Left home country as a refugee", },
}, "46" => {
"4" => { "value" => "Discharged from long-stay hospital or similar institution",
"value" => "Loss of tied accommodation", },
}, "4" => {
"9" => { "value" => "Loss of tied accommodation",
"value" => "Asked to leave by family or friends", },
}, "9" => {
"44" => { "value" => "Asked to leave by family or friends",
"value" => "Death of household member in last settled accommodation", },
}, "8" => {
"8" => { "value" => "Relationship breakdown (non-violent) with partner",
"value" => "Relationship breakdown (non-violent) with partner", },
}, "44" => {
"16" => { "value" => "Death of household member in last settled accommodation",
"value" => "To move nearer to family, friends or school", },
}, "16" => {
"17" => { "value" => "To move nearer to family, friends or school",
"value" => "To move nearer to work", },
}, "17" => {
"48" => { "value" => "To move nearer to work",
"value" => "Domestic abuse - previously joint tenancy with partner", },
}, "48" => {
"49" => { "value" => "Domestic abuse - previously joint tenancy with partner",
"value" => "Domestic abuse - other", },
}, "49" => {
"31" => { "value" => "Domestic abuse - other",
"value" => "Hate crime", },
}, "10" => {
"10" => { "value" => "Racial harassment",
"value" => "Racial harassment", },
}, "31" => {
"11" => { "value" => "Hate crime",
"value" => "Other problems with neighbours", },
}, "11" => {
"35" => { "value" => "Other problems with neighbours",
"value" => "Couldn’t afford fees attached to renewing the tenancy", },
}, "34" => {
"36" => { "value" => "Repossession",
"value" => "Couldn’t afford increase in rent", },
}, "54" => {
"38" => { "value" => "Could no longer afford rent or mortgage",
"value" => "Couldn’t afford rent or mortgage (employment)", },
}, "12" => {
"37" => { "value" => "Property unsuitable because of overcrowding",
"value" => "Couldn’t afford rent or mortgage (welfare reforms)", },
}, "13" => {
"39" => { "value" => "Property unsuitable because of ill health or disability",
"value" => "Couldn’t afford rent or mortgage (other)", },
}, "14" => {
"34" => { "value" => "Property unsuitable because of poor condition",
"value" => "Repossession", },
}, "29" => {
"12" => { "value" => "Under occupation (offered incentive to downsize)",
"value" => "Property unsuitable because of overcrowding", },
}, "30" => {
"13" => { "value" => "Under occupation (no incentive)",
"value" => "Property unsuitable because of ill health or disability", },
}, "18" => {
"14" => { "value" => "To move to accommodation with support",
"value" => "Property unsuitable because of poor condition", },
}, "19" => {
"18" => { "value" => "To move to independent accommodation",
"value" => "To move to accommodation with support", },
}, "20" => {
"19" => { "value" => "Other",
"value" => "To move to independent accommodation", },
}, "28" => {
"30" => { "value" => "Don’t know",
"value" => "Under occupation (no incentive)", },
}, "divider" => {
"29" => { "value" => true,
"value" => "Under occupation (offered incentive to downsize)", },
}, "47" => {
"20" => { "value" => "Tenant prefers not to say",
"value" => "Other", },
}, }.freeze
"47" => { else
"value" => "Tenant prefers not to say", {
}, "40" => {
"divider" => { "value" => "End of assured shorthold tenancy (no fault)",
"value" => true, },
}, "41" => {
"28" => { "value" => "End of assured shorthold tenancy (eviction or tenant at fault)",
"value" => "Don’t know", },
}, "42" => {
}.freeze "value" => "End of fixed term tenancy (no fault)",
},
"43" => {
"value" => "End of fixed term tenancy (eviction or tenant at fault)",
},
"1" => {
"value" => "Permanently decanted from another property owned by this landlord",
},
"46" => {
"value" => "Discharged from long-stay hospital or similar institution",
},
"45" => {
"value" => "Discharged from prison",
},
"2" => {
"value" => "Left home country as a refugee",
},
"4" => {
"value" => "Loss of tied accommodation",
},
"9" => {
"value" => "Asked to leave by family or friends",
},
"44" => {
"value" => "Death of household member in last settled accommodation",
},
"8" => {
"value" => "Relationship breakdown (non-violent) with partner",
},
"16" => {
"value" => "To move nearer to family, friends or school",
},
"17" => {
"value" => "To move nearer to work",
},
"48" => {
"value" => "Domestic abuse - previously joint tenancy with partner",
},
"49" => {
"value" => "Domestic abuse - other",
},
"31" => {
"value" => "Hate crime",
},
"10" => {
"value" => "Racial harassment",
},
"11" => {
"value" => "Other problems with neighbours",
},
"35" => {
"value" => "Couldn’t afford fees attached to renewing the tenancy",
},
"36" => {
"value" => "Couldn’t afford increase in rent",
},
"38" => {
"value" => "Couldn’t afford rent or mortgage (employment)",
},
"37" => {
"value" => "Couldn’t afford rent or mortgage (welfare reforms)",
},
"39" => {
"value" => "Couldn’t afford rent or mortgage (other)",
},
"34" => {
"value" => "Repossession",
},
"12" => {
"value" => "Property unsuitable because of overcrowding",
},
"13" => {
"value" => "Property unsuitable because of ill health or disability",
},
"14" => {
"value" => "Property unsuitable because of poor condition",
},
"18" => {
"value" => "To move to accommodation with support",
},
"19" => {
"value" => "To move to independent accommodation",
},
"30" => {
"value" => "Under occupation (no incentive)",
},
"29" => {
"value" => "Under occupation (offered incentive to downsize)",
},
"20" => {
"value" => "Other",
},
"47" => {
"value" => "Tenant prefers not to say",
},
"divider" => {
"value" => true,
},
"28" => {
"value" => "Don’t know",
},
}.freeze
end
end
end end

64
app/models/form/lettings/questions/reason_renewal.rb
Unescape View File

@ -7,7 +7,6 @@ class Form::Lettings::Questions::ReasonRenewal < ::Form::Question
@type = "radio" @type = "radio"
@check_answers_card_number = 0 @check_answers_card_number = 0
@hint_text = "You told us this letting is a renewal. We have removed some options because of this." @hint_text = "You told us this letting is a renewal. We have removed some options because of this."
@answer_options = ANSWER_OPTIONS
@question_number = 77 @question_number = 77
@conditional_for = { @conditional_for = {
"reasonother" => [ "reasonother" => [
@ -16,20 +15,51 @@ class Form::Lettings::Questions::ReasonRenewal < ::Form::Question
} }
end end
ANSWER_OPTIONS = { def answer_options
"40" => { "value" => "End of assured shorthold tenancy (no fault)" }, if form.start_year_after_2024?
"42" => { "value" => "End of fixed term tenancy (no fault)" }, {
"20" => { "50" => {
"value" => "Other", "value" => "End of social housing tenancy - no fault",
}, },
"47" => { "51" => {
"value" => "Tenant prefers not to say", "value" => "End of social housing tenancy - evicted due to anti-social behaviour (ASB)",
}, },
"divider" => { "52" => {
"value" => true, "value" => "End of social housing tenancy - evicted due to rent arrears",
}, },
"28" => { "53" => {
"value" => "Don’t know", "value" => "End of social housing tenancy - evicted for any other reason",
}, },
}.freeze "20" => {
"value" => "Other",
},
"47" => {
"value" => "Tenant prefers not to say",
},
"divider" => {
"value" => true,
},
"28" => {
"value" => "Don’t know",
},
}.freeze
else
{
"40" => { "value" => "End of assured shorthold tenancy (no fault)" },
"42" => { "value" => "End of fixed term tenancy (no fault)" },
"20" => {
"value" => "Other",
},
"47" => {
"value" => "Tenant prefers not to say",
},
"divider" => {
"value" => true,
},
"28" => {
"value" => "Don’t know",
},
}.freeze
end
end
end end

111
spec/models/form/lettings/questions/reason_renewal_spec.rb
Unescape View File

@ -0,0 +1,111 @@
require "rails_helper"
RSpec.describe Form::Lettings::Questions::ReasonRenewal, type: :model do
subject(:question) { described_class.new(question_id, question_definition, page) }
let(:question_id) { nil }
let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) }
let(:subsection) { instance_double(Form::Subsection) }
let(:form) { instance_double(Form) }
before do
allow(form).to receive(:start_year_after_2024?).and_return(false)
allow(page).to receive(:subsection).and_return(subsection)
allow(subsection).to receive(:form).and_return(form)
end
it "has correct page" do
expect(question.page).to eq(page)
end
it "has the correct id" do
expect(question.id).to eq("reason")
end
it "has the correct header" do
expect(question.header).to eq("What is the tenant’s main reason for the household leaving their last settled home?")
end
it "has the correct check_answer_label" do
expect(question.check_answer_label).to eq("Reason for leaving last settled home")
end
it "has the correct type" do
expect(question.type).to eq("radio")
end
it "has the correct check_answers_card_number" do
expect(question.check_answers_card_number).to eq(0)
end
it "has the correct hint" do
expect(question.hint_text).to eq("You told us this letting is a renewal. We have removed some options because of this.")
end
it "has the correct conditional_for" do
expect(question.conditional_for).to eq({ "reasonother" => [20] })
end
it "is not marked as derived" do
expect(question).not_to be_derived
end
context "with 2023/24 form" do
it "has the correct answer_options" do
expect(question.answer_options).to eq({
"40" => { "value" => "End of assured shorthold tenancy (no fault)" },
"42" => { "value" => "End of fixed term tenancy (no fault)" },
"20" => {
"value" => "Other",
},
"47" => {
"value" => "Tenant prefers not to say",
},
"divider" => {
"value" => true,
},
"28" => {
"value" => "Don’t know",
},
})
end
end
context "with 2024/25 form" do
before do
allow(form).to receive(:start_year_after_2024?).and_return(true)
end
it "has the correct answer_options" do
expect(question.answer_options).to eq({
"50" => {
"value" => "End of social housing tenancy - no fault",
},
"51" => {
"value" => "End of social housing tenancy - evicted due to anti-social behaviour (ASB)",
},
"52" => {
"value" => "End of social housing tenancy - evicted due to rent arrears",
},
"53" => {
"value" => "End of social housing tenancy - evicted for any other reason",
},
"20" => {
"value" => "Other",
},
"47" => {
"value" => "Tenant prefers not to say",
},
"divider" => {
"value" => true,
},
"28" => {
"value" => "Don’t know",
},
})
end
end
end

351
spec/models/form/lettings/questions/reason_spec.rb
Unescape View File

@ -6,6 +6,14 @@ RSpec.describe Form::Lettings::Questions::Reason, type: :model do
let(:question_id) { nil } let(:question_id) { nil }
let(:question_definition) { nil } let(:question_definition) { nil }
let(:page) { instance_double(Form::Page) } let(:page) { instance_double(Form::Page) }
let(:subsection) { instance_double(Form::Subsection) }
let(:form) { instance_double(Form) }
before do
allow(form).to receive(:start_year_after_2024?).and_return(false)
allow(page).to receive(:subsection).and_return(subsection)
allow(subsection).to receive(:form).and_return(form)
end
it "has correct page" do it "has correct page" do
expect(question.page).to eq(page) expect(question.page).to eq(page)
@ -31,10 +39,6 @@ RSpec.describe Form::Lettings::Questions::Reason, type: :model do
expect(question.check_answers_card_number).to eq(0) expect(question.check_answers_card_number).to eq(0)
end end
it "has the correct hint" do
expect(question.hint_text).to eq("The tenant’s ‘last settled home’ is their last long-standing home. For tenants who were in temporary accommodation or sleeping rough, their last settled home is where they were living previously.")
end
it "has the correct conditional_for" do it "has the correct conditional_for" do
expect(question.conditional_for).to eq({ "reasonother" => [20] }) expect(question.conditional_for).to eq({ "reasonother" => [20] })
end end
@ -43,116 +47,233 @@ RSpec.describe Form::Lettings::Questions::Reason, type: :model do
expect(question).not_to be_derived expect(question).not_to be_derived
end end
it "has the correct answer_options" do context "with 2023/24 form" do
expect(question.answer_options).to eq({ it "has the correct hint" do
"40" => { expect(question.hint_text).to eq("The tenant’s ‘last settled home’ is their last long-standing home. For tenants who were in temporary accommodation or sleeping rough, their last settled home is where they were living previously.")
"value" => "End of assured shorthold tenancy (no fault)", end
},
"41" => { it "has the correct answer_options" do
"value" => "End of assured shorthold tenancy (eviction or tenant at fault)", expect(question.answer_options).to eq({
}, "40" => {
"42" => { "value" => "End of assured shorthold tenancy (no fault)",
"value" => "End of fixed term tenancy (no fault)", },
}, "41" => {
"43" => { "value" => "End of assured shorthold tenancy (eviction or tenant at fault)",
"value" => "End of fixed term tenancy (eviction or tenant at fault)", },
}, "42" => {
"1" => { "value" => "End of fixed term tenancy (no fault)",
"value" => "Permanently decanted from another property owned by this landlord", },
}, "43" => {
"46" => { "value" => "End of fixed term tenancy (eviction or tenant at fault)",
"value" => "Discharged from long-stay hospital or similar institution", },
}, "1" => {
"45" => { "value" => "Permanently decanted from another property owned by this landlord",
"value" => "Discharged from prison", },
}, "46" => {
"2" => { "value" => "Discharged from long-stay hospital or similar institution",
"value" => "Left home country as a refugee", },
}, "45" => {
"4" => { "value" => "Discharged from prison",
"value" => "Loss of tied accommodation", },
}, "2" => {
"9" => { "value" => "Left home country as a refugee",
"value" => "Asked to leave by family or friends", },
}, "4" => {
"44" => { "value" => "Loss of tied accommodation",
"value" => "Death of household member in last settled accommodation", },
}, "9" => {
"8" => { "value" => "Asked to leave by family or friends",
"value" => "Relationship breakdown (non-violent) with partner", },
}, "44" => {
"16" => { "value" => "Death of household member in last settled accommodation",
"value" => "To move nearer to family, friends or school", },
}, "8" => {
"17" => { "value" => "Relationship breakdown (non-violent) with partner",
"value" => "To move nearer to work", },
}, "16" => {
"48" => { "value" => "To move nearer to family, friends or school",
"value" => "Domestic abuse - previously joint tenancy with partner", },
}, "17" => {
"49" => { "value" => "To move nearer to work",
"value" => "Domestic abuse - other", },
}, "48" => {
"31" => { "value" => "Domestic abuse - previously joint tenancy with partner",
"value" => "Hate crime", },
}, "49" => {
"10" => { "value" => "Domestic abuse - other",
"value" => "Racial harassment", },
}, "31" => {
"11" => { "value" => "Hate crime",
"value" => "Other problems with neighbours", },
}, "10" => {
"35" => { "value" => "Racial harassment",
"value" => "Couldn’t afford fees attached to renewing the tenancy", },
}, "11" => {
"36" => { "value" => "Other problems with neighbours",
"value" => "Couldn’t afford increase in rent", },
}, "35" => {
"38" => { "value" => "Couldn’t afford fees attached to renewing the tenancy",
"value" => "Couldn’t afford rent or mortgage (employment)", },
}, "36" => {
"37" => { "value" => "Couldn’t afford increase in rent",
"value" => "Couldn’t afford rent or mortgage (welfare reforms)", },
}, "38" => {
"39" => { "value" => "Couldn’t afford rent or mortgage (employment)",
"value" => "Couldn’t afford rent or mortgage (other)", },
}, "37" => {
"34" => { "value" => "Couldn’t afford rent or mortgage (welfare reforms)",
"value" => "Repossession", },
}, "39" => {
"12" => { "value" => "Couldn’t afford rent or mortgage (other)",
"value" => "Property unsuitable because of overcrowding", },
}, "34" => {
"13" => { "value" => "Repossession",
"value" => "Property unsuitable because of ill health or disability", },
}, "12" => {
"14" => { "value" => "Property unsuitable because of overcrowding",
"value" => "Property unsuitable because of poor condition", },
}, "13" => {
"18" => { "value" => "Property unsuitable because of ill health or disability",
"value" => "To move to accommodation with support", },
}, "14" => {
"19" => { "value" => "Property unsuitable because of poor condition",
"value" => "To move to independent accommodation", },
}, "18" => {
"30" => { "value" => "To move to accommodation with support",
"value" => "Under occupation (no incentive)", },
}, "19" => {
"29" => { "value" => "To move to independent accommodation",
"value" => "Under occupation (offered incentive to downsize)", },
}, "30" => {
"20" => { "value" => "Under occupation (no incentive)",
"value" => "Other", },
}, "29" => {
"47" => { "value" => "Under occupation (offered incentive to downsize)",
"value" => "Tenant prefers not to say", },
}, "20" => {
"divider" => { "value" => "Other",
"value" => true, },
}, "47" => {
"28" => { "value" => "Tenant prefers not to say",
"value" => "Don’t know", },
}, "divider" => {
}) "value" => true,
},
"28" => {
"value" => "Don’t know",
},
})
end
end
context "with 2024/25 form" do
before do
allow(form).to receive(:start_year_after_2024?).and_return(true)
end
it "has the correct hint" do
expect(question.hint_text).to eq("The tenant’s ‘last settled home’ is their last long-standing home. For tenants who were in temporary accommodation, sleeping rough or otherwise homeless, their last settled home is where they were living previously.")
end
it "has the correct answer_options" do
expect(question.answer_options).to eq({
"50" => {
"value" => "End of social housing tenancy - no fault",
},
"51" => {
"value" => "End of social housing tenancy - evicted due to anti-social behaviour (ASB)",
},
"52" => {
"value" => "End of social housing tenancy - evicted due to rent arrears",
},
"53" => {
"value" => "End of social housing tenancy - evicted for any other reason",
},
"1" => {
"value" => "Permanently decanted from another property owned by this landlord",
},
"2" => {
"value" => "Left home country as a refugee",
},
"45" => {
"value" => "Discharged from prison",
},
"46" => {
"value" => "Discharged from long-stay hospital or similar institution",
},
"4" => {
"value" => "Loss of tied accommodation",
},
"9" => {
"value" => "Asked to leave by family or friends",
},
"8" => {
"value" => "Relationship breakdown (non-violent) with partner",
},
"44" => {
"value" => "Death of household member in last settled accommodation",
},
"16" => {
"value" => "To move nearer to family, friends or school",
},
"17" => {
"value" => "To move nearer to work",
},
"48" => {
"value" => "Domestic abuse - previously joint tenancy with partner",
},
"49" => {
"value" => "Domestic abuse - other",
},
"10" => {
"value" => "Racial harassment",
},
"31" => {
"value" => "Hate crime",
},
"11" => {
"value" => "Other problems with neighbours",
},
"34" => {
"value" => "Repossession",
},
"54" => {
"value" => "Could no longer afford rent or mortgage",
},
"12" => {
"value" => "Property unsuitable because of overcrowding",
},
"13" => {
"value" => "Property unsuitable because of ill health or disability",
},
"14" => {
"value" => "Property unsuitable because of poor condition",
},
"29" => {
"value" => "Under occupation (offered incentive to downsize)",
},
"30" => {
"value" => "Under occupation (no incentive)",
},
"18" => {
"value" => "To move to accommodation with support",
},
"19" => {
"value" => "To move to independent accommodation",
},
"20" => {
"value" => "Other",
},
"28" => {
"value" => "Don’t know",
},
"divider" => {
"value" => true,
},
"47" => {
"value" => "Tenant prefers not to say",
},
})
end
end end
end end

Write Preview
Loading…
Cancel
Save