Merge 1497128 into 72f8d86

lib/lamian/raven_context_extension.rb

@@ -6,8 +6,6 @@ module Lamian::RavenContextExtension
   # @see https://www.rubydoc.info/gems/sentry-raven/0.9.2/Raven/Context#extra-instance_method
   def extra
     log = Lamian.dump(format: :txt)
-    return super unless log
-    extra = super || {}
-    extra.merge(lamian_log: log)
+    log ? super.merge!(lamian_log: log) : super
   end
 end

lib/lamian/sidekiq_raven_middleware.rb

@@ -8,7 +8,7 @@ def call(*)
       begin
         yield
       rescue Exception # rubocop:disable Lint/RescueException
-        Raven.extra_context(lamian_log: Lamian.dump(format: :txt))
+        Raven.extra_context # Just trigger saving of the current log
         raise
       end
     end

lib/lamian/version.rb

@@ -13,5 +13,5 @@ module Lamian
   # According to this, it is enought to specify '~> a.b'
   # if private API was not used and to specify '~> a.b.c' if it was
 
-  VERSION = "1.1.0"
+  VERSION = "1.1.1"
 end

spec/lamian/raven_context_extension_spec.rb

@@ -1,18 +1,8 @@
 # frozen_string_literal: true
 
-require "raven"
-require "raven/transports/dummy"
-
-Raven::Context.prepend(Lamian::RavenContextExtension)
-
-Raven.configure do |config|
-  config.dsn = "dummy://public@example.com/project-id"
-  config.encoding = "json"
-  config.logger = Logger.new(nil)
-end
-
 describe Lamian::RavenContextExtension, :cool_loggers do
   after { sent_events.clear }
+  after { Raven::Context.clear! }
 
   let(:sent_events) { Raven.client.transport.events }
   let(:extra_info) { JSON.parse(sent_events.last[1]).fetch("extra") }

spec/lamian/sidekiq_raven_middleware_spec.rb

@@ -2,11 +2,10 @@
 
 describe Lamian::SidekiqRavenMiddleware, :cool_loggers do
   it "call Raven.extra_context with proper lamian_log" do
-    expect(Raven).to receive(:extra_context).with(lamian_log: "some log\n")
+    expect(Raven).to receive(:extra_context)
 
     middleware_call = proc do
       Lamian::SidekiqRavenMiddleware.new.call do
-        generic_logger.info "some log"
         raise "some error"
       end
     end

spec/spec_helper.rb

@@ -9,6 +9,8 @@
 
 require "lamian"
 
+require_relative "support/raven"
+
 RSpec.configure do |config|
   config.order = :random
 end

spec/support/raven.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require "raven"
+require "raven/transports/dummy"
+
+Raven::Context.prepend(Lamian::RavenContextExtension)
+
+Raven.configure do |config|
+  config.dsn = "dummy://public@example.com/project-id"
+  config.encoding = "json"
+  config.logger = Logger.new(nil)
+end