1

假设我有一个列出文章的页面。控制器中的代码曾经是

# articles#index
@articles = Article.paginate(page: params[:page], per_page: 10, order: :title)

我的测试就像

# spec/requests/article_pages_spec
Article.paginate(page: 1, per_page:10, order: :title).each do |a|
  a.should have_selector('h3', text: a.title)
end

好的。现在我的代码改变了一堆。索引就像

@articles = Article.find(:all, conditions: complicated_status_conditions)
  .sort_by { |a| complicated_weighted_rating_stuff }
  .select { |a| complicated_filters }
  .paginate(...)

或者其他的东西。那么我的请求规范现在应该是什么样子？我不想只是将应用程序代码复制并粘贴到测试中，但与此同时，条件和排序现在相当复杂，所以测试所有预期元素的存在和顺序肯定会失败，除非我模拟索引控制器。

最好的方法是什么，避免专门测试，复制应用程序代码？将查询重构到某个中心位置（例如模型）并在测试中重新使用它？

4

1 回答 1

3 
# articles#index
@articles = Article.paginate(page: params[:page], per_page: 10, order: :title)

我们测试的方式不是Article.paginate(page: params[:page], per_page: 10, order: :title)在规范中再次编写。规范必须测试您的程序代码的结果，而不是复制您的程序代码本身！

长话短说 - 你必须调用articles#index控制器，然后检查@articles变量。IE

# We usually call this as a controller spec
# spec/controllers/articles_controller
# But feel free to put it anywhere you want
describe ArticlesController do
  it "should ..." do
    get :index

    # assigns[:articles] will give the @articles variable contents
    assigns[:articles].each do |a|
      response.should have_selector('h3', text: a.title)
    end
  end
end

这样，您可以直接使用@articles变量本身进行测试，而无需进行第二次查询（这既会消耗不必要的时间，也会导致复制代码）。

如果您想测试实际查询本身，那么由于您的查询很复杂，您应该编写如下规范：

it "should ..." do
  # Create 10 articles in the database
  # out of which only 5 are expected to match the expected output
  article1  = Article.create! ...
  ...
  article10 = Article.create! ...

  get :index

  # Test whether the articles are correctly filtered and ordered
  assigns[:articles].should == [article5, article3, article7, article1, article4]

编辑：脚注编辑 2：添加了测试实际查询的额外示例

于 2013-06-08T17:41:50.773 回答