Question

我有这样的一些测试：

it 'should invite user again' do
    admin_user = create(:invited_admin_user)
    expect_any_instance_of(AdminUser).to receive(:invite!).and_return(true)
    patch :reinvite, params: { id: admin_user.to_param }
end

我真的想这样写：

it 'should invite user again' do
    admin_user = create(:invited_admin_user)
    expect(admin_user).to receive(:invite!).and_return(true)
    patch :reinvite, params: { id: admin_user.to_param }
end

但如果我这样做，测试就会失败。知道为什么会这样吗？我使用factory_bot创建了AdminUser实例。

我已尝试将puts语句放入测试和邀请方法以确认ID。

def invite!(_param1 = AdminUser.new, _param2 = {})
    puts 'ID in invite!' + self.id.inspect
    super(_param1, _param2)
end

it 'should invite user again' do
    admin_user = create(:invited_admin_user)
    puts 'adminuser created' + admin_user.id.inspect
    expect(admin_user).to receive(:invite!).and_return(true)
    patch :reinvite, params: { id: admin_user.to_param }
end

结果

adminuser created7768
邀请中的
ID！7768

Answer 1

问题是您的代码重新查找记录并将其实例化为新对象，因此接收消息的对象与测试中的对象不同。

为了解决这个问题：

it 'should invite user again' do
  admin_user = create(:invited_admin_user)
  expect(User).to receive(:find).with(admin_user.id).and_return(admin_user)
  expect(admin_user).to receive(:invite!).and_return(true)
  patch :reinvite, params: { id: admin_user.to_param }
end

这只是拦截User.find调用并返回你的测试对象，而不是它通常初始化的对象。

Answer 2

为什么你想首先在这里使用模拟？

it 'should invite user again' do
  admin_user = create(:invited_admin_user)
  patch :reinvite, params: { id: admin_user.to_param }
  expect(admin_user.reload.invited).to eq(true)
end

如果你想避免对数据库的冗余调用，整个测试应该写成a）没有真正的数据库对象创建（FactoryGirl#build，）b）没有patch调用（直接调用相应的控制器的方法，）和c）模拟在两者之间调用的所有内容。

NB 我个人认为没有任何理由可以对所有内容进行模拟测试：它们与代码本身几乎没有区别。我的意思是，我们可能在测试和代码中犯了一个错误，并检查patch调用相应控制器的方法是愚蠢的：它已经在Rails测试中检查过了。我总是尝试在适用的时候测试真正的事物（比如用户确实被更改了，而不是调用某些方法。）

如何避免使用expect_any_instance_of锤子？

2 个答案: