cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

Refactor: make components testable #392

Closed hrk091 closed 1 year ago

hrk091 commented 1 year ago

最近機能追加したTrackSelectorを題材に、testableな構成への変更を実施しました。

主に、以下の変更が入っています。

各コンポーネントの状態が大幅に減り、可読性が上がり、テスト容易性が上がっているかと思いますが、そのあたりの実装や方針の是非についてレビュー・コメントいただけると嬉しいです。

testableにするための方針について合意できたら、この方針で他のコンポーネントもリファクタリングを進めていこうと思います。

github-actions[bot] commented 1 year ago

Review app

takaishi commented 1 year ago

方針の理解などで時間がかかりそうなので、レビューはリハ後とさせてください 🙇

hrk091 commented 1 year ago

Track.tsxの責務の分離が終わって、あとはchatとtrailMapだけになりました。 リハで結合検証しないと不安な箇所の修正は終わり、残りはリハ後でも配信への影響なく修正できるところだけです。

IVSとMediaLiveを建てて検証したら、一応ちゃんと配信ができることは確認できました。 このPRをmergeした状態でリハの検証をさせていただきたく、暫定でapprove & mergeしていただけると助かります(リハ後に一旦revertして、その後再度PRを出すでも構いません)