Railsで開発するにあたって、rubocop-railsをLinterとして利用しており、
自動修正機能を適用したときにエラーに繋ってしまったのでメモしておきます。

背景

Railsではfind_by_idやfind_by_nameのように、ActiveModelのカラム名を利用して検索できるメソッドが動的に定義されます。

しかし、このような個別のメソッドを利用するのではなくfind_by形式に統一するスタイルがRuboCopにおいては推奨されており、検知&自動修正するためのCopが存在します。

一方で、自前でfind_by_hogeのようなメソッドを定義した場合にもこのCopの判定対象になってしまい、自動修正の対象になってしまいます。

自分の開発しているプロジェクトでは自動修正をかけた際に意図せず変更されてしまい、メソッド未定義のエラーに繋りました。

回避方法

一部のメソッドをこのCopの対象から除外したい場合には以下のアプローチがあります:

2つの方法があるなかで、私は後者の選択をしました。

判断の理由としては、前者のインラインコメントで都度対応をするという方法は

といったいくつかのデメリットがあると考えたためです。

そのため、後者の.rubocop.yml 中に記述するアプローチを取りました。

.rubocop.yml

Rails/DynamicFindBy:
  Whitelist:
    - find_by_sql
    - find_by_hoge

Whitelistに該当メソッドだけでなく、find_by_sqlについても追加していますが、こちらのメソッドもRailsの標準のメソッドであり自動修正の対象とはしたくないためです。

RuboCopの設定は、デフォルトの設定を自動的に継承するようになっていますが、

継承されている設定と同じ項目についての設定を記述した時に以下のようなルールで上書きがされます。

今回は後者の配列の上書きに該当しますので、標準でWhitelistに追加されているfind_by_sqlについても自分で追加しておく必要がありました。