hutch 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9ad4ea66e507cf8c01b3e402c933223e0c525503
-  data.tar.gz: 736c571f2a6cc790ab2b3dda796a1c29b624ec74
+  metadata.gz: 49a9fedccdbf581b5874f8a546ce20545d683b96
+  data.tar.gz: 1783f96443d36488ed329271e5f382123ed79a98
 SHA512:
-  metadata.gz: d16e7ef03f63fdedf9501bee91dd49e6b5edb03db74e1027d7a0fa1ed140994dd9e545f757484f86441d078271523df98d3d9d9d83156ad33329ca761cfbaae9
-  data.tar.gz: 719bd84eced5c8876a2dbc43b785db900bc7300a091ffb479a6949fb0797ef66eb354a273b438f92ef3932eba07a1e1703dc8c2abed31cea7fd4bbd52f109a83
+  metadata.gz: d59463f950dd59b7a5cb575b6eeb47d97400fcea4626d168902b7d5c442f791d907ece465a2841c68c6c09a85b253da33eae08d3b6228cf1b19315680bf972b2
+  data.tar.gz: 3f7d8abae79d3227d656c427bc3d6201060545a8fab2523d89c1048011b52506481f8041e0ed7923e2c32a18d1db78ca2e9ff36d5eee07c8edbccb6f4f70c92b

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.5.1 - October 17, 2013
+
+- Raise an exception when publishing fails
+
 ## 0.5.0 - October 17, 2013
 
 - Support for the `--mq-tls-key` and `--mq-tls-cert` configuration options.

data/README.md

@@ -29,7 +29,7 @@ gem install hutch
 To use in a project managed with Bundler:
 
 ``` ruby
-gem "hutch", "~> 0.5.0"
+gem "hutch", "~> 0.5.1"
 ```
 
 ## Overview

data/lib/hutch/broker.rb

@@ -151,14 +151,23 @@ module Hutch
     def publish(routing_key, message)
       payload = JSON.dump(message)
 
-      if @connection && @connection.open?
-        logger.info "publishing message '#{message.inspect}' to #{routing_key}"
-        @exchange.publish(payload, routing_key: routing_key, persistent: true,
-                          timestamp: Time.now.to_i, message_id: generate_id)
-      else
-        logger.error "Unable to publish : routing key: #{routing_key}, " +
-                     "message: #{message}"
+      unless @connection
+        msg = "Unable to publish - no connection to broker. " +
+              "Message: #{message.inspect}, Routing key: #{routing_key}."
+        logger.error(msg)
+        raise PublishError, msg
+      end
+
+      unless @connection.open?
+        msg = "Unable to publish - connection is closed. " +
+              "Message: #{message.inspect}, Routing key: #{routing_key}."
+        logger.error(msg)
+        raise PublishError, msg
       end
+
+      logger.info("publishing message '#{message.inspect}' to #{routing_key}")
+      @exchange.publish(payload, routing_key: routing_key, persistent: true,
+                        timestamp: Time.now.to_i, message_id: generate_id)
     end
 
     private

data/lib/hutch/exceptions.rb

@@ -2,4 +2,5 @@ module Hutch
   class ConnectionError < StandardError; end
   class AuthenticationError < StandardError; end
   class WorkerSetupError < StandardError; end
+  class PublishError < StandardError; end
 end

data/lib/hutch/version.rb

@@ -1,4 +1,4 @@
 module Hutch
-  VERSION = '0.5.0'.freeze
+  VERSION = '0.5.1'.freeze
 end
 

data/spec/hutch/broker_spec.rb

@@ -147,9 +147,14 @@ describe Hutch::Broker do
     end
 
     context 'without a valid connection' do
+      it 'raises an exception' do
+        expect { broker.publish('test.key', 'message') }.
+          to raise_exception(Hutch::PublishError)
+      end
+
       it 'logs an error' do
         broker.logger.should_receive(:error)
-        broker.publish('test.key', 'message')
+        broker.publish('test.key', 'message') rescue nil
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hutch
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Harry Marr