instana 1.13.0 → 1.193.3

data/test/instrumentation/{rest-client_test.rb → rest_client_test.rb}

@@ -1,6 +1,12 @@
 require 'test_helper'
+require 'support/apps/http_endpoint/boot'
 
 class RestClientTest < Minitest::Test
+  def setup
+    # See https://github.com/rest-client/rest-client/issues/612
+    OpenSSL::SSL::SSLContext::DEFAULT_PARAMS[:ciphers] = OpenSSL::SSL::SSLContext.new.ciphers
+  end
+
   def test_config_defaults
     assert ::Instana.config[:'rest-client'].is_a?(Hash)
     assert ::Instana.config[:'rest-client'].key?(:enabled)

data/test/instrumentation/sidekiq-client_test.rb

@@ -1,4 +1,5 @@
 require 'test_helper'
+require 'support/apps/sidekiq/boot'
 
 class SidekiqClientTest < Minitest::Test
   def setup

data/test/instrumentation/sidekiq-worker_test.rb

@@ -1,4 +1,5 @@
 require 'test_helper'
+require 'support/apps/sidekiq/boot'
 
 class SidekiqServerTest < Minitest::Test
   def setup

data/test/support/apps/active_record/active_record.rb

@@ -0,0 +1,21 @@
+require 'rails'
+require 'active_record/railtie'
+
+migration_class = if ActiveRecord::Migration.respond_to?(:[])
+                    ActiveRecord::Migration[4.2]
+                  else
+                    ActiveRecord::Migration
+                  end
+
+class CreateBlocks < migration_class
+  def change
+    create_table :blocks do |t|
+      t.string :name
+      t.string :color
+      t.timestamps
+    end
+  end
+end
+
+class Block < ActiveRecord::Base
+end

data/test/{servers/grpc_50051.rb → support/apps/grpc/boot.rb}

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/../apps/grpc_server.rb')
+require_relative 'grpc_server'
 
 ::Instana.logger.info "Booting instrumented gRPC server on port 50051 for tests."
 

data/test/{servers/rails_3205.rb → support/apps/rails/boot.rb}

@@ -1,22 +1,16 @@
-
 ::Instana.logger.warn "Starting background Ruby on Rails #{Rails::VERSION::STRING} application on port 3205"
 
-if Rails::VERSION::STRING >= '5.0' && ::Instana::Test.mysql?
-  ::Instana.logger.fatal "Rails 5.x doesn't support the mysql adapter (discontinued).  Set DB_FLAVOR=msyql2 instead.  This will fail as is."
-end
-
-require "rails/all"
-require "action_controller/railtie" # require more if needed
+require "rails"
+require "active_record/railtie"
+require "active_model/railtie"
+require "action_controller/railtie"
+require "active_model/railtie"
 require 'rack/handler/puma'
 
 if Rails::VERSION::STRING >= '6.0'
-  require File.expand_path(File.dirname(__FILE__) + '/../models/block6')
-  system("mkdir -p app/assets/config && echo '{}' > app/assets/config/manifest.js")
-  # Rails 6 Issue: https://github.com/rails/rails/issues/37183
-  class ApplicationController < ActionController::Base
-  end
+  require_relative 'models/block6'
 else
-  require File.expand_path(File.dirname(__FILE__) + '/../models/block')
+  require_relative 'models/block'
 end
 
 ActiveRecord::Base.establish_connection(ENV['DATABASE_URL'])
@@ -62,7 +56,7 @@ class RailsTestApp < Rails::Application
   # uncomment below to display errors
   # config.consider_all_requests_local = true
 
-  config.paths['app/views'].unshift(File.expand_path(File.dirname(__FILE__) + '/../views'))
+  config.paths['app/views'].unshift(File.join(__dir__, 'views'))
 
   config.active_support.deprecation = :stderr
 

data/test/{models → support/apps/rails/models}/block.rb

@@ -7,7 +7,7 @@ class Block < ActiveRecord::Base
   end
 end
 
-class CreateBlocks < ActiveRecord::Migration
+class CreateBlocks < ActiveRecord::Migration[4.2]
   def change
     create_table :blocks do |t|
       t.string :name

data/test/support/apps/resque/boot.rb

@@ -0,0 +1,2 @@
+require_relative 'jobs/resque_error_job'
+require_relative 'jobs/resque_fast_job'

