From 509f9db7db819c930ea94480e14ccf69b8396bf4 Mon Sep 17 00:00:00 2001
From: natdeanlewissoftwire <nat.dean-lewis@softwire.com>
Date: Wed, 14 Dec 2022 15:58:06 +0000
Subject: [PATCH] test: update tests and refactor

---
 .../sales/pages/person1_relationship_to_buyer1.rb    |  5 +----
 .../person1_relationship_to_buyer1_joint_purchase.rb |  5 +----
 db/schema.rb                                         |  2 +-
 spec/models/form/sales/pages/person1_age_spec.rb     |  4 ++--
 spec/models/form/sales/pages/person1_known_spec.rb   |  8 ++++----
 spec/models/form/sales/pages/person2_age_spec.rb     |  4 ++--
 spec/models/form/sales/pages/person2_known_spec.rb   |  6 +++---
 spec/models/form/sales/pages/person3_age_spec.rb     |  4 ++--
 spec/models/form/sales/pages/person3_known_spec.rb   |  3 +--
 spec/models/form/sales/pages/person4_age_spec.rb     |  4 ++--
 spec/models/form/sales/pages/person4_known_spec.rb   |  2 +-
 .../form/sales/questions/person1_age_known_spec.rb   | 12 ++++++------
 spec/models/form/sales/questions/person1_age_spec.rb |  8 ++++----
 .../sales/questions/person1_gender_identity_spec.rb  |  2 +-
 .../form/sales/questions/person1_known_spec.rb       |  4 ++--
 .../form/sales/questions/person2_age_known_spec.rb   | 10 +++++-----
 spec/models/form/sales/questions/person2_age_spec.rb |  8 ++++----
 .../form/sales/questions/person2_known_spec.rb       |  4 ++--
 .../form/sales/questions/person3_age_known_spec.rb   | 10 +++++-----
 spec/models/form/sales/questions/person3_age_spec.rb |  8 ++++----
 .../form/sales/questions/person3_known_spec.rb       |  4 ++--
 .../form/sales/questions/person4_age_known_spec.rb   | 10 +++++-----
 spec/models/form/sales/questions/person4_age_spec.rb |  8 ++++----
 .../form/sales/questions/person4_known_spec.rb       |  4 ++--
 .../subsections/household_characteristics_spec.rb    |  8 ++++++++
 spec/models/form_handler_spec.rb                     |  4 ++--
 26 files changed, 76 insertions(+), 75 deletions(-)

diff --git a/app/models/form/sales/pages/person1_relationship_to_buyer1.rb b/app/models/form/sales/pages/person1_relationship_to_buyer1.rb
index 7ee7787a8..dea5a23b9 100644
--- a/app/models/form/sales/pages/person1_relationship_to_buyer1.rb
+++ b/app/models/form/sales/pages/person1_relationship_to_buyer1.rb
@@ -6,10 +6,7 @@ class Form::Sales::Pages::Person1RelationshipToBuyer1 < ::Form::Page
     @description = ""
     @subsection = subsection
     @depends_on = [
-      { "hholdcount" => 1, "jointpur" => 2 },
-      { "hholdcount" => 2, "jointpur" => 2 },
-      { "hholdcount" => 3, "jointpur" => 2 },
-      { "hholdcount" => 4, "jointpur" => 2 },
+      { "details_known_1" => 1, "jointpur" => 2 },
     ]
   end
 
diff --git a/app/models/form/sales/pages/person1_relationship_to_buyer1_joint_purchase.rb b/app/models/form/sales/pages/person1_relationship_to_buyer1_joint_purchase.rb
index 5c91452c0..e09da2924 100644
--- a/app/models/form/sales/pages/person1_relationship_to_buyer1_joint_purchase.rb
+++ b/app/models/form/sales/pages/person1_relationship_to_buyer1_joint_purchase.rb
@@ -6,10 +6,7 @@ class Form::Sales::Pages::Person1RelationshipToBuyer1JointPurchase < ::Form::Pag
     @description = ""
     @subsection = subsection
     @depends_on = [
-      { "hholdcount" => 1, "jointpur" => 1 },
-      { "hholdcount" => 2, "jointpur" => 1 },
-      { "hholdcount" => 3, "jointpur" => 1 },
-      { "hholdcount" => 4, "jointpur" => 1 },
+      { "details_known_1" => 1, "jointpur" => 1 },
     ]
   end
 
diff --git a/db/schema.rb b/db/schema.rb
index 7eaee9ecb..00ffe3065 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -405,13 +405,13 @@ ActiveRecord::Schema[7.0].define(version: 2022_12_13_130736) do
     t.integer "savingsnk"
     t.integer "savings"
     t.string "sex3"
-    t.bigint "updated_by_id"
     t.integer "details_known_1"
     t.string "relat3"
     t.integer "income1_value_check"
     t.integer "mortgage"
     t.integer "inc2mort"
     t.integer "mortgage_value_check"
+    t.bigint "updated_by_id"
     t.index ["created_by_id"], name: "index_sales_logs_on_created_by_id"
     t.index ["managing_organisation_id"], name: "index_sales_logs_on_managing_organisation_id"
     t.index ["owning_organisation_id"], name: "index_sales_logs_on_owning_organisation_id"
diff --git a/spec/models/form/sales/pages/person1_age_spec.rb b/spec/models/form/sales/pages/person1_age_spec.rb
index 553a7cff4..4228ce161 100644
--- a/spec/models/form/sales/pages/person1_age_spec.rb
+++ b/spec/models/form/sales/pages/person1_age_spec.rb
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Pages::Person1Age, type: :model do
   end
 
   it "has correct questions" do
-    expect(page.questions.map(&:id)).to eq(%w[age3_known age3])
+    expect(page.questions.map(&:id)).to eq(%w[age2_known age2])
   end
 
   it "has the correct id" do
@@ -30,7 +30,7 @@ RSpec.describe Form::Sales::Pages::Person1Age, type: :model do
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
       [
-        { "details_known_1" => 1 },
+        { "details_known_1" => 1, "jointpur" => 2 },
       ],
     )
   end
diff --git a/spec/models/form/sales/pages/person1_known_spec.rb b/spec/models/form/sales/pages/person1_known_spec.rb
index 3e3969bc6..ebe0e963c 100644
--- a/spec/models/form/sales/pages/person1_known_spec.rb
+++ b/spec/models/form/sales/pages/person1_known_spec.rb
@@ -34,10 +34,10 @@ RSpec.describe Form::Sales::Pages::Person1Known, type: :model do
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
       [
-        { "hholdcount" => 1 },
-        { "hholdcount" => 2 },
-        { "hholdcount" => 3 },
-        { "hholdcount" => 4 },
+        { "hholdcount" => 1, "jointpur" => 2 },
+        { "hholdcount" => 2, "jointpur" => 2 },
+        { "hholdcount" => 3, "jointpur" => 2 },
+        { "hholdcount" => 4, "jointpur" => 2 },
       ],
     )
   end
diff --git a/spec/models/form/sales/pages/person2_age_spec.rb b/spec/models/form/sales/pages/person2_age_spec.rb
index 85d2813f1..c054bf4e0 100644
--- a/spec/models/form/sales/pages/person2_age_spec.rb
+++ b/spec/models/form/sales/pages/person2_age_spec.rb
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Pages::Person2Age, type: :model do
   end
 
   it "has correct questions" do
-    expect(page.questions.map(&:id)).to eq(%w[age4_known age4])
+    expect(page.questions.map(&:id)).to eq(%w[age3_known age3])
   end
 
   it "has the correct id" do
@@ -29,7 +29,7 @@ RSpec.describe Form::Sales::Pages::Person2Age, type: :model do
 
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
-      [{ "details_known_2" => 1 }],
+      [{ "details_known_2" => 1, "jointpur" => 2 }],
     )
   end
 end
