newrelic_rpm 4.0.0.332 → 6.13.1

data/test/multiverse/suites/rails/middleware_instrumentation_test.rb

@@ -1,41 +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.
-
-# https://newrelic.atlassian.net/browse/RUBY-927
-
-require './app'
-
-if Rails::VERSION::MAJOR.to_i >= 3
-
-class MiddlewareInstrumentationTest < RailsMultiverseTest
-  def test_rails_middleware_records_metrics
-    get('/')
-    assert_metrics_recorded(
-      ['Middleware/all', 'Middleware/Rack/Rails::Rack::Logger/call']
-    )
-  end
-
-  def test_rails_routeset_is_instrumented
-    get('/')
-    assert_metrics_recorded(
-      'Middleware/Rack/ActionDispatch::Routing::RouteSet/call'
-    )
-  end
-
-  if Rails::VERSION::MAJOR >= 4
-    def test_rails_middlewares_constructed_by_name
-      get('/')
-      assert response.headers['NamedMiddleware'], "NamedMiddleware should have been called, but wasn't"
-      assert_metrics_recorded('Middleware/Rack/NamedMiddleware/call')
-    end
-
-    def test_rails_middlewares_passed_as_instances
-      get('/')
-      assert response.headers['InstanceMiddleware'], "InstanceMiddleware should have been called, but wasn't"
-      assert_metrics_recorded('Middleware/Rack/InstanceMiddleware/call')
-    end
-  end
-end
-
-end

data/test/multiverse/suites/rails/middlewares.rb

@@ -1,19 +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.
-
-# Middlewares (potentially) for use from both Rails 2 and Rails 3+ tests
-
-class ErrorMiddleware
-  def initialize(app, options={})
-    @app = app
-  end
-
-  def call(env)
-    path = ::Rack::Request.new(env).path_info
-    raise "middleware error" if path.match(/\/middleware_error\/before/)
-    result = @app.call(env)
-    raise "middleware error" if path.match(/\/middleware_error\/after/)
-    result
-  end
-end

data/test/multiverse/suites/rails/parameter_capture_test.rb

