newrelic_rpm 4.1.0.333 → 4.2.0.334

data/test/new_relic/agent/transaction/slowest_sample_buffer_test.rb

@@ -1,67 +0,0 @@
-# encoding: utf-8
-# This file is distributed under New Relic's license terms.
-# See https://github.com/newrelic/rpm/blob/master/LICENSE for complete details.
-
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
-
-class NewRelic::Agent::Transaction
-  class SlowestSampleBufferTest < Minitest::Test
-    def setup
-      @buffer = SlowestSampleBuffer.new
-    end
-
-    def test_store_sample
-      sample = stub(:duration => 2.0, :threshold => 1.0)
-      @buffer.store(sample)
-      assert_equal([sample], @buffer.samples)
-    end
-
-    def test_store_new_is_slowest
-      old_sample = stub(:duration => 3.0, :threshold => 1.0)
-      new_sample = stub(:duration => 4.0, :threshold => 1.0)
-
-      @buffer.store(old_sample)
-      @buffer.store(new_sample)
-
-      assert_equal([new_sample], @buffer.samples)
-    end
-
-    def test_store_new_is_faster
-      old_sample = stub(:duration => 4.0, :threshold => 1.0)
-      new_sample = stub(:duration => 3.0, :threshold => 1.0)
-
-      @buffer.store(old_sample)
-      @buffer.store(new_sample)
-
-      assert_equal([old_sample], @buffer.samples)
-    end
-
-    def test_store_does_not_store_if_faster_than_threshold
-      old_sample = stub('old', :duration => 1.0, :threshold => 0.5)
-      new_sample = stub('new', :duration => 2.0, :threshold => 4.0)
-
-      @buffer.store(old_sample)
-      @buffer.store(new_sample)
-
-      assert_equal([old_sample], @buffer.samples)
-    end
-
-    def test_harvest_samples
-      sample = stub(:duration => 1.0, :threshold => 1.0)
-      @buffer.store(sample)
-
-      harvested = @buffer.harvest_samples
-
-      assert_equal([sample], harvested)
-    end
-
-    def test_harvest_samples_resets
-      sample = stub(:duration => 1.0, :threshold => 1.0)
-      @buffer.store(sample)
-
-      @buffer.harvest_samples
-
-      assert(@buffer.samples.empty?)
-    end
-  end
-end

data/test/new_relic/agent/transaction/synthetics_sample_buffer_test.rb

@@ -1,38 +0,0 @@
-# encoding: utf-8
-# This file is distributed under New Relic's license terms.
-# See https://github.com/newrelic/rpm/blob/master/LICENSE for complete details.
-
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
-
-class NewRelic::Agent::Transaction
-  class SyntheticsSampleBufferTest < Minitest::Test
-    def setup
-      @buffer = SyntheticsSampleBuffer.new
-    end
-
-    def test_doesnt_store_if_not_synthetics
-      sample = stub(:synthetics_resource_id => nil)
-      @buffer.store(sample)
-      assert_empty @buffer.samples
-    end
-
-    def test_stores_if_synthetics
-      sample = stub(:synthetics_resource_id => 42)
-      @buffer.store(sample)
-      assert_equal [sample], @buffer.samples
-    end
-
-    def test_applies_limits
-      last_sample = nil
-
-      try_count = @buffer.capacity + 1
-      try_count.times do |i|
-        last_sample = stub(:synthetics_resource_id => i)
-        @buffer.store(last_sample)
-      end
-
-      assert_equal @buffer.capacity, @buffer.samples.length
-      refute_includes @buffer.samples, last_sample
-    end
-  end
-end

data/test/new_relic/agent/transaction/trace_node_test.rb

