Skip to content

Test_2024.09.19.21.19_model_spec_test(BookmarkOfShuffledOverviewモデル)#190

Merged
maixhashi merged 1 commit intomasterfrom
Test_2024.09.19.21.19_model_spec_test(BookmarkOfShuffledOverviewモデル)_#189
Sep 19, 2024

Hidden character warning

The head ref may contain hidden characters: "Test_2024.09.19.21.19_model_spec_test\uff08BookmarkOfShuffledOverview\u30e2\u30c7\u30eb\uff09_#189"
Merged

Test_2024.09.19.21.19_model_spec_test(BookmarkOfShuffledOverviewモデル)#190
maixhashi merged 1 commit intomasterfrom
Test_2024.09.19.21.19_model_spec_test(BookmarkOfShuffledOverviewモデル)_#189

Conversation

@maixhashi
Copy link
Copy Markdown
Owner

GitHub Issue Ticket

やった事

このプルリクエストにて何をしたのか?
BookmarkOfShuffledOverviewモデルのモデルスペックテストの実施

class BookmarkOfShuffledOverview < ApplicationRecord
  belongs_to :user
  belongs_to :shuffled_overview

  validates :user_id, uniqueness: { scope: :shuffled_overview_id }
end

なぜやるのか

GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい

動作確認

どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
下記コマンドでテスト実施、テストパスを確認

 docker-compose run web bundle exec rspec spec/models/bookmark_of_shuffled_overview_spec.rb    

Refs (レビューにあたって参考にすべき情報)(Optional)

関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報

@maixhashi maixhashi merged commit c53e282 into master Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test_2024.09.19.21.19_model spec test(BookmarkOfShuffledOverviewモデル)_#189

1 participant