tops_connect 0.4.1 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 345f5053904bc5be6db1f16a2b862d926c0822ce
-  data.tar.gz: 05222a7f429dcdfc451f163630058426e3a140b6
+  metadata.gz: c4b4ccf4ab5dad5fd3680bff0ea869b7683b3c74
+  data.tar.gz: '0328eed8bce785b4bf4a57f1ca50bb7c3de23727'
 SHA512:
-  metadata.gz: 1b24a11f52aad027934c00cb69b84c3932660828b5928329763c0f67169ffe939cf194f6ef74e29df8a322ccaafb7cb9d9a98f9f3d9235d2d32d23486b40c616
-  data.tar.gz: c29194c29660d1d478808527a8bd8bfa5fe7cfb0c41ec4188c8bfe3c48d0f3a5ea529d3e9aeac66dbc3b9d3353b1379fd4a112e499d5d9751901ad3be0619c80
+  metadata.gz: efd3af5e304363d9701b9f7d49a63e0857c7abb6d8b36624c5426e575febbeb8a9a4f6fef3a627c4b41cbed3f545b715d107b90d386c761e1a3cf9b86436126e
+  data.tar.gz: e743ec9665adf8f358aabf9e8cd1f78bb61d1564138e2e6d17a4ddd0a4f43885770ec4a360e0067fb12cc2087ccbf6b57b32821a619c54eb303ceec5e9feaeac

data/lib/tops_connect/client.rb

@@ -33,24 +33,46 @@ module TopsConnect
         headers: headers.merge('community-api-key' => @community_api_key)
       )
 
+      raise_exception(response) unless response.code == 200
+
+      response.parsed_response
+    end
+
+    protected
+
+    def raise_exception(response)
       case response.code
-      when 200
-        response.parsed_response
+      when 404
+        raise TopsConnect::NotFoundError, response
       when 400..499
-        raise TopsConnect::BadRequest, response.parsed_response['Message']
+        raise TopsConnect::ClientError, response
       when 500..599
-        raise TopsConnect::InternalError, response.parsed_response['Message']
+        raise TopsConnect::InternalError, response
       else
-        raise TopsConnect::ApiError,
-              "#{response.code}: #{response.parsed_response['Message']}"
+        # As far as I'm aware, Tops does not return 100 - 199 or 201 - 399.
+        raise TopsConnect::ApiError, response
       end
     end
   end
 
   class ApiError < ::RuntimeError
+    def initialize(response)
+      @response = response
+    end
+
+    def to_s
+      '%{code}: %{message} (%{uri})'.format(
+        code: @response.code,
+        message: @response.parsed_response&.dig('Message'),
+        uri: @response.request.last_uri.to_s
+      )
+    end
+  end
+
+  class ClientError < ApiError
   end
 
-  class BadRequest < ApiError
+  class NotFoundError < ClientError
   end
 
   class InternalError < ApiError

data/lib/tops_connect/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module TopsConnect
-  VERSION = '0.4.1'
+  VERSION = '0.4.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tops_connect
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: ruby
 authors:
 - Steven Hoffman
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-01-13 00:00:00.000000000 Z
+date: 2017-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler