danger-rubocop 0.12.0 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e62c69c270c52a9db56563f52892f3e3f04ac0f154a70144f20ef66f2a5db517
-  data.tar.gz: 7db336caf7afd0d8574a66096ee0276b9e92f725d914e57e5720bfe6a83fc27a
+  metadata.gz: 228bd1f2c8294e6ade21c5a33424af6ddfa0ceb08613f62ca22db33bdfecd212
+  data.tar.gz: cafedd57a18b594548ef05c3287fd96acbf9a5692306e291ad85f7cf3456e6ad
 SHA512:
-  metadata.gz: e88de395cb97400f569e38461f45dfd978483bbaf42f048865b4e3f33fefb86632c5ca55f63796cb45a5f5947065200cb6e2bc35fcaa3f42c932c565de7d2cb8
-  data.tar.gz: c192a3cf0f8b3b199d2e6526c3df8f73c96cffed6cb683f7216dfd00ff9a2c92c83cac77bca63b28eabc215f26bae67c003737977ea112c62d2b73dcb4939c8f
+  metadata.gz: e30f4a47b31ce14a58e263da6a7d0cd80a81c016b989be786487393d60f45a131bbabf76f4dbbb08b22058331d1878ff30ddd917cceb3db1e34b5f3cb85fd039
+  data.tar.gz: 705e56f918fe479ee9633f513aaa9ce3396616456112be2d4fa2b1f9358b9c58c620f367ba8cedbb6159e5463d2bad2168e3fd1a949ff3477e6696b4fd24acc4

data/Gemfile.lock

@@ -108,7 +108,8 @@ GEM
       ffi (~> 1.0)
     rchardet (1.8.0)
     regexp_parser (2.1.1)
-    rexml (3.2.5)
+    rexml (3.2.8)
+      strscan (>= 3.0.9)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)
@@ -139,11 +140,12 @@ GEM
       addressable (>= 2.3.5)
       faraday (> 0.8, < 2.0)
     shellany (0.0.1)
+    strscan (3.1.0)
     terminal-table (3.0.1)
       unicode-display_width (>= 1.1.1, < 3)
     thor (1.1.0)
     unicode-display_width (2.0.0)
-    yard (0.9.26)
+    yard (0.9.36)
 
 PLATFORMS
   ruby

data/README.md

@@ -57,6 +57,7 @@ The following keys are supported:
   (this option will instruct rubocop to ignore the files that your rubocop config ignores,
   despite the plugin providing the list of files explicitly)
 * `inline_comment`: pass `true` to comment inline of the diffs.
+* `group_inline_comments`: pass `true` to group inline comments to be a single comment on each line with all issues for that line.
 * `fail_on_inline_comment`: pass `true` to use `fail` instead of `warn` on inline comment.
 * `report_severity`: pass `true` to use `fail` or `warn` based on Rubocop severity.
 * `report_danger`: pass true to report errors to Danger, and break CI.
@@ -66,7 +67,7 @@ The following keys are supported:
    Note that this won't mark offenses for _Metrics/XXXLength_ if you add lines to an already existing scope.
 * `include_cop_names`: Prepends cop names to the output messages. Example: "Layout/EmptyLinesAroundBlockBody: Extra empty line detected at block body end."
 * `rubocop_cmd`: Allows you to change the rubocop executable that's invoked. This is used to support rubocop wrappers like [Standard](https://github.com/testdouble/standard/) by passing `standardrb` as the value.
