klout-rb 1.0.0 → 1.0.1

data/lib/klout/client.rb

@@ -8,6 +8,7 @@ require 'singleton'
 module Klout
   class Error < StandardError; end
   class Error::Forbidden < Klout::Error; end
+  class Error::GatewayTimeout < Klout::Error; end
   class Error::InternalServerError < Klout::Error; end
   class Error::ServiceUnavailable < Klout::Error; end
 
@@ -106,18 +107,25 @@ module Klout
     # Raises Klout::Error::Forbidden if Klout api key is not valid.
     # Raises Klout::Error::InternalServerError if there is a server error in Klout.
     # Raises Klout::Error::ServiceUnavailable if Klout endpoint is unavailable.
+    # Raises Klout::Error::GatewayTimeout if Klout endpoint respond with timeout.
+    # Raises Klout::Error for a non 200 response code.
     def parse_response(response)
       case response.code.to_i
+      when 200
+        body = ::MultiJson.decode(response.body)
+        raise Klout::Error.new(body['body']['error']) if body.has_key?('body') && body['body'].has_key?('error')
+        Hashie::Mash.new(body)
       when 403
         raise Klout::Error::Forbidden.new
       when 500
         raise Klout::Error::InternalServerError.new
       when 503
         raise Klout::Error::ServiceUnavailable.new
+      when 504
+        raise Klout::Error::GatewayTimeout.new
+      else
+        raise Klout::Error.new("#{response.code} - #{response.headers['X-Mashery-Error-Code']}")
       end
-      body = ::MultiJson.decode(response.body)
-      raise Klout::Error.new(body['body']['error']) if body.has_key?('body') && body['body'].has_key?('error')
-      Hashie::Mash.new(body)
     end
 
     # Internal: Builds the request url.

data/lib/klout/version.rb

@@ -1,4 +1,4 @@
 module Klout
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end
 

data/spec/klout_spec.rb

@@ -49,6 +49,28 @@ describe Klout::Client do
         expect { Klout::Client.score("damiancaruso") }.to raise_error(Klout::Error::ServiceUnavailable)
       end
     end
+
+    context "gateway timeout" do
+      before do
+        stub_get("/1/klout.json").with(:query => @query.merge({:users => "damiancaruso"})).
+          to_return(:status => 504)
+      end
+
+      it "should raise an error if a gateway timeout is received" do
+        expect { Klout::Client.score("damiancaruso") }.to raise_error(Klout::Error::GatewayTimeout)
+      end
+    end
+
+    context "non sucessful response" do
+      before do
+        stub_get("/1/klout.json").with(:query => @query.merge({:users => "damiancaruso"})).
+          to_return(:status => 502)
+      end
+
+      it "should raise an error for non 200 response codes" do
+        expect { Klout::Client.score("damiancaruso") }.to raise_error(Klout::Error)
+      end
+    end
   end
 
   describe ".score" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klout-rb
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hashie
-  requirement: &73583900 !ruby/object:Gem::Requirement
+  requirement: &74836180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 1.2.0
   type: :runtime
   prerelease: false
-  version_requirements: *73583900
+  version_requirements: *74836180
 - !ruby/object:Gem::Dependency
   name: httpi
-  requirement: &73583560 !ruby/object:Gem::Requirement
+  requirement: &74835750 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 0.9.5
   type: :runtime
   prerelease: false
-  version_requirements: *73583560
+  version_requirements: *74835750
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &73583260 !ruby/object:Gem::Requirement
+  requirement: &74835350 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 1.0.3
   type: :runtime
   prerelease: false
-  version_requirements: *73583260
+  version_requirements: *74835350
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &73582970 !ruby/object:Gem::Requirement
+  requirement: &74835100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: 0.9.2.2
   type: :development
   prerelease: false
-  version_requirements: *73582970
+  version_requirements: *74835100
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &73582710 !ruby/object:Gem::Requirement
+  requirement: &74834830 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,10 +65,10 @@ dependencies:
         version: 2.7.0
   type: :development
   prerelease: false
-  version_requirements: *73582710
+  version_requirements: *74834830
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &73582450 !ruby/object:Gem::Requirement
+  requirement: &74834440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -76,7 +76,7 @@ dependencies:
         version: 1.7.7
   type: :development
   prerelease: false
-  version_requirements: *73582450
+  version_requirements: *74834440
 description: Ruby wrapper for the Klout REST API
 email:
 - damian.caruso@gmail.com
@@ -118,7 +118,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 947582353
+      hash: 714302189
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -127,7 +127,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 947582353
+      hash: 714302189
 requirements: []
 rubyforge_project: klout-rb
 rubygems_version: 1.8.10