Rspec的受让人失败上增加更多的代码

Question

控制器：测试失败，如果：

def index 
    @products = Product.all 
    @products.present? ? @no_products_found = "" : @no_products_found = "No Products available." 
end 

失败消息： 预期：2 得到：0

(compared using ==) 

Rspec的：

it "assigns products" do 
    get :index 
    FactoryGirl.create(:product, name: "product 1") 
    FactoryGirl.create(:product, name: "product 2") 
    expect(assigns(:products).size).to eq(2) 
end 

如果我将动作更改为：

def index 
    @products = Product.all 
end 

PS：新的导轨。感谢您的输入。

Answer 1

你需要做get之前调用FactoryGirl.create。当然，记录必须在提出请求之前存在。

it "assigns products" do 
    FactoryGirl.create(:product, name: "product 1") 
    FactoryGirl.create(:product, name: "product 2") 
    get :index 
    expect(assigns(:products).size).to eq(2) 
end 

虽然我亲自做的规范一点更加明确：

it "assigns products" do 
    get :index 
    products = [ 
     FactoryGirl.create(:product, name: "product 1"), 
     FactoryGirl.create(:product, name: "product 2") 
    ] 
    expect(assigns(:products)).to eq(products) 
    end 
end 

再有就是你的控制器动作的另一个问题。应谨慎使用三元条件运算符，在这种情况下，我强烈建议不要使用它，因为它会妨碍可读性。这条线：

@products.present? ? @no_products_found = "" : @no_products_found = "No Products available." 

应该再次写成

@no_products_found = @products.present? ? "" : "No Products available." 

甚至

@no_products_found = if @products.present? 
         "" 
        else 
         "No Products available." 
        end 

，我个人不喜欢乱用这种简单的观点的逻辑我的控制器动作。也许它只用在一个地方，所以我的解决方案将完全位于视图层：

<% if @products.present? %> 
    <% @products.each do |product| %> 
    <!-- Render products --> 
    <% end %> 
<% else %> 
    No products available. 
<% end %> 

Answer 2

这是因为您在调用索引方法后创建产品。更改规格，以这样的：

it "assigns products" do 
    FactoryGirl.create(:product, name: "product 1") 
    FactoryGirl.create(:product, name: "product 2") 
    # created products above and then make call to index: 
    get :index 
    expect(assigns(:products).size).to eq(2) 
end