Merge pull request #31 from zendesk/mciupak/support_precondition_failed_error

Add ApiClient::Errors::PreconditionFailed

Author: marekciupak

CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.5.26
+
+* Add support for HTTP status code: 412 Precondition Failed
+
 # 0.5.25
 
 * Fix broken gem build (gemspec files)

lib/api_client/connection/basic.rb

@@ -109,6 +109,8 @@ def handle_response(request, response)
             raise ApiClient::Errors::Conflict.new(nil, request, response)
           when 410
             raise ApiClient::Errors::Gone.new(nil, request, response)
+          when 412
+            raise ApiClient::Errors::PreconditionFailed.new(nil, request, response)
           when 422
             raise ApiClient::Errors::UnprocessableEntity.new(response.body, request, response)
           when 423

lib/api_client/errors.rb

@@ -24,6 +24,7 @@ class Conflict < ApiClientError; end
     class Gone < ApiClientError; end
     class ServerError < ApiClientError; end
     class UnprocessableEntity < ApiClientError; end
+    class PreconditionFailed < ApiClientError; end
     class Locked < ApiClientError; end
     class TooManyRequests < ApiClientError; end
   end

lib/api_client/version.rb

@@ -1,3 +1,3 @@
 module ApiClient
-  VERSION = '0.5.25'
+  VERSION = '0.5.26'
 end

spec/api_client/connection/basic_spec.rb

@@ -149,6 +149,13 @@
       }.should raise_error(ApiClient::Errors::Gone, "Status code: 410")
     end
 
+    it "raises an ApiClient::Errors::PreconditionFailed if status is 412" do
+      @response.env[:status] = 412
+      lambda {
+        @instance.send :handle_response, request, @response
+      }.should raise_error(ApiClient::Errors::PreconditionFailed, "Status code: 412")
+    end
+
     it "raises an ApiClient::Errors::Unsupported if status is 422" do
       @response.env[:status] = 422
       lambda {