action_reporter 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 13e5a1b759408acb2b8af922f9075aad25fa5b1dcc39e931a6d171ea81bc4b9a
-  data.tar.gz: 5ac0c5e94e6c9b56bb25c88df82ce80350c4d43fa67284c60e8af7c8b87bba19
+  metadata.gz: a55d0da65743cb598db1ca915e69704f6fd3871921b015193d6445eaf5f1702b
+  data.tar.gz: 469868c0928f40fbefaabf40a797ce86d3b3aaa16c2f283faaee8dc2ee4a4ee5
 SHA512:
-  metadata.gz: 9a8c8d4991ed7b85e5822ec28a56d4b5f3a383cec09e5651be332d12943b6fdc63cd97e71f9b56d00e54b675faddaac657027d9e9290a8cde570ac70483ae5c1
-  data.tar.gz: f155b3a4484e67f1e2b4e06d885b0759335eda30ce47b739f293dbb2cfdf541b51a13211762598bc894e8d8c57650d581cc00d011e0d47ef261ee274319074d7
+  metadata.gz: 9a280cc47e2b244299dd1279e480b1b65454e810f99ad675f5abfefe4b93dc8b80ed2e84816f28222c11d10977384d5c41fb5c23bcbe53f67d517c28b68dbddb
+  data.tar.gz: b80979582abbedc6db34ba723a8f225200e37c3e2fe5bf5eddd05a4d4301aef0a442fcd97002ac94dc6f4d4fa7cdb383e591afad2108039cb8f80881eb7aeaf6

data/CHANGELOG.md

@@ -1,3 +1,13 @@
+# 1.2.0
+
+* Major fixes for class resolvers
+* Implemented ActionReporter::Error class
+* Improved test coverage
+
+# 1.1.1
+
+* Update check-in logic
+
 # 1.1.0
 
 * Add reporter check-in method

data/action_reporter.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = "action_reporter"
-  s.version = "1.1.0"
+  s.version = "1.2.0"
 
   s.license = "MIT"
 

data/lib/action_reporter/base.rb

@@ -1,10 +1,12 @@
 module ActionReporter
   class Base
     def self.class_accessor(class_name)
-      return unless Object.const_defined?(class_name)
+      method_name = class_name.gsub("::", "_").downcase + "_class"
+      define_method(method_name) do
+        raise Error.new("#{class_name} is not defined") unless Object.const_defined?(class_name)
 
-      const_name = class_name.gsub("::", "")
-      const_set(const_name, Object.const_get(class_name))
+        Object.const_get(class_name)
+      end
     end
 
     def transform_context(context)
@@ -18,12 +20,12 @@ module ActionReporter
     end
 
     def resolve_check_in_id(identifier)
-      if identifier.is_a?(String)
-        identifier
-      elsif identifier.respond_to?(:reporter_check_in)
+      if identifier.respond_to?(:reporter_check_in)
         identifier.reporter_check_in
+      elsif identifier.respond_to?(:to_s)
+        identifier.to_s
       else
-        raise ArgumentError, "Unknown check-in identifier: #{identifier.inspect}"
+        raise ArgumentError.new("Unknown check-in identifier: #{identifier.inspect}")
       end
     end
 

data/lib/action_reporter/error.rb

@@ -0,0 +1,3 @@
+module ActionReporter
+  class Error < StandardError; end
+end

data/lib/action_reporter/honeybadger_reporter.rb

@@ -4,21 +4,25 @@ module ActionReporter
 
     def notify(error, context: {})
       new_context = transform_context(context)
-      Honeybadger.notify(error, context: new_context)
+      honeybadger_class.notify(error, context: new_context)
     end
 
     def context(args)
       new_context = transform_context(args)
-      Honeybadger.context(new_context)
+      honeybadger_class.context(new_context)
     end
 
     def reset_context
-      Honeybadger.context.clear!
+      honeybadger_class.context.clear!
     end
 
     def check_in(identifier)
       check_in_id = resolve_check_in_id(identifier)
-      Honeybadger.check_in(check_in_id)
+      honeybadger_class.check_in(check_in_id)
+    end
+
+    def audited_user=(user)
+      honeybadger_class.context(user_global_id: user.to_global_id.to_s) if user
     end
   end
 end

data/lib/action_reporter/rails_reporter.rb

@@ -4,19 +4,19 @@ module ActionReporter
 
     def notify(error, context: {})
       new_context = transform_context(context)
-      Rails.logger.info(
+      rails_class.logger.info(
         "Reporter notification: #{error.inspect}, #{new_context.inspect}"
       )
     end
 
     def context(args)
       new_context = transform_context(args)
-      Rails.logger.info("Reporter context: #{new_context.inspect}")
+      rails_class.logger.info("Reporter context: #{new_context.inspect}")
     end
 
     def check_in(identifier)
       check_in_id = resolve_check_in_id(identifier)
-      Rails.logger.info("Reporter check-in: #{check_in_id}")
+      rails_class.logger.info("Reporter check-in: #{check_in_id}")
     end
   end
 end

data/lib/action_reporter/scout_apm_reporter.rb

@@ -5,12 +5,16 @@ module ActionReporter
 
     def notify(error, context: {})
       self.context(context)
-      ScoutApmError.capture(error)
+      scoutapm_error_class.capture(error)
     end
 
     def context(args)
       new_context = transform_context(args)
-      ScoutApmContext.add(new_context)
+      scoutapm_context_class.add(new_context)
+    end
+
+    def audited_user=(user)
+      scoutapm_context_class.add_user(user_global_id: user&.to_global_id&.to_s)
     end
   end
 end

data/lib/action_reporter/sentry_reporter.rb

@@ -3,23 +3,27 @@ module ActionReporter
     class_accessor "Sentry"
 
     def notify(error, context: {})
-      Sentry.with_scope do |temp_scope|
+      sentry_class.with_scope do |temp_scope|
         temp_scope.set_context("context", transform_context(context))
 
         if error.is_a?(StandardError)
-          Sentry.capture_exception(error)
+          sentry_class.capture_exception(error)
         else
-          Sentry.capture_message(error)
+          sentry_class.capture_message(error)
         end
       end
     end
 
     def context(args)
-      Sentry.get_current_scope.set_context("context", transform_context(args))
+      sentry_class.get_current_scope.set_context("context", transform_context(args))
+    end
+
+    def reset_context
+      sentry_class.get_current_scope.set_context("context", {})
     end
 
     def audited_user=(user)
-      Sentry.set_user(global_id: user.to_global_id.to_s) if user
+      sentry_class.set_user(user_global_id: user&.to_global_id&.to_s)
     end
   end
 end

data/lib/action_reporter/version.rb

@@ -1,3 +1,3 @@
 module ActionReporter
-  VERSION = '1.1.0'
+  VERSION = '1.2.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: action_reporter
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Andrei Makarov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-18 00:00:00.000000000 Z
+date: 2023-04-20 00:00:00.000000000 Z
 dependencies: []
 description: Ruby wrapper for multiple reporting services
 email:
@@ -24,6 +24,7 @@ files:
 - lib/action_reporter.rb
 - lib/action_reporter/audited_reporter.rb
 - lib/action_reporter/base.rb
+- lib/action_reporter/error.rb
 - lib/action_reporter/honeybadger_reporter.rb
 - lib/action_reporter/rails_reporter.rb
 - lib/action_reporter/scout_apm_reporter.rb