system_specにて、Couldn't find User with 'id'エラーが出る

Question

前提

プロフィール作成フォームにて、user_nameが空欄で入力されたときに、
バリデーションがかかり、「user_nameが空欄の時、登録できないこと」のテストが通るようにしたい
のテストを実装したいがタイトルの通り、Couldn't find User with 'id'エラーが発生しテストが通りません。

実現したいこと

「user_nameが空欄の時、登録できないこと」のテストが通るようにしたい

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

Failure/Error: expect(user.reload.user_name).to be_invalid
     
     ActiveRecord::RecordNotFound:
       Couldn't find User with 'id'=9
     # /usr/local/bundle/gems/activestorage-7.0.4/lib/active_storage/attached/model.rb:237:in `reload'
     # ./spec/models/user_spec.rb:33:in `block (3 levels) in <main>'

該当のソースコード

require 'rails_helper'

RSpec.describe User, type: :model do
  describe 'ユーザー登録' do
    it 'email,passwordが存在すれば登録出来ること' do
      user = build(:user)
      expect(user).to be_valid
    end
    
    it 'emailが空欄の時、登録できないこと' do
      user = build(:user)
      user.email = ''
      expect(user).to be_invalid
    end
    
    it 'passwordが空欄の時、登録できないこと' do
      user = build(:user)
      user.password = ''
      expect(user).to be_invalid
    end
    
  end

  describe 'プロフィール登録' do
    it 'user_nameが存在すれば登録出来ること' do
      user = build(:user)
      expect(user).to be_valid
    end

    it 'user_nameが空欄の時、登録できないこと' do    <-----------問題のコード
      user = build(:user)
      user.user_name = ''
      expect(user.reload.user_name).to be_invalid
    end
  end
end

FactoryBot.define do
  factory :user do
    sequence(:id)      { |n| n}
    sequence(:email)   { |n| "person#{n}@example.com" }
    password           { "123456" }
    user_name          { "test" }
    self_introduce     { "test" }
  end
end

class HomeController < ApplicationController
  def top
    @q = User.ransack(params[:q])
    @services = Service.all
  end

  def profile               
    @user = current_user
  end

  def update
    @user = current_user
    if @user.update(params.permit(:profile_image, :user_name, :self_introduce, :twitter_url, :youtube_url, :instagram_url))
      flash[:notice] = "Profile was successfully updated."
      redirect_to "/"
    else
      render 'profile', status: :unprocessable_entity
    end
  end

  def show
    @user = User.find(params[:id])
    @user_movies = @user.movies
    @services = @user.services
  end

  def search
    @q = User.ransack(params[:q])
    @results = @q.result
  end
end

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise  :database_authenticatable, :registerable,
          :recoverable, :rememberable, :validatable

  has_one_attached :profile_image

  has_many :services, dependent: :destroy
  has_many :movies, dependent: :destroy

#プロフィール画面でのみバリデーションがかかるように
  validates :user_name, presence: true, on: :update
end

補足

rails 7
ruby3.1.1

よろしくお願いします。

Answer 1

ほかは
expect(user).to be_valid
なのに、なぜここだけ
expect(user.reload.user_name).to be_invalid
なのですか？
expect(user).to be_invalid
にしてみて。

追記
user.reload.user_name とすれば、UPDATEの時として validateしてくれるか？ というのも問題ですが、おかしい所が他にも２点あります。

1. user.reload は userをもう一度DBから読み込みなおす ということです。

　しかしこの時点では userはDBにはありません。buildしただけで save していませんから。 なので、ActiveRecord::RecordNotFound です
2. user.reload がうまくいったとして、expect(user.reload.user_name).to be_invalid は user が .to be_invalid かどうか、は見ていません。user_name が be_imvalidかどうかを見ています。
つまり、class Nill か class Stringにたいしてみています。 これは エラーになりそうですね

Comments

[.taniyan]

ご回答ありがとうございます。
書き忘れておりましたが、userモデルにてupdateメソッドでのみバリデーションがかかるようにしております。
（ユーザーの新規登録時にバリデーションがかかるのを防ぐため。）

models/user.rb
class User < ApplicationRecord
# Include default devise modules. Others available are:
# :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
devise :database_authenticatable, :registerable,
:recoverable, :rememberable, :validatable

has_one_attached :profile_image

has_many :services, dependent: :destroy
has_many :movies, dependent: :destroy

validates :user_name, presence: true, on: :update
end

ですのでupdate時にバリデーションがかかるようにと思い、
expect(user.reload.user_name).to be_invalid
とさせていただきました。

[winterboum]

そうすると 「update時にバリデーションがかかる」状態になるのですか？

[.taniyan]

＞そうすると 「update時にバリデーションがかかる」状態になるのですか？

validates :user_name, presence: true, on: :update

はい、これによってプロフィール画面でのupdateメソッドでのみ、バリデーションがかかる状態になっています。

[winterboum]

その validtes の code がUPDATEの時に動く、というのはわかります。
聞きたいのは、user.reload.user_name とすると .to be_invalid が 「これは UPDATEのときだ」と判断してくれる、のですか？
という質問です

[.taniyan]

言われてみればおっしゃる通りです。今回のエラーについて新しく質問させていただいたので、よろしければそちらも確認していただけると幸いです。