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