-
+* `skip_bundle_exec`: When there is a `Gemfile` in the project, Rubocop will be executed using [Bundler](https://bundler.io). When `true`, this flag will force Rubocop to run without `bundle exec`.
 
 Passing `files` as only argument is also supported for backward compatibility.
 

data/lib/danger_plugin.rb

@@ -34,18 +34,22 @@ module Danger
       report_danger = config[:report_danger] || false
       only_report_new_offenses = config[:only_report_new_offenses] || false
       inline_comment = config[:inline_comment] || false
+      group_inline_comments = config[:group_inline_comments] || false
       fail_on_inline_comment = config[:fail_on_inline_comment] || false
       report_severity = config[:report_severity] || false
       include_cop_names = config[:include_cop_names] || false
       rubocop_cmd = config[:rubocop_cmd] || 'rubocop'
+      skip_bundle_exec = config[:skip_bundle_exec] || false
 
       files_to_lint = fetch_files_to_lint(files)
-      files_to_report = rubocop(files_to_lint, force_exclusion, only_report_new_offenses, cmd: rubocop_cmd, config_path: config_path)
+      files_to_report = rubocop(files_to_lint, force_exclusion, only_report_new_offenses, cmd: rubocop_cmd, config_path: config_path, skip_bundle_exec: skip_bundle_exec)
 
       return if files_to_report.empty?
       return report_failures(files_to_report, include_cop_names: include_cop_names) if report_danger
 
-      if inline_comment
+      if inline_comment && group_inline_comments
+        add_grouped_violation_for_each_line(files_to_report, fail_on_inline_comment, report_severity, include_cop_names: include_cop_names)
+      elsif inline_comment
         add_violation_for_each_line(files_to_report, fail_on_inline_comment, report_severity, include_cop_names: include_cop_names)
       else
         markdown offenses_message(files_to_report, include_cop_names: include_cop_names)
@@ -54,12 +58,12 @@ module Danger
 
     private
 
-    def rubocop(files_to_lint, force_exclusion, only_report_new_offenses, cmd: 'rubocop', config_path: nil)
+    def rubocop(files_to_lint, force_exclusion, only_report_new_offenses, cmd: 'rubocop', config_path: nil, skip_bundle_exec: false)
       base_command = [cmd, '-f', 'json', '--only-recognized-file-types']
       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.join(' ')} #{files_to_lint}`
+      rubocop_output = `#{'bundle exec ' if File.exist?('Gemfile') && !skip_bundle_exec}#{base_command.join(' ')} #{files_to_lint}`
 
       return [] if rubocop_output.empty?
 
@@ -144,6 +148,37 @@ module Danger
       end
     end
 
+    def add_grouped_violation_for_each_line(offending_files, fail_on_inline_comment, report_severity, include_cop_names: false)
+      grouped_offense_messages = Hash.new { |h, k| h[k] = [] }
+      offending_files.flat_map do |file|
+        file['offenses'].map do |offense|
+          offense_message = offense_message(offense, include_cop_names: include_cop_names)
+          kargs = {
+            file: file['path'],
+            line: offense['location']['line']
+          }
+          grouped_offense_messages[kargs] << offense_message
+        end
+      end
+
+      grouped_offense_messages.each do |kargs, offense_messages|
+        grouped_offense_message = if offense_messages.length > 1
+          "\n" + offense_messages.map do |offense_message|
+            "* #{offense_message}"
+          end.join("\n")
+        else
+          offense_messages[0]
+        end
+        if fail_on_inline_comment
+          fail(grouped_offense_message, **kargs)
+        elsif report_severity && %w[error fatal].include?(offense['severity'])
+          fail(grouped_offense_message, **kargs)
+        else
+          warn(grouped_offense_message, **kargs)
+        end
+      end
+    end
+
     def fetch_files_to_lint(files = nil)
       to_lint = if files.nil?
         # when files are renamed, git.modified_files contains the old name not the new one, so we need to do the convertion

data/lib/version.rb

@@ -1,3 +1,3 @@
 module DangerRubocop
-  VERSION = '0.12.0'.freeze
+  VERSION = '0.13.0'.freeze
 end

data/spec/danger_plugin_spec.rb

@@ -323,6 +323,81 @@ EOS
                 .to eq("Violation Syntax/WhetherYouShouldDoThat: Don't do that! { sticky: false, file: spec/fixtures/ruby_file.rb, line: 13, type: error }")
             end
           end
