exception_no 0.0.3 → 0.0.4

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ODJlYTU1M2UyY2RmZjJjMzgyN2E1ZjIwMjcyYmNjNGZmMTFiYzhjMQ==
+    OGYwNDYzMDA0NmQzN2U4YTU2MGYxMzQ3OTQ5YTY5NTQ5Y2JiNTExNA==
   data.tar.gz: !binary |-
-    NDc5Y2Y2OWI1OWI0YmQ3YTE5YTEzZTgzMGExNTM2M2RlMGYwMDY5NA==
+    NWY4ODFiOGIwNDI3YzcwZjBkOGE3YjE0ZGY4NTQ3MzEzNDM5ZDM2OQ==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    OGNiMTBlNmU3OTFiYjM5Mjg5OTQ5ZDMyNzczZWM4MjhmMGFiNTljNzQzMzgz
-    YzA2OGQwY2Y0ZDUzMTIxNTQzYzgxZjhkN2YyOGQyMDNiNzNkN2Y0YTZlYWQy
-    MDBkNzJjZDE1MjA4ZDJhMTYwMTE4ODExZDA4YjE5NjliMzdhMzU=
+    N2Q3NWE4ZGEyZDI4NTg4MTA5Y2ZkMTQ0MmU4MWQyODUxMzAyMzYwZTk3ZjFm
+    NjBhZTFmZTcxMTg4M2JjN2NlZjEwNzAyZGNiNTQyMGI4MmNhNzEzMzBiMDQ4
+    YzRlYjI1ZGVkNzBkMzM0MTUzYzFkY2I3YzZhYjk4NjRhMDFkMDc=
   data.tar.gz: !binary |-
-    NTdhNTUwYmZmNDg1ZTFlZTVlMmFkZGNiZDEyYzM2MWIwZmRhY2Y0Y2YyOWFk
-    NGY2MTYwNmFjNTFkMDk0NTdjYTQ3NzQ0MmEzYzc4YjczZDc2MzgyNzYwY2Iw
-    MjlmMDZmZjI5ZmE5MjRkZWU3N2NiZTQyOTEzNzQwMDE3NTlmOTU=
+    MWI4YWI5MzQwNTU3ZjNiYzZjYmU1ZDBhNjUwNDkxYmIwMTI3OGMzNDJiYTgz
+    Y2Y3ZDk5OGQzYTU3OTViNmNiMTc2MGUyODBjY2MxYzFjOTNmNTMyY2UwYzhl
+    ZjBlYjJkMGM2MmFjMmQ0MmNhMDQ5NWFhOTZkMGMxNjU0MWEyZDk=

data/lib/exception_no.rb

@@ -2,20 +2,20 @@ require "net/smtp"
 require "erb"
 
 class ExceptionNo
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 
   attr_accessor :backtrace_filter
-  attr_accessor :deliver
+  attr_accessor :behaviors
 
   def initialize(config = {})
     @config = config
     @template = ERB.new(TEMPLATE)
-    @deliver = true
+    @behaviors = [:deliver]
 
     @backtrace_filter = -> line { true }
   end
 
-  def _notify(exception, options = {})
+  def _deliver(exception, options = {})
     body = @template.result(binding)
 
     Net::SMTP.start(@config.fetch(:host), @config.fetch(:port, 25)) do |smtp|
@@ -23,11 +23,9 @@ class ExceptionNo
     end
   end
 
-  def notify(exception, options = {})
-    return unless @deliver
-
+  def deliver(exception, options)
     begin
-      _notify(exception, options)
+      _deliver(exception, options)
     rescue => notification_error
       $stderr.write("*** FAILED SENDING ERROR NOTIFICATION\n")
       $stderr.write("*** #{notification_error.class}: #{notification_error}\n")
@@ -39,6 +37,11 @@ class ExceptionNo
     end
   end
 
+  def notify(exception, options = {})
+    deliver(exception, options) if @behaviors.include?(:deliver)
+    raise exception if @behaviors.include?(:raise)
+  end
+
   TEMPLATE = (<<-'EMAIL').gsub(/^ {2}/, '')
   From: <%= @config[:from_alias] %> <<%= @config[:from] %>>
   To: <<%= @config[:to] %>>

data/test/exception_no.rb

@@ -10,6 +10,8 @@ setup do
 end
 
 test "deliver exception notification" do |notifier|
+  notifier.behaviors = [:deliver]
+
   ex = ArgumentError.new("Really bad argument")
 
   notifier.notify(ex)
@@ -26,6 +28,8 @@ test "deliver exception notification" do |notifier|
 end
 
 test "exception messages with multiple lines" do |notifier|
+  notifier.behaviors = [:deliver]
+
   notifier.notify(ArgumentError.new("A really\nbad\nargument"))
 
   headers, body = parse_email($smtp.outbox.last[:data])
@@ -34,6 +38,8 @@ test "exception messages with multiple lines" do |notifier|
 end
 
 test "includes backtrace information" do |notifier|
+  notifier.behaviors = [:deliver]
+
   begin
     raise ArgumentError, "A bad argument"
   rescue Exception => ex
@@ -47,6 +53,8 @@ test "includes backtrace information" do |notifier|
 end
 
 test "allows to filter the backtrace" do |notifier|
+  notifier.behaviors = [:deliver]
+
   notifier.backtrace_filter = -> line do
     !line.include?(Gem.path.first)
   end
@@ -64,12 +72,28 @@ test "allows to filter the backtrace" do |notifier|
 end
 
 test "disable delivery" do |notifier|
-  notifier.deliver = false
+  notifier.behaviors = []
 
   notifier.notify(ArgumentError.new)
 
   assert_equal $smtp.outbox.size, 0
 end
 
+test "raise exception" do |notifier|
+  notifier.behaviors = [:raise]
+
+  assert_raise(ArgumentError) { notifier.notify(ArgumentError.new) }
+
+  assert_equal $smtp.outbox.size, 0
+end
+
+test "raise exception and deliver notification" do |notifier|
+  notifier.behaviors = [:raise, :deliver]
+
+  assert_raise(ArgumentError) { notifier.notify(ArgumentError.new) }
+
+  assert_equal $smtp.outbox.size, 1
+end
+
 $smtp.stop
 $smtp.join

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: exception_no
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Educabilia
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-18 00:00:00.000000000 Z
+date: 2014-03-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mini-smtp-server