Monkey patch class and method length cops

Rubocop `ClassLength` and `MethodLength` cops have a function called
`check_code_length` that determines the lines to report for the
violation.

Rubocop passes in `keyword` as the method to be called on `node`
which always returns `1` since it's grabbing the defining keyword, eg:
`class` or `def`.

This monkey patch passes in the node's `loc` property directly to return
the correct lines.

Author: BlakeWilliams

lib/cc/engine/rubocop.rb

@@ -2,6 +2,8 @@
 require "pathname"
 require "rubocop"
 require "rubocop/cop/method_complexity_patch"
+require "rubocop/cop/method_length"
+require "rubocop/cop/class_length"
 require "cc/engine/category_parser"
 require "cc/engine/file_list_resolver"
 require "cc/engine/violation_decorator"

lib/rubocop/cop/class_length.rb

@@ -0,0 +1,10 @@
+RuboCop::Cop::Metrics::ClassLength.module_eval do
+  def check_code_length(node, *_)
+    length = code_length(node)
+    return unless length > max_length
+
+    add_offense(node, node.loc, message(length, max_length)) do
+      self.max = length
+    end
+  end
+end

lib/rubocop/cop/method_length.rb

@@ -0,0 +1,10 @@
+RuboCop::Cop::Metrics::MethodLength.module_eval do
+  def check_code_length(node, *_)
+    length = code_length(node)
+    return unless length > max_length
+
+    add_offense(node, node.loc, message(length, max_length)) do
+      self.max = length
+    end
+  end
+end

spec/cc/engine/rubocop_spec.rb

@@ -283,6 +283,39 @@ def method
         refute includes_check?(output, "Lint/UselessAssignment")
       end
 
+      it "shows full source of long methods" do
+        create_source_file("test.rb", <<-EORUBY)
+          def method
+            #{"puts 'hi'\n" * 10}
+            return false
+          end
+        EORUBY
+        output = run_engine
+        issues = output.split("\0").map { |issue| JSON.parse(issue) }
+        violation = issues.find do |issue|
+          issue["check_name"] == "Rubocop/Metrics/MethodLength"
+        end
+
+        assert_equal 1, violation["location"]["positions"]["begin"]["line"]
+        assert_equal 14, violation["location"]["positions"]["end"]["line"]
+      end
+
+      it "shows full source of long classes" do
+        create_source_file("test.rb", <<-EORUBY)
+          class Awesome
+            #{"foo = 1\n" * 102}
+          end
+        EORUBY
+        output = run_engine
+        issues = output.split("\0").map { |issue| JSON.parse(issue) }
+        violation = issues.find do |issue|
+          issue["check_name"] == "Rubocop/Metrics/ClassLength"
+        end
+
+        assert_equal 1, violation["location"]["positions"]["begin"]["line"]
+        assert_equal 105, violation["location"]["positions"]["end"]["line"]
+      end
+
       def includes_check?(output, cop_name)
         issues(output).any? { |i| i["check_name"] =~ /#{cop_name}$/ }
       end