控制器 rspec 不会从数据库重新加载 current_user 或 @user

Question

特点

用户拥有个人资料并且应该能够对其进行更新。

---

问题

我更新了配置文件，例如将名称更改为“Homer Simpson”，但所有断言都失败了，因为数据库记录似乎没有更新。

我似乎无法获得更新的属性：

 Failure/Error: expect(subject.current_user.first_name).to eq('Homer')

   expected: "Homer"
        got: "Lew"

   (compared using ==)
 # ./spec/controllers/registrations_controller_spec.rb:67:in `block (3 levels) in <top (required)>'

注意@user.reload和subject.current_user.reload我都试过了

规范仍未通过。

---

代码

我正在使用：

• 导轨 (4.0.0)
• 设计 (3.0.3)
• rspec-rails (2.14.0)
• 水豚 (2.1.0)
• factory_girl (4.2.0)
• database_cleaner (1.1.1)

我已经检查过了：

• 我已将devise.mapping 设置为用户
• 我没有像 per this other Stackoverflow thread 那样污染设计会话的 valid_session。

registrations_controller_spec.rb

describe "User Profiles" do
  login_user

  it "Update - changes the user's attributes" do
    put :update, id: @user, user: attributes_for(:user, first_name: 'Homer')
    @user.reload
    expect(@user.first_name).to eq('Homer') # FAILS
  end
end

我尝试将@user 换成subject.current_user，就像在这个 Stackoverflow 线程中一样："Devise Rspec registration controller test failing on update as if it was trying to confirm email address"

  put :update, id: subject.current_user, user: attributes_for(:user, first_name: 'Homer')
  subject.current_user.reload
  expect(subject.current_user.first_name).to eq('Homer') # Still FAILS

但还是失败了。

控制器有问题吗？我通过current_user.id而不是params[:id]找到用户。

registrations_controller.rb

def update
  @user = User.find(current_user.id)
  email_changed = @user.email != params[:user][:email]
  password_changed = !params[:user][:password].blank?

  if email_changed or password_changed
    successfully_updated = @user.update_with_password(user_params)
  else
    successfully_updated = @user.update_without_password(user_params)
  end

  if successfully_updated
    sign_in @user, bypass: true # Sign in the user bypassing validation in case his password changed
    redirect_to user_profile_path, notice: 'Profile was successfully updated.'
  else
    render "edit"
  end
end

controller_macros.rb - 定义login_user helper

module ControllerMacros
  def login_user    
    before(:each) do
      @request.env["devise.mapping"] = Devise.mappings[:user]
      @user = FactoryGirl.create(:user)
      @user.confirm!
      sign_in @user
    end
  end
end

我的集成规范通过了。我在控制器中缺少什么？

Answer 1

我的回答可以解决您的问题，但不能直接修复您代码中的错误。为此，我需要编写更多的测试和动手调试，我没有那么多经验来通过只读来解决它:)

我不建议您像有问题的那样覆盖 Devise 的 RegistrationsController。与原始代码相比，您的代码至少缺少两点：

1. 没有 current_user 对象的副本。在实际应用中，current_user 将通过提交不好的表单来注销。

2. 缺乏清理参数

还有剩下的错误。

我的建议是直接使用Devise的方法，因为你的代码中没什么特别的，不需要重写完整的代码。 p>

class RegistrationsController < Devise::RegistrationsController
  def update
  end
  # Or even without this method.
end

就是这样。

不需要密码

def update
  params.merge!(password: current_user.password) if params[:password].blank?
  super
end

对于测试，只需编写一些随意的集成测试。 Devise 具有全面的功能测试，因此无需重复。

Answer 2

试试assigns

it "Update - changes the user's attributes" do
  put :update, id: @user, user: attributes_for(:user, first_name: 'Homer')
  homer = assigns(:user)
  @user.reload
  expect(homer.first_name).to eq('Homer')
end

更新：根据 Billy Chan 的评论，这应该可以正确测试名称是否正在更新

it "Update - changes the user's attributes" do
  put :update, id: @user, user: attributes_for(:user, first_name: 'Homer')
  homer = assigns(:user)
  @user.reload
  #expect(homer.first_name).to eq('Homer') Peter and Billy are right, this only tests
  # that the attribute was actually assigned, not that the update was successful
  expect(@user.first_name).to eq(homer.first_name)
  #however this test that the users updated `first_name` matches the attribute 
  #in the test 
end

注意：

我的这个答案基于我几个月前阅读的 Michael Hartl 教程 - 他使用这种方法，我相信他解释了原因 - 尽管我面前没有屏幕投射片刻。我稍后会查一下。

视频：

Here's the video - 它的质量非常低，因为我只使用了 quicktime 的屏幕记录 - 开始时有一些残酷的反馈循环，所以在开始的几秒钟内让你的电脑静音。

Answer 3

回答

感谢大家的建议，这些建议帮助我整理代码并找到问题所在。

失败原因：默认出厂包含的参数包括电子邮件和密码，因此控制器测试不断尝试更改用户密码。

具体来说，我在 registrations_controller_spec.rb

中更改了这行代码

put :update, id: @user, user: attributes_for(:user, first_name: 'Homer')

到：

patch :update, id: @user, user: attributes_for(:user_params, first_name: 'Homer', last_name: 'Simpson')

然后我必须更新我的工厂，所以我可以使用 :user_params 代替更新：

FactoryGirl.define do

  factory :user do
    first_name          { Faker::Name.first_name }
    last_name           { Faker::Name.last_name }
    sequence(:username) { |n| "user-#{n}" }
    email               { Faker::Internet.email }
    password            { Faker::Lorem.characters(8) }
  end

  factory :user_params, class: :user do
    first_name     { Faker::Name.first_name }
    last_name      { Faker::Name.last_name }

    factory :user_params_with_email, class: :user do
      email        { Faker::Internet.email }
    end

    factory :user_params_with_password, class: :user do
      password    { Faker::Lorem.characters(8) }
    end
  end

end

感谢所有提出建议的人。它帮助我解开我的代码，@billy-chan 正确地指出了我修复的问题。

• 未清理参数（正在完成 rails4 升级）
• 其他杂项。错误

经验教训

比较进出控制器的参数。

我的集成测试通过了，因为我没有尝试更改电子邮件或密码。