diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2017-01-14 05:58:41 +0300 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2017-01-14 05:58:41 +0300 |
commit | bf8e174f0ae21b320c17b5a8f8d45aefcfef9520 (patch) | |
tree | 161fdc51f8076d06ff9f9cf4c023957ffd695944 /spec | |
parent | 77756efc22af33dd1cdb16671e39192d5761eaf6 (diff) | |
parent | 892ff3a3ae640272f8712fb190242f2b1fe010a0 (diff) |
Merge branch '26587-metrics-middleware-endpoint-is-nil' into 'master'
Resolve "Metrics middleware: endpoint is nil"
Closes #26587
See merge request !8544
Diffstat (limited to 'spec')
-rw-r--r-- | spec/lib/gitlab/metrics/rack_middleware_spec.rb | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/spec/lib/gitlab/metrics/rack_middleware_spec.rb b/spec/lib/gitlab/metrics/rack_middleware_spec.rb index 7371b578a48..fb470ea7568 100644 --- a/spec/lib/gitlab/metrics/rack_middleware_spec.rb +++ b/spec/lib/gitlab/metrics/rack_middleware_spec.rb @@ -126,5 +126,16 @@ describe Gitlab::Metrics::RackMiddleware do expect(transaction.action).to eq('Grape#GET /projects/:id/archive') end + + it 'does not tag a transaction if route infos are missing' do + endpoint = double(:endpoint) + allow(endpoint).to receive(:route).and_raise + + env['api.endpoint'] = endpoint + + middleware.tag_endpoint(transaction, env) + + expect(transaction.action).to be_nil + end end end |