rspec.yml in onkcop-0.45.0.0

- old
+ new
@@ -8,19 +8,32 @@
 # feature spec は exclude でも良いかもしれない。
 # ヒアドキュメント使うと一瞬で超えるので disable も検討。
 RSpec/ExampleLength:
   Max: 8
 
+# one-liner の should は書きやすいし意味が通りやすいし副作用も無いので撥ねる必要がない。
+# ただ expect 派に対して強制するほどでもないので統一はしない。
+RSpec/ImplicitExpect:
+  Enabled: false
+
 # let を使うのは context 間で条件が違うものが存在する時だけにしたい。
 # before の方が事前条件を整えていることが分かりやすい。
 RSpec/InstanceVariable:
   Enabled: false
 
-# 変に名前つけて呼ぶ方が分かりづらい。
-# テスト対象メソッドを呼ぶだけの subject 以外を書かないようにする方が効く。
-RSpec/NamedSubject:
-  Enabled: false
+# mock ではなく spy を使うのを推奨したい。
+# ただメッセージが悪く、「mock ではなく stub を使え」と言っているように読み取れる
+# (そのため Enabled: false になった) ので
+# https://github.com/backus/rubocop-rspec/pull/224
+# が cop 名も分かりやすくなって良さそう。
+RSpec/MessageExpectation:
+  Enabled: true
 
 # 強く 1 example 1 assertion の立場は取らないが、多すぎてもツラいので。
 # aggregate_failures で囲われていたら無視する的なオプション欲しい。
 RSpec/MultipleExpectations:
   Max: 3
+
+# 変に名前つけて呼ぶ方が分かりづらい。
+# テスト対象メソッドを呼ぶだけの subject 以外を書かないようにする方が効く。
+RSpec/NamedSubject:
+  Enabled: false