使用shoulda和factory_girl测试REST - destroy

Question

我正在使用shoulda和factory_girl开发REST测试。 下面的代码

 context "on :delete to :destroy" do
    setup do
      @controller = NewsArticlesController.new
      @request = ActionController::TestRequest.new
      @response = ActionController::TestResponse.new

      @news_article =  Factory.create(:news_article)

    end

    should "destroy new NewsArticle" do
      assert_difference('NewsArticle.count', -1) do
        delete :destroy, :id => @news_article.id
      end
    end

    should_redirect_to news_articles_path
  end

我看到

  1) Error:
test: on :delete to :destroy should redirect to index. (NewsArticlesControllerTest):
ArgumentError: block not supplied
    c:/develop/ruby/lib/ruby/gems/1.8/gems/thoughtbot-shoulda-2.10.2/lib/shoulda/action_controller/macros.rb:201:in `instance_eval'
    c:/develop/ruby/lib/ruby/gems/1.8/gems/thoughtbot-shoulda-2.10.2/lib/shoulda/action_controller/macros.rb:201:in `__bind_1248853182_16800
0'
    c:/develop/ruby/lib/ruby/gems/1.8/gems/thoughtbot-shoulda-2.10.2/lib/shoulda/context.rb:351:in `call'
    c:/develop/ruby/lib/ruby/gems/1.8/gems/thoughtbot-shoulda-2.10.2/lib/shoulda/context.rb:351:in `test: on :delete to :destroy should redirect to index. '

，你能告诉我 - 出了什么问题以及我如何修改测试以使它们正常工作吗？

UPD：路线看起来不错

news_articles GET    /news(.:format)                    {:controller=>"news_articles", :action=>"index"}

Answer 1

问题出在 should_redirect_to 上，它现在使用 block 来评估重定向代码。 遗憾的是，thoughtbot wiki 和 github 上的自述文件都没有反映这一点，并且仍然包含旧的示例。

正确的代码是

should_redirect_to "news articles page" { news_articles_path }

第一个参数只是一个文本描述（它不像旧版本那样进行评估），用于生成测试名称，因此您会得到一个像 ' 这样的测试名称'应该重定向到新闻文章页面'

Answer 2

也许您应该在调用删除时使用符号和 post 方法：（

 assert_difference 'Article.count', -1 do
    post :delete, :id => ...
  end

引用自 http://api.rubyonrails.org/classes/ActiveSupport/Testing/Assertions.html#M001427）

Answer 3

tkramar 解决方案指向正确的方向，但我必须将代码编写为：

should_redirect_to("news articles page") { news_articles_path }

另请参阅 http://dev.thoughtbot.com/shoulda/classes/Shoulda/ActionController/Macros.html#M000015