newrelic_rpm 2.14.1 → 3.0.0.beta1

data/test/config/test_control.rb

@@ -19,10 +19,10 @@ class NewRelic::Control::Frameworks::Test < parent_class
       :rails
     end
   end
-  
+
   def initialize *args
     super
-    setup_log 
+    setup_log
   end
   # when running tests, don't write out stderr
   def log!(msg, level=:info)
@@ -37,7 +37,7 @@ class NewRelic::Control::Frameworks::Test < parent_class
       def draw_with_test_route
         draw_without_test_route do | map |
           map.connect ':controller/:action/:id'
-          yield map        
+          yield map
         end
       end
       alias_method_chain :draw, :test_route

data/test/new_relic/agent/agent/connect_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.join(File.dirname(__FILE__),'..', '..', '..','test_helper')) 
+require File.expand_path(File.join(File.dirname(__FILE__),'..', '..', '..','test_helper'))
 class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
   require 'new_relic/agent/agent'
   include NewRelic::Agent::Agent::Connect
@@ -9,7 +9,7 @@ class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
     @connect_attempts = 1
     @connect_retry_period = 0
   end
-  
+
   def test_tried_to_connect?
     # base case, should default to false
     assert !tried_to_connect?({})
@@ -20,7 +20,7 @@ class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
     @connected = true
     assert tried_to_connect?({})
   end
-  
+
   def test_tried_to_connect_forced
     # is false if force_reconnect is true
     assert !tried_to_connect?({:force_reconnect => true})
@@ -116,7 +116,7 @@ class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
     log.expects(:debug).never
     log_seed_token
   end
-  
+
   def mocks_for_positive_environment_for_connect(value_for_control)
     control = mocked_control
     control.expects(:'[]').with('send_environment_info').once.returns(value_for_control)
@@ -124,7 +124,7 @@ class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
     fake_env.expects(:snapshot).once.returns("snapshot")
     control.expects(:local_env).once.returns(fake_env)
   end
-  
+
   def test_environment_for_connect_nil
     mocks_for_positive_environment_for_connect(nil)
     assert_equal 'snapshot', environment_for_connect
@@ -195,6 +195,27 @@ class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
     enable_random_samples!(sampling_rate)
   end
 
+  def test_enable_random_samples_with_no_sampling_rate
+    # testing that we set a sane default for sampling rate
+    sampling_rate = 0
+    ts = @transaction_sampler = mock('ts')
+    ts.expects(:random_sampling=).with(true)
+    ts.expects(:sampling_rate=).with(10)
+    ts.expects(:sampling_rate).returns(10)
+    log.expects(:info).with("Transaction sampling enabled, rate = 10")
+    enable_random_samples!(sampling_rate)
+  end
+
+  def test_configure_transaction_tracer_with_random_sampling
+    @config_should_send_samples = true
+    @should_send_random_samples = true
+    @slowest_transaction_threshold = 5
+    log.expects(:debug).with('Transaction tracing threshold is 5 seconds.')
+    self.expects(:enable_random_samples!).with(10)
+    configure_transaction_tracer!(true, 10)
+    assert @should_send_samples
+  end
+
   def test_configure_transaction_tracer_positive
     @config_should_send_samples = true
     @slowest_transaction_threshold = 5
@@ -266,7 +287,7 @@ class NewRelic::Agent::Agent::ConnectTest < Test::Unit::TestCase
     assert_equal 'pasta sauce', @report_period
     assert_equal 'tamales', @url_rules
   end
-  
+
   private
 
   def mocked_control

data/test/new_relic/agent/agent/start_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper')) 
+require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
 class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
   require 'new_relic/agent/agent'
   include NewRelic::Agent::Agent::Start
@@ -14,7 +14,7 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     self.expects(:started?).returns(false)
     assert !already_started?
   end
-  
+
   def test_disabled_positive
     control = mocked_control
     control.expects(:agent_enabled?).returns(false)
@@ -79,14 +79,14 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     set_sql_recording!
     assert_equal :off, @record_sql, "should be set to :off, was #{@record_sql}"
   end
-  
+
   def test_set_sql_recording_none
     self.expects(:sampler_config).returns({'record_sql' => 'none'})
     self.expects(:log_sql_transmission_warning?)
     set_sql_recording!
     assert_equal :off, @record_sql, "should be set to :off, was #{@record_sql}"
   end
-  
+
   def test_set_sql_recording_raw
     self.expects(:sampler_config).returns({'record_sql' => 'raw'})
     self.expects(:log_sql_transmission_warning?)
