Avis: move notify_gestionnaire to private method

app/models/avis.rb

@@ -14,10 +14,6 @@ class Avis < ApplicationRecord
     gestionnaire.try(:email) || email
   end
 
-  def notify_gestionnaire
-    AvisMailer.avis_invitation(self).deliver_now
-  end
-
   def self.link_avis_to_gestionnaire(gestionnaire)
     Avis.where(email: gestionnaire.email).update_all(email: nil, gestionnaire_id: gestionnaire.id)
   end
@@ -26,4 +22,10 @@ class Avis < ApplicationRecord
     avis = Avis.find_by(id: avis_id)
     avis.present? && avis.email == email
   end
+
+  private
+
+  def notify_gestionnaire
+    AvisMailer.avis_invitation(self).deliver_now
+  end
 end

spec/models/avis_spec.rb

@@ -86,4 +86,16 @@ RSpec.describe Avis, type: :model do
       end
     end
   end
+
+  describe '#notify_gestionnaire' do
+    context 'when an avis is created' do
+      before do
+        avis_invitation_double = double('avis_invitation', deliver_now: true)
+        allow(AvisMailer).to receive(:avis_invitation).and_return(avis_invitation_double)
+        Avis.create(claimant: claimant, email: 'email@l.com')
+      end
+
+      it { expect(AvisMailer).to have_received(:avis_invitation) }
+    end
+  end
 end