remember meをoffした時にcookieが記録されていないことを確認するテストに通らない

Question

前提・実現したいこと

Railsチュートリアル9章に取り組んでいます。
以下のようなテスト（remember meチェックボックスがoffになっている際に、cookieにremember_tokenが記録されていないことを確認するテストであると理解しています）で

(users_login_test.rb)
test "login without remembering" do
    # cookieを保存してログイン
    log_in_as(@user, remember_me: '1')
    delete logout_path
    # cookieを削除してログイン
    log_in_as(@user, remember_me: '0')
    assert_empty cookies[:remember_token]
  end

最後の行の
assert_empty cookies[:remember_token]
にfailしてしまいます。
railsチュートリアルのコードの通りに進めたはずなのですが。。。
対処法も見当がつきません。

程度の低い質問であったら、大変申し訳ございません。
また、回答に必要な情報が不足している場合、お手数ですがコメントいただければ、追記いたします。

よろしくお願いいたします。

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

（ターミナルでテストを実行した時の結果です）
ubuntu:~/environment/sample_apps/6_0_0/ch08 (advanced-login) $ rails t
The dependency tzinfo-data (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for ruby but the dependency is only for x86-mingw32, x86-mswin32, x64-mingw32, java. To add those platforms to the bundle, run `bundle lock --add-platform x86-mingw32 x86-mswin32 x64-mingw32 java`.
Started with run options --seed 40658

 FAIL["test_login_without_remembering", #<Minitest::Reporters::Suite:0x000055eecbdbb658 @name="UsersLoginTest">, 2.328885603001254]
 test_login_without_remembering#UsersLoginTest (2.33s)
        Expected "fWcP-vGR3IJovyUMQhhSsw" to be empty.
        test/integration/users_login_test.rb:54:in `block in <class:UsersLoginTest>'

  24/24: [==============================================================] 100% Time: 00:00:02, Time: 00:00:02

Finished in 2.33303s
24 tests, 60 assertions, 1 failures, 0 errors, 0 skips

該当のソースコード

(test_helper.rb)
ENV['RAILS_ENV'] ||= 'test'
require_relative '../config/environment'
require 'rails/test_help'
require "minitest/reporters"
Minitest::Reporters.use!

class ActiveSupport::TestCase
  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

(user.rb)
class User < ApplicationRecord
  attr_accessor :remember_token
  before_save { self.email = email.downcase }
  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 }
  
  # ここにrememberを移動しました
  
  
  # ここにauthenticated?を移動しました
  

# class << self
  # 渡された文字列のハッシュ値を返す
  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
  
  # ここにrememberがありました
  def remember
    self.remember_token = User.new_token
    update_attribute(:remember_digest, User.digest(remember_token))
  end
  
  # 渡されたトークンがダイジェストと一致したらtrueを返す
  # ここにauthenticated?がありました
  def authenticated?(remember_token)
    return false if remember_digest.nil?
    BCrypt::Password.new(remember_digest).is_password?(remember_token)
  end
  
    # ユーザーのログイン情報を破棄する
  def forget
    update_attribute(:remember_digest, nil)
  end

  
# end
end

(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

  # 現在ログイン中のユーザーを返す (いる場合)
  def current_user
    if (user_id = session[:user_id])
      @current_user ||= User.find_by(id: user_id)
    elsif(user_id = cookies.signed[: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
  
  

  # ユーザーがログインしていれば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
end

(new.html.erb)
<% provide(:title, "Log in") %>
<h1>Log in</h1>

<div class="row">
  <div class="col-md-6 col-md-offset-3">
    <%= form_with(url: login_path, scope: :session, local: true) do |f| %>

      <%= f.label :email %>
      <%= f.email_field :email, class: 'form-control' %>

      <%= f.label :password %>
      <%= f.password_field :password, class: 'form-control' %>
      
      <%= f.label :remember_me, class: "checkbox inline" do %>
        <%= f.check_box :remember_me %>
        <span>Remember me on this computer</span>
      <% end %>  

      <%= f.submit "Log in", class: "btn btn-primary" %>
    <% end %>

    <p>New user? <%= link_to "Sign up now!", signup_path %></p>
  </div>
</div>

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

ruby 2.6.3
rails 6.0.3
AWS cloud9を使用

Comments

[asm]

コントローラーのloginアクションのソースをご提示ねがいます。

[takuma_lol]

追記修正依頼の場合メールで通知が来ないこと知らず、返信が遅くなってしまいました。
大変申し訳ございません。

ルーティングが以下のようになっており
```ruby
(routes.rb)
Rails.application.routes.draw do

get 'users/new'
root 'static_pages#home'
get '/help', to: 'static_pages#help'
get '/about', to: 'static_pages#about'
get '/contact', to: 'static_pages#contact'
get '/signup', to: 'users#new'
get '/login', to: 'sessions#new'
post '/login', to: 'sessions#create'
delete '/logout', to: 'sessions#destroy'
resources :users

end
```

new・createアクションは以下のようになっております。
```ruby
(sessions_controller.rb )
class SessionsController < ApplicationController

def new
end

def create
user = User.find_by(email: params[:session][:email].downcase)
if user&.authenticate(params[:session][:password])
log_in user
remember user
redirect_to user
else
flash.now[:danger] = 'Invalid email/password combination'
render 'new'
end
end

def destroy
log_out
redirect_to root_url
end
end

```

返信が遅くなりましたこと、大変失礼いたしました。重ねてお詫び申し上げます。

よろしくお願いいたします。

Answer 1

def create

user = User.find_by(email: params[:session][:email].downcase)
if user&.authenticate(params[:session][:password])
log_in user
remember user
redirect_to user
else
flash.now[:danger] = 'Invalid email/password combination'
render 'new'
end
end

remember userが無条件になっているのが問題です。

def create
  user = User.find_by(email: params[:session][:email].downcase)
  if user&.authenticate(params[:session][:password])
    log_in user
    remember user if params[:session][:remember_me] == '1'
    redirect_to user
  else
    flash.now[:danger] = 'Invalid email/password combination'
    render 'new'
  end
end

なお、念の為にRailsチュートリアルを確認したところ
remember meがoff時はforget処理が行われるようになっていました。

おそらく、ブラウザAでremember me状態でログインしている時に
別のブラウザBでログインした際、自動でブラウザAのremember_tokenを無効化する為の処理だと思いますが
まぁ模倣するかは好みかと思いそのままにしています。

Comments

[takuma_lol]

回答ありがとうございます。

`remember user if params[:session][:remember_me] == '1'`、試してみましたが、
["test_login_without_remembering", #<Minitest::Reporters::Suite:0x000055d64ed51108 @name="UsersLoginTest">, 3.4183716000000004]
test_login_without_remembering#UsersLoginTest (3.42s)
Expected "GMObKId3Bl1Tz46p3jqicQ" to be empty.
test/integration/users_login_test.rb:54:in `block in <class:UsersLoginTest>'
以上のメッセージが出てfailしてしまいました。
railsチュートリアル上の
params[:session][:remember_me] == '1' ? remember(user) : forget(user)
も試してみたのですが、同様の仕方でfailしました。

何か他に原因等考えられるものがあれば、お手数ですが御教示いただけると幸いです。

よろしくお願いいたします。

[asm]

うーん、思いつきませんね。

書き換え行直前にbinding.irbやconsoleを挟みparams[:session]およびparams[:session][:remember_me]に何が入ってるか
forget(user)を手動で呼んだらcookiesはどうなるかなどをデバッグしていく事になるかと思います。

[takuma_lol]

ありがとうございます。　
もう少し色々と試してみようと思います。

回答ありがとうございました。