receive_message_chain和臭味代码

Question

我正在测试我的控制器，并且有一个产生AR查询的字符串： current_user.providers.find(params[:id])。我需要它返回到我测试的对象，否则，控制器获取的参考信息与我在spec中的参考信息不同，并且某些存根（如allow(provider).to receive(:recreate)）不起作用。

我发现要做到这一点的唯一方法是使用这样的receive_message_chain： allow(provider.user).to receive_message_chain(:providers, :find => provider)。但是rspec文档says考虑使用receive_message_chain作为有异味的代码。 另外，我想后面我可能需要用另一个ID调用current_user.providers.find(otherid)来获得另一个对象，这样就不再适合我了。

有什么办法可以做得更好吗？我已经设法避免allow_any_instance_of，这也被认为是臭，所以我相信有一种方法可以避免这一点，我只是看不到它。 如果没有，我至少想知道是否有什么方法可以将with添加到receive_message_chain？

===========

我只是想测试我控制器的方法update。

# app/controllers/restream/facebooks_controller.rb 
class Restream::FacebooksController < Restream::BaseController 
    def update 
    current_user.providers.find(params[:id]) 

    if @fb.update_attributes(facebook_params) 
     if event_changed? 
     @fb.recreate 
     else 
     @fb.update 
     end 
     redirect_to restreams_path 
    else 
     render 'edit' 
    end 
    end 
end 

#spec/controllers/restream/facebooks_controller_spec.rb 

require 'rails_helper' 

describe Restream::FacebooksController do 
    let!(:facebook) { create(:restream_facebook) } 
    let!(:restream) { facebook.restream } 

    before do 
    login(restream.user) 
    end 

    describe '#update' do 
    let!(:params_hash) { { 
     :title   => facebook.title, 
     :privacy  => facebook.privacy, 
     :destination => facebook.destination, 
     :destination_id => facebook.destination_id, 
     :description => facebook.description 
     } } 
    let!(:request_hash) { { 
      :restream_facebook => params_hash, 
      :id     => facebook.id 
     } } 

    before do 
     allow(facebook.user). 
     to receive_message_chain(:providers, :find => facebook) 
     allow(facebook).to receive(:update) 
     allow(facebook).to receive(:recreate) 
    end 

    context 'updates' do 
     it 'title' do 
     params_hash[:title] = SecureRandom.hex(2) 
     post :update, request_hash 

     expect(facebook.reload.title).to eq params_hash[:title] 
     end 
    end 
    end 
end 

Answer 1

您的控制器的工作原理是从链中删除current_user？ Provider.find(params[:id])这样你就可以获得更少的链接方法和更简单的代码来测试。我不认为current_user.providers.find的连锁店做的不仅仅是Provider.find。

Answer 2

receive_messaged_chain可以是一种气味，但使用双打时测试控制器是更大的臭味。

既然你没有提供任何控制器动作的代码，我会给你常见的例子：

def destroy 
    @provider = current_user.providers.find(params[:id]) 
    @provider.delete 
end 

，不测试，如果delete方法被调用。测试物体是否从DB中消失，例如：

let(:current_user) { FactoryGirl.create(:user) } 
let!(:provider) { FactoryGirl.create(:provider, user: current_user } 
it do 
    delete :destroy, id: provider.id 
    expect(Provider.find(provider.id).to raise_error(ActiveRecord::RecordNotFound) # writing from memory, don't remember exactly how the exception is called 
end 

# or 

it do 
    expect { delete :destroy } 
    .to change{ Provider.where(id: provider.id).count }.from(1).to(0) 
end 

依此类推。一般来说，你想使用双打作为最后的解决方案。想想按照它所做的实际效果来测试代码的方法，而不是它所调用的方法。这适用于任何集成测试（哪些控制器是）。如果你正在编写单元测试并需要隔离 - 那么就模拟一些东西来实现这种隔离。