Browse Source

mark organisations as merged

pull/1801/head
Kat 3 years ago
parent
commit
53189bcbe4
  1. 6
      app/services/merge/merge_organisations_service.rb
  2. 5
      spec/services/merge/merge_organisations_service_spec.rb

6
app/services/merge/merge_organisations_service.rb

@ -16,9 +16,9 @@ class Merge::MergeOrganisationsService
merge_schemes_and_locations(merging_organisation)
merge_lettings_logs(merging_organisation)
merge_sales_logs(merging_organisation)
mark_organisation_as_merged(merging_organisation)
end
@absorbing_organisation.save!
mark_organisations_as_merged
Rails.logger.info(@users_success_message)
Rails.logger.info(@schemes_success_message)
rescue ActiveRecord::RecordInvalid => e
@ -99,8 +99,8 @@ private
end
end
def mark_organisations_as_merged
# @merging_organisations.update_all(merge_date: Time.zone.today)
def mark_organisation_as_merged(merging_organisation)
merging_organisation.update(merge_date: Time.zone.today)
end
def merge_boolean_organisation_attribute(attribute)

5
spec/services/merge/merge_organisations_service_spec.rb

@ -22,10 +22,11 @@ RSpec.describe Merge::MergeOrganisationsService do
expect(merging_organisation_user.organisation).to eq(absorbing_organisation)
end
xit "sets merge date on merged organisation" do
it "sets merge date on merged organisation" do
merge_organisations_service.call
expect(merging_organisation.merge_date).to eq(Time.zone.today)
merging_organisation.reload
expect(merging_organisation.merge_date.to_date).to eq(Time.zone.today)
end
it "combines organisation data" do

Loading…
Cancel
Save