为什么这个删除测试没有删除我的对象？

Question

这是我的规范的样子：

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

在我的UsersController 中调用此操作

  def destroy
    @user = User.find(params[:id])
    if @user.destroy
      redirect_to root_path, notice: "You have successfully cancelled your account."
    else
      redirect_to :back
    end
  end

但是当我运行该规范时，我得到了这个：

RSpec::Expectations::ExpectationNotMetError: expected result to have changed by -1, but was changed by 0

我该如何解决这个问题？

编辑 1

@user1 在 context 块顶部的 before_each 调用中被初始化，it 语句所在的块。看起来像这样：

context "when the inviter is being deleted and invited has not accepted invitation" do
  before :each do
    @user1 = create(:user, gender: 0)
    @user2 = create(:user)
    @member = create(:member, email: @user2.email, first_name: @user2.first_name, last_name: @user2.last_name, bio: @user2.bio, gender: @user2.gender)
    @membership = create(:membership, member: @member, family_tree: @user1.family_tree, inviter: @user1, relation: "sister")
    @connection = create(:connection, inviter_membership: @membership)
    login_user
  end

编辑 2

当我用这个替换测试时，它可以工作：

  it "should successfullly destroy that user record" do
    delete :destroy, id: @user1
    expect(response).to redirect_to(root_path)
    # expect {
    #   delete :destroy, id: @user1
    # }.to change(User,:count).by(-1)
  end

该测试成功通过。

Finished in 1.2 seconds (files took 6.08 seconds to load)
19 examples, 0 failures, 15 pending

编辑 3

对于它的价值，在我的UsersController 中，我在顶部调用load_and_authorize_resource。当我删除它时，测试似乎确实通过了。所以问题是，我如何让它与 CanCanCan 一起工作？

这就是我的ability.rb 的样子：

class Ability
  include CanCan::Ability

  def initialize(user)
    user ||= User.new # guest user (not logged in)
    if user.has_role? :admin
      can :manage, :all
    end

    can :manage, FamilyTree, user_id: user.id
    can :manage, Comment, user_id: user.id
    can :manage, Node, user_id: user.id
    can :manage, Event, user_id: user.id
    can :manage, User, id: user.id

    can :read, Comment, user_id: user.id
    can :read, Event, user_id: user.id
  end
end

编辑 4

当我将raise 添加到我的destroy 操作时...什么也没有发生。所以看来我的destroy 动作确实没有被执行。

编辑 5

这就是我的login_user 方法的样子：

module SpecAuthentication
  def login_user
    @request.env["devise.mapping"] = Devise.mappings[:user]
    @user = FactoryGirl.create :user
    sign_in @user
  end
end

Answer 1

使用CanCanCan gem，您必须允许控制器操作。您需要将此添加到您的 ability.rb 文件中。

can :destroy, User

这将允许具有该角色的任何人销毁用户对象。您可以添加更多细节以适合您的用例。

编辑：刚刚看到 CanCanCan gem 上的 :manage 文档。

您的能力文件目前允许用户管理自己。您的 login_user 方法正在动态创建用户，而您的测试正在尝试销毁单独的用户。您可以更新您的 CanCan 功能以销毁所有用户，或者尝试销毁 current_user.id（假设您有 current_user 方法）。我强烈建议您更改您的 login_user 方法以接受用户对象，以便您知道哪个用户已登录并有权访问该对象。

编辑：

如上所述，使用方法登录特定用户，以便您可以将其作为对象访问。将 login_user 调用替换为 sign_in @user1 或接受用户对象的类似方法。

Answer 2

您的destroy action 没有受到 spec 的影响，请检查是否有某种 before_action 阻止它执行所以。

我会这样做：

  def destroy
    raise # The action was hit
    @user = User.find(params[:id])
    if @user.destroy
      redirect_to root_path, notice: "You have successfully cancelled your account."
    else
      redirect_to :back
    end
  end