data/test/support/apps/sidekiq/boot.rb

@@ -0,0 +1,21 @@
+# Hook into sidekiq to control the current mode
+$sidekiq_mode = :client
+class << Sidekiq
+  def server?
+    $sidekiq_mode == :server
+  end
+end
+
+# Configure redis for sidekiq client
+Sidekiq.configure_client do |config|
+  config.redis = { url: ENV['REDIS_URL'] }
+end
+
+# Configure redis for sidekiq worker
+$sidekiq_mode = :server
+::Sidekiq.configure_server do |config|
+  config.redis = { url: ENV['REDIS_URL'] }
+end
+$sidekiq_mode = :client
+
+require_relative 'worker'

data/test/{servers → support/apps}/sidekiq/worker.rb

@@ -3,8 +3,8 @@ require 'sidekiq/cli'
 require 'sidekiq/api'
 require 'sidekiq/processor'
 
-require Dir.pwd + '/test/jobs/sidekiq_job_1.rb'
-require Dir.pwd + '/test/jobs/sidekiq_job_2.rb'
+require_relative 'jobs/sidekiq_job_1'
+require_relative 'jobs/sidekiq_job_2'
 
 ::Instana.logger.info "Booting instrumented sidekiq worker for tests."
 ::Sidekiq.logger.level = ::Logger::FATAL

data/test/support/helpers.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+module Instana
+  module TestHelpers
+    # Used to reset the gem to boot state.  It clears out any queued and/or staged
+    # traces and resets the tracer to no active trace.
+    #
+    def clear_all!
+      ::Instana.processor.clear!
+      ::Instana.tracer.clear!
+      nil
+    end
+
+    def disable_redis_instrumentation
+      ::Instana.config[:redis][:enabled] = false
+    end
+
+    def enable_redis_instrumentation
+      ::Instana.config[:redis][:enabled] = true
+    end
+
+    def validate_sdk_span(json_span, sdk_hash = {}, errored = false, _ec = 1)
+      assert_equal :sdk, json_span[:n]
+      assert json_span.key?(:k)
+      assert json_span.key?(:d)
+      assert json_span.key?(:ts)
+
+      sdk_hash.each do |k, v|
+        assert_equal v, json_span[:data][:sdk][k]
+      end
+
+      return unless errored
+
+      assert_equal true, json_span[:error]
+      assert_equal 1, json_span[:ec]
+    end
+
+    def find_spans_by_name(spans, name)
+      spans.select do |span|
+        (span[:n] == :sdk && span[:data][:sdk][:name] == name) || span[:n] == name
+      end.tap do |result|
+        raise StandardError, "No SDK spans (#{name}) could be found" if result.empty?
+      end
+    end
+
+    def find_first_span_by_name(spans, name)
+      spans.each do |span|
+        case span[:n]
+        when :sdk
+          return span if span[:data][:sdk][:name] == name
+        when name
+          return span
+        end
+      end
+      raise StandardError, "Span (#{name}) not found"
+    end
+
+    def find_span_by_id(spans, id)
+      spans.each do |span|
+        return span if span[:s] == id
+      end
+      raise StandardError, "Span with id (#{id}) not found"
+    end
+
+    # Finds the first span in +spans+ for which +block+ returns true
+    #
+    #     ar_span = find_first_span_by_qualifier(ar_spans) do |span|
+    #       span[:data][:activerecord][:sql] == sql
+    #     end
+    #
+    # This helper will raise an exception if no span evaluates to true against he provided block.
+    #
+    # +spans+: +Array+ of spans to search
+    # +block+: The Ruby block to evaluate against each span
+    def find_first_span_by_qualifier(spans, &block)
+      spans.each do |span|
+        return span if block.call(span)
+      end
+      raise StandardError, 'Span with qualifier not found'
+    end
+
+    def has_postgres_database?
+      URI(ENV.fetch('DATABASE_URL', '')).scheme == 'postgres'
+    end
+
+    def has_mysql2_database?
+      URI(ENV.fetch('DATABASE_URL', '')).scheme == 'mysql2'
+    end
+
+    def has_mysql_database?
+      URI(ENV.fetch('DATABASE_URL', '')).scheme == 'mysql'
+    end
+  end
+end

data/test/test_helper.rb

@@ -1,159 +1,39 @@
-$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
 ENV['INSTANA_TEST'] = 'true'
-require "rubygems"
-require "bundler/setup"
-Bundler.require(:default, :test)
 
-require "minitest/spec"
-require "minitest/autorun"
-require "minitest/reporters"
-require "minitest/debugger" if ENV['DEBUG']
-require "minitest/benchmark"
-require 'webmock/minitest'
-
-require "instana/test"
-::Instana::Test.setup_environment
+begin
+  require 'simplecov'
+  require 'simplecov_json_formatter'
 
-# Webmock: Whitelist local IPs
-whitelist = ['127.0.0.1', 'localhost', '172.17.0.1', '172.0.12.100']
-allowed_sites = lambda{|uri|
-  whitelist.include?(uri.host)
-}
-::WebMock.disable_net_connect!(allow: allowed_sites)
+  SimpleCov.start do
+    enable_coverage :branch
 
-# Boot background webservers to test against.
-require "./test/servers/rackapp_6511"
+    add_group 'Frameworks', 'lib/instana/frameworks'
+    add_group 'Instrumentation', 'lib/instana/instrumentation'
 
-case File.basename(ENV['BUNDLE_GEMFILE'])
-when /rails/
-  require './test/servers/rails_3205'
-when /libraries/
-  # Configure gRPC
-  require './test/servers/grpc_50051.rb'
+    add_filter %r{^/test/}
 
-  # Hook into sidekiq to control the current mode
-  $sidekiq_mode = :client
-  class << Sidekiq
-    def server?
-      $sidekiq_mode == :server
+    if ENV['CI']
+      formatter SimpleCov::Formatter::JSONFormatter
     end
   end
-
-  ENV['REDIS_URL'] ||= 'redis://127.0.0.1:6379'
-
-  # Configure redis for sidekiq client
-  Sidekiq.configure_client do |config|
-    config.redis = { url: ENV['REDIS_URL'] }
-  end
-
-  # Configure redis for sidekiq worker
-  $sidekiq_mode = :server
-  ::Sidekiq.configure_server do |config|
-    config.redis = { url: ENV['REDIS_URL'] }
-  end
-  $sidekiq_mode = :client
-
-  require './test/servers/sidekiq/worker'
-end
-
-Minitest::Reporters.use! MiniTest::Reporters::SpecReporter.new
-
-# Used to reset the gem to boot state.  It clears out any queued and/or staged
-# traces and resets the tracer to no active trace.
-#
-def clear_all!
-  ::Instana.processor.clear!
-  ::Instana.tracer.clear!
+rescue LoadError => _e
   nil
 end
 
-def disable_redis_instrumentation
-  ::Redis::Client.class_eval do
-    alias call call_without_instana
-    alias call_pipeline call_pipeline_without_instana
-  end
-end
+require 'bundler/setup'
+Bundler.require
 
-def enable_redis_instrumentation
-  ::Redis::Client.class_eval do
-    alias call call_with_instana
-    alias call_pipeline call_pipeline_with_instana
-  end
-end
-
-def validate_sdk_span(json_span, sdk_hash = {}, errored = false, ec = 1)
-  assert_equal :sdk, json_span[:n]
-  assert json_span.key?(:k)
-  assert json_span.key?(:d)
-  assert json_span.key?(:ts)
-
-  for k,v in sdk_hash
-    assert_equal v, json_span[:data][:sdk][k]
-  end
-
-  if errored
-    assert_equal true, json_span[:error]
-    assert_equal 1, json_span[:ec]
-  end
-end
-
-def find_spans_by_name(spans, name)
-  result = []
-  for span in spans
-    if span[:n] == :sdk
-      if span[:data][:sdk][:name] == name
-        result << span
-      end
-    elsif span[:n] == name
-      result << span
-    end
-  end
-  if result.empty?
-    raise Exception.new("No SDK spans (#{name}) could be found")
-  else
-    return result
-  end
-end
+require "minitest/spec"
+require "minitest/autorun"
+require "minitest/reporters"
 
-def find_first_span_by_name(spans, name)
-  for span in spans
-    if span[:n] == :sdk
-      if span[:data][:sdk][:name] == name
-        return span
-      end
-    else
-      if span[:n] == name
-        return span
-      end
-    end
-  end
-  raise Exception.new("Span (#{name}) not found")
-end
+require 'webmock/minitest'
+# Webmock: Whitelist local IPs
+WebMock.disable_net_connect!(
+  allow: ->(uri) { %w[localhost 127.0.0.1 172.17.0.1 172.0.12.100].include?(uri.host) }
+)
 
-def find_span_by_id(spans, id)
-  for span in spans
-    if span[:s] == id
-      return span
-    end
-  end
-  raise Exception.new("Span with id (#{id}) not found")
-end
+Dir['test/support/*.rb'].each { |f| load(f) }
 
-# Finds the first span in +spans+ for which +block+ returns true
-#
-#     ar_span = find_first_span_by_qualifier(ar_spans) do |span|
-#       span[:data][:activerecord][:sql] == sql
-#     end
-#
-# This helper will raise an exception if no span evaluates to true against he provided block.
-#
-# +spans+: +Array+ of spans to search
-# +block+: The Ruby block to evaluate against each span
-def find_first_span_by_qualifier(spans, &block)
-  spans.each do |span|
-    if block.call(span)
-      return span
-    end
-  end
-  raise Exception.new("Span with qualifier not found")
-end
+Minitest::Reporters.use! MiniTest::Reporters::SpecReporter.new
+Minitest::Test.include(Instana::TestHelpers)

data/test/tracing/id_management_test.rb

@@ -49,82 +49,20 @@ class TracerIDMgmtTest < Minitest::Test
     converted_id = Instana::Util.header_to_id(header_id)
 
     # Assert that it is an Integer
-    assert converted_id.is_a?(Integer)
+    assert converted_id.is_a?(String)
   end
 
   def test_header_to_id_conversion_with_bogus_header
     # Bogus nil arg
     bogus_result = Instana::Util.header_to_id(nil)
-    assert_equal 0, bogus_result
+    assert_equal '', bogus_result
 
     # Bogus Integer arg
     bogus_result = Instana::Util.header_to_id(1234)
-    assert_equal 0, bogus_result
+    assert_equal '', bogus_result
 
     # Bogus Array arg
     bogus_result = Instana::Util.header_to_id([1234])
-    assert_equal 0, bogus_result
-  end
-
-  def test_id_conversion_back_and_forth
-    # id --> header --> id
-    original_id = ::Instana::Util.generate_id
-    header_id = Instana::Util.id_to_header(original_id)
-    converted_back_id = Instana::Util.header_to_id(header_id)
-    assert original_id == converted_back_id
-
-    # header --> id --> header
-    original_header_id = "c025ee93b1aeda7b"
-    id = Instana::Util.header_to_id(original_header_id)
-    converted_back_header_id = Instana::Util.id_to_header(id)
-    assert_equal original_header_id, converted_back_header_id
-
-    # Test a random value
-    id = -7815363404733516491
-    header = "938a406416457535"
-
-    result = Instana::Util.header_to_id(header)
-    assert_equal id, result
-
-    result = Instana::Util.id_to_header(id)
-    assert_equal header, result
-
-    10000.times do
-      original_id = ::Instana::Util.generate_id
-      header_id = Instana::Util.id_to_header(original_id)
-      converted_back_id = Instana::Util.header_to_id(header_id)
-      assert original_id == converted_back_id
-    end
-  end
-
-  def test_id_max_value_and_conversion
-    max_id = 9223372036854775807
-    min_id = -9223372036854775808
-    max_hex = "7fffffffffffffff"
-    min_hex = "8000000000000000"
-
-    assert_equal max_hex, Instana::Util.id_to_header(max_id)
-    assert_equal min_hex, Instana::Util.id_to_header(min_id)
-
-    assert_equal max_id, Instana::Util.header_to_id(max_hex)
-    assert_equal min_id, Instana::Util.header_to_id(min_hex)
-  end
-
-  def test_that_leading_zeros_handled_correctly
-
-    header = ::Instana::Util.id_to_header(16)
-    assert_equal "10", header
-
-    id = ::Instana::Util.header_to_id("10")
-    assert_equal 16, id
-
-    id = ::Instana::Util.header_to_id("0000000000000010")
-    assert_equal 16, id
-
-    id = ::Instana::Util.header_to_id("88b6c735206ca42")
-    assert_equal 615705016619420226, id
-
-    id = ::Instana::Util.header_to_id("088b6c735206ca42")
-    assert_equal 615705016619420226, id
+    assert_equal '', bogus_result
   end
 end