instructeurs: replace calls to Instructeur.find_by(email: …)

app/controllers/admin/instructeurs_controller.rb

@@ -13,7 +13,7 @@ class Admin::InstructeursController < AdminController
 
   def create
     email = params[:instructeur][:email].downcase
-    @instructeur = Instructeur.find_by(email: email)
+    @instructeur = Instructeur.by_email(email)
     procedure_id = params[:procedure_id]
 
     if @instructeur.nil?

app/controllers/users/passwords_controller.rb

@@ -15,7 +15,7 @@ class Users::PasswordsController < Devise::PasswordsController
       @devise_mapping = Devise.mappings[:administrateur]
       params[:administrateur] = params[:user]
       # uncomment to check password complexity for Instructeur
-      # elsif Instructeur.find_by(email: email)
+      # elsif Instructeur.by_email(email)
       #   @devise_mapping = Devise.mappings[:instructeur]
       #   params[:instructeur] = params[:user]
     end
@@ -46,7 +46,7 @@ class Users::PasswordsController < Devise::PasswordsController
 
   def try_to_authenticate_instructeur
     if user_signed_in?
-      instructeur = Instructeur.find_by(email: current_user.email)
+      instructeur = Instructeur.by_email(current_user.email)
 
       if instructeur
         sign_in(instructeur.user)

app/controllers/webhook_controller.rb

@@ -4,8 +4,8 @@ class WebhookController < ActionController::Base
   def helpscout
     email = params[:customer][:email].downcase
     user = User.find_by(email: email)
-    instructeur = Instructeur.find_by(email: email)
-    administrateur = Administrateur.find_by(email: email)
+    instructeur = user.instructeur
+    administrateur = user.administrateur
     html = []
 
     if user

app/models/administrateur.rb

@@ -74,7 +74,7 @@ class Administrateur < ApplicationRecord
   end
 
   def instructeur
-    Instructeur.find_by(email: email)
+    user.instructeur
   end
 
   def can_be_deleted?

app/models/avis.rb

@@ -50,7 +50,7 @@ class Avis < ApplicationRecord
   private
 
   def try_to_assign_instructeur
-    instructeur = Instructeur.find_by(email: email)
+    instructeur = Instructeur.by_email(email)
     if instructeur
       self.instructeur = instructeur
       self.email = nil

app/models/instructeur.rb

@@ -24,6 +24,10 @@ class Instructeur < ApplicationRecord
 
   has_one :user
 
+  def self.by_email(email)
+    Instructeur.eager_load(:user).find_by(users: { email: email })
+  end
+
   def follow(dossier)
     begin
       followed_dossiers << dossier

spec/controllers/admin/instructeurs_controller_spec.rb

@@ -112,7 +112,7 @@ describe Admin::InstructeursController, type: :controller do
     end
 
     context 'when an other admin will add the same email' do
-      let(:instructeur) { Instructeur.find_by(email: email) }
+      let(:instructeur) { Instructeur.by_email(email) }
 
       before do
         create :instructeur, email: email, administrateurs: [admin]
@@ -133,7 +133,7 @@ describe Admin::InstructeursController, type: :controller do
 
     context 'when an other admin will add the same email with some uppercase in it' do
       let(:email) { 'Test@Plop.com' }
-      let(:instructeur) { Instructeur.find_by(email: email.downcase) }
+      let(:instructeur) { Instructeur.by_email(email.downcase) }
 
       before do
         create :instructeur, email: email, administrateurs: [admin]

spec/controllers/instructeurs/avis_controller_spec.rb

@@ -284,7 +284,7 @@ describe Instructeurs::AvisController, type: :controller do
       let!(:avis) { create(:avis, email: invited_email, dossier: dossier) }
       let(:avis_id) { avis.id }
       let(:password) { 'démarches-simplifiées-pwd' }
-      let(:created_instructeur) { Instructeur.find_by(email: invited_email) }
+      let(:created_instructeur) { Instructeur.by_email(invited_email) }
       let(:invitations_email) { true }
 
       before do

spec/models/administration_spec.rb

@@ -24,7 +24,7 @@ describe Administration, type: :model do
 
     it 'creates a corresponding instructeur account for the email' do
       subject
-      instructeur = Instructeur.find_by(email: valid_email)
+      instructeur = Instructeur.by_email(valid_email)
       expect(instructeur).to be_present
     end