zydronium/internetee-registry
1
0
Fork 0
mirror of https://github.com/internetee/registry.git synced 2025-07-01 08:43:37 +02:00
Code Issues Projects Releases Packages Wiki Activity

Log domain name in domain_expire_mailer

Browse source 
#186
This commit is contained in:
Artur Beljajev 2016-11-25 11:46:15 +02:00
parent e9d959d373
commit 9ad84d78aa
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/mailers/domain_expire_mailer.rb
View file

@ -6,7 +6,7 @@ class DomainExpireMailer < ApplicationMailer
recipient = filter_invalid_emails(emails: domain.primary_contact_emails, domain: domain) recipient = filter_invalid_emails(emails: domain.primary_contact_emails, domain: domain)
subject = default_i18n_subject(domain_name: domain.name) subject = default_i18n_subject(domain_name: domain.name)
logger.info("Send DomainExpireMailer#expired email for domain ##{domain.id}" \ logger.info("Send DomainExpireMailer#expired email for domain #{domain.name} (##{domain.id})" \
" to #{domain.primary_contact_emails.join(', ')}") " to #{domain.primary_contact_emails.join(', ')}")
mail(to: recipient, subject: subject) mail(to: recipient, subject: subject)
@ -28,7 +28,7 @@ class DomainExpireMailer < ApplicationMailer
valid = EmailValidator.new(email).valid? valid = EmailValidator.new(email).valid?
unless valid unless valid
logger.info("Unable to send DomainExpireMailer#expired email for domain ##{domain.id}" \ logger.info("Unable to send DomainExpireMailer#expired email for domain #{domain.name} (##{domain.id})" \
" to invalid recipient #{email}") " to invalid recipient #{email}")
end end

4
spec/mailers/domain_expire_mailer_spec.rb
View file

@ -32,7 +32,7 @@ RSpec.describe DomainExpireMailer do
end end
it 'creates log record' do it 'creates log record' do
log_message = 'Send DomainExpireMailer#expired email for domain #1 to recipient@test.com,' \ log_message = 'Send DomainExpireMailer#expired email for domain test.com (#1) to recipient@test.com,' \
' recipient@test.com' ' recipient@test.com'
expect(described_class.logger).to receive(:info).with(log_message) expect(described_class.logger).to receive(:info).with(log_message)
message.deliver_now message.deliver_now
@ -55,7 +55,7 @@ RSpec.describe DomainExpireMailer do
end end
it 'creates log record' do it 'creates log record' do
log_message = 'Unable to send DomainExpireMailer#expired email for domain #1 to' \ log_message = 'Unable to send DomainExpireMailer#expired email for domain test.com (#1) to' \
' invalid recipient invalid_email' ' invalid recipient invalid_email'
expect(described_class.logger).to receive(:info).with(log_message) expect(described_class.logger).to receive(:info).with(log_message)
message.deliver_now message.deliver_now