search

kufu / smarthr-ui

React components for creating SmartHR applications.
https://story.smarthr-ui.dev
MIT License
880 stars 141 forks source link

chore: RSCのテスト用ページで、コンポーネントをレンダリングするところまで検証するように修正 #4997

Closed s-sasaki-0529 closed 1 month ago

s-sasaki-0529 commented 1 month ago

関連URL

前回PR

概要

前回PRにて、コンポーネントがRSCレンダリング可能かを検証するページを用意してE2Eで担保するようにしたが、検証方法が不十分で判定が正しく出来ていなかったことがわかったため、判定方法を見直す。

変更内容

変更前は、コンポーネントを import して、コンソール出力するだけだった。

これだとモジュールのトップレベルで useContext などの依存が発生しているコンポーネントの場合はその時点でエラーとなるが、コンポーネントをレンダリングするタイミングで初めて useState や useRef に依存する場合のコンポーネントを検知できていなかった。

そこで、本PRではすべてのコンポーネントについて、最小限のレンダリングを行うように修正して、E2Eテストの期待値も見直した。

確認方法

CIが通ればOKです。ローカルで検証する場合は前回PRを参照してください。

pkg-pr-new[bot] commented 1 month ago

Open in Stackblitz

pnpm add https://pkg.pr.new/kufu/smarthr-ui@4997

commit: d9c5c4e