fix(ShippingMethod): fix response object class of shipping method req…

…uest by cart or location
commercetools · Oct 26, 2016 · 76b18bc · 76b18bc
1 parent 178596b
commit 76b18bc
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 4 deletions.
diff --git a/src/Request/ShippingMethods/ShippingMethodByCartIdGetRequest.php b/src/Request/ShippingMethods/ShippingMethodByCartIdGetRequest.php
@@ -5,6 +5,7 @@
 
 namespace Commercetools\Core\Request\ShippingMethods;
 
+use Commercetools\Core\Response\ResourceResponse;
 use Psr\Http\Message\ResponseInterface;
 use Commercetools\Core\Client\HttpMethod;
 use Commercetools\Core\Client\HttpRequest;
@@ -74,6 +75,6 @@ public function httpRequest()
      */
     public function buildResponse(ResponseInterface $response)
     {
-        return new PagedQueryResponse($response, $this, $this->getContext());
+        return new ResourceResponse($response, $this, $this->getContext());
     }
 }
diff --git a/src/Request/ShippingMethods/ShippingMethodByLocationGetRequest.php b/src/Request/ShippingMethods/ShippingMethodByLocationGetRequest.php
@@ -5,6 +5,7 @@
 
 namespace Commercetools\Core\Request\ShippingMethods;
 
+use Commercetools\Core\Response\ResourceResponse;
 use Psr\Http\Message\ResponseInterface;
 use Commercetools\Core\Client\HttpMethod;
 use Commercetools\Core\Client\HttpRequest;
@@ -102,6 +103,6 @@ public function httpRequest()
      */
     public function buildResponse(ResponseInterface $response)
     {
-        return new PagedQueryResponse($response, $this, $this->getContext());
+        return new ResourceResponse($response, $this, $this->getContext());
     }
 }
diff --git a/tests/integration/ShippingMethod/ShippingMethodQueryRequestTest.php b/tests/integration/ShippingMethod/ShippingMethodQueryRequestTest.php
@@ -14,6 +14,7 @@
 use Commercetools\Core\Model\ShippingMethod\ZoneRate;
 use Commercetools\Core\Model\ShippingMethod\ZoneRateCollection;
 use Commercetools\Core\Request\ShippingMethods\ShippingMethodByIdGetRequest;
+use Commercetools\Core\Request\ShippingMethods\ShippingMethodByLocationGetRequest;
 use Commercetools\Core\Request\ShippingMethods\ShippingMethodCreateRequest;
 use Commercetools\Core\Request\ShippingMethods\ShippingMethodDeleteRequest;
 use Commercetools\Core\Request\ShippingMethods\ShippingMethodQueryRequest;
@@ -81,6 +82,18 @@ public function testGetById()
 
         $this->assertInstanceOf('\Commercetools\Core\Model\ShippingMethod\ShippingMethod', $shippingMethod);
         $this->assertSame($shippingMethod->getId(), $result->getId());
+    }
+
+    public function testByLocation()
+    {
+        $draft = $this->getDraft();
+        $shippingMethod = $this->createShippingMethod($draft);
+
+        $request = ShippingMethodByLocationGetRequest::ofCountry('DE')->withState($this->getRegion());
+        $response = $request->executeWithClient($this->getClient());
+        $result = $request->mapResponse($response);
 
+        $this->assertInstanceOf('\Commercetools\Core\Model\ShippingMethod\ShippingMethodCollection', $result);
+        $this->assertSame($shippingMethod->getId(), $result->current()->getId());
     }
 }
diff --git a/tests/unit/Request/ShippingMethods/ShippingMethodByCartIdGetRequestTest.php b/tests/unit/Request/ShippingMethods/ShippingMethodByCartIdGetRequestTest.php
@@ -58,6 +58,6 @@ public function testBuildResponse()
         $request = ShippingMethodByCartIdGetRequest::ofCartId('id');
         $response = $request->buildResponse($guzzleResponse);
 
-        $this->assertInstanceOf('\Commercetools\Core\Response\PagedQueryResponse', $response);
+        $this->assertInstanceOf('\Commercetools\Core\Response\ResourceResponse', $response);
     }
 }
diff --git a/tests/unit/Request/ShippingMethods/ShippingMethodByLocationGetRequestTest.php b/tests/unit/Request/ShippingMethods/ShippingMethodByLocationGetRequestTest.php
@@ -82,6 +82,6 @@ public function testBuildResponse()
         $request = ShippingMethodByLocationGetRequest::ofCountry('DE');
         $response = $request->buildResponse($guzzleResponse);
 
-        $this->assertInstanceOf('\Commercetools\Core\Response\PagedQueryResponse', $response);
+        $this->assertInstanceOf('\Commercetools\Core\Response\ResourceResponse', $response);
     }
 }