Railsチュートリアル１１章エラー

Question

前提・実現したいこと

Railsチュートリアル学習中のものです。
１１章のアカウント認証のところまで進めましたがここでrails testコマンド入力をすると下記エラーが出ました。
リンク内容
このエラーの解消方法がわからず詰まっております。
該当されると思われるファイルのコードは下記に貼り付けております。

発生している問題・エラーメッセージ

ERROR["test_current_user_returns_nil_when_remember_digest_is_wrong", SessionsHelperTest, 2.259955172000005]
 test_current_user_returns_nil_when_remember_digest_is_wrong#SessionsHelperTest (2.26s)
ArgumentError:         ArgumentError: wrong number of arguments (given 1, expected 2)
            app/models/user.rb:32:in `authenticated?'
            app/helpers/sessions_helper.rb:60:in `current_user'
            test/helpers/sessions_helper_test.rb:17:in `block in <class:SessionsHelperTest>'

ERROR["test_current_user_returns_right_user_when_session_is_nil", SessionsHelperTest, 2.2706089260000084]
 test_current_user_returns_right_user_when_session_is_nil#SessionsHelperTest (2.27s)
ArgumentError:         ArgumentError: wrong number of arguments (given 1, expected 2)
            app/models/user.rb:32:in `authenticated?'
            app/helpers/sessions_helper.rb:60:in `current_user'
            test/helpers/sessions_helper_test.rb:11:in `block in <class:SessionsHelperTest>'

該当のソースコード

app/models/user.rb

class User < ApplicationRecord
  attr_accessor :remember_token, :activation_token
  before_save   :downcase_email
  before_create :create_activation_digest
  validates :name,  presence: true, length: { maximum: 50 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+.[a-z]+\z/i
  validates :email, presence: true, length: { maximum: 255 },
                    format: { with: VALID_EMAIL_REGEX },
                    uniqueness: { case_sensitive: false }
  has_secure_password
  validates :password, presence: true, length: { minimum: 6 }, allow_nil: true

  # 渡された文字列のハッシュ値を返す
  def User.digest(string)
    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                  BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  # ランダムなトークンを返す
  def User.new_token
    SecureRandom.urlsafe_base64
  end

  # 永続セッションのためにユーザーをデータベースに記憶する
  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end

  # トークンがダイジェストと一致したらtrueを返す
  def authenticated?(attribute, token)
    digest = send("#{attribute}_digest")
    return false if digest.nil?
    BCrypt::Password.new(digest).is_password?(token)
  end
  
  # ユーザーのログイン情報を破棄する
  def forget
    update_attribute(:remember_digest, nil)
  end
  
  private

    # メールアドレスをすべて小文字にする
    def downcase_email
      self.email = email.downcase
    end

    # 有効化トークンとダイジェストを作成および代入する
    def create_activation_digest
      self.activation_token  = User.new_token
      self.activation_digest = User.digest(activation_token)
    end  
    
end

app/helpers/sessions_helper.rb

module SessionsHelper

  # 渡されたユーザーでログインする
  def log_in(user)
    session[:user_id] = user.id
  end
  
  # ユーザーのセッションを永続的にする
  def remember(user)
    user.remember
    cookies.permanent.signed[:user_id] = user.id
    cookies.permanent[:remember_token] = user.remember_token
  end
  
  # 渡されたユーザーがログイン済みユーザーであればtrueを返す
  def current_user?(user)
    user == current_user
  end

  # 現在ログイン中のユーザーを返す (いる場合)
  def current_user
    if (user_id = session[:user_id])
      @current_user ||= User.find_by(id: user_id)
    elsif (user_id = cookies.signed[:user_id])
      raise       # テストがパスすれば、この部分がテストされていないことがわかる
      user = User.find_by(id: user_id)
      if user && user.authenticated?(:remember, cookies[:remember_token])
        log_in user
        @current_user = user
      end
    end
  end

  # ユーザーがログインしていればtrue、その他ならfalseを返す
  def logged_in?
    !current_user.nil?
  end
  
  # 永続的セッションを破棄する
  def forget(user)
    user.forget
    cookies.delete(:user_id)
    cookies.delete(:remember_token)
  end
  
   # 現在のユーザーをログアウトする
  def log_out
    forget(current_user)
    session.delete(:user_id)
    @current_user = nil
  end

test/helpers/sessions_helper_test.rb

require 'test_helper'

class SessionsHelperTest < ActionView::TestCase

  def setup
    @user = users(:michael)
    remember(@user)
  end

  test "current_user returns right user when session is nil" do
    assert_equal @user, current_user
    assert is_logged_in?
  end

  test "current_user returns nil when remember digest is wrong" do
    @user.update_attribute(:remember_digest, User.digest(User.new_token))
    assert_nil current_user
  end
end

試したこと

ネットで検索すると引数の数が違うということでエラーが出ていますと書かれていますが、どこをどのように直せばよいのかわかりません。

補足情報（FW/ツールのバージョンなど）

AWS : Cloud9

Answer 1

こんばんは。
もし解決済みとかでしたら、ご容赦くださいませ。
質問的なところもありますが、コメントしてみますね。

メッセージの件

引数の数が違うということでエラーが出ています

というので合っていますので、ログを丁寧に見ていくと良いです。

「sessions_helper.rbのcurrent_userのところで、user.rbのauthenticated?を呼び出してる。でも、引数が合わないよ」というメッセージになります。

一見すると問題なさそうな気がするのですが、ちょっと気になったのが、エラーで出ている行番号です。

app/models/user.rb:32:in `authenticated?'

        app/helpers/sessions_helper.rb:60:in `current_user'
        test/helpers/sessions_helper_test.rb:11:in `block in <class:SessionsHelperTest>'

sessions_helper.rbのソースコードを貼っていただいていますが、実際は、本当に60行までありますか？

また、60行目のcurrent_userのところで、user.rbのauthenticated?を呼び出している、という処理が書かれていますか？

もしかしたら編集の際のインデントやendの抜け漏れがあったりするかもしれません。ファイルの末尾までチェックするといいのでは。

チュートリアルの補足

Railsチュートリアルの各章、すこしずつ変えて進めていくと、抜粋だけだとわかりにくい点があるかもしれません。
チュートリアルのサイトを運用してくださっているYassLabさまのリポジトリで、各章の状態を追えるようになっているリポジトリがあります。

困ったらこちらを参考にされると良いと思います。

https://github.com/yasslab/sample_apps

今回話題にしている11章の課題が終わった段階だと、こちらになりますね。

https://github.com/yasslab/sample_apps/tree/master/5_1_2/ch11

Comments

[takuo5]

suamaさん、コメント頂きましてありがとうございます。
とてもわかりやすいご解説をしていただいて本当にありがとうございます。

YassLabさまのソースコードと自分のコードを見比べてみると下記のようにケアレスミスがありました。

①def current_userが２つ定義されていました。
⇛　１つ削除しました。
②def current_userの中のコメントアウト部分がコメントアウトされていなかった。　⇛　コメントアウトしました。

上記２点を修正したところエラーが無くなりました。

誠にお恥ずかしい間違えで申し訳ありません。
Railsチュートリアル進めるにあたって今後は下記のことに気をつけていこうと思います。

①エラーメッセージのしっかりと読んで理解する。
②エラーメッセージにある箇所を確認。
③調べてわからない場合は完成版のソースコードと見比べて見る。

非常に助かりました。
ありがとうございました。

[suama]

エラーが解消されたようで何よりです！
エラーの読み取り方、原因の探り方も、ご自身の中でだいぶ理解されているかと思います。がんばってください！