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

第21章 数え上げ #26

Merged
merged 4 commits into from
Jan 9, 2018
Merged

第21章 数え上げ #26

merged 4 commits into from
Jan 9, 2018

Conversation

at-grandpa
Copy link
Owner

@at-grandpa at-grandpa commented Jan 9, 2018

TODO

  • テストメソッドを呼び出す
  • setUpを最初に呼び出す
  • tearDownを後で呼び出す
  • テストメソッドが失敗したとしてもtearDownを呼び出す
  • 複数のテストを走らせる
  • 収集したテスト結果を出力する
  • WasRunで文字列をログに記録する

@at-grandpa
Copy link
Owner Author

この章で、テストが失敗しても必ずtearDownが呼ばれるようにしたかった。それには例外をキャッチする必要があるが、実装に失敗しても例外が出ないので、間違いに気づきにくい。

@at-grandpa
Copy link
Owner Author

テストを書く順番は重要。あるテストを書いて、次のテストで動きが止まってしまってはだめだ。二歩戻る。

@at-grandpa
Copy link
Owner Author

5 run, 2 failed 的な感じで数え上げたいが、今の自作フレームワークに全てのテスト結果を収集してレポーティングさせるのは大きすぎる。

@at-grandpa
Copy link
Owner Author

runメソッドがTestResultクラスを返すようにしたらどうか。

テストから書こう。

@at-grandpa
Copy link
Owner Author

at-grandpa commented Jan 9, 2018

TODO

  • テストメソッドを呼び出す
  • setUpを最初に呼び出す
  • tearDownを後で呼び出す
  • テストメソッドが失敗したとしてもtearDownを呼び出す
  • 複数のテストを走らせる
  • 収集したテスト結果を出力する
  • WasRunで文字列をログに記録する
  • 失敗したテストを出力する

@at-grandpa
Copy link
Owner Author

例外が発生してしまうが、テストをコメントアウトして一旦保留。

より小さなテストに移る。

@at-grandpa
Copy link
Owner Author

at-grandpa commented Jan 9, 2018

振り返り

  • まずは仮実装(p. 217)を書き、ベタ書きの値を変数に置き換えることで段階的に実装を本物に近づた。
  • もう 1 つテストを書いた。
  • 追加したテストが失敗したので、もう少し小さいテスト(p. 207)を別に書いて、失敗したテストを通すための実装を導くことにした。

@at-grandpa
Copy link
Owner Author

この章で得た知見。

  • 仮実装、三角測量、変数への書き換え
    • 当たり前だけど強力

@at-grandpa at-grandpa merged commit 76ed0f1 into master Jan 9, 2018
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

Successfully merging this pull request may close these issues.

1 participant