airbrake-ruby 4.9.0-java → 4.10.0-java
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/lib/airbrake-ruby.rb +55 -6
- data/lib/airbrake-ruby/async_sender.rb +3 -3
- data/lib/airbrake-ruby/backtrace.rb +2 -2
- data/lib/airbrake-ruby/code_hunk.rb +1 -1
- data/lib/airbrake-ruby/config.rb +1 -1
- data/lib/airbrake-ruby/config/validator.rb +3 -3
- data/lib/airbrake-ruby/deploy_notifier.rb +1 -1
- data/lib/airbrake-ruby/filters/exception_attributes_filter.rb +2 -2
- data/lib/airbrake-ruby/filters/git_last_checkout_filter.rb +2 -2
- data/lib/airbrake-ruby/filters/keys_filter.rb +1 -1
- data/lib/airbrake-ruby/filters/sql_filter.rb +3 -3
- data/lib/airbrake-ruby/filters/thread_filter.rb +1 -1
- data/lib/airbrake-ruby/inspectable.rb +2 -2
- data/lib/airbrake-ruby/notice.rb +7 -7
- data/lib/airbrake-ruby/notice_notifier.rb +1 -1
- data/lib/airbrake-ruby/performance_breakdown.rb +1 -1
- data/lib/airbrake-ruby/performance_notifier.rb +36 -20
- data/lib/airbrake-ruby/query.rb +1 -1
- data/lib/airbrake-ruby/queue.rb +2 -2
- data/lib/airbrake-ruby/request.rb +1 -1
- data/lib/airbrake-ruby/stat.rb +1 -1
- data/lib/airbrake-ruby/version.rb +1 -1
- data/spec/airbrake_spec.rb +107 -48
- data/spec/async_sender_spec.rb +4 -4
- data/spec/backtrace_spec.rb +18 -18
- data/spec/code_hunk_spec.rb +9 -9
- data/spec/config/validator_spec.rb +5 -5
- data/spec/config_spec.rb +5 -5
- data/spec/deploy_notifier_spec.rb +2 -2
- data/spec/filter_chain_spec.rb +1 -1
- data/spec/filters/dependency_filter_spec.rb +1 -1
- data/spec/filters/gem_root_filter_spec.rb +5 -5
- data/spec/filters/git_last_checkout_filter_spec.rb +1 -1
- data/spec/filters/git_repository_filter.rb +1 -1
- data/spec/filters/git_revision_filter_spec.rb +10 -10
- data/spec/filters/keys_blacklist_spec.rb +22 -22
- data/spec/filters/keys_whitelist_spec.rb +21 -21
- data/spec/filters/root_directory_filter_spec.rb +5 -5
- data/spec/filters/sql_filter_spec.rb +53 -53
- data/spec/filters/system_exit_filter_spec.rb +1 -1
- data/spec/filters/thread_filter_spec.rb +28 -28
- data/spec/fixtures/project_root/code.rb +9 -9
- data/spec/notice_notifier/options_spec.rb +12 -12
- data/spec/notice_notifier_spec.rb +17 -17
- data/spec/notice_spec.rb +5 -5
- data/spec/performance_notifier_spec.rb +88 -68
- data/spec/query_spec.rb +1 -1
- data/spec/request_spec.rb +1 -1
- data/spec/response_spec.rb +8 -8
- data/spec/spec_helper.rb +2 -2
- data/spec/stat_spec.rb +2 -2
- data/spec/sync_sender_spec.rb +12 -12
- data/spec/tdigest_spec.rb +6 -6
- data/spec/thread_pool_spec.rb +5 -5
- data/spec/timed_trace_spec.rb +1 -1
- data/spec/truncator_spec.rb +12 -12
- metadata +2 -2
@@ -15,7 +15,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
15
15
|
|
16
16
|
Airbrake::Config.instance = Airbrake::Config.new(
|
17
17
|
project_id: project_id,
|
18
|
-
project_key: project_key
|
18
|
+
project_key: project_key,
|
19
19
|
)
|
20
20
|
end
|
21
21
|
|
@@ -64,7 +64,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
64
64
|
describe ":root_directory" do
|
65
65
|
before do
|
66
66
|
subject.add_filter(
|
67
|
-
Airbrake::Filters::RootDirectoryFilter.new('/home/kyrylo/code')
|
67
|
+
Airbrake::Filters::RootDirectoryFilter.new('/home/kyrylo/code'),
|
68
68
|
)
|
69
69
|
end
|
70
70
|
|
@@ -73,7 +73,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
73
73
|
|
74
74
|
expect(
|
75
75
|
a_request(:post, endpoint)
|
76
|
-
.with(body: %r|{"file":"/PROJECT_ROOT/airbrake/ruby/spec/airbrake_spec.+|)
|
76
|
+
.with(body: %r|{"file":"/PROJECT_ROOT/airbrake/ruby/spec/airbrake_spec.+|),
|
77
77
|
).to have_been_made.once
|
78
78
|
end
|
79
79
|
|
@@ -85,7 +85,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
85
85
|
subject.notify_sync(ex)
|
86
86
|
expect(
|
87
87
|
a_request(:post, endpoint)
|
88
|
-
.with(body: %r{"rootDirectory":"/bingo/bango"})
|
88
|
+
.with(body: %r{"rootDirectory":"/bingo/bango"}),
|
89
89
|
).to have_been_made.once
|
90
90
|
end
|
91
91
|
end
|
@@ -105,7 +105,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
105
105
|
WEBrick::HTTPServer.new(
|
106
106
|
Port: 0,
|
107
107
|
Logger: WEBrick::Log.new('/dev/null'),
|
108
|
-
AccessLog: []
|
108
|
+
AccessLog: [],
|
109
109
|
)
|
110
110
|
end
|
111
111
|
|
@@ -121,7 +121,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
121
121
|
before do
|
122
122
|
Airbrake::Config.instance.merge(
|
123
123
|
proxy: proxy_params,
|
124
|
-
host: "http://localhost:#{proxy.config[:Port]}"
|
124
|
+
host: "http://localhost:#{proxy.config[:Port]}",
|
125
125
|
)
|
126
126
|
|
127
127
|
proxy.mount_proc '/' do |req, res|
|
@@ -139,7 +139,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
139
139
|
if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("2.6.0")
|
140
140
|
skip(
|
141
141
|
"We use Webmock 2, which doesn't support Ruby 2.6+. It's " \
|
142
|
-
"safe to run this test on 2.6+ once we upgrade to Webmock 3.5+"
|
142
|
+
"safe to run this test on 2.6+ once we upgrade to Webmock 3.5+",
|
143
143
|
)
|
144
144
|
end
|
145
145
|
subject.notify_sync(ex)
|
@@ -164,7 +164,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
164
164
|
subject.notify_sync(ex)
|
165
165
|
expect(
|
166
166
|
a_request(:post, endpoint)
|
167
|
-
.with(body: /"context":{.*"environment":"production".*}/)
|
167
|
+
.with(body: /"context":{.*"environment":"production".*}/),
|
168
168
|
).to have_been_made.once
|
169
169
|
end
|
170
170
|
end
|
@@ -192,7 +192,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
192
192
|
context "when env is set and ignore_environments doesn't mention it" do
|
193
193
|
params = {
|
194
194
|
environment: :development,
|
195
|
-
ignore_environments: [:production]
|
195
|
+
ignore_environments: [:production],
|
196
196
|
}
|
197
197
|
|
198
198
|
include_examples 'sent notice', params
|
@@ -201,7 +201,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
201
201
|
context "when the current env and notify envs are the same" do
|
202
202
|
params = {
|
203
203
|
environment: :development,
|
204
|
-
ignore_environments: %i[production development]
|
204
|
+
ignore_environments: %i[production development],
|
205
205
|
}
|
206
206
|
|
207
207
|
include_examples 'ignored notice', params
|
@@ -227,7 +227,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
227
227
|
context "when ignore_environments specifies a Regexp pattern" do
|
228
228
|
params = {
|
229
229
|
environment: :testing,
|
230
|
-
ignore_environments: ['staging', /test.+/]
|
230
|
+
ignore_environments: ['staging', /test.+/],
|
231
231
|
}
|
232
232
|
|
233
233
|
include_examples 'ignored notice', params
|
@@ -241,7 +241,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
241
241
|
before do
|
242
242
|
Airbrake::Config.instance.merge(
|
243
243
|
blacklist_keys: %i[password password_confirmation],
|
244
|
-
whitelist_keys: [:email, /user/i, 'account_id']
|
244
|
+
whitelist_keys: [:email, /user/i, 'account_id'],
|
245
245
|
)
|
246
246
|
end
|
247
247
|
|
@@ -4,7 +4,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
4
4
|
project_id: 1,
|
5
5
|
project_key: 'abc',
|
6
6
|
logger: Logger.new('/dev/null'),
|
7
|
-
performance_stats: true
|
7
|
+
performance_stats: true,
|
8
8
|
)
|
9
9
|
end
|
10
10
|
|
@@ -32,7 +32,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
32
32
|
let(:body) do
|
33
33
|
{
|
34
34
|
'id' => '00054414-b147-6ffa-85d6-1524d83362a6',
|
35
|
-
'url' => 'http://localhost/locate/00054414-b147-6ffa-85d6-1524d83362a6'
|
35
|
+
'url' => 'http://localhost/locate/00054414-b147-6ffa-85d6-1524d83362a6',
|
36
36
|
}.to_json
|
37
37
|
end
|
38
38
|
|
@@ -110,7 +110,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
110
110
|
before do
|
111
111
|
Airbrake::Config.instance.merge(
|
112
112
|
environment: 'test',
|
113
|
-
ignore_environments: %w[test]
|
113
|
+
ignore_environments: %w[test],
|
114
114
|
)
|
115
115
|
end
|
116
116
|
|
@@ -132,7 +132,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
132
132
|
let(:body) do
|
133
133
|
{
|
134
134
|
'id' => '00054414-b147-6ffa-85d6-1524d83362a6',
|
135
|
-
'url' => 'http://localhost/locate/00054414-b147-6ffa-85d6-1524d83362a6'
|
135
|
+
'url' => 'http://localhost/locate/00054414-b147-6ffa-85d6-1524d83362a6',
|
136
136
|
}
|
137
137
|
end
|
138
138
|
|
@@ -153,8 +153,8 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
153
153
|
subject.notify_sync('foo', bingo: 'bango')
|
154
154
|
expect(
|
155
155
|
a_request(:post, endpoint).with(
|
156
|
-
body: /"params":{.*"bingo":"bango".*}
|
157
|
-
)
|
156
|
+
body: /"params":{.*"bingo":"bango".*}/,
|
157
|
+
),
|
158
158
|
).to have_been_made.once
|
159
159
|
end
|
160
160
|
|
@@ -190,7 +190,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
190
190
|
context "when the provided environment is ignored" do
|
191
191
|
before do
|
192
192
|
Airbrake::Config.instance.merge(
|
193
|
-
environment: 'test', ignore_environments: %w[test]
|
193
|
+
environment: 'test', ignore_environments: %w[test],
|
194
194
|
)
|
195
195
|
end
|
196
196
|
|
@@ -256,7 +256,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
256
256
|
it "returns the full generated backtrace" do
|
257
257
|
backtrace = [
|
258
258
|
"/lib/airbrake-ruby/a.rb:84:in `build_notice'",
|
259
|
-
"/lib/airbrake-ruby/b.rb:124:in `send_notice'"
|
259
|
+
"/lib/airbrake-ruby/b.rb:124:in `send_notice'",
|
260
260
|
]
|
261
261
|
allow(Kernel).to receive(:caller).and_return(backtrace)
|
262
262
|
|
@@ -265,8 +265,8 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
265
265
|
expect(notice[:errors].first[:backtrace]).to eq(
|
266
266
|
[
|
267
267
|
{ file: '/lib/airbrake-ruby/a.rb', line: 84, function: 'build_notice' },
|
268
|
-
{ file: '/lib/airbrake-ruby/b.rb', line: 124, function: 'send_notice' }
|
269
|
-
]
|
268
|
+
{ file: '/lib/airbrake-ruby/b.rb', line: 124, function: 'send_notice' },
|
269
|
+
],
|
270
270
|
)
|
271
271
|
end
|
272
272
|
end
|
@@ -276,7 +276,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
276
276
|
backtrace = [
|
277
277
|
"/airbrake-ruby/lib/airbrake-ruby/a.rb:84:in `b'",
|
278
278
|
"/airbrake-ruby/lib/foo/b.rb:84:in `build'",
|
279
|
-
"/airbrake-ruby/lib/bar/c.rb:124:in `send'"
|
279
|
+
"/airbrake-ruby/lib/bar/c.rb:124:in `send'",
|
280
280
|
]
|
281
281
|
allow(Kernel).to receive(:caller).and_return(backtrace)
|
282
282
|
|
@@ -285,8 +285,8 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
285
285
|
expect(notice[:errors].first[:backtrace]).to eq(
|
286
286
|
[
|
287
287
|
{ file: '/airbrake-ruby/lib/foo/b.rb', line: 84, function: 'build' },
|
288
|
-
{ file: '/airbrake-ruby/lib/bar/c.rb', line: 124, function: 'send' }
|
289
|
-
]
|
288
|
+
{ file: '/airbrake-ruby/lib/bar/c.rb', line: 124, function: 'send' },
|
289
|
+
],
|
290
290
|
)
|
291
291
|
end
|
292
292
|
end
|
@@ -303,7 +303,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
303
303
|
backtrace = [
|
304
304
|
"org/jruby/RubyKernel.java:998:in `eval'",
|
305
305
|
"/ruby/stdlib/irb/workspace.rb:87:in `evaluate'",
|
306
|
-
"/ruby/stdlib/irb.rb:489:in `block in eval_input'"
|
306
|
+
"/ruby/stdlib/irb.rb:489:in `block in eval_input'",
|
307
307
|
]
|
308
308
|
allow(Kernel).to receive(:caller).and_return(backtrace)
|
309
309
|
|
@@ -314,8 +314,8 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
314
314
|
[
|
315
315
|
{ file: 'org/jruby/RubyKernel.java', line: 998, function: 'eval' },
|
316
316
|
{ file: '/ruby/stdlib/irb/workspace.rb', line: 87, function: 'evaluate' },
|
317
|
-
{ file: '/ruby/stdlib/irb.rb:489', line: 489, function: 'block in eval_input' }
|
318
|
-
]
|
317
|
+
{ file: '/ruby/stdlib/irb.rb:489', line: 489, function: 'block in eval_input' },
|
318
|
+
],
|
319
319
|
)
|
320
320
|
# rubocop:enable Metrics/LineLength
|
321
321
|
end
|
@@ -330,7 +330,7 @@ RSpec.describe Airbrake::NoticeNotifier do
|
|
330
330
|
it "raises error" do
|
331
331
|
expect { subject.build_notice(Exception.new) }.to raise_error(
|
332
332
|
Airbrake::Error,
|
333
|
-
'attempted to build Exception with closed Airbrake instance'
|
333
|
+
'attempted to build Exception with closed Airbrake instance',
|
334
334
|
)
|
335
335
|
end
|
336
336
|
end
|
data/spec/notice_spec.rb
CHANGED
@@ -17,7 +17,7 @@ RSpec.describe Airbrake::Notice do
|
|
17
17
|
before do
|
18
18
|
Airbrake::Config.instance.merge(
|
19
19
|
app_version: "1.2.3",
|
20
|
-
root_directory: "/one/two"
|
20
|
+
root_directory: "/one/two",
|
21
21
|
)
|
22
22
|
end
|
23
23
|
|
@@ -34,7 +34,7 @@ RSpec.describe Airbrake::Notice do
|
|
34
34
|
context "when versions is empty" do
|
35
35
|
it "doesn't set the 'versions' payload" do
|
36
36
|
expect(notice.to_json).not_to match(
|
37
|
-
/"context":{"versions":{"dep":"1.2.3"}}
|
37
|
+
/"context":{"versions":{"dep":"1.2.3"}}/,
|
38
38
|
)
|
39
39
|
end
|
40
40
|
end
|
@@ -43,7 +43,7 @@ RSpec.describe Airbrake::Notice do
|
|
43
43
|
it "sets the 'versions' payload" do
|
44
44
|
notice[:context][:versions] = { 'dep' => '1.2.3' }
|
45
45
|
expect(notice.to_json).to match(
|
46
|
-
/"context":{.*"versions":{"dep":"1.2.3"}.*}
|
46
|
+
/"context":{.*"versions":{"dep":"1.2.3"}.*}/,
|
47
47
|
)
|
48
48
|
end
|
49
49
|
end
|
@@ -192,7 +192,7 @@ RSpec.describe Airbrake::Notice do
|
|
192
192
|
it "doesn't fail" do
|
193
193
|
notice[:params] = { a: { b: { c: ObjectWithIoIvars.new } } }
|
194
194
|
expect(notice.to_json).to match(
|
195
|
-
/"params":{"a":{"b":{"c":".+ObjectWithIoIvars.+"}}.*}
|
195
|
+
/"params":{"a":{"b":{"c":".+ObjectWithIoIvars.+"}}.*}/,
|
196
196
|
)
|
197
197
|
end
|
198
198
|
end
|
@@ -201,7 +201,7 @@ RSpec.describe Airbrake::Notice do
|
|
201
201
|
it "doesn't fail" do
|
202
202
|
notice[:params] = { a: [[ObjectWithIoIvars.new]] }
|
203
203
|
expect(notice.to_json).to match(
|
204
|
-
/"params":{"a":\[\[".+ObjectWithIoIvars.+"\]\].*}
|
204
|
+
/"params":{"a":\[\[".+ObjectWithIoIvars.+"\]\].*}/,
|
205
205
|
)
|
206
206
|
end
|
207
207
|
end
|
@@ -15,7 +15,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
15
15
|
project_key: 'banana',
|
16
16
|
performance_stats: true,
|
17
17
|
performance_stats_flush_period: 0,
|
18
|
-
query_stats: true
|
18
|
+
query_stats: true,
|
19
19
|
)
|
20
20
|
end
|
21
21
|
|
@@ -30,8 +30,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
30
30
|
file: 'foo.rb',
|
31
31
|
line: 123,
|
32
32
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
33
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
34
|
-
)
|
33
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
34
|
+
),
|
35
35
|
)
|
36
36
|
subject.close
|
37
37
|
|
@@ -49,7 +49,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
49
49
|
"sum":60000.0,
|
50
50
|
"sumsq":3600000000.0,
|
51
51
|
"tdigest":"AAAAAkA0AAAAAAAAAAAAAUdqYAAB"
|
52
|
-
}\]}\z|x)
|
52
|
+
}\]}\z|x),
|
53
53
|
).to have_been_made
|
54
54
|
end
|
55
55
|
|
@@ -60,8 +60,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
60
60
|
route: '/foo',
|
61
61
|
status_code: 200,
|
62
62
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
63
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
64
|
-
)
|
63
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
64
|
+
),
|
65
65
|
)
|
66
66
|
subject.close
|
67
67
|
|
@@ -76,7 +76,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
76
76
|
"sum":60000.0,
|
77
77
|
"sumsq":3600000000.0,
|
78
78
|
"tdigest":"AAAAAkA0AAAAAAAAAAAAAUdqYAAB"
|
79
|
-
}\]}\z|x)
|
79
|
+
}\]}\z|x),
|
80
80
|
).to have_been_made
|
81
81
|
end
|
82
82
|
|
@@ -88,8 +88,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
88
88
|
response_type: 'json',
|
89
89
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
90
90
|
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
91
|
-
groups: { db: 131, view: 421 }
|
92
|
-
)
|
91
|
+
groups: { db: 131, view: 421 },
|
92
|
+
),
|
93
93
|
)
|
94
94
|
subject.close
|
95
95
|
|
@@ -118,7 +118,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
118
118
|
"tdigest":"AAAAAkA0AAAAAAAAAAAAAUPSgAAB"
|
119
119
|
}
|
120
120
|
}
|
121
|
-
}\]}\z|x)
|
121
|
+
}\]}\z|x),
|
122
122
|
).to have_been_made
|
123
123
|
end
|
124
124
|
|
@@ -129,8 +129,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
129
129
|
error_count: 2,
|
130
130
|
groups: { redis: 131, sql: 421 },
|
131
131
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
132
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
133
|
-
)
|
132
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
133
|
+
),
|
134
134
|
)
|
135
135
|
subject.close
|
136
136
|
|
@@ -158,7 +158,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
158
158
|
"tdigest":"AAAAAkA0AAAAAAAAAAAAAUPSgAAB"
|
159
159
|
}
|
160
160
|
}
|
161
|
-
}\]}\z/x)
|
161
|
+
}\]}\z/x),
|
162
162
|
).to have_been_made
|
163
163
|
end
|
164
164
|
|
@@ -168,13 +168,13 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
168
168
|
method: 'GET',
|
169
169
|
route: '/foo',
|
170
170
|
status_code: 200,
|
171
|
-
start_time: Time.new(2018, 1, 1, 0, 0, 20, 0)
|
172
|
-
)
|
171
|
+
start_time: Time.new(2018, 1, 1, 0, 0, 20, 0),
|
172
|
+
),
|
173
173
|
)
|
174
174
|
subject.close
|
175
175
|
|
176
176
|
expect(
|
177
|
-
a_request(:put, routes).with(body: /"time":"2018-01-01T00:00:00\+00:00"/)
|
177
|
+
a_request(:put, routes).with(body: /"time":"2018-01-01T00:00:00\+00:00"/),
|
178
178
|
).to have_been_made
|
179
179
|
end
|
180
180
|
|
@@ -184,21 +184,21 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
184
184
|
method: 'GET',
|
185
185
|
route: '/foo',
|
186
186
|
status_code: 200,
|
187
|
-
start_time: Time.new(2018, 1, 1, 0, 0, 20, 0)
|
188
|
-
)
|
187
|
+
start_time: Time.new(2018, 1, 1, 0, 0, 20, 0),
|
188
|
+
),
|
189
189
|
)
|
190
190
|
subject.notify(
|
191
191
|
Airbrake::Request.new(
|
192
192
|
method: 'GET',
|
193
193
|
route: '/foo',
|
194
194
|
status_code: 200,
|
195
|
-
start_time: Time.new(2018, 1, 1, 0, 0, 50, 0)
|
196
|
-
)
|
195
|
+
start_time: Time.new(2018, 1, 1, 0, 0, 50, 0),
|
196
|
+
),
|
197
197
|
)
|
198
198
|
subject.close
|
199
199
|
|
200
200
|
expect(
|
201
|
-
a_request(:put, routes).with(body: /"count":2/)
|
201
|
+
a_request(:put, routes).with(body: /"count":2/),
|
202
202
|
).to have_been_made
|
203
203
|
end
|
204
204
|
|
@@ -209,8 +209,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
209
209
|
route: '/foo',
|
210
210
|
status_code: 200,
|
211
211
|
start_time: Time.new(2018, 1, 1, 0, 0, 49, 0),
|
212
|
-
end_time: Time.new(2018, 1, 1, 0, 0, 50, 0)
|
213
|
-
)
|
212
|
+
end_time: Time.new(2018, 1, 1, 0, 0, 50, 0),
|
213
|
+
),
|
214
214
|
)
|
215
215
|
subject.notify(
|
216
216
|
Airbrake::Request.new(
|
@@ -218,8 +218,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
218
218
|
route: '/foo',
|
219
219
|
status_code: 200,
|
220
220
|
start_time: Time.new(2018, 1, 1, 0, 1, 49, 0),
|
221
|
-
end_time: Time.new(2018, 1, 1, 0, 1, 55, 0)
|
222
|
-
)
|
221
|
+
end_time: Time.new(2018, 1, 1, 0, 1, 55, 0),
|
222
|
+
),
|
223
223
|
)
|
224
224
|
subject.close
|
225
225
|
|
@@ -233,8 +233,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
233
233
|
{"method":"GET","route":"/foo","statusCode":200,
|
234
234
|
"time":"2018-01-01T00:01:00\+00:00","count":1,"sum":6000.0,
|
235
235
|
"sumsq":36000000.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUW7gAAB"}\]}
|
236
|
-
\z|x
|
237
|
-
)
|
236
|
+
\z|x,
|
237
|
+
),
|
238
238
|
).to have_been_made
|
239
239
|
end
|
240
240
|
|
@@ -245,8 +245,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
245
245
|
route: '/foo',
|
246
246
|
status_code: 200,
|
247
247
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
248
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
249
|
-
)
|
248
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
249
|
+
),
|
250
250
|
)
|
251
251
|
subject.notify(
|
252
252
|
Airbrake::Request.new(
|
@@ -254,8 +254,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
254
254
|
route: '/foo',
|
255
255
|
status_code: 200,
|
256
256
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
257
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
258
|
-
)
|
257
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
258
|
+
),
|
259
259
|
)
|
260
260
|
subject.close
|
261
261
|
|
@@ -269,8 +269,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
269
269
|
{"method":"POST","route":"/foo","statusCode":200,
|
270
270
|
"time":"2018-01-01T00:49:00\+00:00","count":1,"sum":60000.0,
|
271
271
|
"sumsq":3600000000.0,"tdigest":"AAAAAkA0AAAAAAAAAAAAAUdqYAAB"}\]}
|
272
|
-
\z|x
|
273
|
-
)
|
272
|
+
\z|x,
|
273
|
+
),
|
274
274
|
).to have_been_made
|
275
275
|
end
|
276
276
|
|
@@ -282,8 +282,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
282
282
|
response_type: 'json',
|
283
283
|
start_time: Time.new(2018, 1, 1, 0, 0, 20, 0),
|
284
284
|
end_time: Time.new(2018, 1, 1, 0, 0, 22, 0),
|
285
|
-
groups: { db: 131, view: 421 }
|
286
|
-
)
|
285
|
+
groups: { db: 131, view: 421 },
|
286
|
+
),
|
287
287
|
)
|
288
288
|
subject.notify(
|
289
289
|
Airbrake::PerformanceBreakdown.new(
|
@@ -292,8 +292,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
292
292
|
response_type: 'json',
|
293
293
|
start_time: Time.new(2018, 1, 1, 0, 0, 30, 0),
|
294
294
|
end_time: Time.new(2018, 1, 1, 0, 0, 32, 0),
|
295
|
-
groups: { db: 55, view: 11 }
|
296
|
-
)
|
295
|
+
groups: { db: 55, view: 11 },
|
296
|
+
),
|
297
297
|
)
|
298
298
|
subject.close
|
299
299
|
|
@@ -322,7 +322,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
322
322
|
"tdigest":"AAAAAkA0AAAAAAAAAAAAAkEwAABDzQAAAQE="
|
323
323
|
}
|
324
324
|
}
|
325
|
-
}\]}\z|x)
|
325
|
+
}\]}\z|x),
|
326
326
|
).to have_been_made
|
327
327
|
end
|
328
328
|
|
@@ -333,8 +333,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
333
333
|
error_count: 2,
|
334
334
|
groups: { redis: 131, sql: 421 },
|
335
335
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
336
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
337
|
-
)
|
336
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
337
|
+
),
|
338
338
|
)
|
339
339
|
subject.notify(
|
340
340
|
Airbrake::Queue.new(
|
@@ -342,8 +342,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
342
342
|
error_count: 3,
|
343
343
|
groups: { redis: 131, sql: 421 },
|
344
344
|
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
345
|
-
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0)
|
346
|
-
)
|
345
|
+
end_time: Time.new(2018, 1, 1, 0, 50, 0, 0),
|
346
|
+
),
|
347
347
|
)
|
348
348
|
subject.close
|
349
349
|
|
@@ -371,7 +371,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
371
371
|
"tdigest":"AAAAAkA0AAAAAAAAAAAAAUPSgAAC"
|
372
372
|
}
|
373
373
|
}
|
374
|
-
}\]}\z/x)
|
374
|
+
}\]}\z/x),
|
375
375
|
).to have_been_made
|
376
376
|
end
|
377
377
|
|
@@ -381,8 +381,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
381
381
|
method: 'GET',
|
382
382
|
route: '/foo',
|
383
383
|
status_code: 200,
|
384
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
385
|
-
)
|
384
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
385
|
+
),
|
386
386
|
)
|
387
387
|
subject.close
|
388
388
|
|
@@ -392,7 +392,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
392
392
|
|
393
393
|
it "checks performance stat configuration" do
|
394
394
|
request = Airbrake::Request.new(
|
395
|
-
method: 'GET', route: '/foo', status_code: 200, start_time: Time.new
|
395
|
+
method: 'GET', route: '/foo', status_code: 200, start_time: Time.new,
|
396
396
|
)
|
397
397
|
expect(Airbrake::Config.instance).to receive(:check_performance_options)
|
398
398
|
.with(request).and_return(Airbrake::Promise.new)
|
@@ -408,15 +408,15 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
408
408
|
method: 'POST',
|
409
409
|
route: '/foo',
|
410
410
|
status_code: 200,
|
411
|
-
start_time: Time.new
|
412
|
-
)
|
411
|
+
start_time: Time.new,
|
412
|
+
),
|
413
413
|
)
|
414
414
|
subject.close
|
415
415
|
|
416
416
|
expect(
|
417
417
|
a_request(:put, routes).with(
|
418
|
-
body: /\A{"routes":\[.+\],"environment":"test"}\z/x
|
419
|
-
)
|
418
|
+
body: /\A{"routes":\[.+\],"environment":"test"}\z/x,
|
419
|
+
),
|
420
420
|
).to have_been_made
|
421
421
|
end
|
422
422
|
|
@@ -437,7 +437,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
437
437
|
project_id: 1,
|
438
438
|
project_key: 'banana',
|
439
439
|
performance_stats: true,
|
440
|
-
performance_stats_flush_period: flush_period
|
440
|
+
performance_stats_flush_period: flush_period,
|
441
441
|
)
|
442
442
|
|
443
443
|
subject.notify(
|
@@ -445,8 +445,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
445
445
|
method: 'GET',
|
446
446
|
route: '/foo',
|
447
447
|
status_code: 200,
|
448
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
449
|
-
)
|
448
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
449
|
+
),
|
450
450
|
)
|
451
451
|
|
452
452
|
subject.notify(
|
@@ -454,8 +454,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
454
454
|
method: 'POST',
|
455
455
|
route: '/foo',
|
456
456
|
query: 'SELECT * FROM things',
|
457
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
458
|
-
)
|
457
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
458
|
+
),
|
459
459
|
)
|
460
460
|
|
461
461
|
sleep(flush_period + 0.5)
|
@@ -474,8 +474,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
474
474
|
method: 'GET',
|
475
475
|
route: '/foo',
|
476
476
|
status_code: 200,
|
477
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
478
|
-
)
|
477
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
478
|
+
),
|
479
479
|
)
|
480
480
|
subject.close
|
481
481
|
|
@@ -488,8 +488,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
488
488
|
method: 'POST',
|
489
489
|
route: '/foo',
|
490
490
|
query: 'SELECT * FROM things',
|
491
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
492
|
-
)
|
491
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
492
|
+
),
|
493
493
|
)
|
494
494
|
subject.close
|
495
495
|
|
@@ -510,20 +510,40 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
510
510
|
method: 'POST',
|
511
511
|
route: '/foo',
|
512
512
|
query: 'SELECT * FROM things',
|
513
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
514
|
-
)
|
513
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
514
|
+
),
|
515
515
|
)
|
516
516
|
subject.close
|
517
517
|
|
518
518
|
expect(
|
519
519
|
a_request(:put, queries).with(
|
520
|
-
body: /\A{"queries":\[{"method":"POST","route":"\[Filtered\]"
|
521
|
-
)
|
520
|
+
body: /\A{"queries":\[{"method":"POST","route":"\[Filtered\]"/,
|
521
|
+
),
|
522
522
|
).to have_been_made
|
523
523
|
end
|
524
524
|
end
|
525
525
|
end
|
526
526
|
|
527
|
+
describe "#notify_sync" do
|
528
|
+
it "notifies synchronously" do
|
529
|
+
retval = subject.notify_sync(
|
530
|
+
Airbrake::Query.new(
|
531
|
+
method: 'POST',
|
532
|
+
route: '/foo',
|
533
|
+
query: 'SELECT * FROM things',
|
534
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
535
|
+
),
|
536
|
+
)
|
537
|
+
|
538
|
+
expect(
|
539
|
+
a_request(:put, queries).with(
|
540
|
+
body: %r|\A{"queries":\[{"method":"POST","route":"/foo"|,
|
541
|
+
),
|
542
|
+
).to have_been_made
|
543
|
+
expect(retval).to eq('' => nil)
|
544
|
+
end
|
545
|
+
end
|
546
|
+
|
527
547
|
describe "#close" do
|
528
548
|
before do
|
529
549
|
Airbrake::Config.instance.merge(performance_stats_flush_period: 0.1)
|
@@ -540,8 +560,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
540
560
|
method: 'POST',
|
541
561
|
route: '/foo',
|
542
562
|
query: 'SELECT * FROM things',
|
543
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
544
|
-
)
|
563
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
564
|
+
),
|
545
565
|
)
|
546
566
|
subject.close
|
547
567
|
end
|
@@ -549,7 +569,7 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
549
569
|
it "logs the exit message" do
|
550
570
|
allow(Airbrake::Loggable.instance).to receive(:debug)
|
551
571
|
expect(Airbrake::Loggable.instance).to receive(:debug).with(
|
552
|
-
/performance notifier closed
|
572
|
+
/performance notifier closed/,
|
553
573
|
)
|
554
574
|
subject.close
|
555
575
|
end
|
@@ -571,8 +591,8 @@ RSpec.describe Airbrake::PerformanceNotifier do
|
|
571
591
|
method: 'POST',
|
572
592
|
route: '/foo',
|
573
593
|
status_code: 200,
|
574
|
-
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0)
|
575
|
-
)
|
594
|
+
start_time: Time.new(2018, 1, 1, 0, 49, 0, 0),
|
595
|
+
),
|
576
596
|
)
|
577
597
|
subject.close
|
578
598
|
|