RSpec バリデーションテスト

#実現したいこと
親モデルのuserは子モデルhabitを複数登録できる 1対多の関係です。
そこでuserはhabitモデルを6つまでしか登録できないという独自のバリデーションを加え,そのテストを書いています。

先にhabitモデルを6つ生成して
create_list(:habit, 6, user: user)
7つ目を登録しようとするとfalseになる
expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey

と言うふうにしたいのですが7つ目もtrueとなってしまいます。
consoleで確認するとちゃんとバリデーションエラーになります。

わかる方いましたらぜひ知恵を貸して頂きたいです。
よろしくお願い致しますm(__)m

#エラーメッセージ

Failure/Error: expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey
     
       expected: falsey value
            got: true

habit.rb

class Habit < ApplicationRecord
belongs_to :user

validates :task, presence: true, length: { maximum: 12 }, uniqueness: { scope: :user }
validates :frequency, presence: true
validate :user_habits_size_validate

 HABIT_MAX = 6
def user_habits_size_validate
    if self.user && self.user.habits.size >=  HABIT_MAX
        errors.add(:task,"は#{HABIT_MAX}つまでしか登録出来ません")
    end
end

end

habits.rb(FactroyBot)

FactoryBot.define do
  factory :habit do
    sequence(:task) { |n| "ランニング#{n}"}
    frequency { 3 }
    user
  end
end

habit_spec.rb

require 'rails_helper'

RSpec.describe Habit, type: :model do
  let(:user){ create(:user) }
  let(:habit)  { create(:habit) }
  # given!(:habits) { 6.time.collect { |n| create(:habit, task: "ランニング#{n}")}}


  describe 'バリデーション' do
    it 'taskが空の場合は保存できない' do
        habit.task = ""
        expect(habit).to_not be_valid
    end
    it 'taskが12文字以上の場合保存できない' do
        habit.task = "a" * 13
        expect(habit).to_not be_valid
    end
    it 'userは同じtaskカラムを持つhabitモデルを保存できない' do
        expect(build(:habit, task: "ランニング", user: user).save).to be_truthy
        expect(build(:habit, task: "ランニング", user: user).save).to be_falsey
    end
    it 'userはhabitモデルを6つ以上持てない' do     #エラーになるテスト
        create_list(:habit, 6, user: user)
        expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey
    end
    it "frequencyがnilの場合は保存できない" do
        habit.frequency = nil
        expect(habit).to_not be_valid
    end
  end
end

habitを6つ登録するところまではできてると思います。

 22:     it 'userはhabitモデルを6つ以上持てない' do
    23:         habits = create_list(:habit, 6, user: user)
 => 24:         binding.pry
    25:         expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey
    26:     end
    27:     it "frequencyがnilの場合は保存できない" do
    28:         habit.frequency = nil
    29:         expect(habit).to_not be_valid

