Throw Api\Exception when client returns a non 200 response

Author: chadicus

src/Collection.php

@@ -156,7 +156,12 @@ public function next()
         $indexResponse = $this->client->index($this->resource, $this->filters);
 
         $httpCode = $indexResponse->getHttpCode();
-        Util::ensure(200, $httpCode, "Did not receive 200 from API. Instead received {$httpCode}");
+        Util::ensure(
+            200,
+            $httpCode,
+            Exception::class,
+            ["Did not receive 200 from API. Instead received {$httpCode}", $indexResponse]
+        );
 
         $response = $indexResponse->getResponse();
         $this->limit = $response['pagination']['limit'];

tests/CollectionTest.php

@@ -300,6 +300,44 @@ public function selectMissingKeys()
         );
     }
 
+    /**
+     * @test
+     * @covers ::next
+     */
+    public function collectionThrowsApiExceptionWith400Response()
+    {
+        $response = new Response(400, ['Content-Type' => ['application/json']], []);
+        $mockClient = $this->getMockBuilder(ClientInterface::class)->getMock();
+        $mockClient->method('index')->willReturn($response);
+        $collection = new Collection($mockClient, 'basic', []);
+        try {
+            $collection->next();
+            $this->fail('No Exception throw');
+        } catch (Exception $e) {
+            $this->assertSame('Did not receive 200 from API. Instead received 400', $e->getMessage());
+            $this->assertSame($response, $e->getResponse());
+        }
+    }
+
+    /**
+     * @test
+     * @covers ::next
+     */
+    public function collectionThrowsApiExceptionWith500Response()
+    {
+        $response = new Response(500, ['Content-Type' => ['application/json']], []);
+        $mockClient = $this->getMockBuilder(ClientInterface::class)->getMock();
+        $mockClient->method('index')->willReturn($response);
+        $collection = new Collection($mockClient, 'basic', []);
+        try {
+            $collection->next();
+            $this->fail('No Exception throw');
+        } catch (Exception $e) {
+            $this->assertSame('Did not receive 200 from API. Instead received 500', $e->getMessage());
+            $this->assertSame($response, $e->getResponse());
+        }
+    }
+
     private function getClient(array $items = self::DEFAULT_RESULT_SET) : ClientInterface
     {
         $callback = function (string $resource, array $filters) use ($items) {