search

raisetech-for-student / banana-crud-restfulapi

0 stars 1 forks source link

論理削除を追加 #28

Closed renangton closed 2 years ago

renangton commented 2 years ago

概要

削除機能の追加およびテストの作成

論理削除機能及び論理削除の単体/結合テストを追加しました。

SpotBugsの修正

下記サイトを参考にし、一部警告を除外するように設定しました。 https://www.monotalk.xyz/blog/lombok%E3%81%AEfindbugs%E8%AD%A6%E5%91%8A%E3%82%92%E6%8A%91%E5%88%B6%E3%81%99%E3%82%8B/

renangton commented 2 years ago

論理削除成功時

banana_delete_succes

論理削除失敗時（指定したIDが存在しない時）

banana_delete_notfound

yoshi-koyama commented 2 years ago

結合テストも作っておきましょう。 Controller -> Service -> Mapperまで一気通貫で動かして問題ないことを確認するためのテストがほしいです。 https://onedrive.live.com/?authkey=%21ABDVqdvuCWnIC1M&cid=D2DFE9880240895A&id=D2DFE9880240895A%2132420&parId=D2DFE9880240895A%2130567&o=OneUp

Jsonの確認はassertThatJsonを使えるようにしてもいいです。 （assertThatJsonのほうが実際の値と期待値の検証順がAssertJと同じなので全体的な統一感を大事にするならassertThatJsonのほうがいいかと思います）