RSpec バリデーションエラー

Question

#実現したいこと
親モデルのuserがhabitという子モデルを複数持っています。
habitにはtaskというカラムがあり、userは、habitモデルは複数持てるが、taskカラムの重複は許さないというようなテストを書いています。

そこでFactroyBotで同じtaskカラムを持つhabitモデルを2つ用意し、1つ目は保存が成功するが2つ目は失敗するというテストを書きたいのですが同じuserを２回作成しようとしてしまうためかメールアドレスは既に存在しますというuserのバリデーションエラーになってしまいます。

自分なりに調べてみましたが、上手くいくやり方が中々見つかりませんでした。

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

#エラーメッセージ

1) Habit バリデーション userは同じtaskカラムを持つhabitモデルを保存できない
     Failure/Error: let(:habit2) { create(:habit2) }
     
     ActiveRecord::RecordInvalid:
       バリデーションに失敗しました: メールアドレスはすでに存在します

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

FactoryBot

FactoryBot.define do
  factory :habit do
    task {"ランニング"}
    frequency { 3 }
    user
  end
  factory :habit2, class: Habit do
    task {"ランニング"}
    frequency { 2 }
    user
  end
end

habit_spec.rb

require 'rails_helper'

RSpec.describe Habit, type: :model do
  let(:habit)  { create(:habit) }
  let(:habit2) { create(:habit2) }
  
  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(habit.save).to be_truthy
        expect(habit2.save).to be_falsey
    end
    it "frequencyがnilの場合は保存できない" do
      habit.frequency = nil
      expect(habit).to_not be_valid
    end
  end
end

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

Answer 1

Userファクトリーのメールアドレスはsequenceで定義していますか？

factory :user do
  sequence(:email) { |n| "tama#{n}@cat.com" }
end

こうするとuserをbuild/createするたびにnの数が増えていき、
一意なメールアドレスのユーザーを作成できます。

create(:user).email # "[e-mail]"
create(:user).email # "[e-mail]"
create(:user).email # "[e-mail]"

factory_bot sequences

Comments

[hanayama]

neko_daisuki様またご回答頂きありがとうございますm(__)m
sequenceを使うと別のuserということになってしまい、`expect(habit2.save).to be_falsey`がエラーになってしまいませんか？
`user,habits.new(task: "ランニング")` => 保存成功
`user,habits.new(task: "ランニング")` => 保存失敗
`other_user.habits.new(task: "ランニング")` => 保存成功
と言うふうに書きたいのですがうまくいきません。
説明不足ですみませんm(__)m

[neko_daisuki]

なるほど。同じユーザーを指定するんですね。
fabricationだと以下のように書けます。

let(:user){ create(:user) }

expect(build(:habit, task: "ランニング", user: user).save).to be_truthy
expect(build(:habit, task: "ランニング", user: user).save).to be_falsey

factorybot でも同じようにできると思うんですが、もしダメな場合は
invalidのuser を valid の user にすればうまくいくと思います。

valid = build(:habit, task: "ランニング")
expect(valid.save).to be_truthy

invalid = build(:habit, task: "ランニング")
invalid.user = valid.user
expect(invalid.save).to be_falsey

[hanayama]

なるほど！！

expect(build(:habit, task: "ランニング", user: user).save).to be_truthy
expect(build(:habit, task: "ランニング", user: user).save).to be_falsey

で無事テスト通りました！
factorybotの方でももっとうまく書けるように勉強します！
neko_daisuki様本当にありがとうございました！！m(__)m