Rails 3 учебник Ch. 10.6 из 5 rspec провал

Question

В решении упражнения 5 я столкнулся с проблемой, аналогичной этой вопрос .Я произвел рефакторинг на основе ответа на этот вопрос, но все еще получаю ошибку:

1) UsersController DELETE 'destroy' as an admin user should not self-destruct
     Failure/Error: lambda do
       count should have been changed by 0, but was changed by -1
    # ./spec/controllers/users_controller_spec.rb:354:in `block (4 levels) in <top     (required)>'

Моя спецификация:

  it "should destroy the user" do
    lambda do
      delete :destroy, :id => @user
    end.should change(User, :count).by(-1)
  end

  it "should redirect to the users page" do
    delete :destroy, :id => @user
    response.should redirect_to(users_path)
  end

  it "should not self-destruct" do
    lambda do
      delete :destroy, :id => @user.id
    end.should change(User, :count).by(0)
  end

и мой контроллер:

def destroy
  @user = User.find(params[:id])
  if current_user == @user
    flash[:notice] = "You cannot destroy yourself"
  else
    @user.destroy
     flash[:success] = "User destroyed"  
  end
  redirect_to users_path
end

Я проверил поведение в браузере, и он работает, как ожидалось.Как всегда, любая помощь приветствуется.Спасибо!

---

Обновленный, рабочий код:

describe "as an admin user" do

  before(:each) do
    @admin = Factory(:user, :email => "admin@example.com", :admin => "true")
    test_sign_in(@admin)
  end

  it "should have links to destroy a user" do
    get :index
    response.should have_selector("a", :content => "delete" )
  end

  it "should destroy the user" do
    lambda do
      delete :destroy, :id => @user
    end.should change{ User.count }.by(-1)
  end

  it "should redirect to the users page" do
    delete :destroy, :id => @user
    response.should redirect_to(users_path)
  end

  it "should not be allowed to delete itself" do
    lambda do
      delete :destroy, :id => @admin
    end.should_not change{ User.count }
  end
end

Answer 1

Я только что понял, что прочитал неправильный один из трех тестов, которые вы опубликовали (было бы понятнее, если бы вы опубликовали только тест с ошибкой :)

Однако я запутался, ваш "it should not self destruct" тест точно такой же, как "it should destroy the user":

it "should destroy the user" do
  lambda do
    delete :destroy, :id => @user
  end.should change(User, :count).by(-1)
end

it "should not self-destruct" do
  lambda do
    delete :destroy, :id => @user.id
  end.should change(User, :count).by(0)
end

Вы проводите один и тот же тест оба раза, но ожидаете разных результатов. Я бы еще раз взглянул на «это не должно самоуничтожаться» и увидел бы, что вы подразумеваете под этим (я не уверен).

Я бы предположил, что нужно изменить то, что находится в `change (xxxx) .by (0). Что не должно меняться в ходе теста?

Answer 2

Это укусило меня тоже надолго. Попробуйте изменить свой синтаксис с:

should change(User, :count).by(-1)

до:

should change{ User.count }.by(-1)

Или попробуйте сделать:

Это немного отходит от книги, но вы можете попробовать изменить свой синтаксис. Рекомендация RSpec для этого типа ожидания:

it "should destroy the user" do
  expect{
    delete :destroy, :id => @user
  }.to change{ User.count }.by(-1)
end

Который я на самом деле считаю намного чище и в любом случае более читабельным. Не могли бы вы дать ссылку на пример, как он есть в книге?