如何测试黄瓜的错误？

Question

我写了这个场景来测试当用户访问一个未发布的条目时，他们会看到一个错误页面：

Scenario: Unpublished entry
    Given there is an unpublished entry
    When a user visits the entry page
    Then he will see an error page

步骤

Given /^there is (?:an|one) unpublished entry$/ do
  @entry = create(:entry, published: false)
end

When /^a user visits the entry page$/ do
  visit entry_path(@entry)
end

Then(/^he will see an error page$/) do
  expect(response).to raise_error(ActiveRecord::RecordNotFound)
end

运行测试时，它不会通过第二步，因为它失败并出现ActiveRecord::RecordNotFound 错误；这是我想要的，但它发生在错误的步骤上。

When a user visits the entry page   # features/step_definitions/entry_steps.rb:17
  Couldn't find Entry with id=93 [WHERE "entries"."published" = 't'] (ActiveRecord::RecordNotFound)

显然我做错了什么。如何编写场景以更好地表达我的意图，即“访问未发布的条目会引发错误”？

Answer 1

如果您将 ActiveRecord::RecordNotFound 返回给用户，则您的 Web 应用程序存在问题。用户不应该看到 Rails 错误，因为它可能会给他们提供有关您服务器的敏感信息。而且它看起来很时髦！

这是模式应该是什么：

1. 用户请求无效项目
2. Rails 尝试查找无效项目并抛出 ActiveRecord::RecordNotFound
3. Rails 捕获 ActiveRecord::RecordNotFound 并将用户重定向到 404 页面。

然后在 Cucumber 中，您将测试 404 页面。

这是common HTML error codes 的列表，供您启迪。

编辑

这是捕捉ActiveRecord::RecordNotFound 错误的好方法。

class SomeModel
  around_filter :catch_not_found

private
  def catch_not_found
    yield
  rescue
     redirect_to not_found_url
  end
end

编辑 2

您可能希望返回 403 Forbidden 而不是 404 Not Found

编辑 3

Here 详细讨论了为什么用 cucumber 捕获异常很奇怪。这是捕获和重定向的另一个论据。