Q&A
メソッドの中で何をやってるかをもとに「通るテスト」を書くのではなく、そのメソッドに求められる仕様をもとに「仕様を満たしていることを保証するためのテスト」を書きます。
何をするメソッドなのか、ドキュメントを読んでください。
今までテストがなかったコードにJestでユニットテストを追加したいです。ただ、メソッドの中で何をやっているのかがわからなかったり外部のライブラリを使っているのかそれとも作成した変数なのかわからなかったりして何から手をつけていけばわかりません。私の今のアプローチとしてはreturnしている箇所に注目し、その値が何であるかを確かめてからテストコードをそれに合わせて書いていますがこのアプローチの仕方で合っているのかがわかりません。1足す1は2であることを期待するみたいな簡単なコードであればテストケースも簡単ですが、実際のコードは複雑でどうしていいかわかりません。ちなみにリファクタリングはテストコードを書いてからしようと思っているのですぐに複雑なコードをリファクタリングしてからテストを書くというのはバグを含めそうなのでやっていません。以上のような場合、他人が書いたコードにテストを追加する際のアプローチ方法を教えていただけますと幸いです。
> ちなみにリファクタリングはテストコードを書いてからしようと思っているのですぐに複雑なコードをリファクタリングしてからテストを書くというのはバグを含めそうなのでやっていません。
元々の作りがリファクタリングしないとユニットテストと言えるテストが出来ないような作りをしているのではないですか？
「その仕事振った上司に聞け」ってのが第一で、あとはすでに指摘されているとおり。
ドキュメントがないならまず目の前のコード読んで仕様を起こすところからやる必要があるけど、それができないなら上司に「無理です」って言おう。
なるほど、やはりドキュメント等ないと厳しいのですね。それがあるかどうか聞いてみて無ければGitHubやタスク管理ツールなどをみてみます。リファクタリングしなければならない箇所もありますが、全てがスパゲッティではないのでできるところから手をつけていきたいと思います。
確かにアプローチの仕方が違っていたのでそこがクリアになってよかったです。
皆さんのコメントが役に立ちましたので決めづらいのですが、ここはいちばん早くに回答いただいたZuishinさんにベストアンサーをつけさせていただきたいのでよろしければご回答いただけますと幸いです。
質問の意図がよくわからなかったので回答していません。
解決したのであれば、今後この質問を見る他の人の参考になるような回答を自分でしてそれをベストアンサーに選んでください。
みなさん、ありがとうございました！
回答1件
0
自己解決
ユニットテストを書く際は
- ドキュメントをしっかり読み何をテストすべきなのかを把握する。
- リファクタリングが必要な際は先にリファクタリングをし、テストがしやすいコードにする。
- ドキュメントがないのであればコードをみて作成する。 それができなければ厳しいのでそのことを報告する（こういった状況を作らない）
投稿2021/11/25 10:00
総合スコア24
> リファクタリングが必要な際は先にリファクタリングをし、テストがしやすいコードにする。
言葉が足りないだけなのかもしれませんが、テストは、リファクタリングする前と後の結果が同じことを確認、担保する為に書くので、リファクタリングする前にテストを書ける項目があるのであれば、先にテストを書かなければいけないのではないでしょうか。リファクタリングすることによって追加できるテストが増えるのであれば、それは後で良いのでしょうけど。
そうですね！質問ではリファクタリングより先にテストコードを書きたいと言っていたのですが、回答ではそこが言葉足らずになってしまいました。ご指摘ありがとうございます????♂️
自分のコメントのせいですね
> 元々の作りがリファクタリングしないとユニットテストと言えるテストが出来ないような作りをしているのではないですか？
密結合なクソコードではユニットテスト出来ないでしょって意味です。結合テスト並のものをユニットテストと言ってるなら知りませんけど。
テストを書くためにリファクタリングしたらどうかという意味ではありません。
なるほど、私が読み誤っていた様です。良く分かりました。レスをいただき、どうもありがとうございます。
あなたの回答
tips
プレビュー
