mirror of
https://github.com/internetee/registry.git
synced 2025-07-27 13:06:18 +02:00
Merge pull request #2718 from internetee/daily-fd-domains-subscription
Daily email notification to admins about new FD statuses
This commit is contained in:
commit
e38c46757b
4 changed files with 155 additions and 0 deletions
41
app/jobs/force_delete_daily_admin_notifier_job.rb
Normal file
41
app/jobs/force_delete_daily_admin_notifier_job.rb
Normal file
|
@ -0,0 +1,41 @@
|
|||
class ForceDeleteDailyAdminNotifierJob < ApplicationJob
|
||||
queue_as :default
|
||||
|
||||
def perform
|
||||
domains = Domain.where("'#{DomainStatus::FORCE_DELETE}' = ANY (statuses)")
|
||||
.where("force_delete_start = ?", Time.zone.now)
|
||||
|
||||
return if domains.empty?
|
||||
|
||||
notify_admins(domains)
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def notify_admins(domains)
|
||||
summary = generate_summary(domains)
|
||||
AdminMailer.force_delete_daily_summary(summary).deliver_now
|
||||
end
|
||||
|
||||
def generate_summary(domains)
|
||||
domains.map do |domain|
|
||||
{
|
||||
name: domain.name,
|
||||
reason: determine_reason(domain),
|
||||
force_delete_type: domain.force_delete_type,
|
||||
force_delete_start: domain.force_delete_start,
|
||||
force_delete_date: domain.force_delete_date
|
||||
}
|
||||
end
|
||||
end
|
||||
|
||||
def determine_reason(domain)
|
||||
if domain.template_name.present?
|
||||
domain.template_name
|
||||
elsif domain.status_notes[DomainStatus::FORCE_DELETE].present?
|
||||
"Manual force delete: #{domain.status_notes[DomainStatus::FORCE_DELETE]}"
|
||||
else
|
||||
'Unknown reason'
|
||||
end
|
||||
end
|
||||
end
|
9
app/mailers/admin_mailer.rb
Normal file
9
app/mailers/admin_mailer.rb
Normal file
|
@ -0,0 +1,9 @@
|
|||
class AdminMailer < ApplicationMailer
|
||||
def force_delete_daily_summary(domains_summary)
|
||||
@domains = domains_summary
|
||||
mail(
|
||||
to: ENV['admin_notification_email'] || 'admin@registry.test',
|
||||
subject: "Force Delete Daily Summary - #{Date.current}"
|
||||
)
|
||||
end
|
||||
end
|
24
app/views/admin_mailer/force_delete_daily_summary.html.erb
Normal file
24
app/views/admin_mailer/force_delete_daily_summary.html.erb
Normal file
|
@ -0,0 +1,24 @@
|
|||
<h1>Force Delete Daily Summary - <%= Date.current %></h1>
|
||||
|
||||
<table>
|
||||
<thead>
|
||||
<tr>
|
||||
<th>Domain</th>
|
||||
<th>Reason</th>
|
||||
<th>Type</th>
|
||||
<th>Start Date</th>
|
||||
<th>Delete Date</th>
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody>
|
||||
<% @domains.each do |domain| %>
|
||||
<tr>
|
||||
<td><%= domain[:name] %></td>
|
||||
<td><%= domain[:reason] %></td>
|
||||
<td><%= domain[:force_delete_type] %></td>
|
||||
<td><%= domain[:force_delete_start]&.to_date %></td>
|
||||
<td><%= domain[:force_delete_date]&.to_date %></td>
|
||||
</tr>
|
||||
<% end %>
|
||||
</tbody>
|
||||
</table>
|
81
test/jobs/force_delete_daily_admin_notifier_job_test.rb
Normal file
81
test/jobs/force_delete_daily_admin_notifier_job_test.rb
Normal file
|
@ -0,0 +1,81 @@
|
|||
require 'test_helper'
|
||||
|
||||
class ForceDeleteDailyAdminNotifierJobTest < ActiveSupport::TestCase
|
||||
include ActionMailer::TestHelper
|
||||
|
||||
setup do
|
||||
@domain = domains(:shop)
|
||||
travel_to Time.zone.parse('2010-07-05')
|
||||
ActionMailer::Base.deliveries.clear
|
||||
end
|
||||
|
||||
def test_sends_notification_for_domains_with_force_delete_today
|
||||
@domain.schedule_force_delete(type: :soft)
|
||||
@domain.update!(force_delete_start: Time.zone.now.to_date)
|
||||
@domain.reload
|
||||
|
||||
assert_emails 1 do
|
||||
ForceDeleteDailyAdminNotifierJob.perform_now
|
||||
end
|
||||
|
||||
email = ActionMailer::Base.deliveries.last
|
||||
assert_includes email.body.to_s, @domain.name
|
||||
assert_includes email.body.to_s, @domain.force_delete_type
|
||||
end
|
||||
|
||||
def test_does_not_send_notification_when_no_force_delete_domains_today
|
||||
travel_to Time.zone.parse('2010-07-06')
|
||||
@domain.schedule_force_delete(type: :soft)
|
||||
@domain.reload
|
||||
|
||||
assert_no_emails do
|
||||
ForceDeleteDailyAdminNotifierJob.perform_now
|
||||
end
|
||||
end
|
||||
|
||||
def test_includes_multiple_domains_in_notification
|
||||
@domain.schedule_force_delete(type: :soft)
|
||||
@domain.update!(force_delete_start: Time.zone.now.to_date)
|
||||
|
||||
domain2 = domains(:airport)
|
||||
domain2.schedule_force_delete(type: :fast_track)
|
||||
domain2.update!(force_delete_start: Time.zone.now.to_date)
|
||||
|
||||
assert_emails 1 do
|
||||
ForceDeleteDailyAdminNotifierJob.perform_now
|
||||
end
|
||||
|
||||
email = ActionMailer::Base.deliveries.last
|
||||
assert_includes email.body.to_s, @domain.name
|
||||
assert_includes email.body.to_s, domain2.name
|
||||
end
|
||||
|
||||
def test_includes_correct_reason_for_invalid_email_template
|
||||
@domain.update!(template_name: 'invalid_email')
|
||||
@domain.schedule_force_delete(type: :soft)
|
||||
@domain.update!(force_delete_start: Time.zone.now.to_date)
|
||||
@domain.reload
|
||||
|
||||
assert_emails 1 do
|
||||
ForceDeleteDailyAdminNotifierJob.perform_now
|
||||
end
|
||||
|
||||
email = ActionMailer::Base.deliveries.last
|
||||
assert_includes email.body.to_s, 'invalid_email'
|
||||
end
|
||||
|
||||
def test_includes_correct_reason_for_manual_force_delete
|
||||
manual_reason = "Manual deletion requested"
|
||||
@domain.status_notes = { DomainStatus::FORCE_DELETE => manual_reason }
|
||||
@domain.schedule_force_delete(type: :fast_track)
|
||||
@domain.update!(force_delete_start: Time.zone.now.to_date)
|
||||
@domain.reload
|
||||
|
||||
assert_emails 1 do
|
||||
ForceDeleteDailyAdminNotifierJob.perform_now
|
||||
end
|
||||
|
||||
email = ActionMailer::Base.deliveries.last
|
||||
assert_includes email.body.to_s, "Manual force delete: #{manual_reason}"
|
||||
end
|
||||
end
|
Loading…
Add table
Add a link
Reference in a new issue