Browse Source

CLDC-4248: change waityear and layear response copy (#3217)

* CLDC-4236: make review deploy manual

* CLDC-4236: use pr comment trigger

* CLDC-4236: refactor

* CLDC-4248: less than > under in waityear responses

* CLDC-4248: make change for 25/26 also, and update layear too

* CLDC-4248: revert changes from other branch

* CLDC-4248: update csv tests
CLDC-4236-temp-revert-pipeline-updates
Nat Dean-Lewis 1 week ago committed by GitHub
parent
commit
e6962f3222
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 15
      app/models/form/lettings/questions/layear.rb
  2. 2
      app/models/form/lettings/questions/waityear.rb
  3. 2
      spec/fixtures/files/lettings_log_csv_export_labels_25.csv
  4. 2
      spec/fixtures/files/lettings_log_csv_export_labels_26.csv
  5. 2
      spec/fixtures/files/lettings_log_csv_export_non_support_labels_25.csv
  6. 2
      spec/fixtures/files/lettings_log_csv_export_non_support_labels_26.csv
  7. 23
      spec/models/form/lettings/questions/layear_spec.rb
  8. 2
      spec/models/form/lettings/questions/waityear_spec.rb

15
app/models/form/lettings/questions/layear.rb

@ -8,7 +8,20 @@ class Form::Lettings::Questions::Layear < ::Form::Question
end
def answer_options
if form.start_year_2024_or_later?
if form.start_year_2025_or_later?
{
"1" => { "value" => "Just moved to local authority area with this new let" },
"2" => { "value" => "Under 1 year" },
"7" => { "value" => "1 year but under 2 years" },
"8" => { "value" => "2 years but under 3 years" },
"9" => { "value" => "3 years but under 4 years" },
"10" => { "value" => "4 years but under 5 years" },
"11" => { "value" => "5 years but under 10 years" },
"12" => { "value" => "10 years or more" },
"divider" => { "value" => true },
"6" => { "value" => "Don’t know" },
}
elsif form.start_year_2024_or_later?
{
"1" => { "value" => "Just moved to local authority area with this new let" },
"2" => { "value" => "Less than 1 year" },

2
app/models/form/lettings/questions/waityear.rb

@ -11,7 +11,7 @@ class Form::Lettings::Questions::Waityear < ::Form::Question
if form.start_year_2025_or_later?
return {
"13" => { "value" => "Household not on the housing register (or waiting list) in this area" },
"2" => { "value" => "Less than 1 year" },
"2" => { "value" => "Under 1 year" },
"7" => { "value" => "1 year but under 2 years" },
"8" => { "value" => "2 years but under 3 years" },
"9" => { "value" => "3 years but under 4 years" },

2
spec/fixtures/files/lettings_log_csv_export_labels_25.csv vendored

File diff suppressed because one or more lines are too long

2
spec/fixtures/files/lettings_log_csv_export_labels_26.csv vendored

File diff suppressed because one or more lines are too long

2
spec/fixtures/files/lettings_log_csv_export_non_support_labels_25.csv vendored

File diff suppressed because one or more lines are too long

2
spec/fixtures/files/lettings_log_csv_export_non_support_labels_26.csv vendored

File diff suppressed because one or more lines are too long

23
spec/models/form/lettings/questions/layear_spec.rb

@ -10,7 +10,7 @@ RSpec.describe Form::Lettings::Questions::Layear, type: :model do
let(:form) { instance_double(Form, start_date: Time.zone.local(2023, 4, 1)) }
before do
allow(form).to receive(:start_year_2024_or_later?).and_return(false)
allow(form).to receive_messages(start_year_2024_or_later?: false, start_year_2025_or_later?: false)
allow(page).to receive(:subsection).and_return(subsection)
allow(subsection).to receive(:form).and_return(form)
end
@ -68,6 +68,27 @@ RSpec.describe Form::Lettings::Questions::Layear, type: :model do
end
end
context "with 2025/26 form" do
before do
allow(form).to receive(:start_year_2025_or_later?).and_return(true)
end
it "has the correct answer_options" do
expect(question.answer_options).to eq({
"1" => { "value" => "Just moved to local authority area with this new let" },
"2" => { "value" => "Under 1 year" },
"7" => { "value" => "1 year but under 2 years" },
"8" => { "value" => "2 years but under 3 years" },
"9" => { "value" => "3 years but under 4 years" },
"10" => { "value" => "4 years but under 5 years" },
"11" => { "value" => "5 years but under 10 years" },
"12" => { "value" => "10 years or more" },
"divider" => { "value" => true },
"6" => { "value" => "Don’t know" },
})
end
end
it "has the correct check_answers_card_number" do
expect(question.check_answers_card_number).to eq(0)
end

2
spec/models/form/lettings/questions/waityear_spec.rb

@ -85,7 +85,7 @@ RSpec.describe Form::Lettings::Questions::Waityear, type: :model do
it "has the correct answer_options" do
expect(question.answer_options).to eq({
"13" => { "value" => "Household not on the housing register (or waiting list) in this area" },
"2" => { "value" => "Less than 1 year" },
"2" => { "value" => "Under 1 year" },
"7" => { "value" => "1 year but under 2 years" },
"8" => { "value" => "2 years but under 3 years" },
"9" => { "value" => "3 years but under 4 years" },

Loading…
Cancel
Save