Railsチュートリアル第７版の11章 11.3.1authenticated?メソッドの抽象化の突破。 エラーの原因が不明です

Question

実現したいこと

• エラー原因の特定。
• テストのパス

発生している問題・分からないこと

11.3.1authenticated?メソッドの抽象化
https://railstutorial.jp/chapters/account_activation?version=7.0#sec-generalizing_the_authenticated_method
の進行中でテストをしたところエラーが２件発生しました。

エラーメッセージ

ERROR SessionsHelperTest#test_current_user_returns_nil_when_remember_digest_is_wrong (0.13s)
Minitest::UnexpectedError:         ArgumentError: wrong number of arguments (given 1, expected 2)
            app/models/user.rb:39:in `authenticated?'
            test/helpers/sessions_helper_test.rb:26:in `current_user'
            test/helpers/sessions_helper_test.rb:17:in `block in <class:SessionsHelperTest>'

ERROR SessionsHelperTest#test_current_user_returns_right_user_when_session_is_nil (0.14s)
Minitest::UnexpectedError:         ArgumentError: wrong number of arguments (given 1, expected 2)
            app/models/user.rb:39:in `authenticated?'
            test/helpers/sessions_helper_test.rb:26:in `current_user'
            test/helpers/sessions_helper_test.rb:11:in `block in <class:SessionsHelperTest>'

該当のソースコード

authenticated?メソッドを含めた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: true
  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))
    remember_digest
  end

  # セッションハイジャック防止のためにセッショントークンを返す
  # この記憶ダイジェストを再利用しているのは単に利便性のため
  def session_token
    remember_digest || remember
  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

current_user内の抽象化したauthenticated?メソッド 

app/helpers/sessions_helper.rb

module SessionsHelper

  # 渡されたユーザーでログインする
  def log_in(user)
    session[:user_id] = user.id
    # セッションリプレイ攻撃から保護する
    # 詳しくは https://techracho.bpsinc.jp/hachi8833/2023_06_02/130443 を参照
    session[:session_token] = user.session_token
  end

  # 永続的セッションのためにユーザーをデータベースに記憶する
  def remember(user)
    user.remember
    cookies.permanent.encrypted[:user_id] = user.id
    cookies.permanent[:remember_token] = user.remember_token
  end

  # 記憶トークンcookieに対応するユーザーを返す
  def current_user
    if (user_id = session[:user_id])
      user = User.find_by(id: user_id)
      @current_user ||= user if session[:session_token] == user.session_token
    elsif (user_id = cookies.encrypted[:user_id])
      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を返す
  def current_user?(user)
    user && user == current_user
  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)
    reset_session
    @current_user = nil
  end

  # アクセスしようとしたURLを保存する
  def store_location
    session[:forwarding_url] = request.original_url if request.get?
  end

end

Userテスト内の抽象化したauthenticated?メソッド 

test/models/user_test.rb

require "test_helper"

class UserTest < ActiveSupport::TestCase

  def setup
    @user = User.new(name: "Example User", email: "[e-mail]",
                    password: "foobar", password_confirmation: "foobar")
  end

  test "should be valid" do
    assert @user.valid?
  end

  test "name should be present" do
    @user.name = ""
    assert_not @user.valid?
  end

  test "email should be present" do
    @user.email = "     "
    assert_not @user.valid?
  end

  test "name should not be too long" do
    @user.name = "a" * 51
    assert_not @user.valid?
  end

  test "email should not be too long" do
    @user.email = "a" * 244 + "@example.com"
    assert_not @user.valid?
  end

  test "email validation should accept valid addresses" do
    valid_addresses = %w[[e-mail] [e-mail] [e-mail]
                        [e-mail] [e-mail]]
    valid_addresses.each do |valid_address|
      @user.email = valid_address
      assert @user.valid?, "#{valid_address.inspect} should be valid"
    end
  end

  test "email validation should reject invalid addresses" do
    invalid_addresses = %w[user@example,com user_at_foo.org user.name@example.
                           foo@bar_baz.com foo@bar+baz.com]
    invalid_addresses.each do |invalid_address|
      @user.email = invalid_address
      assert_not @user.valid?, "#{invalid_address.inspect} should be invalid"
    end
  end


  test "email addresses should be unique" do
    duplicate_user = @user.dup
    @user.save
    assert_not duplicate_user.valid?
  end

  test "password should be present (nonblank)" do
    @user.password = @user.password_confirmation = " " * 6
    assert_not @user.valid?
  end

  test "password should have a minimum length" do
    @user.password = @user.password_confirmation = "a" * 5
    assert_not @user.valid?
  end

  test "authenticated? should return false for a user with nil digest" do　⇦追加した部分です。
    assert_not @user.authenticated?(:remember, '')
  end
