Научиться spec..seem, возникли проблемы

Question

Вот мой код, я spec'ing:

def vote_up 
    get_vote 
    @vote.value += 1 unless @vote.value == 1 
    @vote.save 
    respond_to do |format| 
    format.js { render :action => "vote", :layout => false } 
    end 
end 

Кажется довольно просто. Это то, что я пытаюсь чертежу его:

it "should vote up" do 
    @mock_cat = Factory.create(:category) 
    Category.stub(:mock_cat) 
    @mock_post = Factory.create(:post) 
    Post.stub(:current_post).and_return(@mock_post) 

    @vote = Factory(:vote) 

    get :vote_up, :id => @vote 
    @vote.reload.value.should == 1  
    end 

Это возвращение этого:

undefined method `to_i' for #<Vote:0x1052a4af8> 

Я не могу понять, почему же. Если бы я запустил свой mock_vote как (: vote), не пропустил бы он через метод контроллера и не прислал бы +1 к нему?

Update

Вот частный метод из моего posts_controller.rb

private 

def get_vote 
    current_post = Post.all.detect{|r| r.id == params[:id].to_i} 
    @post = current_post 
    @vote = current_post.votes.find_by_user_id(current_user.id) 
    unless @vote 
    @vote = Vote.create(:user_id => current_user.id, :value => 0) 
    current_post.votes << @vote 
    end 
end 

Ответ:

it "should vote up" do 
    @mock_cat = Factory.create(:category) 
    Category.stub(:mock_cat) 
    @post = Factory(:post) 

    get :vote_up, :id => @post.id 
    @post.reload.vote_score.should == 1  
    end 

Answer 1

Это трудно следовать, что именно ваши окурки делаете, потому что вы не размещал код для get_vote. Но я думаю, что вы переусердствовали с заглушками, когда могли просто воспользоваться теми фабриками, которые вы уже создали.

it "should vote up" do 
    # Does your Vote belong to a post or a category or anything? I don't know. 
    # Modify as needed -- Factory(:vote, :post => Factory(:post)) 
    @vote = Factory(:vote) 

    get :vote_up, :id => @vote 
    @vote.reload.value.should == 1 
end 

Примечание reload. Ваш контроллер изменяет запись, которую мы уже вытащили из базы данных, поэтому нам нужно перезагрузить ее, чтобы проверить ее новое значение.

Answer 2

Я рекомендую перемещать логику от контроллера к модели. Модели намного проще в спецификации. Вместо того, чтобы:

def vote_up 
    get_vote 
    @vote.value += 1 unless @vote.value == 1 
    @vote.save 
    respond_to do |format| 
    format.js { render :action => "vote", :layout => false } 
    end 
end 

Ваш контроллер должен выглядеть следующим образом:

def vote_up 
    @vote = Vote.vote_up(params[:id]) 
    respond_to do |format| 
    format.js { render :action => "vote", :layout => false } 
    end 
end 

или, если вам действительно нужно get_vote (может быть, вы должны переместить его в before_filter?)

def vote_up 
    @vote.vote_up 
    respond_to do |format| 
    format.js { render :action => "vote", :layout => false } 
    end 
end 

Если необходимо. добавить исключения (может быть с rescue_from)

Затем вам нужно спецификации только в модели, и некоторая интеграция (rspec, steac, огурец)