magento/graphql-ce#985: Remove unnecessary exceptions

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

magento/graphql-ce#985: Remove unnecessary exceptions #995

Closed atwixfirster closed 4 years ago

atwixfirster commented 4 years ago

Description (*)

This PR removes unnecessary exceptions in Magento\CustomerGraphQl\Model\Customer\CheckCustomerPassword.

Fixed Issues (if relevant)

https://github.com/magento/graphql-ce/issues/985: [Test coverage] Cover exceptions in CustomerGraphQl\Model\Customer\CheckCustomerPassword

Manual testing scenarios (*)

Questions or comments

Contribution checklist (*)

[ ] Pull request has a meaningful description of its purpose
[ ] All commits are accompanied by meaningful commit messages
[ ] All new or changed code is covered with unit/integration tests (if applicable)
[ ] All automated tests passed successfully (all builds are green)

magento-engcom-team commented 4 years ago

Hi @lenaorobei, thank you for the review. ENGCOM-6054 has been created to process this Pull Request :eight_spoked_asterisk: @lenaorobei, could you please add one of the following labels to the Pull Request?

Label	Description
`Auto-Tests: Covered`	All changes in Pull Request is covered by auto-tests
`Auto-Tests: Not Covered`	Changes in Pull Request requires coverage by auto-tests
`Auto-Tests: Not Required`	Changes in Pull Request does not require coverage by auto-tests

ghost commented 4 years ago

Hi @atwixfirster, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.