search

hiroaki514 / contents_management_app

0 stars 0 forks source link

組織削除ログのテスト追加 #205

Closed hiroaki514 closed 11 months ago

hiroaki514 commented 12 months ago

PRの目的

重点的に見て欲しいポイント

対象のissues

備考

hiroaki514 commented 12 months ago

@yoshichiba 

organizations_spec.rb
・論理削除がされること
・論理削除された組織は一覧に表示されないこと

上記ですが、削除されることのテストで表示も確認をしているのに気づきました、１まとめという事でも大丈夫でしょうか？

またログのテストですが、beforeで削除を実行させて表示を確認する形で記述したのですが、合っていますでしょうか。 よろしくお願いします！

yoshichiba commented 12 months ago

@hiroaki514

１まとめという事でも大丈夫でしょうか？

こちらおっしゃる通りですね！OKです！

またログのテストですが、beforeで削除を実行させて表示を確認する形で記述したのですが、合っていますでしょうか。

こちらも問題なしです！

hiroaki514 commented 12 months ago

@yoshichiba ありがとうございます、確認よろしくお願いします！