newrelic_rpm 3.14.0.305 → 3.14.1.311

data/test/new_relic/agent/pipe_service_test.rb

@@ -63,6 +63,13 @@ class PipeServiceTest < Minitest::Test
       assert_equal ['err'], received_data[:error_data]
     end
 
+    def test_error_event_data
+      received_data = data_from_forked_process do
+        @service.error_event_data(['err_ev'])
+      end
+      assert_equal ['err_ev'], received_data[:error_event_data]
+    end
+
     def test_sql_trace_data
       received_data = data_from_forked_process do
         @service.sql_trace_data(['sql'])

data/test/new_relic/marshalling_test_cases.rb

@@ -107,6 +107,41 @@ module MarshallingTestCases
     assert_equal(expected_event, events.first)
   end
 
+  def test_sends_error_events
+    t0 = freeze_time(Time.at(Time.now.to_i))
+
+    with_around_hook do
+      Transactioner.new.break_it
+    end
+
+    transmit_data
+
+    result = $collector.calls_for('error_event_data')
+
+    assert_equal 1, result.length
+    events = result.first.error_events
+    assert_equal 1, events.length
+
+    expected_event = [
+      {
+        "type" => "TransactionError",
+        "error.class" => "StandardError",
+        "error.message" => "Sorry!",
+        "timestamp" => t0.to_f,
+        "transactionName" => "TestTransaction/break_it",
+        "duration" => 0.0
+      },
+      {},
+      {}
+    ]
+
+    event = events.first
+    # this is only present on REE, and we don't really care - the point of this
+    # test is just to validate basic marshalling
+    event[0].delete("gcCumulative")
+    assert_equal(expected_event, event)
+  end
+
   class Transactioner
     include NewRelic::Agent::Instrumentation::ControllerInstrumentation
 
@@ -115,6 +150,13 @@ module MarshallingTestCases
     end
 
     add_transaction_tracer :do_it
+
+    def break_it
+      NewRelic::Agent.set_transaction_name("break_it", :category => "TestTransaction")
+      NewRelic::Agent.notice_error StandardError.new("Sorry!")
+    end
+
+    add_transaction_tracer :break_it
   end
 
   def with_around_hook(&blk)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: newrelic_rpm
 version: !ruby/object:Gem::Version
-  version: 3.14.0.305
+  version: 3.14.1.311
 platform: ruby
 authors:
 - Tim Krajcar
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-14 00:00:00.000000000 Z
+date: 2015-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -813,6 +813,7 @@ files:
 - test/multiverse/suites/delayed_job/Envfile
 - test/multiverse/suites/delayed_job/before_suite.rb
 - test/multiverse/suites/delayed_job/config/newrelic.yml
+- test/multiverse/suites/delayed_job/delayed_job_instrumentation_test.rb
 - test/multiverse/suites/delayed_job/delayed_job_sampler_test.rb
 - test/multiverse/suites/delayed_job/unsupported_backend_test.rb
 - test/multiverse/suites/excon/Envfile
@@ -863,10 +864,12 @@ files:
 - test/multiverse/suites/padrino/config/newrelic.yml
 - test/multiverse/suites/padrino/padrino_test.rb
 - test/multiverse/suites/rack/Envfile
+- test/multiverse/suites/rack/before_suite.rb
 - test/multiverse/suites/rack/builder_map_test.rb
 - test/multiverse/suites/rack/example_app.rb
 - test/multiverse/suites/rack/http_response_code_test.rb
 - test/multiverse/suites/rack/nested_non_rack_app_test.rb
+- test/multiverse/suites/rack/puma_rack_builder_test.rb
 - test/multiverse/suites/rack/rack_auto_instrumentation_test.rb
 - test/multiverse/suites/rack/rack_cascade_test.rb
 - test/multiverse/suites/rack/rack_env_mutation_test.rb
@@ -1028,6 +1031,7 @@ files:
 - test/new_relic/agent/instrumentation/active_record_helper_test.rb
 - test/new_relic/agent/instrumentation/active_record_subscriber_test.rb
 - test/new_relic/agent/instrumentation/controller_instrumentation_test.rb
+- test/new_relic/agent/instrumentation/delayed_job_instrumentation_test.rb
 - test/new_relic/agent/instrumentation/instrumentation_test.rb
 - test/new_relic/agent/instrumentation/metric_frame_test.rb
 - test/new_relic/agent/instrumentation/middleware_proxy_test.rb