@@ -1,342 +0,0 @@
-# encoding: utf-8
-# This file is distributed under New Relic's license terms.
-# See https://github.com/newrelic/rpm/blob/master/LICENSE for complete details.
-
-require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', '..', 'test_helper'))
-require 'new_relic/agent/transaction/trace_node'
-
-class NewRelic::Agent::Transaction::TraceNodeTest < Minitest::Test
-  def test_node_creation
-    # basic smoke test
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal NewRelic::Agent::Transaction::TraceNode, s.class
-  end
-
-  def test_readers
-    t = Time.now
-    s = NewRelic::Agent::Transaction::TraceNode.new(t, 'Custom/test/metric')
-    assert_equal(t, s.entry_timestamp)
-    assert_equal(nil, s.exit_timestamp)
-    assert_equal(nil, s.parent_node)
-    assert_equal('Custom/test/metric', s.metric_name)
-  end
-
-  def test_end_trace
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    t = Time.now
-    s.end_trace(t)
-    assert_equal(t, s.exit_timestamp)
-  end
-
-  def test_add_called_node
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal [], s.called_nodes
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    s.add_called_node(fake_node)
-    assert_equal([fake_node], s.called_nodes)
-  end
-
-  def test_to_s
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    s.expects(:to_debug_str).with(0)
-    s.to_s
-  end
-
-  def test_to_array
-    parent = NewRelic::Agent::Transaction::TraceNode.new(1, 'Custom/test/parent')
-    parent.params[:test] = 'value'
-    child = NewRelic::Agent::Transaction::TraceNode.new(2, 'Custom/test/child')
-    child.end_trace(3)
-    parent.add_called_node(child)
-    parent.end_trace(4)
-    expected_array = [1000, 4000, 'Custom/test/parent', {:test => 'value'},
-                      [[2000, 3000, 'Custom/test/child', {}, []]]]
-    assert_equal(expected_array, parent.to_array)
-  end
-
-  def test_to_array_with_bad_values
-    node = NewRelic::Agent::Transaction::TraceNode.new(nil, nil)
-    node.end_trace(Rational(10, 1))
-    expected = [0, 10_000.0, "<unknown>", {}, []]
-    assert_equal(expected, node.to_array)
-  end
-
-  def test_path_string
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal("Custom/test/metric[]", s.path_string)
-
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    fake_node.expects(:path_string).returns('Custom/other/metric[]')
-
-
-    s.add_called_node(fake_node)
-    assert_equal("Custom/test/metric[Custom/other/metric[]]", s.path_string)
-  end
-
-  def test_to_s_compact
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal("Custom/test/metric", s.to_s_compact)
-
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    fake_node.expects(:to_s_compact).returns('Custom/other/metric')
-    s.add_called_node(fake_node)
-
-    assert_equal("Custom/test/metric{Custom/other/metric}", s.to_s_compact)
-  end
-
-  def test_to_debug_str_basic
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n<<  n/a Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_to_debug_str_with_params
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    s.params = {:whee => 'a param'}
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n    -whee            : a param\n<<  n/a Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_to_debug_str_closed
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    s.end_trace(0.1)
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n<< 100 ms Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_to_debug_str_closed_with_nonnumeric
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    s.end_trace("0.1")
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n<< 0.1 Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_to_debug_str_one_child
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    s.add_called_node(NewRelic::Agent::Transaction::TraceNode.new(0.1, 'Custom/test/other'))
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n  >> 100 ms [TraceNode] Custom/test/other \n  <<  n/a Custom/test/other\n<<  n/a Custom/test/metric\n", s.to_debug_str(0))
-    # try closing it
-    s.called_nodes.first.end_trace(0.15)
-    s.end_trace(0.2)
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n  >> 100 ms [TraceNode] Custom/test/other \n  << 150 ms Custom/test/other\n<< 200 ms Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_to_debug_str_multichild
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    s.add_called_node(NewRelic::Agent::Transaction::TraceNode.new(0.1, 'Custom/test/other'))
-    s.add_called_node(NewRelic::Agent::Transaction::TraceNode.new(0.11, 'Custom/test/extra'))
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n  >> 100 ms [TraceNode] Custom/test/other \n  <<  n/a Custom/test/other\n  >> 110 ms [TraceNode] Custom/test/extra \n  <<  n/a Custom/test/extra\n<<  n/a Custom/test/metric\n", s.to_debug_str(0))
-    ending = 0.12
-    s.called_nodes.each { |x| x.end_trace(ending += 0.01) }
-    s.end_trace(0.2)
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n  >> 100 ms [TraceNode] Custom/test/other \n  << 130 ms Custom/test/other\n  >> 110 ms [TraceNode] Custom/test/extra \n  << 140 ms Custom/test/extra\n<< 200 ms Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_to_debug_str_nested
-    inner = NewRelic::Agent::Transaction::TraceNode.new(0.2, 'Custom/test/inner')
-    middle = NewRelic::Agent::Transaction::TraceNode.new(0.1, 'Custom/test/middle')
-    s = NewRelic::Agent::Transaction::TraceNode.new(0.0, 'Custom/test/metric')
-    middle.add_called_node(inner)
-    s.add_called_node(middle)
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n  >> 100 ms [TraceNode] Custom/test/middle \n    >> 200 ms [TraceNode] Custom/test/inner \n    <<  n/a Custom/test/inner\n  <<  n/a Custom/test/middle\n<<  n/a Custom/test/metric\n", s.to_debug_str(0))
-
-    # close them
-    inner.end_trace(0.21)
-    middle.end_trace(0.22)
-    s.end_trace(0.23)
-    assert_equal(">>   0 ms [TraceNode] Custom/test/metric \n  >> 100 ms [TraceNode] Custom/test/middle \n    >> 200 ms [TraceNode] Custom/test/inner \n    << 210 ms Custom/test/inner\n  << 220 ms Custom/test/middle\n<< 230 ms Custom/test/metric\n", s.to_debug_str(0))
-  end
-
-  def test_called_nodes_default
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal([], s.called_nodes)
-  end
-
-  def test_called_nodes_with_nodes
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    s.add_called_node(fake_node)
-
-    assert_equal([fake_node], s.called_nodes)
-  end
-
-  def test_duration
-    fake_entry_timestamp = mock('entry timestamp')
-    fake_exit_timestamp = mock('exit timestamp')
-    fake_result = mock('numeric')
-    fake_exit_timestamp.expects(:-).with(fake_entry_timestamp).returns(fake_result)
-    fake_result.expects(:to_f).returns(0.5)
-
-    s = NewRelic::Agent::Transaction::TraceNode.new(fake_entry_timestamp, 'Custom/test/metric')
-    s.end_trace(fake_exit_timestamp)
-    assert_equal(0.5, s.duration)
-  end
-
-  def test_exclusive_duration_no_children
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    s.expects(:duration).returns(0.5)
-    assert_equal(0.5, s.exclusive_duration)
-  end
-
-  def test_exclusive_duration_with_children
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-
-    s.expects(:duration).returns(0.5)
-
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    fake_node.expects(:duration).returns(0.1)
-
-    s.add_called_node(fake_node)
-
-    assert_equal(0.4, s.exclusive_duration)
-  end
-
-  def test_count_nodes_default
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal(1, s.count_nodes)
-  end
-
-  def test_count_nodes_with_children
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    fake_node.expects(:count_nodes).returns(1)
-
-    s.add_called_node(fake_node)
-
-    assert_equal(2, s.count_nodes)
-  end
-
-  def test_key_equals
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    # doing this to hold the reference to the hash
-    params = {}
-    s.params = params
-    assert_equal(params, s.params)
-
-    # should delegate to the same hash we have above
-    s[:foo] = 'correct'
-
-    assert_equal('correct', params[:foo])
-  end
-
-  def test_key
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    s.params = {:foo => 'correct'}
-    assert_equal('correct', s[:foo])
-  end
-
-  def test_params
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-
-    # should have a default value
-    assert_equal({}, s.instance_variable_get(:@params))
-    assert_equal({}, s.params)
-
-    # should otherwise take the value from the @params var
-    s.instance_eval { @params = {:foo => 'correct'} }
-    assert_equal({:foo => 'correct'}, s.params)
-  end
-
-  def test_each_node_default
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    # in the base case it just yields the block to itself
-    count = 0
-    s.each_node do |x|
-      count += 1
-      assert_equal(s, x)
-    end
-    # should only run once
-    assert_equal(1, count)
-  end
-
-  def test_each_node_with_children
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-
-    fake_node = mock('node')
-    fake_node.expects(:parent_node=).with(s)
-    fake_node.expects(:each_node).yields(fake_node)
-
-    s.add_called_node(fake_node)
-
-    count = 0
-    s.each_node do |x|
-      count += 1
-    end
-
-    assert_equal(2, count)
-  end
-
-  def test_each_node_with_nest_tracking
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-
-    summary = mock('summary')
-    summary.expects(:current_nest_count).twice.returns(0).then.returns(1)
-    summary.expects(:current_nest_count=).twice
-    s.each_node_with_nest_tracking do |x|
-      summary
-    end
-  end
-
-  def test_explain_sql_raising_an_error
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    config = { :adapter => 'mysql' }
-    statement = NewRelic::Agent::Database::Statement.new('SELECT')
-    statement.config = config
-    statement.explainer = NewRelic::Agent::Instrumentation::ActiveRecord::EXPLAINER
-    s.params = {:sql => statement}
-    NewRelic::Agent::Database.expects(:get_connection).with(config).raises(RuntimeError.new("whee"))
-    s.explain_sql
-  end
-
-  def test_explain_sql_can_handle_missing_config
-    # If TT node came over from Resque child, might not be a Statement
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    s.params = { :sql => "SELECT * FROM galaxy" }
-    s.explain_sql
-  end
-
-  def test_explain_sql_can_use_already_existing_plan
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    s.params = {
-      :sql => "SELECT * FROM galaxy",
-      :explain_plan => "EXPLAIN IT!"
-    }
-
-    assert_equal("EXPLAIN IT!", s.explain_sql)
-  end
-
-  def test_params_equal
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal({}, s.instance_variable_get(:@params))
-
-    params = {:foo => 'correct'}
-
-    s.params = params
-    assert_equal(params, s.instance_variable_get(:@params))
-  end
-
-  def test_obfuscated_sql
-    sql = 'select * from table where id = 1'
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    s[:sql] = NewRelic::Agent::Database::Statement.new(sql)
-    assert_equal('select * from table where id = ?', s.obfuscated_sql)
-  end
-
-  def test_called_nodes_equals
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal(nil, s.instance_eval { @called_nodes })
-    s.called_nodes = [1, 2, 3]
-    assert_equal([1, 2, 3], s.instance_eval { @called_nodes })
-  end
-
-  def test_parent_node_equals
-    s = NewRelic::Agent::Transaction::TraceNode.new(Time.now, 'Custom/test/metric')
-    assert_equal(nil, s.parent_node)
-    fake_node = mock('node')
-    s.send(:parent_node=, fake_node)
-    assert_equal(fake_node, s.parent_node)
-  end
-end

