From 014045e5fca80f1ddb92cd6e0b1b0ebad3e6713c Mon Sep 17 00:00:00 2001 From: Kat Date: Wed, 11 Oct 2023 16:12:38 +0100 Subject: [PATCH] Refactor guidance to allow both top and bottom guidance at the same time --- .../form/lettings/questions/declaration.rb | 2 +- app/models/form/lettings/questions/earnings.rb | 3 +-- .../form/lettings/questions/net_income_known.rb | 3 +-- app/models/form/lettings/questions/rent_type.rb | 3 +-- app/models/form/lettings/questions/scheme_id.rb | 3 +-- app/models/form/lettings/questions/voiddate.rb | 3 +-- app/models/form/question.rb | 15 +++++---------- .../sales/questions/discounted_ownership_type.rb | 3 +-- .../form/sales/questions/mortgage_lender.rb | 3 +-- .../sales/questions/outright_ownership_type.rb | 3 +-- .../form/sales/questions/privacy_notice.rb | 2 +- .../sales/questions/shared_ownership_type.rb | 3 +-- app/views/form/_checkbox_question.html.erb | 4 ++-- app/views/form/_date_question.html.erb | 4 ++-- app/views/form/_numeric_output_question.html.erb | 4 ++-- app/views/form/_numeric_question.html.erb | 4 ++-- app/views/form/_radio_question.html.erb | 4 ++-- app/views/form/_select_question.html.erb | 4 ++-- app/views/form/_text_question.html.erb | 4 ++-- app/views/form/_textarea_question.html.erb | 4 ++-- config/forms/2021_2022.json | 6 +++--- config/forms/2022_2023.json | 9 ++++----- spec/fixtures/forms/2021_2022.json | 2 +- .../form/lettings/questions/rent_type_spec.rb | 2 +- .../form/lettings/questions/voiddate_spec.rb | 4 ++-- .../questions/discounted_ownership_type_spec.rb | 8 ++++---- .../form/sales/questions/mortgage_lender_spec.rb | 10 +++++++--- .../questions/outright_ownership_type_spec.rb | 16 ++++++++++++---- .../questions/shared_ownership_type_spec.rb | 8 ++++---- 29 files changed, 70 insertions(+), 73 deletions(-) diff --git a/app/models/form/lettings/questions/declaration.rb b/app/models/form/lettings/questions/declaration.rb index 742545b96..03fc72f4a 100644 --- a/app/models/form/lettings/questions/declaration.rb +++ b/app/models/form/lettings/questions/declaration.rb @@ -6,7 +6,7 @@ class Form::Lettings::Questions::Declaration < ::Form::Question @header = "Declaration" @type = "checkbox" @check_answers_card_number = 0 - @guidance_partial = "privacy_notice_tenant" + @top_guidance_partial = "privacy_notice_tenant" @answer_options = ANSWER_OPTIONS @question_number = 30 end diff --git a/app/models/form/lettings/questions/earnings.rb b/app/models/form/lettings/questions/earnings.rb index 4169547c1..84d9ff9c5 100644 --- a/app/models/form/lettings/questions/earnings.rb +++ b/app/models/form/lettings/questions/earnings.rb @@ -8,8 +8,7 @@ class Form::Lettings::Questions::Earnings < ::Form::Question @width = 5 @check_answers_card_number = 0 @min = 0 - @guidance_partial = "what_counts_as_income" - @guidance_position = GuidancePosition::TOP + @top_guidance_partial = "what_counts_as_income" @hint_text = "" @step = 0.01 @prefix = "£" diff --git a/app/models/form/lettings/questions/net_income_known.rb b/app/models/form/lettings/questions/net_income_known.rb index 4c232a85d..d3008f44f 100644 --- a/app/models/form/lettings/questions/net_income_known.rb +++ b/app/models/form/lettings/questions/net_income_known.rb @@ -6,8 +6,7 @@ class Form::Lettings::Questions::NetIncomeKnown < ::Form::Question @header = "Do you know the household’s combined income after tax?" @type = "radio" @check_answers_card_number = 0 - @guidance_partial = "what_counts_as_income" - @guidance_position = GuidancePosition::TOP + @top_guidance_partial = "what_counts_as_income" @hint_text = "" @answer_options = ANSWER_OPTIONS @question_number = 86 diff --git a/app/models/form/lettings/questions/rent_type.rb b/app/models/form/lettings/questions/rent_type.rb index b902539a3..4ff1a0d32 100644 --- a/app/models/form/lettings/questions/rent_type.rb +++ b/app/models/form/lettings/questions/rent_type.rb @@ -5,8 +5,7 @@ class Form::Lettings::Questions::RentType < ::Form::Question @check_answer_label = "Rent type" @header = "What is the rent type?" @type = "radio" - @guidance_partial = "rent_type_definitions" - @guidance_position = GuidancePosition::TOP + @top_guidance_partial = "rent_type_definitions" @answer_options = ANSWER_OPTIONS @conditional_for = { "irproduct_other" => [5] } @question_number = 6 diff --git a/app/models/form/lettings/questions/scheme_id.rb b/app/models/form/lettings/questions/scheme_id.rb index 5d140b21b..6eb5eeb66 100644 --- a/app/models/form/lettings/questions/scheme_id.rb +++ b/app/models/form/lettings/questions/scheme_id.rb @@ -6,8 +6,7 @@ class Form::Lettings::Questions::SchemeId < ::Form::Question @hint_text = "Enter scheme name or postcode" @type = "select" @answer_options = answer_options - @guidance_position = GuidancePosition::BOTTOM - @guidance_partial = "scheme_selection" + @bottom_guidance_partial = "scheme_selection" @question_number = 9 @inferred_answers = { "location.name": { diff --git a/app/models/form/lettings/questions/voiddate.rb b/app/models/form/lettings/questions/voiddate.rb index 0431b9adb..d20219b0b 100644 --- a/app/models/form/lettings/questions/voiddate.rb +++ b/app/models/form/lettings/questions/voiddate.rb @@ -7,7 +7,6 @@ class Form::Lettings::Questions::Voiddate < ::Form::Question @type = "date" @check_answers_card_number = 0 @question_number = 23 - @guidance_partial = "void_date" - @guidance_position = GuidancePosition::BOTTOM + @bottom_guidance_partial = "void_date" end end diff --git a/app/models/form/question.rb b/app/models/form/question.rb index 9f0a8aac9..eac75dc5c 100644 --- a/app/models/form/question.rb +++ b/app/models/form/question.rb @@ -3,22 +3,17 @@ class Form::Question :type, :min, :max, :step, :width, :fields_to_add, :result_field, :conditional_for, :readonly, :answer_options, :page, :check_answer_label, :inferred_answers, :hidden_in_check_answers, :inferred_check_answers_value, - :guidance_partial, :prefix, :suffix, :requires_js, :fields_added, :derived, + :top_guidance_partial, :bottom_guidance_partial, :prefix, :suffix, :requires_js, :fields_added, :derived, :check_answers_card_number, :unresolved_hint_text, :question_number, :hide_question_number_on_page, :plain_label, :error_label - module GuidancePosition - TOP = 1 - BOTTOM = 2 - end - def initialize(id, hsh, page) @id = id @page = page if hsh @check_answer_label = hsh["check_answer_label"] @header = hsh["header"] - @guidance_partial = hsh["guidance_partial"] - @guidance_position = hsh["guidance_position"] || GuidancePosition::TOP + @top_guidance_partial = hsh["top_guidance_partial"] + @bottom_guidance_partial = hsh["bottom_guidance_partial"] @hint_text = hsh["hint_text"] @type = hsh["type"] @min = hsh["min"] @@ -232,11 +227,11 @@ class Form::Question end def top_guidance? - @guidance_partial && @guidance_position == GuidancePosition::TOP + @top_guidance_partial.present? end def bottom_guidance? - @guidance_partial && @guidance_position == GuidancePosition::BOTTOM + @bottom_guidance_partial.present? end def is_derived_or_has_inferred_check_answers_value?(log) diff --git a/app/models/form/sales/questions/discounted_ownership_type.rb b/app/models/form/sales/questions/discounted_ownership_type.rb index aeb00fac1..be4d3c47b 100644 --- a/app/models/form/sales/questions/discounted_ownership_type.rb +++ b/app/models/form/sales/questions/discounted_ownership_type.rb @@ -5,8 +5,7 @@ class Form::Sales::Questions::DiscountedOwnershipType < ::Form::Question @check_answer_label = "Type of discounted ownership sale" @header = "What is the type of discounted ownership sale?" @type = "radio" - @guidance_partial = guidance_partial - @guidance_position = GuidancePosition::TOP + @top_guidance_partial = guidance_partial @answer_options = ANSWER_OPTIONS @question_number = 5 end diff --git a/app/models/form/sales/questions/mortgage_lender.rb b/app/models/form/sales/questions/mortgage_lender.rb index ad584aab5..e2f57306e 100644 --- a/app/models/form/sales/questions/mortgage_lender.rb +++ b/app/models/form/sales/questions/mortgage_lender.rb @@ -8,8 +8,7 @@ class Form::Sales::Questions::MortgageLender < ::Form::Question @hint_text = "" @page = page @answer_options = ANSWER_OPTIONS - @guidance_position = GuidancePosition::BOTTOM - @guidance_partial = "mortgage_lender" + @bottom_guidance_partial = "mortgage_lender" @ownershipsch = ownershipsch @question_number = question_number end diff --git a/app/models/form/sales/questions/outright_ownership_type.rb b/app/models/form/sales/questions/outright_ownership_type.rb index 71fa86e65..eb587d396 100644 --- a/app/models/form/sales/questions/outright_ownership_type.rb +++ b/app/models/form/sales/questions/outright_ownership_type.rb @@ -5,8 +5,7 @@ class Form::Sales::Questions::OutrightOwnershipType < ::Form::Question @check_answer_label = "Type of outright sale" @header = "What is the type of outright sale?" @type = "radio" - @guidance_partial = guidance_partial - @guidance_position = GuidancePosition::TOP + @top_guidance_partial = guidance_partial @answer_options = ANSWER_OPTIONS @conditional_for = { "othtype" => [12], diff --git a/app/models/form/sales/questions/privacy_notice.rb b/app/models/form/sales/questions/privacy_notice.rb index c08e4be6a..d440e414b 100644 --- a/app/models/form/sales/questions/privacy_notice.rb +++ b/app/models/form/sales/questions/privacy_notice.rb @@ -6,7 +6,7 @@ class Form::Sales::Questions::PrivacyNotice < ::Form::Question @header = "Declaration" @type = "checkbox" @answer_options = ANSWER_OPTIONS - @guidance_partial = "privacy_notice_buyer" + @top_guidance_partial = "privacy_notice_buyer" @question_number = 19 end diff --git a/app/models/form/sales/questions/shared_ownership_type.rb b/app/models/form/sales/questions/shared_ownership_type.rb index 053af2e1a..658aa3139 100644 --- a/app/models/form/sales/questions/shared_ownership_type.rb +++ b/app/models/form/sales/questions/shared_ownership_type.rb @@ -5,8 +5,7 @@ class Form::Sales::Questions::SharedOwnershipType < ::Form::Question @check_answer_label = "Type of shared ownership sale" @header = "What is the type of shared ownership sale?" @hint_text = "A shared ownership sale is when the purchaser buys up to 75% of the property value and pays rent to the Private Registered Provider (PRP) on the remaining portion" - @guidance_partial = guidance_partial - @guidance_position = GuidancePosition::TOP + @top_guidance_partial = guidance_partial @type = "radio" @answer_options = answer_options @question_number = 4 diff --git a/app/views/form/_checkbox_question.html.erb b/app/views/form/_checkbox_question.html.erb index 2a9c33941..2e8585e15 100644 --- a/app/views/form/_checkbox_question.html.erb +++ b/app/views/form/_checkbox_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <%= f.govuk_check_boxes_fieldset question.id.to_sym, caption: caption(caption_text, page_header, conditional), @@ -21,4 +21,4 @@ <% end %> <% end %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_date_question.html.erb b/app/views/form/_date_question.html.erb index 167e02a31..c84d60d0e 100644 --- a/app/views/form/_date_question.html.erb +++ b/app/views/form/_date_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <%= f.govuk_date_field question.id.to_sym, caption: caption(caption_text, page_header, conditional), @@ -9,4 +9,4 @@ <%= govuk_inset_text(text: question.unresolved_hint_text) if question.unresolved_hint_text.present? && @log.unresolved %> <% end %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_numeric_output_question.html.erb b/app/views/form/_numeric_output_question.html.erb index 7d681f7ea..66b5e5dab 100644 --- a/app/views/form/_numeric_output_question.html.erb +++ b/app/views/form/_numeric_output_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %>
-<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_numeric_question.html.erb b/app/views/form/_numeric_question.html.erb index b92d0f736..aafa68d93 100644 --- a/app/views/form/_numeric_question.html.erb +++ b/app/views/form/_numeric_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <%= f.govuk_number_field( question.id.to_sym, @@ -16,4 +16,4 @@ **stimulus_html_attributes(question), ) %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_radio_question.html.erb b/app/views/form/_radio_question.html.erb index accc62d8c..d126b89d8 100644 --- a/app/views/form/_radio_question.html.erb +++ b/app/views/form/_radio_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <% banner = question.notification_banner(@log) %> <% if banner %> <%= govuk_notification_banner( @@ -44,4 +44,4 @@ <% end %> <% end %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_select_question.html.erb b/app/views/form/_select_question.html.erb index 5f410deef..2af828ed4 100644 --- a/app/views/form/_select_question.html.erb +++ b/app/views/form/_select_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <% selected = @log.public_send(question.id) || "" %> <% answers = question.displayed_answer_options(@log, current_user).map { |key, value| OpenStruct.new(id: key, name: value.respond_to?(:service_name) ? value.service_name : nil, resource: value) } %> @@ -17,4 +17,4 @@ <% end %> <% end %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_text_question.html.erb b/app/views/form/_text_question.html.erb index 74e7e9b26..920eaf883 100644 --- a/app/views/form/_text_question.html.erb +++ b/app/views/form/_text_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <%= f.govuk_text_field question.id.to_sym, caption: caption(caption_text, page_header, conditional), @@ -7,4 +7,4 @@ width: question.width || nil, **stimulus_html_attributes(question) %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/app/views/form/_textarea_question.html.erb b/app/views/form/_textarea_question.html.erb index a9768e4d0..fcb8c0a42 100644 --- a/app/views/form/_textarea_question.html.erb +++ b/app/views/form/_textarea_question.html.erb @@ -1,4 +1,4 @@ -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.top_guidance? %> +<%= render partial: "form/guidance/#{question.top_guidance_partial}" if question.top_guidance? %> <%= f.govuk_text_area question.id.to_sym, caption: caption(caption_text, page_header, conditional), @@ -7,4 +7,4 @@ width: question.width || nil, **stimulus_html_attributes(question) %> -<%= render partial: "form/guidance/#{question.guidance_partial}" if question.bottom_guidance? %> +<%= render partial: "form/guidance/#{question.bottom_guidance_partial}" if question.bottom_guidance? %> diff --git a/config/forms/2021_2022.json b/config/forms/2021_2022.json index 9751bf569..006a716de 100644 --- a/config/forms/2021_2022.json +++ b/config/forms/2021_2022.json @@ -1139,7 +1139,7 @@ "questions": { "declaration": { "header": "", - "guidance_partial": "privacy_notice_tenant", + "top_guidance_partial": "privacy_notice_tenant", "check_answer_label": "Tenant has seen the privacy notice", "check_answers_card_number": 0, "type": "checkbox", @@ -7328,7 +7328,7 @@ "net_income_known": { "check_answer_label": "Do you know the household’s combined income?", "header": "Do you know the household’s combined income after tax?", - "guidance_partial": "what_counts_as_income", + "top_guidance_partial": "what_counts_as_income", "hint_text": "", "type": "radio", "answer_options": { @@ -7360,7 +7360,7 @@ "earnings": { "check_answer_label": "Total household income", "header": "How much income does the household have in total?", - "guidance_partial": "what_counts_as_income", + "top_guidance_partial": "what_counts_as_income", "hint_text": "", "type": "numeric", "min": 0, diff --git a/config/forms/2022_2023.json b/config/forms/2022_2023.json index c3e7d5cef..5dc249050 100644 --- a/config/forms/2022_2023.json +++ b/config/forms/2022_2023.json @@ -721,8 +721,7 @@ "header": "What is the void date?", "hint_text": "For example, 27 3 2021.", "type": "date", - "guidance_partial": "void_date", - "guidance_position": 2 + "bottom_guidance_partial": "void_date" } }, "depends_on": [ @@ -1170,7 +1169,7 @@ "questions": { "declaration": { "header": "", - "guidance_partial": "privacy_notice_tenant", + "top_guidance_partial": "privacy_notice_tenant", "check_answer_label": "Tenant has seen the privacy notice", "check_answers_card_number": 0, "type": "checkbox", @@ -7286,7 +7285,7 @@ "net_income_known": { "check_answer_label": "Do you know the household’s combined income?", "header": "Do you know the household’s combined income after tax?", - "guidance_partial": "what_counts_as_income", + "top_guidance_partial": "what_counts_as_income", "hint_text": "", "type": "radio", "answer_options": { @@ -7318,7 +7317,7 @@ "earnings": { "check_answer_label": "Total household income", "header": "How much income does the household have in total?", - "guidance_partial": "what_counts_as_income", + "top_guidance_partial": "what_counts_as_income", "hint_text": "", "type": "numeric", "min": 0, diff --git a/spec/fixtures/forms/2021_2022.json b/spec/fixtures/forms/2021_2022.json index 300b658b6..9f4195223 100644 --- a/spec/fixtures/forms/2021_2022.json +++ b/spec/fixtures/forms/2021_2022.json @@ -688,7 +688,7 @@ "earnings": { "check_answer_label": "Income", "header": "What is the tenant’s /and partner’s combined income after tax?", - "guidance_partial": "what_counts_as_income", + "top_guidance_partial": "what_counts_as_income", "type": "numeric", "min": 0, "step": 1, diff --git a/spec/models/form/lettings/questions/rent_type_spec.rb b/spec/models/form/lettings/questions/rent_type_spec.rb index f78233a44..da1951c09 100644 --- a/spec/models/form/lettings/questions/rent_type_spec.rb +++ b/spec/models/form/lettings/questions/rent_type_spec.rb @@ -51,6 +51,6 @@ RSpec.describe Form::Lettings::Questions::RentType, type: :model do end it "has the guidance partial" do - expect(question.guidance_partial).to eq("rent_type_definitions") + expect(question.top_guidance_partial).to eq("rent_type_definitions") end end diff --git a/spec/models/form/lettings/questions/voiddate_spec.rb b/spec/models/form/lettings/questions/voiddate_spec.rb index f35f3a575..c8198bc89 100644 --- a/spec/models/form/lettings/questions/voiddate_spec.rb +++ b/spec/models/form/lettings/questions/voiddate_spec.rb @@ -31,8 +31,8 @@ RSpec.describe Form::Lettings::Questions::Voiddate, type: :model do expect(question.question_number).to eq(23) end - it "has the correct guidance_partial" do - expect(question.guidance_partial).to eq("void_date") + it "has the correct bottom_guidance_partial" do + expect(question.bottom_guidance_partial).to eq("void_date") end it "has the correct type" do diff --git a/spec/models/form/sales/questions/discounted_ownership_type_spec.rb b/spec/models/form/sales/questions/discounted_ownership_type_spec.rb index 766466e74..89fe9378a 100644 --- a/spec/models/form/sales/questions/discounted_ownership_type_spec.rb +++ b/spec/models/form/sales/questions/discounted_ownership_type_spec.rb @@ -49,8 +49,8 @@ RSpec.describe Form::Sales::Questions::DiscountedOwnershipType, type: :model do context "when the form is for 2023/24" do let(:start_date) { Time.zone.local(2023, 4, 8) } - it "shows shows correct guidance_partial" do - expect(question.guidance_partial).to eq("discounted_ownership_type_definitions") + it "shows shows correct top_guidance_partial" do + expect(question.top_guidance_partial).to eq("discounted_ownership_type_definitions") end it "is at the top" do @@ -62,8 +62,8 @@ RSpec.describe Form::Sales::Questions::DiscountedOwnershipType, type: :model do context "when the form is for before 2023/24" do let(:start_date) { Time.zone.local(2022, 4, 8) } - it "does not show a guidance_partial" do - expect(question.guidance_partial).to eq(nil) + it "does not show a top_guidance_partial" do + expect(question.top_guidance_partial).to eq(nil) end end end diff --git a/spec/models/form/sales/questions/mortgage_lender_spec.rb b/spec/models/form/sales/questions/mortgage_lender_spec.rb index 811746dbe..1fcf928a3 100644 --- a/spec/models/form/sales/questions/mortgage_lender_spec.rb +++ b/spec/models/form/sales/questions/mortgage_lender_spec.rb @@ -31,13 +31,17 @@ RSpec.describe Form::Sales::Questions::MortgageLender, type: :model do expect(question.derived?).to be false end - it "is has correct guidance_position" do + it "is has correct guidance position" do expect(question.top_guidance?).to be false expect(question.bottom_guidance?).to be true end - it "is has correct guidance_partial" do - expect(question.guidance_partial).to eq("mortgage_lender") + it "is has correct bottom_guidance_partial" do + expect(question.bottom_guidance_partial).to eq("mortgage_lender") + end + + it "is has correct top_guidance_partial" do + expect(question.top_guidance_partial).to be_nil end it "has the correct answer_options" do diff --git a/spec/models/form/sales/questions/outright_ownership_type_spec.rb b/spec/models/form/sales/questions/outright_ownership_type_spec.rb index 42804f938..e3bd043b4 100644 --- a/spec/models/form/sales/questions/outright_ownership_type_spec.rb +++ b/spec/models/form/sales/questions/outright_ownership_type_spec.rb @@ -50,8 +50,12 @@ RSpec.describe Form::Sales::Questions::OutrightOwnershipType, type: :model do context "when the form is for year 2023/24" do let(:start_date) { Time.zone.local(2023, 4, 8) } - it "has the correct guidance_partial" do - expect(question.guidance_partial).to eq("outright_sale_type_definitions") + it "has the correct top_guidance_partial" do + expect(question.top_guidance_partial).to eq("outright_sale_type_definitions") + end + + it "has the correct bottom_guidance_partial" do + expect(question.bottom_guidance_partial).to be_nil end it "is at the top" do @@ -63,8 +67,12 @@ RSpec.describe Form::Sales::Questions::OutrightOwnershipType, type: :model do context "when the form is for before year 2023/24" do let(:start_date) { Time.zone.local(2022, 4, 8) } - it "does not display a guidance partial" do - expect(question.guidance_partial).to eq(nil) + it "does not display a top guidance partial" do + expect(question.top_guidance_partial).to eq(nil) + end + + it "does not display a bottom guidance partial" do + expect(question.bottom_guidance_partial).to eq(nil) end end end diff --git a/spec/models/form/sales/questions/shared_ownership_type_spec.rb b/spec/models/form/sales/questions/shared_ownership_type_spec.rb index 669acd9a6..693d58de9 100644 --- a/spec/models/form/sales/questions/shared_ownership_type_spec.rb +++ b/spec/models/form/sales/questions/shared_ownership_type_spec.rb @@ -53,8 +53,8 @@ RSpec.describe Form::Sales::Questions::SharedOwnershipType, type: :model do }) end - it "does not show a guidance_partial" do - expect(question.guidance_partial).to eq(nil) + it "does not show a top_guidance_partial" do + expect(question.top_guidance_partial).to eq(nil) end end @@ -74,8 +74,8 @@ RSpec.describe Form::Sales::Questions::SharedOwnershipType, type: :model do }) end - it "shows shows correct guidance_partial" do - expect(question.guidance_partial).to eq("shared_ownership_type_definitions") + it "shows shows correct top_guidance_partial" do + expect(question.top_guidance_partial).to eq("shared_ownership_type_definitions") end end end