appsignal 2.11.1-java → 3.0.0.beta.1-java
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.rubocop.yml +6 -1
- data/.semaphore/semaphore.yml +197 -23
- data/CHANGELOG.md +25 -0
- data/README.md +9 -0
- data/Rakefile +9 -6
- data/build_matrix.yml +13 -4
- data/ext/agent.yml +17 -17
- data/ext/base.rb +12 -9
- data/gemfiles/no_dependencies.gemfile +7 -0
- data/gemfiles/resque-2.gemfile +0 -1
- data/gemfiles/webmachine.gemfile +1 -0
- data/lib/appsignal.rb +1 -27
- data/lib/appsignal/auth_check.rb +2 -8
- data/lib/appsignal/cli.rb +1 -23
- data/lib/appsignal/cli/diagnose/utils.rb +8 -11
- data/lib/appsignal/cli/install.rb +5 -8
- data/lib/appsignal/config.rb +0 -24
- data/lib/appsignal/event_formatter.rb +0 -25
- data/lib/appsignal/helpers/instrumentation.rb +32 -0
- data/lib/appsignal/hooks.rb +0 -23
- data/lib/appsignal/hooks/action_cable.rb +3 -34
- data/lib/appsignal/hooks/active_support_notifications.rb +7 -86
- data/lib/appsignal/hooks/celluloid.rb +5 -9
- data/lib/appsignal/hooks/net_http.rb +2 -12
- data/lib/appsignal/hooks/puma.rb +3 -5
- data/lib/appsignal/hooks/que.rb +1 -1
- data/lib/appsignal/hooks/rake.rb +2 -24
- data/lib/appsignal/hooks/redis.rb +2 -13
- data/lib/appsignal/hooks/resque.rb +2 -43
- data/lib/appsignal/hooks/shoryuken.rb +43 -4
- data/lib/appsignal/hooks/unicorn.rb +3 -24
- data/lib/appsignal/hooks/webmachine.rb +1 -7
- data/lib/appsignal/integrations/action_cable.rb +34 -0
- data/lib/appsignal/integrations/active_support_notifications.rb +77 -0
- data/lib/appsignal/integrations/net_http.rb +16 -0
- data/lib/appsignal/integrations/object.rb +44 -17
- data/lib/appsignal/integrations/padrino.rb +5 -7
- data/lib/appsignal/integrations/que.rb +26 -33
- data/lib/appsignal/integrations/railtie.rb +1 -4
- data/lib/appsignal/integrations/rake.rb +26 -2
- data/lib/appsignal/integrations/redis.rb +17 -0
- data/lib/appsignal/integrations/resque.rb +39 -10
- data/lib/appsignal/integrations/unicorn.rb +28 -0
- data/lib/appsignal/integrations/webmachine.rb +22 -24
- data/lib/appsignal/minutely.rb +0 -12
- data/lib/appsignal/system.rb +4 -0
- data/lib/appsignal/transaction.rb +30 -2
- data/lib/appsignal/version.rb +1 -1
- data/spec/lib/appsignal/auth_check_spec.rb +1 -24
- data/spec/lib/appsignal/cli_spec.rb +1 -1
- data/spec/lib/appsignal/config_spec.rb +0 -66
- data/spec/lib/appsignal/event_formatter_spec.rb +0 -37
- data/spec/lib/appsignal/hooks/celluloid_spec.rb +6 -1
- data/spec/lib/appsignal/hooks/rake_spec.rb +1 -2
- data/spec/lib/appsignal/hooks/redis_spec.rb +50 -15
- data/spec/lib/appsignal/hooks/resque_spec.rb +10 -2
- data/spec/lib/appsignal/hooks/shoryuken_spec.rb +151 -104
- data/spec/lib/appsignal/hooks/sidekiq_spec.rb +4 -2
- data/spec/lib/appsignal/hooks/unicorn_spec.rb +14 -3
- data/spec/lib/appsignal/hooks/webmachine_spec.rb +2 -13
- data/spec/lib/appsignal/hooks_spec.rb +0 -57
- data/spec/lib/appsignal/integrations/object_spec.rb +25 -10
- data/spec/lib/appsignal/integrations/padrino_spec.rb +2 -3
- data/spec/lib/appsignal/integrations/railtie_spec.rb +0 -45
- data/spec/lib/appsignal/integrations/webmachine_spec.rb +26 -8
- data/spec/lib/appsignal/minutely_spec.rb +0 -19
- data/spec/lib/appsignal/transaction_spec.rb +56 -14
- data/spec/lib/appsignal/transmitter_spec.rb +1 -1
- data/spec/lib/appsignal_spec.rb +30 -69
- data/spec/spec_helper.rb +1 -15
- metadata +13 -22
- data/lib/appsignal/cli/notify_of_deploy.rb +0 -131
- data/lib/appsignal/integrations/resque_active_job.rb +0 -19
- data/lib/appsignal/js_exception_transaction.rb +0 -56
- data/lib/appsignal/rack/js_exception_catcher.rb +0 -80
- data/spec/lib/appsignal/cli/notify_of_deploy_spec.rb +0 -180
- data/spec/lib/appsignal/integrations/resque_active_job_spec.rb +0 -28
- data/spec/lib/appsignal/integrations/resque_spec.rb +0 -28
- data/spec/lib/appsignal/js_exception_transaction_spec.rb +0 -128
- data/spec/lib/appsignal/rack/js_exception_catcher_spec.rb +0 -170
@@ -1,56 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module Appsignal
|
4
|
-
class JSExceptionTransaction
|
5
|
-
attr_reader :uuid, :ext
|
6
|
-
|
7
|
-
def initialize(data)
|
8
|
-
@data = data
|
9
|
-
@uuid = SecureRandom.uuid
|
10
|
-
@ext = Appsignal::Extension.start_transaction(@uuid, Appsignal::Transaction::FRONTEND, 0)
|
11
|
-
|
12
|
-
set_action
|
13
|
-
set_metadata
|
14
|
-
set_error
|
15
|
-
set_sample_data
|
16
|
-
end
|
17
|
-
|
18
|
-
def set_action
|
19
|
-
return unless @data["action"]
|
20
|
-
ext.set_action(@data["action"])
|
21
|
-
end
|
22
|
-
|
23
|
-
def set_metadata
|
24
|
-
return unless @data["path"]
|
25
|
-
ext.set_metadata("path", @data["path"])
|
26
|
-
end
|
27
|
-
|
28
|
-
def set_error
|
29
|
-
ext.set_error(
|
30
|
-
@data["name"],
|
31
|
-
@data["message"] || "",
|
32
|
-
Appsignal::Utils::Data.generate(@data["backtrace"] || [])
|
33
|
-
)
|
34
|
-
end
|
35
|
-
|
36
|
-
def set_sample_data
|
37
|
-
{
|
38
|
-
:params => @data["params"],
|
39
|
-
:session_data => @data["session_data"],
|
40
|
-
:environment => @data["environment"],
|
41
|
-
:tags => @data["tags"]
|
42
|
-
}.each do |key, data|
|
43
|
-
next unless data.is_a?(Array) || data.is_a?(Hash)
|
44
|
-
ext.set_sample_data(
|
45
|
-
key.to_s,
|
46
|
-
Appsignal::Utils::Data.generate(data)
|
47
|
-
)
|
48
|
-
end
|
49
|
-
end
|
50
|
-
|
51
|
-
def complete!
|
52
|
-
ext.finish(0)
|
53
|
-
ext.complete
|
54
|
-
end
|
55
|
-
end
|
56
|
-
end
|
@@ -1,80 +0,0 @@
|
|
1
|
-
# frozen_string_literal: true
|
2
|
-
|
3
|
-
module Appsignal
|
4
|
-
# @api private
|
5
|
-
module Rack
|
6
|
-
# JavaScript error catching middleware.
|
7
|
-
#
|
8
|
-
# Listens to the endpoint specified in the `frontend_error_catching_path`
|
9
|
-
# configuration option.
|
10
|
-
#
|
11
|
-
# This is automatically included middleware in Rails apps if the
|
12
|
-
# `frontend_error_catching_path` configuration option is active.
|
13
|
-
#
|
14
|
-
# If AppSignal is not active in the current environment, but does have
|
15
|
-
# JavaScript error catching turned on, we assume that a JavaScript script
|
16
|
-
# still sends errors to this endpoint. When AppSignal is not active in this
|
17
|
-
# scenario this middleware still listens to the endpoint, but won't record
|
18
|
-
# the error. It will return HTTP status code 202.
|
19
|
-
#
|
20
|
-
# @example with a Sinatra app
|
21
|
-
# Sinatra::Application.use(Appsignal::Rack::JSExceptionCatcher)
|
22
|
-
#
|
23
|
-
# @see http://docs.appsignal.com/front-end/error-handling.html
|
24
|
-
# @api private
|
25
|
-
class JSExceptionCatcher
|
26
|
-
include Appsignal::Utils::DeprecationMessage
|
27
|
-
|
28
|
-
def initialize(app, _options = nil)
|
29
|
-
Appsignal.logger.debug \
|
30
|
-
"Initializing Appsignal::Rack::JSExceptionCatcher"
|
31
|
-
deprecation_message "The Appsignal::Rack::JSExceptionCatcher is " \
|
32
|
-
"deprecated and will be removed in a future version. Please use " \
|
33
|
-
"the official AppSignal JavaScript integration by disabling " \
|
34
|
-
"`enable_frontend_error_catching` in your configuration and " \
|
35
|
-
"installing AppSignal for JavaScript instead. " \
|
36
|
-
"(https://docs.appsignal.com/front-end/)"
|
37
|
-
@app = app
|
38
|
-
end
|
39
|
-
|
40
|
-
def call(env)
|
41
|
-
# Ignore other paths than the error catching path.
|
42
|
-
return @app.call(env) unless error_cathing_endpoint?(env)
|
43
|
-
|
44
|
-
# Prevent raising a 404 not found when a non-active environment posts
|
45
|
-
# to this endpoint.
|
46
|
-
unless Appsignal.active?
|
47
|
-
return [
|
48
|
-
202,
|
49
|
-
{},
|
50
|
-
["AppSignal JavaScript error catching endpoint is not active."]
|
51
|
-
]
|
52
|
-
end
|
53
|
-
|
54
|
-
begin
|
55
|
-
body = JSON.parse(env["rack.input"].read)
|
56
|
-
rescue JSON::ParserError
|
57
|
-
return [400, {}, ["Request payload is not valid JSON."]]
|
58
|
-
end
|
59
|
-
|
60
|
-
if body["name"].is_a?(String) && !body["name"].empty?
|
61
|
-
transaction = JSExceptionTransaction.new(body)
|
62
|
-
transaction.complete!
|
63
|
-
code = 200
|
64
|
-
else
|
65
|
-
Appsignal.logger.debug \
|
66
|
-
"JSExceptionCatcher: Could not send exception, 'name' is empty."
|
67
|
-
code = 422
|
68
|
-
end
|
69
|
-
|
70
|
-
[code, {}, []]
|
71
|
-
end
|
72
|
-
|
73
|
-
private
|
74
|
-
|
75
|
-
def error_cathing_endpoint?(env)
|
76
|
-
env["PATH_INFO"] == Appsignal.config[:frontend_error_catching_path]
|
77
|
-
end
|
78
|
-
end
|
79
|
-
end
|
80
|
-
end
|
@@ -1,180 +0,0 @@
|
|
1
|
-
require "appsignal/cli"
|
2
|
-
|
3
|
-
describe Appsignal::CLI::NotifyOfDeploy do
|
4
|
-
include CLIHelpers
|
5
|
-
|
6
|
-
let(:out_stream) { std_stream }
|
7
|
-
let(:output) { out_stream.read }
|
8
|
-
let(:err_stream) { std_stream }
|
9
|
-
let(:stderr) { err_stream.read }
|
10
|
-
|
11
|
-
define :include_deploy_notification do
|
12
|
-
match do |log|
|
13
|
-
log.include?("Notifying AppSignal of deploy with: ") &&
|
14
|
-
log.include?("AppSignal has been notified of this deploy!")
|
15
|
-
end
|
16
|
-
end
|
17
|
-
define :include_deploy_notification_with do |options|
|
18
|
-
match do |log|
|
19
|
-
return false unless options
|
20
|
-
values = "revision: #{options[:revision]}, user: #{options[:user]}"
|
21
|
-
log.include?("Notifying AppSignal of deploy with: #{values}") &&
|
22
|
-
log.include?("AppSignal has been notified of this deploy!")
|
23
|
-
end
|
24
|
-
end
|
25
|
-
define :include_config_error do
|
26
|
-
match do |log|
|
27
|
-
log.include? "Error: No valid config found."
|
28
|
-
end
|
29
|
-
end
|
30
|
-
define :include_missing_options do |options|
|
31
|
-
match do |log|
|
32
|
-
log.include? "Error: Missing options: #{options.join(", ")}"
|
33
|
-
end
|
34
|
-
end
|
35
|
-
|
36
|
-
def run
|
37
|
-
capture_std_streams(out_stream, err_stream) do
|
38
|
-
run_cli("notify_of_deploy", options)
|
39
|
-
end
|
40
|
-
end
|
41
|
-
|
42
|
-
context "without config" do
|
43
|
-
let(:config) { Appsignal::Config.new(tmp_dir, "production") }
|
44
|
-
let(:options) { {} }
|
45
|
-
around do |example|
|
46
|
-
Dir.chdir tmp_dir do
|
47
|
-
example.run
|
48
|
-
end
|
49
|
-
end
|
50
|
-
|
51
|
-
it "prints a config error" do
|
52
|
-
expect { run }.to raise_error(SystemExit)
|
53
|
-
expect(output).to include_config_error
|
54
|
-
expect(output).to_not include_deploy_notification
|
55
|
-
end
|
56
|
-
end
|
57
|
-
|
58
|
-
context "with config" do
|
59
|
-
let(:config) { project_fixture_config }
|
60
|
-
before do
|
61
|
-
config[:name] = options[:name] if options[:name]
|
62
|
-
stub_api_request config, "markers", :revision => options[:revision],
|
63
|
-
:user => options[:user]
|
64
|
-
end
|
65
|
-
around do |example|
|
66
|
-
Dir.chdir project_fixture_path do
|
67
|
-
example.run
|
68
|
-
end
|
69
|
-
end
|
70
|
-
|
71
|
-
context "without environment" do
|
72
|
-
let(:options) { { :environment => "", :revision => "foo", :user => "thijs" } }
|
73
|
-
before do
|
74
|
-
# Makes the config "active"
|
75
|
-
ENV["APPSIGNAL_PUSH_API_KEY"] = "foo"
|
76
|
-
end
|
77
|
-
|
78
|
-
it "requires environment option" do
|
79
|
-
expect { run }.to raise_error(SystemExit)
|
80
|
-
expect(output).to include_missing_options(%w[environment])
|
81
|
-
expect(output).to_not include_deploy_notification
|
82
|
-
end
|
83
|
-
end
|
84
|
-
|
85
|
-
context "without known environment" do
|
86
|
-
let(:options) { { :environment => "foo" } }
|
87
|
-
|
88
|
-
it "prints a config error" do
|
89
|
-
expect { run }.to raise_error(SystemExit)
|
90
|
-
expect(output).to include_config_error
|
91
|
-
expect(output).to_not include_missing_options([])
|
92
|
-
expect(output).to_not include_deploy_notification
|
93
|
-
end
|
94
|
-
end
|
95
|
-
|
96
|
-
context "with known environment" do
|
97
|
-
context "without required options" do
|
98
|
-
let(:options) { { :environment => "production" } }
|
99
|
-
|
100
|
-
it "prints a missing required options error" do
|
101
|
-
expect { run }.to raise_error(SystemExit)
|
102
|
-
expect(output).to_not include_config_error
|
103
|
-
expect(output).to include_missing_options(%w[revision user])
|
104
|
-
expect(output).to_not include_deploy_notification
|
105
|
-
end
|
106
|
-
|
107
|
-
context "with empty required option" do
|
108
|
-
let(:options) { { :environment => "production", :revision => "foo", :user => "" } }
|
109
|
-
|
110
|
-
it "prints a missing required option error" do
|
111
|
-
expect { run }.to raise_error(SystemExit)
|
112
|
-
expect(output).to_not include_config_error
|
113
|
-
expect(output).to include_missing_options(%w[user])
|
114
|
-
expect(output).to_not include_deploy_notification
|
115
|
-
end
|
116
|
-
end
|
117
|
-
end
|
118
|
-
|
119
|
-
context "with required options" do
|
120
|
-
let(:options) { { :environment => "production", :revision => "aaaaa", :user => "thijs" } }
|
121
|
-
let(:log_stream) { std_stream }
|
122
|
-
let(:log) { log_contents(log_stream) }
|
123
|
-
before { Appsignal.logger = test_logger(log_stream) }
|
124
|
-
|
125
|
-
it "notifies of a deploy" do
|
126
|
-
run
|
127
|
-
expect(output).to_not include_config_error
|
128
|
-
expect(output).to_not include_missing_options([])
|
129
|
-
expect(output).to include_deploy_notification_with(options)
|
130
|
-
end
|
131
|
-
|
132
|
-
it "prints a deprecation message" do
|
133
|
-
run
|
134
|
-
deprecation_message = "This command (appsignal notify_of_deploy) has been deprecated"
|
135
|
-
expect(stderr).to include("appsignal WARNING: #{deprecation_message}")
|
136
|
-
expect(log).to contains_log :warn, deprecation_message
|
137
|
-
end
|
138
|
-
|
139
|
-
context "with no app name configured" do
|
140
|
-
before { ENV["APPSIGNAL_APP_NAME"] = "" }
|
141
|
-
|
142
|
-
context "without name option" do
|
143
|
-
let(:options) { { :environment => "production", :revision => "aaaaa", :user => "thijs" } }
|
144
|
-
|
145
|
-
it "requires name option" do
|
146
|
-
expect { run }.to raise_error(SystemExit)
|
147
|
-
expect(output).to_not include_config_error
|
148
|
-
expect(output).to include_missing_options(%w[name])
|
149
|
-
expect(output).to_not include_deploy_notification
|
150
|
-
end
|
151
|
-
end
|
152
|
-
|
153
|
-
context "with name option" do
|
154
|
-
let(:options) { { :environment => "production", :revision => "aaaaa", :user => "thijs", :name => "foo" } }
|
155
|
-
|
156
|
-
it "notifies of a deploy with a custom name" do
|
157
|
-
run
|
158
|
-
expect(output).to_not include_config_error
|
159
|
-
expect(output).to_not include_missing_options([])
|
160
|
-
expect(output).to include_deploy_notification_with(options)
|
161
|
-
end
|
162
|
-
end
|
163
|
-
end
|
164
|
-
|
165
|
-
context "with name option" do
|
166
|
-
let(:options) do
|
167
|
-
{ :environment => "production", :revision => "aaaaa", :user => "thijs", :name => "foo" }
|
168
|
-
end
|
169
|
-
|
170
|
-
it "notifies of a deploy with a custom name" do
|
171
|
-
run
|
172
|
-
expect(output).to_not include_config_error
|
173
|
-
expect(output).to_not include_missing_options([])
|
174
|
-
expect(output).to include_deploy_notification_with(options)
|
175
|
-
end
|
176
|
-
end
|
177
|
-
end
|
178
|
-
end
|
179
|
-
end
|
180
|
-
end
|
@@ -1,28 +0,0 @@
|
|
1
|
-
require "appsignal/integrations/resque_active_job"
|
2
|
-
|
3
|
-
describe "Legacy Resque ActiveJob integration" do
|
4
|
-
let(:err_stream) { std_stream }
|
5
|
-
let(:stderr) { err_stream.read }
|
6
|
-
let(:log_stream) { std_stream }
|
7
|
-
let(:log) { log_contents(log_stream) }
|
8
|
-
|
9
|
-
it "logs and prints a deprecation message on extend" do
|
10
|
-
Appsignal.logger = test_logger(log_stream)
|
11
|
-
|
12
|
-
capture_std_streams(std_stream, err_stream) do
|
13
|
-
Class.new do
|
14
|
-
include Appsignal::Integrations::ResqueActiveJobPlugin
|
15
|
-
end
|
16
|
-
end
|
17
|
-
|
18
|
-
deprecation_message =
|
19
|
-
"The AppSignal ResqueActiveJobPlugin is deprecated and does " \
|
20
|
-
"nothing on extend. In this version of the AppSignal Ruby gem " \
|
21
|
-
"the integration with Resque is automatic on all Resque workers. " \
|
22
|
-
"Please remove the following line from this file to remove this " \
|
23
|
-
"message: include Appsignal::Integrations::ResqueActiveJobPlugin\n" \
|
24
|
-
"#{__FILE__}:"
|
25
|
-
expect(stderr).to include "appsignal WARNING: #{deprecation_message}"
|
26
|
-
expect(log).to contains_log :warn, deprecation_message
|
27
|
-
end
|
28
|
-
end
|
@@ -1,28 +0,0 @@
|
|
1
|
-
require "appsignal/integrations/resque"
|
2
|
-
|
3
|
-
describe "Legacy Resque integration" do
|
4
|
-
let(:err_stream) { std_stream }
|
5
|
-
let(:stderr) { err_stream.read }
|
6
|
-
let(:log_stream) { std_stream }
|
7
|
-
let(:log) { log_contents(log_stream) }
|
8
|
-
|
9
|
-
it "logs and prints a deprecation message on extend" do
|
10
|
-
Appsignal.logger = test_logger(log_stream)
|
11
|
-
|
12
|
-
capture_std_streams(std_stream, err_stream) do
|
13
|
-
Class.new do
|
14
|
-
extend Appsignal::Integrations::ResquePlugin
|
15
|
-
end
|
16
|
-
end
|
17
|
-
|
18
|
-
deprecation_message =
|
19
|
-
"The AppSignal ResquePlugin is deprecated and does " \
|
20
|
-
"nothing on extend. In this version of the AppSignal Ruby gem " \
|
21
|
-
"the integration with Resque is automatic on all Resque workers. " \
|
22
|
-
"Please remove the following line from this file to remove this " \
|
23
|
-
"message: extend Appsignal::Integrations::ResquePlugin\n" \
|
24
|
-
"#{__FILE__}:"
|
25
|
-
expect(stderr).to include "appsignal WARNING: #{deprecation_message}"
|
26
|
-
expect(log).to contains_log :warn, deprecation_message
|
27
|
-
end
|
28
|
-
end
|
@@ -1,128 +0,0 @@
|
|
1
|
-
describe Appsignal::JSExceptionTransaction do
|
2
|
-
before { allow(SecureRandom).to receive(:uuid).and_return("123abc") }
|
3
|
-
|
4
|
-
let!(:transaction) { Appsignal::JSExceptionTransaction.new(data) }
|
5
|
-
let(:data) do
|
6
|
-
{
|
7
|
-
"name" => "TypeError",
|
8
|
-
"message" => "foo is not a valid method",
|
9
|
-
"action" => "ExceptionIncidentComponent",
|
10
|
-
"path" => "foo.bar/moo",
|
11
|
-
"environment" => "development",
|
12
|
-
"backtrace" => [
|
13
|
-
"foo.bar/js:11:1",
|
14
|
-
"foo.bar/js:22:2"
|
15
|
-
],
|
16
|
-
"tags" => [
|
17
|
-
"tag1"
|
18
|
-
]
|
19
|
-
}
|
20
|
-
end
|
21
|
-
|
22
|
-
describe "#initialize" do
|
23
|
-
it "should call all required methods" do
|
24
|
-
expect(Appsignal::Extension).to receive(:start_transaction).with("123abc", "frontend", 0).and_return(1)
|
25
|
-
|
26
|
-
expect(transaction).to receive(:set_action)
|
27
|
-
expect(transaction).to receive(:set_metadata)
|
28
|
-
expect(transaction).to receive(:set_error)
|
29
|
-
expect(transaction).to receive(:set_sample_data)
|
30
|
-
|
31
|
-
transaction.send :initialize, data
|
32
|
-
|
33
|
-
expect(transaction.ext).to_not be_nil
|
34
|
-
end
|
35
|
-
end
|
36
|
-
|
37
|
-
describe "#set_action" do
|
38
|
-
it "should call `Appsignal::Extension.set_action`" do
|
39
|
-
expect(transaction.ext).to receive(:set_action).with(
|
40
|
-
"ExceptionIncidentComponent"
|
41
|
-
)
|
42
|
-
|
43
|
-
transaction.set_action
|
44
|
-
end
|
45
|
-
end
|
46
|
-
|
47
|
-
describe "#set_metadata" do
|
48
|
-
it "should call `Appsignal::Extension.set_transaction_metadata`" do
|
49
|
-
expect(transaction.ext).to receive(:set_metadata).with(
|
50
|
-
"path",
|
51
|
-
"foo.bar/moo"
|
52
|
-
)
|
53
|
-
|
54
|
-
transaction.set_metadata
|
55
|
-
end
|
56
|
-
end
|
57
|
-
|
58
|
-
describe "#set_error" do
|
59
|
-
it "should call `Appsignal::Extension.set_transaction_error`" do
|
60
|
-
expect(transaction.ext).to receive(:set_error).with(
|
61
|
-
"TypeError",
|
62
|
-
"foo is not a valid method",
|
63
|
-
Appsignal::Utils::Data.generate(["foo.bar/js:11:1", "foo.bar/js:22:2"])
|
64
|
-
)
|
65
|
-
|
66
|
-
transaction.set_error
|
67
|
-
end
|
68
|
-
end
|
69
|
-
|
70
|
-
describe "#set_sample_data" do
|
71
|
-
it "should call `Appsignal::Extension.set_transaction_error_data`" do
|
72
|
-
expect(transaction.ext).to receive(:set_sample_data).with(
|
73
|
-
"tags",
|
74
|
-
Appsignal::Utils::Data.generate(["tag1"])
|
75
|
-
)
|
76
|
-
|
77
|
-
transaction.set_sample_data
|
78
|
-
end
|
79
|
-
end
|
80
|
-
|
81
|
-
context "when sending just the name" do
|
82
|
-
let(:data) { { "name" => "TypeError" } }
|
83
|
-
|
84
|
-
describe "#set_action" do
|
85
|
-
it "should not call `Appsignal::Extension.set_action`" do
|
86
|
-
expect(transaction.ext).to_not receive(:set_action)
|
87
|
-
|
88
|
-
transaction.set_action
|
89
|
-
end
|
90
|
-
end
|
91
|
-
|
92
|
-
describe "#set_metadata" do
|
93
|
-
it "should not call `Appsignal::Extension.set_transaction_metadata`" do
|
94
|
-
expect(transaction.ext).to_not receive(:set_metadata)
|
95
|
-
|
96
|
-
transaction.set_metadata
|
97
|
-
end
|
98
|
-
end
|
99
|
-
|
100
|
-
describe "#set_error" do
|
101
|
-
it "should call `Appsignal::Extension.set_transaction_error` with just the name" do
|
102
|
-
expect(transaction.ext).to receive(:set_error).with(
|
103
|
-
"TypeError",
|
104
|
-
"",
|
105
|
-
Appsignal::Utils::Data.generate([])
|
106
|
-
)
|
107
|
-
|
108
|
-
transaction.set_error
|
109
|
-
end
|
110
|
-
end
|
111
|
-
|
112
|
-
describe "#set_sample_data" do
|
113
|
-
it "should not call `Appsignal::Extension.set_transaction_error_data`" do
|
114
|
-
expect(transaction.ext).to_not receive(:set_sample_data)
|
115
|
-
|
116
|
-
transaction.set_sample_data
|
117
|
-
end
|
118
|
-
end
|
119
|
-
end
|
120
|
-
|
121
|
-
describe "#complete!" do
|
122
|
-
it "should call all required methods" do
|
123
|
-
expect(transaction.ext).to receive(:finish).and_call_original
|
124
|
-
expect(transaction.ext).to receive(:complete).and_call_original
|
125
|
-
transaction.complete!
|
126
|
-
end
|
127
|
-
end
|
128
|
-
end
|