- <%= 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" %>
+ <%= 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é" %>
diff --git a/config/routes.rb b/config/routes.rb
index e69a44a60..09f0d8fb8 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -32,7 +32,6 @@ 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
deleted file mode 100644
index 3a0164d4b..000000000
--- a/spec/controllers/manager/instructeurs_controller_spec.rb
+++ /dev/null
@@ -1,17 +0,0 @@
-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 3e6864466..9f0c30923 100644
--- a/spec/models/administrateur_spec.rb
+++ b/spec/models/administrateur_spec.rb
@@ -46,21 +46,6 @@ 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 78737bf89..37917bf18 100644
--- a/spec/models/instructeur_spec.rb
+++ b/spec/models/instructeur_spec.rb
@@ -424,40 +424,9 @@ 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, instructeur_assigne: instructeur)
- create :assign_to, instructeur: instructeur_assigne, procedure: procedure_to_assign, groupe_instructeur: procedure_to_assign.defaut_groupe_instructeur
+ def assign(procedure_to_assign)
+ create :assign_to, instructeur: instructeur, 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 19b317f40..b127d35c6 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -224,24 +224,6 @@ 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
@@ -259,29 +241,12 @@ describe User, type: :model do
let!(:dossier_en_construction) { create(:dossier, :en_construction, user: user) }
let!(:dossier_brouillon) { create(:dossier, user: user) }
- context 'without a hidden dossier' do
- it "keep track of dossiers and delete user" do
- user.delete_and_keep_track_dossiers(administration)
+ 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
- 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
+ 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