fix: update destroy account extend and test

Author: Stef-Rousset

lib/decidim/cleaner/extends/commands/decidim/destroy_account.rb

@@ -12,17 +12,19 @@ module DestroyAccount
 
           # Invalidate all sessions after cleaning Decidim::User record to prevent Active Record error
           def destroy_user_account!
-            @user.name = ""
-            @user.nickname = ""
-            @user.email = ""
-            @user.delete_reason = @form.delete_reason
-            @user.admin = false if @user.admin?
-            @user.deleted_at = Time.current
-            @user.skip_reconfirmation!
-            @user.avatar.purge
-            @user.save!
+            current_user.name = ""
+            current_user.nickname = ""
+            current_user.email = ""
+            current_user.personal_url = ""
+            current_user.about = ""
+            current_user.delete_reason = @form.delete_reason
+            current_user.admin = false if current_user.admin?
+            current_user.deleted_at = Time.current
+            current_user.skip_reconfirmation!
+            current_user.avatar.purge
+            current_user.save!
 
-            @user.invalidate_all_sessions!
+            current_user.invalidate_all_sessions!
           end
         end
       end

spec/commands/decidim/destroy_account_spec.rb

@@ -4,7 +4,7 @@
 
 module Decidim
   describe DestroyAccount do
-    let(:command) { described_class.new(user, form) }
+    let(:command) { described_class.new(form) }
     let(:user) { create(:user, :confirmed) }
     let!(:identity) { create(:identity, user:) }
     let(:valid) { true }
@@ -17,7 +17,8 @@ module Decidim
     let(:form) do
       form = double(
         delete_reason: data[:delete_reason],
-        valid?: valid
+        valid?: valid,
+        current_user: user
       )
 
       form
@@ -50,11 +51,14 @@ module Decidim
         expect(user.reload.deleted_at).not_to be_nil
       end
 
-      it "set name, nickname and email to blank string" do
+      it "set name, nickname, personal_url, about and email to blank string" do
         command.call
-        expect(user.reload.name).to eq("")
-        expect(user.reload.nickname).to eq("")
-        expect(user.reload.email).to eq("")
+        user.reload
+        expect(user.name).to eq("")
+        expect(user.nickname).to eq("")
+        expect(user.email).to eq("")
+        expect(user.personal_url).to eq("")
+        expect(user.about).to eq("")
       end
 
       it "destroys the current user avatar" do