Add specific error class for 503s

I've also cleaned up the use of `last_response` versus `@last_response`.

Author: bittersweet

lib/help_scout.rb

@@ -8,6 +8,7 @@ class NotFoundError < StandardError; end
   class TooManyRequestsError < StandardError; end
   class InternalServerError < StandardError; end
   class ForbiddenError < StandardError; end
+  class ServiceUnavailable < StandardError; end
 
   # Status codes used by Help Scout, not all are implemented in this gem yet.
   # http://developer.helpscout.net/help-desk-api/status-codes/
@@ -19,6 +20,7 @@ class ForbiddenError < StandardError; end
   HTTP_NOT_FOUND = 404
   HTTP_TOO_MANY_REQUESTS = 429
   HTTP_INTERNAL_SERVER_ERROR = 500
+  HTTP_SERVICE_UNAVAILABLE = 503
 
   attr_accessor :last_response
 
@@ -229,10 +231,9 @@ def request(method, path, options)
     }.merge(options)
 
     @last_response = HTTParty.send(method, uri, options)
-
-    case @last_response.code
+    case last_response.code
     when HTTP_OK, HTTP_CREATED, HTTP_NO_CONTENT
-      @last_response.parsed_response
+      last_response.parsed_response
     when HTTP_BAD_REQUEST
       raise ValidationError, last_response.parsed_response["validationErrors"]
     when HTTP_FORBIDDEN
@@ -241,12 +242,15 @@ def request(method, path, options)
       raise NotFoundError
     when HTTP_INTERNAL_SERVER_ERROR
       raise InternalServerError
+    when HTTP_SERVICE_UNAVAILABLE
+      raise ServiceUnavailable
     when HTTP_TOO_MANY_REQUESTS
       retry_after = last_response.headers["Retry-After"]
-      error_message = "Rate limit of 200 RPM or 12 POST/PUT/DELETE requests per 5 seconds reached. Next request possible in #{retry_after} seconds."
-      raise TooManyRequestsError, error_message
+      message = "Rate limit of 200 RPM or 12 POST/PUT/DELETE requests per 5 " +
+        "seconds reached. Next request possible in #{retry_after} seconds."
+      raise TooManyRequestsError, message
     else
-      raise NotImplementedError, "Help Scout returned something that is not implemented by the help_scout gem yet: #{@last_response.code}: #{@last_response.parsed_response["message"] if @last_response.parsed_response}"
+      raise NotImplementedError, "Help Scout returned something that is not implemented by the help_scout gem yet: #{last_response.code}: #{last_response.parsed_response["message"] if last_response.parsed_response}"
     end
   end
 end

spec/helpscout_spec.rb

@@ -73,14 +73,23 @@
       end
     end
 
+    context 'with a 503 status code' do
+      it 'returns ServiceUnavailable' do
+        url = 'https://api.helpscout.net/v1/conversations/1337.json'
+        stub_request(:get, url).to_return(status: 503)
+
+        expect { client.get_conversation(1337) }.to raise_error(HelpScout::ServiceUnavailable)
+      end
+    end
+
     context 'with a not implemented status code' do
       it 'returns a not implemented error' do
         data = { subject: "Help me!" }
 
         url = 'https://api.helpscout.net/v1/conversations.json'
         stub_request(:post, url).
           to_return(
-            status: 503,
+            status: 402,
           )
 
         expect { client.create_conversation(data) }.to raise_error(HelpScout::NotImplementedError)