RSpec - 没有对应视图的控制器规范

Question

对于更高级的 Rails 开发人员来说，这是一个简单的问题，但我无法确定答案。我有两个非常基本的控制器规格没有通过。我没有关联的视图（我通过RailsAdmin 添加/删除产品）但这不应该是一个问题，因为在控制器中我在每个操作中都进行了重定向。

下面附上相关代码。非常感谢您的帮助。谢谢！

products_controller_spec.rb 需要'spec_helper'

describe ProductsController do
   describe 'GET #new' do
it "creates a new product" do
  get :new
    response.should be_success
  end
end

describe 'POST #create' do
  it "creates a new product and saves it" do
    expect{
  post :create, product: FactoryGirl.attributes_for(:product)
  }.to change(Product, :count).by(1)
end
  end
end

controllers/products.rb

class ProductsController < ApplicationController

  def new
    @product = Product.new
    redirect_to 'home'
  end

  def create
   @product = Product.create(params[:product])
   redirect_to 'home'
   end
 end

错误信息：

Failures:

  1) ProductsController GET #new creates a new product
     Failure/Error: response.should be_success
       expected success? to return true, got false
     # ./spec/controllers/products_controller_spec.rb:7:in `block (3 levels) in <top (required)>'

  2) ProductsController POST #create creates a new product and saves it
     Failure/Error: expect{
       count should have been changed by 1, but was changed by 0
     # ./spec/controllers/products_controller_spec.rb:13:in `block (3 levels) in <top (required)>'

Finished in 0.07747 seconds
2 examples, 2 failures

Answer 1

如果您真的不想呈现响应，请使用此而不是重定向。

head :ok

您应该使用 Product.create！

def create
  Product.create!(params[:product])
  head :ok
end

或者查看Product.create的返回结果

def create
  if Product.create(params[:product])
    head :ok
  else
    head :unprocessable_entity
  end
end

或者更好的是，在响应中返回错误

def create
  product = Product.new(params[:product])

  if product.save
    head :ok
  else
    render json: product.errors
  end
end

您当前的代码无法创建产品，并且创建调用返回 false，但您没有检查它。使用创建！如果创建失败会为你抛出错误，否则你需要检查 create 的返回值并处理返回值是 true 还是 false 的逻辑。如果你不这样做，你会得到无声的失败，这是最糟糕的失败。

Answer 2

对于第一种情况response.should be_success 只通过 如果返回状态码 200。在您的情况下，您将重定向到 home 所以 状态码将在300 - 399 之间。将第一个测试更改为：

 response.should be_redirect

目前还不确定第二个。