more robust error message on queue full

thomaszurkan-optimizely · thomaszurkan-optimizely · commit 33d4ec59f56d · 2019-12-09T15:07:15.000-08:00
diff --git a/lib/optimizely/event/batch_event_processor.rb b/lib/optimizely/event/batch_event_processor.rb
@@ -91,7 +91,7 @@ def process(user_event)
       begin
         @event_queue.push(user_event, true)
       rescue => e
-        @logger.log(Logger::WARN, 'Payload not accepted by the queue. ' + e.message)
+        @logger.log(Logger::WARN, 'Payload not accepted by the queue: ' + e.message)
         return
       end
     end
diff --git a/spec/event/batch_event_processor_spec.rb b/spec/event/batch_event_processor_spec.rb
@@ -344,7 +344,7 @@
     # Wait until other thread has processed the event.
     while @event_processor.current_batch.length != 10; end
     expect(@event_dispatcher).not_to have_received(:dispatch_event)
-    expect(spy_logger).to have_received(:log).with(Logger::WARN, 'Payload not accepted by the queue.').once
+    expect(spy_logger).to have_received(:log).with(Logger::WARN, 'Payload not accepted by the queue: queue full').once
   end
 
   it 'should not process and log when Executor is not running' do
