第14章学習用テストと回帰テスト #18

at-grandpa · 2017-11-10T00:42:44Z

TODO

at-grandpa · 2017-11-10T13:10:15Z

一旦本書とのズレを精算する。

at-grandpa · 2017-11-10T13:36:03Z

２フランを１ドルに変換するテストを書く。

at-grandpa · 2017-11-10T13:38:31Z

コンパイルエラー。

 crystal spec' <

Error in line 1: while requiring "./spec/money_package_spec.cr"

in spec/money_package_spec.cr:62: undefined method 'addRate' for MoneyPackage::Bank

      bank.addRate("CHF", "USD", 2)
           ^~~~~~~

Rerun with --error-trace to show a complete error trace.

at-grandpa · 2017-11-10T13:40:56Z

コンパイルは通るが、テストがコケる。

 crystal spec' <

.......F

Failures:

  1) MoneyPackage testReduceMoneyDifferentCurrency() 異なる通貨を計算して変換できること
     Failure/Error: result.should eq Money.dollar(1)

       Expected: #<MoneyPackage::Money:0x101c20a00 @amount=1, @currency="USD">
            got: #<MoneyPackage::Money:0x101c20a20 @amount=2, @currency="CHF">

     # spec/money_package_spec.cr:64

Finished in 1.2 milliseconds
8 examples, 1 failures, 0 errors, 0 pending

Failed examples:

crystal spec spec/money_package_spec.cr:60 # MoneyPackage testReduceMoneyDifferentCurrency() 異なる通貨を計算して変換できること

at-grandpa · 2017-11-10T13:53:41Z

Moneyが為替レートを知っている状態になってしまった。為替レートはBankが一手に担うべき。なので、Expressionのreduceには、Bankを引数にとらないといけない。

at-grandpa · 2017-11-10T14:03:57Z

テストが通る。

 crystal spec' <

........

Finished in 868 microseconds
8 examples, 0 failures, 0 errors, 0 pending

at-grandpa · 2017-11-10T14:04:39Z

マジックナンバーが残っているので、Bank内に表形式でデータを持ち、検索できるようにしたい。

at-grandpa · 2017-11-10T15:27:40Z

配列の等価比較がどのようになるかを学習するためのテストを書いた。crystalの場合は一致してしまうことがわかった。

とりあえず本書通り進めていくことにする。

at-grandpa · 2017-11-10T15:28:07Z

テストを消して、Pairクラスを作成する。

at-grandpa · 2017-11-10T15:31:01Z

PairクラスをHashテーブルのkeyに使うには、equalsメソッドとhashCodeメソッドを実装しないといけない。

ｆｍ
crystalだとどうなるのか検証しよう。

at-grandpa · 2017-11-10T15:36:29Z

今はPairのテストを書かない。リファクタリングの一貫でPairクラスを抽出しているから。リファクタリングが完了していれば、Pairクラスもテストできていることと同じだ。

at-grandpa · 2017-11-10T21:25:56Z

equalsメソッドとhashCodeメソッド

equalsは==でok。hashCodeはObject#hashでok。

https://crystal-lang.org/api/0.23.1/Object.html#hash-instance-method

これは、Hashのkeyの比較にも使われている。

The hash value is used along with #== by the Hash class to determine if two objects reference the same hash key.

at-grandpa · 2017-11-10T21:51:05Z

レッドバー

 crystal spec' <

......E.

Failures:

  1) MoneyPackage testReduceMoney() BankのreduceメソッドにMoneyを渡した場合い指定通貨に換算できること

       Missing hash key: #<MoneyPackage::Pair:0x1042419e0 @from="USD", @to="USD">
       0x103ba6695: *CallStack::unwind:Array(Pointer(Void)) at ??
       0x103ba6631: *CallStack#initialize:Array(Pointer(Void)) at ??
       0x103ba6608: *CallStack::new:CallStack at ??
       0x103ba2375: *raise<KeyError>:NoReturn at ??
       0x103c24ff1: *Hash(MoneyPackage::Pair, Int32)@Hash(K, V)#fetch<MoneyPackage::Pair>:Int32 at ??
       0x103c24f19: *Hash(MoneyPackage::Pair, Int32)@Hash(K, V)#[]<MoneyPackage::Pair>:Int32 at ??
       0x103c24da7: *MoneyPackage::Bank#rate<String, String>:Int32 at ??
       0x103c21e59: *MoneyPackage::Money#reduce<MoneyPackage::Bank, String>:MoneyPackage::Money at ??
       0x103c24d78: *MoneyPackage::Bank#reduce<MoneyPackage::Money, String>:MoneyPackage::Money at ??
       0x103ba4edb: ~procProc(Nil)@spec/money_package_spec.cr:53 at ??
       0x103ba437b: *it<String, String, Int32, Int32, &Proc(Nil)>:(Array(Spec::Result) | Nil) at ??
       0x103ba4e90: ~procProc(Nil)@spec/money_package_spec.cr:52 at ??
       0x103c02502: *Spec::RootContext::describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x103ba4789: *describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x103ba3f80: ~procProc(Nil)@spec/money_package_spec.cr:5 at ??
       0x103c02502: *Spec::RootContext::describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x103ba5073: *describe<MoneyPackage:Module, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x103b8dd57: __crystal_main at ??
       0x103ba1798: main at ??

