Switched ApiException to use title for exception message only if detail does not exist

Author: rbibby

src/Exception/ApiException.php

@@ -24,7 +24,12 @@ public function __construct($response)
         }
 
         if (!empty($this->errors)) {
-            $this->message = $this->errors[0]->title;
+            $message = $this->errors[0]->detail;
+            if (empty($message)) {
+                $message = $this->errors[0]->title;
+            }
+
+            $this->message = $message;
         }
 
         $this->response = $response;

tests/ClientTest.php

@@ -257,7 +257,7 @@ public function wraps_client_exceptions_as_ukfast_exceptions()
             $client->request('GET', '/');
         } catch (ApiException $e) {
             $this->assertEquals(1, count($e->getErrors()));
-            $this->assertEquals('Testing errors', $e->getMessage());
+            $this->assertEquals('Testing errors detail', $e->getMessage());
             return;
         }
 
@@ -293,6 +293,33 @@ public function wraps_server_exceptions_as_ukfast_exceptions()
         $this->expectException(ApiException::class);
     }
 
+    /**
+     * @test
+     */
+    public function defaults_exception_message_to_title_if_no_detail_is_set()
+    {
+        $mock = new MockHandler([
+            new Response(500, [], json_encode([
+                'errors' => [[
+                    'title' => 'Testing errors title',
+                    'status' => 500,
+                ]]
+            ])),
+        ]);
+        $handler = HandlerStack::create($mock);
+        $guzzle = new Guzzle(['handler' => $handler]);
+        $client = new Client($guzzle);
+
+        try {
+            $client->request('GET', '/');
+        } catch (ApiException $e) {
+            $this->assertEquals('Testing errors title', $e->getMessage());
+            return;
+        }
+
+        $this->expectException(ApiException::class);
+    }
+
     /**
      * @test
      */