From dd79527e434eb1d5db5b24d95093c4c5f25f2ee6 Mon Sep 17 00:00:00 2001 From: Kat Date: Thu, 4 Apr 2024 16:45:57 +0100 Subject: [PATCH] Update test selectord and urls --- spec/features/lettings_log_spec.rb | 4 ++-- spec/requests/form_controller_spec.rb | 16 ++++++++-------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/spec/features/lettings_log_spec.rb b/spec/features/lettings_log_spec.rb index 997d209fa..3c5f32667 100644 --- a/spec/features/lettings_log_spec.rb +++ b/spec/features/lettings_log_spec.rb @@ -152,7 +152,7 @@ RSpec.describe "Lettings Log Features" do click_link("Set up this lettings log") select(support_user.organisation.name, from: "lettings-log-owning-organisation-id-field") click_button("Save and continue") - select("#{support_user.name} (#{support_user.email})", from: "lettings-log-created-by-id-field") + select("#{support_user.name} (#{support_user.email})", from: "lettings-log-assigned-to-id-field") click_button("Save and continue") log_id = page.current_path.scan(/\d/).join visit("lettings-logs/#{log_id}/setup/check-answers") @@ -333,7 +333,7 @@ RSpec.describe "Lettings Log Features" do visit("/lettings-logs") click_button("Create a new lettings log") click_link("Set up this lettings log") - select(user.name, from: "lettings-log-created-by-id-field") + select(user.name, from: "lettings-log-assigned-to-id-field") click_button("Save and continue") log_id = page.current_path.scan(/\d/).join expect(page).to have_current_path("/lettings-logs/#{log_id}/needs-type") diff --git a/spec/requests/form_controller_spec.rb b/spec/requests/form_controller_spec.rb index f7878c786..4d8839a98 100644 --- a/spec/requests/form_controller_spec.rb +++ b/spec/requests/form_controller_spec.rb @@ -105,7 +105,7 @@ RSpec.describe FormController, type: :request do it "resets created by and renders the next page" do post "/lettings-logs/#{lettings_log.id}/net-income", params: params - expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}/created-by") + expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}/assigned-to") follow_redirect! lettings_log.reload expect(lettings_log.assigned_to).to eq(nil) @@ -130,7 +130,7 @@ RSpec.describe FormController, type: :request do it "does not reset created by" do post "/lettings-logs/#{lettings_log.id}/net-income", params: params - expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}/created-by") + expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}/assigned-to") follow_redirect! lettings_log.reload expect(lettings_log.assigned_to).to eq(user) @@ -155,7 +155,7 @@ RSpec.describe FormController, type: :request do it "sets managing organisation to owning organisation" do post "/lettings-logs/#{lettings_log.id}/stock-owner", params: params - expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}/created-by") + expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}/assigned-to") follow_redirect! lettings_log.reload expect(lettings_log.owning_organisation).to eq(managing_organisation) @@ -182,7 +182,7 @@ RSpec.describe FormController, type: :request do it "correctly sets owning organisation" do post "/sales-logs/#{sales_log.id}/owning-organisation", params: params - expect(response).to redirect_to("/sales-logs/#{sales_log.id}/created-by") + expect(response).to redirect_to("/sales-logs/#{sales_log.id}/assigned-to") follow_redirect! sales_log.reload expect(sales_log.owning_organisation).to eq(managing_organisation) @@ -209,7 +209,7 @@ RSpec.describe FormController, type: :request do it "does not reset created by" do post "/sales-logs/#{sales_log.id}/owning-organisation", params: params - expect(response).to redirect_to("/sales-logs/#{sales_log.id}/created-by") + expect(response).to redirect_to("/sales-logs/#{sales_log.id}/assigned-to") follow_redirect! sales_log.reload expect(sales_log.assigned_to).to eq(assigned_to) @@ -238,7 +238,7 @@ RSpec.describe FormController, type: :request do it "does not reset created by" do post "/sales-logs/#{sales_log.id}/owning-organisation", params: params - expect(response).to redirect_to("/sales-logs/#{sales_log.id}/created-by") + expect(response).to redirect_to("/sales-logs/#{sales_log.id}/assigned-to") follow_redirect! sales_log.reload expect(sales_log.assigned_to).to eq(assigned_to) @@ -527,14 +527,14 @@ RSpec.describe FormController, type: :request do let(:user) { create(:user, :support) } it "routes to the page" do - get "/lettings-logs/#{lettings_log.id}/created-by" + get "/lettings-logs/#{lettings_log.id}/assigned-to" expect(response).to have_http_status(:ok) end end context "when the dependency is not met" do it "redirects to the tasklist page" do - get "/lettings-logs/#{lettings_log.id}/created-by" + get "/lettings-logs/#{lettings_log.id}/assigned-to" expect(response).to redirect_to("/lettings-logs/#{lettings_log.id}") end end