end

※追加した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

# 記憶トークンcookieに対応するユーザーを返す
  def current_user
    if (user_id = session[:user_id])
      @current_user ||= User.find_by(id: user_id)
    elsif (user_id = cookies.encrypted[:user_id])
      user = User.find_by(id: user_id)
      if user && user.authenticated?(cookies[:remember_token])
        log_in user
        @current_user = user
      end
    end
  end

end

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

• チャットgptにて質問した結果を引用

エラーメッセージから判断すると、authenticated? メソッドに渡す引数が間違っている可能性があります。authenticated? メソッドは2つの引数を期待していますが、渡された引数が1つだけであることが原因でエラーが発生しているようです。

authenticated? メソッドに渡す引数を確認し、適切なトークン（cookies[:remember_token] など）を渡してみてください。

修正方法
authenticated? メソッドに渡す引数を確認し、適切なトークン（cookies[:remember_token] など）を渡してみてください。

以下のように修正してみてください：

修正例:
ruby
コードをコピーする

user.rb の authenticated? メソッド

def authenticated?(attribute, token)
digest = send("#{attribute}_digest")
return false if digest.nil?
BCrypt::Password.new(digest).is_password?(token)
end

sessions_helper.rb 内での呼び出し

if user && user.authenticated?(:remember, cookies[:remember_token]) # ここでトークンを渡す
log_in user
@current_user = user
end

修正箇所がないように見えます。
sessions_helper_test.rbを追記しました。リスト 11.30のrails testでgreenになるはずがエラーによってRedの状態です。

補足

Rails 7.0.8.4

Comments

[maisumakun]

「環境を再起動して、それでもエラーになるかチェックしてみる」「あえてエラーの出るようなコードを仕込むことで、本当に書き換えたコードを実行していると確認する」ことなどをまず行ってみましょう。

[winterboum]

test_current_user_returns_nil_when_remember_digest_is_wrong
のcodeが載ってないですね

[12a1070]

>> maisumakun　様
Rails sをしても同じでした。

>> winterboum　様
test_current_user_returns_nil_when_remember_digest_is_wrongのコード。sessions_helper_test.rbを追記しました。テストのエラー内容は同じです。

[winterboum]

「テストのエラー内容は同じです。」＜＝ これ本当？ エラーが指摘してる行とcodeとずれてる気がする。

Answer 1

app/helpers/sessions_helper.rbの

# 現在ログイン中のユーザーを返す（いる場合）
  def current_user
    if (user_id = session[:user_id])
      user = User.find_by(id: user_id)
      @current_user ||= user if session[:session_token] == user.session_token
    elsif (user_id = cookies.encrypted[:user_id])
      user = User.find_by(id: user_id)
      if user && user.authenticated?(cookies[:remember_token])
        log_in user
        @current_user = user
      end
    end
  end

の

 if user && user.authenticated?(cookies[:remember_token])

を

if user && user.authenticated?(:remember, cookies[:remember_token])

に変えればパスできました。

Answer 2

エラーメッセージは 17行 → 26行 → user.rb:39行 ってなってるけど、
「追加したsessions_helper_test.rb」の 17行から26行には行かないと思われる。また user.rb39行は何もない行だからここでエラーが起きるはずばない。
エラーメッセージを採ったときと1行ずれてて今載ってる sessions_helper_test.rb の 18行→27行 → user.rb:41行 と思われるのでその線で回答すると
if user && user.authenticated?(cookies[:remember_token]) の user.authenticated? は引数が一つ、 user.rb:41行 の def authenticated?(attribute, token) は引数２つを要求してる。

ということで
ArgumentError: wrong number of arguments (given 1, expected 2)

Comments

[12a1070]

>> winterboum　様

改めてテストを行なったところ
```Ruby

ERROR SessionsHelperTest#test_current_user_returns_right_user_when_session_is_nil (1.59s)
Minitest::UnexpectedError: ArgumentError: wrong number of arguments (given 1, expected 2)
app/models/user.rb:39:in `authenticated?'
test/helpers/sessions_helper_test.rb:26:in `current_user'
test/helpers/sessions_helper_test.rb:11:in `block in <class:SessionsHelperTest>'

