From 749348e21bb053ee5427bc45e19879914edfe93f Mon Sep 17 00:00:00 2001 From: Kat Date: Fri, 19 Jan 2024 09:01:28 +0000 Subject: [PATCH] Do not update updated_at to prevent export --- lib/tasks/set_duplicate_references.rake | 8 ++++++-- spec/lib/tasks/set_duplicate_references_spec.rb | 4 ++++ 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/lib/tasks/set_duplicate_references.rake b/lib/tasks/set_duplicate_references.rake index fb01c3950..d1fc9b18c 100644 --- a/lib/tasks/set_duplicate_references.rake +++ b/lib/tasks/set_duplicate_references.rake @@ -5,7 +5,9 @@ task set_duplicate_references: :environment do next if duplicate_set.any? { |_log_id| SalesLog.exists?(duplicate_set_id:) } duplicate_set.each do |log_id| - SalesLog.find(log_id).update!(duplicate_set_id:) + log = SalesLog.find(log_id) + log.duplicate_set_id = duplicate_set_id + log.save!(touch: false, validate: false) end end @@ -14,7 +16,9 @@ task set_duplicate_references: :environment do next if duplicate_set.any? { |_log_id| LettingsLog.exists?(duplicate_set_id:) } duplicate_set.each do |log_id| - LettingsLog.find(log_id).update!(duplicate_set_id:) + log = LettingsLog.find(log_id) + log.duplicate_set_id = duplicate_set_id + log.save!(touch: false, validate: false) end end end diff --git a/spec/lib/tasks/set_duplicate_references_spec.rb b/spec/lib/tasks/set_duplicate_references_spec.rb index 3346caa3e..32918b974 100644 --- a/spec/lib/tasks/set_duplicate_references_spec.rb +++ b/spec/lib/tasks/set_duplicate_references_spec.rb @@ -20,6 +20,7 @@ RSpec.describe "set_duplicate_references" do let!(:sales_log_without_duplicates) { create(:sales_log, created_by: user) } it "creates duplicate references for sales logs" do + initial_sales_log_updated_at = sales_log.updated_at expect(sales_log.duplicates.count).to eq(0) expect(sales_log.duplicate_set_id).to be_nil expect(duplicate_sales_log.duplicates.count).to eq(0) @@ -41,6 +42,7 @@ RSpec.describe "set_duplicate_references" do expect(sales_log_without_duplicates.duplicates.count).to eq(0) expect(sales_log.duplicate_set_id).to eq(duplicate_sales_log.duplicate_set_id) expect(sales_log.duplicate_set_id).to eq(second_duplicate_sales_log.duplicate_set_id) + expect(sales_log.updated_at).to eq(initial_sales_log_updated_at) end end @@ -126,6 +128,7 @@ RSpec.describe "set_duplicate_references" do let!(:lettings_log_without_duplicates) { create(:lettings_log, created_by: user) } it "creates duplicate references for lettings logs" do + initial_lettings_log_updated_at = lettings_log.updated_at expect(lettings_log.duplicates.count).to eq(0) expect(lettings_log.duplicate_set_id).to be_nil expect(duplicate_lettings_log.duplicates.count).to eq(0) @@ -148,6 +151,7 @@ RSpec.describe "set_duplicate_references" do expect(lettings_log_without_duplicates.duplicate_set_id).to be_nil expect(lettings_log.duplicate_set_id).to eq(duplicate_lettings_log.duplicate_set_id) expect(lettings_log.duplicate_set_id).to eq(second_duplicate_lettings_log.duplicate_set_id) + expect(lettings_log.updated_at).to eq(initial_lettings_log_updated_at) end end