feat(Cart): support add line item by sky

Closes #341

Author: Jens Schulze

src/Core/Model/Cart/LineItemDraft.php

@@ -6,6 +6,7 @@
 namespace Commercetools\Core\Model\Cart;
 
 use Commercetools\Core\Model\Channel\ChannelReference;
+use Commercetools\Core\Model\Common\Context;
 use Commercetools\Core\Model\Common\JsonObject;
 use Commercetools\Core\Model\Common\LocalizedString;
 use Commercetools\Core\Model\Common\Money;
@@ -38,6 +39,8 @@
  * @method LineItemDraft setExternalPrice(Money $externalPrice = null)
  * @method ExternalLineItemTotalPrice getExternalTotalPrice()
  * @method LineItemDraft setExternalTotalPrice(ExternalLineItemTotalPrice $externalTotalPrice = null)
+ * @method string getSku()
+ * @method LineItemDraft setSku(string $sku = null)
  */
 class LineItemDraft extends JsonObject
 {
@@ -53,6 +56,29 @@ public function fieldDefinitions()
             'externalPrice' => [static::TYPE => Money::class],
             'externalTotalPrice' => [static::TYPE => ExternalLineItemTotalPrice::class],
             'custom' => [static::TYPE => CustomFieldObject::class],
+            'sku' => [static::TYPE => 'string'],
         ];
     }
+
+    /**
+     * @param string $productId
+     * @param Context|callable $context
+     * @return LineItemDraft
+     */
+    public static function ofProductId($productId, $context = null)
+    {
+        $draft = static::of($context);
+        return $draft->setProductId($productId);
+    }
+
+    /**
+     * @param string $sku
+     * @param Context|callable $context
+     * @return LineItemDraft
+     */
+    public static function ofSku($sku, $context = null)
+    {
+        $draft = static::of($context);
+        return $draft->setSku($sku);
+    }
 }

src/Core/Request/Carts/Command/CartAddLineItemAction.php

@@ -36,6 +36,8 @@
  * @method CartAddLineItemAction setExternalPrice(Money $externalPrice = null)
  * @method ExternalLineItemTotalPrice getExternalTotalPrice()
  * @method CartAddLineItemAction setExternalTotalPrice(ExternalLineItemTotalPrice $externalTotalPrice = null)
+ * @method string getSku()
+ * @method CartAddLineItemAction setSku(string $sku = null)
  */
 class CartAddLineItemAction extends AbstractAction
 {
@@ -45,6 +47,7 @@ public function fieldDefinitions()
             'action' => [static::TYPE => 'string'],
             'productId' => [static::TYPE => 'string'],
             'variantId' => [static::TYPE => 'int'],
+            'sku' => [static::TYPE => 'string'],
             'quantity' => [static::TYPE => 'int'],
             'supplyChannel' => [static::TYPE => ChannelReference::class],
             'distributionChannel' => [static::TYPE => ChannelReference::class],
@@ -67,6 +70,17 @@ public static function ofProductIdVariantIdAndQuantity($productId, $variantId, $
         return static::of($context)->setProductId($productId)->setVariantId($variantId)->setQuantity($quantity);
     }
 
+    /**
+     * @param string $sku
+     * @param Context|callable $context
+     * @param int $quantity
+     * @return CartAddLineItemAction
+     */
+    public static function ofSkuAndQuantity($sku, $quantity, $context = null)
+    {
+        return static::of($context)->setSku($sku)->setQuantity($quantity);
+    }
+
     /**
      * @param array $data
      * @param Context|callable $context

tests/integration/Cart/CartUpdateRequestTest.php

@@ -176,6 +176,29 @@ public function testLineItem()
         $this->assertCount(0, $cart->getLineItems());
     }
 
+    public function testLineItemBySku()
+    {
+        $product = $this->getProduct();
+        $variant = $product->getMasterData()->getCurrent()->getMasterVariant();
+
+        $draft = $this->getDraft();
+        $draft->setLineItems(LineItemDraftCollection::of()->add(LineItemDraft::ofSku($variant->getSku())));
+        $cart = $this->createCart($draft);
+
+        $this->assertSame(1, $cart->getLineItems()->current()->getQuantity());
+        $request = CartUpdateRequest::ofIdAndVersion($cart->getId(), $cart->getVersion())
+            ->addAction(
+                CartAddLineItemAction::ofSkuAndQuantity($variant->getSku(), 1)
+            )
+        ;
+        $response = $request->executeWithClient($this->getClient());
+        $cart = $request->mapResponse($response);
+        var_dump((string)$response->getBody());
+        $this->deleteRequest->setVersion($cart->getVersion());
+
+        $this->assertSame(2, $cart->getLineItems()->current()->getQuantity());
+    }
+
     public function testSetExternalLineItemTotalPrice()
     {
         $draft = $this->getDraft();

tests/unit/Request/GenericActionTest.php

@@ -598,7 +598,7 @@ public function actionFieldProvider()
             ],
             [
                 CartAddLineItemAction::class,
-                ['action', 'productId', 'variantId', 'quantity', 'supplyChannel', 'distributionChannel', 'custom', 'externalTaxRate', 'externalPrice', 'externalTotalPrice']
+                ['action', 'productId', 'variantId', 'quantity', 'supplyChannel', 'distributionChannel', 'custom', 'externalTaxRate', 'externalPrice', 'externalTotalPrice', 'sku']
             ],
             [
                 CartChangeLineItemQuantityAction::class,