Browse Source

feat: test fix wip

pull/1509/head
natdeanlewissoftwire 3 years ago
parent
commit
4242ea1b10
  1. 2
      spec/factories/lettings_log.rb
  2. 4
      spec/features/form/tasklist_page_spec.rb
  3. 2
      spec/features/lettings_log_spec.rb
  4. 2
      spec/models/validations/financial_validations_spec.rb
  5. 4
      spec/requests/form_controller_spec.rb

2
spec/factories/lettings_log.rb

@ -3,7 +3,7 @@ FactoryBot.define do
created_by { FactoryBot.create(:user) } created_by { FactoryBot.create(:user) }
owning_organisation { created_by.organisation } owning_organisation { created_by.organisation }
managing_organisation { created_by.organisation } managing_organisation { created_by.organisation }
trait :about_completed do trait :setup_completed do
renewal { 0 } renewal { 0 }
needstype { 1 } needstype { 1 }
rent_type { 1 } rent_type { 1 }

4
spec/features/form/tasklist_page_spec.rb

@ -33,7 +33,7 @@ RSpec.describe "Task List" do
let(:setup_completed_log) do let(:setup_completed_log) do
FactoryBot.create( FactoryBot.create(
:lettings_log, :lettings_log,
:about_completed, :setup_completed,
owning_organisation: user.organisation, owning_organisation: user.organisation,
managing_organisation: user.organisation, managing_organisation: user.organisation,
created_by: user, created_by: user,
@ -69,7 +69,7 @@ RSpec.describe "Task List" do
it "shows number of completed sections if one section is completed" do it "shows number of completed sections if one section is completed" do
visit("/lettings-logs/#{setup_completed_log.id}") visit("/lettings-logs/#{setup_completed_log.id}")
expect(page).to have_content("1 of 8 sections completed.") expect(page).to have_content("1 of 8 subsections completed.")
end end
it "show skip link for next incomplete section" do it "show skip link for next incomplete section" do

2
spec/features/lettings_log_spec.rb

@ -95,7 +95,7 @@ RSpec.describe "Lettings Log Features" do
end end
context "when visiting a subsection check answers page" do context "when visiting a subsection check answers page" do
let(:lettings_log) { FactoryBot.create(:lettings_log, :about_completed) } let(:lettings_log) { FactoryBot.create(:lettings_log, :setup_completed) }
it "has the correct breadcrumbs with the correct links" do it "has the correct breadcrumbs with the correct links" do
visit lettings_log_setup_check_answers_path(lettings_log) visit lettings_log_setup_check_answers_path(lettings_log)

2
spec/models/validations/financial_validations_spec.rb

@ -86,7 +86,7 @@ RSpec.describe Validations::FinancialValidations do
end end
context "when outstanding rent or charges is yes" do context "when outstanding rent or charges is yes" do
let(:record) { FactoryBot.create(:lettings_log, :about_completed, startdate: Time.zone.now) } let(:record) { FactoryBot.create(:lettings_log, :setup_completed, startdate: Time.zone.now) }
it "expects that a shortfall is provided" do it "expects that a shortfall is provided" do
record.hbrentshortfall = 1 record.hbrentshortfall = 1

4
spec/requests/form_controller_spec.rb

@ -15,7 +15,7 @@ RSpec.describe FormController, type: :request do
let(:setup_complete_lettings_log) do let(:setup_complete_lettings_log) do
create( create(
:lettings_log, :lettings_log,
:about_completed, :setup_completed,
status: 1, status: 1,
created_by: user, created_by: user,
) )
@ -203,7 +203,7 @@ RSpec.describe FormController, type: :request do
context "with form pages" do context "with form pages" do
context "when forms exist for multiple years" do context "when forms exist for multiple years" do
let(:lettings_log_year_1) { create(:lettings_log, owning_organisation: organisation, created_by: user) } let(:lettings_log_year_1) { create(:lettings_log, owning_organisation: organisation, created_by: user) }
let(:lettings_log_year_2) { create(:lettings_log, :about_completed, startdate: Time.zone.local(2022, 5, 1), owning_organisation: organisation, created_by: user) } let(:lettings_log_year_2) { create(:lettings_log, :setup_completed, startdate: Time.zone.local(2022, 5, 1), owning_organisation: organisation, created_by: user) }
before do before do
Timecop.freeze(Time.zone.local(2021, 5, 1)) Timecop.freeze(Time.zone.local(2021, 5, 1))

Loading…
Cancel
Save