fix(client): Handle Guzzle ClientException (#324)

ankitpokhrel · Nov 27, 2020 · 0c548f0 · 0c548f0
1 parent 126a185
commit 0c548f0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/src/Tus/Client.php b/src/Tus/Client.php
@@ -437,9 +437,13 @@ public function create(string $key) : string
             $headers += ['Upload-Concat' => 'partial'];
         }
 
-        $response = $this->getClient()->post($this->apiPath, [
-            'headers' => $headers,
-        ]);
+        try {
+            $response = $this->getClient()->post($this->apiPath, [
+                'headers' => $headers,
+            ]);
+        } catch (ClientException $e) {
+            $response = $e->getResponse();
+        }
 
         $statusCode = $response->getStatusCode();
 

diff --git a/tests/Tus/ClientTest.php b/tests/Tus/ClientTest.php
@@ -1367,6 +1367,12 @@ public function it_throws_exception_when_unable_to_create_resource() : void
 
         $this->tusClientMock->file($filePath, $fileName);
 
+        $clientExceptionMock = m::mock(ClientException::class);
+        $clientExceptionMock
+            ->shouldReceive('getResponse')
+            ->once()
+            ->andReturn($responseMock);
+
         $guzzleMock
             ->shouldReceive('post')
             ->once()
@@ -1378,7 +1384,7 @@ public function it_throws_exception_when_unable_to_create_resource() : void
                     'Upload-Metadata' => 'filename ' . base64_encode($fileName),
                 ],
             ])
-            ->andReturn($responseMock);
+            ->andThrow($clientExceptionMock);
 
         $this->tusClientMock->create($key);
     }