Finished in 1.39 milliseconds
8 examples, 0 failures, 1 errors, 0 pending

Failed examples:

crystal spec spec/money_package_spec.cr:53 # MoneyPackage testReduceMoney() BankのreduceメソッドにMoneyを渡した場合い指定通貨に換算できること

#<MoneyPackage::Pair:0x1042419e0 @from="USD", @to="USD">の時に1を返さないとだめだが、実装されてない。テストから書こう。

at-grandpa · 2017-11-10T21:51:28Z

なるほど。予期しないエラーがでたら、まずテストを書くのか。

at-grandpa · 2017-11-10T21:51:55Z

まー確かに、そのほうが検証早いよなぁ。他のテストが落ちていないかも検証できるし。

at-grandpa · 2017-11-10T21:55:00Z

レッドバーが二つになった。

 crystal spec' <

......E.E

Failures:

  1) MoneyPackage testReduceMoney() BankのreduceメソッドにMoneyを渡した場合い指定通貨に換算できること

       Missing hash key: #<MoneyPackage::Pair:0x102d5c9e0 @from="USD", @to="USD">
       0x1026c4015: *CallStack::unwind:Array(Pointer(Void)) at ??
       0x1026c3fb1: *CallStack#initialize:Array(Pointer(Void)) at ??
       0x1026c3f88: *CallStack::new:CallStack at ??
       0x1026bfbc5: *raise<KeyError>:NoReturn at ??
       0x102742a81: *Hash(MoneyPackage::Pair, Int32)@Hash(K, V)#fetch<MoneyPackage::Pair>:Int32 at ??
       0x1027429a9: *Hash(MoneyPackage::Pair, Int32)@Hash(K, V)#[]<MoneyPackage::Pair>:Int32 at ??
       0x102742837: *MoneyPackage::Bank#rate<String, String>:Int32 at ??
       0x10273f8a9: *MoneyPackage::Money#reduce<MoneyPackage::Bank, String>:MoneyPackage::Money at ??
       0x102742808: *MoneyPackage::Bank#reduce<MoneyPackage::Money, String>:MoneyPackage::Money at ??
       0x1026c276b: ~procProc(Nil)@spec/money_package_spec.cr:53 at ??
       0x1026c1c0b: *it<String, String, Int32, Int32, &Proc(Nil)>:(Array(Spec::Result) | Nil) at ??
       0x1026c2720: ~procProc(Nil)@spec/money_package_spec.cr:52 at ??
       0x10271ff52: *Spec::RootContext::describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026c2019: *describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026c17d0: ~procProc(Nil)@spec/money_package_spec.cr:5 at ??
       0x10271ff52: *Spec::RootContext::describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026c29f3: *describe<MoneyPackage:Module, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026ab597: __crystal_main at ??
       0x1026befe8: main at ??

  2) MoneyPackage testIdentityRate() 同一通貨の場合はrateとして１を返すこと

       Missing hash key: #<MoneyPackage::Pair:0x102d5c8c0 @from="USD", @to="USD">
       0x1026c4015: *CallStack::unwind:Array(Pointer(Void)) at ??
       0x1026c3fb1: *CallStack#initialize:Array(Pointer(Void)) at ??
       0x1026c3f88: *CallStack::new:CallStack at ??
       0x1026bfbc5: *raise<KeyError>:NoReturn at ??
       0x102742a81: *Hash(MoneyPackage::Pair, Int32)@Hash(K, V)#fetch<MoneyPackage::Pair>:Int32 at ??
       0x1027429a9: *Hash(MoneyPackage::Pair, Int32)@Hash(K, V)#[]<MoneyPackage::Pair>:Int32 at ??
       0x102742837: *MoneyPackage::Bank#rate<String, String>:Int32 at ??
       0x1026c2972: ~procProc(Nil)@spec/money_package_spec.cr:68 at ??
       0x1026c1c0b: *it<String, String, Int32, Int32, &Proc(Nil)>:(Array(Spec::Result) | Nil) at ??
       0x1026c2940: ~procProc(Nil)@spec/money_package_spec.cr:67 at ??
       0x10271ff52: *Spec::RootContext::describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026c2019: *describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026c1850: ~procProc(Nil)@spec/money_package_spec.cr:5 at ??
       0x10271ff52: *Spec::RootContext::describe<String, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026c29f3: *describe<MoneyPackage:Module, String, Int32, &Proc(Nil)>:Spec::Context+ at ??
       0x1026ab597: __crystal_main at ??
       0x1026befe8: main at ??

