danger-rubocop 0.6.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 541340d181c21fb3a3c0f3ca5274df9a89610380ecf1572453cfbfd480e07f4a
-  data.tar.gz: 740f0a0e8da3d6c77c62a888984ec5636eddca707806c09674366bcc89aafe24
+  metadata.gz: ad658f09ac619e760282d3c744fb30c479711f934919fa337561a890cc810348
+  data.tar.gz: f85f21294bdbf3e1878fab539fbc5d0edc8cdc4a6b47880af049b756693b33e4
 SHA512:
-  metadata.gz: 2bde76c6fa48ecc75aa06805c5d04ed84504d15def8685e05ffd62cb1e50aa417fb55e536c621cb2cff0af1e93222652436462d7719f5d5aa39a5b2e4e8d32de
-  data.tar.gz: 909df7927367d9fc75389ffdfef64764ae38165828c82e40500667f167912d408000c03fcf6eb21241ec08c92add6dc4235ccfd4589b88fe1267925d9b335e2e
+  metadata.gz: 85c71ea2747fab34cdc91ebf7183ff8d18100416e11338a6207f7fe7a31cce0763fc6e7af63f4aec5e0b687042d208a02e92d88599e71c47c867a0468e55b0cf
+  data.tar.gz: 0aec485ea365fd126af04967a0f8267ed268bce85a3ba5ca5fd6513cbe76222d7e76de73ba2fe442855af462966da72aacbc79fbce791a4912ae23184b0869d3

data/Gemfile.lock

@@ -1,18 +1,18 @@
 PATH
   remote: .
   specs:
-    danger-rubocop (0.6.0)
+    danger-rubocop (0.7.0)
       danger
       rubocop
 
 GEM
   remote: https://rubygems.org/
   specs:
-    addressable (2.5.2)
-      public_suffix (>= 2.0.2, < 4.0)
-    ast (2.3.0)
+    addressable (2.7.0)
+      public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.0)
     bacon (1.2.0)
-    claide (1.0.2)
+    claide (1.0.3)
     claide-plugins (0.9.2)
       cork
       nap
@@ -21,26 +21,27 @@ GEM
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (5.5.5)
+    danger (6.0.9)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
       cork (~> 0.1)
       faraday (~> 0.9)
-      faraday-http-cache (~> 1.0)
-      git (~> 1)
-      kramdown (~> 1.5)
+      faraday-http-cache (~> 2.0)
+      git (~> 1.5)
+      kramdown (~> 2.0)
+      kramdown-parser-gfm (~> 1.0)
       no_proxy_fix
       octokit (~> 4.7)
       terminal-table (~> 1)
     diff-lcs (1.2.5)
-    faraday (0.13.1)
+    faraday (0.15.4)
       multipart-post (>= 1.2, < 3)
-    faraday-http-cache (1.3.1)
+    faraday-http-cache (2.0.0)
       faraday (~> 0.8)
     ffi (1.9.10)
     formatador (0.2.5)
-    git (1.3.0)
+    git (1.5.0)
     guard (2.14.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
@@ -55,7 +56,10 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    kramdown (1.16.2)
+    jaro_winkler (1.5.3)
+    kramdown (2.1.0)
+    kramdown-parser-gfm (1.1.0)
+      kramdown (~> 2.0)
     listen (3.0.7)
       rb-fsevent (>= 0.9.3)
       rb-inotify (>= 0.9.7)
@@ -66,27 +70,26 @@ GEM
       metaclass (~> 0.0.1)
     mocha-on-bacon (0.2.2)
       mocha (>= 0.13.0)
-    multipart-post (2.0.0)
+    multipart-post (2.1.1)
     nap (1.1.0)
     nenv (0.3.0)
     no_proxy_fix (0.1.2)
     notiffany (0.1.0)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    octokit (4.8.0)
+    octokit (4.14.0)
       sawyer (~> 0.8.0, >= 0.5.3)
     open4 (1.3.4)
-    parallel (1.12.1)
-    parser (2.4.0.2)
-      ast (~> 2.3)
-    powerpack (0.1.1)
+    parallel (1.17.0)
+    parser (2.6.4.1)
+      ast (~> 2.4.0)
     prettybacon (0.0.2)
       bacon (~> 1.2)
     pry (0.10.3)
       coderay (~> 1.1.0)
       method_source (~> 0.8.1)
       slop (~> 3.4)
-    public_suffix (3.0.1)
+    public_suffix (4.0.1)
     rainbow (3.0.0)
     rake (10.5.0)
     rb-fsevent (0.9.7)
@@ -105,23 +108,23 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.4.0)
     rspec-support (3.4.1)
-    rubocop (0.52.0)
+    rubocop (0.74.0)
+      jaro_winkler (~> 1.5.1)
       parallel (~> 1.10)
-      parser (>= 2.4.0.2, < 3.0)
-      powerpack (~> 0.1)
+      parser (>= 2.6)
       rainbow (>= 2.2.2, < 4.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.9.0)
-    sawyer (0.8.1)
-      addressable (>= 2.3.5, < 2.6)
-      faraday (~> 0.8, < 1.0)
+      unicode-display_width (>= 1.4.0, < 1.7)
+    ruby-progressbar (1.10.1)
+    sawyer (0.8.2)
+      addressable (>= 2.3.5)
+      faraday (> 0.8, < 2.0)
     shellany (0.0.1)
     slop (3.6.0)
     terminal-table (1.8.0)
       unicode-display_width (~> 1.1, >= 1.1.1)
     thor (0.19.1)
-    unicode-display_width (1.3.0)
+    unicode-display_width (1.6.0)
     yard (0.9.12)
 
 PLATFORMS
@@ -146,4 +149,4 @@ RUBY VERSION
    ruby 2.3.1p112
 
 BUNDLED WITH
-   1.15.4
+   1.17.3

data/README.md

@@ -47,6 +47,9 @@ The following keys are supported:
 * `inline_comment`: pass `true` to comment inline of the diffs.
 * `fail_on_inline_comment`: pass `true` to use `fail` instead of `warn` on inline comment.
 * `report_danger`: pass true to report errors to Danger, and break CI.
+* `config`: path to the `.rubocop.yml` file.
+* `only_report_new_offenses`: pass `true` to only report offenses that are in current user's scope.
+   Note that this won't mark offenses for _Metrics/XXXLength_ if you add lines to an already existing scope.
 
 
 Passing `files` as only argument is also supported for backward compatibility.

data/lib/danger_plugin.rb

@@ -30,13 +30,14 @@ module Danger
       config = config.is_a?(Hash) ? config : { files: config }
       files = config[:files]
       force_exclusion = config[:force_exclusion] || false
-
+      config_path = config[:config]
       report_danger = config[:report_danger] || false
+      only_report_new_offenses = config[:only_report_new_offenses] || false
       inline_comment = config[:inline_comment] || false
       fail_on_inline_comment = config[:fail_on_inline_comment] || false
 
       files_to_lint = fetch_files_to_lint(files)
-      files_to_report = rubocop(files_to_lint, force_exclusion)
+      files_to_report = rubocop(files_to_lint, force_exclusion, only_report_new_offenses, config_path: config_path)
 
       return if files_to_report.empty?
       return report_failures files_to_report if report_danger
@@ -46,21 +47,50 @@ module Danger
       else
         markdown offenses_message(files_to_report)
       end
-
     end
 
     private
 
-    def rubocop(files_to_lint, force_exclusion)
-      base_command = 'rubocop -f json'
-      base_command << ' --force-exclusion' if force_exclusion
+    def rubocop(files_to_lint, force_exclusion, only_report_new_offenses, config_path: nil)
+      base_command = ['rubocop', '-f', 'json']
+      base_command.concat(['--force-exclusion']) if force_exclusion
+      base_command.concat(['--config', config_path.shellescape]) unless config_path.nil?
 
-      rubocop_output = `#{'bundle exec ' if File.exist?('Gemfile')}#{base_command} #{files_to_lint}`
+      rubocop_output = `#{'bundle exec ' if File.exist?('Gemfile')}#{base_command.join(' ')} #{files_to_lint}`
 
       return [] if rubocop_output.empty?
 
-      JSON.parse(rubocop_output)['files']
-        .select { |f| f['offenses'].any? }
+      files = JSON.parse(rubocop_output)['files']
+
+      filter_out_offenses(files) if only_report_new_offenses
+
+      files.select { |f| f['offenses'].any? }
+    end
+
+    def filter_out_offenses(files)
+      files.each do |file|
+        added_lines = added_lines(file['path']).to_set
+        file['offenses'].select! do |offense|
+          added_lines.include?(offense['location']['line'])
+        end
+      end
+    end
+
+    def added_lines(path)
+      git.diff_for_file(path)
+         .patch
+         .split("\n@@")
+         .tap(&:shift)
+         .flat_map do |chunk|
+           first_line, *diff = chunk.split("\n")
+           # Get start from diff.
+           lineno = first_line.match(/\+(\d+),(\d+)/).captures.first.to_i
+           diff.each_with_object([]) do |current_line, added_lines|
+             lineno += 1 unless current_line.start_with?('-')
+             added_lines << lineno if current_line.start_with?('+')
+             added_lines
+           end
+         end
     end
 
     def offenses_message(offending_files)

data/lib/version.rb

@@ -1,3 +1,3 @@
 module DangerRubocop
-  VERSION = '0.6.1'.freeze
+  VERSION = '0.7.0'.freeze
 end

data/spec/danger_plugin_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../spec_helper', __FILE__)
+require File.expand_path('spec_helper', __dir__)
 
 module Danger
   describe DangerRubocop do
@@ -12,6 +12,82 @@ module Danger
         @rubocop = testing_dangerfile.rubocop
       end
 
+      describe "#filter_out_offenses" do
+        let(:files_within_pr) do
+          [
+            {
+              'path' => 'lib/version.rb',
+              'offenses' => [
+                {
+                  'message' => 'No.',
+                  'location' => { 'line' => 42 }
+                }
+              ]
+            }
+          ]
+        end
+
+        let(:files_outside_pr) do
+          [
+            {
+              'path' => 'lib/version.rb',
+              'offenses' => [
+                {
+                  'message' => 'No.',
+                  'location' => { 'line' => 20 }
+                }
+              ]
+            }
+          ]
+        end
+        subject { @rubocop }
+
+        before do
+          allow(@rubocop.git).to receive(:diff_for_file).with('lib/version.rb') do
+            instance_double('Git::Diff::DiffFile', patch: <<~DIFF)
+            diff --git a/lib/version.rb b/lib/version.rb
+            index 66d3a986..5e8074a8 100644
+            --- a/lib/version.rb
+            +++ b/lib/version.rb
+            @@ -32,4 +40,3 @@
+             line 1
+            -removed
+            -line 2 old version
+            +line 2 with offense
+             line 3
+            DIFF
+          end
+        end
+
+        it 'filters out offenses not in the pr' do
+          p described_class
+          expect(subject.send(:filter_out_offenses, files_outside_pr)).to eq(
+            [
+              {
+                'path' => 'lib/version.rb',
+                'offenses' => []
+              }
+            ]
+          )
+        end
+
+        it 'keeps offenses in the pr' do
+          expect(subject.send(:filter_out_offenses, files_within_pr.dup)).to eq(
+            [
+              {
+                'path' => 'lib/version.rb',
+                'offenses' => [
+                  {
+                    'message' => 'No.',
+                    'location' => { 'line' => 42 }
+                  }
+                ]
+              }
+            ]
+          )
+        end
+      end
+
       describe :lint_files do
         let(:response_ruby_file) do
           {
@@ -47,11 +123,11 @@ module Danger
 
         it 'handles a rubocop report for specified files' do
           allow(@rubocop).to receive(:`)
-            .with('bundle exec rubocop -f json spec/fixtures/ruby_file.rb')
+            .with('bundle exec rubocop -f json --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
             .and_return(response_ruby_file)
 
           # Do it
-          @rubocop.lint(files: 'spec/fixtures/ruby*.rb')
+          @rubocop.lint(files: 'spec/fixtures/ruby*.rb', config: 'path/to/rubocop.yml')
 
           output = @rubocop.status_report[:markdowns].first.message
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-rubocop
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.7.0
 platform: ruby
 authors:
 - Ash Furrow
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-11-21 00:00:00.000000000 Z
+date: 2019-09-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger