「privateメソッドのテストについての考え方」を読んで #yochiyochirb - アジャイルSEの憂鬱

highwide.hatenablog.com を読んで、「自分なら設計を変えて、publicにしてからテストを書くなー」と思ったので、考え方・直し方の一例としてブログを書く。 元の設計 元記事のスライドの途中に出てくるコードはこんな感じで、バッチ処理などでよくある設計。 module Tasks module Hoge class Sender def self.execute data = aggregate_data processed_data = process_data(data) send_s3(processed_data) end end end end コードの臭い 個人的なリファクタリング原則で「引数が1つのメソッドは、その引数のインスタンスメソッドに書き換えられる」がある。*1 あと、元コードだと「データ収集、加工、s3に置く」のが1メソッドになっていてテストし辛い

[ledsun]

「引数が1つのメソッドは、その引数のインスタンスメソッドに書き換えられる」わかる。僕の中では「第一引数だけを破壊するstaticなメソッドは、」が判定条件かな？