newrelic_rpm 4.1.0.333 → 4.2.0.334

data/test/new_relic/agent/harvester_test.rb

@@ -1,79 +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/harvester'
-
-module NewRelic
-  module Agent
-    class HarvesterTest < Minitest::Test
-
-      attr_reader :harvester
-
-      def setup
-        @after_forker = stub_everything
-        @harvester = Harvester.new(nil, @after_forker)
-      end
-
-      def test_doesnt_mark_started_process_on_initialize
-        assert_nil harvester.starting_pid
-      end
-
-      def test_skips_out_early_if_already_started
-        harvester.mark_started
-        @after_forker.expects(:after_fork).never
-
-        with_config(:restart_thread_in_children => true) do
-          harvester.on_transaction
-        end
-      end
-
-      def test_calls_to_restart_by_default
-        pretend_started_in_another_process
-        @after_forker.expects(:after_fork).once
-
-        harvester.on_transaction
-      end
-
-      def test_calls_to_restart_if_explicitly_enabled
-        pretend_started_in_another_process
-        @after_forker.expects(:after_fork).once
-
-        with_config(:restart_thread_in_children => true) do
-          harvester.on_transaction
-        end
-      end
-
-      def test_doesnt_call_to_restart_if_explicitly_disabled
-        pretend_started_in_another_process
-        @after_forker.expects(:after_fork).never
-
-        with_config(:restart_thread_in_children => false) do
-          harvester.on_transaction
-        end
-      end
-
-      def test_doesnt_call_to_restart_if_harvest_thread_disabled
-        pretend_started_in_another_process
-        @after_forker.expects(:after_fork).never
-
-        with_config(:disable_harvest_thread => true) do
-          harvester.on_transaction
-        end
-      end
-
-      def test_calls_restart_every_time_until_marked_started
-        pretend_started_in_another_process
-        @after_forker.expects(:after_fork).times(2)
-
-        harvester.on_transaction
-        harvester.on_transaction
-      end
-
-      def pretend_started_in_another_process
-        harvester.mark_started(Process.pid - 1)
-      end
-    end
-  end
-end

data/test/new_relic/agent/hostname_test.rb

@@ -1,121 +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/hostname'
-
-module NewRelic
-  module Agent
-    class HostnameTest < Minitest::Test
-      def setup
-        NewRelic::Agent::Hostname.instance_variable_set(:@hostname, nil)
-        Socket.stubs(:gethostname).returns('Rivendell')
-      end
-
-      def teardown
-        NewRelic::Agent::Hostname.instance_variable_set(:@hostname, nil)
-      end
-
-      def test_get_returns_socket_hostname
-        assert_equal 'Rivendell', NewRelic::Agent::Hostname.get
-      end
-
-      def test_get_uses_dyno_name_if_dyno_env_set_and_dyno_names_enabled
-        with_dyno_name('Imladris', :'heroku.use_dyno_names' => true) do
-          assert_equal 'Imladris', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_get_uses_socket_gethostname_if_dyno_env_set_and_dyno_names_disabled
-        with_dyno_name('Imladris', :'heroku.use_dyno_names' => false) do
-          assert_equal 'Rivendell', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_shortens_to_prefix_if_using_dyno_names_and_matches
-        with_dyno_name('Imladris.1', :'heroku.use_dyno_names' => true,
-                                     :'heroku.dyno_name_prefixes_to_shorten' => ['Imladris']) do
-          assert_equal 'Imladris.*', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_does_not_shorten_if_not_using_dyno_names
-        with_dyno_name('Imladris', :'heroku.use_dyno_names' => false,
-                                   :'heroku.dyno_name_prefixes_to_shorten' => ['Rivendell']) do
-          Socket.stubs(:gethostname).returns('Rivendell.1')
-          assert_equal 'Rivendell.1', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_only_shortens_if_matches_prefix_and_dot
-        with_dyno_name('ImladrisImladrisFakeout.1',
-                       :'heroku.use_dyno_names' => true,
-                       :'heroku.dyno_name_prefixes_to_shorten' => ['Imladris']) do
-          assert_equal 'ImladrisImladrisFakeout.1', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_shortens_to_prefixes_from_string
-        with_dyno_name('Imladris.1', :'heroku.use_dyno_names' => true,
-                                     :'heroku.dyno_name_prefixes_to_shorten' => 'Imladris') do
-          assert_equal 'Imladris.*', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_shortens_to_prefixes_from_string_allows_csv
-        with_dyno_name('Imladris.1', :'heroku.use_dyno_names' => true,
-                                     :'heroku.dyno_name_prefixes_to_shorten' => 'Rivendell,Imladris') do
-          assert_equal 'Imladris.*', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_shortens_to_prefixes_with_empty_string
-        with_dyno_name('Imladris.1', :'heroku.use_dyno_names' => true,
-                                     :'heroku.dyno_name_prefixes_to_shorten' => '') do
-          assert_equal 'Imladris.1', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_shortens_to_prefixes_with_unsupported_object
-        with_dyno_name('Imladris.1', :'heroku.use_dyno_names' => true,
-                                     :'heroku.dyno_name_prefixes_to_shorten' => Object.new) do
-          expects_logging(:error, includes('heroku.dyno_name_prefixes_to_shorten'), instance_of(ArgumentError))
-          assert_equal 'Imladris.1', NewRelic::Agent::Hostname.get
-        end
-      end
-
-      def test_local_predicate_true_when_host_local
-        hosts = %w(localhost 0.0.0.0 127.0.0.1 0:0:0:0:0:0:0:1
-                   0:0:0:0:0:0:0:0 ::1 ::)
-        hosts.each do |host|
-          assert NewRelic::Agent::Hostname.local?(host)
-        end
-      end
-
-      def test_localhost_predicate_false_when_host_nonlocal
-        hosts = %w(drscheffler jonan-show jonan.tm)
-        hosts.each do |host|
-          refute NewRelic::Agent::Hostname.local?(host)
-        end
-      end
-
-      def test_get_external_returns_host_for_localhost
-        assert_equal "Rivendell", NewRelic::Agent::Hostname.get_external("localhost")
-      end
-
-      def test_get_external_returns_argument_for_nonlocalhost
-        assert_equal "drscheffler", NewRelic::Agent::Hostname.get_external("drscheffler")
-      end
-
-      def with_dyno_name(dyno_name, config_options)
-        with_config(config_options) do
-          ENV['DYNO'] = dyno_name
-          yield
-        end
-      ensure
-        ENV.delete('DYNO')
-      end
-    end
-  end
-end

data/test/new_relic/agent/http_clients/uri_util_test.rb

@@ -1,64 +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/http_clients/uri_util'
-
-class URIUtilTest < Minitest::Test
-
-  def test_filter_uri
-    assert_filtered("http://foo.com/bar/baz",
-                    "http://foo.com/bar/baz")
-  end
-
-  def test_filter_uri_custom_port
-    assert_filtered("http://foo.com:1234/bar/baz",
-                    "http://foo.com:1234/bar/baz")
-  end
-
-  def test_filtered_uri_omits_query_params
-    assert_filtered("http://foo.com/bar/baz?a=1&b=2",
-                    "http://foo.com/bar/baz")
-  end
-
-  def test_filtered_uri_omits_fragment
-    assert_filtered("http://foo.com/bar/baz#fragment",
-                    "http://foo.com/bar/baz")
-  end
-
-  def test_filtered_uri_omits_query_params_and_fragment
-    assert_filtered("http://foo.com/bar/baz?a=1&b=2#fragment",
-                    "http://foo.com/bar/baz")
-  end
-
-  def test_filtered_uri_reflects_use_of_ssl
-    assert_filtered('https://foo.com/bar/baz',
-                    "https://foo.com/bar/baz")
-  end
-
-  def test_filtered_uri_reflects_use_of_ssl_with_custom_port
-    assert_filtered('https://foo.com:9999/bar/baz',
-                    "https://foo.com:9999/bar/baz")
-  end
-
-  def test_filter_uri_with_full_uri_request_path
-    assert_filtered("http://foo.com/bar/baz?a=1&b=2#fragment",
-                    "http://foo.com/bar/baz")
-  end
-
-  def test_filter_uri_with_full_uri_request_path_https
-    assert_filtered("https://foo.com/bar/baz?a=1&b=2#fragment",
-                    "https://foo.com/bar/baz")
-  end
-
-  def test_strips_credentials_embedded_in_uri
-    assert_filtered("http://user:pass@foo.com/bar/baz",
-                    "http://foo.com/bar/baz")
-  end
-
-  def assert_filtered(original, expected)
-    assert_equal(expected, NewRelic::Agent::HTTPClients::URIUtil.filter_uri(URI(original)))
-  end
-
-end

