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 

依此類推。一般來說，你想使用雙打作爲最後的解決方案。想想按照它所做的實際效果來測試代碼的方法，而不是它所調用的方法。這適用於任何集成測試（哪些控制器是）。如果你正在編寫單元測試並需要隔離 - 那麼就模擬一些東西來實現這種隔離。