diff --git a/spec/models/form/sales/pages/person2_known_spec.rb b/spec/models/form/sales/pages/person2_known_spec.rb
index 57bac950f..e06f61ad2 100644
--- a/spec/models/form/sales/pages/person2_known_spec.rb
+++ b/spec/models/form/sales/pages/person2_known_spec.rb
@@ -34,9 +34,9 @@ RSpec.describe Form::Sales::Pages::Person2Known, type: :model do
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
       [
-        { "hholdcount" => 2 },
-        { "hholdcount" => 3 },
-        { "hholdcount" => 4 },
+        { "hholdcount" => 2, "jointpur" => 2 },
+        { "hholdcount" => 3, "jointpur" => 2 },
+        { "hholdcount" => 4, "jointpur" => 2 },
       ],
     )
   end
diff --git a/spec/models/form/sales/pages/person3_age_spec.rb b/spec/models/form/sales/pages/person3_age_spec.rb
index 4cb16be5e..9ff9b520d 100644
--- a/spec/models/form/sales/pages/person3_age_spec.rb
+++ b/spec/models/form/sales/pages/person3_age_spec.rb
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Pages::Person3Age, type: :model do
   end
 
   it "has correct questions" do
-    expect(page.questions.map(&:id)).to eq(%w[age5_known age5])
+    expect(page.questions.map(&:id)).to eq(%w[age4_known age4])
   end
 
   it "has the correct id" do
@@ -29,7 +29,7 @@ RSpec.describe Form::Sales::Pages::Person3Age, type: :model do
 
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
-      [{ "details_known_3" => 1 }],
+      [{ "details_known_3" => 1, "jointpur" => 2 }],
     )
   end
 end
diff --git a/spec/models/form/sales/pages/person3_known_spec.rb b/spec/models/form/sales/pages/person3_known_spec.rb
index e05b8b452..bb82ba9f0 100644
--- a/spec/models/form/sales/pages/person3_known_spec.rb
+++ b/spec/models/form/sales/pages/person3_known_spec.rb
@@ -34,8 +34,7 @@ RSpec.describe Form::Sales::Pages::Person3Known, type: :model do
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
       [
-        { "hholdcount" => 3 },
-        { "hholdcount" => 4 },
+        { "hholdcount" => 3, "jointpur" => 2 }, { "hholdcount" => 4, "jointpur" => 2 }
       ],
     )
   end
diff --git a/spec/models/form/sales/pages/person4_age_spec.rb b/spec/models/form/sales/pages/person4_age_spec.rb
index 530e9ad22..28fe3d1f8 100644
--- a/spec/models/form/sales/pages/person4_age_spec.rb
+++ b/spec/models/form/sales/pages/person4_age_spec.rb
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Pages::Person4Age, type: :model do
   end
 
   it "has correct questions" do
-    expect(page.questions.map(&:id)).to eq(%w[age6_known age6])
+    expect(page.questions.map(&:id)).to eq(%w[age5_known age5])
   end
 
   it "has the correct id" do
@@ -29,7 +29,7 @@ RSpec.describe Form::Sales::Pages::Person4Age, type: :model do
 
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
-      [{ "details_known_4" => 1 }],
+      [{ "details_known_4" => 1, "jointpur" => 2 }],
     )
   end
 end
diff --git a/spec/models/form/sales/pages/person4_known_spec.rb b/spec/models/form/sales/pages/person4_known_spec.rb
index fdf8ad7d5..f8b791c94 100644
--- a/spec/models/form/sales/pages/person4_known_spec.rb
+++ b/spec/models/form/sales/pages/person4_known_spec.rb
@@ -34,7 +34,7 @@ RSpec.describe Form::Sales::Pages::Person4Known, type: :model do
   it "has correct depends_on" do
     expect(page.depends_on).to eq(
       [
-        { "hholdcount" => 4 },
+        { "hholdcount" => 4, "jointpur" => 2 },
       ],
     )
   end
diff --git a/spec/models/form/sales/questions/person1_age_known_spec.rb b/spec/models/form/sales/questions/person1_age_known_spec.rb
index 98dd81056..31d0585b2 100644
--- a/spec/models/form/sales/questions/person1_age_known_spec.rb
+++ b/spec/models/form/sales/questions/person1_age_known_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person1AgeKnown, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age2_known" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person1AgeKnown, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age3_known")
+    expect(question.id).to eq("age2_known")
   end
 
   it "has the correct header" do
@@ -40,7 +40,7 @@ RSpec.describe Form::Sales::Questions::Person1AgeKnown, type: :model do
 
   it "has correct conditional for" do
     expect(question.conditional_for).to eq({
-      "age3" => [0],
+      "age2" => [0],
     })
   end
 
@@ -53,10 +53,10 @@ RSpec.describe Form::Sales::Questions::Person1AgeKnown, type: :model do
       {
         "depends_on" => [
           {
-            "age3_known" => 0,
+            "age2_known" => 0,
           },
           {
-            "age3_known" => 1,
+            "age2_known" => 1,
           },
         ],
       },
@@ -64,6 +64,6 @@ RSpec.describe Form::Sales::Questions::Person1AgeKnown, type: :model do
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(3)
+    expect(question.check_answers_card_number).to eq(2)
   end
 end
diff --git a/spec/models/form/sales/questions/person1_age_spec.rb b/spec/models/form/sales/questions/person1_age_spec.rb
index b904fd75f..8542e0313 100644
--- a/spec/models/form/sales/questions/person1_age_spec.rb
+++ b/spec/models/form/sales/questions/person1_age_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person1Age, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age2" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person1Age, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age3")
+    expect(question.id).to eq("age2")
   end
 
   it "has the correct header" do
@@ -41,12 +41,12 @@ RSpec.describe Form::Sales::Questions::Person1Age, type: :model do
 
   it "has the correct inferred check answers value" do
     expect(question.inferred_check_answers_value).to eq({
-      "condition" => { "age3_known" => 1 },
+      "condition" => { "age2_known" => 1 },
       "value" => "Not known",
     })
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(3)
+    expect(question.check_answers_card_number).to eq(2)
   end
 end
diff --git a/spec/models/form/sales/questions/person1_gender_identity_spec.rb b/spec/models/form/sales/questions/person1_gender_identity_spec.rb
index b89ca5560..9a4bc67cf 100644
--- a/spec/models/form/sales/questions/person1_gender_identity_spec.rb
+++ b/spec/models/form/sales/questions/person1_gender_identity_spec.rb
@@ -32,7 +32,7 @@ RSpec.describe Form::Sales::Questions::Person1GenderIdentity, type: :model do
   end
 
   it "has expected check answers card number" do
-    expect(question.check_answers_card_number).to eq(3)
+    expect(question.check_answers_card_number).to eq(2)
   end
 
   it "has the correct answer_options" do
diff --git a/spec/models/form/sales/questions/person1_known_spec.rb b/spec/models/form/sales/questions/person1_known_spec.rb
index e1207ea1b..4930d9710 100644
--- a/spec/models/form/sales/questions/person1_known_spec.rb
+++ b/spec/models/form/sales/questions/person1_known_spec.rb
@@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Questions::Person1Known, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
   let(:question_id) { nil }
-  let(:question_definition) { nil }
+  let(:question_definition) { { check_answers_card_number: 2 } }
   let(:page) { instance_double(Form::Page) }
 
   it "has correct page" do
@@ -59,6 +59,6 @@ RSpec.describe Form::Sales::Questions::Person1Known, type: :model do
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(3)
+    expect(question.check_answers_card_number).to eq(2)
   end
 end
diff --git a/spec/models/form/sales/questions/person2_age_known_spec.rb b/spec/models/form/sales/questions/person2_age_known_spec.rb
index 203ea0ecc..7d2f2657f 100644
--- a/spec/models/form/sales/questions/person2_age_known_spec.rb
+++ b/spec/models/form/sales/questions/person2_age_known_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person2AgeKnown, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age3_known" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person2AgeKnown, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age4_known")
+    expect(question.id).to eq("age3_known")
   end
 
   it "has the correct header" do
@@ -40,7 +40,7 @@ RSpec.describe Form::Sales::Questions::Person2AgeKnown, type: :model do
 
   it "has correct conditional for" do
     expect(question.conditional_for).to eq({
-      "age4" => [0],
+      "age3" => [0],
     })
   end
 
@@ -51,12 +51,12 @@ RSpec.describe Form::Sales::Questions::Person2AgeKnown, type: :model do
   it "has the correct hidden_in_check_answers" do
     expect(question.hidden_in_check_answers).to eq(
       {
-        "depends_on" => [{ "age4_known" => 0 }, { "age4_known" => 1 }],
+        "depends_on" => [{ "age3_known" => 0 }, { "age3_known" => 1 }],
       },
     )
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(4)
+    expect(question.check_answers_card_number).to eq(3)
   end
 end
diff --git a/spec/models/form/sales/questions/person2_age_spec.rb b/spec/models/form/sales/questions/person2_age_spec.rb
index 2dba0d695..78cf77954 100644
--- a/spec/models/form/sales/questions/person2_age_spec.rb
+++ b/spec/models/form/sales/questions/person2_age_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person2Age, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age3" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person2Age, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age4")
+    expect(question.id).to eq("age3")
   end
 
   it "has the correct header" do
@@ -41,12 +41,12 @@ RSpec.describe Form::Sales::Questions::Person2Age, type: :model do
 
   it "has the correct inferred check answers value" do
     expect(question.inferred_check_answers_value).to eq({
-      "condition" => { "age4_known" => 1 },
+      "condition" => { "age3_known" => 1 },
       "value" => "Not known",
     })
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(4)
+    expect(question.check_answers_card_number).to eq(3)
   end
 end
diff --git a/spec/models/form/sales/questions/person2_known_spec.rb b/spec/models/form/sales/questions/person2_known_spec.rb
index 6d76874df..6b8b7e3ab 100644
--- a/spec/models/form/sales/questions/person2_known_spec.rb
+++ b/spec/models/form/sales/questions/person2_known_spec.rb
@@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Questions::Person2Known, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
   let(:question_id) { nil }
-  let(:question_definition) { nil }
+  let(:question_definition) { { check_answers_card_number: 3 } }
   let(:page) { instance_double(Form::Page) }
 
   it "has correct page" do
@@ -59,6 +59,6 @@ RSpec.describe Form::Sales::Questions::Person2Known, type: :model do
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(4)
+    expect(question.check_answers_card_number).to eq(3)
   end
 end
diff --git a/spec/models/form/sales/questions/person3_age_known_spec.rb b/spec/models/form/sales/questions/person3_age_known_spec.rb
index 8f6e4f022..c794f96a7 100644
--- a/spec/models/form/sales/questions/person3_age_known_spec.rb
+++ b/spec/models/form/sales/questions/person3_age_known_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person3AgeKnown, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age4_known" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person3AgeKnown, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age5_known")
+    expect(question.id).to eq("age4_known")
   end
 
   it "has the correct header" do
@@ -40,7 +40,7 @@ RSpec.describe Form::Sales::Questions::Person3AgeKnown, type: :model do
 
   it "has correct conditional for" do
     expect(question.conditional_for).to eq({
-      "age5" => [0],
+      "age4" => [0],
     })
   end
 
@@ -51,12 +51,12 @@ RSpec.describe Form::Sales::Questions::Person3AgeKnown, type: :model do
   it "has the correct hidden_in_check_answers" do
     expect(question.hidden_in_check_answers).to eq(
       {
-        "depends_on" => [{ "age5_known" => 0 }, { "age5_known" => 1 }],
+        "depends_on" => [{ "age4_known" => 0 }, { "age4_known" => 1 }],
       },
     )
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(5)
+    expect(question.check_answers_card_number).to eq(4)
   end
 end
diff --git a/spec/models/form/sales/questions/person3_age_spec.rb b/spec/models/form/sales/questions/person3_age_spec.rb
index 50ec25c3f..c0267d06b 100644
--- a/spec/models/form/sales/questions/person3_age_spec.rb
+++ b/spec/models/form/sales/questions/person3_age_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person3Age, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age4" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person3Age, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age5")
+    expect(question.id).to eq("age4")
   end
 
   it "has the correct header" do
@@ -41,12 +41,12 @@ RSpec.describe Form::Sales::Questions::Person3Age, type: :model do
 
   it "has the correct inferred check answers value" do
     expect(question.inferred_check_answers_value).to eq({
-      "condition" => { "age5_known" => 1 },
+      "condition" => { "age4_known" => 1 },
       "value" => "Not known",
     })
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(5)
+    expect(question.check_answers_card_number).to eq(4)
   end
 end
diff --git a/spec/models/form/sales/questions/person3_known_spec.rb b/spec/models/form/sales/questions/person3_known_spec.rb
index 39efdcb3a..d5f5ee4e9 100644
--- a/spec/models/form/sales/questions/person3_known_spec.rb
+++ b/spec/models/form/sales/questions/person3_known_spec.rb
@@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Questions::Person3Known, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
   let(:question_id) { nil }
-  let(:question_definition) { nil }
+  let(:question_definition) { { check_answers_card_number: 4 } }
   let(:page) { instance_double(Form::Page) }
 
   it "has correct page" do
@@ -59,6 +59,6 @@ RSpec.describe Form::Sales::Questions::Person3Known, type: :model do
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(5)
+    expect(question.check_answers_card_number).to eq(4)
   end
 end
diff --git a/spec/models/form/sales/questions/person4_age_known_spec.rb b/spec/models/form/sales/questions/person4_age_known_spec.rb
index 559ef62b5..e8fa286d7 100644
--- a/spec/models/form/sales/questions/person4_age_known_spec.rb
+++ b/spec/models/form/sales/questions/person4_age_known_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person4AgeKnown, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age5_known" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person4AgeKnown, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age6_known")
+    expect(question.id).to eq("age5_known")
   end
 
   it "has the correct header" do
@@ -40,7 +40,7 @@ RSpec.describe Form::Sales::Questions::Person4AgeKnown, type: :model do
 
   it "has correct conditional for" do
     expect(question.conditional_for).to eq({
-      "age6" => [0],
+      "age5" => [0],
     })
   end
 
@@ -51,12 +51,12 @@ RSpec.describe Form::Sales::Questions::Person4AgeKnown, type: :model do
   it "has the correct hidden_in_check_answers" do
     expect(question.hidden_in_check_answers).to eq(
       {
-        "depends_on" => [{ "age6_known" => 0 }, { "age6_known" => 1 }],
+        "depends_on" => [{ "age5_known" => 0 }, { "age5_known" => 1 }],
       },
     )
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(6)
+    expect(question.check_answers_card_number).to eq(5)
   end
 end
diff --git a/spec/models/form/sales/questions/person4_age_spec.rb b/spec/models/form/sales/questions/person4_age_spec.rb
index d1c6ebead..3686a9ad8 100644
--- a/spec/models/form/sales/questions/person4_age_spec.rb
+++ b/spec/models/form/sales/questions/person4_age_spec.rb
@@ -3,7 +3,7 @@ require "rails_helper"
 RSpec.describe Form::Sales::Questions::Person4Age, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
-  let(:question_id) { nil }
+  let(:question_id) { "age5" }
   let(:question_definition) { nil }
   let(:page) { instance_double(Form::Page) }
 
