Session/9

[mqkotoo]

課題

close #10

対応箇所

• yumemi_weather API の呼び出しから Widget へ通知する部分までの Unit tests を書く
  • 以下のクラスのunitテストを実装しました
  • WeatherService
  • WeatherRequest
  • WeatherForecast
  • WeatherStateNotifier
• 依存しているものが、成功・失敗するケースも網羅する
• 余力があれば、JSON のエンコード・デコードの Unit tests も書く
  • jsonEncode: WeatherRequest
  • jsonDecode: WeatherForecast

動作

動作変更なし

[github-actions]

Ready for review 🚀

[mqkotoo]

[memo]
json.encode()とjsonEncode()等の違い

jsonは変数名と被ることが多いからjsonEncode が用意されているみたい
jsonEncode内でjson.encodeを呼び出している

ref:
flutter - What is the difference and preferred way between jsonEncode() and json.encode() in Dart? - Stack Overflow

https://api.flutter.dev/flutter/dart-convert/jsonEncode.html#:~:text=Shorthand%20for%20json.encode.%20Useful%20if%20a%20local%20variable%20shadows%20the%20global%20json%20constant.

[Yamasaki-pan961]

一旦レビューしました。
テストの実装に関してはテストの基本ができており、さまざまなテストケースを考慮できているので非常に良いです！

一つ気になったのは、テストのdescriptionで"case"というワードがよく使われていることです。
個人的には文章的な説明をするように心がけているのでコメントのように書くことが多いです。

（間違ってコメントなしでレビューを送ってしましました）

[Yamasaki-pan961]

1点だけ指摘していますが問題ないです。ご対応ありがとうございます。