@@ -1,328 +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 './app'
-
-class ParameterCaptureController < ApplicationController
-  def transaction
-    render :text => 'hi!'
-  end
-
-  def create
-    raise 'problem' if params[:raise]
-    render :text => 'created'
-  end
-
-  def sql
-    NewRelic::Agent.agent.sql_sampler.notice_sql(
-      'SELECT * FROM table',
-      'ActiveRecord/foos/find',
-      {},
-      100.0
-    )
-  end
-
-  def error
-    raise 'wut'
-  end
-
-  # For Rails 3+, this is configured globally in the config block for the app
-  if Rails::VERSION::MAJOR == 2
-    filter_parameter_logging(:secret)
-  end
-end
-
-class ParameterCaptureTest < RailsMultiverseTest
-  include MultiverseHelpers
-  setup_and_teardown_agent
-
-  def test_no_params_captured_on_errors_when_disabled
-    with_config(:capture_params => false) do
-      get '/parameter_capture/error?other=1234&secret=4567'
-      refute_contains_request_params(agent_attributes_for_single_error_posted)
-    end
-  end
-
-  def test_no_params_captured_on_tts_when_disabled
-    with_config(:capture_params => false) do
-      get '/parameter_capture/transaction?other=1234&secret=4567'
-    end
-    assert_equal({}, last_transaction_trace_request_params)
-  end
-
-  def test_uri_on_traced_errors_never_contains_query_string_without_capture_params
-    with_config(:capture_params => false) do
-      get '/parameter_capture/error?other=1234&secret=4567'
-      assert_equal('/parameter_capture/error', attributes_for_single_error_posted("request_uri"))
-    end
-  end
-
-  def test_uri_on_traced_errors_never_contains_query_string_with_capture_params
-    with_config(:capture_params => true) do
-      get '/parameter_capture/error?other=1234&secret=4567'
-      assert_equal('/parameter_capture/error', attributes_for_single_error_posted("request_uri"))
-    end
-  end
-
-  def test_referrer_on_traced_errors_never_contains_query_string_without_capture_params
-    with_config(:capture_params => false) do
-      get '/parameter_capture/error?other=1234&secret=4567', {}, { 'HTTP_REFERER' => '/foo/bar?other=123&secret=456' }
-      attributes = agent_attributes_for_single_error_posted
-      assert_equal('/foo/bar', attributes["request.headers.referer"])
-    end
-  end
-
-  def test_referrer_on_traced_errors_never_contains_query_string_even_with_capture_params
-    with_config(:capture_params => true) do
-      get '/parameter_capture/error?other=1234&secret=4567', {}, { 'HTTP_REFERER' => '/foo/bar?other=123&secret=456' }
-      attributes = agent_attributes_for_single_error_posted
-      assert_equal('/foo/bar', attributes["request.headers.referer"])
-    end
-  end
-
-  def test_controller_and_action_excluded_from_error_parameters
-    with_config(:capture_params => true) do
-      get '/parameter_capture/error'
-      run_harvest
-
-      refute_error_has_agent_attribute('request.parameters.controller')
-      refute_error_has_agent_attribute('request.parameters.action')
-    end
-  end
-
-  def test_controller_and_action_excluded_from_transaction_trace_parameters
-    with_config(:capture_params => true, :'transaction_tracer.transaction_threshold' => -10) do
-      get '/parameter_capture/transaction'
-      run_harvest
-
-      refute_transaction_trace_has_agent_attribute('request.parameters.controller')
-      refute_transaction_trace_has_agent_attribute('request.parameters.action')
-    end
-  end
-
-  def test_uri_on_tts_never_contains_query_string
-    with_config(:capture_params => false) do
-      get '/parameter_capture/transaction?other=1234&secret=4567'
-    end
-    assert_equal('/parameter_capture/transaction', last_transaction_trace.uri)
-
-    with_config(:capture_params => true) do
-      get '/parameter_capture/transaction?other=1234&secret=4567'
-    end
-    assert_equal('/parameter_capture/transaction', last_transaction_trace.uri)
-  end
-
-  def test_filters_parameters_on_traced_errors
-    with_config(:capture_params => true) do
-      get '/parameter_capture/error?other=1234&secret=4567'
-
-      captured_params = agent_attributes_for_single_error_posted
-      assert_equal('[FILTERED]', captured_params['request.parameters.secret'])
-      assert_equal('1234',       captured_params['request.parameters.other'])
-    end
-  end
-
-  def test_filters_parameters_on_transaction_traces
-    with_config(:capture_params => true) do
-      get '/parameter_capture/transaction?other=1234&secret=4567'
-    end
-
-    captured_params = last_transaction_trace_request_params
-    assert_equal('[FILTERED]', captured_params['request.parameters.secret'])
-    assert_equal('1234',       captured_params['request.parameters.other'])
-  end
-
-  def test_no_traced_error_params_captured_when_bails_before_rails
-    with_config(:capture_params => false) do
-      get '/middleware_error/before?other=1234&secret=4567'
-      refute_contains_request_params(agent_attributes_for_single_error_posted)
-    end
-  end
-
-  def test_no_transaction_trace_params_captured_when_bails_before_rails
-    with_config(:capture_params => false) do
-      get '/middleware_error/before?other=1234&secret=4567'
-    end
-
-    assert_equal({}, last_transaction_trace_request_params)
-  end
-
-  def test_no_params_captured_on_error_when_bails_before_rails_even_when_enabled
-    with_config(:capture_params => true) do
-      get '/middleware_error/before?other=1234&secret=4567'
-      refute_contains_request_params(agent_attributes_for_single_error_posted)
-    end
-  end
-
-  def test_no_params_captured_on_tt_when_bails_before_rails_even_when_enabled
-    with_config(:capture_params => true) do
-      get '/middleware_error/before?other=1234&secret=4567'
-    end
-    assert_equal({}, last_transaction_trace_request_params)
-  end
-
-  def test_uri_on_tt_should_not_contain_query_string_with_capture_params_off
-    with_config(:capture_params => false) do
-      get '/parameter_capture/transaction?param1=value1&param2=value2'
-    end
-    assert_equal('/parameter_capture/transaction', last_transaction_trace.uri)
-  end
-
-  def test_uri_on_tt_should_not_contain_query_string_with_capture_params_on
-    with_config(:capture_params => true) do
-      get '/parameter_capture/transaction?param1=value1&param2=value2'
-    end
-    assert_equal('/parameter_capture/transaction', last_transaction_trace.uri)
-  end
-
-  def test_uri_on_traced_error_should_not_contain_query_string_with_capture_params_off
-    with_config(:capture_params => false) do
-      get '/parameter_capture/error?param1=value1&param2=value2'
-      assert_equal('/parameter_capture/error', attributes_for_single_error_posted("request_uri"))
-    end
-  end
-
-  def test_uri_on_traced_error_should_not_contain_query_string_with_capture_params_on
-    with_config(:capture_params => true) do
-      get '/parameter_capture/error?param1=value1&param2=value2'
-      assert_equal('/parameter_capture/error', attributes_for_single_error_posted("request_uri"))
-    end
-  end
-
-  def test_uri_on_sql_trace_should_not_contain_query_string_with_capture_params_off
-    with_config(:capture_params => false) do
-      get '/parameter_capture/sql?param1=value1&param2=value2'
-    end
-    assert_equal('/parameter_capture/sql', last_sql_trace.url)
-  end
-
-  def test_uri_on_sql_trace_should_not_contain_query_string_with_capture_params_on
-    with_config(:capture_params => true) do
-      get '/parameter_capture/sql?param1=value1&param2=value2'
-    end
-    assert_equal('/parameter_capture/sql', last_sql_trace.url)
-  end
-
-  def test_parameters_attached_to_transaction_events_if_enabled
-    with_config(:'attributes.include' => 'request.parameters.*',
-                :'attributes.exclude' => ['request.*', 'response.*']) do
-      get '/parameter_capture/transaction?param1=value1&param2=value2'
-    end
-
-    actual = agent_attributes_for_single_event_posted_without_ignored_attributes
-
-    expected = {"request.parameters.param1" => "value1",
-      "request.parameters.param2" => "value2"
-    }
-
-    assert_equal expected, actual
-  end
-
-  def test_request_and_response_attributes_recorded_as_agent_attributes
-      get '/parameter_capture/transaction'
-
-      expected = {
-        "response.headers.contentType" => "#{response.content_type}; charset=#{response.charset}",
-        "request.headers.contentLength" => request.content_length.to_i,
-        "request.headers.contentType" => request.content_type,
-        "request.headers.host" => request.host,
-        "request.headers.accept" => request.accept
-      }
-
-      # ActionController::IntegrationTest sets this header whereas ActionDispatch::IntegrationTest
-      # does not. Since we test using both we need to conditionally expect content_length to be set.
-
-      unless defined?(ActionDispatch::IntegrationTest)
-        expected["response.headers.contentLength"] = response.content_length
-      end
-
-      actual = agent_attributes_for_single_event_posted_without_ignored_attributes
-
-      # request method may be a symbol or string based on Rails versions
-      request_method = actual.delete("request.method")
-      assert_equal request_method, request.request_method.to_s.upcase
-
-      assert_equal(expected, actual)
-  end
-
-
-  if Rails::VERSION::MAJOR > 2
-    def test_params_tts_should_be_filtered_when_serviced_by_rack_app
-      params = {"secret" => "shhhhhhh", "name" => "name"}
-      with_config(:capture_params => true) do
-        post '/filtering_test/', params
-      end
-
-      expected = {
-        "request.parameters.secret" => "[FILTERED]",
-        "request.parameters.name" => "name"
-      }
-      assert_equal expected, last_transaction_trace_request_params
-    end
-
-    def test_params_on_errors_should_be_filtered_when_serviced_by_rack_app
-      params = {"secret" => "shhhhhhh", "name" => "name"}
-      with_config(:capture_params => true) do
-        post '/filtering_test?raise=1', params
-
-        expected = {
-          "request.parameters.secret" => "[FILTERED]",
-          "request.parameters.name" => "name",
-          "request.parameters.raise" => "1"
-        }
-
-        attributes = agent_attributes_for_single_error_posted
-        expected.each do |key, value|
-          assert_equal value, attributes[key]
-        end
-      end
-    end
-
-    def test_parameter_filtering_should_not_mutate_argument
-      input = { "foo" => "bar", "secret" => "baz" }
-      env   = { "action_dispatch.parameter_filter" => ["secret"] }
-      filtered = NewRelic::Agent::ParameterFiltering.apply_filters(env, input)
-
-      assert_equal({ "foo" => "bar", "secret" => "[FILTERED]" }, filtered)
-      assert_equal({ "foo" => "bar", "secret" => "baz" }, input)
-    end
-  end
-
-  if Rails::VERSION::MAJOR > 2 && defined?(Sinatra)
-    def test_params_tts_should_be_filtered_when_serviced_by_sinatra_app
-      with_config(:capture_params => true) do
-        get '/sinatra_app/', "secret" => "shhhhhhh", "name" => "name"
-      end
-
-      expected = {
-        "request.parameters.secret" => "[FILTERED]",
-        "request.parameters.name" => "name"
-      }
-      assert_equal expected, last_transaction_trace_request_params
-    end
-
-    def test_params_on_errors_should_be_filtered_when_serviced_by_sinatra_app
-      with_config(:capture_params => true) do
-        get '/sinatra_app?raise=1', "secret" => "shhhhhhh", "name" => "name"
-
-        attributes = agent_attributes_for_single_error_posted
-        assert_equal "[FILTERED]", attributes["request.parameters.secret"]
-        assert_equal "name", attributes["request.parameters.name"]
-        assert_equal "1", attributes["request.parameters.raise"]
-      end
-    end
-
-    def test_file_upload_params_are_replaced_with_placeholder
-      with_config(:capture_params => true, :'transaction_tracer.transaction_threshold' => -10) do
-        post '/parameter_capture', :file => Rack::Test::UploadedFile.new(__FILE__, 'text/plain')
-
-        run_harvest
-
-        result = single_transaction_trace_posted
-        assert_equal "#<ActionDispatch::Http::UploadedFile>", result.agent_attributes["request.parameters.file"]
-      end
-    end
-  end
-
-end

