File: settings_sections_spec.rb

package info (click to toggle)
gitlab 17.6.5-19
  • links: PTS, VCS
  • area: main
  • in suites: sid
  • size: 629,368 kB
  • sloc: ruby: 1,915,304; javascript: 557,307; sql: 60,639; xml: 6,509; sh: 4,567; makefile: 1,239; python: 406
file content (66 lines) | stat: -rw-r--r-- 2,394 bytes parent folder | download
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
# frozen_string_literal: true

require 'gitlab/dangerfiles/spec_helper'
require 'fast_spec_helper'

require_relative '../../../tooling/danger/settings_sections'

RSpec.describe Tooling::Danger::SettingsSections, feature_category: :tooling do
  include_context 'with dangerfile'

  subject(:settings_section_check) { fake_danger.new(helper: fake_helper) }

  let(:fake_danger) { DangerSpecHelper.fake_danger.include(described_class) }
  let(:matching_changed_files) { ['app/views/foo/bar.html.haml', 'app/assets/js/foo/bar.vue'] }
  let(:changed_lines) { ['-render SettingsBlockComponent.new(id: "foo") do', '<settings-section id="foo">'] }
  let(:stable_branch?) { false }

  before do
    allow(fake_helper).to receive(:changed_files).and_return(matching_changed_files)
    allow(fake_helper).to receive(:changed_lines).and_return(changed_lines)
    allow(fake_helper).to receive(:stable_branch?).and_return(stable_branch?)
  end

  context 'when on stable branch' do
    let(:stable_branch?) { true }

    it 'does not write any markdown' do
      expect(settings_section_check).not_to receive(:markdown)
      settings_section_check.check!
    end
  end

  context 'when none of the changed files are Haml or Vue files' do
    let(:matching_changed_files) { [] }

    it 'does not write any markdown' do
      expect(settings_section_check).not_to receive(:markdown)
      settings_section_check.check!
    end
  end

  context 'when none of the changed lines match the pattern' do
    let(:changed_lines) { ['-foo', '+bar'] }

    it 'does not write any markdown' do
      expect(settings_section_check).not_to receive(:markdown)
      settings_section_check.check!
    end
  end

  context 'when some files match the pattern but in ignored folders' do
    let(:matching_changed_files) { ['app/views/admin/foo/bar.html.haml', 'ee/app/views/profiles/foo/bar.html.haml'] }

    it 'does not write any markdown' do
      expect(settings_section_check).not_to receive(:markdown)
      settings_section_check.check!
    end
  end

  it 'adds a new markdown section listing every matching line' do
    expect(settings_section_check).to receive(:markdown).with(/Searchable setting sections/)
    expect(settings_section_check).to receive(:markdown).with(/SettingsBlock/)
    expect(settings_section_check).to receive(:markdown).with(/settings-section/)
    settings_section_check.check!
  end
end