Workflow実行時のPipelineテスト #486
Conversation
tsuchiyama-araya
requested review from
emuemuJP,
itutu-tienday and
milesAraya
| } | ||
|
|
||
| // Helper to create a fulfilled action for a given action type and payload | ||
| const createFulfilledAction = ( |
There was a problem hiding this comment.
そもそもですが、このファイルのテストコードは、実際のコードのどの範囲をテスト（coverage）対象としたものでしょうか？
- 他の PilelineXXXX ファイルは、テストの対象外でしょうか？ （PipelineSlice.ts のみがテスト対象？）
- 上記が対象外の場合、その事由を確認させてください（文面では分かりづらそうなので、会話共有しましょう）
There was a problem hiding this comment.
こちらは資料でまとめますので、少々お待ちください。
完了したら、Slackの方で報告します。
There was a problem hiding this comment.
テストケース資料について、cover率が低いファイルについて（Pipeline(Actions|Hook|Selectors)）、事由をシートに補足解説しておいてください
There was a problem hiding this comment.
こちらはPipeline Hooksにコメントを追加しましたが、
他の完全ではないファイルにもメモを記載した方が良いでしょうか？
There was a problem hiding this comment.
cover率が低いファイルについて（Pipeline(Actions|Hook|Selectors)）、
例えば PipelineActions など、coverage が 11% 程度?（ほぼテスト対象外）となっていませんか？
→ ほぼテスト対象外なのであれば、解説を残しておくと良いと思われます。
|
@itutu-tienday なお、テストのカバレッジについては現在資料を作成中ですので、少々お時間をいただければと思います。 |
|
今回のテストCoverageを資料に追加しました。 また、Jest Coverageを追加しましたので、yarn test-coverage {filePath} を実行すれば、 お手数ですが、ご確認お願い致します。 |
| } | ||
|
|
||
| // Helper to create a fulfilled action for a given action type and payload | ||
| const createFulfilledAction = ( |
There was a problem hiding this comment.
cover率が低いファイルについて（Pipeline(Actions|Hook|Selectors)）、
例えば PipelineActions など、coverage が 11% 程度?（ほぼテスト対象外）となっていませんか？
→ ほぼテスト対象外なのであれば、解説を残しておくと良いと思われます。
| // Helper to create a fulfilled action for a given action type and payload | ||
| const createFulfilledAction = ( | ||
| // eslint-disable-next-line @typescript-eslint/no-explicit-any | ||
| actionType: AsyncThunk<any, any, any>, |
There was a problem hiding this comment.
FulfilledAction のみ、type 指定に generics を利用しているのは、どのような事由からでしょうか？
※pending や reject では、generics を使用せず 個別に ActionClass が定義されているように見えています。
【問題】
ワークフロースクリーンのテストは、コンポーネント（React Flow）とJestの互換性の問題により、まだ実装されていません。
【解決策】
ワークフローの状態管理に焦点を当てたテストの実装
ワークフロースクリーンのテストで重要なポイントは、ワークフローの実行状態をテストすることです。そのため、Reactコンポーネントに依存しないテスト方法を検討します。
【実装】
Reduxの状態管理や非同期アクションの動作をモックデータを用いて詳細に検証しています。特に、非同期アクションが成功/失敗した場合の状態遷移や、各プロパティが期待通りの値を持つことを確認することに重点を置いています。
関連リンク