Skip to content

Commit

Permalink
MAGETWO-57426: [Github] Braintree Vault payments causing GET order AP…
Browse files Browse the repository at this point in the history
…I to throw error for 2.1.x

- phpmd fixes
  • Loading branch information
Dmytro Yushkin committed Sep 9, 2016
1 parent cd29741 commit cbb2688
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
1 change: 1 addition & 0 deletions app/code/Magento/Vault/Model/Method/Vault.php
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,7 @@ final class Vault implements VaultPaymentInterface
* @param PaymentTokenManagementInterface $tokenManagement
* @param OrderPaymentExtensionInterfaceFactory $paymentExtensionFactory
* @param string $code
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
*/
public function __construct(
ConfigInterface $config,
Expand Down
4 changes: 4 additions & 0 deletions app/code/Magento/Vault/Test/Unit/Model/Method/VaultTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,10 @@
use Magento\Vault\Model\Method\Vault;
use Magento\Vault\Model\VaultPaymentInterface;

/**
* Class VaultTest
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
*/
class VaultTest extends \PHPUnit_Framework_TestCase
{
/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,8 @@ public function testOrderList()
// check what additional information is single dimension array
static::assertEquals(
count($item['payment']['additional_information']),
count($item['payment']['additional_information']), COUNT_RECURSIVE
count($item['payment']['additional_information']),
COUNT_RECURSIVE
);
}
}

0 comments on commit cbb2688

Please sign in to comment.