- <%= button_to "supprimer", delete_manager_user_path(page.resource), method: :delete, disabled: !page.resource.can_be_deleted?, class: "button", data: { confirm: "Confirmez-vous la suppression de l'utilisateur ?" }, title: page.resource.can_be_deleted? ? "Supprimer" : "Cet utilisateur a des dossiers dont l'instruction a commencé et ne peut être supprimé" %>
+ <%= button_to "supprimer", delete_manager_user_path(page.resource), method: :delete, disabled: !page.resource.can_be_deleted?, class: "button", data: { confirm: "Confirmez-vous la suppression de l'utilisateur ?" }, title: page.resource.can_be_deleted? ? "Supprimer" : "Cet utilisateur ne peut être supprimé. Il a des dossiers dont l'instruction a commencé ou il est administrateur ou instructeur" %>
diff --git a/config/routes.rb b/config/routes.rb
index 09f0d8fb8..e69a44a60 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -32,6 +32,7 @@ Rails.application.routes.draw do
resources :instructeurs, only: [:index, :show] do
post 'reinvite', on: :member
+ delete 'delete', on: :member
end
resources :dossiers, only: [:show]
diff --git a/spec/controllers/manager/instructeurs_controller_spec.rb b/spec/controllers/manager/instructeurs_controller_spec.rb
new file mode 100644
index 000000000..3a0164d4b
--- /dev/null
+++ b/spec/controllers/manager/instructeurs_controller_spec.rb
@@ -0,0 +1,17 @@
+describe Manager::InstructeursController, type: :controller do
+ let(:administration) { create(:administration) }
+
+ describe '#delete' do
+ let!(:instructeur) { create(:instructeur) }
+
+ before { sign_in administration }
+
+ subject { delete :delete, params: { id: instructeur.id } }
+
+ it 'deletes the instructeur' do
+ subject
+
+ expect(Instructeur.find_by(id: instructeur.id)).to be_nil
+ end
+ end
+end
diff --git a/spec/models/administrateur_spec.rb b/spec/models/administrateur_spec.rb
index 9f0c30923..3e6864466 100644
--- a/spec/models/administrateur_spec.rb
+++ b/spec/models/administrateur_spec.rb
@@ -46,6 +46,21 @@ describe Administrateur, type: :model do
end
end
+ describe '#delete_and_transfer_services' do
+ let!(:administrateur) { create(:administrateur) }
+ let!(:autre_administrateur) { create(:administrateur) }
+ let!(:procedure) { create(:procedure, :with_service, administrateurs: [administrateur, autre_administrateur]) }
+ let(:service) { procedure.service }
+
+ it "delete and transfer services to other admin" do
+ service.update(administrateur: administrateur)
+ administrateur.delete_and_transfer_services
+
+ expect(Administrateur.find_by(id: administrateur.id)).to be_nil
+ expect(service.reload.administrateur).to eq(autre_administrateur)
+ end
+ end
+
# describe '#password_complexity' do
# let(:email) { 'mail@beta.gouv.fr' }
# let(:passwords) { ['pass', '12pass23', 'démarches ', 'démarches-simple', 'démarches-simplifiées-pwd'] }
diff --git a/spec/models/instructeur_spec.rb b/spec/models/instructeur_spec.rb
index 37917bf18..78737bf89 100644
--- a/spec/models/instructeur_spec.rb
+++ b/spec/models/instructeur_spec.rb
@@ -424,9 +424,40 @@ describe Instructeur, type: :model do
it { expect(instructeur_a.procedures.all.to_ary).to eq([procedure_a]) }
end
+ describe "#can_be_deleted?" do
+ subject { instructeur.can_be_deleted? }
+
+ context 'when the instructeur is an administrateur' do
+ let!(:administrateur) { create(:administrateur) }
+ let(:instructeur) { administrateur.instructeur }
+
+ it { is_expected.to be false }
+ end
+
+ context "when the instructeur's procedures have other instructeurs" do
+ let(:instructeur_not_admin) { create(:instructeur) }
+ let(:autre_instructeur) { create(:instructeur) }
+
+ it "can be deleted" do
+ assign(procedure, instructeur_assigne: instructeur_not_admin)
+ assign(procedure, instructeur_assigne: autre_instructeur)
+ expect(autre_instructeur.can_be_deleted?).to be_truthy
+ end
+ end
+
+ context "when the instructeur's procedures is the only one" do
+ let(:instructeur_not_admin) { create :instructeur }
+ let(:autre_procedure) { create :procedure }
+ it "can be deleted" do
+ assign(autre_procedure, instructeur_assigne: instructeur_not_admin)
+ expect(instructeur_not_admin.can_be_deleted?).to be_falsy
+ end
+ end
+ end
+
private
- def assign(procedure_to_assign)
- create :assign_to, instructeur: instructeur, procedure: procedure_to_assign, groupe_instructeur: procedure_to_assign.defaut_groupe_instructeur
+ def assign(procedure_to_assign, instructeur_assigne: instructeur)
+ create :assign_to, instructeur: instructeur_assigne, procedure: procedure_to_assign, groupe_instructeur: procedure_to_assign.defaut_groupe_instructeur
end
end
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index b127d35c6..19b317f40 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -224,6 +224,24 @@ describe User, type: :model do
context 'when the user has no dossier in instruction' do
it { is_expected.to be true }
end
+
+ context 'when the user is an administrateur' do
+ it 'cannot be deleted' do
+ administrateur = create(:administrateur)
+ user = administrateur.user
+
+ expect(user.can_be_deleted?).to be_falsy
+ end
+ end
+
+ context 'when the user is an instructeur' do
+ it 'cannot be deleted' do
+ instructeur = create(:instructeur)
+ user = instructeur.user
+
+ expect(user.can_be_deleted?).to be_falsy
+ end
+ end
end
describe '#delete_and_keep_track_dossiers' do
@@ -241,12 +259,29 @@ describe User, type: :model do
let!(:dossier_en_construction) { create(:dossier, :en_construction, user: user) }
let!(:dossier_brouillon) { create(:dossier, user: user) }
- it "keep track of dossiers and delete user" do
- user.delete_and_keep_track_dossiers(administration)
+ context 'without a hidden dossier' do
+ it "keep track of dossiers and delete user" do
+ user.delete_and_keep_track_dossiers(administration)
- expect(DeletedDossier.find_by(dossier_id: dossier_en_construction)).to be_present
- expect(DeletedDossier.find_by(dossier_id: dossier_brouillon)).to be_present
- expect(User.find_by(id: user.id)).to be_nil
+ expect(DeletedDossier.find_by(dossier_id: dossier_en_construction)).to be_present
+ expect(DeletedDossier.find_by(dossier_id: dossier_brouillon)).to be_present
+ expect(User.find_by(id: user.id)).to be_nil
+ end
+ end
+
+ context 'with a hidden dossier' do
+ let!(:dossier_cache) do
+ create(:dossier, :en_construction, user: user)
+ end
+
+ it "keep track of dossiers and delete user" do
+ dossier_cache.delete_and_keep_track(administration)
+ user.delete_and_keep_track_dossiers(administration)
+
+ expect(DeletedDossier.find_by(dossier_id: dossier_en_construction)).to be_present
+ expect(DeletedDossier.find_by(dossier_id: dossier_brouillon)).to be_present
+ expect(User.find_by(id: user.id)).to be_nil
+ end
end
end
end