1 files changed, 192 insertions, 0 deletions
diff --git a/spec/tooling/danger/base_linter_spec.rb b/spec/tooling/danger/base_linter_spec.rb
new file mode 100644
index 00000000000..54d8f3dc1f7
--- /dev/null
+++ b/spec/tooling/danger/base_linter_spec.rb
@@ -0,0 +1,192 @@
+# frozen_string_literal: true
+
+require 'rspec-parameterized'
+require_relative 'danger_spec_helper'
+
+require_relative '../../../tooling/danger/base_linter'
+
+RSpec.describe Tooling::Danger::BaseLinter do
+  let(:commit_class) do
+    Struct.new(:message, :sha, :diff_parent)
+  end
+
+  let(:commit_message) { 'A commit message' }
+  let(:commit) { commit_class.new(commit_message, anything, anything) }
+
+  subject(:commit_linter) { described_class.new(commit) }
+
+  describe '#failed?' do
+    context 'with no failures' do
+      it { expect(commit_linter).not_to be_failed }
+    end
+
+    context 'with failures' do
+      before do
+        commit_linter.add_problem(:subject_too_long, described_class.subject_description)
+      end
+
+      it { expect(commit_linter).to be_failed }
+    end
+  end
+
+  describe '#add_problem' do
+    it 'stores messages in #failures' do
+      commit_linter.add_problem(:subject_too_long, '%s')
+
+      expect(commit_linter.problems).to eq({ subject_too_long: described_class.problems_mapping[:subject_too_long] })
+    end
+  end
+
+  shared_examples 'a valid commit' do
+    it 'does not have any problem' do
+      commit_linter.lint_subject
+
+      expect(commit_linter.problems).to be_empty
+    end
+  end
+
+  describe '#lint_subject' do
+    context 'when subject valid' do
+      it_behaves_like 'a valid commit'
+    end
+
+    context 'when subject is too short' do
+      let(:commit_message) { 'A B' }
+
+      it 'adds a problem' do
+        expect(commit_linter).to receive(:add_problem).with(:subject_too_short, described_class.subject_description)
+
+        commit_linter.lint_subject
+      end
+    end
+
+    context 'when subject is too long' do
+      let(:commit_message) { 'A B ' + 'C' * described_class::MAX_LINE_LENGTH }
+
+      it 'adds a problem' do
+        expect(commit_linter).to receive(:add_problem).with(:subject_too_long, described_class.subject_description)
+
+        commit_linter.lint_subject
+      end
+    end
+
+    context 'when ignoring length issues for subject having not-ready wording' do
+      using RSpec::Parameterized::TableSyntax
+
+      let(:final_message) { 'A B C' }
+
+      context 'when used as prefix' do
+        where(prefix: [
+          'WIP: ',
+          'WIP:',
+          'wIp:',
+          '[WIP] ',
+          '[WIP]',
+          '[draft]',
+          '[draft] ',
+          '(draft)',
+          '(draft) ',
+          'draft - ',
+          'draft: ',
+          'draft:',
+          'DRAFT:'
+        ])
+
+        with_them do
+          it 'does not have any problems' do
+            commit_message = prefix + final_message + 'D' * (described_class::MAX_LINE_LENGTH - final_message.size)
+            commit = commit_class.new(commit_message, anything, anything)
+
+            linter = described_class.new(commit).lint_subject
+
+            expect(linter.problems).to be_empty
+          end
+        end
+      end
+
+      context 'when used as suffix' do
+        where(suffix: %w[WIP draft])
+
+        with_them do
+          it 'does not have any problems' do
+            commit_message = final_message + 'D' * (described_class::MAX_LINE_LENGTH - final_message.size) + suffix
+            commit = commit_class.new(commit_message, anything, anything)
+
+            linter = described_class.new(commit).lint_subject
+
+            expect(linter.problems).to be_empty
+          end
+        end
+      end
+    end
+
+    context 'when subject does not have enough words and is too long' do
+      let(:commit_message) { 'A ' + 'B' * described_class::MAX_LINE_LENGTH }
+
+      it 'adds a problem' do
+        expect(commit_linter).to receive(:add_problem).with(:subject_too_short, described_class.subject_description)
+        expect(commit_linter).to receive(:add_problem).with(:subject_too_long, described_class.subject_description)
+
+        commit_linter.lint_subject
+      end
+    end
+
+    context 'when subject starts with lowercase' do
+      let(:commit_message) { 'a B C' }
+
+      it 'adds a problem' do
+        expect(commit_linter).to receive(:add_problem).with(:subject_starts_with_lowercase, described_class.subject_description)
+
+        commit_linter.lint_subject
+      end
+    end
+
+    [
+      '[ci skip] A commit message',
+      '[Ci skip] A commit message',
+      '[API] A commit message',
+      'api: A commit message',
+      'API: A commit message',
+      'API: a commit message',
+      'API: a commit message'
+    ].each do |message|
+      context "when subject is '#{message}'" do
+        let(:commit_message) { message }
+
+        it 'does not add a problem' do
+          expect(commit_linter).not_to receive(:add_problem)
+
+          commit_linter.lint_subject
+        end
+      end
+    end
+
+    [
+      '[ci skip]A commit message',
+      '[Ci skip]  A commit message',
+      '[ci skip] a commit message',
+      'api: a commit message',
+      '! A commit message'
+    ].each do |message|
+      context "when subject is '#{message}'" do
+        let(:commit_message) { message }
+
+        it 'adds a problem' do
+          expect(commit_linter).to receive(:add_problem).with(:subject_starts_with_lowercase, described_class.subject_description)
+
+          commit_linter.lint_subject
+        end
+      end
+    end
+
+    context 'when subject ends with a period' do
+      let(:commit_message) { 'A B C.' }
+
+      it 'adds a problem' do
+        expect(commit_linter).to receive(:add_problem).with(:subject_ends_with_a_period, described_class.subject_description)
+
+        commit_linter.lint_subject
+      end
+    end
+  end
+end