Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UnitTest課題が、結合テストと勘違いしそう #63

Open
novr opened this issue Jan 15, 2024 · 3 comments
Open

UnitTest課題が、結合テストと勘違いしそう #63

novr opened this issue Jan 15, 2024 · 3 comments

Comments

@novr
Copy link
Member

novr commented Jan 15, 2024

https://github.com/yumemi-inc/ios-training/blob/main/Documentation/UnitTest.md について

テストコードを書くためにリファクタリングする部分 の例が
Modelについて書かれているためViewのテストをすべきところ、View+Modelの結合テストを想定してしまう
yumemi-inc/ios-training-bjung#8 (comment)

ViewのUnitTestであることを明記すべきか?

@el-hoshino
Copy link
Member

確かに、Testって言われてもさまざまなテストがあるから非常に曖昧ですので、UnitTestって明記した方がいいと思いました 🙋

@macneko-ayu
Copy link

これ、どういうコードを書いたら正解なのかを私も理解できていなくて、OJTで悩んでたんですが、ここで聞いちゃうのが早そう。
「スタブなりで用意したModelをVCに渡して、UIImageViewに晴れ画像が表示されること」がゴールであって、「ロジックを通して生成されたModelをVCに渡して、UIImageViewに晴れ画像が表示されること」は求めていない、という認識であってますか…?

@novr
Copy link
Member Author

novr commented Sep 24, 2024

画面に晴れ画像が表示されること を確認しようとするとUITestになってしまうので
文言も揃えてこんな感じかな

  • 天気予報がxxxだったら、画面にxxx画像が設定されること
  • 天気予報の最高気温が画面に設定されること

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants