ActionMailer方法返回全部发送邮件数组的测试实现问询

嘿，我明白你的困扰——你想在Mailer里批量给用户发邮件，还想拿到所有发送的邮件实例来做测试，但现在只返回最后一封。问题出在Action Mailer的mail方法的工作机制上：每次调用mail，它本质上就像控制器里的render，会终止当前Mailer的处理流程，返回这封邮件的响应。所以你循环里前面的mail调用都会被覆盖，最后只留下最后一封。

下面给你两种可行的解决方案，优先推荐第一种，因为它更符合Rails的设计规范：

方案一：外部处理批量逻辑，Mailer专注单封邮件 #

这是Rails官方推荐的做法——把“筛选用户”“循环发送”的逻辑放在Mailer外面，让Mailer只负责生成单封邮件。这样代码更清晰，测试也更容易。

第一步：重构你的Mailer方法 #

让它只处理单个用户：

class UserReminderMailer < ApplicationMailer
  def no_products_reminder(user)
    @user = user # 如果邮件模板需要用到用户数据的话
    mail to: user.email, subject: "You Haven't Submitted Any Designs Yet"
  end
end

第二步：在外部实现批量发送 #

比如可以写一个任务、service类，或者控制器方法：

def send_no_products_reminders
  # 先筛选符合条件的用户
  eligible_users = User.created_days_ago(2).select { |user| user.buyer_products.empty? }
  # 逐个调用Mailer发送
  eligible_users.each do |user|
    UserReminderMailer.no_products_reminder(user).deliver_now
    # 如果是异步发送，用deliver_later即可
  end
end

第三步：测试验证 #

测试的时候，直接检查Rails提供的ActionMailer::Base.deliveries数组就行，它会记录所有发送的邮件：

test "sends reminders to all eligible users" do
  # 创建测试用户：2个符合条件，1个不符合
  user1 = create(:user, created_at: 2.days.ago, buyer_products: [])
  user2 = create(:user, created_at: 2.days.ago, buyer_products: [])
  non_eligible = create(:user, created_at: 2.days.ago, buyer_products: create_list(:product, 1))

  # 调用发送方法
  send_no_products_reminders

  # 验证邮件数量和收件人
  assert_equal 2, ActionMailer::Base.deliveries.size
  assert_includes ActionMailer::Base.deliveries.map(&:to), [user1.email]
  assert_includes ActionMailer::Base.deliveries.map(&:to), [user2.email]
  refute_includes ActionMailer::Base.deliveries.map(&:to), [non_eligible.email]
end

方案二：在Mailer内部生成多封邮件（不推荐） #

如果你一定要在Mailer内部完成所有逻辑，也可以手动构建邮件对象，但这种方式不符合Mailer的设计初衷，容易出问题（比如实例变量污染）。

实现方式如下：

def no_products_2
  eligible_users = User.created_days_ago(2).select { |user| user.buyer_products.empty? }
  # 用map生成所有邮件对象，手动触发发送
  eligible_users.map do |user|
    mail(
      to: user.email,
      subject: "You Haven't Submitted Any Designs Yet"
    ).tap(&:deliver_now)
  end
end

这里用map替代tap，因为tap返回的是原用户数组，而map会返回所有构建好的邮件对象。但要注意，同一个Mailer实例会被复用，如果你在邮件里用了实例变量，可能会出现数据混乱的情况，所以还是优先选方案一。

总结 #

把批量逻辑抽离到Mailer外部是更优的选择，既符合Rails的设计模式，也让测试变得简单直接——通过ActionMailer::Base.deliveries就能轻松验证所有邮件是否正确发送。

内容的提问来源于stack exchange，提问作者Josh Kestenberg