From 2c12a50ba35d3e2045f54952770493c04ff43e1d Mon Sep 17 00:00:00 2001
From: Arthur Campbell <arfa.camble@gmail.com>
Date: Mon, 13 Mar 2023 17:07:24 +0000
Subject: [PATCH] write a test covering the fix

---
 .../requests/lettings_logs_controller_spec.rb | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/spec/requests/lettings_logs_controller_spec.rb b/spec/requests/lettings_logs_controller_spec.rb
index 066cd1f35..cb6ccbe0c 100644
--- a/spec/requests/lettings_logs_controller_spec.rb
+++ b/spec/requests/lettings_logs_controller_spec.rb
@@ -891,6 +891,36 @@ RSpec.describe LettingsLogsController, type: :request do
             end
           end
 
+          context "when a lettings log is for a renewal of supported housing, so property information does not need to show" do
+            let(:lettings_log) {
+              FactoryBot.create(:lettings_log,
+                owning_organisation: user.organisation,
+                managing_organisation: user.organisation,
+                created_by: user,
+                startdate: Time.zone.now,
+                renewal: 1,
+                needstype: 2,
+                rent_type: 3,
+                postcode_known: 0
+              )}
+
+            before do
+              sign_in user
+            end
+
+            around do |example|
+              FormHandler.instance.use_real_forms!
+              example.run
+              FormHandler.instance.use_fake_forms!
+            end
+
+            it "does not crash the app if postcode_known is not nil" do
+              expect {
+                get "/lettings-logs/#{lettings_log.id}", headers:, params: {}
+              }.not_to raise_error(ActionView::Template::Error)
+            end
+          end
+
           context "with a lettings log with a single section complete" do
             let(:section_completed_lettings_log) do
               FactoryBot.create(