appsignal 1.3.1 → 1.3.2
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.rspec +1 -1
- data/CHANGELOG.md +5 -1
- data/README.md +21 -6
- data/Rakefile +7 -3
- data/ext/extconf.rb +3 -3
- data/lib/appsignal.rb +15 -4
- data/lib/appsignal/cli.rb +2 -1
- data/lib/appsignal/cli/diagnose.rb +2 -2
- data/lib/appsignal/cli/install.rb +1 -1
- data/lib/appsignal/config.rb +1 -1
- data/lib/appsignal/hooks.rb +5 -1
- data/lib/appsignal/transaction.rb +18 -1
- data/lib/appsignal/version.rb +1 -1
- data/spec/lib/appsignal/auth_check_spec.rb +3 -6
- data/spec/lib/appsignal/cli/install_spec.rb +4 -4
- data/spec/lib/appsignal/config_spec.rb +37 -37
- data/spec/lib/appsignal/event_formatter/action_view/render_formatter_spec.rb +3 -3
- data/spec/lib/appsignal/event_formatter/active_record/instantiation_formatter_spec.rb +1 -1
- data/spec/lib/appsignal/event_formatter/active_record/sql_formatter_spec.rb +1 -1
- data/spec/lib/appsignal/event_formatter/faraday/request_formatter_spec.rb +1 -1
- data/spec/lib/appsignal/event_formatter/moped/query_formatter_spec.rb +7 -7
- data/spec/lib/appsignal/event_formatter_spec.rb +4 -4
- data/spec/lib/appsignal/hooks/redis_spec.rb +1 -1
- data/spec/lib/appsignal/hooks/sidekiq_spec.rb +1 -1
- data/spec/lib/appsignal/hooks_spec.rb +13 -13
- data/spec/lib/appsignal/integrations/mongo_ruby_driver_spec.rb +1 -1
- data/spec/lib/appsignal/integrations/railtie_spec.rb +6 -6
- data/spec/lib/appsignal/integrations/webmachine_spec.rb +1 -1
- data/spec/lib/appsignal/rack/rails_instrumentation_spec.rb +2 -2
- data/spec/lib/appsignal/subscriber_spec.rb +4 -4
- data/spec/lib/appsignal/transaction_spec.rb +44 -25
- data/spec/lib/appsignal/transmitter_spec.rb +9 -9
- data/spec/lib/appsignal/utils_spec.rb +2 -2
- data/spec/lib/appsignal_spec.rb +14 -15
- metadata +2 -2
@@ -36,7 +36,7 @@ describe Appsignal::Transmitter do
|
|
36
36
|
|
37
37
|
subject { instance.transmit(:the => :payload) }
|
38
38
|
|
39
|
-
it { should
|
39
|
+
it { should eq '200' }
|
40
40
|
end
|
41
41
|
|
42
42
|
describe "#http_post" do
|
@@ -46,12 +46,12 @@ describe Appsignal::Transmitter do
|
|
46
46
|
|
47
47
|
subject { instance.send(:http_post, 'the' => 'payload') }
|
48
48
|
|
49
|
-
its(:body) { should
|
50
|
-
its(:path) { should
|
49
|
+
its(:body) { should eq Zlib::Deflate.deflate("{\"the\":\"payload\"}", Zlib::BEST_SPEED) }
|
50
|
+
its(:path) { should eq instance.uri.request_uri }
|
51
51
|
|
52
52
|
it "should have the correct headers" do
|
53
|
-
subject['Content-Type'].should
|
54
|
-
subject['Content-Encoding'].should
|
53
|
+
subject['Content-Type'].should eq 'application/json; charset=UTF-8'
|
54
|
+
subject['Content-Encoding'].should eq 'gzip'
|
55
55
|
end
|
56
56
|
end
|
57
57
|
|
@@ -59,7 +59,7 @@ describe Appsignal::Transmitter do
|
|
59
59
|
subject { Appsignal::Transmitter::CA_FILE_PATH }
|
60
60
|
|
61
61
|
it { should include('resources/cacert.pem') }
|
62
|
-
it("should exist") { File.
|
62
|
+
it("should exist") { File.exist?(subject).should be_true }
|
63
63
|
end
|
64
64
|
|
65
65
|
describe "#http_client" do
|
@@ -79,7 +79,7 @@ describe Appsignal::Transmitter do
|
|
79
79
|
it { should be_instance_of(Net::HTTP) }
|
80
80
|
its(:proxy?) { should be_false }
|
81
81
|
its(:use_ssl?) { should be_true }
|
82
|
-
its(:verify_mode) { should
|
82
|
+
its(:verify_mode) { should eq OpenSSL::SSL::VERIFY_PEER }
|
83
83
|
its(:ca_file) { Appsignal::Transmitter::CA_FILE_PATH }
|
84
84
|
end
|
85
85
|
|
@@ -87,8 +87,8 @@ describe Appsignal::Transmitter do
|
|
87
87
|
let(:config) { project_fixture_config('production', :http_proxy => 'http://localhost:8080') }
|
88
88
|
|
89
89
|
its(:proxy?) { should be_true }
|
90
|
-
its(:proxy_address) { should
|
91
|
-
its(:proxy_port) { should
|
90
|
+
its(:proxy_address) { should eq 'localhost' }
|
91
|
+
its(:proxy_port) { should eq 8080 }
|
92
92
|
end
|
93
93
|
end
|
94
94
|
end
|
@@ -18,7 +18,7 @@ describe Appsignal::Utils do
|
|
18
18
|
}
|
19
19
|
end
|
20
20
|
|
21
|
-
it { should
|
21
|
+
it { should eq %({"the":"payload","1":true,"":"test","foo":[1,2,"three"],"bar":null,"baz":{"foo":"bar"}}) }
|
22
22
|
end
|
23
23
|
|
24
24
|
context "with a body that contains strings with invalid utf-8 content" do
|
@@ -34,7 +34,7 @@ describe Appsignal::Utils do
|
|
34
34
|
}
|
35
35
|
} }
|
36
36
|
|
37
|
-
it { should
|
37
|
+
it { should eq %({"field_one":"aa","field_two":"aa�","field_three":["one","aa�"],"field_four":{"one":"aa�"}}) }
|
38
38
|
end
|
39
39
|
end
|
40
40
|
end
|
data/spec/lib/appsignal_spec.rb
CHANGED
@@ -17,7 +17,7 @@ describe Appsignal do
|
|
17
17
|
Appsignal.logger.should_not_receive(:level=)
|
18
18
|
|
19
19
|
Appsignal.config = config
|
20
|
-
Appsignal.config.should
|
20
|
+
Appsignal.config.should eq config
|
21
21
|
end
|
22
22
|
end
|
23
23
|
|
@@ -54,7 +54,7 @@ describe Appsignal do
|
|
54
54
|
|
55
55
|
it "should initialize logging" do
|
56
56
|
Appsignal.start
|
57
|
-
Appsignal.logger.level.should
|
57
|
+
Appsignal.logger.level.should eq Logger::INFO
|
58
58
|
end
|
59
59
|
|
60
60
|
it "should start native" do
|
@@ -154,7 +154,7 @@ describe Appsignal do
|
|
154
154
|
|
155
155
|
it "should change the log level" do
|
156
156
|
Appsignal.start
|
157
|
-
Appsignal.logger.level.should
|
157
|
+
Appsignal.logger.level.should eq Logger::DEBUG
|
158
158
|
end
|
159
159
|
end
|
160
160
|
end
|
@@ -259,7 +259,7 @@ describe Appsignal do
|
|
259
259
|
lambda {
|
260
260
|
Appsignal.monitor_transaction('perform_job.nothing') do
|
261
261
|
object.some_method
|
262
|
-
end.should
|
262
|
+
end.should eq 1
|
263
263
|
}.should_not raise_error
|
264
264
|
end
|
265
265
|
end
|
@@ -334,10 +334,10 @@ describe Appsignal do
|
|
334
334
|
background_env_with_data
|
335
335
|
) do
|
336
336
|
current = Appsignal::Transaction.current
|
337
|
-
current.namespace.should
|
337
|
+
current.namespace.should eq Appsignal::Transaction::BACKGROUND_JOB
|
338
338
|
current.request.should be_a(Appsignal::Transaction::GenericRequest)
|
339
339
|
object.some_method
|
340
|
-
end.should
|
340
|
+
end.should eq 1
|
341
341
|
end
|
342
342
|
|
343
343
|
it "should instrument and complete for a http request" do
|
@@ -353,7 +353,7 @@ describe Appsignal do
|
|
353
353
|
http_request_env_with_data
|
354
354
|
) do
|
355
355
|
current = Appsignal::Transaction.current
|
356
|
-
current.namespace.should
|
356
|
+
current.namespace.should eq Appsignal::Transaction::HTTP_REQUEST
|
357
357
|
current.request.should be_a(::Rack::Request)
|
358
358
|
object.some_method
|
359
359
|
end
|
@@ -571,9 +571,9 @@ describe Appsignal do
|
|
571
571
|
context "when the log path is writable" do
|
572
572
|
it "should log to file" do
|
573
573
|
Appsignal.start_logger
|
574
|
-
Appsignal.logger.level.should
|
574
|
+
Appsignal.logger.level.should eq Logger::INFO
|
575
575
|
Appsignal.logger.error('Log to file')
|
576
|
-
File.
|
576
|
+
File.exist?(log_file).should be_true
|
577
577
|
File.open(log_file).read.should include 'Log to file'
|
578
578
|
File.open(log_file).read.should include 'Log something'
|
579
579
|
end
|
@@ -587,7 +587,7 @@ describe Appsignal do
|
|
587
587
|
|
588
588
|
it "should log to stdout" do
|
589
589
|
Appsignal.start_logger
|
590
|
-
Appsignal.logger.level.should
|
590
|
+
Appsignal.logger.level.should eq Logger::INFO
|
591
591
|
Appsignal.logger.error('Log to stdout')
|
592
592
|
out_stream.string.should include 'appsignal: Log to stdout'
|
593
593
|
out_stream.string.should include 'Log something'
|
@@ -605,7 +605,7 @@ describe Appsignal do
|
|
605
605
|
it "should set the log level to info" do
|
606
606
|
Appsignal.config = nil
|
607
607
|
Appsignal.start_logger
|
608
|
-
Appsignal.logger.level.should
|
608
|
+
Appsignal.logger.level.should eq Logger::INFO
|
609
609
|
end
|
610
610
|
end
|
611
611
|
|
@@ -613,7 +613,7 @@ describe Appsignal do
|
|
613
613
|
it "should set the log level to debug" do
|
614
614
|
Appsignal.config.config_hash[:debug] = true
|
615
615
|
Appsignal.start_logger
|
616
|
-
Appsignal.logger.level.should
|
616
|
+
Appsignal.logger.level.should eq Logger::DEBUG
|
617
617
|
end
|
618
618
|
end
|
619
619
|
end
|
@@ -623,8 +623,7 @@ describe Appsignal do
|
|
623
623
|
|
624
624
|
it "should format a log line" do
|
625
625
|
Process.stub(:pid => 100)
|
626
|
-
subject.call('Debug', Time.parse('2015-07-08'), nil, 'log line').should
|
627
|
-
"[2015-07-08T00:00:00 (process) #100][Debug] log line\n"
|
626
|
+
subject.call('Debug', Time.parse('2015-07-08'), nil, 'log line').should eq "[2015-07-08T00:00:00 (process) #100][Debug] log line\n"
|
628
627
|
end
|
629
628
|
end
|
630
629
|
|
@@ -633,7 +632,7 @@ describe Appsignal do
|
|
633
632
|
|
634
633
|
it { should be_a Appsignal::Config }
|
635
634
|
it 'should return configuration' do
|
636
|
-
subject[:endpoint].should
|
635
|
+
subject[:endpoint].should eq 'https://push.appsignal.com'
|
637
636
|
end
|
638
637
|
end
|
639
638
|
|
metadata
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: appsignal
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 1.3.
|
4
|
+
version: 1.3.2
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Robert Beekman
|
@@ -9,7 +9,7 @@ authors:
|
|
9
9
|
autorequire:
|
10
10
|
bindir: bin
|
11
11
|
cert_chain: []
|
12
|
-
date: 2016-
|
12
|
+
date: 2016-09-01 00:00:00.000000000 Z
|
13
13
|
dependencies:
|
14
14
|
- !ruby/object:Gem::Dependency
|
15
15
|
name: rack
|