如何编写复杂轨道索引的集成测试(最佳实践)

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

# 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(...) 

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

最好的方法是什么,避免如此具体地进行测试,复制应用程序代码? 将查询重构为某个中心位置(如模型)并在测试中重复使用它?

 # 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 controller,然后只需检查@articles变量。 即

 # 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:添加了测试实际查询的额外示例