rspec 测试 has_many :through 和 after_save

我有一个（我认为）相对简单的has_many :through与连接表的关系：

class User < ActiveRecord::Base
  has_many :user_following_thing_relationships
  has_many :things, :through => :user_following_thing_relationships
end

class Thing < ActiveRecord::Base
  has_many :user_following_thing_relationships
  has_many :followers, :through => :user_following_thing_relationships, :source => :user
end

class UserFollowingThingRelationship < ActiveRecord::Base
  belongs_to :thing
  belongs_to :user
end

这些 rspec 测试（我知道这些不一定是好的测试，这些只是为了说明正在发生的事情）：

describe Thing do     
  before(:each) do
    @user = User.create!(:name => "Fred")
    @thing = Thing.create!(:name => "Foo")    
    @user.things << @thing
  end

  it "should have created a relationship" do
    UserFollowingThingRelationship.first.user.should == @user
    UserFollowingThingRelationship.first.thing.should == @thing
  end

  it "should have followers" do
    @thing.followers.should == [@user]
  end     
end

这工作正常，直到我添加一个after_save to the Thing引用其的模型followers。也就是说，如果我这样做

class Thing < ActiveRecord::Base
  after_save :do_stuff
  has_many :user_following_thing_relationships
  has_many :followers, :through => :user_following_thing_relationships, :source => :user

  def do_stuff
    followers.each { |f| puts "I'm followed by #{f.name}" }
  end
end

然后第二个测试失败 - 即关系仍然添加到连接表中，但是@thing.followers返回一个空数组。此外，回调的那部分永远不会被调用（就像followers模型内为空）。如果我添加一个puts "HI"在之前的回调中followers.each行，“HI”显示在标准输出上，所以我知道正在调用回调。如果我注释掉followers.each行，然后测试再次通过。

如果我通过控制台执行此操作，则效果很好。即，我可以做

>> t = Thing.create!(:name => "Foo")
>> t.followers # []
>> u = User.create!(:name => "Bar")
>> u.things << t
>> t.followers  # [u]
>> t.save    # just to be super duper sure that the callback is triggered
>> t.followers  # still [u]

为什么 rspec 会失败？我做错了什么可怕的事情吗？

Update

如果我手动定义一切正常Thing#followers as

def followers
  user_following_thing_relationships.all.map{ |r| r.user }
end

这让我相信也许我正在定义我的has_many :through with :source错误地？

Update

我创建了一个最小的示例项目并将其放在 github 上：https://github.com/dantswain/RspecHasMany https://github.com/dantswain/RspecHasMany

另一个更新

非常感谢@PeterNixey 和@kikuchiyo 在下面提出的建议。最终的答案是两个答案的组合，我希望我可以在它们之间分配功劳。我已经用我认为最干净的解决方案更新了 github 项目并推送了更改：https://github.com/dantswain/RspecHasMany https://github.com/dantswain/RspecHasMany

如果有人能给我一个关于这里发生的事情的真正可靠的解释，我仍然会喜欢它。对我来说最麻烦的是，为什么在最初的问题陈述中，如果我注释掉对的引用，所有内容（除了回调本身的操作）都会起作用followers.

我过去也遇到过类似的问题，这些问题已通过重新加载关联（而不是父对象）来解决。

重新加载后还能用吗thing.followers在 RSpec 中？

it "should have followers" do
  @thing.followers.reload
  @thing.followers.should == [@user]
end 

EDIT

如果（正如您所提到的）您遇到回调未触发的问题，那么您可以在对象本身中进行重新加载：

class Thing < ActiveRecord::Base
  after_save { followers.reload}
  after_save :do_stuff
  ...
end

or

class Thing < ActiveRecord::Base
  ...
  def do_stuff
    followers.reload
    ...
  end
end

我不知道为什么 RSpec 存在不重新加载关联的问题，但我自己也遇到过同样类型的问题

Edit 2

尽管 @dantswain 证实followers.reload帮助缓解了一些问题，但仍然没有解决所有问题。

为此，该解决方案需要 @kikuchiyo 的修复，需要调用save完成回调后Thing:

describe Thing do
  before :each do
    ...
    @user.things << @thing
    @thing.run_callbacks(:save)
  end 
  ...
end

最终建议

我相信这是因为使用了<< on a has_many_through手术。我不认为<<实际上应该触发你的after_save事件：

您当前的代码是这样的：

describe Thing do
  before(:each) do
    @user = User.create!(:name => "Fred")
    @thing = Thing.create!(:name => "Foo")    
    @user.things << @thing
  end
end

class Thing < ActiveRecord::Base
  after_save :do_stuff
  ...

  def do_stuff
   followers.each { |f| puts "I'm followed by #{f.name}" }
  end
end

问题是do_stuff没有接到电话。我认为这是正确的行为。

让我们看一下 RSpec：

describe Thing do
  before(:each) do
    @user = User.create!(:name => "Fred")
    # user is created and saved

    @thing = Thing.create!(:name => "Foo")    
    # thing is created and saved

    @user.things << @thing
    # user_thing_relationship is created and saved
    # no call is made to @user.save since nothing is updated on the user
  end
end

问题是第三步实际上并不需要thing要重新保存的对象 - 它只是在连接表中创建一个条目。

如果您想确保 @user 确实调用 save 您可能会得到您想要的效果，如下所示：

describe Thing do
  before(:each) do
    @thing = Thing.create!(:name => "Foo")    
    # thing is created and saved

    @user = User.create!(:name => "Fred")
    # user is created BUT NOT SAVED

    @user.things << @thing
    # user_thing_relationship is created and saved
    # @user.save is also called as part of the addition
  end
end

您可能还会发现after_save事实上，回调位于错误的对象上，您更希望将其放在关系对象上。最后，如果回调确实属于用户并且您确实需要在创建关系后触发它，您可以使用touch创建新关系时更新用户。