search

cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

test: Add tests of TalkSelector/TalkInfo/Chat/hooks/selectors/actions #402

Closed hrk091 closed 1 year ago

hrk091 commented 1 year ago

TrailMap関連以外について、一通りテストを書きました。 backend callのmockingは、mswを使用しています。

Chat以外は、関数レベルでテストを書きました。 Chatも細かな粒度でテストを書きたかったのですが、各componentごとにテストを書くのがしんどかったのと、Chatのtop levelのシグネチャもシンプルでテストしやすい形になっていたので、Chatコンポーネントの仕様化テストとして実装しています。最低限のテストとして、チャットのCRUDの各ケースを実装しました。

また、前回のPRで「snapshot testは意味ないから消しました」としましたが、DOMレベルの精度での詳細確認はいらないとしてもmodel側のリファクタリングをしているときにviewに影響が出ていないことを確認するうえではやはり有用だと思い直し、snapshotテストを復活させています。

github-actions[bot] commented 1 year ago

Review app

hrk091 commented 1 year ago

テストを書いててnon-null-assertionが使えないのが煩わしすぎるので、無効化しました。 （個人的には、non-null-assertionは、本当に使っていいシーンなら使っていい派です。objectionあればコメントください）