Actually output validation errors in HelpScout::ValidationError

Author: dennispaagman

lib/help_scout.rb

@@ -234,7 +234,7 @@ def request(method, path, options)
     when HTTP_OK, HTTP_CREATED, HTTP_NO_CONTENT
       @last_response.parsed_response
     when HTTP_BAD_REQUEST
-      raise ValidationError, last_response.parsed_response["message"]
+      raise ValidationError, last_response.parsed_response["validationErrors"]
     when HTTP_FORBIDDEN
       raise ForbiddenError
     when HTTP_NOT_FOUND

spec/helpscout_spec.rb

@@ -21,7 +21,7 @@
 
     context 'with invalid input' do
       it 'returns validation errors' do
-        data = { subject: "Help me!", customer: { email: "" } }
+        data = { subject: "Help me!", customer: { email: "[email protected]" } }
 
         url = 'https://api.helpscout.net/v1/conversations.json'
         stub_request(:post, url).
@@ -30,10 +30,19 @@
             headers: {
                'Content-Type' => 'application/json'
             },
-            body: { error: "Input could not be validated", message: "Email is not valid" }.to_json
+            body: {
+              "error" => "Input could not be validated",
+              "validationErrors" => [
+                {
+                  "property" => "customer:email",
+                  "value" => "[email protected]",
+                  "message" => "Email is not valid"
+                }
+              ]
+            }.to_json
           )
 
-        expect { client.create_conversation(data) }.to raise_error(HelpScout::ValidationError, "Email is not valid")
+        expect { client.create_conversation(data) }.to raise_error(HelpScout::ValidationError, '[{"property"=>"customer:email", "value"=>"[email protected]", "message"=>"Email is not valid"}]')
       end
     end