Merge pull request #137 from raphaeladrien/handling-payload-bigger-than-10KB

Refactor request.rb to handling HTTP 413

Author: jameshfisher

lib/pusher/request.rb

@@ -94,6 +94,8 @@ def handle_response(status_code, body)
         raise Error, "404 Not found (#{@uri.path})"
       when 407
         raise Error, "Proxy Authentication Required"
+      when 413
+        raise Error, "Payload Too Large > 10KB"
       else
         raise Error, "Unknown error (status code #{status_code}): #{body}"
       end

spec/client_spec.rb

@@ -430,6 +430,11 @@
             expect { call_api }.to raise_error(Pusher::Error, 'Proxy Authentication Required')
           end
 
+          it "should raise Pusher::Error if pusher returns 413" do
+            stub_request(verb, @url_regexp).to_return({:status => 413})
+            expect { call_api }.to raise_error(Pusher::Error, 'Payload Too Large > 10KB')
+          end
+
           it "should raise Pusher::Error if pusher returns 500" do
             stub_request(verb, @url_regexp).to_return({:status => 500, :body => "some error"})
             expect { call_api }.to raise_error(Pusher::Error, 'Unknown error (status code 500): some error')