@@ -12,7 +12,7 @@ RSpec.describe Form::Sales::Questions::Person4Age, type: :model do
   end
 
   it "has the correct id" do
-    expect(question.id).to eq("age6")
+    expect(question.id).to eq("age5")
   end
 
   it "has the correct header" do
@@ -41,12 +41,12 @@ RSpec.describe Form::Sales::Questions::Person4Age, type: :model do
 
   it "has the correct inferred check answers value" do
     expect(question.inferred_check_answers_value).to eq({
-      "condition" => { "age6_known" => 1 },
+      "condition" => { "age5_known" => 1 },
       "value" => "Not known",
     })
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(6)
+    expect(question.check_answers_card_number).to eq(5)
   end
 end
diff --git a/spec/models/form/sales/questions/person4_known_spec.rb b/spec/models/form/sales/questions/person4_known_spec.rb
index f23dfa741..829ccc869 100644
--- a/spec/models/form/sales/questions/person4_known_spec.rb
+++ b/spec/models/form/sales/questions/person4_known_spec.rb
@@ -4,7 +4,7 @@ RSpec.describe Form::Sales::Questions::Person4Known, type: :model do
   subject(:question) { described_class.new(question_id, question_definition, page) }
 
   let(:question_id) { nil }
-  let(:question_definition) { nil }
+  let(:question_definition) { { check_answers_card_number: 5 } }
   let(:page) { instance_double(Form::Page) }
 
   it "has correct page" do
@@ -59,6 +59,6 @@ RSpec.describe Form::Sales::Questions::Person4Known, type: :model do
   end
 
   it "has the correct check_answers_card_number" do
-    expect(question.check_answers_card_number).to eq(6)
+    expect(question.check_answers_card_number).to eq(5)
   end
 end
diff --git a/spec/models/form/sales/subsections/household_characteristics_spec.rb b/spec/models/form/sales/subsections/household_characteristics_spec.rb
index 1fbe02a1d..55927ab73 100644
--- a/spec/models/form/sales/subsections/household_characteristics_spec.rb
+++ b/spec/models/form/sales/subsections/household_characteristics_spec.rb
@@ -35,17 +35,25 @@ RSpec.describe Form::Sales::Subsections::HouseholdCharacteristics, type: :model
         buyer_2_live_in_property
         number_of_others_in_property
         person_1_known
+        person_1_known_joint_purchase
         person_1_relationship_to_buyer_1
         person_1_relationship_to_buyer_1_joint_purchase
         person_1_age
+        person_1_age_joint_purchase
         person_1_gender_identity
         person_1_gender_identity_joint_purchase
         person_2_known
+        person_2_known_joint_purchase
         person_2_age
+        person_2_age_joint_purchase
         person_3_known
+        person_3_known_joint_purchase
         person_3_age
+        person_3_age_joint_purchase
         person_4_known
+        person_4_known_joint_purchase
         person_4_age
+        person_4_age_joint_purchase
       ],
     )
   end
diff --git a/spec/models/form_handler_spec.rb b/spec/models/form_handler_spec.rb
index 7656fe7fb..406b7f212 100644
--- a/spec/models/form_handler_spec.rb
+++ b/spec/models/form_handler_spec.rb
@@ -52,14 +52,14 @@ RSpec.describe FormHandler do
     it "is able to load a current sales form" do
       form = form_handler.get_form("current_sales")
       expect(form).to be_a(Form)
-      expect(form.pages.count).to eq(56)
+      expect(form.pages.count).to eq(66)
       expect(form.name).to eq("2022_2023_sales")
     end
 
     it "is able to load a previous sales form" do
       form = form_handler.get_form("previous_sales")
       expect(form).to be_a(Form)
-      expect(form.pages.count).to eq(56)
+      expect(form.pages.count).to eq(66)
       expect(form.name).to eq("2021_2022_sales")
     end
   end