rails tutorialの9章の9.26のテストでエラー

Question

前提・実現したいこと

rails tutorialの9章の9.26のテストがどうしてもクリアできません。リンク内容
test "login without remembering" doの部分が間違っていると思われるのですが、何が間違っていてエラーになっているのかわかりません。

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

$ rails test
Running via Spring preloader in process 6836
Started with run options --seed 63036

 FAIL["test_login_without_remembering", UsersLoginTest, 0.5039926859999468]
 test_login_without_remembering#UsersLoginTest (0.50s)
        Expected "3ro8et89_FbpgDDVzl93Ng" to be empty.
        test/integration/users_login_test.rb:53:in `block in <class:UsersLoginTest>'

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

Finished in 0.68396s
24 tests, 59 assertions, 1 failures, 0 errors, 0 skips

sessions_helper.rbのソースコード

require 'test_helper'

class UsersLoginTest < ActionDispatch::IntegrationTest

  def setup
    @user = users(:michael)
  end


  test "login with invalid information" do
    get login_path
    assert_template 'sessions/new'
    post login_path, params: { session: { email: "", password: "" } }
    assert_template 'sessions/new'
    assert_not flash.empty?
    get root_path
    assert flash.empty?
  end

  test "login with valid information followed by logout" do
    get login_path
    post login_path, params: { session: { email:    @user.email,
                                          password: 'password' } }
    assert is_logged_in?
    assert_redirected_to @user
    follow_redirect!
    assert_template 'users/show'
    assert_select "a[href=?]", login_path, count: 0
    assert_select "a[href=?]", logout_path
    assert_select "a[href=?]", user_path(@user)
    delete logout_path
    assert_not is_logged_in?
    assert_redirected_to root_url
    # 2番目のウィンドウでログアウトをクリックするユーザーをシミュレートする
    delete logout_path
    follow_redirect!
    assert_select "a[href=?]", login_path
    assert_select "a[href=?]", logout_path,      count: 0
    assert_select "a[href=?]", user_path(@user), count: 0
  end  
  
  test "login with remembering" do
    log_in_as(@user, remember_me: '1')
    assert_not_empty cookies['remember_token']
  end

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

試したこと

ここに問題に対して試したことを記載してください。

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

環境
AWS cloud9

Answer 1

app/controllers/sessions_controller.rb の

class SessionsController < ApplicationController

  # GET /login
  def new
    #@session = Session.new
  end

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

  # DELETE /logout
  def destroy
    log_out if logged_in?
    redirect_to root_url
  end
end

      # Success
      log_in user
      params[:session][:remember_me] == '1' ? remember(user) : forget(user)
      redirect_to user

が、この順番となっているか確認してみてください。

Comments

[takuo5]

ありがとうございます。
if user && user.authenticate(params[:session][:password])
log_in user
params[:session][:remember_me] == '1' ? remember(user) : forget(user)
remember user
redirect_to user
else
flash.now[:danger] = 'Invalid email/password combination'
render 'new'
end
となっていまして、remember userの行を消すとエラーが消えました。
しかしながら、なぜそこだと気づかれましたか？
色々と勉強不足で、差し支えなければ教えていただけないでしょうか。

[mochizuki-pg]

当然ではありますが、テストは実装したコードをもとに実行されます。

今回sessionに関するエラーが起きていることが、エラーメッセージよりわかりましたので、sessions_controllerに問題があることを予想しました。

[mochizuki-pg]

今回は、三項演算子を使用してそれぞれtrue、falseの場合の処理を書いていますが、その結果に関わらず remember userを実行してしまっていたことによって、sessionの中身が空でなくなってしまっていたので、テストが失敗していました。

[takuo5]

なるほどです、’テストは実装したコードをもとに実行されます’　というのでしっくり来ました！
ありがとうございます！