Fixes #42 - Call to undefined method GuzzleHttp\Exception\ConnectException::getResponse(). (#51)

* Fixes: #42 - Call to undefined method GuzzleHttp\Exception\ConnectException::getResponse().

Author: mgruner

src/HTTPClient.php

@@ -181,7 +181,10 @@ public function request(string $method, $uri = '', array $options = [] ): Respon
             $response = parent::request( $method, $uri, $options );
         }
         catch ( \GuzzleHttp\Exception\TransferException $e ) {
-            $response = $e->getResponse();
+            if (method_exists($e, 'hasResponse') && $e->hasResponse()) {
+                return $e->getResponse();
+            }
+            throw $e;
         }
 
         return $response;

test/ZammadAPIClient/ClientTest.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace ZammadAPIClient;
+
+use PHPUnit\Framework\TestCase;
+
+use ZammadAPIClient\Client;
+use GuzzleHttp\Exception\ConnectException;
+
+class ClientTest extends TestCase
+{
+    public function testNetworkError()
+    {
+        // When providing a wrong URL, there must be a proper exception thrown.
+        $this->expectException( \GuzzleHttp\Exception\ConnectException::class );
+
+        $client = new Client([
+            'url'      => 'https://non.existing.ci/',
+            'username' => 'nonexisting',
+            'password' => 'nonexisting',
+        ]);
+        $client->get('/nonexisting');
+    }
+}