diff --git a/spec/models/form/lettings/pages/person_lead_partner_spec.rb b/spec/models/form/lettings/pages/person_lead_partner_spec.rb index 38404a198..18a14ef21 100644 --- a/spec/models/form/lettings/pages/person_lead_partner_spec.rb +++ b/spec/models/form/lettings/pages/person_lead_partner_spec.rb @@ -9,8 +9,6 @@ RSpec.describe Form::Lettings::Pages::PersonLeadPartner, type: :model do let(:form) { instance_double(Form, start_date: Time.zone.local(2024, 4, 1), start_year_2026_or_later?: start_year_2026_or_later?, person_question_count:) } let(:subsection) { instance_double(Form::Subsection, form:) } let(:person_index) { 2 } - let(:is_any_person_partner?) { false } - let(:log) { instance_double(LettingsLog, is_any_person_partner?: is_any_person_partner?) } it "has correct subsection" do expect(page.subsection).to eq(subsection) @@ -20,46 +18,6 @@ RSpec.describe Form::Lettings::Pages::PersonLeadPartner, type: :model do expect(page.description).to be_nil end - describe "#skip_page_in_form_flow?" do - context "with start year < 2026", metadata: { year: 25 } do - context "when no other person is the partner of the lead tenant" do - let(:is_any_person_partner?) { false } - - it "returns false" do - expect(page.skip_page_in_form_flow?(log)).to be false - end - end - - context "when another person is the partner of the lead tenant" do - let(:is_any_person_partner?) { true } - - it "returns false" do - expect(page.skip_page_in_form_flow?(log)).to be false - end - end - end - - context "with start year >= 2026", metadata: { year: 26 } do - let(:start_year_2026_or_later?) { true } - - context "when no other person is the partner of the lead tenant" do - let(:is_any_person_partner?) { false } - - it "returns false" do - expect(page.skip_page_in_form_flow?(log)).to be false - end - end - - context "when another person is the partner of the lead tenant" do - let(:is_any_person_partner?) { true } - - it "returns true" do - expect(page.skip_page_in_form_flow?(log)).to be true - end - end - end - end - context "with person 2" do it "has correct questions" do expect(page.questions.map(&:id)).to eq(%w[relat2]) diff --git a/spec/models/form/lettings/questions/person_partner_spec.rb b/spec/models/form/lettings/questions/person_partner_spec.rb index 4c19c58eb..4c99818e2 100644 --- a/spec/models/form/lettings/questions/person_partner_spec.rb +++ b/spec/models/form/lettings/questions/person_partner_spec.rb @@ -24,6 +24,8 @@ RSpec.describe Form::Lettings::Questions::PersonPartner, type: :model do ) end let(:person_index) { 2 } + let(:is_any_person_partner?) { false } + let(:log) { instance_double(LettingsLog, is_any_person_partner?: is_any_person_partner?) } it "has correct page" do expect(question.page).to eq(page) @@ -47,6 +49,48 @@ RSpec.describe Form::Lettings::Questions::PersonPartner, type: :model do expect(question.hidden_in_check_answers).to be_nil end + describe "#skip_page_in_form_flow?" do + context "with start year < 2026", metadata: { year: 25 } do + let(:year) { 2025 } + + context "when no other person is the partner of the lead tenant" do + let(:is_any_person_partner?) { false } + + it "returns false" do + expect(question.skip_question_in_form_flow?(log)).to be false + end + end + + context "when another person is the partner of the lead tenant" do + let(:is_any_person_partner?) { true } + + it "returns false" do + expect(question.skip_question_in_form_flow?(log)).to be false + end + end + end + + context "with start year >= 2026", metadata: { year: 26 } do + let(:year) { 2026 } + + context "when no other person is the partner of the lead tenant" do + let(:is_any_person_partner?) { false } + + it "returns false" do + expect(question.skip_question_in_form_flow?(log)).to be false + end + end + + context "when another person is the partner of the lead tenant" do + let(:is_any_person_partner?) { true } + + it "returns true" do + expect(question.skip_question_in_form_flow?(log)).to be true + end + end + end + end + context "with person 2" do it "has the correct id" do expect(question.id).to eq("relat2")