From bb4af880b2ad80ecccbbe6100db56d757fa113e7 Mon Sep 17 00:00:00 2001 From: Claire Date: Wed, 15 Jul 2026 16:41:20 +0200 Subject: [PATCH] Rename `ASSOCIATIONS_ON_SUSPEND` to `ASSOCIATIONS_ON_PURGE` (#39831) --- app/services/delete_account_service.rb | 6 +++--- spec/services/delete_account_service_spec.rb | 6 ++++++ 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/app/services/delete_account_service.rb b/app/services/delete_account_service.rb index ce998c79f9c..e9ae117ceb2 100644 --- a/app/services/delete_account_service.rb +++ b/app/services/delete_account_service.rb @@ -3,7 +3,7 @@ class DeleteAccountService < BaseService include Payloadable - ASSOCIATIONS_ON_SUSPEND = %w( + ASSOCIATIONS_ON_PURGE = %w( account_notes account_pins active_relationships @@ -304,9 +304,9 @@ class DeleteAccountService < BaseService def associations_for_destruction if keep_account_record? - ASSOCIATIONS_ON_SUSPEND + ASSOCIATIONS_ON_PURGE else - ASSOCIATIONS_ON_SUSPEND + ASSOCIATIONS_ON_DESTROY + ASSOCIATIONS_ON_PURGE + ASSOCIATIONS_ON_DESTROY end end diff --git a/spec/services/delete_account_service_spec.rb b/spec/services/delete_account_service_spec.rb index 60bcf48df37..f3eb0ca70fb 100644 --- a/spec/services/delete_account_service_spec.rb +++ b/spec/services/delete_account_service_spec.rb @@ -63,6 +63,12 @@ RSpec.describe DeleteAccountService do end end + describe 'ASSOCIATIONS_WITHOUT_SIDE_EFFECTS' do + it 'is a subset of ASSOCIATIONS_ON_PURGE' do + expect(described_class::ASSOCIATIONS_WITHOUT_SIDE_EFFECTS - described_class::ASSOCIATIONS_ON_PURGE).to eq [] + end + end + describe '#call on local account', :inline_jobs do before do stub_request(:post, remote_alice.inbox_url).to_return(status: 201)