data/test/new_relic/agent/transaction/trace_test.rb

@@ -1,394 +0,0 @@
-# encoding: utf-8
-# This file is distributed under New Relic's license terms.
-# See https://github.com/newrelic/rpm/blob/master/LICENSE for complete details.
-
-require File.expand_path('../../../../test_helper.rb', __FILE__)
-require 'new_relic/agent/transaction/trace'
-
-class NewRelic::Agent::Transaction::TraceTest < Minitest::Test
-  def setup
-    freeze_time
-    @start_time = Time.now
-    @trace = NewRelic::Agent::Transaction::Trace.new(@start_time)
-    @trace.root_node.end_trace(@start_time)
-
-    filter = NewRelic::Agent.instance.attribute_filter
-    @fake_attributes = NewRelic::Agent::Transaction::Attributes.new(filter)
-
-    @trace.attributes = @fake_attributes
-  end
-
-  def test_start_time
-    assert_equal @start_time, @trace.start_time
-  end
-
-  def test_node_count
-    assert_equal 0, @trace.node_count
-  end
-
-  def test_sample_id_is_the_object_id
-    assert_equal @trace.object_id, @trace.sample_id
-  end
-
-  def test_create_node_increases_node_count
-    @trace.create_node(0.0, 'foo')
-    assert_equal 1, @trace.node_count
-  end
-
-  def test_duration_is_the_root_node_duration
-    assert_equal @trace.duration, @trace.root_node.duration
-  end
-
-  def test_create_node
-    result = @trace.create_node(0.0, 'goo')
-    assert_equal 0.0, result.entry_timestamp
-    assert_equal 'goo', result.metric_name
-  end
-
-  def test_each_node_delegates_to_root_node
-    block = Proc.new {}
-    @trace.root_node.expects(:each_node)
-    @trace.each_node(&block)
-  end
-
-  def test_collector_array_contains_start_time
-    expected = NewRelic::Helper.time_to_millis(@start_time)
-    assert_collector_array_contains(:start_time, expected)
-  end
-
-  def test_root_node
-    assert_equal 0.0, @trace.root_node.entry_timestamp
-    assert_equal "ROOT", @trace.root_node.metric_name
-  end
-
-  def test_prepare_to_send_returns_self
-    result = @trace.prepare_to_send!
-    assert_equal @trace, result
-  end
-
-  def test_prepare_to_send_returns_self_if_already_prepared
-    @trace.prepare_to_send!
-
-    result = @trace.prepare_to_send!
-    assert_equal @trace, result
-  end
-
-  def test_prepare_to_send_is_idempotent
-    NewRelic::Agent::Database.stubs(:should_record_sql?).returns true
-    @trace.expects(:collect_explain_plans!).once
-    @trace.expects(:prepare_sql_for_transmission!).once
-    @trace.prepare_to_send!
-    @trace.prepare_to_send!
-  end
-
-  def test_prepare_to_send_handles_sql_and_does_not_strip_if_should_record_sql_is_true
-    NewRelic::Agent::Database.stubs(:should_record_sql?).returns true
-    @trace.expects(:collect_explain_plans!).once
-    @trace.expects(:prepare_sql_for_transmission!).once
-    @trace.expects(:strip_sql!).never
-    @trace.prepare_to_send!
-  end
-
-  def test_prepare_to_send_strips_and_does_not_handle_sql_if_should_record_sql_is_false
-    NewRelic::Agent::Database.stubs(:should_record_sql?).returns false
-    @trace.expects(:collect_explain_plans!).never
-    @trace.expects(:prepare_sql_for_transmission!).never
-    @trace.expects(:strip_sql!).once
-    @trace.prepare_to_send!
-  end
-
-  def test_prepare_to_send_collects_explain_plans
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(2)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = ''
-
-    @trace.root_node.add_called_node(node)
-
-    with_config(:'transaction_tracer.explain_threshold' => 1) do
-      @trace.prepare_to_send!
-    end
-
-    assert node[:explain_plan]
-  end
-
-  def test_prepare_to_send_prepares_sql_for_transmission
-    NewRelic::Agent::Database.stubs(:record_sql_method).returns :obfuscated
-
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(2)
-    node[:sql] = "select * from pelicans where name = '1337807';"
-    @trace.root_node.add_called_node(node)
-
-    @trace.prepare_to_send!
-    assert_equal "select * from pelicans where name = ?;", node[:sql]
-  end
-
-  def test_prepare_to_send_strips_sql
-    NewRelic::Agent::Database.stubs(:should_record_sql?).returns false
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(2)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = 'select * from pelicans;'
-
-    @trace.root_node.add_called_node(node)
-    @trace.prepare_to_send!
-
-    refute node[:sql]
-  end
-
-  def test_collect_explain_plans!
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(2)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = ''
-
-    @trace.root_node.add_called_node(node)
-
-    with_config(:'transaction_tracer.explain_threshold' => 1) do
-      @trace.collect_explain_plans!
-    end
-
-    assert node[:explain_plan]
-  end
-
-  def test_collect_explain_plans_does_not_attach_explain_plans_if_duration_is_too_short
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(1)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = ''
-
-    @trace.root_node.add_called_node(node)
-
-    with_config(:'transaction_tracer.explain_threshold' => 2) do
-      @trace.collect_explain_plans!
-    end
-
-    refute node[:explain_plan]
-  end
-
-  def test_collect_explain_plans_does_not_attach_explain_plans_without_sql
-    node = @trace.create_node(0.0, 'nope_sql')
-    node.stubs(:duration).returns(2)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = nil
-
-    @trace.root_node.add_called_node(node)
-
-    with_config(:'transaction_tracer.explain_threshold' => 1) do
-      @trace.collect_explain_plans!
-    end
-
-    refute node[:explain_plan]
-  end
-
-  def test_collect_explain_plans_does_not_attach_explain_plans_if_db_says_not_to
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(2)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = ''
-
-    NewRelic::Agent::Database.stubs(:should_collect_explain_plans?).returns(false)
-
-    @trace.root_node.add_called_node(node)
-
-    with_config(:'transaction_tracer.explain_threshold' => 1) do
-      @trace.collect_explain_plans!
-    end
-
-    refute node[:explain_plan]
-  end
-
-  def test_prepare_sql_for_transmission_obfuscates_sql_if_record_sql_method_is_obfuscated
-    NewRelic::Agent::Database.stubs(:record_sql_method).returns :obfuscated
-
-    node = @trace.create_node(0.0, 'has_sql')
-    node[:sql] = "select * from pelicans where name = '1337807';"
-    @trace.root_node.add_called_node(node)
-
-    @trace.prepare_sql_for_transmission!
-    assert_equal "select * from pelicans where name = ?;", node[:sql]
-  end
-
-  def test_prepare_sql_for_transmission_does_not_modify_sql_if_record_sql_method_is_raw
-    NewRelic::Agent::Database.stubs(:record_sql_method).returns :raw
-
-    node = @trace.create_node(0.0, 'has_sql')
-    node[:sql] = NewRelic::Agent::Database::Statement.new "select * from pelicans where name = '1337807';"
-    @trace.root_node.add_called_node(node)
-
-    @trace.prepare_sql_for_transmission!
-    assert_equal "select * from pelicans where name = '1337807';", node[:sql]
-  end
-
-  def test_prepare_sql_for_transmission_removes_sql_if_record_sql_method_is_off
-    NewRelic::Agent::Database.stubs(:record_sql_method).returns :off
-
-    node = @trace.create_node(0.0, 'has_sql')
-    node[:sql] = "select * from pelicans where name = '1337807';"
-    @trace.root_node.add_called_node(node)
-
-    @trace.prepare_sql_for_transmission!
-    refute node[:sql]
-  end
-
-  def test_strip_sql!
-    node = @trace.create_node(0.0, 'has_sql')
-    node.stubs(:duration).returns(2)
-    node.stubs(:explain_sql).returns('')
-    node[:sql] = 'select * from pelicans;'
-
-    @trace.root_node.add_called_node(node)
-    @trace.strip_sql!
-
-    refute node[:sql]
-  end
-
-  def test_collector_array_contains_root_node_duration
-    @trace.root_node.end_trace(1)
-    assert_collector_array_contains(:duration, 1000)
-  end
-
-  def test_collector_array_contains_transaction_name
-    @trace.transaction_name = 'zork'
-    assert_collector_array_contains(:transaction_name, 'zork')
-  end
-
-  def test_transaction_name_gets_coerced_into_a_string
-    @trace.transaction_name = 1337807
-    assert_collector_array_contains(:transaction_name, '1337807')
-  end
-
-  def test_collector_array_contains_uri
-    @trace.uri = 'http://windows95tips.com/'
-    assert_collector_array_contains(:uri, 'http://windows95tips.com/')
-  end
-
-  def test_uri_gets_coerced_into_a_string
-    @trace.uri = 95
-    assert_collector_array_contains(:uri, '95')
-  end
-
-  def test_collector_array_contains_trace_tree
-    assert_collector_array_contains(:trace_tree, @trace.trace_tree)
-  end
-
-  def test_collector_array_contains_guid
-    @trace.guid = 'DEADBEEF8BADF00D'
-    assert_collector_array_contains(:guid, 'DEADBEEF8BADF00D')
-  end
-
-  def test_guid_gets_coerced_into_a_string
-    @trace.guid = 42
-    assert_collector_array_contains(:guid, '42')
-  end
-
-  def test_collector_array_contains_nil_for_reserved
-    assert_collector_array_contains(:reserved, nil)
-  end
-
-  def test_collector_array_contains_forced_true_if_in_an_xray_session
-    @trace.xray_session_id = 7
-    assert_collector_array_contains(:forced?, true)
-  end
-
-  def test_collector_array_contains_forced_false_if_not_in_an_xray_session
-    @trace.xray_session_id = nil
-    assert_collector_array_contains(:forced?, false)
-  end
-
-  def test_collector_array_contains_xray_session_id
-    @trace.xray_session_id = 112357
-    assert_collector_array_contains(:xray_session_id, 112357)
-  end
-
-  def test_xray_session_id_gets_coerced_to_an_integer
-    @trace.xray_session_id = '112357'
-    assert_collector_array_contains(:xray_session_id, 112357)
-  end
-
-  def test_xray_session_id_does_not_coerce_nil_to_an_integer
-    @trace.xray_session_id = nil
-    assert_collector_array_contains(:xray_session_id, nil)
-  end
-
-  def test_collector_array_contains_synthetics_resource_id
-    @fake_attributes.add_intrinsic_attribute(:synthetics_resource_id, '31415926')
-    assert_collector_array_contains(:synthetics_resource_id, '31415926')
-  end
-
-  def test_to_collector_array_encodes_trace_tree_with_given_encoder
-    fake_encoder = mock
-    fake_encoder.expects(:encode).with(@trace.trace_tree)
-    @trace.to_collector_array(fake_encoder)
-  end
-
-  def test_synthetics_resource_id_gets_coerced_to_a_string
-    @fake_attributes.add_intrinsic_attribute(:synthetics_resource_id, 31415926)
-    assert_collector_array_contains(:synthetics_resource_id, '31415926')
-  end
-
-  def test_trace_tree_coerces_start_time_to_a_float
-    assert_kind_of Float, @trace.trace_tree.first
-  end
-
-  def test_trace_tree_includes_start_time
-    assert_trace_tree_contains(:start_time, @start_time.to_f)
-  end
-
-  def test_trace_tree_includes_unused_legacy_request_params
-    assert_trace_tree_contains(:unused_legacy_request_params, {})
-  end
-
-  def test_trace_tree_includes_unused_legacy_custom_params
-    assert_trace_tree_contains(:unused_legacy_custom_params, {})
-  end
-
-  def test_trace_tree_contains_serialized_root_node
-    assert_trace_tree_contains(:root_node, @trace.root_node.to_array)
-  end
-
-  def test_trace_tree_contains_attributes
-    @fake_attributes.add_agent_attribute(:foo, 'bar', NewRelic::Agent::AttributeFilter::DST_ALL)
-    @fake_attributes.add_intrinsic_attribute(:foo, 'bar')
-    @fake_attributes.merge_custom_attributes(:foo => 'bar')
-
-    expected = {
-      'agentAttributes' => { :foo => 'bar' },
-      'userAttributes'  => { "foo" => 'bar' },
-      'intrinsics'      => { :foo => 'bar' }
-    }
-
-    assert_trace_tree_contains(:attributes, expected)
-  end
-
-  def assert_trace_tree_contains(key, expected)
-    indices = [
-      :start_time,
-      :unused_legacy_request_params,
-      :unused_legacy_custom_params,
-      :root_node,
-      :attributes
-    ]
-
-    assert_equal expected, @trace.trace_tree[indices.index(key)]
-  end
-
-  def assert_collector_array_contains(key, expected)
-    indices = [
-      :start_time,
-      :duration,
-      :transaction_name,
-      :uri,
-      :trace_tree,
-      :guid,
-      :reserved,
-      :forced?,
-      :xray_session_id,
-      :synthetics_resource_id
-    ]
-
-    encoder = NewRelic::Agent::NewRelicService::Encoders::Identity
-    assert_equal expected, @trace.to_collector_array(encoder)[indices.index(key)]
-  end
-end