diff --git a/app/models/user.rb b/app/models/user.rb index 5d67e6854..cb4432cab 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -208,6 +208,8 @@ class User < ApplicationRecord targeted_role.merge(old_role) end end + + old_user.destroy end private diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index e4055cb22..9b78c1475 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -459,6 +459,11 @@ describe User, type: :model do expect(targeted_administrateur).to receive(:merge).with(administrateur) subject + + expect { instructeur.reload }.to raise_error(ActiveRecord::RecordNotFound) + expect { expert.reload }.to raise_error(ActiveRecord::RecordNotFound) + expect { administrateur.reload }.to raise_error(ActiveRecord::RecordNotFound) + expect { old_user.reload }.to raise_error(ActiveRecord::RecordNotFound) end end end