From 0e4ba1863d1d2ceb8d51eaa21aa01c1e37526a5a Mon Sep 17 00:00:00 2001 From: Kat Date: Fri, 25 Nov 2022 12:14:01 +0000 Subject: [PATCH] Route affected logs to tenancy start date question --- app/controllers/lettings_logs_controller.rb | 2 +- spec/requests/lettings_logs_controller_spec.rb | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/app/controllers/lettings_logs_controller.rb b/app/controllers/lettings_logs_controller.rb index 9f2c07e9a..0878eb721 100644 --- a/app/controllers/lettings_logs_controller.rb +++ b/app/controllers/lettings_logs_controller.rb @@ -51,7 +51,7 @@ class LettingsLogsController < LogsController def edit @log = current_user.lettings_logs.find_by(id: params[:id]) if @log - render "logs/edit", locals: { current_user: } + @log.impacted_by_deactivation ? redirect_to(lettings_log_tenancy_start_date_path(@log)) : render("logs/edit", locals: { current_user: }) else render_not_found end diff --git a/spec/requests/lettings_logs_controller_spec.rb b/spec/requests/lettings_logs_controller_spec.rb index 382e68a20..0a2a31ca4 100644 --- a/spec/requests/lettings_logs_controller_spec.rb +++ b/spec/requests/lettings_logs_controller_spec.rb @@ -829,6 +829,21 @@ RSpec.describe LettingsLogsController, type: :request do expect(page).to have_content("You need to update 2 logs") end end + + context "when viewing a specific log affected by deactivated location" do + let!(:affected_lettings_log) { FactoryBot.create(:lettings_log, impacted_by_deactivation: true, created_by: user) } + let(:headers) { { "Accept" => "text/html" } } + + before do + allow(user).to receive(:need_two_factor_authentication?).and_return(false) + sign_in user + end + + it "routes to the tenancy date question" do + get "/lettings-logs/#{affected_lettings_log.id}", headers:, params: {} + expect(response).to redirect_to("/lettings-logs/#{affected_lettings_log.id}/tenancy-start-date") + end + end end describe "PATCH" do