Finished in 1.31 milliseconds
9 examples, 0 failures, 2 errors, 0 pending

Failed examples:

crystal spec spec/money_package_spec.cr:53 # MoneyPackage testReduceMoney() BankのreduceメソッドにMoneyを渡した場合い指定通貨に換算できること
crystal spec spec/money_package_spec.cr:68 # MoneyPackage testIdentityRate() 同一通貨の場合はrateとして１を返すこと

at-grandpa · 2017-11-10T21:56:59Z

グリーン。

 crystal spec' <

.........

Finished in 1.27 milliseconds
9 examples, 0 failures, 0 errors, 0 pending

at-grandpa · 2017-11-10T21:57:34Z

TODO

at-grandpa · 2017-11-10T21:58:58Z

「これだけ今やったんだっけ？」ってなる。細かいステップに気を取られていて、全体的に今どうしているんだっけ？が迷う。

at-grandpa · 2017-11-10T22:00:02Z

こういう時は、テストをみよう。やったことがすべてそこに書かれている。

at-grandpa · 2017-11-10T22:00:29Z

TODOリストを増やすタイミングは、疑念が湧いたとき、なのかなー

at-grandpa · 2017-11-10T22:01:54Z

こういう迷いも考慮して、本書には「振り返り」があるんだろう。良いなぁ。

at-grandpa · 2017-11-10T22:04:49Z

振り返り。

必要になると予想されていたパラメータ追加をすぐに行った
コードとテストの間のデータの重複を括りだした
言語仕様を調べるテストを書いた
実装内部で使うためだけのヘルパークラスを個別のテスト無しで書いた
リファクタリング中にミスを犯したが、問題を再現するテストを追加して、着実に前進した

at-grandpa · 2017-11-10T22:04:57Z

14章終了

yoshiyuki-tsuchida added 2 commits November 10, 2017 09:41

first commit

15e0c75

コメント削除

05c8134

yoshiyuki-tsuchida added 2 commits November 10, 2017 22:32

本書とのズレを一旦精算する

44d46a1

精算

4822852

テストの追加

702d7bb

addRateの空実装

4122a2c

reduceの仮実装でグリーンバーにした

70b7e55

yoshiyuki-tsuchida added 2 commits November 10, 2017 23:00

reduceメソッドの引数にBankを渡すようにした

0ddf54f

Bankにrateを計算させた

9d98825

学習用テスト

1243f7b

学習用テストの削除

fd11b42

yoshiyuki-tsuchida added 2 commits November 11, 2017 06:28

Pairクラスを実装し、==とhashメソッドを追加した

57daa35

rateを格納するhashを定義し、addRateとrateのメソッドを書き換えた

1649565

同一通貨の場合のrateのテストを追加

6bafcb6

同一通貨の場合に1を返す実装

625476b

READMEの修正

a12476a

at-grandpa merged commit 058e8f5 into master Nov 10, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

第14章学習用テストと回帰テスト #18

第14章学習用テストと回帰テスト #18

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

第14章 学習用テストと回帰テスト #18

第14章 学習用テストと回帰テスト #18

Conversation

at-grandpa commented Nov 10, 2017 • edited Loading

TODO

at-grandpa commented Nov 10, 2017 • edited Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017 • edited Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017 • edited Loading

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017 • edited Loading

TODO

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

at-grandpa commented Nov 10, 2017

第14章学習用テストと回帰テスト #18

第14章学習用テストと回帰テスト #18

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017 •

edited

Loading

at-grandpa commented Nov 10, 2017 •

edited

Loading