Browse Source

Make clear the counts are only for affected records

pull/2685/head
Manny Dinssa 2 years ago
parent
commit
79a8f0b7d7
  1. 4
      lib/tasks/duplicate_rent_periods.rake
  2. 4
      spec/lib/tasks/dupicate_rent_periods_spec.rb

4
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"

4
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

Loading…
Cancel
Save