From 29009c2482529e2d67598897c56192a9dcafc2df Mon Sep 17 00:00:00 2001 From: Kat Date: Mon, 30 Jan 2023 08:21:01 +0000 Subject: [PATCH] Update check answers card headers --- .../check_answers_summary_list_card_component.rb | 2 +- spec/factories/sales_log.rb | 8 ++++---- .../form/check_answers_page_sales_logs_spec.rb | 10 +++++++--- 3 files changed, 12 insertions(+), 8 deletions(-) diff --git a/app/components/check_answers_summary_list_card_component.rb b/app/components/check_answers_summary_list_card_component.rb index 874ac8c10..af8ebd36a 100644 --- a/app/components/check_answers_summary_list_card_component.rb +++ b/app/components/check_answers_summary_list_card_component.rb @@ -30,7 +30,7 @@ class CheckAnswersSummaryListCardComponent < ViewComponent::Base when 1..number_of_buyers "Buyer #{question.check_answers_card_number}" when (number_of_buyers + 1)..(number_of_buyers + 4) - "Person #{question.check_answers_card_number - number_of_buyers}" + "Person #{question.check_answers_card_number}" end end end diff --git a/spec/factories/sales_log.rb b/spec/factories/sales_log.rb index 085d4ca03..bfce386c7 100644 --- a/spec/factories/sales_log.rb +++ b/spec/factories/sales_log.rb @@ -38,16 +38,16 @@ FactoryBot.define do ecstat2 { "1" } hholdcount { "1" } wheel { 1 } - details_known_1 { 1 } + details_known_3 { 1 } age3_known { 0 } age3 { 14 } - details_known_2 { 1 } + details_known_4 { 1 } age4_known { 0 } age4 { 18 } - details_known_3 { 1 } + details_known_5 { 1 } age5_known { 0 } age5 { 40 } - details_known_4 { 1 } + details_known_6 { 1 } age6_known { 0 } age6 { 40 } income1nk { 0 } diff --git a/spec/features/form/check_answers_page_sales_logs_spec.rb b/spec/features/form/check_answers_page_sales_logs_spec.rb index 2c6dc050d..1b3ccd364 100644 --- a/spec/features/form/check_answers_page_sales_logs_spec.rb +++ b/spec/features/form/check_answers_page_sales_logs_spec.rb @@ -43,8 +43,10 @@ RSpec.describe "Sales Log Check Answers Page" do visit("/sales-logs/#{completed_sales_log_joint_purchase.id}/#{subsection}/check-answers") assert_selector ".x-govuk-summary-card__title", text: "Buyer 1", count: 1 assert_selector ".x-govuk-summary-card__title", text: "Buyer 2", count: 1 - assert_selector ".x-govuk-summary-card__title", text: "Person 1", count: 1 + assert_selector ".x-govuk-summary-card__title", text: "Person 1", count: 0 assert_selector ".x-govuk-summary-card__title", text: "Person 2", count: 0 + assert_selector ".x-govuk-summary-card__title", text: "Person 3", count: 1 + assert_selector ".x-govuk-summary-card__title", text: "Person 4", count: 1 end end @@ -53,8 +55,10 @@ RSpec.describe "Sales Log Check Answers Page" do visit("/sales-logs/#{completed_sales_log_non_joint_purchase.id}/#{subsection}/check-answers") assert_selector ".x-govuk-summary-card__title", text: "Buyer 1", count: 1 assert_selector ".x-govuk-summary-card__title", text: "Buyer 2", count: 0 - assert_selector ".x-govuk-summary-card__title", text: "Person 1", count: 1 - assert_selector ".x-govuk-summary-card__title", text: "Person 2", count: 0 + assert_selector ".x-govuk-summary-card__title", text: "Person 1", count: 0 + assert_selector ".x-govuk-summary-card__title", text: "Person 2", count: 1 + assert_selector ".x-govuk-summary-card__title", text: "Person 3", count: 1 + assert_selector ".x-govuk-summary-card__title", text: "Person 4", count: 1 end end end