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

[wip] Kadai2-fujiokayu #21

Open
wants to merge 12 commits into
base: kadai1-fujiokayu
Choose a base branch
from
Open

Conversation

fujiokayu
Copy link

@fujiokayu fujiokayu commented Jun 11, 2019

  • io.Readerとio.Writerについて調べてみよう

  • 1回目の宿題のテストを作ってみて下さい
    converter パッケージのみを対象に実施しました。

    • テストのしやすさを考えてリファクタリングしてみる
      • 趣旨は異なるのですが、テストカバレッジを高める中でエラーハンドリングが足りていない箇所を発見、修正しました。
    • テストのカバレッジを取ってみる
    • テーブル駆動テストを行う
    • テストヘルパーを作ってみる
    • go test -cover
PASS  
coverage: 96.6% of statements
ok      myConverter/converter   6.992s

カバレッジが低いとテスト自体を Fail にするような仕組みがあると良いな、と思いました。
Makefile で頑張るのが近道でしょうか。

@fujiokayu fujiokayu requested a review from a team June 11, 2019 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant