ENGCOM-4873: GraphQL-641: Schema Inconsistency of "cartItems" declaration #642

naydav · web-flow · commit 53e8c50d1164 · 2019-05-01T15:17:09.000-05:00
diff --git a/app/code/Magento/ConfigurableProductGraphQl/etc/schema.graphqls b/app/code/Magento/ConfigurableProductGraphQl/etc/schema.graphqls
@@ -41,7 +41,7 @@ type ConfigurableProductOptionsValues @doc(description: "ConfigurableProductOpti
 
 input AddConfigurableProductsToCartInput {
     cart_id: String!
-    cartItems: [ConfigurableProductCartItemInput!]!
+    cart_items: [ConfigurableProductCartItemInput!]!
 }
 
 type AddConfigurableProductsToCartOutput {
diff --git a/app/code/Magento/QuoteGraphQl/Model/Resolver/AddSimpleProductsToCart.php b/app/code/Magento/QuoteGraphQl/Model/Resolver/AddSimpleProductsToCart.php
@@ -52,12 +52,12 @@ public function resolve(Field $field, $context, ResolveInfo $info, array $value
         }
         $maskedCartId = $args['input']['cart_id'];
 
-        if (!isset($args['input']['cartItems']) || empty($args['input']['cartItems'])
-            || !is_array($args['input']['cartItems'])
+        if (!isset($args['input']['cart_items']) || empty($args['input']['cart_items'])
+            || !is_array($args['input']['cart_items'])
         ) {
-            throw new GraphQlInputException(__('Required parameter "cartItems" is missing'));
+            throw new GraphQlInputException(__('Required parameter "cart_items" is missing'));
         }
-        $cartItems = $args['input']['cartItems'];
+        $cartItems = $args['input']['cart_items'];
 
         $cart = $this->getCartForUser->execute($maskedCartId, $context->getUserId());
         $this->addProductsToCart->execute($cart, $cartItems);
diff --git a/app/code/Magento/QuoteGraphQl/etc/schema.graphqls b/app/code/Magento/QuoteGraphQl/etc/schema.graphqls
@@ -27,7 +27,7 @@ input createEmptyCartInput {
 
 input AddSimpleProductsToCartInput {
     cart_id: String!
-    cartItems: [SimpleProductCartItemInput!]!
+    cart_items: [SimpleProductCartItemInput!]!
 }
 
 input SimpleProductCartItemInput {
@@ -37,7 +37,7 @@ input SimpleProductCartItemInput {
 
 input AddVirtualProductsToCartInput {
     cart_id: String!
-    cartItems: [VirtualProductCartItemInput!]!
+    cart_items: [VirtualProductCartItemInput!]!
 }
 
 input VirtualProductCartItemInput {
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/CatalogInventory/AddProductToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/CatalogInventory/AddProductToCartTest.php
@@ -112,7 +112,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity) :
   addSimpleProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}", 
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: $quantity
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/ConfigurableProduct/AddConfigurableProductToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/ConfigurableProduct/AddConfigurableProductToCartTest.php
@@ -93,7 +93,7 @@ private function getQuery(string $maskedQuoteId, string $variantSku, int $quanti
   addConfigurableProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}"
-      cartItems: [
+      cart_items: [
         {
           variant_sku: "{$variantSku}"
           data: {
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/PageCache/Quote/Guest/CartCacheTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/PageCache/Quote/Guest/CartCacheTest.php
@@ -79,7 +79,7 @@ private function addSimpleProductToCart(string $maskedCartId, int $qty, string $
         addSimpleProductsToCart(
           input: {
             cart_id: "{$maskedCartId}"
-            cartItems: [
+            cart_items: [
               {
                 data: {
                   qty: $qty
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/AddSimpleProductWithCustomOptionsToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/AddSimpleProductWithCustomOptionsToCartTest.php
@@ -59,7 +59,7 @@ public function testAddSimpleProductWithOptions()
   addSimpleProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}", 
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: $quantity
@@ -118,7 +118,7 @@ public function testAddSimpleProductWithNoRequiredOptionsSet()
   addSimpleProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}", 
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: $quantity
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/AddVirtualProductWithCustomOptionsToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/AddVirtualProductWithCustomOptionsToCartTest.php
@@ -59,7 +59,7 @@ public function testAddVirtualProductWithOptions()
   addVirtualProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}", 
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: $quantity
@@ -118,7 +118,7 @@ public function testAddVirtualProductWithNoRequiredOptionsSet()
   addVirtualProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}", 
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: $quantity
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Customer/AddSimpleProductToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Customer/AddSimpleProductToCartTest.php
@@ -139,7 +139,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity):
 mutation {
   addSimpleProductsToCart(input: {
     cart_id: "{$maskedQuoteId}", 
-    cartItems: [
+    cart_items: [
       {
         data: {
           quantity: {$quantity}
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Customer/AddVirtualProductToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Customer/AddVirtualProductToCartTest.php
@@ -139,7 +139,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity):
 mutation {
   addVirtualProductsToCart(input: {
     cart_id: "{$maskedQuoteId}", 
-    cartItems: [
+    cart_items: [
       {
         data: {
           quantity: {$quantity}
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Customer/CheckoutEndToEndTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Customer/CheckoutEndToEndTest.php
@@ -212,7 +212,7 @@ private function addProductToCart(string $cartId, float $qty, string $sku): void
   addSimpleProductsToCart(
     input: {
       cart_id: "{$cartId}"
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: {$qty}
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/AddSimpleProductToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/AddSimpleProductToCartTest.php
@@ -112,7 +112,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity):
   addSimpleProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}"
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: $quantity
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/AddVirtualProductToCartTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/AddVirtualProductToCartTest.php
@@ -113,7 +113,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity):
   addVirtualProductsToCart(
     input: {
       cart_id: "{$maskedQuoteId}"
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: {$quantity}
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/CheckoutEndToEndTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/CheckoutEndToEndTest.php
@@ -172,7 +172,7 @@ private function addProductToCart(string $cartId, float $quantity, string $sku):
   addSimpleProductsToCart(
     input: {
       cart_id: "{$cartId}"
-      cartItems: [
+      cart_items: [
         {
           data: {
             quantity: {$quantity}

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ type ConfigurableProductOptionsValues @doc(description: "ConfigurableProductOpti`
`41`	`41`
`42`	`42`	`input AddConfigurableProductsToCartInput {`
`43`	`43`	`cart_id: String!`
`44`		`- cartItems: [ConfigurableProductCartItemInput!]!`
	`44`	`+ cart_items: [ConfigurableProductCartItemInput!]!`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`type AddConfigurableProductsToCartOutput {`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ input createEmptyCartInput {`
`27`	`27`
`28`	`28`	`input AddSimpleProductsToCartInput {`
`29`	`29`	`cart_id: String!`
`30`		`- cartItems: [SimpleProductCartItemInput!]!`
	`30`	`+ cart_items: [SimpleProductCartItemInput!]!`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`input SimpleProductCartItemInput {`
`@@ -37,7 +37,7 @@ input SimpleProductCartItemInput {`
`37`	`37`
`38`	`38`	`input AddVirtualProductsToCartInput {`
`39`	`39`	`cart_id: String!`
`40`		`- cartItems: [VirtualProductCartItemInput!]!`
	`40`	`+ cart_items: [VirtualProductCartItemInput!]!`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`input VirtualProductCartItemInput {`
Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity) :`
`112`	`112`	`addSimpleProductsToCart(`
`113`	`113`	`input: {`
`114`	`114`	`cart_id: "{$maskedQuoteId}",`
`115`		`- cartItems: [`
	`115`	`+ cart_items: [`
`116`	`116`	`{`
`117`	`117`	`data: {`
`118`	`118`	`quantity: $quantity`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ private function getQuery(string $maskedQuoteId, string $variantSku, int $quanti`
`93`	`93`	`addConfigurableProductsToCart(`
`94`	`94`	`input: {`
`95`	`95`	`cart_id: "{$maskedQuoteId}"`
`96`		`- cartItems: [`
	`96`	`+ cart_items: [`
`97`	`97`	`{`
`98`	`98`	`variant_sku: "{$variantSku}"`
`99`	`99`	`data: {`
Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ private function addSimpleProductToCart(string $maskedCartId, int $qty, string $`
`79`	`79`	`addSimpleProductsToCart(`
`80`	`80`	`input: {`
`81`	`81`	`cart_id: "{$maskedCartId}"`
`82`		`- cartItems: [`
	`82`	`+ cart_items: [`
`83`	`83`	`{`
`84`	`84`	`data: {`
`85`	`85`	`qty: $qty`
Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity):`
`139`	`139`	`mutation {`
`140`	`140`	`addSimpleProductsToCart(input: {`
`141`	`141`	`cart_id: "{$maskedQuoteId}",`
`142`		`- cartItems: [`
	`142`	`+ cart_items: [`
`143`	`143`	`{`
`144`	`144`	`data: {`
`145`	`145`	`quantity: {$quantity}`
Original file line number	Diff line number	Diff line change
`@@ -212,7 +212,7 @@ private function addProductToCart(string $cartId, float $qty, string $sku): void`
`212`	`212`	`addSimpleProductsToCart(`
`213`	`213`	`input: {`
`214`	`214`	`cart_id: "{$cartId}"`
`215`		`- cartItems: [`
	`215`	`+ cart_items: [`
`216`	`216`	`{`
`217`	`217`	`data: {`
`218`	`218`	`quantity: {$qty}`
Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ private function getQuery(string $maskedQuoteId, string $sku, float $quantity):`
`113`	`113`	`addVirtualProductsToCart(`
`114`	`114`	`input: {`
`115`	`115`	`cart_id: "{$maskedQuoteId}"`
`116`		`- cartItems: [`
	`116`	`+ cart_items: [`
`117`	`117`	`{`
`118`	`118`	`data: {`
`119`	`119`	`quantity: {$quantity}`
Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ private function addProductToCart(string $cartId, float $quantity, string $sku):`
`172`	`172`	`addSimpleProductsToCart(`
`173`	`173`	`input: {`
`174`	`174`	`cart_id: "{$cartId}"`
`175`		`- cartItems: [`
	`175`	`+ cart_items: [`
`176`	`176`	`{`
`177`	`177`	`data: {`
`178`	`178`	`quantity: {$quantity}`