rorvswild 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3543194c8dc3ff5474934185936a3e514d347416
-  data.tar.gz: ef3d398dc36f503b8adaacacd24a0396c187eac6
+  metadata.gz: 91ff73c055e4c1842915157c1efa958eb24cfad1
+  data.tar.gz: 6e6d5b3070ba4156499b9710c94260a857df5ac0
 SHA512:
-  metadata.gz: adc6cf77472d1657f151c4287dba352ceaf5675de5cde8d6d7a9c2801ef4d99967955601b43acf32684f07476f9992117a4ac13d7102ddd02595ba015bf8ce15
-  data.tar.gz: 5b6c56513e62c187b9871381eba3bde99c9ff71c6c3ab593406c96f434ca33fd85eeb00437af213c6eee4b05dc7fc3c671428cfc7fb14930f99ae6c7d5400823
+  metadata.gz: 7ea535f196c3f3964d9b8bd33654e2591637bdd2949ef599d6aed5359b3ade919254d1ff489c0f00108eb780b5b2fb8cd48ae84598fedfa0066de314ceb41acd
+  data.tar.gz: cc15c40d7b585a8496e4ba2d12f9681f5a7951b61acee64fc180ffe254b4ec236cbf22335375a41a75c429ec6917d8fe99d95e1bc0310c184f16c40e6f4ee736

data/lib/rorvswild/version.rb

@@ -1,3 +1,3 @@
 module RorVsWild
-  VERSION = "0.4.0".freeze
+  VERSION = "0.4.1".freeze
 end

data/lib/rorvswild.rb

@@ -84,7 +84,7 @@ module RorVsWild
         @app_root ||= Rails.root.to_s
         config = Rails.application.config
         @parameter_filter = ActionDispatch::Http::ParameterFilter.new(config.filter_parameters)
-        @ignored_exceptions = %w[ActionController::RoutingError] + config.action_dispatch.rescue_responses.map { |(key,value)| key }
+        @ignored_exceptions ||= %w[ActionController::RoutingError] + config.action_dispatch.rescue_responses.map { |(key,value)| key }
       end
 
       @logger ||= Logger.new(STDERR)
@@ -150,7 +150,7 @@ module RorVsWild
     end
 
     def after_exception(exception, controller)
-      if !ignored_exceptions.include?(exception.class.to_s)
+      if !ignored_exception?(exception)
         file, line = exception.backtrace.first.split(":")
         request[:error] = exception_to_hash(exception).merge(
           session: controller.session.to_hash,
@@ -185,8 +185,8 @@ module RorVsWild
       cpu_time_offset = cpu_time
       begin
         block.call
-      rescue Exception => exception
-        job[:error] = exception_to_hash(exception)
+      rescue Exception => ex
+        job[:error] = exception_to_hash(ex) if !ignored_exception?(ex)
         raise
       ensure
         job[:runtime] = (Time.now - started_at) * 1000
@@ -198,9 +198,9 @@ module RorVsWild
     def catch_error(extra_details = nil, &block)
       begin
         block.call
-      rescue Exception => exception
-        record_error(exception, extra_details)
-        exception
+      rescue Exception => ex
+        record_error(ex, extra_details) if !ignored_exception?(ex)
+        ex
       end
     end
 
@@ -385,6 +385,10 @@ module RorVsWild
       hash.clone.keep_if { |key,value| key == key.upcase }
     end
 
+    def ignored_exception?(exception)
+      ignored_exceptions.include?(exception.class.to_s)
+    end
+
     def log_error(exception)
       @logger.error("[RorVsWild] " + exception.inspect)
       @logger.error("[RorVsWild] " + exception.backtrace.join("\n[RorVsWild] "))

data/test/ror_vs_wild_test.rb

@@ -27,6 +27,13 @@ class RorVsWildTest < Minitest::Test
     assert(client.send(:job)[:error])
   end
 
+  def test_mesure_block_when_exception_is_ignored
+    client = initialize_client(ignored_exceptions: %w[ZeroDivisionError])
+    client.expects(:post_job)
+    assert_raises(ZeroDivisionError) { RorVsWild.measure_code("1/0") }
+    refute(client.send(:job)[:error])
+  end
+
   def test_measure_code_when_no_client
     RorVsWild.register_default_client(nil)
     RorVsWild::Client.any_instance.expects(:post_job).never

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rorvswild
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Alexis Bernard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-27 00:00:00.000000000 Z
+date: 2017-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler