Заглушка before_filter с RSpec - PullRequest
5 голосов
/ 11 июня 2010

Мне трудно понять, почему я не могу заглушить этот метод контроллера :load_user, так как все мои тесты не пройдены, если я изменю фактическую реализацию :load_user, чтобы она не возвращалась, и экземпляр @user.

Кто-нибудь может понять, почему моя заглушка (controller.stub!(:load_user).and_return(@user)) фактически не вызывается, когда RSpec делает запрос к контроллеру?

require 'spec_helper'

describe TasksController do

  before(:each) do
    @user = Factory(:user)
    sign_in @user
    @task = Factory(:task)
    User.stub_chain(:where, :first).and_return(@user)
    controller.stub!(:load_user).and_return(@user)
  end

  #GET Index
  describe "GET Index" do

    before(:each) do
      @tasks = 7.times{Factory(:task, :user => @user)}
      @user.stub!(:tasks).and_return(@tasks)
    end

    it "should should find all of the tasks owned by a user" do
      @user.should_receive(:tasks).and_return(@tasks)
      get :index, :user_id => @user.id
    end

    it "should assign all of the user's tasks to the view" do
      get :index, :user_id => @user.id
      assigns[:tasks].should be(@tasks)      
    end
  end

  #GET New
  describe "GET New" do

    before(:each) do
      @user.stub_chain(:tasks, :new).and_return(@task)
    end

    it "should return a new Task" do
      @user.tasks.should_receive(:new).and_return(@task)
      get :new, :user_id => @user.id
    end
  end

  #POST Create
  describe "POST Create" do

    before(:each) do
      @user.stub_chain(:tasks, :new).and_return(@task)
    end

    it "should create a new task" do
     @user.tasks.should_receive(:new).and_return(@task)
      post :create, :user_id => @user.id, :task => @task.to_s
    end

    it "saves the task" do
      @task.should_receive(:save)
      post :create, :user_id => @user.id, :task => @task
    end

    context "when the task is saved successfully" do

      before(:each) do
        @task.stub!(:save).and_return(true)
      end

      it "should set the flash[:notice] message to 'Task Added Successfully'"do 
        post :create, :user_id => @user.id, :task => @task
        flash[:notice].should == "Task Added Successfully!"
      end

      it "should redirect to the user's task page" do
        post :create, :user_id => @user.id, :task => @task
        response.should redirect_to(user_tasks_path(@user.id))
      end
    end

    context "when the task isn't saved successfully" do

      before(:each) do
        @task.stub(:save).and_return(false)
      end

      it "should return to the 'Create New Task' page do" do
        post :create, :user_id => @user.id, :task => @task
        response.should render_template('new')
      end
    end
  end

  it "should attempt to authenticate and load the user who owns the tasks" do

    context "when the tasks belong to the currently logged in user" do

      it "should set the user instance variable to the currently logged in user" do
        pending 
      end

    end

    context "when the tasks belong to another user" do

      it "should set the flash[:notice] to 'Sorry but you can't view other people's tasks.'" do
        pending
      end

      it "should redirect to the home page" do
        pending
      end
    end
  end
end

class TasksController < ApplicationController
  before_filter :load_user 

  def index
    @tasks = @user.tasks
  end

  def new
    @task = @user.tasks.new
  end

  def create
    @task = @user.tasks.new
    if @task.save
      flash[:notice] = "Task Added Successfully!"
      redirect_to user_tasks_path(@user.id)
    else
      render :action => 'new'
    end
  end

  private

  def load_user
    if current_user.id == params[:user_id].to_i
      @user = User.where(:id => params[:user_id]).first 
    else
      flash[:notice] = "Sorry but you can't view other people's tasks."
      redirect_to root_path
    end
  end
end

Кто-нибудь может понять, почему моя заглушка не работает? Как я уже сказал, мои тесты проходят только в том случае, если я убедился, что load_user работает, если нет, все мои тесты не пройдены, что заставляет меня думать, что RSpec не использует созданную мной заглушку.

Ответы [ 2 ]

8 голосов
/ 16 июня 2010

Уничтожение load_user нарушает ваши тесты, потому что очищая метод нейтрализует его. Когда контроллер вызывает load_user, он больше не выполняет ваш исходный код. Теперь он просто возвращает то, что вы указали в and_return(...) (которое возвращается в стек обратного вызова ActionController, который игнорирует все, кроме false).

Код вашего контроллера не использует возвращаемое значение этого метода; он использует переменную внутри it. Поскольку исходный код для метода load_user не запускается, переменная экземпляра @user никогда не создается. (Переменная @user в ваших тестах видна только вашим тестам.)

Но со всеми остальными окурками, которые у вас есть, я не вижу никакой причины, по которой вам вообще нужно заглушить load_user. Пока вы глушите current_user, чтобы вернуть @user (что, как я полагаю, делается в методе sign_in), в этом не должно быть никакой необходимости.

2 голосов
/ 16 июня 2010

Вы также можете попробовать проверить работоспособность заглушки, выполнив утверждение типа

controller.current_user.should == @user
...