onkcop 0.44.1.1 → 0.45.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 71b5bce1c4bc5855da59a997e7784600d7a500f6
-  data.tar.gz: 13271f7395df44277bd66f59680b85a097c03b83
+  metadata.gz: 1592b33c4ff3db3988cbfbeba1b8443382b8f265
+  data.tar.gz: 51d2ddfae92c082b28450fe26f127b9f278d0042
 SHA512:
-  metadata.gz: bb6de1cd8f2e113af48578d3c0754f1c6d57bb96d6c8a8417d755c5a81217206717db8f79d1cb7704cea04471e94a6142f2e96c95449be0d8f3ed47e515ff101
-  data.tar.gz: accb348a27844f4e1335da230987c5e9fe87b7af25598715664189053fe2857d3442eae8c25bd121d5262dba830ffc44b9f6cfcb43c513977f84f02826ef52eb
+  metadata.gz: a22fce88c3ad8677e3017616397ccf88fcfb9de3779bd7111b278ba9ec7ebbdb71154e78aa9e3cb68dadab99e8b76fc5bf716ef85b83844a01f035e2e5a29a98
+  data.tar.gz: 20c9632930b74cdfa01c888cf5d04bb16046be657d41fae14db69852e27b86c7d988e56cf840f1013192db5a2ff30c764ede7cc5d85f9dff8b860943ee8e55e4

data/config/rails.yml

@@ -1,10 +1,6 @@
 Rails:
   Enabled: true
 
-# v0.44.1 には session, xhr, format を解釈しないバグがある
-Rails/HttpPositionalArguments:
-  Enabled: false
-
 # slug とか created_by とか、NOT NULL だが create_table 時に
 # default 値を定義できないカラムは存在する。
 Rails/NotNullColumn:

data/config/rspec.yml

@@ -10,17 +10,30 @@ RSpec/DescribedClass:
 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

data/config/rubocop.yml

@@ -1,5 +1,5 @@
 # target_version:
-#   rubocop v0.44.1
+#   rubocop v0.45.0
 
 # 自動生成されるものはチェック対象から除外する
 AllCops:
@@ -200,6 +200,11 @@ Style/StringLiteralsInInterpolation:
 Style/SingleLineBlockParams:
   Enabled: false
 
+# 三項演算子は分かりやすく使いたい。
+# () を外さない方が条件式が何なのか読み取りやすいと感じる。
+Style/TernaryParentheses:
+  Enabled: false
+
 # 複数行の場合はケツカンマを入れる
 Style/TrailingCommaInLiteral:
   EnforcedStyleForMultiline: comma
@@ -228,12 +233,15 @@ Lint/UnusedMethodArgument:
 Metrics/AbcSize:
   Max: 24
 
+# Gemfile, Guardfile は DSL 的で基本的に複雑にはならないので除外
 # routes は巨大な block 不可避なので除外
 Metrics/BlockLength:
   Exclude:
     - "Rakefile"     # default
     - "**/*.rake"    # default
     - "spec/**/*.rb" # default
+    - "Gemfile"
+    - "Guardfile"
     - "config/routes.rb"
     - "config/routes/**/*.rb"
 

data/lib/onkcop/version.rb

@@ -1,3 +1,3 @@
 module Onkcop
-  VERSION = "0.44.1.1"
+  VERSION = "0.45.0.0"
 end

data/onkcop.gemspec

@@ -19,8 +19,8 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rubocop", "~> 0.44.1"
-  spec.add_dependency "rubocop-rspec", ">= 1.7.0"
+  spec.add_dependency "rubocop", "~> 0.45.0"
+  spec.add_dependency "rubocop-rspec", ">= 1.8.0"
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: onkcop
 version: !ruby/object:Gem::Version
-  version: 0.44.1.1
+  version: 0.45.0.0
 platform: ruby
 authors:
 - Takafumi ONAKA
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-10-13 00:00:00.000000000 Z
+date: 2016-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.44.1
+        version: 0.45.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.44.1
+        version: 0.45.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.7.0
+        version: 1.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.7.0
+        version: 1.8.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement