DGNum/demarches-normaliennes
13
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Rename gestionnaire in code to instructeur

Browse source
This commit is contained in:
simon lehericey 2019-08-06 11:02:54 +02:00
parent 6902f84b85
commit 3fde2a6f70
175 changed files with 1162 additions and 1162 deletions
Download patch file Download diff file Expand all files Collapse all files

8
spec/features/help_spec.rb
View file

@ -77,15 +77,15 @@ feature 'Getting help:' do
end
end
context 'as a gestionnaire' do
let(:gestionnaire) { create(:gestionnaire) }
context 'as a instructeur' do
let(:instructeur) { create(:instructeur) }
before do
login_as gestionnaire, scope: :gestionnaire
login_as instructeur, scope: :instructeur
end
scenario 'a Help menu is visible on signed-in pages' do
visit gestionnaire_procedures_path
visit instructeur_procedures_path
within('.new-header') do
expect(page).to have_help_menu
end

84
spec/features/instructeurs/instructeur_spec.rb
View file

@ -1,27 +1,27 @@
require 'spec_helper'
feature 'The gestionnaire part' do
feature 'The instructeur part' do
include ActiveJob::TestHelper
let(:password) { 'démarches-simplifiées-pwd' }
let!(:gestionnaire) { create(:gestionnaire, password: password) }
let!(:instructeur) { create(:instructeur, password: password) }
let!(:procedure) { create(:procedure, :published, gestionnaires: [gestionnaire]) }
let!(:procedure) { create(:procedure, :published, instructeurs: [instructeur]) }
let!(:dossier) { create(:dossier, state: Dossier.states.fetch(:en_construction), procedure: procedure) }
before do
Flipflop::FeatureSet.current.test!.switch!(:enable_email_login_token, true)
end
context 'when the gestionnaire is also a user' do
let!(:user) { create(:user, email: gestionnaire.email, password: password) }
context 'when the instructeur is also a user' do
let!(:user) { create(:user, email: instructeur.email, password: password) }
scenario 'a gestionnaire can fill a dossier' do
scenario 'a instructeur can fill a dossier' do
visit commencer_path(path: procedure.path)
click_on 'J’ai déjà un compte'
expect(page).to have_current_path new_user_session_path
sign_in_with(gestionnaire.email, password, true)
sign_in_with(instructeur.email, password, true)
expect(page).to have_current_path(commencer_path(path: procedure.path))
click_on 'Commencer la démarche'
@ -32,16 +32,16 @@ feature 'The gestionnaire part' do
end
end
scenario 'A gestionnaire can accept a dossier', :js do
log_in(gestionnaire.email, password)
scenario 'A instructeur can accept a dossier', :js do
log_in(instructeur.email, password)
expect(page).to have_current_path(gestionnaire_procedures_path)
expect(page).to have_current_path(instructeur_procedures_path)
click_on procedure.libelle
expect(page).to have_current_path(gestionnaire_procedure_path(procedure))
expect(page).to have_current_path(instructeur_procedure_path(procedure))
click_on dossier.user.email
expect(page).to have_current_path(gestionnaire_dossier_path(procedure, dossier))
expect(page).to have_current_path(instructeur_dossier_path(procedure, dossier))
click_on 'En construction'
accept_confirm do
@ -73,36 +73,36 @@ feature 'The gestionnaire part' do
expect(dossier.motivation).to eq('a good reason')
end
scenario 'A gestionnaire can follow/unfollow a dossier' do
log_in(gestionnaire.email, password)
scenario 'A instructeur can follow/unfollow a dossier' do
log_in(instructeur.email, password)
click_on procedure.libelle
test_statut_bar(a_suivre: 1, tous_les_dossiers: 1)
dossier_present?(dossier.id, 'en construction')
click_on 'Suivre le dossier'
expect(page).to have_current_path(gestionnaire_procedure_path(procedure))
expect(page).to have_current_path(instructeur_procedure_path(procedure))
test_statut_bar(suivi: 1, tous_les_dossiers: 1)
expect(page).to have_text('Aucun dossier')
click_on 'suivi'
expect(page).to have_current_path(gestionnaire_procedure_path(procedure, statut: 'suivis'))
expect(page).to have_current_path(instructeur_procedure_path(procedure, statut: 'suivis'))
dossier_present?(dossier.id, 'en construction')
click_on 'Ne plus suivre'
expect(page).to have_current_path(gestionnaire_procedure_path(procedure, statut: 'suivis'))
expect(page).to have_current_path(instructeur_procedure_path(procedure, statut: 'suivis'))
test_statut_bar(a_suivre: 1, tous_les_dossiers: 1)
expect(page).to have_text('Aucun dossier')
end
scenario 'A gestionnaire can use avis' do
log_in(gestionnaire.email, password)
scenario 'A instructeur can use avis' do
log_in(instructeur.email, password)
click_on procedure.libelle
click_on dossier.user.email
click_on 'Avis externes'
expect(page).to have_current_path(avis_gestionnaire_dossier_path(procedure, dossier))
expect(page).to have_current_path(avis_instructeur_dossier_path(procedure, dossier))
expert_email = 'expert@tps.com'
@ -113,24 +113,24 @@ feature 'The gestionnaire part' do
log_out
avis = dossier.avis.first
test_mail(expert_email, sign_up_gestionnaire_avis_path(avis, expert_email))
test_mail(expert_email, sign_up_instructeur_avis_path(avis, expert_email))
avis_sign_up(avis, expert_email)
expect(page).to have_current_path(gestionnaire_avis_index_path)
expect(page).to have_current_path(instructeur_avis_index_path)
expect(page).to have_text('avis à donner 1')
expect(page).to have_text('avis donnés 0')
click_on dossier.user.email
expect(page).to have_current_path(gestionnaire_avis_path(dossier.avis.first))
expect(page).to have_current_path(instructeur_avis_path(dossier.avis.first))
within(:css, '.tabs') do
click_on 'Avis'
end
expect(page).to have_current_path(instruction_gestionnaire_avis_path(dossier.avis.first))
expect(page).to have_current_path(instruction_instructeur_avis_path(dossier.avis.first))
within(:css, '.give-avis') do
expect(page).to have_text("Demandeur : #{gestionnaire.email}")
expect(page).to have_text("Demandeur : #{instructeur.email}")
expect(page).to have_text('a good introduction')
expect(page).to have_text('Cet avis est confidentiel')
fill_in 'avis_answer', with: 'a great answer'
@ -139,7 +139,7 @@ feature 'The gestionnaire part' do
log_out
log_in(gestionnaire.email, password, check_email: false)
log_in(instructeur.email, password, check_email: false)
click_on procedure.libelle
click_on dossier.user.email
@ -148,38 +148,38 @@ feature 'The gestionnaire part' do
expect(page).to have_text('a great answer')
end
scenario 'A gestionnaire can see the personnes impliquées' do
gestionnaire2 = FactoryBot.create(:gestionnaire, password: password)
scenario 'A instructeur can see the personnes impliquées' do
instructeur2 = FactoryBot.create(:instructeur, password: password)
log_in(gestionnaire.email, password)
log_in(instructeur.email, password)
click_on procedure.libelle
click_on dossier.user.email
click_on 'Avis externes'
expect(page).to have_current_path(avis_gestionnaire_dossier_path(procedure, dossier))
expect(page).to have_current_path(avis_instructeur_dossier_path(procedure, dossier))
expert_email = 'expert@tps.com'
ask_confidential_avis(expert_email, 'a good introduction')
expert_email = gestionnaire2.email
expert_email = instructeur2.email
ask_confidential_avis(expert_email, 'a good introduction')
click_on 'Personnes impliquées'
expect(page).to have_text(expert_email)
expect(page).to have_text(gestionnaire2.email)
expect(page).to have_text(instructeur2.email)
end
scenario 'A gestionnaire can send a dossier to several instructeurs', js: true do
instructeur_2 = FactoryBot.create(:gestionnaire)
instructeur_3 = FactoryBot.create(:gestionnaire)
procedure.gestionnaires << [instructeur_2, instructeur_3]
scenario 'A instructeur can send a dossier to several instructeurs', js: true do
instructeur_2 = FactoryBot.create(:instructeur)
instructeur_3 = FactoryBot.create(:instructeur)
procedure.instructeurs << [instructeur_2, instructeur_3]
send_dossier = double()
expect(GestionnaireMailer).to receive(:send_dossier).and_return(send_dossier).twice
expect(InstructeurMailer).to receive(:send_dossier).and_return(send_dossier).twice
expect(send_dossier).to receive(:deliver_later).twice
log_in(gestionnaire.email, password)
log_in(instructeur.email, password)
click_on procedure.libelle
click_on dossier.user.email
@ -203,7 +203,7 @@ feature 'The gestionnaire part' do
sign_in_with(email, password, check_email)
expect(page).to have_current_path(gestionnaire_procedures_path)
expect(page).to have_current_path(instructeur_procedures_path)
end
def log_out
@ -236,10 +236,10 @@ feature 'The gestionnaire part' do
end
def avis_sign_up(avis, email)
visit sign_up_gestionnaire_avis_path(avis, email)
fill_in 'gestionnaire_password', with: 'démarches-simplifiées-pwd'
visit sign_up_instructeur_avis_path(avis, email)
fill_in 'instructeur_password', with: 'démarches-simplifiées-pwd'
click_on 'Créer un compte'
expect(page).to have_current_path(gestionnaire_avis_index_path)
expect(page).to have_current_path(instructeur_avis_index_path)
end
def dossier_present?(id, statut)

6
spec/features/instructeurs/procedure_filters_spec.rb
View file

@ -2,7 +2,7 @@ require "spec_helper"
feature "procedure filters" do
let(:procedure) { create(:procedure, :published, :with_type_de_champ) }
let(:gestionnaire) { create(:gestionnaire, procedures: [procedure]) }
let(:instructeur) { create(:instructeur, procedures: [procedure]) }
let!(:type_de_champ) { procedure.types_de_champ.first }
let!(:new_unfollow_dossier) { create(:dossier, procedure: procedure, state: Dossier.states.fetch(:en_instruction)) }
let!(:champ) { Champ.find_by(type_de_champ_id: type_de_champ.id, dossier_id: new_unfollow_dossier.id) }
@ -12,8 +12,8 @@ feature "procedure filters" do
before do
champ.update(value: "Mon champ rempli")
champ_2.update(value: "Mon autre champ rempli différemment")
login_as gestionnaire, scope: :gestionnaire
visit gestionnaire_procedure_path(procedure)
login_as instructeur, scope: :instructeur
visit instructeur_procedure_path(procedure)
end
scenario "should display demandeur by default" do