Browse Source

Route affected logs to tenancy start date question

pull/1035/head
Kat 4 years ago
parent
commit
0e4ba1863d
  1. 2
      app/controllers/lettings_logs_controller.rb
  2. 15
      spec/requests/lettings_logs_controller_spec.rb

2
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

15
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

Loading…
Cancel
Save