Rspec validateの中でparameterを使用しているときのモデルspecについて

Question

Rails APIを使用してアプリを作成しています。

現在下記のようなagreementモデルがあります。
roomで話し合い、userとhostが合意したらagreementを作成できます。
agreementのupdate機能を追加した際に、自分自身に対しても時間帯(start_timeとfinish_time)の重複チェックをしてしまうため、room_idからroomを特定しroom.agreementがnilでない場合にroom.agreement.idを重複チェックの要素に入れ、自分自身を除いた重複チェックができるようにしました。

しかしモデルspecでテストを実行した際に、room = Room.find(room_id)の部分でidがないと怒られました。

パラメータを渡してないからだと思うのですが、ではどのようにしてモデルテストを記述したら良いのか分かりません。。。。。
リクエストspecに書き換えたほうがいいでしょうか？
ベストプラクティスをご教授お願い致します。

# agreement.rb

  belongs_to :user
  belongs_to :host
  belongs_to :room
validate :duplication_of_work_hours_for_same_user

def duplication_of_work_hours_for_same_user
    room = Room.find(room_id)
    if room.agreement.nil?
      Agreement.where('finish_time >= ? && ? >= start_time && user_id = ?', start_time, finish_time, user_id).exists?
      errors.add(:start_time, "userの勤務期間が他の契約と重複していますneeee。")
    else
      agreement = room.agreement
      if Agreement.where('id != ? && finish_time >= ? && ? >= start_time && user_id = ?', agreement.id, start_time, finish_time, user_id).exists?
        errors.add(:start_time, "userの勤務期間が他の契約と重複しています。")
      end
    end
  end

8) Agreement start_time, finish_time １時間なら有効
     Failure/Error: room = Room.find(room_id)
     
     ActiveRecord::RecordNotFound:
       Couldn't find Room without an ID
     # ./app/models/agreement.rb:33:in `duplication_of_work_hours_for_same_user'
     # ./spec/models/agreement_spec.rb:58:in `block (3 levels) in <top (required)>'

it "１時間ちょうどなら有効" do
      agreement = build(:agreement, finish_time: Time.current + 25.hour)
      expect(agreement).to be_valid
    end

FactoryBot.define do
  factory :agreement do
    association :user
    association :host
    association :room
    start_time { Time.current + 24.hour }
    finish_time { Time.current + 32.hour }
    state { 0 }
  end
end

#
このコードで解決しました。
it "ちょうど１時間なら有効" do
      agreement = build(:agreement, room: room, user: room.user, host: room.host, finish_time: Time.current + 25.hour)
      expect(agreement).to be_valid
    end

そして

if room.agreement.nil?
      Agreement.where('finish_time >= ? && ? >= start_time && user_id = ?', start_time, finish_time, user_id).exists?
      errors.add(:start_time, "userの勤務期間が他の契約と重複していますneeee。")
    else

ここにifをつけ忘れていたこともややこしくしていました泣泣
正しくは

if room.agreement.nil?
  if Agreement.where('finish_time >= ? && ? >= start_time && user_id = ?', start_time, finish_time, user_id).exists?
      errors.add(:start_time, "userの勤務期間が他の契約と重複していますneeee。")
  end
    else

に修正しました

Answer 1

おそらく、build した agreement の room が保存されていません。
FactoryBot でbuildしたとき、関連先がcreateされなくなっていた（デフォルトの挙動として）

記事の通りにするか、strategy に create を指定する、または以下のようにしてみてください。

agreement = build(:agreement, finish_time: Time.current + 25.hour)
agreement.room.save

expect(agreement).to be_valid

Comments

[IRIESS]

回答ありがとうございます！
指定の通りにテストを記述してみたら、room_idがやはり渡されませんでした。今回は実装ではroomは先に存在した上でagreementが作られる実装にしているので、先にroomを作るようにテストを記述したところ、テストが通るようになりました。解決したテストを追記しました。
ご回答ありがとうございました！

Answer 2

パラメータを渡してないからだと思うのですが

いえ、パラメーターの問題ではありません。エラーメッセージのとおり、このモデルへroom_idをセットしない場合、本当に例外を発生させてしまいます。

room_idがない場合はどのような動作をさせたいかを考えて、それに合わせた実装・テストを書く必要があります。

Comments

[maisumakun]

roomとagreementを同時作成しようとした場合には、「roomからは取れるけど、DBには未保存なのでroom_idはnil」となることが考えられます。このような場合まで考えるなら、room_idからRoomを取ること自体が妥当ではないかもしれません。

[IRIESS]

なるほど、ファクトリーボッドのアソシエーション任せにすると、roomとagreementが同時作成されるからroom_idはnilになってしまうということですか。
実際にagreementが作成される場合にはroomから作るため必ずroomは存在するので、実装は変えずに、テストを追記のような形で記入することでとりあえず解決できました。テストは長くなってしまいましたが。
今回もご回答ありがとうございました。