Merge pull request #10 from codeclimate/will/monkeypatch-failfast

change monkey patch to use module_eval

Author: wfleming

lib/cc/engine/rubocop.rb

@@ -1,7 +1,7 @@
 require "json"
 require "pathname"
-require "rubocop/cop/method_complexity_patch"
 require "rubocop"
+require "rubocop/cop/method_complexity_patch"
 require "cc/engine/category_parser"
 
 module CC

lib/rubocop/cop/method_complexity_patch.rb

@@ -1,18 +1,14 @@
 # Monkey patching! Here be dragons.
-module RuboCop
-  module Cop
-    module MethodComplexity
-      # RuboCop's default implementation of `add_offense` in `Cop` only gets the
-      # location of the keyword associated with the problem which, for things
-      # like complexity checkers, is just the method def line. This isn't very
-      # useful for checkers where the entire method body is relevant. Fetching
-      # this information from an `Offense` instance is difficult, since the
-      # original AST is no longer available. So it's easier to monkey-path
-      # this method on complexity checkers to send the location of the entire
-      # method to the created `Offense`.
-      def add_offense(node, loc, message = nil, severity = nil)
-        super(node, node.loc, message, severity)
-      end
-    end
+RuboCop::Cop::MethodComplexity.module_eval do
+  # RuboCop's default implementation of `add_offense` in `Cop` only gets the
+  # location of the keyword associated with the problem which, for things
+  # like complexity checkers, is just the method def line. This isn't very
+  # useful for checkers where the entire method body is relevant. Fetching
+  # this information from an `Offense` instance is difficult, since the
+  # original AST is no longer available. So it's easier to monkey-path
+  # this method on complexity checkers to send the location of the entire
+  # method to the created `Offense`.
+  def add_offense(node, loc, message = nil, severity = nil)
+    super(node, node.loc, message, severity)
   end
 end