MAGETWO-63910: Create infrastructure for mapper in Payment module

YevSent · YevSent · commit 5227ef25e28c · 2017-02-14T15:26:14.000+02:00
- Refactored code according to code review changes.
diff --git a/app/code/Magento/Braintree/Model/AvsEmsCodeMapper.php b/app/code/Magento/Braintree/Model/AvsEmsCodeMapper.php
@@ -43,11 +43,10 @@ class AvsEmsCodeMapper implements PaymentVerificationInterface
 
     /**
      * Gets payment AVS verification code.
-     * Throws an exception if specified order payment has different payment method code.
      *
      * @param OrderPaymentInterface $orderPayment
      * @return string
-     * @throws \InvalidArgumentException
+     * @throws \InvalidArgumentException If specified order payment has different payment method code.
      */
     public function getCode(OrderPaymentInterface $orderPayment)
     {
diff --git a/app/code/Magento/Braintree/Model/CvvEmsCodeMapper.php b/app/code/Magento/Braintree/Model/CvvEmsCodeMapper.php
@@ -42,11 +42,10 @@ class CvvEmsCodeMapper implements PaymentVerificationInterface
 
     /**
      * Gets payment CVV verification code.
-     * Throws an exception if specified order payment has different payment method code.
      *
      * @param OrderPaymentInterface $orderPayment
      * @return string
-     * @throws \Exception
+     * @throws \InvalidArgumentException If specified order payment has different payment method code.
      */
     public function getCode(OrderPaymentInterface $orderPayment)
     {

Original file line number	Diff line number	Diff line change
`@@ -43,11 +43,10 @@ class AvsEmsCodeMapper implements PaymentVerificationInterface`
`43`	`43`
`44`	`44`	`/**`
`45`	`45`	`* Gets payment AVS verification code.`
`46`		`- * Throws an exception if specified order payment has different payment method code.`
`47`	`46`	`*`
`48`	`47`	`* @param OrderPaymentInterface $orderPayment`
`49`	`48`	`* @return string`
`50`		`- * @throws \InvalidArgumentException`
	`49`	`+ * @throws \InvalidArgumentException If specified order payment has different payment method code.`
`51`	`50`	`*/`
`52`	`51`	`public function getCode(OrderPaymentInterface $orderPayment)`
`53`	`52`	`{`
Original file line number	Diff line number	Diff line change
`@@ -42,11 +42,10 @@ class CvvEmsCodeMapper implements PaymentVerificationInterface`
`42`	`42`
`43`	`43`	`/**`
`44`	`44`	`* Gets payment CVV verification code.`
`45`		`- * Throws an exception if specified order payment has different payment method code.`
`46`	`45`	`*`
`47`	`46`	`* @param OrderPaymentInterface $orderPayment`
`48`	`47`	`* @return string`
`49`		`- * @throws \Exception`
	`48`	`+ * @throws \InvalidArgumentException If specified order payment has different payment method code.`
`50`	`49`	`*/`
`51`	`50`	`public function getCode(OrderPaymentInterface $orderPayment)`
`52`	`51`	`{`