Shoulda Matcher条件校验报错求助：新增用户跳过邮箱校验测试失败

首先，咱们先理清楚核心问题：你的校验逻辑是「新增用户时跳过邮箱必填校验，更新时必须校验」，但你的测试用例是直接校验邮箱必填，这就和实际的条件校验逻辑冲突了，所以测试会失败。

第一步：确认模型校验代码的正确性 #

先检查你模型里的校验规则是不是写反了。要实现「new_record?为true（新增）时跳过校验」，正确的写法应该是：

# 写法1：用unless指定跳过条件
validates :email, presence: true, unless: :new_record?

# 写法2：用persisted?更直观（persisted?表示记录已保存，即更新场景）
validates :email, presence: true, if: -> { persisted? }

如果之前你写的是if: :new_record?，那刚好搞反了——会变成新增时必须校验邮箱，更新时跳过，这肯定不符合你的需求。

第二步：调整测试用例匹配条件校验逻辑 #

原来的测试用例it { expect(subject).to validate_presence_of(:email) }会失败，因为subject默认是一个新创建的实例（new_record?为true），这时候你的校验是跳过的，shoulda-matchers检测到邮箱为空也能通过，自然会报错。

你需要分场景测试，或者用shoulda-matchers的上下文参数来指定测试场景：

方式1：分场景编写测试（更清晰）

# 测试新增时邮箱可以为空
it "allows blank email when creating a new user" do
  user = User.new(email: nil)
  expect(user).to be_valid
end

# 测试更新时邮箱不能为空
it "requires email when updating an existing user" do
  user = User.create!(email: "test@example.com")
  user.email = nil
  expect(user).not_to be_valid
  expect(user.errors[:email]).to include("can't be blank")
end

方式2：用shoulda-matchers的on: :update参数

如果你习惯用shoulda-matchers的简洁语法，可以指定只在update场景下校验：

it { is_expected.to validate_presence_of(:email).on(:update) }

这样shoulda-matchers会自动模拟记录已保存的状态（即更新场景）来测试校验规则，就不会和新增时的跳过逻辑冲突了。

最后再确认生产环境逻辑 #

你说生产环境运行正常，说明实际的校验逻辑是符合预期的（新增时邮箱为空也能保存），只是测试用例没有匹配这个条件逻辑而已，调整测试用例后就能解决报错问题啦。

内容的提问来源于stack exchange，提问作者Ian Tran