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
|
# frozen_string_literal: true
# Requires sentry_api_request and subject to be defined
RSpec.shared_examples 'calls sentry api' do
it 'calls sentry api' do
subject
expect(sentry_api_request).to have_been_requested
end
end
# Requires sentry_api_url and subject to be defined
RSpec.shared_examples 'no Sentry redirects' do |http_method|
let(:redirect_to) { 'https://redirected.example.com' }
let(:other_url) { 'https://other.example.org' }
let!(:redirected_req_stub) { stub_sentry_request(other_url) }
let!(:redirect_req_stub) do
stub_sentry_request(
sentry_api_url,
http_method || :get,
status: 302,
headers: { location: redirect_to }
)
end
it 'does not follow redirects' do
expect { subject }.to raise_exception(ErrorTracking::SentryClient::Error, 'Sentry response status code: 302')
expect(redirect_req_stub).to have_been_requested
expect(redirected_req_stub).not_to have_been_requested
end
end
RSpec.shared_examples 'maps Sentry exceptions' do |http_method|
exceptions = {
Gitlab::HTTP::Error => 'Error when connecting to Sentry',
Net::OpenTimeout => 'Connection to Sentry timed out',
SocketError => 'Received SocketError when trying to connect to Sentry',
OpenSSL::SSL::SSLError => 'Sentry returned invalid SSL data',
Errno::ECONNREFUSED => 'Connection refused',
StandardError => 'Sentry request failed due to StandardError'
}
exceptions.each do |exception, message|
context exception do
before do
stub_request(
http_method || :get,
sentry_request_url
).to_raise(exception)
end
it do
expect { subject }
.to raise_exception(ErrorTracking::SentryClient::Error, message)
end
end
end
end
RSpec.shared_examples 'non-numeric input handling in Sentry response' do |field|
context 'with non-numeric error id' do
where(:id_input) do
['string', '-1', '1\n2']
end
with_them do
it 'raises exception' do
message = %(Sentry API response contains invalid value for field "#{field}": #{id_input.inspect} is not numeric)
expect { subject }.to raise_error(ErrorTracking::SentryClient::InvalidFieldValueError, message)
end
end
end
end
# Expects to following variables:
# - subject
# - sentry_api_response
# - sentry_url, token
RSpec.shared_examples 'Sentry API response size limit' do
let(:response_body) { Gitlab::Json.generate(sentry_api_response) }
context 'when response body is within limit' do
before do
allow(Gitlab::Utils::DeepSize).to receive(:new).and_call_original
end
it 'checks parsed response' do
subject
expect(Gitlab::Utils::DeepSize).to have_received(:new)
end
end
context 'when response body is too large' do
before do
stub_const('ErrorTracking::SentryClient::RESPONSE_SIZE_LIMIT', response_body.bytesize - 1)
stub_const('ErrorTracking::SentryClient::RESPONSE_MEMORY_SIZE_LIMIT', 1)
end
it 'raises an exception' do
expect { subject }.to raise_error(
ErrorTracking::SentryClient::ResponseInvalidSizeError,
/Sentry API response is too big\. Limit is \d+(\.\d+)? \w+\. Got \d+ bytes\./
)
end
end
context 'when resulting memory size of the parsed response is too large' do
before do
stub_const('ErrorTracking::SentryClient::RESPONSE_SIZE_LIMIT', response_body.bytesize + 1)
stub_const('ErrorTracking::SentryClient::RESPONSE_MEMORY_SIZE_LIMIT', 1)
end
it 'raises an exception' do
expect { subject }.to raise_error(
ErrorTracking::SentryClient::ResponseInvalidSizeError,
/Sentry API response memory footprint is too big. Limit is \d+(\.\d+)? \w+\./
)
end
end
end
|