[1] pry(#<RSpec::ExampleGroups::Habit::Nested>)> habits
=> [#<Habit:0x000055ede6be1f20
  id: 1,
  task: "ランニング3",
  frequency: 3,
  user_id: 1,
  created_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  updated_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  end_time: nil,
  complete: 0>,
 #<Habit:0x000055ede6beece8
  id: 2,
  task: "ランニング4",
  frequency: 3,
  user_id: 1,
  created_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  updated_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  end_time: nil,
  complete: 0>,
 #<Habit:0x000055ede6c0a498
  id: 3,
  task: "ランニング5",
  frequency: 3,
  user_id: 1,
  created_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  updated_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  end_time: nil,
  complete: 0>,
 #<Habit:0x000055ede6bfdc70
  id: 4,
  task: "ランニング6",
  frequency: 3,
  user_id: 1,
  created_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  updated_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  end_time: nil,
  complete: 0>,
 #<Habit:0x000055ede6c11428
  id: 5,
  task: "ランニング7",
  frequency: 3,
  user_id: 1,
  created_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  updated_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  end_time: nil,
  complete: 0>,
 #<Habit:0x000055ede6c1cbc0
  id: 6,
  task: "ランニング8",
  frequency: 3,
  user_id: 1,
  created_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  updated_at: Sun, 06 Dec 2020 23:40:44 JST +09:00,
  end_time: nil,
  complete: 0>]

#バージョン
Rails 6.0.3.4
ruby 2.6.3p62
RSpec 3.10

行動規範の内容に同意します

回答2件

今更ですが、たぶんキャッシュが効いてます。

ruby
1user = User.create
2user.habits.size # => 0
3
43.times { Habit.create(user: user) }
5
6# キャッシュを参照してる
7user.habits.size # => 0
8
9# count は常にSQLを発行して確認する
10user.habits.count # => 3

ruby
1user = User.create
2habit = Habit.create(user: user)
3
4# キャッシュする
5habit.user.habits
6
73.times { Habit.create(user: user) }
8
9habit.user.habits.size #=> 1
10habit.user.habits.count # => 4
11
12habit.user.habits.size #=> 1
13habit.user.reload.habits.size #=> 4

ruby
1user = User.create
2user.habits.size # => 0
3
4user.habits.create
5user.habits.size # => 1
6
73.times { Habit.create(user: user) }
8user.habits.size # => 1
9
10user.habits.create
11# createはキャッシュに 1 プラスしてるだけ
12user.habits.size # => 2
13
14user.habits.count # => 5

create_list(:habit, 6, user: user)の際にHabitのvalidateが走り、
self.user.habitsがキャッシュされ、self.user.habits.size が常に 0 を返してるのかも。

投稿2020/12/07 07:52

編集2020/12/07 08:04

neko_daisuki

総合スコア2090

hanayama

2020/12/07 09:06

neko_daisuki様お世話になっておりますm(__)m sizeとcountでデータの取得の仕方が違うんですね！このように書き換えてみたらうまくテストが通りました！ `haibt_spec.rb` habits = create_list(:habit, 6, user: user) expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey `habit.rb` self.user.habits.reload.size >= HABIT_MAX userやhabitモデルは正常にデータベースに保存されていたけどhabit.rbの self.user.habits.size >= HABIT_MAX の部分がデータベースではなくキャッシュを参照しているので常にtrueになってしまう。 reload.size or count とすれば生成されてるデータベースを正しくカウントしてくれる。という解釈でよろしいでしょうか？

neko_daisuki

2020/12/07 09:17

false ですが、その通りだと思います。僕も以前同じ問題でハマったのですが、キャッシュなんか普段意識しないので分かりにくいですよね。

hanayama

2020/12/07 09:26

すいませんfalseですね！そうですね　指摘頂けてなかったらこの答えには辿り着いてないと思います。質問できる人がいないので本当に助かってます。連日お世話になってますね笑いつも本当にありがとうございますm(__)m

行動規範の内容に同意します

ベストアンサー

create_listしただけでは、データベースに保存されないので、別に作ったuserからhabitを回収できない、ということではないでしょうか？

投稿2020/12/06 23:26

maisumakun

総合スコア145975

hanayama

2020/12/07 03:43

ご回答ありがとうござます！！ create_listしただけではデータベースに保存されないんですね。 create_listでデータを保存できてないので expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey が　１つ目のデータとして認識され、trueになってしまうと言うことでしょうか？解釈を間違えていたらすみませんm(__)m

hanayama

2020/12/07 05:31

回収できないというのはuserからhabitを取得できないという意味でしょうか？確かにuser.habits => [] となってしまいました。

maisumakun

2020/12/07 05:32

> expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey が　１つ目のデータとして認識され、trueになってしまうと言うことでしょうか？そういうことです。

hanayama

2020/12/07 06:18

ご返信ありがとうございます！データベースに保存できてないという事でこのように書き換えてみました habits = create_list(:habit, 6, user: user) habits.map(&:save) expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey ですがやはり同じエラーになってしまいます。これでもデータベースに保存されてないのでしょうか？試しにhabitを6から10に変えて確認すると habits.map(&:save) => [true, true, true, true, true, true, true, true, true, true] と全てtrueになってしまいます。質問ばかりですみません。よろしければご返答お待ちしておりますm(__)m

maisumakun

2020/12/07 06:19

userもデータベースに保存していますか？

hanayama

2020/12/07 07:07

habitモデルにuser_idを付けているのでuserも保存されているものと思い込んでおりました。生成できてたのはhabitモデルだけでuser_idのuserが保存できていなかったのでうまくいかなかったという事ですね。綺麗なコードではないと思いますが一応 6.times do |n| user.habits.create(task: "ランニング#{n}", frequency: 3) end expect(build(:habit, task: "筋トレ", user: user).save).to be_falsey とすることでテストが通りました！！ FactroyBotでhas_manyの関係性をうまく書けるよう学んでみたいと思います。何度も質問に答えて頂き、本当にありがとうございました！！m(__)m

行動規範の内容に同意します

あなたの回答