Skip to content

kadai2-micchie#17

Open
mi-bear wants to merge 8 commits into
masterfrom
kadai2-micchie
Open

kadai2-micchie#17
mi-bear wants to merge 8 commits into
masterfrom
kadai2-micchie

Conversation

@mi-bear

@mi-bear mi-bear commented Jul 30, 2019

Copy link
Copy Markdown
Member

Overview

  • io.Reader と io.Writer が標準パッケージでどのように使われているか (README.md)
  • io.Reader と io.Writer があることでどういう利点があるのか具体例を挙げて考えてみる (README.md)
  • テストのしやすさを考えてリファクタリングしてみる
  • テストのカバレッジを取ってみる
  • テーブル駆動テストを行う
  • テストヘルパーを作ってみる

Command

  -after string
    	image format after conversion (default "png")
  -before string
    	image format before conversion (default "jpg")
  -d string
    	path of images to convert [required]

Cover

kadai2-cover

分からないところ:

  • filepath.Walk の部分はどんな風にテストを書けば良いのか。
  • err のテストの分離。
  • テストヘルパーはどんなときに使うと良いものなのか。

@mi-bear mi-bear added the kadai2 label Jul 30, 2019
@mi-bear mi-bear requested a review from tenntenn July 30, 2019 06:05
@mi-bear mi-bear self-assigned this Jul 30, 2019
@mi-bear

mi-bear commented Jul 31, 2019

Copy link
Copy Markdown
Member Author

TODO:

  • ヘルパーを書く
  • t.Parallel() を入れる
  • テストパッケージにする
  • サブテストを使う
  • interface{} の使い所がありそう

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant