Update tests for later version error messaging

pmclain · pmclain · commit fe9d262f5998 · 2019-08-21T20:59:26.000-04:00
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Customer/CreateCustomerAddressTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Customer/CreateCustomerAddressTest.php
@@ -303,8 +303,8 @@ public function invalidInputDataProvider()
     {
         return [
             ['', 'Syntax Error: Expected Name, found )'],
-            ['input: ""', 'Expected type CustomerAddressInput!, found "".'],
-            ['input: "foo"', 'Expected type CustomerAddressInput!, found "foo".']
+            ['input: ""', 'requires type CustomerAddressInput!, found "".'],
+            ['input: "foo"', 'requires type CustomerAddressInput!, found "foo".']
         ];
     }
 
diff --git a/dev/tests/api-functional/testsuite/Magento/GraphQl/Customer/UpdateCustomerAddressTest.php b/dev/tests/api-functional/testsuite/Magento/GraphQl/Customer/UpdateCustomerAddressTest.php
@@ -306,8 +306,8 @@ public function invalidInputDataProvider()
     {
         return [
             ['', '"input" value must be specified'],
-            ['input: ""', 'Expected type CustomerAddressInput, found ""'],
-            ['input: "foo"', 'Expected type CustomerAddressInput, found "foo"']
+            ['input: ""', 'requires type CustomerAddressInput, found ""'],
+            ['input: "foo"', 'requires type CustomerAddressInput, found "foo"']
         ];
     }
 
