Browse Source
* Add person1 working situation question * Add ecstat3 column to the sales log * migration * Add the rest of ecstat questions * Update schema * tests * Reuse pages and questions * Reuse the same page * refactor indexes * Add other ecstat columns * Extract person methods to parent classpull/1092/head v0.2.28
kosiakkatrina
2 years ago
committed by
GitHub
12 changed files with 536 additions and 4 deletions
@ -0,0 +1,28 @@
|
||||
class Form::Sales::Pages::PersonWorkingSituation < ::Form::Page |
||||
def initialize(id, hsh, subsection) |
||||
super |
||||
@header = "" |
||||
@description = "" |
||||
@subsection = subsection |
||||
@depends_on = [ |
||||
{ "details_known_#{person_display_number(PERSON_INDEX)}" => 1, "jointpur" => joint_purchase? ? 1 : 2 }, |
||||
] |
||||
end |
||||
|
||||
def questions |
||||
@questions ||= [ |
||||
Form::Sales::Questions::PersonWorkingSituation.new("ecstat#{person_database_number(PERSON_INDEX)}", nil, self), |
||||
] |
||||
end |
||||
|
||||
PERSON_INDEX = { |
||||
"person_1_working_situation" => 2, |
||||
"person_2_working_situation" => 3, |
||||
"person_3_working_situation" => 4, |
||||
"person_4_working_situation" => 5, |
||||
"person_1_working_situation_joint_purchase" => 3, |
||||
"person_2_working_situation_joint_purchase" => 4, |
||||
"person_3_working_situation_joint_purchase" => 5, |
||||
"person_4_working_situation_joint_purchase" => 6, |
||||
}.freeze |
||||
end |
@ -0,0 +1,33 @@
|
||||
class Form::Sales::Questions::PersonWorkingSituation < ::Form::Question |
||||
def initialize(id, hsh, page) |
||||
super |
||||
@check_answer_label = "Person #{person_display_number(PERSON_INDEX)}’s working situation" |
||||
@header = "Which of these best describes Person #{person_display_number(PERSON_INDEX)}’s working situation?" |
||||
@type = "radio" |
||||
@page = page |
||||
@answer_options = ANSWER_OPTIONS |
||||
@check_answers_card_number = person_database_number(PERSON_INDEX) |
||||
end |
||||
|
||||
ANSWER_OPTIONS = { |
||||
"2" => { "value" => "Part-time - Less than 30 hours" }, |
||||
"1" => { "value" => "Full-time - 30 hours or more" }, |
||||
"3" => { "value" => "In government training into work, such as New Deal" }, |
||||
"4" => { "value" => "Jobseeker" }, |
||||
"6" => { "value" => "Not seeking work" }, |
||||
"8" => { "value" => "Unable to work due to long term sick or disability" }, |
||||
"5" => { "value" => "Retired" }, |
||||
"0" => { "value" => "Other" }, |
||||
"10" => { "value" => "Buyer prefers not to say" }, |
||||
"7" => { "value" => "Full-time student" }, |
||||
"9" => { "value" => "Child under 16" }, |
||||
}.freeze |
||||
|
||||
PERSON_INDEX = { |
||||
"ecstat2" => 2, |
||||
"ecstat3" => 3, |
||||
"ecstat4" => 4, |
||||
"ecstat5" => 5, |
||||
"ecstat6" => 6, |
||||
}.freeze |
||||
end |
@ -0,0 +1,10 @@
|
||||
class AddEcstat3ToSales < ActiveRecord::Migration[7.0] |
||||
def change |
||||
change_table :sales_logs, bulk: true do |t| |
||||
t.column :ecstat3, :integer |
||||
t.column :ecstat4, :integer |
||||
t.column :ecstat5, :integer |
||||
t.column :ecstat6, :integer |
||||
end |
||||
end |
||||
end |
@ -0,0 +1,168 @@
|
||||
require "rails_helper" |
||||
|
||||
RSpec.describe Form::Sales::Pages::PersonWorkingSituation, type: :model do |
||||
subject(:page) { described_class.new(page_id, page_definition, subsection) } |
||||
|
||||
let(:page_definition) { nil } |
||||
let(:subsection) { instance_double(Form::Subsection) } |
||||
|
||||
context "without joint purchase" do |
||||
let(:page_id) { "person_1_working_situation" } |
||||
|
||||
it "has correct subsection" do |
||||
expect(page.subsection).to eq(subsection) |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(page.header).to eq("") |
||||
end |
||||
|
||||
it "has the correct description" do |
||||
expect(page.description).to eq("") |
||||
end |
||||
|
||||
context "with person 1" do |
||||
let(:page_id) { "person_1_working_situation" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat2]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_1_working_situation") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_1" => 1, "jointpur" => 2 }]) |
||||
end |
||||
end |
||||
|
||||
context "with person 2" do |
||||
let(:page_id) { "person_2_working_situation" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat3]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_2_working_situation") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_2" => 1, "jointpur" => 2 }]) |
||||
end |
||||
end |
||||
|
||||
context "with person 3" do |
||||
let(:page_id) { "person_3_working_situation" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat4]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_3_working_situation") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_3" => 1, "jointpur" => 2 }]) |
||||
end |
||||
end |
||||
|
||||
context "with person 4" do |
||||
let(:page_id) { "person_4_working_situation" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat5]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_4_working_situation") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_4" => 1, "jointpur" => 2 }]) |
||||
end |
||||
end |
||||
end |
||||
|
||||
context "with joint purchase" do |
||||
let(:page_id) { "person_1_working_situation_joint_purchase" } |
||||
|
||||
it "has correct subsection" do |
||||
expect(page.subsection).to eq(subsection) |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(page.header).to eq("") |
||||
end |
||||
|
||||
it "has the correct description" do |
||||
expect(page.description).to eq("") |
||||
end |
||||
|
||||
context "with person 1" do |
||||
let(:page_id) { "person_1_working_situation_joint_purchase" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat3]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_1_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_1" => 1, "jointpur" => 1 }]) |
||||
end |
||||
end |
||||
|
||||
context "with person 2" do |
||||
let(:page_id) { "person_2_working_situation_joint_purchase" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat4]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_2_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_2" => 1, "jointpur" => 1 }]) |
||||
end |
||||
end |
||||
|
||||
context "with person 3" do |
||||
let(:page_id) { "person_3_working_situation_joint_purchase" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat5]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_3_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_3" => 1, "jointpur" => 1 }]) |
||||
end |
||||
end |
||||
|
||||
context "with person 4" do |
||||
let(:page_id) { "person_4_working_situation_joint_purchase" } |
||||
|
||||
it "has correct questions" do |
||||
expect(page.questions.map(&:id)).to eq(%w[ecstat6]) |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(page.id).to eq("person_4_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has correct depends_on" do |
||||
expect(page.depends_on).to eq([{ "details_known_4" => 1, "jointpur" => 1 }]) |
||||
end |
||||
end |
||||
end |
||||
end |
@ -0,0 +1,245 @@
|
||||
require "rails_helper" |
||||
|
||||
RSpec.describe Form::Sales::Questions::PersonWorkingSituation, type: :model do |
||||
subject(:question) { described_class.new(question_id, question_definition, page) } |
||||
|
||||
let(:question_id) { "ecstat2" } |
||||
let(:question_definition) { nil } |
||||
let(:page) { instance_double(Form::Page) } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_1_working_situation") |
||||
end |
||||
|
||||
it "has correct page" do |
||||
expect(question.page).to eq(page) |
||||
end |
||||
|
||||
it "has the correct type" do |
||||
expect(question.type).to eq("radio") |
||||
end |
||||
|
||||
it "is not marked as derived" do |
||||
expect(question.derived?).to be false |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(2) |
||||
end |
||||
|
||||
it "has the correct answer_options" do |
||||
expect(question.answer_options).to eq({ |
||||
"2" => { "value" => "Part-time - Less than 30 hours" }, |
||||
"1" => { "value" => "Full-time - 30 hours or more" }, |
||||
"3" => { "value" => "In government training into work, such as New Deal" }, |
||||
"4" => { "value" => "Jobseeker" }, |
||||
"6" => { "value" => "Not seeking work" }, |
||||
"8" => { "value" => "Unable to work due to long term sick or disability" }, |
||||
"5" => { "value" => "Retired" }, |
||||
"0" => { "value" => "Other" }, |
||||
"10" => { "value" => "Buyer prefers not to say" }, |
||||
"7" => { "value" => "Full-time student" }, |
||||
"9" => { "value" => "Child under 16" }, |
||||
}) |
||||
end |
||||
|
||||
context "when person 1" do |
||||
context "and not joint purchase" do |
||||
let(:question_id) { "ecstat2" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_1_working_situation") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat2") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 1’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 1’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(2) |
||||
end |
||||
end |
||||
|
||||
context "and joint purchase" do |
||||
let(:question_id) { "ecstat3" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_1_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat3") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 1’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 1’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(3) |
||||
end |
||||
end |
||||
end |
||||
|
||||
context "when person 2" do |
||||
context "and not joint purchase" do |
||||
let(:question_id) { "ecstat3" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_2_working_situation") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat3") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 2’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 2’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(3) |
||||
end |
||||
end |
||||
|
||||
context "and joint purchase" do |
||||
let(:question_id) { "ecstat4" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_2_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat4") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 2’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 2’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(4) |
||||
end |
||||
end |
||||
end |
||||
|
||||
context "when person 3" do |
||||
context "and not joint purchase" do |
||||
let(:question_id) { "ecstat4" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_3_working_situation") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat4") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 3’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 3’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(4) |
||||
end |
||||
end |
||||
|
||||
context "and joint purchase" do |
||||
let(:question_id) { "ecstat5" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_3_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat5") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 3’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 3’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(5) |
||||
end |
||||
end |
||||
end |
||||
|
||||
context "when person 4" do |
||||
context "and not joint purchase" do |
||||
let(:question_id) { "ecstat5" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_4_working_situation") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat5") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 4’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 4’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(5) |
||||
end |
||||
end |
||||
|
||||
context "and joint purchase" do |
||||
let(:question_id) { "ecstat6" } |
||||
|
||||
before do |
||||
allow(page).to receive(:id).and_return("person_4_working_situation_joint_purchase") |
||||
end |
||||
|
||||
it "has the correct id" do |
||||
expect(question.id).to eq("ecstat6") |
||||
end |
||||
|
||||
it "has the correct header" do |
||||
expect(question.header).to eq("Which of these best describes Person 4’s working situation?") |
||||
end |
||||
|
||||
it "has the correct check_answer_label" do |
||||
expect(question.check_answer_label).to eq("Person 4’s working situation") |
||||
end |
||||
|
||||
it "has expected check answers card number" do |
||||
expect(question.check_answers_card_number).to eq(6) |
||||
end |
||||
end |
||||
end |
||||
end |
Loading…
Reference in new issue