NoMethodError: undefined method `メソッド名' for nil:NilClassの対処法を教えてほしい。

Question

解決したいこと

rails tutorial10章で10.4.3 ユーザー削除のテストを実装して、テストをしてみたところ下記のエラーが出てきて、対処方法を知りたい。

Running via Spring preloader in process 3777
Run options: --seed 40217

# Running:

..................................EE.

Finished in 1.692131s, 21.8659 runs/s, 70.3255 assertions/s.

  1) Error:
UsersIndexTest#test_index_as_non-admin:
NoMethodError: undefined method `email' for nil:NilClass
    test/test_helper.rb:24:in `log_in_as'
    test/integration/users_index_test.rb:36:in `block in <class:UsersIndexTest>'

  2) Error:
UsersIndexTest#test_index_as_admin_including_pagination_and_delete_links:
NoMethodError: undefined method `email' for nil:NilClass
    test/test_helper.rb:24:in `log_in_as'
    test/integration/users_index_test.rb:19:in `block in <class:UsersIndexTest>'

37 runs, 119 assertions, 0 failures, 2 errors, 0 skips

また、NoMethodError: undefined method `メソッド名' for nil:NilClassというエラーはスペルミスかレシバーがnilというパターンがあり、もしレシバーがnilがエラー原因の時に皆さんが良く行う対処方法を教えて頂きたいです。

関連ファイル

test/test.helper.rb

require File.expand_path('../../config/environment', __FILE__)
require 'rails/test_help'

class ActiveSupport::TestCase
  # Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
  fixtures :all

  # テストユーザーがログイン中の場合にtrueを返す
  def is_logged_in?
    !session[:user_id].nil?
  end
  
  # テストユーザーとしてログインする
  def log_in_as(user)
    session[:user_id] = user.id
  end
end

class ActionDispatch::IntegrationTest

  # テストユーザーとしてログインする
  def log_in_as(user, password: 'password', remember_me: '1')
    post login_path, params: { session: { email: user.email,
                                          password: password,
                                          remember_me: remember_me } }
  end
end

test/integration/users_index_test.rb

class UsersIndexTest < ActionDispatch::IntegrationTest
  def setup
    @user = users(:michael)
  end

  test "index including pagination" do
    log_in_as(@user)
    get users_path
    assert_template 'users/index'
    assert_select 'div.pagination'
    User.paginate(page: 1).each do |user|
      assert_select 'a[href=?]', user_path(user), text: user.name
    end
  end
  
  test "index as admin including pagination and delete links" do
    log_in_as(@admin)
    get users_path
    assert_template 'users/index'
    assert_select 'div.pagination'
    first_page_of_users = User.paginate(page: 1)
    first_page_of_users.each do |user|
      assert_select 'a[href=?]', user_path(user), text: user.name
      unless user == @admin
        assert_select 'a[href=?]', user_path(user), text: 'delete'
      end
    end
    assert_difference 'User.count', -1 do
      delete user_path(@non_admin)
    end
  end

  test "index as non-admin" do
    log_in_as(@non_admin)
    get users_path
    assert_select 'a', text: 'delete', count: 0
  end
end

controllers/user_contoller.rb

  before_action :logged_in_user, only: [:index, :edit, :update, :destroy]
  before_action :correct_user, only: [:edit, :update]
  before_action :admin_user, only: :destroy
  
  def index
    @users =User.paginate(page: params[:page])
  end

  def show
    @user = User.find(params[:id])
  end
  
  def new
    @user = User.new
  end
  
  def create
    @user = User.new(user_params)
    if @user.save
      log_in @user
      flash[:success] = "Welcome to the Sample App!"
      redirect_to @user
    else 
      render 'new'
    end
  end
  
  def edit
    @user = User.find(params[:id])
  end
  
  def update
    @user = User.find(params[:id])
    if @user.update_attributes(user_params)
      flash[:success] = "Profile updated"
      redirect_to @user
    else
      render 'edit'
    end
  end
  
  def destroy
    User.find(params[:id]).destroy
    flash[:success] = "User deleted"
    redirect_to users_url
  end

  private
  
  def user_params
    params.require(:user).permit(:name, :email, :password,
                                   :password_confirmation)
  end
  
  # beforeアクション

  # ログイン済みユーザーかどうか確認
  def logged_in_user
    unless logged_in?
      store_location
      flash[:danger] = "Please log in."
      redirect_to login_url
    end
  end
  
  # 正しいユーザーかどうか確認
  def correct_user
    @user = User.find(params[:id])
    redirect_to(root_url) unless current_user?(@user)
  end
  
  # 管理者かどうか確認
  def admin_user
    redirect_to(root_url) unless current_user.admin?
  end
  
end

使用しているツール

AWS、cloud9、rails5.1.2

初心者なのでエラー箇所の見るべきコードが分かっても、どう実装すればいいかまでは理解できていません。
どうか、よろしくお願いいたします。

Answer 1

エラーが出た場合、どの行でエラーが出たかを特定しバックトレースを辿り
どの変数・返り値がnilになっているのかを確認します。

今回でしたらまずいちばん深い部分

test/test_helper.rb:24:in `log_in_as'
def log_in_as(user, password: 'password', remember_me: '1')
post login_path, params: { session: { email: user.email,
password: password,
remember_me: remember_me } }

に注目します。
それを呼び出している部分は

test/integration/users_index_test.rb:36:in `block in <class:UsersIndexTest>'
log_in_as(@non_admin)

どうやら、この@non_adminがnilになっているのが問題のようです。

よってsetupメソッド等で@admin @non_adminを割り当ててやれば解決するはずです。
そのことに注意してよく提示されているソースをお読みください

Comments

[asm]

なお、test/fixtures/users.ymlの
michaelをadmin
archerをnon_adminに名前変更してやると
勝手に@admin, @non_adminが作られるという機能も存在するのがRailsの難しいところ

[yuya-2002]

そうなんですね！
ご教示いただいた事で自分でエラーを解決できるように頑張ります！