3

我正在使用 RSpec 为我的 Viewings 模型在控制器上测试 create 方法。我正在尝试创建一个模拟查看模型以最大限度地减少对我的数据库的调用，将它的新方法和保存方法存根并返回适当的值以测试我的控制器的逻辑是否正常工作： 

describe 'POST #create' do
    let(:office_listing) { create(:office_listing) }
    let(:viewing) { mock_model(Viewing).as_null_object }
····
    before do·
      Viewing.stub(:new).and_return(viewing)
    end

    describe 'with valid attributes' do
      it 'saves the viewing' do·
        Viewing.any_instance.stub(:save).and_return(true)
        viewing.should_receive(:save)
        post :create, office_listing_id: office_listing.id
      end
    end

    describe 'with invalid attributes' do
      it 'fails and renders a failure message' do
        Viewing.any_instance.stub(:save).and_return(false)
        viewing.should_receive(:save)
        post :create, :office_listing_id => office_listing.id
        assigns(:failure_message).should_not be_nil
      end
    end
  end
end

这是我的控制器代码：

def create
    @viewing = Viewing.new(params[:viewing])
    @viewing.broker = current_broker
    @viewing.office_listing = params[:office_listing_id]

    p @viewing.save

    if @viewing && @viewing.save == true
      respond_to do |format|
        format.js
        format.html { redirect_to root_path }
      end
    else
      respond_to do |format|
        @failure_message = "Unable to create viewing."
        format.js
        format.html { redirect_to root_path }
      end
    end
  end
end

问题是，即使我已经根据我所在的测试将保存返回真或假，它总是返回一个查看对象，这让我相信存根不起作用并且控制器正在处理我的将对象模拟为真实对象，当我调用 .save 时进行数据库调用。我是否需要使用 Viewing 以外的类来存根该方法？我应该只调用viewing.stub 而不是Viewing.any_instance 吗？实际上，我认为打字可能已经回答了我自己的问题，但我仍然想听听任何人对这个问题的看法。

4

1 回答 1

3

你不能同时使用

Viewing.any_instance.stub(:save)

和

viewing.should_receive(:save)

第一个用于控制流，而第二个用于测试某些东西。

你应该做：

let!(:viewing) { Viewing.new } # no lazy load, executed right away, no need to mock

def action
  post :create, office_listing_id: office_listing.id
end

before do
  Viewing.stub(:new).and_return(viewing)
end

it "calls save" do
  expect(viewing).to receive(:save).and_return(true)
  action
end
于 2013-08-03T19:46:35.673 回答