From d78d20654441aeb400f2ba3115bcdfad0a484369 Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Mon, 21 Oct 2019 09:24:43 +0000 Subject: [PATCH] instructeurs: fix expert creation not to use Instructeur.email It kind of worked until now, because the email field is disabled, and thus never accessed. But better make it clean, by accessing an object (User) where the email field actually exists. --- app/controllers/instructeurs/avis_controller.rb | 2 +- app/views/instructeurs/avis/sign_up.html.haml | 2 +- spec/controllers/instructeurs/avis_controller_spec.rb | 2 +- spec/features/instructeurs/instruction_spec.rb | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/controllers/instructeurs/avis_controller.rb b/app/controllers/instructeurs/avis_controller.rb index 2cff98e59..5e2e0c8a8 100644 --- a/app/controllers/instructeurs/avis_controller.rb +++ b/app/controllers/instructeurs/avis_controller.rb @@ -81,7 +81,7 @@ module Instructeurs def create_instructeur email = params[:email] - password = params['instructeur']['password'] + password = params[:user][:password] # Not perfect because the password will not be changed if the user already exists user = User.create_or_promote_to_instructeur(email, password) diff --git a/app/views/instructeurs/avis/sign_up.html.haml b/app/views/instructeurs/avis/sign_up.html.haml index 29049546e..44e0ab237 100644 --- a/app/views/instructeurs/avis/sign_up.html.haml +++ b/app/views/instructeurs/avis/sign_up.html.haml @@ -4,7 +4,7 @@ %p.description= @dossier.procedure.libelle %p.dossier Dossier nº #{@dossier.id} .column - = form_for(Instructeur.new, url: { controller: "instructeurs/avis", action: :create_instructeur }, method: :post, html: { class: "form" }) do |f| + = form_for(User.new, url: { controller: "instructeurs/avis", action: :create_instructeur }, method: :post, html: { class: "form" }) do |f| %h1 Créez-vous un compte = f.label :email, "Email" diff --git a/spec/controllers/instructeurs/avis_controller_spec.rb b/spec/controllers/instructeurs/avis_controller_spec.rb index 5c0d60966..1c7c9d007 100644 --- a/spec/controllers/instructeurs/avis_controller_spec.rb +++ b/spec/controllers/instructeurs/avis_controller_spec.rb @@ -296,7 +296,7 @@ describe Instructeurs::AvisController, type: :controller do post :create_instructeur, params: { id: avis_id, email: invited_email, - instructeur: { + user: { password: password } } diff --git a/spec/features/instructeurs/instruction_spec.rb b/spec/features/instructeurs/instruction_spec.rb index 52140f952..884b73631 100644 --- a/spec/features/instructeurs/instruction_spec.rb +++ b/spec/features/instructeurs/instruction_spec.rb @@ -181,7 +181,7 @@ feature 'Instructing a dossier:' do def avis_sign_up(avis, email) visit sign_up_instructeur_avis_path(avis, email) - fill_in 'instructeur_password', with: 'démarches-simplifiées-pwd' + fill_in 'user_password', with: 'démarches-simplifiées-pwd' click_on 'Créer un compte' expect(page).to have_current_path(instructeur_avis_index_path) end