質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
87.20%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

RSpec

RSpecはRuby用のBDD(behaviour-driven development)フレームワークです。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

解決済

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

IRIESS
IRIESS

総合スコア45

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

RSpec

RSpecはRuby用のBDD(behaviour-driven development)フレームワークです。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

2回答

0評価

0クリップ

464閲覧

投稿2021/11/28 12:48

編集2021/11/29 07:16

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に書き換えたほうがいいでしょうか?
ベストプラクティスをご教授お願い致します。

ruby

# 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

ruby

8) Agreement start_time, finish_time 1時間なら有効 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)>'

ruby

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

ruby

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

ruby

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

そして

ruby

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 に修正しました

良い質問の評価を上げる

以下のような質問は評価を上げましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

気になる質問をクリップする

クリップした質問は、後からいつでもマイページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

  • プログラミングに関係のない質問
  • やってほしいことだけを記載した丸投げの質問
  • 問題・課題が含まれていない質問
  • 意図的に内容が抹消された質問
  • 過去に投稿した質問と同じ内容の質問
  • 広告と受け取られるような投稿

評価を下げると、トップページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

まだ回答がついていません

会員登録して回答してみよう

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
87.20%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問

同じタグがついた質問を見る

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

RSpec

RSpecはRuby用のBDD(behaviour-driven development)フレームワークです。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。