Ruby on Rails未登录用户测试报错：undefined method `admin?' for nil:NilClass

问题分析 #

1. 第一个错误（NoMethodError: undefined method 'admin?' for nil:NilClass）：未登录状态下current_user为nil，直接调用current_user.admin?会触发空对象方法调用错误。
2. 修改后出现的跳转不匹配错误：仅调整了admin_user的权限检查逻辑，但未让登录拦截逻辑优先执行，导致未登录用户被admin_user直接重定向到根路径，而非测试预期的登录页。

正确修复步骤 #

1. 调整Controller中before_action的执行顺序 #

在UsersController中，必须让登录检查过滤器优先于管理员权限过滤器。未登录用户首先应被拦截并要求登录，而非先检查管理员权限。

示例代码：

class UsersController < ApplicationController
  # 先执行登录检查，覆盖需要登录的动作（包括destroy）
  before_action :logged_in_user, only: [:index, :edit, :update, :destroy]
  # 再执行管理员权限检查，仅针对destroy动作
  before_action :admin_user,     only: :destroy

  # ...其他控制器动作

  private

  # 登录检查方法（Rails Tutorial标准实现）
  def logged_in_user
    unless logged_in?
      store_location
      redirect_to login_url, status: :see_other
    end
  end

  # 恢复原有的管理员权限检查方法（无需额外判断current_user.present?，因为logged_in_user已确保用户登录）
  def admin_user
    redirect_to(root_url, status: :see_other) unless current_user.admin?
  end
end

2. 验证测试结果 #

重新运行目标测试：

rails test test/controllers/users_controller_test.rb -n test_should_redirect_destroy_when_not_logged_in

未登录用户访问destroy动作时，会先被logged_in_user过滤器拦截，重定向到login_url，完全匹配测试预期，同时也避免了空对象调用错误。

额外说明 #

• 登录检查过滤器（logged_in_user）的核心作用是确保所有需要登录的动作先验证用户身份，这是Rails Tutorial中标准的权限控制流程。
• 管理员权限过滤器（admin_user）仅需在用户已登录的前提下验证管理员身份，因此无需额外判断current_user.present?。

内容的提问来源于stack exchange，提问作者John Sall