magento / graphql-ce

[ARCHIVED] Please use magento/magento2 project
https://github.com/magento/magento2
Open Software License 3.0
131 stars 156 forks source link

[Test Coverage] Extend coverage for CustomerGraphQL #1011

Closed TomashKhamlai closed 4 years ago

TomashKhamlai commented 4 years ago

Description:

Cover with API-functional tests: https://github.com/magento/graphql-ce/blob/f52ba88cf0ba26ae8b30eb6257a80dd2afb9958c/app/code/Magento/CustomerGraphQl/Model/Customer/ExtractCustomerData.php#L82 https://github.com/magento/graphql-ce/blob/3e03eec3c0e831c9536be75248b5c5503f481514/app/code/Magento/CustomerGraphQl/Model/Customer/Address/ExtractCustomerAddressData.php#L98 https://github.com/magento/graphql-ce/blob/da8d6ac580f3b3e47f913b7f0618e330a8693fc8/app/code/Magento/CustomerGraphQl/Model/Customer/GetAllowedCustomerAttributes.php#L84

TestCases:

Examples:

Magento\GraphQl\Customer Magento\GraphQl\Quote\Customer

Guide:

Graphql Functional Testing

magento-engcom-team commented 4 years ago

Hi @TomashKhamlai. Thank you for your report. The issue has been fixed in magento/graphql-ce#1030 by @TomashKhamlai in 2.3-develop branch Related commit(s):

The fix will be available with the upcoming 2.3.4 release.