1

对于更高级的 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

控制器/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
4

2 回答 2

4

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

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

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

于 2013-03-16T00:40:19.967 回答
2

对于第一种情况，只有在返回response.should be_success状态代码时才会通过200。在您的情况下，您正在重定向到，home因此状态代码将介于300 - 399. 将第一个测试更改为：

 response.should be_redirect

还不确定第二个。

于 2013-03-15T18:49:35.140 回答