From 175d58c5575596dfa5d67f26b0837e2013985e0f Mon Sep 17 00:00:00 2001 From: natdeanlewissoftwire Date: Thu, 14 Mar 2024 10:29:55 +0000 Subject: [PATCH] feat: test routing --- .../form/lettings/pages/property_local_authority.rb | 2 +- app/models/form/sales/pages/uprn_selection.rb | 3 ++- .../form/lettings/pages/no_address_found_spec.rb | 10 +++++++++- .../lettings/pages/property_local_authority_spec.rb | 4 +++- spec/models/form/lettings/pages/uprn_selection_spec.rb | 10 +++++++++- spec/models/form/sales/pages/no_address_found_spec.rb | 4 +++- .../form/sales/pages/property_local_authority_spec.rb | 4 +--- spec/models/form/sales/pages/uprn_selection_spec.rb | 4 +++- 8 files changed, 31 insertions(+), 10 deletions(-) diff --git a/app/models/form/lettings/pages/property_local_authority.rb b/app/models/form/lettings/pages/property_local_authority.rb index 9e36030a2..9b36b0936 100644 --- a/app/models/form/lettings/pages/property_local_authority.rb +++ b/app/models/form/lettings/pages/property_local_authority.rb @@ -3,7 +3,7 @@ class Form::Lettings::Pages::PropertyLocalAuthority < ::Form::Page super @id = "property_local_authority" @depends_on = [ - { "is_la_inferred" => false, "is_general_needs?" => true, "address_search_given?" => true } + { "is_la_inferred" => false, "is_general_needs?" => true, "address_search_given?" => true }, ] end diff --git a/app/models/form/sales/pages/uprn_selection.rb b/app/models/form/sales/pages/uprn_selection.rb index 9848ee02c..1c7f251ac 100644 --- a/app/models/form/sales/pages/uprn_selection.rb +++ b/app/models/form/sales/pages/uprn_selection.rb @@ -7,7 +7,8 @@ class Form::Sales::Pages::UprnSelection < ::Form::Page { "uprn_known" => nil, "address_options_present?" => true }, { "uprn_known" => 0, "address_options_present?" => true }, { "uprn_confirmed" => 0, "address_options_present?" => true }, - ] end + ] + end def questions @questions ||= [ diff --git a/spec/models/form/lettings/pages/no_address_found_spec.rb b/spec/models/form/lettings/pages/no_address_found_spec.rb index 34ef22b2a..3f5dd7695 100644 --- a/spec/models/form/lettings/pages/no_address_found_spec.rb +++ b/spec/models/form/lettings/pages/no_address_found_spec.rb @@ -25,7 +25,15 @@ RSpec.describe Form::Lettings::Pages::NoAddressFound, type: :model do end it "has correct depends_on" do - expect(page.depends_on).to eq([{ "address_options_present?" => false }]) + expect(page.depends_on).to eq([{ "address_options_present?" => false, + "is_supported_housing?" => false, + "uprn_known" => nil }, + { "address_options_present?" => false, + "is_supported_housing?" => false, + "uprn_known" => 0 }, + { "address_options_present?" => false, + "is_supported_housing?" => false, + "uprn_confirmed" => 0 }]) end it "is interruption screen page" do diff --git a/spec/models/form/lettings/pages/property_local_authority_spec.rb b/spec/models/form/lettings/pages/property_local_authority_spec.rb index c09d01dc7..1936587b0 100644 --- a/spec/models/form/lettings/pages/property_local_authority_spec.rb +++ b/spec/models/form/lettings/pages/property_local_authority_spec.rb @@ -33,6 +33,8 @@ RSpec.describe Form::Lettings::Pages::PropertyLocalAuthority, type: :model do end it "has the correct depends_on" do - expect(page.depends_on).to match([{ "is_general_needs?" => true, "is_la_inferred" => false }]) + expect(page.depends_on).to match([{ "address_search_given?" => true, + "is_general_needs?" => true, + "is_la_inferred" => false }]) end end diff --git a/spec/models/form/lettings/pages/uprn_selection_spec.rb b/spec/models/form/lettings/pages/uprn_selection_spec.rb index f9ca20894..89e2424d2 100644 --- a/spec/models/form/lettings/pages/uprn_selection_spec.rb +++ b/spec/models/form/lettings/pages/uprn_selection_spec.rb @@ -39,6 +39,14 @@ RSpec.describe Form::Lettings::Pages::UprnSelection, type: :model do end it "has correct depends_on" do - expect(page.depends_on).to eq([{ "address_options_present?" => true }]) + expect(page.depends_on).to eq([{ "address_options_present?" => true, + "is_supported_housing?" => false, + "uprn_known" => nil }, + { "address_options_present?" => true, + "is_supported_housing?" => false, + "uprn_known" => 0 }, + { "address_options_present?" => true, + "is_supported_housing?" => false, + "uprn_confirmed" => 0 }]) end end diff --git a/spec/models/form/sales/pages/no_address_found_spec.rb b/spec/models/form/sales/pages/no_address_found_spec.rb index e275892bf..79b2bd0ca 100644 --- a/spec/models/form/sales/pages/no_address_found_spec.rb +++ b/spec/models/form/sales/pages/no_address_found_spec.rb @@ -25,7 +25,9 @@ RSpec.describe Form::Sales::Pages::NoAddressFound, type: :model do end it "has correct depends_on" do - expect(page.depends_on).to eq([{ "address_options_present?" => false }]) + expect(page.depends_on).to eq([{ "address_options_present?" => false, "uprn_known" => nil }, + { "address_options_present?" => false, "uprn_known" => 0 }, + { "address_options_present?" => false, "uprn_confirmed" => 0 }]) end it "is interruption screen page" do diff --git a/spec/models/form/sales/pages/property_local_authority_spec.rb b/spec/models/form/sales/pages/property_local_authority_spec.rb index 55234ad52..49c39f8e9 100644 --- a/spec/models/form/sales/pages/property_local_authority_spec.rb +++ b/spec/models/form/sales/pages/property_local_authority_spec.rb @@ -52,8 +52,6 @@ RSpec.describe Form::Sales::Pages::PropertyLocalAuthority, type: :model do end it "has the correct depends_on" do - expect(page.depends_on).to eq([{ - "is_la_inferred" => false, - }]) + expect(page.depends_on).to eq([{ "address_search_given?" => true, "is_la_inferred" => false }]) end end diff --git a/spec/models/form/sales/pages/uprn_selection_spec.rb b/spec/models/form/sales/pages/uprn_selection_spec.rb index 28c904c58..75fc6ae24 100644 --- a/spec/models/form/sales/pages/uprn_selection_spec.rb +++ b/spec/models/form/sales/pages/uprn_selection_spec.rb @@ -39,6 +39,8 @@ RSpec.describe Form::Sales::Pages::UprnSelection, type: :model do end it "has correct depends_on" do - expect(page.depends_on).to eq([{ "address_options_present?" => true }]) + expect(page.depends_on).to eq([{ "address_options_present?" => true, "uprn_known" => nil }, + { "address_options_present?" => true, "uprn_known" => 0 }, + { "address_options_present?" => true, "uprn_confirmed" => 0 }]) end end