Railsのログイン認証のテスト(RSpec)

Question

###前提・実現したいこと
Rspecでログイン認証のテストがどうしてもうまくいかないです。
現在はモックを使って試しているのですが、エラーが出てしまいます。
下記のログイン認証のテストの仕方をご教授ください！
（Device等のログイン認証のライブラリは使っていません。）

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

  1) SessionsController sessions#create 正常なログイン情報でログインした場合 TOP画面へ遷移されること
     Failure/Error: user = User.find_by(login_id: params[:session][:login_id])
     
     NoMethodError:
       undefined method `[]' for nil:NilClass
     # ./app/controllers/sessions_controller.rb:9:in `create'
     # ./spec/controllers/sessions_controller_spec.rb:20:in `block (4 levels) in <top (required)>'

###Rspec

#  /sessions_controller_spec.rb

describe 'sessions#create' do
    context '正常なログイン情報でログインした場合' do
      before do
        user = create(:user)
        allow(User).to receive(:find_by).and_return(user)
      end
      it 'TOP画面へ遷移されること' do
        post :create
        expect(response).to redirect_to top_path
      end
   end
end

###Controller

#  /sessions_controller.rb
class SessionsController < ApplicationController

  skip_before_action :user_logged_in?

def create
      user = User.find_by(login_id: params[:session][:login_id])
      if user == user.authenticate(params[:session][:password])
          log_in user
          redirect_to top_path
      else
          redirect_to login_path, flash: { alert: '※ログイン情報が正しくありません'}
    end
  end

###Factory

#  /user.rb

FactoryGirl.define do
    factory :user do
        login_id 'test'
        password 'password'
        last_name '田中'
        first_name '正'
        group_id 1
        role_id 1
    end
emd

###補足情報(言語/FW/ツール等のバージョンなど)
rails -v 5
rspec -v 3.5

Answer 1

これは、params[:session]がnilなため、エラーが出てしまっています。

#sessions_controller
def create
      user = User.find_by(login_id: params[:session][:login_id])

以下のようにして、postする時にパラメーターを渡してあげましょう。
ゆーざーidとパスワードはテストに通る値を入れてください。

post :create, session: { user_id: 'ユーザーid', password: 'パスワード'}

#  /sessions_controller_spec.rb

describe 'sessions#create' do
    context '正常なログイン情報でログインした場合' do
      before do
        user = create(:user)
        allow(User).to receive(:find_by).and_return(user)
      end
      it 'TOP画面へ遷移されること' do
        post :create, session: { user_id: 'ユーザーid', password: 'パスワード'}
        expect(response).to redirect_to top_path
      end
   end
end

Comments

[takeo7]

ご回答ありがとうございます！
そして、以前2回もご回答いただいていたのに返答できず申しわけありません！

パラメーターを渡してみたのですが、同様のエラーになってしまいます。
ハッシュの形を合わせたらいいのかなと思い変なコードですが
params: { session: { login_id: 'id', password: 'pass' } }
ってしてみたらエラーは変わりましたがやはりダメでした。

Mockの書き方がおかしいとかもあるのでしょうか？

よろしくお願いします！

[cameluby]

いえいえ！返事は全然大丈夫ですよ！

変わったエラーは、どのようなエラーでしたか？
もし良かったら、質問に追記していただけたら何か分かるかもです！
paramsのところでエラーになっているなら、mockでは無いと思うのですが…

[takeo7]

ありがとうございます！

いろいろいじっていたら
params: { session: { login_id: 'id', password: 'pass' } }
これでいけました！
昨日の時点のものとどこが変わったのかよくわかりませんが...w

ご丁寧な回答ありがとうございました！