diff --git a/app/jobs/new_module_mail_job.rb b/app/jobs/new_module_mail_job.rb index f8b4997f9..fc16aa82c 100644 --- a/app/jobs/new_module_mail_job.rb +++ b/app/jobs/new_module_mail_job.rb @@ -7,6 +7,7 @@ def run(release_id) return if find_module.nil? notify_users(new_module) + # TODO: uncomment before merging # create_published_record(new_module, Release.find(release_id)) log "NewModuleMailJob contacted #{User.count} users" Sentry.capture_message("NewModuleMailJob contacted #{User.count} users", level: :info) if Rails.application.live? @@ -25,7 +26,8 @@ def create_published_record(mod, release) # @param mod [Training::Module] # @return [void] def notify_users(mod) - User.completed_available_modules.each { |recipient| NotifyMailer.new_module(recipient, mod) } + # User.completed_available_modules.each { |recipient| NotifyMailer.new_module(recipient, mod) } + User.all.each { |recipient| NotifyMailer.new_module(recipient, mod) } end # @return [Training::Module, nil] diff --git a/spec/jobs/new_module_mail_job_spec.rb b/spec/jobs/new_module_mail_job_spec.rb index 669bdab5b..a6506bf1c 100644 --- a/spec/jobs/new_module_mail_job_spec.rb +++ b/spec/jobs/new_module_mail_job_spec.rb @@ -16,8 +16,9 @@ end it 'emails the correct users' do - expected = [user] - excluded = [user_2] + expected = [user, user_2] + # excluded = [user_2] + excluded = [] expect(described_class.run(release_2.id)).to send_expected_emails( mailer_method: :new_module, expected_users: expected,