amqp-boilerplate 0.0.6 → 0.0.7

data/CHANGELOG

@@ -1,3 +1,8 @@
+= version 0.0.7
+
+  * [ENHANCEMENT] Log payload when log level is debug
+  * [ENHANCEMENT] Do not publish +nil+ messages
+
 = version 0.0.6
 
   * [FEATURE] Do not abort consumer thread when a StandardError-like exception

data/lib/amqp/boilerplate/consumer.rb

@@ -86,6 +86,7 @@ module AMQP
       # aborted. See Ruby's exception inheritance hierarchy for a complete
       # list of what is and what is not handled by this wrapper.
       def handle_message_wrapper(metadata, payload)
+        AMQP::Boilerplate.logger.debug("[#{self.class}#handle_message_wrapper] Received message: #{payload}")
         handle_message(metadata, payload)
       rescue StandardError => e
         message = <<-MSG

data/lib/amqp/boilerplate/producer.rb

@@ -94,8 +94,12 @@ module AMQP
         # @return [void]
         def publish
           message = send(self.class.amqp_boilerplate_message.to_sym)
-          exchange.publish(message, self.class.amqp_boilerplate_options) do
-            AMQP::Boilerplate.logger.debug "[#{self.class}] Published message:\n#{message}"
+          if message
+            exchange.publish(message, self.class.amqp_boilerplate_options) do
+              AMQP::Boilerplate.logger.debug "[#{self.class}] Published message:\n#{message}"
+            end
+          else
+            AMQP::Boilerplate.logger.debug "[#{self.class}] Not publishing nil message"
           end
         end
 

data/lib/amqp/boilerplate/version.rb

@@ -1,5 +1,5 @@
 module AMQP
   module Boilerplate
-    VERSION = "0.0.6"
+    VERSION = "0.0.7"
   end
 end

data/spec/amqp/boilerplate/producer_spec.rb

@@ -40,6 +40,13 @@ describe AMQP::Boilerplate::Producer do
       another_producer.publish
     end
 
+    it "should only log nil messages" do
+      @nilproducer = NilProducer.new
+      AMQP::Boilerplate.logger.should_receive(:debug).with("[#{@nilproducer.class}] Not publishing nil message")
+      @nilproducer.publish
+    end
+
+
     it "should pass options to AMQP::Exchange#publish" do
       BarProducer.amqp({ :routing_key => "some.routing.key", :mandatory => true })
       @exchange.should_receive(:publish).with(@producer.message, :routing_key => "some.routing.key", :mandatory => true)

data/spec/spec_helper.rb

@@ -28,6 +28,17 @@ class FooProducer
   end
 end
 
+class NilProducer
+  extend AMQP::Boilerplate::Producer
+
+  amqp :routing_key => "yet.another.routing.key"
+  amqp_message :nil_message
+
+  def nil_message
+    nil
+  end
+end
+
 class BarProducer
   extend AMQP::Boilerplate::Producer
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: amqp-boilerplate
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: 
   segments: 
   - 0
   - 0
-  - 6
-  version: 0.0.6
+  - 7
+  version: 0.0.7
 platform: ruby
 authors: 
 - Patrick Baselier
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-09-21 00:00:00 +02:00
+date: 2011-09-26 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency