structured-event-logger 0.1.0 → 0.1.1

data/lib/structured_event_logger.rb

@@ -4,11 +4,11 @@ class StructuredEventLogger
 
   class Error < ::StandardError; end
 
-  class EndpointException < StructuredEventLogger::Error
-    attr_reader :name, :wrapped_exception
-    def initialize(name, wrapped_exception)
-      @name, @wrapped_exception = name, wrapped_exception
-      super("Endpoint #{name} failed - #{wrapped_exception.class.name}: #{wrapped_exception.message}")
+  class EventHandlingException < StructuredEventLogger::Error
+    attr_reader :exceptions
+    def initialize(scope, name, exceptions)
+      @scope, @name, @exceptions = scope, name, exceptions
+      super("Failed to submit the #{scope}/#{name} event to the following endpoints: #{exceptions.keys.join(", ")}")
     end
   end
 
@@ -25,7 +25,7 @@ class StructuredEventLogger
 
   def event(scope, event, content = {})
     log_event scope, event, flatten_hash(content)
-  rescue EndpointException => e
+  rescue EventHandlingException => e
     error_handler.call(e)
   end
 
@@ -53,13 +53,16 @@ class StructuredEventLogger
     record.update(event_name: event, event_scope: scope, event_uuid: SecureRandom.uuid, event_timestamp: Time.now.utc)
     record.update(hash)
 
+    exceptions = {}
     endpoints.each do |name, endpoint|
       begin
         endpoint.call(scope, event, hash, record)
       rescue => e
-        raise EndpointException.new(name, e)
+        exceptions[name] = e
       end
     end
+
+    raise EventHandlingException.new(scope, event, exceptions) unless exceptions.empty?
   end
 
   def thread_key

data/lib/structured_event_logger/version.rb

@@ -1,3 +1,3 @@
 class StructuredEventLogger
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/test/structured_event_logger_test.rb

@@ -135,28 +135,32 @@ class StructuredEventLoggerTest < Minitest::Test
 
   def test_should_fail_when_syslog_message_is_too_large
     @event_logger.endpoints[:syslog] = StructuredEventLogger::Syslogger.new
-    assert_raises(StructuredEventLogger::EndpointException) do
+    assert_raises(StructuredEventLogger::EventHandlingException) do
       @event_logger.event(:test, :syslog, message: 'a' * (64 * 1024 + 1))
     end
   end
 
-  def test_should_execute_a_custom_error_handler_on_failure
+  def test_should_raise_exception_when_endpoint_fails
     @event_logger.endpoints[:failer] = proc { raise "FAIL" }
+    assert_raises(StructuredEventLogger::EventHandlingException) do
+      @event_logger.event(:test, :fail)
+    end
+  end  
+
+  def test_should_execute_a_custom_error_handler_on_failure
+    @event_logger.endpoints[:failer1] = proc { raise "FAIL" }
+    @event_logger.endpoints[:failer2] = proc { raise "FAIL" }
     @event_logger.error_handler = mock()
     @event_logger.error_handler.expects(:call).with do |exception|
-      assert_kind_of StructuredEventLogger::EndpointException, exception
-      assert_equal 'FAIL', exception.wrapped_exception.message
-      assert_equal 'Endpoint failer failed - RuntimeError: FAIL', exception.message
+      assert_kind_of StructuredEventLogger::EventHandlingException, exception
+      assert_equal 'Failed to submit the test/fail event to the following endpoints: failer1, failer2', exception.message
+      assert_equal 2, exception.exceptions.size
+      assert_equal 'FAIL', exception.exceptions[:failer1].message
+      assert_kind_of RuntimeError, exception.exceptions[:failer2]
     end
     @event_logger.event(:test, :fail)
   end
 
-  def test_should_raise_exception_when_endpoint_fails
-    @event_logger.endpoints[:failer] = proc { raise "FAIL" }
-    assert_raises(StructuredEventLogger::EndpointException) do
-      @event_logger.event(:test, :fail)
-    end
-  end  
 
   private 
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: structured-event-logger
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-19 00:00:00.000000000 Z
+date: 2013-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport