rspec测试协会

Question

我想在我的 rspec 控制器测试中测试员工是否与公司关联。

我想在员工控制器的 create 操作中得到这样的结果：

staff.companies << current_company

其中 current_company 是从会话变量中收集的。

我该如何为此编写测试？

我已经有了这些模型

class Company < ActiveRecord::Base
  has_many :employees
  has_many :staff, :through => :employees
end

class Employee < ActiveRecord::Base
  belongs_to :company
  belongs_to :staff
end

class Staff < ActiveRecord::Base
  has_many :employees
  has_many :companies, :through => :employees
end

以下测试是我尝试规范关联，当我输入关联代码时它失败：

    it "should belong to the current_company" do
      staff.should_receive(:companies)
      post :create
    end

如果我输入 'staff.companies <<我的控制器中的 current_company' 代码运行该测试时出现此错误：

 Failure/Error: post :create
 NoMethodError:
   You have a nil object when you didn't expect it!
   You might have expected an instance of Array.
   The error occurred while evaluating nil.<<

员工控制器创建方法：

  def create
    @staff = Staff.new(params[:staff])

    if @staff.save
      @staff.companies << current_company
      redirect_to staff_index_path, :notice => "Staff created successfully!"
    else
      @company = @staff.firm || current_company
      flash[:alert] = "Staff failed to create"
      render "new"
    end
  end

Answer 1

我会使用不同的方法，因为测试模型应该接收特定消息会使您的测试与实现过于紧密地结合在一起。您真的关心公司是否接收 #<< 或其他方法吗？

实际上，您要测试的是用户在页面上发帖时是否记录了他们的公司。 如何录制并不重要。所以我会做这样的事情：

it "should add the company to the user's list of companies" do
  lambda do 
    post :create
  end.should change(staff.companies, :count).from(0).to(1)
  staff.companies.map(&:name).should include("Acme, Inc.")
end

这是测试行为而不是实现。优点是，当有人将 << 更改为等效的 push 时，您的测试不会失败。它还具有更清楚您的意图并因此更好地记录代码的优点。

Answer 2

如果您在控制器规范中，我将使用 stub_chain

staff.stub_chain(:company, :<<).once.and_return(true)

它将模拟公司调用和 <<调用并期望它被调用一次。

（至少，.once 应该与stub_chain一起工作......）

Answer 3

您可以使用以下命令进行测试：

staff.should have(1).company

或者如果该员工已经拥有其他公司，则获取计数并测试 has(count+1).companies。

Answer 4

代码的问题在于，一旦您删除了一个方法，它就不再存在于模型中。

您已经删除了“公司”方法（当您对其设置期望时），现在它不再调用模型上实际的、真实的公司关联，而是您创建的存根...它返回 nil （因为您没有为其设置返回值）。

然后，当您尝试使用 << 将公司放入这个新的 null 方法时它说它不能这样做。

为了解决这个问题，你可以做你所做的事情，即设置一个返回值：

staff.should_receive(:companies).and_return([])

然后将确保：

@staff.companies << current_company

不会因可怕的 nil 错误而失败（因为公司需要进入实际的、真实的数组）。

但真正要做的最好的事情是正如前面的人所建议的那样，并测试您真正需要测试的内容 - 即保存公司员工将导致新公司保存到数据库中。