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

Merge pull request #10485 from colinux/fix-groupe-inst-mailer

Browse source 
Tech: fix l'envoi d'emails d'ajout à un groupe instructeurs
This commit is contained in:
Colin Darie 2024-06-04 13:01:46 +00:00 committed by GitHub
commit a15604b057
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
app/lib/balancer_delivery_method.rb
View file

@ -45,6 +45,7 @@ class BalancerDeliveryMethod
def prevent_delivery?(mail)
return false if mail[BYPASS_UNVERIFIED_MAIL_PROTECTION].present?
return false if mail.to.blank? # bcc list
user = User.find_by(email: mail.to.first)
return user.unverified_email? if user.present?

11
spec/lib/balancer_delivery_method_spec.rb
View file

@ -2,8 +2,8 @@ RSpec.describe BalancerDeliveryMethod do
class ExampleMailer < ApplicationMailer
include BalancedDeliveryConcern
def greet(name, bypass_unverified_mail_protection: true)
mail(to: name, from: "smtp_from", body: "Hello #{name}")
def greet(name, bypass_unverified_mail_protection: true, **mail_args)
mail(to: name, from: "smtp_from", body: "Hello #{name}", **mail_args)
bypass_unverified_mail_protection! if bypass_unverified_mail_protection
end
@ -202,6 +202,13 @@ RSpec.describe BalancerDeliveryMethod do
it { expect(mail).to have_been_delivered_using(MockSmtp) }
end
end
context 'when there are only bcc recipients' do
let(:bypass_unverified_mail_protection) { false }
let(:mail) { ExampleMailer.greet(nil, bypass_unverified_mail_protection: false, bcc: ["'u@a.com'"]) }
it { expect(mail).to have_been_delivered_using(MockSmtp) }
end
end
# Helpers