data/test/new_relic/agent/inbound_request_monitor_test.rb

@@ -1,49 +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/inbound_request_monitor'
-
-module NewRelic::Agent
-  class InboundRequestMonitorTest < Minitest::Test
-
-    ENCODING_KEY_NOOP         = "\0"
-
-    def setup
-      @events  = NewRelic::Agent::EventListener.new
-      @monitor = NewRelic::Agent::InboundRequestMonitor.new(@events)
-
-      @config = { :encoding_key => ENCODING_KEY_NOOP }
-      NewRelic::Agent.config.add_config_for_testing(@config)
-
-      class << @monitor
-        define_method(:on_finished_configuring) do |*_|
-        end
-      end
-
-      @events.notify(:finished_configuring)
-    end
-
-    def teardown
-      NewRelic::Agent.config.remove_config(@config)
-    end
-
-    def test_deserialize
-      payload = @monitor.obfuscator.obfuscate("[1,2,3]")
-      assert_equal [1, 2, 3], @monitor.deserialize_header(payload, "the_key")
-    end
-
-    def test_deserialize_nonsense
-      expects_logging(:debug, includes("the_key"))
-      assert_nil @monitor.deserialize_header("asdf", "the_key")
-    end
-
-    def test_deserialize_with_invalid_json
-      payload = @monitor.obfuscator.obfuscate("[1,2,3")
-
-      expects_logging(:debug, includes("the_key"))
-      assert_nil @monitor.deserialize_header(payload, "the_key")
-    end
-  end
-end

data/test/new_relic/agent/instrumentation/action_cable_subscriber_test.rb

@@ -1,124 +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.
-
-if defined?(::Rails) && ::Rails::VERSION::MAJOR.to_i >= 5
-
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
-require 'new_relic/agent/instrumentation/action_cable_subscriber'
-
-
-module NewRelic
-  module Agent
-    module Instrumentation
-      class ActionCableSubscriberTest < Minitest::Test
-
-        def setup
-          freeze_time
-          @subscriber = ActionCableSubscriber.new
-
-          NewRelic::Agent.drop_buffered_data
-          @stats_engine = NewRelic::Agent.instance.stats_engine
-          @stats_engine.clear_stats
-          NewRelic::Agent.manual_start
-          NewRelic::Agent::TransactionState.tl_clear_for_testing
-        end
-
-        def teardown
-          NewRelic::Agent.shutdown
-          @stats_engine.clear_stats
-        end
-
-        def test_creates_web_transaction
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          assert NewRelic::Agent::TransactionState.tl_get.in_web_transaction?
-          advance_time(1.0)
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-
-          assert_equal('Controller/ActionCable/TestChannel/test_action',
-                       NewRelic::Agent.instance.transaction_sampler.last_sample.transaction_name)
-          assert_equal('Controller/ActionCable/TestChannel/test_action',
-                       NewRelic::Agent.instance.transaction_sampler.last_sample.root_node.called_nodes[0].metric_name)
-        end
-
-        def test_records_apdex_metrics
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          advance_time(1.5)
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-
-          expected_values = { :apdex_f => 0, :apdex_t => 1, :apdex_s => 0 }
-          assert_metrics_recorded(
-            'Apdex/ActionCable/TestChannel/test_action' => expected_values,
-            'Apdex' => expected_values
-          )
-        end
-
-        def test_sets_default_transaction_name_on_start
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          assert_equal 'Controller/ActionCable/TestChannel/test_action', NewRelic::Agent::Transaction.tl_current.best_name
-        ensure
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-        end
-
-        def test_sets_default_transaction_keeps_name_through_stop
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          txn = NewRelic::Agent::Transaction.tl_current
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-          assert_equal 'Controller/ActionCable/TestChannel/test_action', txn.best_name
-        end
-
-        def test_sets_transaction_name
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          NewRelic::Agent.set_transaction_name('something/else')
-          assert_equal 'Controller/ActionCable/something/else', NewRelic::Agent::Transaction.tl_current.best_name
-        ensure
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-        end
-
-        def test_sets_transaction_name_holds_through_stop
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          txn = NewRelic::Agent::Transaction.tl_current
-          NewRelic::Agent.set_transaction_name('something/else')
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-          assert_equal 'Controller/ActionCable/something/else', txn.best_name
-        end
-
-        def test_creates_tt_node_for_transmit
-          @subscriber.start('perform_action.action_cable', :id, payload_for_perform_action)
-          assert NewRelic::Agent::TransactionState.tl_get.in_web_transaction?
-          @subscriber.start('transmit.action_cable', :id, payload_for_transmit)
-          advance_time(1.0)
-          @subscriber.finish('transmit.action_cable', :id, payload_for_transmit)
-          @subscriber.finish('perform_action.action_cable', :id, payload_for_perform_action)
-
-          sample = NewRelic::Agent.instance.transaction_sampler.last_sample
-
-          assert_equal('Controller/ActionCable/TestChannel/test_action', sample.transaction_name)
-          metric_name = 'Ruby/ActionCable/TestChannel/transmit'
-          refute_nil(find_node_with_name(sample, metric_name), "Expected trace to have node with name: #{metric_name}")
-        end
-
-        def test_does_not_record_unscoped_metrics_nor_create_trace_for_transmit_outside_of_active_txn
-          @subscriber.start('transmit.action_cable', :id, payload_for_transmit)
-          advance_time(1.0)
-          @subscriber.finish('transmit.action_cable', :id, payload_for_transmit)
-
-          sample = NewRelic::Agent.instance.transaction_sampler.last_sample
-
-          assert_nil sample, "Did not expect a transaction to be created for transmit"
-          refute_metrics_recorded ['Ruby/ActionCable/TestChannel/transmit']
-        end
-
-        def payload_for_perform_action action = 'test_action'
-          {:channel_class => "TestChannel", :action => action.to_sym, :data => {"action"=>"#{action}"}}
-        end
-
-        def payload_for_transmit data = {}, via = nil
-          {:channel_class => "TestChannel", :data => data, :via => via}
-        end
-      end
-    end
-  end
-end
-
-end

data/test/new_relic/agent/instrumentation/action_controller_subscriber_test.rb

@@ -1,299 +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.
-
-if defined?(::Rails)
-
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
-require 'new_relic/agent/instrumentation/action_controller_subscriber'
-
-class NewRelic::Agent::Instrumentation::ActionControllerSubscriberTest < Minitest::Test
-  class TestController
-    include NewRelic::Agent::Instrumentation::ControllerInstrumentation
-
-    def self.controller_path
-      'test'
-    end
-
-    def action_name
-      'test'
-    end
-
-    newrelic_ignore :only => :ignored_action
-    newrelic_ignore_apdex :only => :ignored_apdex
-    newrelic_ignore_enduser :only => :ignored_enduser
-  end
-
-  def setup
-    freeze_time
-    @subscriber = NewRelic::Agent::Instrumentation::ActionControllerSubscriber.new
-    NewRelic::Agent.drop_buffered_data
-    @entry_payload = {
-      :controller => TestController.to_s,
-      :action => 'index',
-      :format => :html,
-      :method => 'GET',
-      :path => '/tests',
-      :params => { :controller => 'test_controller', :action => 'index' },
-    }
-    @exit_payload = @entry_payload.merge(:status => 200, :view_runtime => 5.0,
-                                         :db_runtime => 0.5 )
-    @stats_engine = NewRelic::Agent.instance.stats_engine
-    @stats_engine.clear_stats
-    NewRelic::Agent.manual_start
-    NewRelic::Agent::TransactionState.tl_clear_for_testing
-  end
-
-  def teardown
-    NewRelic::Agent.shutdown
-    @stats_engine.clear_stats
-  end
-
-  def test_record_controller_metrics
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    advance_time(2)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    expected_values = { :call_count => 1, :total_call_time => 2.0 }
-    assert_metrics_recorded(
-      'Controller/test/index' => expected_values,
-      'HttpDispatcher' => expected_values
-    )
-  end
-
-  def test_record_apdex_metrics
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    advance_time(1.5)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    expected_values = { :apdex_f => 0, :apdex_t => 1, :apdex_s => 0 }
-    assert_metrics_recorded(
-      'Apdex/test/index' => expected_values,
-      'Apdex' => expected_values
-    )
-  end
-
-  def test_record_apdex_metrics_with_error
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    advance_time(1.5)
-
-    error = StandardError.new("boo")
-    @exit_payload[:exception] = error
-    NewRelic::Agent.notice_error(error)
-
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    expected_values = { :apdex_f => 1, :apdex_t => 0, :apdex_s => 0 }
-    assert_metrics_recorded(
-      'Apdex/test/index' => expected_values,
-      'Apdex' => expected_values
-    )
-  end
-
-  def test_records_scoped_metrics_for_evented_child_txn
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    @subscriber.start('process_action.action_controller', :id, @entry_payload \
-                        .merge(:action => 'child', :path => '/child'))
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload \
-                         .merge(:action => 'child', :path => '/child'))
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    assert_metrics_recorded(
-      ['Nested/Controller/test/child', 'Controller/test/child'] => { :call_count => 1 }
-    )
-  end
-
-  def test_records_scoped_metrics_for_traced_child_txn
-    controller = TestController.new
-    controller.perform_action_with_newrelic_trace(:category => :controller,
-                                                  :name => 'index',
-                                                  :class_name => 'test') do
-      @subscriber.start('process_action.action_controller', :id, @entry_payload \
-                          .merge(:action => 'child', :path => '/child'))
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload \
-                           .merge(:action => 'child', :path => '/child'))
-    end
-
-    assert_metrics_recorded(
-      ['Nested/Controller/test/child', 'Controller/test/child'] => { :call_count => 1 }
-    )
-  end
-
-  def test_sets_default_transaction_name_on_start
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    assert_equal 'Controller/test/index', NewRelic::Agent::Transaction.tl_current.best_name
-  ensure
-    @subscriber.finish('process_action.action_controller', :id, @entry_payload)
-  end
-
-  def test_sets_default_transaction_keeps_name_through_stop
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    txn = NewRelic::Agent::Transaction.tl_current
-    @subscriber.finish('process_action.action_controller', :id, @entry_payload)
-    assert_equal 'Controller/test/index', txn.best_name
-  end
-
-  def test_sets_transaction_name
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    NewRelic::Agent.set_transaction_name('something/else')
-    assert_equal 'Controller/something/else', NewRelic::Agent::Transaction.tl_current.best_name
-  ensure
-    @subscriber.finish('process_action.action_controller', :id, @entry_payload)
-  end
-
-  def test_sets_transaction_name_holds_through_stop
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    txn = NewRelic::Agent::Transaction.tl_current
-    NewRelic::Agent.set_transaction_name('something/else')
-    @subscriber.finish('process_action.action_controller', :id, @entry_payload)
-    assert_equal 'Controller/something/else', txn.best_name
-  end
-
-  def test_record_nothing_for_ignored_action
-    @entry_payload[:action] = 'ignored_action'
-    @exit_payload[:action] = 'ignored_action'
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    assert_metrics_not_recorded([
-      'Controller/test/ignored_action',
-      'Apdex/test/ignored_action',
-      'Apdex',
-      'HttpDispatcher'
-    ])
-  end
-
-  def test_record_no_apdex_metric_for_ignored_apdex_action
-    @entry_payload[:action] = 'ignored_apdex'
-    @exit_payload[:action] = 'ignored_apdex'
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    assert_metrics_recorded(['Controller/test/ignored_apdex', 'HttpDispatcher'])
-    assert_metrics_not_recorded(['Apdex', 'Apdex/test/ignored_apdex'])
-  end
-
-  def test_ignore_end_user
-    @entry_payload[:action] = 'ignored_enduser'
-    @exit_payload[:action] = 'ignored_enduser'
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    txn = NewRelic::Agent::Transaction.tl_current
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    assert txn.ignore_enduser?
-  end
-
-  def test_record_busy_time
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    advance_time(1)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-    NewRelic::Agent::BusyCalculator.harvest_busy
-
-    assert_metrics_recorded('Instance/Busy' => { :call_count => 1, :total_call_time => 1.0 })
-  end
-
-  def test_creates_transaction
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    assert_equal('Controller/test/index',
-                 NewRelic::Agent.instance.transaction_sampler \
-                   .last_sample.transaction_name)
-    assert_equal('Controller/test/index',
-                 NewRelic::Agent.instance.transaction_sampler \
-                   .last_sample.root_node.called_nodes[0].metric_name)
-  end
-
-  def test_applies_txn_name_rules
-    rule_specs = [{'match_expression' => 'test', 'replacement' => 'taste'}]
-
-    with_transaction_renaming_rules(rule_specs) do
-      @subscriber.start('process_action.action_controller', :id, @entry_payload)
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-    end
-
-    assert_metrics_recorded(['Controller/taste/index'])
-    assert_metrics_not_recorded(['Controller/test/index'])
-  end
-
-  def test_record_queue_time_metrics
-    app = lambda do |env|
-      @subscriber.start('process_action.action_controller', :id, @entry_payload)
-      advance_time(2)
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-    end
-
-    t0 = Time.now
-    env = { 'HTTP_X_REQUEST_START' => (t0 - 5).to_f.to_s }
-    ::NewRelic::Rack::AgentHooks.new(app).call(env)
-
-    assert_metrics_recorded(
-      'WebFrontend/QueueTime' => {
-        :call_count => 1,
-        :total_call_time => 5.0
-      }
-    )
-  end
-
-  def test_dont_record_queue_time_if_no_header
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    assert_metrics_not_recorded('WebFrontend/QueueTime')
-  end
-
-  def test_dont_record_queue_time_in_nested_transaction
-    app = lambda do |env|
-      @subscriber.start('process_action.action_controller',  :id, @entry_payload)
-      @subscriber.start('process_action.action_controller',  :id, @entry_payload)
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-    end
-
-    t0 = Time.now
-    env = { 'HTTP_X_REQUEST_START' => (t0 - 5).to_f.to_s }
-    ::NewRelic::Rack::AgentHooks.new(app).call(env)
-
-    assert_metrics_recorded(
-      'WebFrontend/QueueTime' => {
-        :call_count => 1,
-        :total_call_time => 5.0
-      }
-    )
-  end
-
-  def test_records_request_params_in_txn
-    with_config(:capture_params => true) do
-      @entry_payload[:params]['number'] = '666'
-      @subscriber.start('process_action.action_controller', :id, @entry_payload)
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-    end
-
-    sample = NewRelic::Agent.instance.transaction_sampler.last_sample
-    assert_equal('666', attributes_for(sample, :agent)['request.parameters.number'])
-  end
-
-  def test_records_filtered_request_params_in_txn
-    with_config(:capture_params => true) do
-      @entry_payload[:params]['password'] = 'secret'
-      @subscriber.start('process_action.action_controller', :id, @entry_payload)
-      @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-    end
-
-    sample = NewRelic::Agent.instance.transaction_sampler.last_sample
-    assert_equal('[FILTERED]', attributes_for(sample, :agent)['request.parameters.password'])
-  end
-
-  def test_records_custom_parameters_in_txn
-    @subscriber.start('process_action.action_controller', :id, @entry_payload)
-    NewRelic::Agent.add_custom_attributes('number' => '666')
-    @subscriber.finish('process_action.action_controller', :id, @exit_payload)
-
-    sample = NewRelic::Agent.instance.transaction_sampler.last_sample
-    assert_equal('666', attributes_for(sample, :custom)['number'])
-  end
-end if ::Rails::VERSION::MAJOR.to_i >= 4
-
-else
-  puts "Skipping tests in #{__FILE__} because Rails is unavailable"
-end