From 79a8f0b7d706f4a150b03a744bb33a55c34bbe69 Mon Sep 17 00:00:00 2001 From: Manny Dinssa <44172848+Dinssa@users.noreply.github.com> Date: Thu, 10 Oct 2024 17:14:17 +0100 Subject: [PATCH] Make clear the counts are only for affected records --- lib/tasks/duplicate_rent_periods.rake | 4 ++-- spec/lib/tasks/dupicate_rent_periods_spec.rb | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/tasks/duplicate_rent_periods.rake b/lib/tasks/duplicate_rent_periods.rake index dce8b35be..b2f84d3c0 100644 --- a/lib/tasks/duplicate_rent_periods.rake +++ b/lib/tasks/duplicate_rent_periods.rake @@ -38,8 +38,8 @@ task find_redundant_rent_periods: :environment do Rails.logger.info "Total number of records: #{OrganisationRentPeriod.count}" Rails.logger.info "Number of affected records: #{result[:affected_records].size}" - Rails.logger.info "Number of records to delete: #{result[:duplicate_ids].size}" - Rails.logger.info "Number of records to keep: #{result[:to_keep_ids].size}" + Rails.logger.info "Number of affected records to delete: #{result[:duplicate_ids].size}" + Rails.logger.info "Number of affected records to keep: #{result[:to_keep_ids].size}" end desc "Delete duplicate rent periods" diff --git a/spec/lib/tasks/dupicate_rent_periods_spec.rb b/spec/lib/tasks/dupicate_rent_periods_spec.rb index d3bbe1013..00dd05a67 100644 --- a/spec/lib/tasks/dupicate_rent_periods_spec.rb +++ b/spec/lib/tasks/dupicate_rent_periods_spec.rb @@ -26,8 +26,8 @@ RSpec.describe "duplicate_rent_periods" do expect(Rails.logger).to have_received(:info).with(include("Total number of records: 17")) expect(Rails.logger).to have_received(:info).with(include("Number of affected records: 11")) - expect(Rails.logger).to have_received(:info).with(include("Number of records to delete: 7")) - expect(Rails.logger).to have_received(:info).with(include("Number of records to keep: 4")) + expect(Rails.logger).to have_received(:info).with(include("Number of affected records to delete: 7")) + expect(Rails.logger).to have_received(:info).with(include("Number of affected records to keep: 4")) end end