Merge pull request #8886 from colinux/fix-delete-admin-without-published-procedure

fix(admin): peut supprimer un admin sans démarche publiée, sans service

app/models/administrateur.rb

@@ -83,6 +83,8 @@ class Administrateur < ApplicationRecord
     end
 
     procedures.with_discarded.each do |procedure|
+      next if procedure.service.nil?
+
       next_administrateur = procedure.administrateurs.where.not(id: self.id).first
       procedure.service.update(administrateur: next_administrateur)
     end

spec/models/administrateur_spec.rb

@@ -60,6 +60,15 @@ describe Administrateur, type: :model do
       expect(Service.find_by(id: procedure.service.id)).not_to be_nil
       expect(Administrateur.find_by(id: administrateur.id)).to be_nil
     end
+
+    context "proedure without service" do
+      let!(:procedure) { create(:procedure, :draft, administrateurs: [administrateur, autre_administrateur]) }
+
+      it "delete procedure without service" do
+        administrateur.delete_and_transfer_services
+        expect(Administrateur.find_by(id: administrateur.id)).to be_nil
+      end
+    end
   end
 
   describe '#merge' do