sundawg_newrelic_rpm 3.5.8.1

data/test/new_relic/agent/error_collector/notice_error_test.rb

@@ -0,0 +1,268 @@
+require File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','test_helper'))
+class NewRelic::Agent::ErrorCollector::NoticeErrorTest < Test::Unit::TestCase
+  require 'new_relic/agent/error_collector'
+  include NewRelic::Agent::ErrorCollector::NoticeError
+
+  def test_error_params_from_options_mocked
+    options = {:initial => 'options'}
+    self.expects(:uri_ref_and_root).returns({:hi => 'there', :hello => 'bad'})
+    self.expects(:normalized_request_and_custom_params).with({:initial => 'options'}).returns({:hello => 'world'})
+    assert_equal({:hi => 'there', :hello => 'world'}, error_params_from_options(options))
+  end
+
+  module Winner
+    def winner
+      'yay'
+    end
+  end
+
+  def test_sense_method
+    object = Object.new
+    object.extend(Winner)
+    assert !sense_method(object, 'blab')
+    assert_equal 'yay', sense_method(object, 'winner')
+  end
+
+  def test_fetch_from_options
+    options = {:hello => 'world'}
+    assert_equal 'world', fetch_from_options(options, :hello, '')
+    assert_equal '', fetch_from_options(options, :none, '')
+    assert_equal({}, options)
+  end
+
+  def test_uri_ref_and_root_default
+    fake_control = mocked_control
+    fake_control.expects(:root).returns('rootbeer')
+    options = {}
+    assert_equal({:request_referer => '', :rails_root => 'rootbeer', :request_uri => ''}, uri_ref_and_root(options))
+  end
+
+  def test_uri_ref_and_root_values
+    fake_control = mocked_control
+    fake_control.expects(:root).returns('rootbeer')
+    options = {:uri => 'whee', :referer => 'bang'}
+    assert_equal({:request_referer => 'bang', :rails_root => 'rootbeer', :request_uri => 'whee'}, uri_ref_and_root(options))
+  end
+
+  def test_custom_params_from_opts_base
+    assert_equal({}, custom_params_from_opts({}))
+  end
+
+  def test_custom_params_from_opts_custom_params
+    assert_equal({:foo => 'bar'}, custom_params_from_opts({:custom_params => {:foo => 'bar'}}))
+  end
+
+  def test_custom_params_from_opts_merged_params
+    assert_equal({:foo => 'baz'}, custom_params_from_opts({:custom_params => {:foo => 'bar'}, :foo => 'baz'}))
+  end
+
+  def test_request_params_from_opts_positive
+    with_config(:capture_params => true) do
+      val = {:request_params => 'foo'}
+      assert_equal('foo', request_params_from_opts(val))
+      assert_equal({}, val, "should delete request_params key from hash")
+    end
+  end
+
+  def test_request_params_from_opts_negative
+    with_config(:capture_params => false) do
+      val = {:request_params => 'foo'}
+      assert_equal(nil, request_params_from_opts(val))
+      assert_equal({}, val, "should delete request_params key from hash")
+    end
+  end
+
+  def test_normalized_request_and_custom_params_base
+    self.expects(:normalize_params).with(nil).returns(nil)
+    self.expects(:normalize_params).with({}).returns({})
+    with_config(:capture_params => true) do
+      assert_equal({:request_params => nil, :custom_params => {}},
+                   normalized_request_and_custom_params({}))
+    end
+  end
+
+  def test_extract_source_base
+    with_config(:'error_collector.capture_source' => true) do
+      error_collector = NewRelic::Agent::ErrorCollector.new
+      error_collector.expects(:sense_method).with(nil, 'source_extract')
+      assert_equal(nil, error_collector.extract_source(nil))
+    end
+  end
+
+  def test_extract_source_disabled
+    with_config(:'error_collector.capture_source' => false) do
+      error_collector = NewRelic::Agent::ErrorCollector.new
+      assert_equal(nil, error_collector.extract_source(mock('exception')))
+    end
+  end
+
+  def test_extract_source_with_source
+    with_config(:'error_collector.capture_source' => true) do
+      error_collector = NewRelic::Agent::ErrorCollector.new
+      error_collector.expects(:sense_method).with('happy', 'source_extract').returns('THE SOURCE')
+      assert_equal('THE SOURCE', error_collector.extract_source('happy'))
+    end
+  end
+
+  def test_extract_stack_trace
+    exception = mock('exception')
+    self.expects(:sense_method).with(exception, 'original_exception')
+    self.expects(:sense_method).with(exception, 'backtrace')
+    assert_equal('<no stack trace>', extract_stack_trace(exception))
+  end
+
+  def test_extract_stack_trace_positive
+    orig = mock('original')
+    exception = mock('exception')
+    self.expects(:sense_method).with(exception, 'original_exception').returns(orig)
+    self.expects(:sense_method).with(orig, 'backtrace').returns('STACK STACK STACK')
+    assert_equal('STACK STACK STACK', extract_stack_trace(exception))
+  end
+
+  def test_over_queue_limit_negative
+    @errors = []
+    assert !over_queue_limit?(nil)
+  end
+
+  def test_over_queue_limit_positive
+    @errors = %w(1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21)
+    expects_logging(:warn, includes('The error reporting queue has reached 20'))
+    assert over_queue_limit?('hooray')
+  end
+
+  def test_exception_info
+    exception = mock('exception')
+    self.expects(:sense_method).with(exception, 'file_name').returns('file_name')
+    self.expects(:sense_method).with(exception, 'line_number').returns('line_number')
+    self.expects(:extract_source).with(exception).returns('source')
+    self.expects(:extract_stack_trace).with(exception).returns('stack_trace')
+    assert_equal({:file_name => 'file_name', :line_number => 'line_number', :source => 'source', :stack_trace => 'stack_trace'},
+                 exception_info(exception))
+  end
+
+  def test_add_to_error_queue_positive
+    noticed_error = mock('noticed_error')
+    noticed_error.expects(:message).returns('a message')
+    @lock = Mutex.new
+    @errors = []
+    self.expects(:over_queue_limit?).with('a message').returns(false)
+    add_to_error_queue(noticed_error)
+    assert_equal([noticed_error], @errors)
+  end
+
+  def test_add_to_error_queue_negative
+    noticed_error = mock('noticed_error')
+    noticed_error.expects(:message).returns('a message')
+    @lock = Mutex.new
+    @errors = []
+    self.expects(:over_queue_limit?).with('a message').returns(true)
+    add_to_error_queue(noticed_error)
+    assert_equal([], @errors)
+  end
+
+  def test_should_exit_notice_error_disabled
+    error = mocked_error
+    with_error_collector_config(:'error_collector.enabled' => false) do |error_collector|
+      assert error_collector.should_exit_notice_error?(error)
+    end
+  end
+
+  def test_should_exit_notice_error_nil
+    error = nil
+    with_error_collector_config(:'error_collector.enabled' => true) do |error_collector|
+      error_collector.expects(:error_is_ignored?).with(error).returns(false)
+      # we increment it for the case that someone calls
+      # NewRelic::Agent.notice_error(foo) # foo is nil
+      # (which is probably not a good idea but is the existing api)
+      error_collector.expects(:increment_error_count!)
+      assert error_collector.should_exit_notice_error?(error)
+    end
+  end
+
+  def test_should_exit_notice_error_positive
+    error = mocked_error
+    with_error_collector_config(:'error_collector.enabled' => true) do |error_collector|
+      error_collector.expects(:error_is_ignored?).with(error).returns(true)
+      assert error_collector.should_exit_notice_error?(error)
+    end
+  end
+
+  def test_should_exit_notice_error_negative
+    error = mocked_error
+    with_error_collector_config(:'error_collector.enabled' => true) do |error_collector|
+      error_collector.expects(:error_is_ignored?).with(error).returns(false)
+      error_collector.expects(:increment_error_count!)
+      assert !error_collector.should_exit_notice_error?(error)
+    end
+  end
+
+  def test_filtered_error_positive
+    with_error_collector_config(:'error_collector.ignore_errors' => 'an_error') do |error_collector|
+      error = mocked_error
+      error_class = mock('error class')
+      error.expects(:class).returns(error_class)
+      error_class.expects(:name).returns('an_error')
+      assert error_collector.filtered_error?(error)
+    end
+  end
+
+  def test_filtered_error_negative
+    error = mocked_error
+    error_class = mock('error class')
+    error.expects(:class).returns(error_class)
+    error_class.expects(:name).returns('an_error')
+    assert !NewRelic::Agent::ErrorCollector.new.filtered_error?(error)
+  end
+
+  def test_filtered_by_error_filter_empty
+    # should return right away when there's no filter
+    @ignore_filter = nil
+    assert !filtered_by_error_filter?(nil)
+  end
+
+  def test_filtered_by_error_filter_positive
+    error = mocked_error
+    @ignore_filter = lambda { |x| assert_equal error, x; false  }
+    assert filtered_by_error_filter?(error)
+  end
+
+  def test_filtered_by_error_filter_negative
+    error = mocked_error
+    @ignore_filter = lambda { |x| assert_equal error, x; true  }
+    assert !filtered_by_error_filter?(error)
+  end
+
+  def test_error_is_ignored_positive
+    error = mocked_error
+    self.expects(:filtered_error?).with(error).returns(true)
+    assert error_is_ignored?(error)
+  end
+
+  def test_error_is_ignored_negative
+    error = mocked_error
+    self.expects(:filtered_error?).with(error).returns(false)
+    assert !error_is_ignored?(error)
+  end
+
+  def test_error_is_ignored_no_error
+    assert !error_is_ignored?(nil), 'should not ignore nil'
+  end
+
+  private
+
+  def mocked_error
+    mock('error')
+  end
+
+  def mocked_control
+    fake_control = mock('control')
+    NewRelic::Control.stubs(:instance).returns(fake_control)
+    fake_control
+  end
+
+  def with_error_collector_config(config)
+    with_config(config) do
+      yield NewRelic::Agent::ErrorCollector.new
+    end
+  end
+end

