控制器和存根上的 RSpec

Question

我对使用 rspec 很陌生，我正在尝试为我的控制器编写测试。我有这个控制器（我正在使用 mocha 进行存根）：

class CardsController < ApplicationController
  before_filter :require_user

  def show
    @cardset = current_user.cardsets.find_by_id(params[:cardset_id])

    if @cardset.nil?
      flash[:notice] = "That card doesn't exist. Try again."
      redirect_to(cardsets_path)
    else
      @card = @cardset.cards.find_by_id(params[:id])
    end
  end
end

我正在尝试用这样的东西来测试这个动作：

describe CardsController, "for a logged in user" do
  before(:each) do
    @cardset = Factory(:cardset)
    profile = @cardset.profile
    controller.stub!(:current_user).and_return(profile)
  end

  context "and created card" do
    before(:each) do
      @card = Factory(:card)
    end

    context "with get to show" do
      before(:each) do
        get :show, :cardset_id => @cardset.id, :id => @card.id
      end

      context "with valid cardset" do
        before(:each) do
          Cardset.any_instance.stubs(:find).returns(@cardset)
        end

        it "should assign card" do
          assigns[:card].should_not be_nil
        end

        it "should assign cardset" do
          assigns[:cardset].should_not be_nil
        end

      end
    end
  end
end

“应该分配卡组”测试通过，但我不知道如何正确地为“应该分配卡”测试正确地存根@card = @cardset.cards.find_by_id(params[:id]) 这一行。测试此操作的最佳方法是什么，或者如果我在正确的轨道上，我将如何正确地存根我的模型调用？

Answer 1

我最终在哪里寻找的存根

Cardset.stubs(:find_by_id).returns(@cardset)
@cardset.cards.stubs(:find_by_id).returns(@card)

Answer 2

好的，删除了一个错误的答案。

首先：你在存根 find 而不是 find_by_id。尽管您不需要使用 find_by_id，因为这是 find 的默认设置。所以使用find

第二：before :each 排序将在你存根Cardset 之前调用get :show

第三：检查您的 test.log 并确保您没有被重定向。您的 require_user 操作可能会在设置 current_user 之前导致重定向。

class CardsController < ApplicationController
  ...
     @card = @cardset.cards.find(params[:id])
  ...
end

describe CardsController, "for a logged in user" do
  before(:each) do
    @cardset = Factory(:cardset)
    profile = @cardset.profile
    controller.stub!(:current_user).and_return(profile)
  end

  context "and created card" do
    before(:each) do
      @card = Factory(:card)
    end

    context "with get to show" do

      context "with valid cardset" do
        before(:each) do
          Cardset.any_instance.stubs(:find).returns(@cardset)
          get :show, :cardset_id => @cardset.id, :id => @card.id
        end

        it "should assign card" do
          assigns[:card].should_not be_nil
        end

        it "should assign cardset" do
          assigns[:cardset].should_not be_nil
        end

      end
    end
  end
end