From 41cb62b5c7373fcd264b477e5c24a06bce566ac6 Mon Sep 17 00:00:00 2001 From: simon lehericey Date: Thu, 30 Mar 2023 12:31:48 +0200 Subject: [PATCH] add full system spec --- .../routing_rules_component.html.haml | 2 +- .../routing/rules_full_scenario_spec.rb | 262 ++++++++++++++++++ 2 files changed, 263 insertions(+), 1 deletion(-) create mode 100644 spec/system/routing/rules_full_scenario_spec.rb diff --git a/app/components/procedure/routing_rules_component/routing_rules_component.html.haml b/app/components/procedure/routing_rules_component/routing_rules_component.html.haml index 851d74421..6974a72cb 100644 --- a/app/components/procedure/routing_rules_component/routing_rules_component.html.haml +++ b/app/components/procedure/routing_rules_component/routing_rules_component.html.haml @@ -14,7 +14,7 @@ %th.delete-column .conditionnel.mt-2.width-100 - rows.each.with_index do |(targeted_champ, value, groupe_instructeur), row_index| - = form_tag admin_procedure_routing_rules_path, method: :post, class: 'form width-100' do + = form_tag admin_procedure_routing_rules_path, method: :post, class: "form width-100 gi-#{groupe_instructeur.id}" do %table.condition-table.mt-2.width-100 %tbody %tr{ data: { controller: 'autosave' } } diff --git a/spec/system/routing/rules_full_scenario_spec.rb b/spec/system/routing/rules_full_scenario_spec.rb new file mode 100644 index 000000000..01e3d28e2 --- /dev/null +++ b/spec/system/routing/rules_full_scenario_spec.rb @@ -0,0 +1,262 @@ +describe 'The routing with rules', js: true do + let(:password) { 'a very complicated password' } + + let(:procedure) do + create(:procedure, :with_service, :for_individual, :with_zone).tap do |p| + p.draft_revision.add_type_de_champ( + type_champ: :text, + libelle: 'un premier champ text' + ) + p.draft_revision.add_type_de_champ( + type_champ: :drop_down_list, + libelle: 'Spécialité', + options: { "drop_down_other" => "0", "drop_down_options" => ["", "littéraire", "scientifique"] } + ) + end + end + let(:administrateur) { create(:administrateur, procedures: [procedure]) } + let(:scientifique_user) { create(:user, password: password) } + let(:litteraire_user) { create(:user, password: password) } + + before do + Flipper.enable(:routing_rules, procedure) + procedure.defaut_groupe_instructeur.instructeurs << administrateur.instructeur + end + + scenario 'works' do + login_as administrateur.user, scope: :user + visit admin_procedure_path(procedure.id) + find('#groupe-instructeurs').click + + # add littéraire groupe + fill_in 'Ajouter un nom de groupe', with: 'littéraire' + click_on 'Ajouter le groupe' + expect(page).to have_text('Le groupe d’instructeurs « littéraire » a été créé et le routage a été activé.') + + # add victor to littéraire groupe + fill_in 'Emails', with: 'victor@inst.com' + perform_enqueued_jobs { click_on 'Affecter' } + expect(page).to have_text("L’instructeur victor@inst.com a été affecté") + + victor = User.find_by(email: 'victor@inst.com').instructeur + + # add superwoman to littéraire groupe + fill_in 'Emails', with: 'superwoman@inst.com' + perform_enqueued_jobs { click_on 'Affecter' } + expect(page).to have_text("L’instructeur superwoman@inst.com a été affecté") + + superwoman = User.find_by(email: 'superwoman@inst.com').instructeur + + # add inactive groupe + click_on 'Groupes d’instructeurs' + fill_in 'Ajouter un nom de groupe', with: 'non visible car inactif' + click_on 'Ajouter le groupe' + check "Groupe inactif" + click_on 'Modifier' + + # add scientifique groupe + click_on 'Groupes d’instructeurs' + fill_in 'Ajouter un nom de groupe', with: 'scientifique' + click_on 'Ajouter le groupe' + expect(page).to have_text('Le groupe d’instructeurs « scientifique » a été créé.') + + # add marie to scientifique groupe + fill_in 'Emails', with: 'marie@inst.com' + perform_enqueued_jobs { click_on 'Affecter' } + expect(page).to have_text("L’instructeur marie@inst.com a été affecté") + + marie = User.find_by(email: 'marie@inst.com').instructeur + + # add superwoman to scientifique groupe + fill_in 'Emails', with: 'superwoman@inst.com' + perform_enqueued_jobs { click_on 'Affecter' } + expect(page).to have_text("L’instructeur superwoman@inst.com a été affecté") + + # add routing rules + click_on 'Groupes d’instructeurs' + + h = procedure.groupe_instructeurs.index_by(&:label).transform_values(&:id) + + within(".gi-#{h['scientifique']}") do + within('.target') { select('Spécialité') } + within('.value') { select('scientifique') } + end + + within(".gi-#{h['littéraire']}") do + within('.target') { select('Spécialité') } + within('.value') { select('littéraire') } + end + + not_defauts = procedure.groupe_instructeurs.filter { |gi| ['littéraire', 'scientifique'].include?(gi.label) } + not_defauts.each { |gi| wait_until { gi.reload.routing_rule.present? } } + + # publish + publish_procedure(procedure) + log_out + + # 2 users fill a dossier in each group + user_send_dossier(scientifique_user, 'scientifique') + user_send_dossier(litteraire_user, 'littéraire') + + # the litteraires instructeurs only manage the litteraires dossiers + register_instructeur_and_log_in(victor.email) + click_on procedure.libelle + expect(page).to have_text(litteraire_user.email) + expect(page).not_to have_text(scientifique_user.email) + + # the search only show litteraires dossiers + fill_in 'q', with: scientifique_user.email + find('.fr-search-bar .fr-btn').click + expect(page).to have_text('0 dossier trouvé') + + # weird bug, capabary appends text instead of replaces it + # see https://github.com/redux-form/redux-form/issues/686 + fill_in('q', with: litteraire_user.email, fill_options: { clear: :backspace }) + find('.fr-search-bar .fr-btn').click + expect(page).to have_text('1 dossier trouvé') + + ## and the result is clickable + click_on litteraire_user.email + expect(page).to have_current_path(instructeur_dossier_path(procedure, litteraire_user.dossiers.first)) + + # follow the dossier + click_on 'Suivre le dossier' + + log_out + + # the scientifiques instructeurs only manage the scientifiques dossiers + register_instructeur_and_log_in(marie.email) + click_on procedure.libelle + expect(page).not_to have_text(litteraire_user.email) + expect(page).to have_text(scientifique_user.email) + + # follow the dossier + click_on scientifique_user.email + click_on 'Suivre le dossier' + + log_out + + # litteraire_user change its dossier + visit new_user_session_path + sign_in_with litteraire_user.email, password + + click_on litteraire_user.dossiers.first.id.to_s + click_on 'Modifier mon dossier' + + fill_in litteraire_user.dossiers.first.champs_public.first.libelle, with: 'some value' + wait_for_autosave(false) + + log_out + + # the litteraires instructeurs should have a notification + visit new_user_session_path + sign_in_with victor.user.email, password + + ## on the procedures list + expect(page).to have_current_path(instructeur_procedures_path) + expect(find('.procedure-stats')).to have_css('span.notifications') + + ## on the dossiers list + click_on procedure.libelle + expect(page).to have_current_path(instructeur_procedure_path(procedure)) + expect(find('.tabs')).to have_css('span.notifications') + + ## on the dossier itself + click_on 'suivi' + click_on litteraire_user.email + expect(page).to have_current_path(instructeur_dossier_path(procedure, litteraire_user.dossiers.first)) + expect(page).to have_text('Annotations privées') + expect(find('.tabs')).to have_css('span.notifications') + log_out + + # the scientifiques instructeurs should not have a notification + visit new_user_session_path + sign_in_with marie.user.email, password + + expect(page).to have_current_path(instructeur_procedures_path) + expect(find('.procedure-stats')).not_to have_css('span.notifications') + log_out + + # the instructeurs who belong to scientifique AND litteraire groups manage scientifique and litterraire dossiers + register_instructeur_and_log_in(superwoman.email) + visit instructeur_procedure_path(procedure, params: { statut: 'tous' }) + expect(page).to have_text(litteraire_user.email) + expect(page).to have_text(scientifique_user.email) + + # follow the dossier + click_on scientifique_user.email + click_on 'Suivre le dossier' + + visit instructeur_procedure_path(procedure, params: { statut: 'tous' }) + click_on litteraire_user.email + click_on 'Suivre le dossier' + log_out + + # scientifique_user updates its group + user_update_group(scientifique_user, 'littéraire') + + # the instructeurs who belong to scientifique AND litteraire groups should have a notification + visit new_user_session_path + sign_in_with superwoman.user.email, password + + expect(page).to have_current_path(instructeur_procedures_path) + expect(find('.procedure-stats')).to have_css('span.notifications') + end + + def publish_procedure(procedure) + click_on procedure.libelle + find('#publish-procedure-link').click + fill_in 'lien_site_web', with: 'http://some.website' + click_on 'Publier' + + expect(page).to have_text('Démarche publiée') + end + + def user_send_dossier(user, groupe) + login_as user, scope: :user + visit commencer_path(path: procedure.reload.path) + click_on 'Commencer la démarche' + + choose 'Monsieur' + fill_in 'individual_nom', with: 'Nom' + fill_in 'individual_prenom', with: 'Prenom' + click_button('Continuer') + + # the old system should not be present + expect(page).not_to have_selector("#dossier_groupe_instructeur_id") + + choose(groupe) + wait_for_autosave + + click_on 'Déposer le dossier' + expect(page).to have_text('Merci') + + log_out + end + + def user_update_group(user, new_group) + login_as user, scope: :user + visit dossiers_path + click_on user.dossiers.first.id.to_s + click_on "Modifier mon dossier" + + choose(new_group) + wait_for_autosave(false) + + expect(page).to have_text(new_group) + + log_out + end + + def register_instructeur_and_log_in(email) + confirmation_email = emails_sent_to(email) + .find { |m| m.subject == 'Activez votre compte instructeur' } + token_params = confirmation_email.body.match(/token=[^"]+/) + + visit "users/activate?#{token_params}" + fill_in :user_password, with: password + click_button 'Définir le mot de passe' + + expect(page).to have_text('Mot de passe enregistré') + end +end