data/test/new_relic/agent/error_collector_test.rb

@@ -0,0 +1,246 @@
+# Run faster standalone
+ENV['SKIP_RAILS'] = 'true'
+require File.expand_path(File.join(File.dirname(__FILE__),'..','..','test_helper'))
+
+class NewRelic::Agent::ErrorCollectorTest < Test::Unit::TestCase
+  def setup
+    super
+    @test_config = { :capture_params => true }
+    NewRelic::Agent.config.apply_config(@test_config)
+    @error_collector = NewRelic::Agent::ErrorCollector.new
+    @error_collector.stubs(:enabled).returns(true)
+  end
+
+  def teardown
+    super
+    NewRelic::Agent.config.remove_config(@test_config)
+  end
+
+  def test_empty
+    @error_collector.harvest_errors([])
+    @error_collector.notice_error(nil, :metric=> 'path', :request_params => {:x => 'y'})
+    errors = @error_collector.harvest_errors([])
+
+    assert_equal 0, errors.length
+
+    @error_collector.notice_error('Some error message', :metric=> 'path', :request_params => {:x => 'y'})
+    errors = @error_collector.harvest_errors([])
+
+    err = errors.first
+    assert_equal 'Some error message', err.message
+    assert_equal 'y', err.params[:request_params][:x]
+    assert_equal '', err.params[:request_uri]
+    assert_equal '', err.params[:request_referer]
+    assert_equal 'path', err.path
+    assert_equal 'Error', err.exception_class
+  end
+
+  def test_simple
+    @error_collector.notice_error(StandardError.new("message"), :uri => '/myurl/', :metric => 'path', :referer => 'test_referer', :request_params => {:x => 'y'})
+
+    old_errors = []
+    errors = @error_collector.harvest_errors(old_errors)
+
+    assert_equal errors.length, 1
+
+    err = errors.first
+    assert_equal 'message', err.message
+    assert_equal 'y', err.params[:request_params][:x]
+    assert err.params[:request_uri] == '/myurl/'
+    assert err.params[:request_referer] == "test_referer"
+    assert err.path == 'path'
+    assert err.exception_class == 'StandardError'
+
+    # the collector should now return an empty array since nothing
+    # has been added since its last harvest
+    errors = @error_collector.harvest_errors(nil)
+    assert errors.length == 0
+  end
+
+  def test_long_message
+    #yes, times 500. it's a 5000 byte string. Assuming strings are
+    #still 1 byte / char.
+    @error_collector.notice_error(StandardError.new("1234567890" * 500), :uri => '/myurl/', :metric => 'path', :request_params => {:x => 'y'})
+
+    old_errors = []
+    errors = @error_collector.harvest_errors(old_errors)
+
+    assert_equal errors.length, 1
+
+    err = errors.first
+    assert_equal 4096, err.message.length
+    assert_equal ('1234567890' * 500)[0..4095], err.message
+  end
+
+  def test_collect_failover
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'first', :request_params => {:x => 'y'})
+
+    errors = @error_collector.harvest_errors([])
+
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'second', :request_params => {:x => 'y'})
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'path', :request_params => {:x => 'y'})
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'last', :request_params => {:x => 'y'})
+
+    errors = @error_collector.harvest_errors(errors)
+
+    assert_equal 4, errors.length
+    assert_equal 'first', errors.first.path
+    assert_equal 'last', errors.last.path
+
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'first', :request_params => {:x => 'y'})
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'last', :request_params => {:x => 'y'})
+
+    errors = @error_collector.harvest_errors(nil)
+    assert_equal 2, errors.length
+    assert_equal 'first', errors.first.path
+    assert_equal 'last', errors.last.path
+  end
+
+  def test_queue_overflow
+
+    max_q_length = 20     # for some reason I can't read the constant in ErrorCollector
+
+    silence_stream(::STDOUT) do
+     (max_q_length + 5).times do |n|
+        @error_collector.notice_error(StandardError.new("exception #{n}"), :metric => "path", :request_params => {:x => n})
+      end
+    end
+
+    errors = @error_collector.harvest_errors([])
+    assert errors.length == max_q_length
+    errors.each_index do |i|
+      err = errors.shift
+      assert_equal i.to_s, err.params[:request_params][:x], err.params.inspect
+    end
+  end
+
+  # Why would anyone undef these methods?
+  class TestClass
+    undef to_s
+    undef inspect
+  end
+
+
+  def test_supported_param_types
+    types = [[1, '1'],
+    [1.1, '1.1'],
+    ['hi', 'hi'],
+    [:hi, :hi],
+    [StandardError.new("test"), "#<StandardError>"],
+    [TestClass.new, "#<NewRelic::Agent::ErrorCollectorTest::TestClass>"]
+    ]
+
+    types.each do |test|
+      @error_collector.notice_error(StandardError.new("message"), :metric => 'path',
+                                    :request_params => {:x => test[0]})
+      assert_equal test[1], @error_collector.harvest_errors([])[0].params[:request_params][:x]
+    end
+  end
+
+
+  def test_exclude
+    @error_collector.ignore(["IOError"])
+
+    @error_collector.notice_error(IOError.new("message"), :metric => 'path', :request_params => {:x => 'y'})
+
+    errors = @error_collector.harvest_errors([])
+
+    assert_equal 0, errors.length
+  end
+
+  def test_exclude_later_config_changes
+    @error_collector.notice_error(IOError.new("message"))
+
+    NewRelic::Agent.config.apply_config(:'error_collector.ignore_errors' => "IOError")
+    @error_collector.notice_error(IOError.new("message"))
+
+    errors = @error_collector.harvest_errors([])
+
+    assert_equal 1, errors.length
+
+  end
+
+  def test_exclude_block
+    @error_collector.ignore_error_filter &wrapped_filter_proc
+
+    @error_collector.notice_error(IOError.new("message"), :metric => 'path', :request_params => {:x => 'y'})
+    @error_collector.notice_error(StandardError.new("message"), :metric => 'path', :request_params => {:x => 'y'})
+
+    errors = @error_collector.harvest_errors([])
+
+    assert_equal 1, errors.length
+  end
+
+  def test_obfuscates_error_messages_when_high_security_is_set
+    with_config(:high_security => true) do
+      @error_collector.notice_error(StandardError.new("YO SQL BAD: serect * flom test where foo = 'bar'"))
+      @error_collector.notice_error(StandardError.new("YO SQL BAD: serect * flom test where foo in (1,2,3,4,5)"))
+
+      old_errors = []
+      errors = @error_collector.harvest_errors([])
+
+      assert_equal('YO SQL BAD: serect * flom test where foo = ?',
+                   errors[0].message)
+      assert_equal('YO SQL BAD: serect * flom test where foo in (?,?,?,?,?)',
+                   errors[1].message)
+    end
+  end
+
+  def test_increments_count_on_errors
+    expects_error_count_increase(1) do
+      @error_collector.notice_error(StandardError.new("Boo"))
+    end
+  end
+
+  def test_increment_error_count_record_summary_and_txn_metric
+    txn_info = NewRelic::Agent::TransactionInfo.new
+    txn_info.transaction_name = 'Controller/class/method'
+    NewRelic::Agent::TransactionInfo.stubs(:get).returns(txn_info)
+
+    stats_engine = NewRelic::Agent.instance.stats_engine
+    stats_engine.reset_stats
+    @error_collector.increment_error_count!(StandardError.new('Boo'))
+    assert_equal(1, stats_engine.get_stats('Errors/all').call_count,
+                 'Missing Errors/all metric')
+    assert_equal(1, stats_engine.get_stats('Errors/Controller/class/method').call_count,
+                 'Missing Errors/Controller/class/method metric')
+  end
+
+  def test_doesnt_increment_error_count_on_transaction_if_nameless
+    txn_info = NewRelic::Agent::TransactionInfo.new
+    NewRelic::Agent::TransactionInfo.stubs(:get).returns(txn_info)
+
+    stats_engine = NewRelic::Agent.instance.stats_engine
+    stats_engine.reset_stats
+    @error_collector.increment_error_count!(StandardError.new('Boo'))
+    assert_equal(0, stats_engine.get_stats('Errors/(unknown)').call_count)
+  end
+
+  private
+
+  def expects_error_count_increase(increase)
+    count = get_error_stats
+    yield
+    assert_equal increase, get_error_stats - count
+  end
+
+  def get_error_stats
+    NewRelic::Agent.get_stats("Errors/all").call_count
+  end
+
+  def wrapped_filter_proc
+    Proc.new do |e|
+      if e.is_a? IOError
+        return nil
+      else
+        return e
+      end
+    end
+  end
+
+  def silence_stream(*args)
+    super
+  rescue NoMethodError
+    yield
+  end
+end