ヘルパーメソッドの中でいきなり current_user を使うのは変ですか？

Question

ヘルパーメソッドの中でいきなり current_user を使うのは変ですか？

def hogehoge
  shori(current_user)
end

下記の様に引数で current_user を受けた方がいいですか？

def hogehoge(user)
  shori(user)
end

---

いきなりメソッドの中で current_user が出現すると、
テストを書く時に何らかの不都合があったりしますか？

インプットがあって、アウトプットがあるみたいに書いた方がいいですか？
いきなりメソッド内部で current_user が表れているのでおかしいと感じますか？

Answer 1

おそらくdeviseをつかっているのでしょうか？

current_userが何を具体的にどのようなメソッドを指しているのかはわからないのですが、別にそのような書き方をしてもテストに影響はしないと思います

引数にするかどうかは、そのメソッドの動作を引数の種類によって変えたいかどうかにかかっていると思います
もしcurrent_userしか引数に入れる予定がないのなら、引数にしなくてもいいと思います