diff --git a/dev/tests/integration/testsuite/Magento/GraphQl/Controller/GraphQlControllerTest.php b/dev/tests/integration/testsuite/Magento/GraphQl/Controller/GraphQlControllerTest.php
@@ -251,7 +251,7 @@ public function testError() : void
                 foreach ($outputResponse['errors'] as $error) {
                     $this->assertEquals(
                         \Magento\Framework\GraphQl\Exception\GraphQlInputException::EXCEPTION_CATEGORY,
-                        $error['category']
+                        $error['extensions']['category']
                     );
                     if (isset($error['message'])) {
                         $this->assertEquals($error['message'], 'Invalid entity_type specified: invalid');
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PaypalExpressTokenTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PaypalExpressTokenTest.php
@@ -136,7 +136,7 @@ public function testResolveWithPaypalError($paymentMethod): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 
     /**
@@ -173,7 +173,7 @@ public function testResolveWithInvalidRedirectUrl($paymentMethod): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 
     /**
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PaypalPayflowProTokenExceptionTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PaypalPayflowProTokenExceptionTest.php
@@ -73,6 +73,6 @@ public function testResolveWithPaypalError(): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 }
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PaypalPayflowProTokenTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PaypalPayflowProTokenTest.php
@@ -130,6 +130,6 @@ public function testResolveWithInvalidRedirectUrl(): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 }
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PlaceOrderWithHostedProTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PlaceOrderWithHostedProTest.php
@@ -206,7 +206,7 @@ public function testOrderWithHostedProDeclined(): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 
     /**
@@ -258,6 +258,6 @@ public function testSetPaymentMethodInvalidUrls()
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 }
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PlaceOrderWithPayflowLinkTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PlaceOrderWithPayflowLinkTest.php
@@ -273,6 +273,6 @@ public function testResolveWithPayflowLinkDeclined(): void
             $expectedExceptionMessage,
             $actualError['message']
         );
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 }
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PlaceOrderWithPaymentsAdvancedTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/PlaceOrderWithPaymentsAdvancedTest.php
@@ -178,7 +178,7 @@ public function testResolvePaymentsAdvancedWithInvalidUrl(): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 
     /**
@@ -230,7 +230,7 @@ public function testResolveWithPaymentAdvancedDeclined(): void
         $this->assertArrayHasKey('errors', $responseData);
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 
     /**
diff --git a/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/SetPaymentMethodAsPayflowLinkTest.php b/dev/tests/integration/testsuite/Magento/PaypalGraphQl/Model/Resolver/Guest/SetPaymentMethodAsPayflowLinkTest.php
@@ -168,6 +168,6 @@ public function testInvalidUrl(): void
         $expectedExceptionMessage = "Invalid Url.";
         $actualError = $responseData['errors'][0];
         $this->assertEquals($expectedExceptionMessage, $actualError['message']);
-        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);
+        $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -303,8 +303,8 @@ public function invalidInputDataProvider()`
`303`	`303`	`{`
`304`	`304`	`return [`
`305`	`305`	`['', 'Syntax Error: Expected Name, found )'],`
`306`		`- ['input: ""', 'Expected type CustomerAddressInput!, found "".'],`
`307`		`- ['input: "foo"', 'Expected type CustomerAddressInput!, found "foo".']`
	`306`	`+ ['input: ""', 'requires type CustomerAddressInput!, found "".'],`
	`307`	`+ ['input: "foo"', 'requires type CustomerAddressInput!, found "foo".']`
`308`	`308`	`];`
`309`	`309`	`}`
`310`	`310`
Original file line number	Diff line number	Diff line change
`@@ -306,8 +306,8 @@ public function invalidInputDataProvider()`
`306`	`306`	`{`
`307`	`307`	`return [`
`308`	`308`	`['', '"input" value must be specified'],`
`309`		`- ['input: ""', 'Expected type CustomerAddressInput, found ""'],`
`310`		`- ['input: "foo"', 'Expected type CustomerAddressInput, found "foo"']`
	`309`	`+ ['input: ""', 'requires type CustomerAddressInput, found ""'],`
	`310`	`+ ['input: "foo"', 'requires type CustomerAddressInput, found "foo"']`
`311`	`311`	`];`
`312`	`312`	`}`
`313`	`313`
Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,6 @@ public function testResolveWithPaypalError(): void`
`73`	`73`	`$this->assertArrayHasKey('errors', $responseData);`
`74`	`74`	`$actualError = $responseData['errors'][0];`
`75`	`75`	`$this->assertEquals($expectedExceptionMessage, $actualError['message']);`
`76`		`- $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);`
	`76`	`+ $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);`
`77`	`77`	`}`
`78`	`78`	`}`
Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,6 @@ public function testResolveWithInvalidRedirectUrl(): void`
`130`	`130`	`$this->assertArrayHasKey('errors', $responseData);`
`131`	`131`	`$actualError = $responseData['errors'][0];`
`132`	`132`	`$this->assertEquals($expectedExceptionMessage, $actualError['message']);`
`133`		`- $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);`
	`133`	`+ $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);`
`134`	`134`	`}`
`135`	`135`	`}`
Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,7 @@ public function testOrderWithHostedProDeclined(): void`
`206`	`206`	`$this->assertArrayHasKey('errors', $responseData);`
`207`	`207`	`$actualError = $responseData['errors'][0];`
`208`	`208`	`$this->assertEquals($expectedExceptionMessage, $actualError['message']);`
`209`		`- $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);`
	`209`	`+ $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);`
`210`	`210`	`}`
`211`	`211`
`212`	`212`	`/**`
`@@ -258,6 +258,6 @@ public function testSetPaymentMethodInvalidUrls()`
`258`	`258`	`$this->assertArrayHasKey('errors', $responseData);`
`259`	`259`	`$actualError = $responseData['errors'][0];`
`260`	`260`	`$this->assertEquals($expectedExceptionMessage, $actualError['message']);`
`261`		`- $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);`
	`261`	`+ $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);`
`262`	`262`	`}`
`263`	`263`	`}`
Original file line number	Diff line number	Diff line change
`@@ -273,6 +273,6 @@ public function testResolveWithPayflowLinkDeclined(): void`
`273`	`273`	`$expectedExceptionMessage,`
`274`	`274`	`$actualError['message']`
`275`	`275`	`);`
`276`		`- $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);`
	`276`	`+ $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);`
`277`	`277`	`}`
`278`	`278`	`}`
Original file line number	Diff line number	Diff line change
`@@ -168,6 +168,6 @@ public function testInvalidUrl(): void`
`168`	`168`	`$expectedExceptionMessage = "Invalid Url.";`
`169`	`169`	`$actualError = $responseData['errors'][0];`
`170`	`170`	`$this->assertEquals($expectedExceptionMessage, $actualError['message']);`
`171`		`- $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['category']);`
	`171`	`+ $this->assertEquals(GraphQlInputException::EXCEPTION_CATEGORY, $actualError['extensions']['category']);`
`172`	`172`	`}`
`173`	`173`	`}`