From c08e356367758a3f14cac80943d83bc9145d1545 Mon Sep 17 00:00:00 2001 From: simon lehericey Date: Wed, 14 Aug 2019 12:15:13 +0200 Subject: [PATCH] Remove or fix nonsense code --- .../instructeurs/dossiers_controller_spec.rb | 8 +--- .../sessions/sessions_controller_spec.rb | 15 ------- .../users/sessions_controller_spec.rb | 14 ------- .../instructeurs/instructeur_creation_spec.rb | 2 +- spec/models/administrateur_spec.rb | 39 ------------------- spec/models/instructeur_spec.rb | 12 ------ spec/models/user_spec.rb | 14 ------- 7 files changed, 2 insertions(+), 102 deletions(-) diff --git a/spec/controllers/instructeurs/dossiers_controller_spec.rb b/spec/controllers/instructeurs/dossiers_controller_spec.rb index 4ca80d21e..379b77d9e 100644 --- a/spec/controllers/instructeurs/dossiers_controller_spec.rb +++ b/spec/controllers/instructeurs/dossiers_controller_spec.rb @@ -184,13 +184,7 @@ describe Instructeurs::DossiersController, type: :controller do expect(dossier.reload.state).to eq(Dossier.states.fetch(:accepte)) expect(response).to have_http_status(:ok) end - context 'as administrateur' do - let (:current_user) { administrateur } - it 'it is not possible to go back to en_instruction' do - expect(dossier.reload.state).to eq(Dossier.states.fetch(:accepte)) - expect(response).to have_http_status(:ok) - end - end + context 'as superadmin' do let (:current_user) { administration } it 'it is not possible to go back to en_instruction' do diff --git a/spec/controllers/sessions/sessions_controller_spec.rb b/spec/controllers/sessions/sessions_controller_spec.rb index ad1834671..06b4ec51c 100644 --- a/spec/controllers/sessions/sessions_controller_spec.rb +++ b/spec/controllers/sessions/sessions_controller_spec.rb @@ -8,8 +8,6 @@ describe Sessions::SessionsController, type: :controller do end let(:user) { create(:user) } - let(:instructeur) { create(:instructeur) } - let(:administrateur) { create(:administrateur) } describe '#create' do before do @@ -34,17 +32,4 @@ describe Sessions::SessionsController, type: :controller do post :create end end - - describe '#create with administrateur connected' do - before do - @request.env["devise.mapping"] = Devise.mappings[:administrateur] - - allow_any_instance_of(described_class).to receive(:administrateur_signed_in?).and_return(true) - end - - it 'calls sign out for administrateur' do - expect_any_instance_of(described_class).to receive(:sign_out).with(:administrateur) - post :create - end - end end diff --git a/spec/controllers/users/sessions_controller_spec.rb b/spec/controllers/users/sessions_controller_spec.rb index 6a8adde5f..0a42fc26a 100644 --- a/spec/controllers/users/sessions_controller_spec.rb +++ b/spec/controllers/users/sessions_controller_spec.rb @@ -108,20 +108,6 @@ describe Users::SessionsController, type: :controller do expect(response).to redirect_to(root_path) end end - - context "when associated administrateur" do - let(:administrateur) { create(:administrateur, user: user) } - - it 'signs user + instructeur + administrateur out' do - sign_in user - sign_in(administrateur.user) - delete :destroy - expect(@response.redirect?).to be(true) - expect(subject.current_user).to be(nil) - expect(subject.current_instructeur).to be(nil) - expect(subject.current_administrateur).to be(nil) - end - end end describe '#new' do diff --git a/spec/features/instructeurs/instructeur_creation_spec.rb b/spec/features/instructeurs/instructeur_creation_spec.rb index 5ffb3488c..50d284e96 100644 --- a/spec/features/instructeurs/instructeur_creation_spec.rb +++ b/spec/features/instructeurs/instructeur_creation_spec.rb @@ -6,7 +6,7 @@ feature 'As an instructeur', js: true do let(:instructeur_email) { 'new_instructeur@gouv.fr' } before do - login_as administrateur, scope: :administrateur + login_as administrateur.user, scope: :user visit admin_procedure_assigns_path(procedure) fill_in :instructeur_email, with: instructeur_email diff --git a/spec/models/administrateur_spec.rb b/spec/models/administrateur_spec.rb index 6b8f83788..a4e679441 100644 --- a/spec/models/administrateur_spec.rb +++ b/spec/models/administrateur_spec.rb @@ -8,29 +8,6 @@ describe Administrateur, type: :model do it { is_expected.to have_many(:procedures) } end - context 'unified login' do - it 'syncs credentials to associated user' do - administrateur = create(:administrateur) - user = administrateur.instructeur.user - - administrateur.update(email: 'whoami@plop.com', password: 'voilà un super mdp') - - user.reload - expect(user.email).to eq('whoami@plop.com') - expect(user.valid_password?('voilà un super mdp')).to be(true) - end - - it 'syncs credentials to associated administrateur' do - administrateur = create(:administrateur) - instructeur = administrateur.instructeur - - administrateur.update(email: 'whoami@plop.com', password: 'et encore un autre mdp') - - instructeur.reload - expect(instructeur.email).to eq('whoami@plop.com') - end - end - describe "#renew_api_token" do let!(:administrateur) { create(:administrateur) } let!(:token) { administrateur.renew_api_token } @@ -44,22 +21,6 @@ describe Administrateur, type: :model do end end - describe '#find_inactive_by_token' do - let(:administrateur) { create(:administration).invite_admin('paul@tps.fr') } - let(:reset_password_token) { administrateur.invite!(administration.id) } - - it { expect(Administrateur.find_inactive_by_token(reset_password_token)).not_to be_nil } - end - - describe '#reset_password' do - let(:administrateur) { create(:administration).invite_admin('paul@tps.fr') } - let(:reset_password_token) { administrateur.invite!(administration.id) } - - it { expect(Administrateur.reset_password(reset_password_token, "j'aime manger des radis").errors).to be_empty } - it { expect(Administrateur.reset_password('123', "j'aime manger des radis").errors).not_to be_empty } - it { expect(Administrateur.reset_password(reset_password_token, '').errors).not_to be_empty } - end - describe '#feature_enabled?' do let(:administrateur) { create(:administrateur) } diff --git a/spec/models/instructeur_spec.rb b/spec/models/instructeur_spec.rb index 2a98e6336..5cc374a86 100644 --- a/spec/models/instructeur_spec.rb +++ b/spec/models/instructeur_spec.rb @@ -137,18 +137,6 @@ describe Instructeur, type: :model do end end - context 'unified login' do - it 'syncs credentials to associated administrateur' do - admin = create(:administrateur) - user = admin.instructeur.user - - user.update(password: 'démarches-simplifiées-pwd') - - admin.reload - expect(admin.valid_password?('démarches-simplifiées-pwd')).to be(true) - end - end - describe 'last_week_overview' do let!(:instructeur2) { create(:instructeur) } subject { instructeur2.last_week_overview } diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 08346f1f2..1b817db8a 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -100,18 +100,4 @@ describe User, type: :model do it { is_expected.to be_falsey } end end - - context 'unified login' do - it 'syncs credentials to associated administrateur' do - admin = create(:administrateur) - user = admin.instructeur.user - - user.update(email: 'whoami@plop.com', password: 'démarches-simplifiées2') - user.confirm - - admin.reload - expect(admin.email).to eq('whoami@plop.com') - expect(admin.valid_password?('démarches-simplifiées2')).to be(true) - end - end end