roqua-support 0.1.2 → 0.1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5d537f6968d87c89a2c76d6eaa69317470f29207
-  data.tar.gz: b10fe72f54f88222284c8af7b0a93327c9cac436
+  metadata.gz: 5630dc4eb6452f316d41227ba3b233fd0ec4446b
+  data.tar.gz: 3a33380768215e7f9cf6213511c512583af0946e
 SHA512:
-  metadata.gz: b9681dd8f6c01c76aa9e2fd210dc98ea4428239e68110d07c735621384a0dba4b85ad316b849bb563afb88391968d7de6998b8ef3b084d88873afc8dcf847fcf
-  data.tar.gz: eb6966c9414ccfd46d49bfe664396009779213131e0c706cd664a4a148e50bf4346f8e3d8fc1a453596751c1075ad0af00a0aebbcfccfdb3e56c78741b4b386c
+  metadata.gz: 3d3a36dc54de7db6ed108e623d947ad13e91ed93f4a4300774254fdecd773b683290282046a3f90f8453924bb291d94d6ea65911b10970fa04a2f75e11ddb05f
+  data.tar.gz: 54aa2724374a6f7ee746b5d254731007305540e9e405589ef8c57c1a598ee2820519f71a49ef0cbe87d2c6fee99dd6ac05ad69334e6499547b6269c84550e64c

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.1.2.1
+
+* Bugfixes for Errors.
+
 ## 0.1.2
 
 * Added `Roqua::Support::Errors.report(exception, extra_info = {})` which sends exception to all configured exception notifiers

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    roqua-support (0.1.2)
+    roqua-support (0.1.2.1)
       activesupport (>= 3.2, < 5.0)
 
 GEM

data/lib/roqua-support/version.rb

@@ -1,5 +1,5 @@
 module Roqua
   module Support
-    VERSION = "0.1.2"
+    VERSION = "0.1.2.1"
   end
 end

data/lib/roqua/support.rb

@@ -1,5 +1,6 @@
 require 'logger'
 require 'roqua/support/logging'
+require 'roqua/support/errors'
 
 module Roqua
   class << self

data/lib/roqua/support/errors.rb

@@ -1,21 +1,6 @@
 module Roqua
   module Support
     module Errors
-      if const_defined?(:Appsignal)
-        module Appsignal
-          # TODO: If and when https://github.com/appsignal/appsignal/pull/9 is merged,
-          # this functionality should be supported directly by Appsignal.send_exception.
-          def send_exception_with_tags(exception, tags = {})
-            return if is_ignored_exception?(exception)
-            transaction = Appsignal::Transaction.create(SecureRandom.uuid, ENV.to_hash)
-            transaction.set_tags(tags)
-            transaction.add_exception(exception)
-            transaction.complete!
-            Appsignal.agent.send_queue
-          end
-        end
-      end
-
       def self.extra_parameters
         @extra_parameters || {}
       end
@@ -25,6 +10,7 @@ module Roqua
       end
 
       def self.report(exception, extra_params = {})
+        return if const_defined?(:Rails) and Rails.env.test?
         parameters = extra_parameters.merge(extra_params)
 
         # Notify Roqua logging
@@ -42,8 +28,17 @@ module Roqua
         end
 
         # Notify AppSignal
-        if const_defined?(:Appsignal)
-          Appsignal.send_exception_with_tags(exception, parameters: parameters)
+        if const_defined?(:Appsignal) and
+           not Appsignal.is_ignored_exception?(exception)
+          # TODO: If and when https://github.com/appsignal/appsignal/pull/9 is merged,
+          # this functionality should be supported directly by Appsignal.send_exception.
+          # Appsignal.send_exception(exception, parameters: parameters)
+
+          transaction = Appsignal::Transaction.create(SecureRandom.uuid, ENV.to_hash)
+          transaction.set_tags(parameters)
+          transaction.add_exception(exception)
+          transaction.complete!
+          Appsignal.agent.send_queue
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: roqua-support
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.2.1
 platform: ruby
 authors:
 - Marten Veldthuis