data/test/multiverse/suites/rails/queue_time_test.rb

@@ -1,89 +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.
-
-# https://newrelic.atlassian.net/browse/RUBY-927
-
-require './app'
-
-class QueueController < ApplicationController
-  def queued
-    respond_to do |format|
-      format.html { render :text => "<html><head></head><body>Queued</body></html>" }
-    end
-  end
-
-  def nested
-    nested_transaction
-    render :text => 'whatever'
-  end
-
-  def nested_transaction; end
-
-  add_transaction_tracer :nested_transaction
-end
-
-class QueueTimeTest < RailsMultiverseTest
-
-  REQUEST_START_HEADER = 'HTTP_X_REQUEST_START'
-
-  include MultiverseHelpers
-
-  setup_and_teardown_agent(:beacon => "beacon", :browser_key => "key", :js_agent_loader => "loader")
-
-  def test_should_track_queue_time_metric
-    t0 = freeze_time
-    t1 = advance_time(2)
-    get_path('/queue/queued', t0)
-
-    assert_metrics_recorded(
-      'WebFrontend/QueueTime' => {
-        :call_count      => 1,
-        :total_call_time => (t1 - t0)
-      }
-    )
-  end
-
-  def test_should_see_queue_time_in_rum
-    t0 = freeze_time
-    t1 = advance_time(2)
-    get_path('/queue/queued', t0)
-    queue_time = extract_queue_time_from_response
-    assert_equal((t1 - t0) * 1000, queue_time)
-  end
-
-  def test_should_not_track_queue_time_for_nested_transactions
-    t0 = freeze_time
-    t1 = advance_time(2)
-    get_path('/queue/nested', t0)
-    assert_metrics_recorded(
-      'WebFrontend/QueueTime' => {
-        :call_count      => 1,
-        :total_call_time => (t1 - t0)
-      }
-    )
-  end
-
-  def get_path(path, queue_start_time)
-    value = "t=#{(queue_start_time.to_f * 1_000_000).to_i}"
-    get(path, nil, REQUEST_START_HEADER => value)
-  end
-
-  def extract_queue_time_from_response
-    get_last_response_body =~ /\"queueTime\":(\d+.*)/
-    refute_nil $1, "Should have found queue time in #{@response.body.inspect}"
-    $1.to_i
-  end
-
-  def get_last_response_body
-    if Rails::VERSION::MAJOR >= 3
-      @response.body
-    else
-      # In Rails 2 integration tests, @response.body is always the response from
-      # the controller itself, not the middleware stack. Since we want the
-      # response from the middleware stack, we grab it off of the integration
-      # session.
-      @integration_session.body
-    end
-  end
-end