+
+          context 'with group_inline_comments' do
+            context 'with multiple violations on the same line' do
+              let(:response_ruby_file) do
+                {
+                  'files' => [
+                    {
+                      'path' => 'spec/fixtures/ruby_file.rb',
+                      'offenses' => [
+                        {
+                          'cop_name' => 'Syntax/WhetherYouShouldDoThat',
+                          'message' => "Don't do that!",
+                          'severity' => 'warning',
+                          'location' => { 'line' => 13 }
+                        },
+                        {
+                          'cop_name' => 'Syntax/WhetherYouShouldDoThat',
+                          'message' => "Also don't do that!",
+                          'severity' => 'warning',
+                          'location' => { 'line' => 13 }
+                        }
+                      ]
+                    }
+                  ]
+                }.to_json
+              end
+
+              it 'reports multiple violations grouped by line in a bulleted list' do
+                allow(@rubocop.git).to receive(:modified_files)
+                  .and_return(['spec/fixtures/ruby_file.rb'])
+                allow(@rubocop.git).to receive(:added_files).and_return([])
+                allow(@rubocop.git).to receive(:renamed_files).and_return([])
+                allow(@rubocop).to receive(:`)
+                  .with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
+                  .and_return(response_ruby_file)
+
+                @rubocop.lint(inline_comment: true, group_inline_comments: true)
+
+                warning = @rubocop.violation_report[:warnings].first
+                expect(warning.file)
+                  .to eq("spec/fixtures/ruby_file.rb")
+                expect(warning.line)
+                  .to eq(13)
+                expect(warning.message)
+                  .to eq(
+                    <<~HEREDOC
+
+                      * Don't do that!
+                      * Also don't do that!
+                    HEREDOC
+                  .chomp
+                  )
+              end
+            end
+
+            it 'reports single violations grouped by line as normal line by line warnings' do
+              allow(@rubocop.git).to receive(:modified_files)
+                .and_return(['spec/fixtures/ruby_file.rb'])
+              allow(@rubocop.git).to receive(:added_files).and_return([])
+              allow(@rubocop.git).to receive(:renamed_files).and_return([])
+              allow(@rubocop).to receive(:`)
+                .with('bundle exec rubocop -f json --only-recognized-file-types spec/fixtures/ruby_file.rb')
+                .and_return(response_ruby_file)
+
+              @rubocop.lint(inline_comment: true, group_inline_comments: true)
+
+              warning = @rubocop.violation_report[:warnings].first
+              expect(warning.file)
+                .to eq("spec/fixtures/ruby_file.rb")
+              expect(warning.line)
+                .to eq(13)
+              expect(warning.message)
+                .to eq("Don't do that!")
+            end
+          end
         end
 
         context 'with report_severity option' do
@@ -374,6 +449,34 @@ EOS
           end
         end
 
+        context 'using Bundler' do
+          it 'uses `bundle exec` when there is a Gemfile' do
+            allow(@rubocop).to receive(:`)
+              .with('bundle exec rubocop -f json --only-recognized-file-types --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
+              .and_return(response_ruby_file)
+
+            @rubocop.lint(files: 'spec/fixtures/ruby*.rb', config: 'path/to/rubocop.yml')
+          end
+
+          it 'doesn\'t use `bundle exec` when there is no Gemfile' do
+            allow(File).to receive(:exist?).with('Gemfile').and_return(false)
+
+            allow(@rubocop).to receive(:`)
+              .with('rubocop -f json --only-recognized-file-types --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
+              .and_return(response_ruby_file)
+
+            @rubocop.lint(files: 'spec/fixtures/ruby*.rb', config: 'path/to/rubocop.yml')
+          end
+
+          it 'doesn\'t use `bundle exec` when there is a Gemfile but skip_bundle_exec is true' do
+            allow(@rubocop).to receive(:`)
+              .with('rubocop -f json --only-recognized-file-types --config path/to/rubocop.yml spec/fixtures/ruby_file.rb')
+              .and_return(response_ruby_file)
+
+            @rubocop.lint(files: 'spec/fixtures/ruby*.rb', config: 'path/to/rubocop.yml', skip_bundle_exec: true)
+          end
+        end
+
         describe 'a filename with special characters' do
           it 'is shell escaped' do
             modified_files = [

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-rubocop
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.13.0
 platform: ruby
 authors:
 - Ash Furrow
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-01 00:00:00.000000000 Z
+date: 2024-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger