21

我有一个带有“PagesController”的 Rails4 应用程序。

当找不到页面时，show-method 会引发自定义异常“PageNotFoundError”。

在我定义的控制器之上 rescue_from PageNotFoundError, with: :render_not_found

render not found是一个私有方法，PagesController看起来像：

def render_not_found
  flash[:alert]=t(:page_does_not_exists, title: params[:id])
  @pages = Page.all
  render :index, status: :not_found #404
end

开发模式下的 rails-log 显示：

Started GET "/pages/readmef" for 127.0.0.1 at 2013-08-02 23:11:35 +0200
Processing by PagesController#show as HTML
  Parameters: {"id"=>"readmef"}
  ..
  Completed 404 Not Found in 14ms (Views: 12.0ms)

所以，到目前为止，它连接了我的 :status => :not_found 作品。

当我做curl -v http://0.0.0.0:3000/pages/readmef卷曲日志时

curl -v http://localhost:3000/pages/readmef
* About to connect() to localhost port 3000 (#0)
*   Trying 127.0.0.1...
* connected
* Connected to localhost (127.0.0.1) port 3000 (#0)
> GET /pages/readmef HTTP/1.1
> User-Agent: curl/7.24.0 (x86_64-apple-darwin12.0) libcurl/7.24.0 OpenSSL/0.9.8x zlib/1.2.5
> Host: localhost:3000
> Accept: */*
>
< HTTP/1.1 404 Not Found
< X-Frame-Options: SAMEORIGIN

但是使用 RSpec 进行的以下测试失败：

 it 'renders an error if page not found' do
    visit page_path('not_existing_page_321')
    expect(response.status).to eq(404)
    within( '.alert-error' ) do
      page.should have_content('Page not_existing_page_321 doesn\'t exist')
    end
  end

  1) PagesController renders an error if page not found
     Failure/Error: expect(response.status).to eq(404)

       expected: 404
            got: 200

一切看起来都很好，甚至 test.log 说 404

$ tail -f log/test.log
Started GET "/pages/not_existing_page_321" for 127.0.0.1 at 2013-08-03 09:48:13 +0200
Processing by PagesController#show as HTML
  Parameters: {"id"=>"not_existing_page_321"}
  Rendered pages/_page.haml (0.8ms)
  Rendered layouts/_navigation.haml (0.6ms)
  Rendered layouts/_messages.haml (0.2ms)
  Rendered layouts/_locales.haml (0.3ms)
  Rendered layouts/_footer.haml (0.6ms)
Completed 404 Not Found in 6ms (Views: 4.5ms)

我尝试了不同的服务器、WebRICK、Thin、独角兽。在开发和生产模式下，一切都按预期工作。即使 test.log 是正确的，但测试失败。

谁能告诉我为什么测试显示 200 而不是 404？

4

3 回答 3

18

RSpec 3+ 的另一种方法是测试异常：

it 'responds with a 404 if the page is not found' do
  expect { get(:show, id: 'bad_id') }.to raise_error(ActionController::RoutingError)
end
于 2014-12-11T10:52:44.607 回答
14

虽然我对这个解决方案不太满意，但至少这是一种解决方法：

我将测试分成两个单独的规格。一个用于测试响应代码 404（使用 GET 而不是访问），另一个用于测试警报。第二个测试是必要的，因为get它不会渲染视图——即使render_views是在规范文件的顶部定义的。

  it 'response with 404 if page not found' do
    get :show, { controller: 'pages', id: 'not_existing_page_321' }
    expect(response.status).to eq(404)
  end

  it 'renders an error-message if page not found and shows index' do
    visit page_path('page_not_found')
    within '.alert-error' do
      page.should have_content("Page page_not_found doesn't exist")
    end
  end
于 2013-08-04T20:01:31.263 回答
9

这里的问题是您混淆了 Capybara 功能测试和 RSpec 控制器测试。visit是 Capybara 提供的方法和get/response由 RSpec 控制器测试提供 -你不能一起使用它们。

要将其作为单个 RSpec 控制器测试进行测试，您可以执行以下操作：

it "returns a not found response" do
  get :show, { id: 'not_existing_page_321' }
  expect(response.status).to eq(404)
  expect(response.text).to match(/Page page_not_found doesn't exist/)
end

（注意，get 行与您发布的内容不同 - 我没有包含controller参数，就好像您将它放在spec/controllers/pages_controller_spec.rb它所属的地方一样，您不需要那个）

或者作为单个 Capybara 功能测试：

it "renders a not found response" do
  visit page_path('page_not_found')
  expect(page.status_code).to eq(404)
  within '.alert-error' do
    expect(page).to have_content("Page page_not_found doesn't exist")
  end
end
于 2014-08-24T06:41:31.560 回答