@@ -129,13 +129,13 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     fake_sampler_config.expects(:fetch).with('explain_threshold', 0.5)
     fake_sampler_config.expects(:fetch).with('explain_enabled', true)
     self.expects(:set_sql_recording!)
-    
+
     fake_sampler_config.expects(:fetch).with('transaction_threshold', 2.0)
     self.expects(:apdex_f_threshold?).returns(true)
     self.expects(:apdex_f)
     config_transaction_tracer
   end
-  
+
   def test_check_config_and_start_agent_disabled
     self.expects(:monitoring?).returns(false)
     check_config_and_start_agent
@@ -146,7 +146,7 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     self.expects(:has_correct_license_key?).returns(false)
     check_config_and_start_agent
   end
-  
+
   def test_check_config_and_start_agent_forking
     self.expects(:monitoring?).returns(true)
     self.expects(:has_correct_license_key?).returns(true)
@@ -164,7 +164,7 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     self.expects(:install_exit_handler)
     check_config_and_start_agent
   end
-  
+
   def test_check_config_and_start_agent_sync
     self.expects(:monitoring?).returns(true)
     self.expects(:has_correct_license_key?).returns(true)
@@ -176,17 +176,17 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     self.expects(:install_exit_handler)
     check_config_and_start_agent
   end
-  
+
   def test_connect_in_foreground
     self.expects(:connect).with({:keep_retrying => false })
     connect_in_foreground
   end
-  
+
   def at_exit
     yield
   end
   private :at_exit
-  
+
   def test_install_exit_handler_positive
     control = mocked_control
     control.expects(:send_data_on_exit).returns(true)
@@ -199,7 +199,7 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
     self.expects(:shutdown)
     install_exit_handler
   end
-  
+
   def test_install_exit_handler_negative
     control = mocked_control
     control.expects(:send_data_on_exit).returns(false)
@@ -321,7 +321,7 @@ class NewRelic::Agent::Agent::StartTest < Test::Unit::TestCase
   def test_log_if_negative
     assert !log_if(false, :warn, "WHEE")
   end
-  
+
   private
 
   def mocked_log

data/test/new_relic/agent/agent/start_worker_thread_test.rb

@@ -1,8 +1,8 @@
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper')) 
+require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
 class NewRelic::Agent::Agent::StartWorkerThreadTest < Test::Unit::TestCase
   require 'new_relic/agent/agent'
   include NewRelic::Agent::Agent::StartWorkerThread
-  
+
   def test_deferred_work_connects
     self.expects(:catch_errors).yields
     self.expects(:connect).with('connection_options')
@@ -71,7 +71,7 @@ class NewRelic::Agent::Agent::StartWorkerThreadTest < Test::Unit::TestCase
     self.expects(:harvest_and_send_errors)
     create_and_run_worker_loop
   end
-  
+
   def test_handle_force_restart
     # hooray for methods with no branches
     error = mock('exception')
@@ -80,12 +80,12 @@ class NewRelic::Agent::Agent::StartWorkerThreadTest < Test::Unit::TestCase
     log.expects(:info).with('a message')
     self.expects(:reset_stats)
     self.expects(:sleep).with(30)
-    
+
     @metric_ids = 'this is not an empty hash'
     @connected = true
-    
+
     handle_force_restart(error)
-    
+
     assert_equal({}, @metric_ids)
     assert @connected.nil?
   end
@@ -125,7 +125,7 @@ class NewRelic::Agent::Agent::StartWorkerThreadTest < Test::Unit::TestCase
     self.expects(:disconnect)
     handle_other_error(error)
   end
-  
+
   def test_catch_errors_force_restart
     @runs = 0
     error = NewRelic::Agent::ForceRestartException.new
@@ -138,7 +138,7 @@ class NewRelic::Agent::Agent::StartWorkerThreadTest < Test::Unit::TestCase
     end
     assert_equal 3, @runs, 'should retry the block when it fails'
   end
-  
+
   private
 
   def mocked_log

data/test/new_relic/agent/agent_test.rb

