【问题标题】:Why is this attribute lost before it's evaluated in this rspec test?为什么在此 rspec 测试中评估此属性之前会丢失此属性?
【发布时间】:2016-12-08 15:27:02
【问题描述】:

我正在使用设计并已覆盖注册控制器,以允许按照此处的说明“软删除”用户:devise/wiki/How-to:-Soft-delete-a-user

我在代码中添加了一些 puts 命令来跟踪测试,我的 rspec 结果如下所示:

    10:00:30 - INFO - Running: spec/controllers/users/registrations_controller_spec.rb
    ..
>   BEFORE: id:2, deleted_at:
>    -- model: id:2, deleted_at:2016-12-08 15:00:35 UTC
>    -- controller: id:2, deleted_at:2016-12-08 15:00:35 UTC
>   AFTER: id:2, deleted_at:
    F..

    Failures:

      1) Users::RegistrationsController GET #destroy sets 'deleted_at' date for user
         Failure/Error: expect(user.deleted_at).to_not be_nil

           expected: not nil
                got: nil
         # ./spec/controllers/users/registrations_controller_spec.rb:44:in `block (3 levels) in <top (required)>'

    Finished in 0.37438 seconds (files took 3.7 seconds to load)
    5 examples, 1 failure

正如puts 所示,我的用户一开始将deleted_at 字段设置为null。模型将deleted_at 设置为当前时间,控制器确认日期已设置,但是当我在测试中到达expect() 语句时user.deleted_at 又回到nil。这似乎是我的测试有问题,因为应用程序运行正常。

/spec/controllers/users/registrations_controller_spec.rb

  require 'rails_helper'
  require 'factory_girl_rails'

  RSpec.describe Users::RegistrationsController, type: :controller do    
     include Devise::Test::ControllerHelpers

     describe "GET #destroy" do

        it "sets 'deleted_at' date for user" do
          request.env["devise.mapping"] = Devise.mappings[:user]

          # sign-in user
          user = FactoryGirl.create(:user_for_account_update)
          sign_in(user, scope: :user) # note: does not verify correct password

          # soft-delete user
          puts "BEFORE: id:#{user.id}, deleted_at:#{user.deleted_at}"
          delete :destroy, {:id => user.id}
          puts "AFTER: id:#{user.id}, deleted_at:#{user.deleted_at}"

          expect(user.deleted_at).to_not be_nil
        end
     end
  end

/spec/factories/user.rb

require 'ffaker'

FactoryGirl.define do
  factory :user do |u|
    @pass = "password"

    name FFaker::Name.name
    email { |me| "#{name.to_s.gsub(/\s/,'.')}_#{rand(1000).to_s}@testing.com" }
    password @pass

    factory :user_for_account_update do
      password_confirmation @pass
      confirmed_at Date.today
    end
  end
end

/app/models/user.rb

  ...

  # instead of deleting, indicate the user requested a delete & timestamp it
  def soft_delete
    update_attribute(:deleted_at, Time.current)
    puts " -- model: id:#{id}, deleted_at:#{deleted_at.to_s}"
  end

  ...

/app/controllers/users/registrations_controller.rb

# This class overrides devise's registration controller.  All
# other devise actions remain (I believe) handled as usual by 
# devise.  I think it is the route that does that by specifying
# this controller specifically for the purpose of registrations
#
# see also: http://stackoverflow.com/questions/3546289
class Users::RegistrationsController < Devise::RegistrationsController
  before_filter :configure_permitted_parameters, if: :devise_controller?

  # SOFT DELETE /resource
  # Follows devise guide: https://github.com/plataformatec/devise/wiki/How-to:-Soft-delete-a-user-when-user-deletes-account
  def destroy
    resource.soft_delete
    Devise.sign_out_all_scopes ? sign_out : sign_out(resource_name)
    set_flash_message :notice, :destroyed if is_flashing_format?
    yield resource if block_given?
    respond_with_navigational(resource){ redirect_to after_sign_out_path_for(resource_name) }
    puts " -- controller: id:#{resource.id}, deleted_at:#{resource.deleted_at}"
  end

  protected

  # we have to explicitly permit params in overridden controllers like this
  # see https://github.com/plataformatec/devise#strong-parameters
  def configure_permitted_parameters
    devise_parameter_sanitizer.permit(:sign_up) do |u|
      u.permit(:name, :email, :password, :password_confirmation)
    end
    devise_parameter_sanitizer.permit(:account_update) do |u|
      u.permit(:name, :email, :password, :password_confirmation, :current_password)
    end
  end
end

【问题讨论】:

  • 您是否尝试过 user.reload 从 DB 获取更新信息?
  • 我对此一无所知(显然,我对 Rails 很陌生)。非常有意义并解决了问题 - 谢谢!

标签: ruby-on-rails ruby-on-rails-4 rspec devise


【解决方案1】:

您使用的是在控制器操作之前创建的user,因此它具有缓存值(包括deleted_at)。

您可以使用user.reload 从数据库中获取更新的属性

【讨论】:

    猜你喜欢
    • 2012-07-02
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2012-12-03
    • 1970-01-01
    相关资源
    最近更新 更多