fixes a bug, where wrong ApiResponse was returned

src/Response/Context.php

@@ -44,6 +44,15 @@ public function getResponse(): ResponseInterface
         return $this->response;
     }
 
+    /**
+     * Is successful response.
+     */
+    public function isSuccess(): bool
+    {
+        $statusCode = $this->response->getStatusCode();
+        return $statusCode == min(max($statusCode, 200), 208); // [200,208] = HTTP OK
+    }
+
     /**
      * Returns JsonHelper object.
      */

src/Response/ResponseError.php

@@ -43,7 +43,7 @@ public function throw(Context $context)
             return;
         }
         $statusCode = $context->getResponse()->getStatusCode();
-        if ($statusCode == min(max($statusCode, 200), 208)) { // [200,208] = HTTP OK
+        if ($context->isSuccess()) {
             return;
         }
         if (isset($this->errors[strval($statusCode)])) {

src/Response/ResponseHandler.php

@@ -174,6 +174,9 @@ public function getResult(Context $context)
             return null;
         }
         $this->responseError->throw($context);
+        if (!$context->isSuccess()) {
+            return $context->toApiResponse($this->getBody($context));
+        }
         $result = $this->deserializableType->getFrom($context);
         $result = $result ?? $this->responseType->getFrom($context);
         $result = $result ?? $this->responseMultiType->getFrom($context);

tests/ApiCallTest.php

@@ -792,6 +792,16 @@ public function testNullOn404()
         $this->assertNull($result);
     }
 
+    public function testApiResponseWith400()
+    {
+        $response = new MockResponse();
+        $response->setStatusCode(400);
+        $context = new Context(MockHelper::getClient()->getGlobalRequest(), $response, MockHelper::getClient());
+        $result = MockHelper::responseHandler()->type(MockClass::class)->returnApiResponse()->getResult($context);
+        $this->assertInstanceOf(MockApiResponse::class, $result);
+        $this->assertEquals(['res' => 'This is raw body'], $result->getResult());
+    }
+
     public function testGlobalMockException()
     {
         $this->expectException(MockException::class);