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
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
|
# frozen_string_literal: true
require 'fast_spec_helper'
require 'stringio'
require_relative '../support/helpers/next_instance_of'
require_relative '../../rubocop/check_graceful_task'
RSpec.describe RuboCop::CheckGracefulTask do
include NextInstanceOf
let(:output) { StringIO.new }
subject(:task) { described_class.new(output) }
describe '#run' do
let(:status_success) { RuboCop::CLI::STATUS_SUCCESS }
let(:status_offenses) { RuboCop::CLI::STATUS_OFFENSES }
let(:rubocop_status) { status_success }
let(:adjusted_rubocop_status) { rubocop_status }
subject { task.run(args) }
before do
# Don't notify Slack accidentally.
allow(Gitlab::Popen).to receive(:popen).and_raise('Notifications forbidden.')
stub_const('ENV', ENV.to_hash.delete_if { |key, _| key.start_with?('CI_') })
allow_next_instance_of(RuboCop::CLI) do |cli|
allow(cli).to receive(:run).and_return(rubocop_status)
end
allow(RuboCop::Formatter::GracefulFormatter)
.to receive(:adjusted_exit_status).and_return(adjusted_rubocop_status)
end
shared_examples 'rubocop scan' do |rubocop_args:|
it 'invokes a RuboCop scan' do
rubocop_options = %w[--parallel --format RuboCop::Formatter::GracefulFormatter]
rubocop_options.concat(rubocop_args)
expect_next_instance_of(RuboCop::CLI) do |cli|
expect(cli).to receive(:run).with(rubocop_options).and_return(rubocop_status)
end
subject
expect(output.string)
.to include('Running RuboCop in graceful mode:')
.and include("rubocop #{rubocop_options.join(' ')}")
.and include('This might take a while...')
end
end
context 'without args' do
let(:args) { [] }
it_behaves_like 'rubocop scan', rubocop_args: []
context 'with adjusted rubocop status' do
let(:rubocop_status) { status_offenses }
let(:adjusted_rubocop_status) { status_success }
context 'with sufficient environment variables' do
let(:script) { 'scripts/slack' }
let(:channel) { 'f_rubocop' }
let(:emoji) { 'rubocop' }
let(:user_name) { 'GitLab Bot' }
let(:job_name) { 'some job name' }
let(:job_url) { 'some job url' }
let(:docs_link) { 'https://docs.gitlab.com/ee/development/contributing/style_guides.html#silenced-offenses' }
before do
env = {
'CI_SLACK_WEBHOOK_URL' => 'webhook_url',
'CI_JOB_NAME' => job_name,
'CI_JOB_URL' => job_url
}
stub_const('ENV', ENV.to_hash.update(env))
end
it 'notifies slack' do
popen_result = ['', 0]
allow(Gitlab::Popen).to receive(:popen).with(anything).and_return(popen_result)
subject
message = a_kind_of(String).and include(job_name).and include(job_url).and include(docs_link)
expect(Gitlab::Popen).to have_received(:popen)
.with([script, channel, message, emoji, user_name])
expect(output.string).to include("Notifying Slack ##{channel}.")
end
context 'with when notification fails' do
it 'prints that notification failed' do
popen_result = ['', 1]
expect(Gitlab::Popen).to receive(:popen).and_return(popen_result)
subject
expect(output.string).to include("Failed to notify Slack channel ##{channel}.")
end
end
end
context 'with missing environment variables' do
it 'skips slack notification' do
expect(Gitlab::Popen).not_to receive(:popen)
subject
expect(output.string).to include('Skipping Slack notification.')
end
end
end
end
context 'with args' do
let(:args) { %w[a.rb Lint/EmptyFile b.rb Lint/Syntax] }
it_behaves_like 'rubocop scan', rubocop_args: %w[--only Lint/EmptyFile,Lint/Syntax a.rb b.rb]
it 'does not notify slack' do
expect(Gitlab::Popen).not_to receive(:popen)
subject
expect(output.string).not_to include('Skipping Slack notification.')
end
end
end
end
|