danger-wcc 0.0.2

data/spec/fixtures/reek/reek.diff

@@ -0,0 +1,50 @@
+diff --git a/initial.tmp b/final.tmp
+index feafe47..97e1eb3 100644
+--- a/initial.tmp
++++ b/final.tmp
+@@ -23,6 +23,10 @@ lib/wcc/github.rb -- 3 warnings:
+   DuplicateMethodCall: Github#add_labels calls 'github.api' 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+   IrresponsibleModule: Github has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
+   UncommunicativeVariableName: Github#add_labels has the variable name 'l' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
++lib/wcc/plugin.rb -- 3 warnings:
++  TooManyStatements: Danger::DangerWCC#reek has approx 7 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
++  UncommunicativeVariableName: Danger::DangerWCC#reek has the variable name 'l' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
++  UncommunicativeVariableName: Danger::DangerWCC#reek has the variable name 'm' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+ lib/wcc/rubocop_exceptions.rb -- 25 warnings:
+   DuplicateMethodCall: Danger::DangerWCC::RubocopExceptions#build_message calls 'e[:context_lines]' 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+   DuplicateMethodCall: Danger::DangerWCC::RubocopExceptions#find_context calls 'l.content' 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+@@ -68,7 +72,7 @@ lib/wcc/todos.rb -- 18 warnings:
+   UncommunicativeVariableName: Danger::DangerWCC::Todos#find_link_in_context has the variable name 'l' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+   UncommunicativeVariableName: Danger::DangerWCC::Todos#find_link_in_context has the variable name 'm' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+   UtilityFunction: Danger::DangerWCC::Todos#max doesn't depend on instance state (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Utility-Function.md]
+-lib/wcc/utils.rb -- 15 warnings:
++lib/wcc/utils.rb -- 17 warnings:
+   ControlParameter: Utils#issue is controlled by argument 'severity' [https://github.com/troessner/reek/blob/master/docs/Control-Parameter.md]
+   DuplicateMethodCall: Utils#find_in_diff calls 'l.content' 2 times [https://github.com/troessner/reek/blob/master/docs/Duplicate-Method-Call.md]
+   FeatureEnvy: Utils#find_in_diff refers to 'l' more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
+@@ -79,8 +83,10 @@ lib/wcc/utils.rb -- 15 warnings:
+   NestedIterators: Utils#each_addition_in_diff contains iterators nested 3 deep [https://github.com/troessner/reek/blob/master/docs/Nested-Iterators.md]
+   NestedIterators: Utils#each_file_in_diff contains iterators nested 2 deep [https://github.com/troessner/reek/blob/master/docs/Nested-Iterators.md]
+   NestedIterators: Utils#find_in_diff contains iterators nested 3 deep [https://github.com/troessner/reek/blob/master/docs/Nested-Iterators.md]
++  NilCheck: Utils#run_and_diff performs a nil-check [https://github.com/troessner/reek/blob/master/docs/Nil-Check.md]
+   TooManyStatements: Utils#each_addition_in_diff has approx 6 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+   TooManyStatements: Utils#find_in_diff has approx 7 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
++  TooManyStatements: Utils#run_and_diff has approx 18 statements [https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
+   UncommunicativeVariableName: Utils#each_addition_in_diff has the variable name 'l' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+   UncommunicativeVariableName: Utils#find_in_diff has the variable name 'l' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+   UncommunicativeVariableName: Utils#parsed_diffs has the variable name 'd' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+@@ -101,6 +107,8 @@ spec/wcc/commit_lint_spec.rb -- 3 warnings:
+   UtilityFunction: report_counts doesn't depend on instance state (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Utility-Function.md]
+ spec/wcc/github_spec.rb -- 1 warning:
+   IrresponsibleModule: Danger has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
++spec/wcc/plugin_spec.rb -- 1 warning:
++  IrresponsibleModule: Danger has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
+ spec/wcc/rubocop_exceptions_spec.rb -- 1 warning:
+   IrresponsibleModule: Danger has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
+ spec/wcc/todos_spec.rb -- 1 warning:
+@@ -110,4 +118,4 @@ spec/wcc/utils_spec.rb -- 2 warnings:
+   UncommunicativeVariableName: Danger has the variable name 'l' [https://github.com/troessner/reek/blob/master/docs/Uncommunicative-Variable-Name.md]
+ spec/wcc_spec.rb -- 1 warning:
+   IrresponsibleModule: Danger has no descriptive comment [https://github.com/troessner/reek/blob/master/docs/Irresponsible-Module.md]
+-91 total warnings
++97 total warnings

data/spec/fixtures/rubocop_exception.rb

@@ -0,0 +1,39 @@
+
+# frozen_string_literal: true
+
+module RubocopException
+  # rubocop:disable Rails/SomeRule
+  def some_method_with_disabled_rule
+  end
+  # rubocop:enable Rails/SomeRule
+
+  # rubocop:disable Lint/SomeLint
+  def some_method_misspelled_reenable
+  end
+  # rubocop:enable Lint/SomLint
+
+  # rubocop:disable Metrics/LineLength
+  # Disabling because I want to.
+  def some_method_with_context
+  end
+  # rubocop:enable Metrics/LineLength
+
+  # rubocop:disable Metrics/ABC
+  # here is my comment why we are disabling ABC
+  # Some existing comments
+  # Made by another person
+  def some_method_with_existing_comments
+  end
+  # rubocop:enable Metrics/ABC
+
+  def some_method_with_multiline_context
+    # rubocop:disable Layout/AlignHash
+    # I want to disable this because..
+    # I forgot the reasons.
+    {
+      a: 'a',
+        b: 'b'
+    }
+    # rubocop:enable Layout/AlignHash
+  end
+end

data/spec/fixtures/todo.rb

@@ -0,0 +1,21 @@
+
+
+# frozen_string_literal: true
+
+module Todo
+  def this_should_not_trigger_todo
+  end
+
+  # TODO: fix this later
+  def todo_without_comment
+  end
+
+  # TODO: https://zube.io/watermark/asdf1234
+  def todo_with_link_same_line
+  end
+
+  # TODO: fix this when things are right
+  # https://www.github.com/watermarkchurch/asdf/1234
+  def todo_with_link_below
+  end
+end

data/spec/fixtures/todo_link_next_line.diff

@@ -0,0 +1,14 @@
+diff --git a/spec/fixtures/todo.rb b/spec/fixtures/todo.rb
+index 08d4e15..16dc3b5 100644
+--- a/spec/fixtures/todo.rb
++++ b/spec/fixtures/todo.rb
+@@ -13,4 +13,9 @@ module Todo
+   # TODO: https://zube.io/watermark/asdf1234
+   def todo_with_link_same_line
+   end
++
++  # TODO: fix this when things are right
++  # https://www.github.com/watermarkchurch/asdf/1234
++  def todo_with_link_below
++  end
+ end

data/spec/fixtures/todo_link_same_line.diff

@@ -0,0 +1,13 @@
+diff --git a/spec/fixtures/todo.rb b/spec/fixtures/todo.rb
+index f141004..08d4e15 100644
+--- a/spec/fixtures/todo.rb
++++ b/spec/fixtures/todo.rb
+@@ -9,4 +9,8 @@ module Todo
+   # TODO: fix this later
+   def todo_without_comment
+   end
++
++  # TODO: https://zube.io/watermark/asdf1234
++  def todo_with_link_same_line
++  end
+ end

data/spec/fixtures/todo_no_link.diff

@@ -0,0 +1,13 @@
+diff --git a/spec/fixtures/todo.rb b/spec/fixtures/todo.rb
+index 22ca05f..f141004 100644
+--- a/spec/fixtures/todo.rb
++++ b/spec/fixtures/todo.rb
+@@ -5,4 +5,8 @@
+ module Todo
+   def this_should_not_trigger_todo
+   end
++
++  # TODO: fix this later
++  def todo_without_comment
++  end
+ end

data/spec/fixtures/todo_removed.diff

@@ -0,0 +1,13 @@
+diff --git a/spec/fixtures/todo.rb b/spec/fixtures/todo.rb
+index 16dc3b5..e4fada3 100644
+--- a/spec/fixtures/todo.rb
++++ b/spec/fixtures/todo.rb
+@@ -14,8 +14,6 @@ module Todo
+   def todo_with_link_same_line
+   end
+ 
+-  # TODO: fix this when things are right
+-  # https://www.github.com/watermarkchurch/asdf/1234
+   def todo_with_link_below
+   end
+ end

data/spec/fixtures_helper.rb

@@ -0,0 +1,19 @@
+
+# frozen_string_literal: true
+
+module FixturesHelper
+  def load_diff(file_path, fixture_name)
+    raw_diff = load_fixture(fixture_name + '.diff')
+    return unless raw_diff
+
+    double(
+      patch: raw_diff,
+      path: file_path
+    )
+  end
+
+  def load_fixture(file_name)
+    file = "#{File.dirname(__FILE__)}/fixtures/#{file_name}"
+    return File.read(file) if File.exist?(file)
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+require 'pathname'
+ROOT = Pathname.new(File.expand_path('../../', __FILE__))
+$LOAD_PATH.unshift((ROOT + 'lib').to_s)
+$LOAD_PATH.unshift((ROOT + 'spec').to_s)
+
+require 'bundler/setup'
+require 'pry'
+
+require 'rspec'
+require 'danger'
+require 'webmock/rspec'
+
+if `git remote -v` == ''
+  puts 'You cannot run tests without setting a local git remote on this repo'
+  puts "It's a weird side-effect of Danger's internals."
+  exit(0)
+end
+
+require 'fixtures_helper'
+
+# Use coloured output, it's the best.
+RSpec.configure do |config|
+  config.filter_gems_from_backtrace 'bundler'
+  config.color = true
+  config.tty = true
+  config.include FixturesHelper
+end
+
+require 'danger_plugin'
+
+# These functions are a subset of https://github.com/danger/danger/blob/master/spec/spec_helper.rb
+# If you are expanding these files, see if it's already been done ^.
+
+# A silent version of the user interface,
+# it comes with an extra function `.string` which will
+# strip all ANSI colours from the string.
+
+# rubocop:disable Lint/NestedMethodDefinition
+def testing_ui
+  @output = StringIO.new
+  def @output.winsize
+    [20, 9999]
+  end
+
+  cork = Cork::Board.new(out: @output)
+  def cork.string
+    out.string.gsub(/\e\[([;\d]+)?m/, '')
+  end
+  cork
+end
+# rubocop:enable Lint/NestedMethodDefinition
+
+# Example environment (ENV) that would come from
+# running a PR on TravisCI
+def testing_env
+  {
+    'HAS_JOSH_K_SEAL_OF_APPROVAL' => 'true',
+    'TRAVIS_PULL_REQUEST' => '800',
+    'TRAVIS_REPO_SLUG' => 'artsy/eigen',
+    'TRAVIS_COMMIT_RANGE' => '759adcbd0d8f...13c4dc8bb61d',
+    'DANGER_GITHUB_API_TOKEN' => '123sbdq54erfsd3422gdfio'
+  }
+end
+
+# A stubbed out Dangerfile for use in tests
+def testing_dangerfile
+  env = Danger::EnvironmentManager.new(testing_env)
+  Danger::Dangerfile.new(env, testing_ui)
+end
+
+WebMock.disable_net_connect!(allow_localhost: true)

data/spec/wcc/commit_lint_spec.rb

@@ -0,0 +1,392 @@
+# frozen_string_literal: true
+
+# Copied and modified from
+# https://github.com/jonallured/danger-commit_lint
+# which is licensed under the MIT License
+
+require File.expand_path('../../spec_helper', __FILE__)
+
+TEST_MESSAGES = {
+  subject_cap: 'this subject needs a capital',
+  subject_words: 'Fixed',
+  subject_length: 'This is a really long subject line and should result in an'\
+    ' error',
+  subject_period: 'This subject line ends in a period.',
+  empty_line: "This subject line is fine\nBut then I forgot the empty line '\
+    'separating the subject and the body.",
+  all_errors: "this is a really long subject and it even ends in a period.\n'\
+    'Not to mention the missing empty line!",
+  valid:  "This is a valid message\n\nYou can tell because it meets all the '\
+    'criteria and the linter does not complain."
+}.freeze
+
+NOOP_MESSAGE = Danger::DangerWCC::CommitLint::NOOP_MESSAGE
+SUBJECT_LENGTH_CHECK_MESSAGE = 'Please limit commit subject line to 50 '\
+  'characters.'
+
+# rubocop:enable Metrics/LineLength
+
+def report_counts(status_report)
+  status_report.values.flatten.count
+end
+
+module Danger
+  describe Danger::DangerWCC do
+    before do
+      @dangerfile = testing_dangerfile
+      @my_plugin = @dangerfile.wcc
+      @git = @dangerfile.git
+    end
+
+    def message_with_sha(message)
+      [message, sha].join "\n"
+    end
+
+    def commit_lint(options = {})
+      Danger::DangerWCC::CommitLint.new(@my_plugin, options).perform
+    end
+
+    describe 'commit lint' do
+      describe 'check without configuration' do
+        let(:sha) { '1234567' }
+        let(:commit) { double(:commit, message: message, sha: sha) }
+
+        context 'with invalid messages' do
+          it 'fails those checks' do
+            checks = {
+              subject_cap: SubjectCapCheck::MESSAGE,
+              subject_words: SubjectWordsCheck::MESSAGE,
+              subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+              subject_period: SubjectPeriodCheck::MESSAGE,
+              empty_line: EmptyLineCheck::MESSAGE
+            }
+
+            checks.each do |check, warning|
+              commit = double(:commit, message: TEST_MESSAGES[check], sha: sha)
+              allow(@git).to receive(:commits).and_return([commit])
+
+              expect {
+                commit_lint
+              }.to change { @my_plugin.status_report[:errors].count }.by(1)
+
+              status_report = @my_plugin.status_report
+              expect(status_report[:errors].last)
+                .to eq(message_with_sha(warning))
+            end
+          end
+        end
+
+        context 'with all errors' do
+          let(:message) { TEST_MESSAGES[:all_errors] }
+
+          it 'fails every check' do
+            allow(@git).to receive(:commits).and_return([commit])
+
+            commit_lint
+
+            status_report = @my_plugin.status_report
+            expect(report_counts(status_report)).to eq 4
+            expect(status_report[:errors]).to eq [
+              message_with_sha(SubjectCapCheck::MESSAGE),
+              message_with_sha(SUBJECT_LENGTH_CHECK_MESSAGE),
+              message_with_sha(SubjectPeriodCheck::MESSAGE),
+              message_with_sha(EmptyLineCheck::MESSAGE)
+            ]
+          end
+        end
+
+        context 'with valid messages' do
+          let(:message) { TEST_MESSAGES[:valid] }
+
+          it 'does nothing' do
+            checks = {
+              subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+              subject_period: SubjectPeriodCheck::MESSAGE,
+              empty_line: EmptyLineCheck::MESSAGE
+            }
+
+            checks.each do
+              allow(@git).to receive(:commits).and_return([commit])
+
+              commit_lint
+
+              status_report = @my_plugin.status_report
+              expect(report_counts(status_report)).to eq 0
+            end
+          end
+        end
+      end
+
+      describe 'disable configuration' do
+        let(:sha) { '1234567' }
+        let(:commit) { double(:commit, message: message, sha: sha) }
+
+        context 'with individual checks' do
+          context 'with invalid messages' do
+            it 'does nothing' do
+              checks = {
+                subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+                subject_period: SubjectPeriodCheck::MESSAGE,
+                empty_line: EmptyLineCheck::MESSAGE
+              }
+
+              checks.each do |check, _|
+                commit = double(:commit,
+                  message: TEST_MESSAGES[check], sha: sha)
+                allow(@git).to receive(:commits).and_return([commit])
+
+                commit_lint disable: [check]
+
+                status_report = @my_plugin.status_report
+                expect(report_counts(status_report)).to eq 0
+              end
+            end
+          end
+        end
+
+        context 'with all checks, implicitly' do
+          let(:message) { TEST_MESSAGES[:all_errors] }
+
+          it 'warns that nothing was checked' do
+            allow(@git).to receive(:commits).and_return([commit])
+
+            all_checks = %i[
+              subject_cap
+              subject_words
+              subject_length
+              subject_period
+              empty_line
+            ]
+            commit_lint disable: all_checks
+
+            status_report = @my_plugin.status_report
+            expect(report_counts(status_report)).to eq 1
+            expect(status_report[:warnings]).to eq [NOOP_MESSAGE]
+          end
+        end
+
+        context 'with all checks, explicitly' do
+          let(:message) { TEST_MESSAGES[:all_errors] }
+
+          it 'warns that nothing was checked' do
+            allow(@git).to receive(:commits).and_return([commit])
+
+            commit_lint disable: :all
+
+            status_report = @my_plugin.status_report
+            expect(report_counts(status_report)).to eq 1
+            expect(status_report[:warnings]).to eq [NOOP_MESSAGE]
+          end
+        end
+      end
+
+      describe 'warn configuration' do
+        let(:sha) { '1234567' }
+        let(:commit) { double(:commit, message: message, sha: sha) }
+
+        context 'with individual checks' do
+          context 'with invalid messages' do
+            it 'warns instead of failing' do
+              checks = {
+                subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+                subject_period: SubjectPeriodCheck::MESSAGE,
+                empty_line: EmptyLineCheck::MESSAGE
+              }
+
+              checks.each do |check, warning|
+                commit = double(:commit,
+                  message: TEST_MESSAGES[check], sha: sha)
+                allow(@git).to receive(:commits).and_return([commit])
+
+                expect {
+                  commit_lint warn: [check]
+                }.to change { @my_plugin.status_report[:warnings].count }.by(1)
+
+                status_report = @my_plugin.status_report
+                expect(status_report[:warnings].last)
+                  .to eq(message_with_sha(warning))
+              end
+            end
+          end
+
+          context 'with valid messages' do
+            let(:message) { TEST_MESSAGES[:valid] }
+
+            it 'does nothing' do
+              checks = {
+                subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+                subject_period: SubjectPeriodCheck::MESSAGE,
+                empty_line: EmptyLineCheck::MESSAGE
+              }
+
+              checks.each do |check, _|
+                allow(@git).to receive(:commits).and_return([commit])
+
+                commit_lint warn: [check]
+
+                status_report = @my_plugin.status_report
+                expect(report_counts(status_report)).to eq 0
+              end
+            end
+          end
+        end
+
+        context 'with all checks' do
+          context 'with all errors' do
+            let(:message) { TEST_MESSAGES[:all_errors] }
+
+            it 'warns instead of failing' do
+              allow(@git).to receive(:commits).and_return([commit])
+
+              commit_lint warn: :all
+
+              status_report = @my_plugin.status_report
+              expect(report_counts(status_report)).to eq 4
+              expect(status_report[:warnings]).to eq [
+                message_with_sha(SubjectCapCheck::MESSAGE),
+                message_with_sha(SUBJECT_LENGTH_CHECK_MESSAGE),
+                message_with_sha(SubjectPeriodCheck::MESSAGE),
+                message_with_sha(EmptyLineCheck::MESSAGE)
+              ]
+            end
+          end
+
+          context 'with a valid message' do
+            let(:message) { TEST_MESSAGES[:valid] }
+
+            it 'does nothing' do
+              allow(@git).to receive(:commits).and_return([commit])
+
+              commit_lint warn: :all
+
+              status_report = @my_plugin.status_report
+              expect(report_counts(status_report)).to eq 0
+            end
+          end
+        end
+      end
+
+      describe 'fail configuration' do
+        let(:sha) { '1234567' }
+        let(:commit) { double(:commit, message: message, sha: sha) }
+
+        context 'with individual checks' do
+          context 'with invalid messages' do
+            it 'fails those checks' do
+              checks = {
+                subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+                subject_period: SubjectPeriodCheck::MESSAGE,
+                empty_line: EmptyLineCheck::MESSAGE
+              }
+
+              checks.each do |check, warning|
+                commit = double(:commit,
+                  message: TEST_MESSAGES[check], sha: sha)
+                allow(@git).to receive(:commits).and_return([commit])
+
+                expect {
+                  commit_lint fail: [check]
+                }.to change { @my_plugin.status_report[:errors].count }.by(1)
+
+                status_report = @my_plugin.status_report
+
+                expect(status_report[:errors].last)
+                  .to eq(message_with_sha(warning))
+              end
+            end
+          end
+
+          context 'with valid messages' do
+            let(:message) { TEST_MESSAGES[:valid] }
+
+            it 'does nothing' do
+              checks = {
+                subject_length: SUBJECT_LENGTH_CHECK_MESSAGE,
+                subject_period: SubjectPeriodCheck::MESSAGE,
+                empty_line: EmptyLineCheck::MESSAGE
+              }
+
+              checks.each do |check, _|
+                allow(@git).to receive(:commits).and_return([commit])
+
+                commit_lint fail: [check]
+
+                status_report = @my_plugin.status_report
+                expect(report_counts(status_report)).to eq 0
+              end
+            end
+          end
+        end
+
+        context 'with all checks' do
+          context 'with all errors' do
+            let(:message) { TEST_MESSAGES[:all_errors] }
+
+            it 'fails those checks' do
+              allow(@git).to receive(:commits).and_return([commit])
+
+              commit_lint fail: :all
+
+              status_report = @my_plugin.status_report
+              expect(report_counts(status_report)).to eq 4
+              expect(status_report[:errors]).to eq [
+                message_with_sha(SubjectCapCheck::MESSAGE),
+                message_with_sha(SUBJECT_LENGTH_CHECK_MESSAGE),
+                message_with_sha(SubjectPeriodCheck::MESSAGE),
+                message_with_sha(EmptyLineCheck::MESSAGE)
+              ]
+            end
+          end
+
+          context 'with a valid message' do
+            let(:message) { TEST_MESSAGES[:valid] }
+
+            it 'does nothing' do
+              allow(@git).to receive(:commits).and_return([commit])
+
+              commit_lint fail: :all
+
+              status_report = @my_plugin.status_report
+              expect(report_counts(status_report)).to eq 0
+            end
+          end
+        end
+      end
+
+      describe 'individual check options' do
+        let(:sha) { '1234567' }
+
+        context 'with invalid messages for given options' do
+          it 'fails those checks' do
+            options = [
+              [:subject_length, { max: 10 }],
+              [:subject_length, { min: 100, max: 110 }]
+            ]
+            messages = [
+              'Please limit commit subject line to 10 characters.',
+              'Please write a commit subject line of at least 100 characters.'
+            ]
+
+            options.each_with_index do |(check, opts), i|
+              commit = double(:commit,
+                message: TEST_MESSAGES[:valid], sha: sha)
+              allow(@git).to receive(:commits).and_return([commit])
+
+              arg = {}
+              arg[check] = opts
+              expect {
+                commit_lint arg
+              }.to change { @my_plugin.status_report[:errors].count }.by(1)
+
+              status_report = @my_plugin.status_report
+              expect(status_report[:errors].last)
+                .to eq(message_with_sha(messages[i]))
+            end
+          end
+        end
+      end
+    end
+  end
+end
+
+# rubocop:enable Metrics/ClassLength