roqua-support 0.1.2.1 → 0.1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5630dc4eb6452f316d41227ba3b233fd0ec4446b
-  data.tar.gz: 3a33380768215e7f9cf6213511c512583af0946e
+  metadata.gz: 0ac53c450b17e874859934e55be62b8ddc7ae00d
+  data.tar.gz: 2f54b672a70529b33ee906ee88788c3f736e98ec
 SHA512:
-  metadata.gz: 3d3a36dc54de7db6ed108e623d947ad13e91ed93f4a4300774254fdecd773b683290282046a3f90f8453924bb291d94d6ea65911b10970fa04a2f75e11ddb05f
-  data.tar.gz: 54aa2724374a6f7ee746b5d254731007305540e9e405589ef8c57c1a598ee2820519f71a49ef0cbe87d2c6fee99dd6ac05ad69334e6499547b6269c84550e64c
+  metadata.gz: 144657fabc0578bd624c3e28a098da7c3f8623791c87cf1bb40dd5433bbb21e6179c663c201699bf9ef08f87a712363a54a522c1c7c3b765d567005fd2f4bdc6
+  data.tar.gz: 9f26d2f6fb732c0fc89f54619371d12e45804cff76099f1b708ebfccf63c373c9426a51e89e984197ac64e8eaa890354621583e1e1d46f5a1ee4ce9608fa5bef

data/Gemfile.lock

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

data/lib/roqua-support/version.rb

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

data/lib/roqua/support/errors.rb

@@ -34,11 +34,13 @@ module Roqua
           # 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
+          if Appsignal.active?
+            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
     end

data/spec/roqua/support/errors_spec.rb

@@ -25,19 +25,30 @@ describe 'Error reporting' do
   end
 
   it 'sends notifications to airbrake' do
-    stub_const("Airbrake", double("Airbrake"))
+    stub_const("Airbrake", double("Airbrake", is_ignored_exception?: false))
     Airbrake.should_receive(:notify_or_ignore).with(exception, parameters: {})
     Roqua::Support::Errors.report exception
   end
 
-  it 'sends notifications to appsignal' do
-    stub_const("Appsignal", double("Appsignal"))
-    Appsignal.should_receive(:send_exception_with_tags).with(exception, parameters: {})
-    Roqua::Support::Errors.report exception
+  context 'when Appsignal is loaded' do
+    let(:agent)       { double("agent") }
+    let(:transaction) { double("transaction") }
+
+    it 'sends notifications to appsignal' do
+      stub_const("Appsignal", Module.new)
+      Appsignal.stub(is_ignored_exception?: false, agent: agent)
+      stub_const("Appsignal::Transaction", double("Transaction", create: transaction))
+
+      transaction.should_receive(:set_tags).with({})
+      transaction.should_receive(:add_exception).with(exception)
+      transaction.should_receive(:complete!)
+      agent.should_receive(:send_queue)
+      Roqua::Support::Errors.report exception
+    end
   end
 
   it 'supports default extra params' do
-    Roqua::Support::Errors.extra_parameters = {organization: 'some_org'}
+    Roqua::Support::Errors.stub(extra_parameters: {organization: 'some_org'})
     Roqua.logger.should_receive(:error).with('roqua.exception',
                                              class_name: 'Exception',
                                              message: 'exception_message',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: roqua-support
 version: !ruby/object:Gem::Version
-  version: 0.1.2.1
+  version: 0.1.2.2
 platform: ruby
 authors:
 - Marten Veldthuis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-04 00:00:00.000000000 Z
+date: 2014-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport