Disable honeycomb and newrelic in test env

Author: IgorArkhipov

Gemfile

@@ -10,6 +10,7 @@ git_source(:github) do |repo_name|
 end
 
 group :production do
+  gem "honeycomb-rails"
   gem "nakayoshi_fork"
 end
 
@@ -50,7 +51,6 @@ gem "front_matter_parser", "~> 0.2"
 gem "gemoji", "~> 3.0.0"
 gem "gibbon", "~> 2.2"
 gem "google-api-client", "~> 0.27"
-gem "honeycomb-rails"
 gem "html_truncator", "~> 0.4"
 gem "httparty", "~> 0.16"
 gem "inline_svg", "~> 1.3"
@@ -59,7 +59,6 @@ gem "jquery-rails", "~> 4.3"
 gem "kaminari", "~> 1.1"
 gem "libhoney", "~> 1.11"
 gem "liquid", "~> 4.0"
-gem 'newrelic_rpm', "~> 9.6"
 gem "nokogiri", "~> 1.10"
 gem "octokit", "~> 4.13"
 gem "omniauth", "~> 1.9"
@@ -108,6 +107,7 @@ gem "webpush", "~> 0.3"
 
 group :localproduction, :development do
   gem 'meta_request', "~> 0.7"
+  gem 'newrelic_rpm', "~> 9.6"
 end
 
 group :development do
@@ -154,14 +154,14 @@ group :test do
   gem "launchy", "~> 2.4"
   gem "pundit-matchers", "~> 1.6"
   gem "rails-controller-testing", "~> 1.0"
-  gem "ruby-prof", "~> 0.17", require: false
+  gem "ruby-prof", "~> 1.6", require: false
   gem "selenium-webdriver", "~> 3.141"
   gem "shoulda-matchers", "4.0.0.rc1", require: false
   gem "simplecov", "~> 0.16", require: false
   gem "sinatra", "~> 2.0"
   gem "stackprof", "~> 0.2", require: false, platforms: :ruby
   gem "stripe-ruby-mock", "~> 2.5", require: "stripe_mock"
-  gem "test-prof", "~> 0.7"
+  gem "test-prof", "~> 1.3"
   gem "timecop", "~> 0.9"
   gem "webmock", "~> 3.5"
   gem "zonebie", "~> 0.6.1"

Gemfile.lock

@@ -806,7 +806,7 @@ GEM
     rubocop-rspec (1.31.0)
       rubocop (>= 0.60.0)
     ruby-next-core (0.15.3)
-    ruby-prof (0.17.0)
+    ruby-prof (1.6.3)
     ruby-progressbar (1.10.0)
     ruby_dep (1.5.0)
     rubyzip (1.3.0)
@@ -886,7 +886,7 @@ GEM
       activesupport (>= 4.0)
       sprockets (>= 3.0.0)
     staccato (0.5.1)
-    stackprof (0.2.12)
+    stackprof (0.2.25)
     storext (2.2.2)
       rails (>= 4.0, < 6.0)
       virtus
@@ -897,7 +897,7 @@ GEM
       dante (>= 0.2.0)
       multi_json (~> 1.0)
       stripe (>= 2.0.3)
-    test-prof (0.7.3)
+    test-prof (1.3.0)
     thor (0.20.3)
     thread_safe (0.3.6)
     tilt (2.0.9)
@@ -1079,7 +1079,7 @@ DEPENDENCIES
   rspec-retry (~> 0.6)
   rubocop (~> 0.63)
   rubocop-rspec (~> 1.31)
-  ruby-prof (~> 0.17)
+  ruby-prof (~> 1.6)
   rubyzip (~> 1.2, >= 1.2.2)
   s3_direct_upload (~> 0.1)
   sail (~> 1.5)
@@ -1102,7 +1102,7 @@ DEPENDENCIES
   storext (~> 2.2)
   stripe (~> 4.8)
   stripe-ruby-mock (~> 2.5)
-  test-prof (~> 0.7)
+  test-prof (~> 1.3)
   timber (~> 2.6)
   timecop (~> 0.9)
   tty-command (~> 0.10)

config/environments/test.rb

@@ -53,6 +53,6 @@
   logger = Timber::Logger.new(nil)
   logger.level = config.log_level
   # config.logger = ActiveSupport::TaggedLogging.new(logger)
-  config.logger = Logger.new(nil)
+  config.logger = ActiveSupport::TaggedLogging.new(Logger.new(nil))
   config.log_level = :fatal
 end

config/initializers/honeycomb.rb

@@ -1,21 +1,23 @@
-require "libhoney"
+unless Rails.env.test?
+  require "libhoney"
 
-key = ApplicationConfig["HONEYCOMB_API_KEY"]
-dataset = "dev.to-#{Rails.env}"
+  key = ApplicationConfig["HONEYCOMB_API_KEY"]
+  dataset = "dev.to-#{Rails.env}"
 
-$libhoney = if Rails.env.development? || Rails.env.localproduction? || Rails.env.test?
-              Libhoney::NullClient.new
-            else
-              Libhoney::Client.new(
-                writekey: key,
-                dataset: dataset,
-                user_agent_addition: HoneycombRails::USER_AGENT_SUFFIX,
-              )
-            end
+  $libhoney = if Rails.env.development? || Rails.env.localproduction?
+                Libhoney::NullClient.new
+              else
+                Libhoney::Client.new(
+                  writekey: key,
+                  dataset: dataset,
+                  user_agent_addition: HoneycombRails::USER_AGENT_SUFFIX,
+                )
+              end
 
-HoneycombRails.configure do |conf|
-  conf.writekey = key
-  conf.dataset = dataset
-  conf.db_dataset = "dev.to-db-#{Rails.env}"
-  conf.client = $libhoney
+  HoneycombRails.configure do |conf|
+    conf.writekey = key
+    conf.dataset = dataset
+    conf.db_dataset = "dev.to-db-#{Rails.env}"
+    conf.client = $libhoney
+  end
 end

config/newrelic.yml

@@ -58,7 +58,6 @@ localproduction:
   app_name: 'DEV.to fork (Local Production)'
 
 test:
-  <<: *default_settings
   # It doesn't make sense to report to New Relic from automated test runs.
   monitor_mode: false
 

lib/instrumentation.rb

@@ -1,17 +1,22 @@
 module Instrumentation
   def add_param_context(*keys)
+    return if Rails.env.test?
+
     keys.each do |key|
       honeycomb_metadata[key] = params[key]
     end
   end
 
   def add_context(metadata)
+    return if Rails.env.test?
+
     metadata.each do |key, value|
       honeycomb_metadata[key] = value
     end
   end
 
   def append_to_honeycomb(request, controller_name)
+    return if Rails.env.test?
     return if honeycomb_metadata.nil?
 
     honeycomb_metadata["trace.trace_id"] = request.request_id

spec/rails_helper.rb

@@ -12,6 +12,7 @@
 require "pundit/rspec"
 require "webmock/rspec"
 require "test_prof/recipes/rspec/before_all"
+require "test_prof/recipes/rspec/sample"
 
 # Requires supporting ruby files with custom matchers and macros, etc, in
 # spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
@@ -37,6 +38,10 @@
 # Disable internet connection with Webmock
 WebMock.disable_net_connect!(allow_localhost: true)
 
+TestProf::StackProf.configure do |config|
+  config.format = 'json'
+end
+
 RSpec.configure do |config|
   config.fixture_path = "#{::Rails.root}/spec/fixtures"
 

spec/spec_helper.rb

@@ -1,4 +1,4 @@
-require "simplecov"
+# require "simplecov"
 require "zonebie/rspec"
 
 # This file was generated by the `rails generate rspec:install` command. Conventionally, all