Update Rubocop config to match CC styleguide

A small update to bring our rubocop config up-to-date with our styleguide.
Following the refactor in #16, few changes in the codebase were needed.

Author: dblandin

.rubocop.yml

@@ -1,43 +1,110 @@
-AllCops:
-  TargetRubyVersion: 2.2
-Metrics/AbcSize:
-  Enabled: false
+################################################################################
+# Metrics
+################################################################################
 
 Metrics/LineLength:
   Enabled: false
 
-Rails/TimeZone:
+Metrics/AbcSize:
   Enabled: false
 
-SignalException:
+################################################################################
+# Style
+################################################################################
+
+# Executables are conventionally named bin/foo-bar
+Style/FileName:
+  Exclude:
+  - bin/**/*
+
+# We don't (currently) document our code
+Style/Documentation:
   Enabled: false
 
+# Always use double-quotes to keep things simple
 Style/StringLiterals:
-  Enabled: false
+  EnforcedStyle: double_quotes
 
-Style/Documentation:
+# Use a trailing comma to keep diffs clean when elements are inserted or removed
+Style/TrailingComma:
+  EnforcedStyleForMultiline: comma
+
+# We avoid GuardClause because it can result in "suprise return"
+Style/GuardClause:
   Enabled: false
 
-Style/TrailingCommaInLiteral:
+# We avoid IfUnlessModifier because it can result in "suprise if"
+Style/IfUnlessModifier:
   Enabled: false
 
-Style/ClassAndModuleChildren:
+# We don't care about the fail/raise distinction
+Style/SignalException:
+  EnforcedStyle: only_raise
+
+Style/DotPosition:
+  EnforcedStyle: trailing
+
+# Common globals we allow
+Style/GlobalVars:
+  AllowedVariables:
+  - "$statsd"
+  - "$mongo"
+  - "$rollout"
+
+# Allow $! in config/initializers
+Style/SpecialGlobalVars:
   Exclude:
-    - 'spec/**/*'
+  - config/initializers/**/*
 
-Style/IfUnlessModifier:
+# We have common cases where has_ and have_ make sense
+Style/PredicateName:
+  Enabled: true
+  NamePrefixBlacklist:
+  - is_
+
+# We use %w[ ], not %w( ) because the former looks like an array
+Style/PercentLiteralDelimiters:
+  PreferredDelimiters:
+    "%w": []
+    "%W": []
+
+# Allow "trivial" accessors when defined as a predicate? method
+Style/TrivialAccessors:
+  AllowPredicates: true
+
+Style/Next:
   Enabled: false
 
-Style/DotPosition:
+# We think it's OK to use the "extend self" module pattern
+Style/ModuleFunction:
   Enabled: false
 
-Style/GuardClause:
+################################################################################
+# Rails - disable things because we're primarily non-rails
+################################################################################
+
+Rails/Delegate:
   Enabled: false
 
-Style/StringLiteralsInInterpolation:
+Rails/TimeZone:
   Enabled: false
 
-Style/PercentLiteralDelimiters:
-  PreferredDelimiters:
-    '%w': []
-    '%W': []
+################################################################################
+# Specs - be more lenient on length checks and block styles
+################################################################################
+
+Metrics/ModuleLength:
+  Exclude:
+  - spec/**/*
+
+Metrics/MethodLength:
+  Exclude:
+  - spec/**/*
+
+Style/ClassAndModuleChildren:
+  Exclude:
+  - spec/**/*
+
+Style/BlockDelimiters:
+  Exclude:
+  - spec/**/*

Rakefile

@@ -1,4 +1,4 @@
-require 'rspec/core/rake_task'
+require "rspec/core/rake_task"
 RSpec::Core::RakeTask.new(:spec)
 
 task default: :spec

bin/bundler-audit

@@ -2,6 +2,6 @@
 
 $LOAD_PATH.unshift(File.expand_path(File.join(File.dirname(__FILE__), "../lib")))
 
-require 'cc/engine/bundler_audit'
+require "cc/engine/bundler_audit"
 
 CC::Engine::BundlerAudit::Analyzer.new(directory: "/code").run

lib/cc/engine/bundler_audit/issue.rb

@@ -6,7 +6,7 @@ class Issue
         SEVERITIES = {
           high: "critical",
           medium: "normal",
-          low: "info"
+          low: "info",
         }.freeze
 
         def initialize(result, gemfile_lock_lines)
@@ -27,12 +27,12 @@ def to_json(*a)
               path: "Gemfile.lock",
               lines: {
                 begin: line_number,
-                end: line_number
-              }
+                end: line_number,
+              },
             },
             remediation_points: remediation_points,
             severity: severity,
-            type: "Issue"
+            type: "Issue",
           }.to_json(a)
         end
 
@@ -45,7 +45,7 @@ def content_body
             "**Advisory**: #{identifier}",
             "**Criticality**: #{advisory.criticality.capitalize}",
             "**URL**: #{advisory.url}",
-            "**Solution**: #{solution}"
+            "**Solution**: #{solution}",
           ].join("\n\n")
         end
 

spec/cc/engine/bundler_audit/analyzer_spec.rb

@@ -7,8 +7,8 @@ module CC::Engine::BundlerAudit
         directory = fixture_directory("no_gemfile_lock")
         io = StringIO.new
 
-        expect { Analyzer.new(directory: directory, io: io).run }
-          .to raise_error(Analyzer::GemfileLockNotFound)
+        expect { Analyzer.new(directory: directory, io: io).run }.
+          to raise_error(Analyzer::GemfileLockNotFound)
       end
 
       it "emits issues for Gemfile.lock problems" do