@@ -0,0 +1,85 @@
+require File.expand_path(File.join(File.dirname(__FILE__),'..','..','test_helper'))
+module NewRelic
+  module Agent
+    class AgentTest < Test::Unit::TestCase
+      def test_sql_normalization
+
+        # basic statement
+        assert_equal "INSERT INTO X values(?,?, ? , ?)",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "INSERT INTO X values('test',0, 1 , 2)")
+
+        # escaped literals
+        assert_equal "INSERT INTO X values(?, ?,?, ? , ?)",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "INSERT INTO X values('', 'jim''s ssn',0, 1 , 'jim''s son''s son')")
+
+        # multiple string literals
+        assert_equal "INSERT INTO X values(?,?,?, ? , ?)",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "INSERT INTO X values('jim''s ssn','x',0, 1 , 2)")
+
+        # empty string literal
+        # NOTE: the empty string literal resolves to empty string, which for our purposes is acceptable
+        assert_equal "INSERT INTO X values(?,?,?, ? , ?)",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "INSERT INTO X values('','x',0, 1 , 2)")
+
+        # try a select statement
+        assert_equal "select * from table where name=? and ssn=?",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "select * from table where name='jim gochee' and ssn=0012211223")
+
+        # number literals embedded in sql - oh well
+        assert_equal "select * from table_? where name=? and ssn=?",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "select * from table_007 where name='jim gochee' and ssn=0012211223")
+      end
+
+      def test_sql_normalization__single_quotes
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, "INSERT 'this isn''t a real value' into table")
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT '"' into table])
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT ' "some text" \" ' into table])
+        #    could not get this one licked.  no biggie
+        #    assert_equal "INSERT ? into table",
+        #    NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT '\'' into table])
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT ''' ' into table])
+      end
+      def test_sql_normalization__double_quotes
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT "this isn't a real value" into table])
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT "'" into table])
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT " \" " into table])
+        assert_equal "INSERT ? into table",
+        NewRelic::Agent.instance.send(:default_sql_obfuscator, %q[INSERT " 'some text' " into table])
+      end
+      def test_sql_obfuscation_filters
+        orig =  NewRelic::Agent.agent.obfuscator
+
+        NewRelic::Agent.set_sql_obfuscator(:replace) do |sql|
+          sql = "1" + sql
+        end
+
+        sql = "SELECT * FROM TABLE 123 'jim'"
+
+        assert_equal "1" + sql, NewRelic::Agent.instance.obfuscator.call(sql)
+
+        NewRelic::Agent.set_sql_obfuscator(:before) do |sql|
+          sql = "2" + sql
+        end
+
+        assert_equal "12" + sql, NewRelic::Agent.instance.obfuscator.call(sql)
+
+        NewRelic::Agent.set_sql_obfuscator(:after) do |sql|
+          sql = sql + "3"
+        end
+
+        assert_equal "12" + sql + "3", NewRelic::Agent.instance.obfuscator.call(sql)
+
+        NewRelic::Agent.agent.set_sql_obfuscator(:replace, &orig)
+      end
+
+
+    end
+  end
+end

data/test/new_relic/agent/agent_test_controller.rb

@@ -1,4 +1,4 @@
-# Defining a test controller class with a superclass, used to 
+# Defining a test controller class with a superclass, used to
 # verify correct attribute inheritence
 class NewRelic::Agent::SuperclassController <  ActionController::Base
   def base_action
@@ -8,7 +8,7 @@ end
 # This is a controller class used in testing controller instrumentation
 class NewRelic::Agent::AgentTestController < NewRelic::Agent::SuperclassController
   filter_parameter_logging :social_security_number
-  
+
   @@headers_to_add = nil
 
   def index
@@ -21,7 +21,7 @@ class NewRelic::Agent::AgentTestController < NewRelic::Agent::SuperclassControll
   def action_inline
     render(:inline => "<%= 'foo' %>fah")
   end
-  
+
   def action_to_render
     render :text => params.inspect
   end
@@ -40,7 +40,7 @@ class NewRelic::Agent::AgentTestController < NewRelic::Agent::SuperclassControll
   end
   class TestException < RuntimeError
   end
-  
+
   def rescue_action_locally(exception)
     if exception.is_a? TestException
       raise "error in the handler"
@@ -52,22 +52,22 @@ class NewRelic::Agent::AgentTestController < NewRelic::Agent::SuperclassControll
   def entry_action
     perform_action_with_newrelic_trace('internal_action') do
       internal_action
-    end    
+    end
   end
-  
+
   def self.set_some_headers(hash_of_headers)
     @@headers_to_add ||= {}
     @@headers_to_add.merge!(hash_of_headers)
   end
-  
+
   def self.clear_headers
     @@headers_to_add = nil
   end
-  
+
   def newrelic_request_headers
     @@headers_to_add ||= {}
   end
-  
+
   private
   def internal_action
     perform_action_with_newrelic_trace(:name => 'internal_traced_action', :force => true) do

data/test/new_relic/agent/agent_test_controller_test.rb

@@ -1,13 +1,13 @@
-require File.expand_path(File.join(File.dirname(__FILE__),'..','..','test_helper')) 
+require File.expand_path(File.join(File.dirname(__FILE__),'..','..','test_helper'))
 require 'action_controller/test_case'
 class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
   require 'action_controller/base'
   require 'new_relic/agent/agent_test_controller'
-  
+
   self.controller_class = NewRelic::Agent::AgentTestController
-  
+
   attr_accessor :agent, :engine
-  
+
   # Normally you can do this with #setup but for some reason in rails 2.0.2
   # setup is not called.
   def initialize name
@@ -28,7 +28,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     if defined?(Rails) && Rails.respond_to?(:application) && Rails.application.respond_to?(:routes)
       @routes = Rails.application.routes
     end
-  
+
     Thread.current[:newrelic_ignore_controller] = nil
     NewRelic::Agent.manual_start
     @agent = NewRelic::Agent.instance
@@ -40,26 +40,26 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     end
     @engine = @agent.stats_engine
   end
-  
+
   def teardown
     Thread.current[:newrelic_ignore_controller] = nil
     NewRelic::Agent.shutdown
     NewRelic::Agent::AgentTestController.clear_headers
     super
   end
-  
+
   def test_mongrel_queue
-    NewRelic::Agent::AgentTestController.clear_headers        
+    NewRelic::Agent::AgentTestController.clear_headers
     engine.clear_stats
     NewRelic::Control.instance.local_env.stubs(:mongrel).returns( stub('mongrel', :workers => stub('workers', :list => stub('list', :length => '10'))))
-    
+
     get :index
     assert_equal 1, stats('HttpDispatcher').call_count
     assert_equal 1, engine.get_stats_no_scope('Mongrel/Queue Length').call_count
     assert_equal 9, engine.get_stats_no_scope('Mongrel/Queue Length').total_call_time
     assert_equal 0, engine.get_stats_no_scope('WebFrontend/Mongrel/Average Queue Time').call_count
   end
-  
+
   def test_heroku_queue
     engine.clear_stats
     NewRelic::Agent::AgentTestController.set_some_headers 'HTTP_X_HEROKU_QUEUE_DEPTH'=>'15'
@@ -71,7 +71,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
   end
 
   def test_new_queue_integration
-    NewRelic::Agent::AgentTestController.clear_headers    
+    NewRelic::Agent::AgentTestController.clear_headers
     engine.clear_stats
     start = ((Time.now - 1).to_f * 1_000_000).to_i
     NewRelic::Agent::AgentTestController.set_some_headers 'HTTP_X_QUEUE_START'=> "t=#{start}"
@@ -79,7 +79,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
 
     check_metric_time('WebFrontend/QueueTime', 1, 0.1)
   end
-  
+
 
   def test_new_middleware_integration
     engine.clear_stats
@@ -91,7 +91,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
   end
 
   def test_new_server_time_integration
-    NewRelic::Agent::AgentTestController.clear_headers    
+    NewRelic::Agent::AgentTestController.clear_headers
     engine.clear_stats
     start = ((Time.now - 1).to_f * 1_000_000).to_i
     NewRelic::Agent::AgentTestController.set_some_headers 'HTTP_X_REQUEST_START'=> "t=#{start}"
@@ -107,13 +107,13 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     NewRelic::Agent::AgentTestController.set_some_headers({
                                                             'HTTP_X_REQUEST_START'=> "t=#{times[0]}", 'HTTP_X_QUEUE_START' => "t=#{times[1]}", 'HTTP_X_MIDDLEWARE_START' => "t=#{times[2]}"})
     get :index
-    
-    
+
+
     check_metric_time('WebFrontend/WebServer/all', 1, 0.1)
     check_metric_time('Middleware/all', 1, 0.1)
-    check_metric_time('WebFrontend/QueueTime', 1, 0.1)    
+    check_metric_time('WebFrontend/QueueTime', 1, 0.1)
   end
-  
+
   def test_render_inline
     engine.clear_stats
     get :action_inline
@@ -126,7 +126,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     get :action_to_ignore
     compare_metrics [], engine.metrics
   end
-  
+
   def test_controller_rescued_error
     engine.clear_stats
     assert_raise RuntimeError do
@@ -149,7 +149,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     assert_equal 1, score[2], 'failing'
     assert_equal 0, score[1], 'tol'
     assert_equal 0, score[0], 'satisfied'
-   
+
   end
   def test_controller_error
     engine.clear_stats
@@ -226,7 +226,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
       end
     end
     assert_nil Thread.current[:newrelic_ignore_controller]
-    
+
   end
   def test_metric__dispatched
     engine = @agent.stats_engine
@@ -247,7 +247,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
       # you might get here if you don't have the default route installed.
     end
   end
-  
+
   def test_controller_params
     assert agent.transaction_sampler
     num_samples = NewRelic::Agent.instance.transaction_sampler.samples.length
@@ -257,7 +257,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     assert_equal num_samples + 1, samples.length
     assert_equal "[FILTERED]", samples.last.params[:request_params]["social_security_number"]
   end
-  
+
   def test_controller_params
     agent.transaction_sampler.reset!
     get :index, 'number' => "001-555-1212"
@@ -266,10 +266,10 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     assert_equal 5, s.first.params.size
   end
 
-  
+
   def test_busycalculation
     engine.clear_stats
-    
+
     assert_equal 0, NewRelic::Agent::BusyCalculator.busy_count
     get :index, 'social_security_number' => "001-555-1212", 'wait' => '1.0'
     NewRelic::Agent::BusyCalculator.harvest_busy
@@ -280,7 +280,7 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     assert stats('Instance/Busy').total_call_time > 0.5, stats('Instance/Busy').inspect
     assert_equal 0, stats('WebFrontend/Mongrel/Average Queue Time').call_count
   end
-  
+
   def test_histogram
     engine.clear_stats
     get :index, 'social_security_number' => "001-555-1212"
@@ -294,18 +294,18 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
     engine.clear_stats
     queue_length_stat = stats('Mongrel/Queue Length')
     queue_time_stat = stats('WebFrontend/QueueTime')
-    
+
     # no request start header
     get 'index'
     assert_equal 0, queue_length_stat.call_count
   end
 
   def test_queue_headers_apache
-    NewRelic::Agent::AgentTestController.clear_headers    
+    NewRelic::Agent::AgentTestController.clear_headers
     engine.clear_stats
     queue_length_stat = stats('Mongrel/Queue Length')
     queue_time_stat = stats('WebFrontend/QueueTime')
-    
+
     # apache version of header
     request_start = ((Time.now.to_f - 0.5) * 1e6).to_i.to_s
     NewRelic::Agent::AgentTestController.set_some_headers({'HTTP_X_QUEUE_START' => "t=#{request_start}"})
@@ -317,13 +317,13 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
 
   end
   def test_queue_headers_heroku
-    
+
     engine.clear_stats
-    NewRelic::Agent::AgentTestController.clear_headers    
+    NewRelic::Agent::AgentTestController.clear_headers
 
     queue_length_stat = stats('Mongrel/Queue Length')
     queue_time_stat = stats('WebFrontend/QueueTime')
-    
+
     # heroku version
     request_start = ((Time.now.to_f - 0.5) * 1e6).to_i.to_s
     NewRelic::Agent::AgentTestController.set_some_headers({'HTTP_X_QUEUE_START' => "t=#{request_start}", 'HTTP_X_HEROKU_QUEUE_DEPTH' => '0'})
@@ -335,18 +335,18 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
   end
 
   def test_queue_headers_heroku_queue_length
-    
+
     engine.clear_stats
     NewRelic::Agent::AgentTestController.clear_headers
-    
+
     queue_length_stat = stats('Mongrel/Queue Length')
-    queue_time_stat = stats('WebFrontend/QueueTime')    
+    queue_time_stat = stats('WebFrontend/QueueTime')
 
     # heroku version with queue length > 0
     request_start = ((Time.now.to_f - 0.5) * 1e6).to_i.to_s
     NewRelic::Agent::AgentTestController.set_some_headers({'HTTP_X_QUEUE_START' => "t=#{request_start}", 'HTTP_X_HEROKU_QUEUE_DEPTH' => '3'})
     get :index
-    
+
     assert_equal(1, queue_length_stat.call_count, 'queue should have been seen once')
     assert_equal(1, queue_time_stat.call_count, 'should have seen the queue header once')
     assert(queue_time_stat.total_call_time > 0.1, "Queue time should be longer than 100ms")
@@ -355,11 +355,11 @@ class NewRelic::Agent::AgentTestControllerTest < ActionController::TestCase
 
     NewRelic::Agent::AgentTestController.clear_headers
   end
-  
+
   private
   def stats(name)
     engine.get_stats_no_scope(name)
   end
-  
+
 end if defined? Rails