Skip to content

Commit bb28432

Browse files
MrSerthDome-GER
authored andcommitted
Use deliver_later to send emails
Now, we are only using `deliver_now` for login-based emails, such as password reset or confirmation emails.
1 parent c63b85a commit bb28432

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

app/controllers/groups_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ def request_access
6363
@group.admins.each do |admin|
6464
send_message(admin, :group_request)
6565

66-
AccessRequestMailer.send_access_request(current_user, admin, @group).deliver_now
66+
AccessRequestMailer.send_access_request(current_user, admin, @group).deliver_later
6767
end
6868
@group.add(current_user, role: :applicant)
6969
redirect_to groups_path

spec/controllers/groups_controller_spec.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
require 'rails_helper'
44

55
RSpec.describe GroupsController do
6+
include ActiveJob::TestHelper
67
render_views
78

89
let(:user) { create(:user, preferred_locale: user_locale) }
@@ -172,7 +173,7 @@
172173
end
173174

174175
it 'sends mail in correct language for recipient' do
175-
post_request
176+
perform_enqueued_jobs { post_request }
176177
expect(ActionMailer::Base.deliveries.last.body.parts.first.body).to include(I18n.t('groups.access_request_mailer.message_line2', locale: admin_locale))
177178
end
178179
end

0 commit comments

Comments
 (0)