Skip to content

Commit

Permalink
Re-removed debunk attr config tests
Browse files Browse the repository at this point in the history
  • Loading branch information
NatalieWolfe committed Mar 6, 2018
1 parent f53316a commit 0535c68
Showing 1 changed file with 0 additions and 25 deletions.
25 changes: 0 additions & 25 deletions test/unit/config/config.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -746,12 +746,6 @@ describe('the agent configuration', function() {
expect(config.transaction_tracer.enabled).equal(false)
})

it('should disable transaction tracer attributes when told to', function() {
expect(config.transaction_tracer.attributes.enabled).equal(true)
config.onConnect({'transaction_tracer.attributes.enabled': false})
expect(config.transaction_tracer.attributes.enabled).equal(false)
})

it('should always respect collect_errors', function() {
expect(config.collect_errors).equal(true)
config.onConnect({'collect_errors': false})
Expand All @@ -764,12 +758,6 @@ describe('the agent configuration', function() {
expect(config.error_collector.enabled).equal(false)
})

it('should disable the error attributes when told to', function() {
expect(config.error_collector.attributes.enabled).equal(true)
config.onConnect({'error_collector.attributes.enabled': false})
expect(config.error_collector.attributes.enabled).equal(false)
})

it('should set apdex_t', function() {
expect(config.apdex_t).equal(0.1)
config.on('apdex_t', function(value) { expect(value).equal(0.05) })
Expand Down Expand Up @@ -834,12 +822,6 @@ describe('the agent configuration', function() {
expect(config.ignored_params).eql(['a', 'b'])
})

it('should configure global excluded attributes', function() {
expect(config.attributes.exclude).eql([])
config.onConnect({'attributes.exclude': ['a', 'b']})
expect(config.attributes.exclude).eql(['a', 'b'])
})

it('should configure ignored params without stomping local config', function() {
config.ignored_params = ['b', 'c']
config.onConnect({'ignored_params': ['a', 'b']})
Expand Down Expand Up @@ -1081,13 +1063,6 @@ describe('the agent configuration', function() {
expect(config.transaction_events.enabled).equals(false)
})

it('does not blow up when transaction_events.attributes.enabled is set', function() {
expect(function() {
config.onConnect({'transaction_events.attributes.enabled': false})
}).not.throws()
expect(config.transaction_events.attributes.enabled).equals(false)
})

describe('when data_report_period is set', function() {
it('should emit data_report_period when harvest interval is changed',
function(done) {
Expand Down

0 comments on commit 0535c68

Please sign in to comment.