From 3d413c5781577903963259de3cc0765a0c7bc60a Mon Sep 17 00:00:00 2001
From: Arthur Campbell <51094020+arfacamble@users.noreply.github.com>
Date: Fri, 7 Jun 2024 11:11:36 +0100
Subject: [PATCH] CLDC-3503 rent periods bug (#2454)

* ensure in the update method that it is not possible to delete rent periods that are in use by that organisation

* lint

---------

Co-authored-by: Kat <katrina@kosiak.co.uk>
---
 app/controllers/organisations_controller.rb   |  3 +-
 ...anisations_controller_rent_periods_spec.rb | 32 +++++++++++++++++--
 2 files changed, 31 insertions(+), 4 deletions(-)

diff --git a/app/controllers/organisations_controller.rb b/app/controllers/organisations_controller.rb
index 583a64fdc..357a5f272 100644
--- a/app/controllers/organisations_controller.rb
+++ b/app/controllers/organisations_controller.rb
@@ -136,7 +136,8 @@ class OrganisationsController < ApplicationController
         else
           flash[:notice] = I18n.t("organisation.updated")
         end
-        rent_periods_to_delete = rent_period_params[:all_rent_periods] - selected_rent_periods
+        used_rent_periods = @organisation.lettings_logs.pluck(:period).uniq.compact.map(&:to_s)
+        rent_periods_to_delete = rent_period_params[:all_rent_periods] - selected_rent_periods - used_rent_periods
         OrganisationRentPeriod.transaction do
           selected_rent_periods.each { |period| OrganisationRentPeriod.create(organisation: @organisation, rent_period: period) }
           OrganisationRentPeriod.where(organisation: @organisation, rent_period: rent_periods_to_delete).destroy_all
diff --git a/spec/requests/OrganisationsController/organisations_controller_rent_periods_spec.rb b/spec/requests/OrganisationsController/organisations_controller_rent_periods_spec.rb
index 390b747b8..126a3a624 100644
--- a/spec/requests/OrganisationsController/organisations_controller_rent_periods_spec.rb
+++ b/spec/requests/OrganisationsController/organisations_controller_rent_periods_spec.rb
@@ -96,11 +96,9 @@ RSpec.describe OrganisationsController, type: :request do
       }
     end
 
-    before do
+    it "creates and destroys organisation rent periods as appropriate" do
       create(:organisation_rent_period, organisation:, rent_period: initially_checked_rent_period_id)
-    end
 
-    it "creates and destroys organisation rent periods as appropriate" do
       rent_periods = Organisation.includes(:organisation_rent_periods)
                                  .find(organisation.id)
                                  .organisation_rent_periods
@@ -115,5 +113,33 @@ RSpec.describe OrganisationsController, type: :request do
       expect(rent_periods.count).to be 1
       expect(rent_periods.first.rent_period.to_s).to eq initially_unchecked_rent_period_id
     end
+
+    context "when a user creates a log with a certain rent period while another user is on the edit page" do
+      let(:period_to_not_delete) { "4" }
+      let(:period_to_create) { "5" }
+      let(:coordinator) { create(:user, :data_coordinator, organisation:) }
+      let(:params) do
+        {
+          "organisation": {
+            name: organisation.name,
+            rent_periods: [period_to_create],
+            all_rent_periods: [period_to_not_delete],
+          },
+        }
+      end
+
+      it "does not delete rent periods that are in use by that organisation" do
+        create(:organisation_rent_period, organisation:, rent_period: period_to_not_delete)
+        create(:lettings_log, :setup_completed, period: period_to_not_delete, assigned_to: coordinator, owning_organisation: organisation, managing_organisation: organisation)
+
+        patch organisation_path(organisation, headers:, params:)
+
+        rent_periods = Organisation.includes(:organisation_rent_periods)
+                                   .find(organisation.id)
+                                   .organisation_rent_periods
+
+        expect(rent_periods.map(&:rent_period).map(&:to_s)).to match_array([period_to_create, period_to_not_delete])
+      end
+    end
   end
 end