ERROR SessionsHelperTest#test_current_user_returns_nil_when_remember_digest_is_wrong (1.60s)
Minitest::UnexpectedError: ArgumentError: wrong number of arguments (given 1, expected 2)
app/models/user.rb:39:in `authenticated?'
test/helpers/sessions_helper_test.rb:26:in `current_user'
test/helpers/sessions_helper_test.rb:17:in `block in <class:SessionsHelperTest>'

39/39: [========] 100% Time: 00:00:01, Time: 00:00:01

Finished in 1.72135s
39 tests, 162 assertions, 0 failures, 2 errors, 0 skips

```
関連するコードは

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

# 渡されたトークンがダイジェストと一致したらtrueを返す
def authenticated?(attribute, token)　⇦ app/models/user.rb:39:in `authenticated?'
digest = send("#{attribute}_digest")
return false if digest.nil?
BCrypt::Password.new(digest).is_password?(token)
end
```

```Ruby
test/helpers/sessions_helper_test.rb:26:in `current_user'

# 記憶トークンcookieに対応するユーザーを返す
def current_user
if (user_id = session[:user_id])
@current_user ||= User.find_by(id: user_id)
elsif (user_id = cookies.encrypted[:user_id])
user = User.find_by(id: user_id)
if user && user.authenticated?(cookies[:remember_token])　⇦２６行目
log_in user
@current_user = user
end
end
end

```
```Ruby
test/helpers/sessions_helper_test.rb:11:in `block in <class:SessionsHelperTest>'

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

```
エラーが起きたのは分かったのですがテキストを噛み砕いてやっているのですが変数やインスタンスはどう変えれば良いのか分からず。

[winterboum]

エラーが発覚したのは app/models/user.rb:39:in `authenticated?' ですが、その原因となったのがどのfileのどの行か、というのは理解できました？

[12a1070]

エラー文が

```Ruby

ERROR SessionsHelperTest#test_current_user_returns_nil_when_remember_digest_is_wrong (1.03s)
Minitest::UnexpectedError: ArgumentError: wrong number of arguments (given 1, expected 2)
app/models/user.rb:39:in `authenticated?'
test/helpers/sessions_helper_test.rb:26:in `current_user'
test/helpers/sessions_helper_test.rb:17:in `block in <class:SessionsHelperTest>'

ERROR SessionsHelperTest#test_current_user_returns_right_user_when_session_is_nil (1.04s)
Minitest::UnexpectedError: ArgumentError: wrong number of arguments (given 1, expected 2)
app/models/user.rb:39:in `authenticated?'
test/helpers/sessions_helper_test.rb:26:in `current_user'
test/helpers/sessions_helper_test.rb:11:in `block in <class:SessionsHelperTest>'
```

39行目に当たる
```Ruby
def authenticated?(attribute, token) ⇦３９行目でエラー発覚
digest = send("#{attribute}_digest")
return false if digest.nil?
BCrypt::Password.new(digest).is_password?(token)
end
```

session_helper_test.rbの
```Ruby
# 現在ログイン中のユーザーを返す（いる場合）
def current_user
if (user_id = session[:user_id])
@current_user ||= User.find_by(id: user_id)
elsif (user_id = cookies.encrypted[:user_id])
user = User.find_by(id: user_id)
if user && user.authenticated?(cookies[:remember_token])　⇦ 26:in `current_user'
log_in user
@current_user = user
end
end
end
```

おそらく２６行目にcurrent_user'がないことが原因だと思います。その後のコードをどう書けば良いのか分からないです。

if user && user.authenticated?(cookies[:remember_token])の上にbinding.pryして

[2] pry(#<SessionsHelperTest>)> @current_user
したのですがnillでした。

[winterboum]

エラーメッセージきちんと読んで。
どのfileの何行目でどんなエラーが起きているか
そこへは どのfileの何行目から行ってるのか

が書かれているから、そこに注目。

「２６行目に」は正しいけど、「current_user'がないことが原因」ってなるのはエラーメッセージきちんと読んでいない。

[12a1070]

回答ありがとうございます。
ベストアンサーは別の方を選ばせていただきましたが、こちらの回答も非常に参考になりました。