Closed maixhashi closed 3 days ago
このプルリクエストにて何をしたのか? Notificationモデルのモデルスペックテストを実行
このプルリクエストにて何をしたのか?
class Notification < ApplicationRecord default_scope -> { order(created_at: :desc) } belongs_to :shuffled_overview, optional: true belongs_to :movie, optional: true # belongs_to :comment, optional: true belongs_to :visitor, class_name: 'User', foreign_key: 'visitor_id' belongs_to :visited, class_name: 'User', foreign_key: 'visited_id' end
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要) なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
どの環境でどんな動作チェックをしたか 動作確認をした事についてスクショなどがあるとわかりやすくて良い
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
下記コマンドでテスト実行、テストパスを確認
docker-compose run web bundle exec rspec spec/models/notification_spec.rb
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
GitHub Issue Ticket
やった事
このプルリクエストにて何をしたのか?
Notificationモデルのモデルスペックテストを実行なぜやるのか
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
動作確認
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
下記コマンドでテスト実行、テストパスを確認
Refs (レビューにあたって参考にすべき情報)(Optional)
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報