Merge pull request #3370 from magento-tsg-csl3/2.2-develop-pr7

[TSG-CSL3] 2.2-develop (pr7)

Author: viktym

app/code/Magento/Braintree/Model/Paypal/Helper/QuoteUpdater.php

@@ -148,7 +148,7 @@ private function updateBillingAddress(Quote $quote, array $details)
     {
         $billingAddress = $quote->getBillingAddress();
 
-        if ($this->config->isRequiredBillingAddress()) {
+        if ($this->config->isRequiredBillingAddress() && !empty($details['billingAddress'])) {
             $this->updateAddressData($billingAddress, $details['billingAddress']);
         } else {
             $this->updateAddressData($billingAddress, $details['shippingAddress']);

app/code/Magento/Braintree/Test/Unit/Model/Paypal/Helper/QuoteUpdaterTest.php

@@ -3,62 +3,67 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+declare(strict_types=1);
+
 namespace Magento\Braintree\Test\Unit\Model\Paypal\Helper;
 
+use Magento\Braintree\Gateway\Config\PayPal\Config;
+use Magento\Braintree\Model\Paypal\Helper\QuoteUpdater;
+use Magento\Braintree\Model\Ui\PayPal\ConfigProvider;
+use Magento\Braintree\Observer\DataAssignObserver;
+use Magento\Quote\Api\CartRepositoryInterface;
+use Magento\Quote\Api\Data\CartExtensionInterface;
 use Magento\Quote\Model\Quote;
 use Magento\Quote\Model\Quote\Address;
 use Magento\Quote\Model\Quote\Payment;
-use Magento\Quote\Api\CartRepositoryInterface;
-use Magento\Braintree\Model\Ui\PayPal\ConfigProvider;
-use Magento\Braintree\Observer\DataAssignObserver;
-use Magento\Braintree\Gateway\Config\PayPal\Config;
-use Magento\Braintree\Model\Paypal\Helper\QuoteUpdater;
+use PHPUnit_Framework_MockObject_MockObject as MockObject;
 
 /**
  * Class QuoteUpdaterTest
  *
- * @see \Magento\Braintree\Model\Paypal\Helper\QuoteUpdater
- *
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
 class QuoteUpdaterTest extends \PHPUnit\Framework\TestCase
 {
     const TEST_NONCE = '3ede7045-2aea-463e-9754-cd658ffeeb48';
 
     /**
-     * @var Config|\PHPUnit_Framework_MockObject_MockObject
+     * @var Config|MockObject
      */
-    private $configMock;
+    private $config;
 
     /**
-     * @var CartRepositoryInterface|\PHPUnit_Framework_MockObject_MockObject
+     * @var CartRepositoryInterface|MockObject
      */
-    private $quoteRepositoryMock;
+    private $quoteRepository;
 
     /**
-     * @var Address|\PHPUnit_Framework_MockObject_MockObject
+     * @var Address|MockObject
      */
-    private $billingAddressMock;
+    private $billingAddress;
 
     /**
-     * @var Address|\PHPUnit_Framework_MockObject_MockObject
+     * @var Address|MockObject
      */
-    private $shippingAddressMock;
+    private $shippingAddress;
 
     /**
      * @var QuoteUpdater
      */
     private $quoteUpdater;
 
+    /**
+     * @inheritdoc
+     */
     protected function setUp()
     {
-        $this->configMock = $this->getMockBuilder(Config::class)
+        $this->config = $this->getMockBuilder(Config::class)
             ->disableOriginalConstructor()
             ->getMock();
-        $this->quoteRepositoryMock = $this->getMockBuilder(CartRepositoryInterface::class)
+        $this->quoteRepository = $this->getMockBuilder(CartRepositoryInterface::class)
             ->getMockForAbstractClass();
 
-        $this->billingAddressMock = $this->getMockBuilder(Address::class)
+        $this->billingAddress = $this->getMockBuilder(Address::class)
             ->setMethods(
                 [
                     'setLastname',
@@ -73,9 +78,10 @@ protected function setUp()
                     'setShouldIgnoreValidation',
                     'getEmail'
                 ]
-            )->disableOriginalConstructor()
+            )
+            ->disableOriginalConstructor()
             ->getMock();
-        $this->shippingAddressMock = $this->getMockBuilder(Address::class)
+        $this->shippingAddress = $this->getMockBuilder(Address::class)
             ->setMethods(
                 [
                     'setLastname',
@@ -89,54 +95,61 @@ protected function setUp()
                     'setPostcode',
                     'setShouldIgnoreValidation'
                 ]
-            )->disableOriginalConstructor()
+            )
+            ->disableOriginalConstructor()
             ->getMock();
 
         $this->quoteUpdater = new QuoteUpdater(
-            $this->configMock,
-            $this->quoteRepositoryMock
+            $this->config,
+            $this->quoteRepository
         );
     }
 
+    /**
+     * Checks if quote details can be update by the response from Braintree.
+     *
+     * @throws \Magento\Framework\Exception\LocalizedException
+     */
     public function testExecute()
     {
         $details = $this->getDetails();
-        $quoteMock = $this->getQuoteMock();
-        $paymentMock = $this->getPaymentMock();
+        $quote = $this->getQuoteMock();
+        $payment = $this->getPaymentMock();
 
-        $quoteMock->expects(self::once())
-            ->method('getPayment')
-            ->willReturn($paymentMock);
+        $quote->method('getPayment')
+            ->willReturn($payment);
 
-        $paymentMock->expects(self::once())
-            ->method('setMethod')
+        $payment->method('setMethod')
             ->with(ConfigProvider::PAYPAL_CODE);
-        $paymentMock->expects(self::once())
-            ->method('setAdditionalInformation')
+        $payment->method('setAdditionalInformation')
             ->with(DataAssignObserver::PAYMENT_METHOD_NONCE, self::TEST_NONCE);
 
-        $this->updateQuoteStep($quoteMock, $details);
+        $this->updateQuoteStep($quote, $details);
 
-        $this->quoteUpdater->execute(self::TEST_NONCE, $details, $quoteMock);
+        $this->quoteUpdater->execute(self::TEST_NONCE, $details, $quote);
     }
 
+    /**
+     * Disables quote's addresses validation.
+     *
+     * @return void
+     */
     private function disabledQuoteAddressValidationStep()
     {
-        $this->billingAddressMock->expects(self::once())
-            ->method('setShouldIgnoreValidation')
+        $this->billingAddress->method('setShouldIgnoreValidation')
             ->with(true);
-        $this->shippingAddressMock->expects(self::once())
-            ->method('setShouldIgnoreValidation')
+        $this->shippingAddress->method('setShouldIgnoreValidation')
             ->with(true);
-        $this->billingAddressMock->expects(self::once())
-            ->method('getEmail')
+        $this->billingAddress->method('getEmail')
             ->willReturn('bt_buyer_us@paypal.com');
     }
 
     /**
+     * Gets quote's details.
+     *
      * @return array
      */
-    private function getDetails()
+    private function getDetails(): array
     {
         return [
             'email' => 'bt_buyer_us@paypal.com',
@@ -166,54 +179,51 @@ private function getDetails()
     }
 
     /**
+     * Updates shipping address details.
+     *
      * @param array $details
      */
     private function updateShippingAddressStep(array $details)
     {
-        $this->shippingAddressMock->expects(self::once())
-            ->method('setLastname')
+        $this->shippingAddress->method('setLastname')
             ->with($details['lastName']);
-        $this->shippingAddressMock->expects(self::once())
-            ->method('setFirstname')
+        $this->shippingAddress->method('setFirstname')
             ->with($details['firstName']);
-        $this->shippingAddressMock->expects(self::once())
-            ->method('setEmail')
+        $this->shippingAddress->method('setEmail')
             ->with($details['email']);
-        $this->shippingAddressMock->expects(self::once())
-            ->method('setCollectShippingRates')
+        $this->shippingAddress->method('setCollectShippingRates')
             ->with(true);
 
-        $this->updateAddressDataStep($this->shippingAddressMock, $details['shippingAddress']);
+        $this->updateAddressDataStep($this->shippingAddress, $details['shippingAddress']);
     }
 
     /**
-     * @param \PHPUnit_Framework_MockObject_MockObject $addressMock
+     * Updates address details.
+     *
+     * @param MockObject $address
      * @param array $addressData
      */
-    private function updateAddressDataStep(\PHPUnit_Framework_MockObject_MockObject $addressMock, array $addressData)
+    private function updateAddressDataStep(MockObject $address, array $addressData)
     {
-        $addressMock->expects(self::once())
-            ->method('setStreet')
+        $address->method('setStreet')
             ->with([$addressData['streetAddress'], $addressData['extendedAddress']]);
-        $addressMock->expects(self::once())
-            ->method('setCity')
+        $address->method('setCity')
             ->with($addressData['locality']);
-        $addressMock->expects(self::once())
-            ->method('setRegionCode')
+        $address->method('setRegionCode')
             ->with($addressData['region']);
-        $addressMock->expects(self::once())
-            ->method('setCountryId')
+        $address->method('setCountryId')
             ->with($addressData['countryCodeAlpha2']);
-        $addressMock->expects(self::once())
-            ->method('setPostcode')
+        $address->method('setPostcode')
             ->with($addressData['postalCode']);
     }
 
     /**
-     * @param \PHPUnit_Framework_MockObject_MockObject $quoteMock
+     * Updates quote's address details.
+     *
+     * @param MockObject $quoteMock
      * @param array $details
      */
-    private function updateQuoteAddressStep(\PHPUnit_Framework_MockObject_MockObject $quoteMock, array $details)
+    private function updateQuoteAddressStep(MockObject $quoteMock, array $details)
     {
         $quoteMock->expects(self::exactly(2))
             ->method('getIsVirtual')
@@ -224,82 +234,92 @@ private function updateQuoteAddressStep(\PHPUnit_Framework_MockObject_MockObject
     }
 
     /**
+     * Updates billing address details.
+     *
      * @param array $details
      */
     private function updateBillingAddressStep(array $details)
     {
-        $this->configMock->expects(self::once())
-            ->method('isRequiredBillingAddress')
+        $this->config->method('isRequiredBillingAddress')
             ->willReturn(true);
 
-        $this->updateAddressDataStep($this->billingAddressMock, $details['billingAddress']);
+        $this->updateAddressDataStep($this->billingAddress, $details['billingAddress']);
 
-        $this->billingAddressMock->expects(self::once())
-            ->method('setLastname')
+        $this->billingAddress->method('setLastname')
             ->with($details['lastName']);
-        $this->billingAddressMock->expects(self::once())
-            ->method('setFirstname')
+        $this->billingAddress->method('setFirstname')
             ->with($details['firstName']);
-        $this->billingAddressMock->expects(self::once())
-            ->method('setEmail')
+        $this->billingAddress->method('setEmail')
             ->with($details['email']);
     }
 
     /**
-     * @param \PHPUnit_Framework_MockObject_MockObject $quoteMock
+     * Updates quote details.
+     *
+     * @param MockObject $quote
      * @param array $details
      */
-    private function updateQuoteStep(\PHPUnit_Framework_MockObject_MockObject $quoteMock, array $details)
+    private function updateQuoteStep(MockObject $quote, array $details)
     {
-        $quoteMock->expects(self::once())
-            ->method('setMayEditShippingAddress')
+        $quote->method('setMayEditShippingAddress')
             ->with(false);
-        $quoteMock->expects(self::once())
-            ->method('setMayEditShippingMethod')
+        $quote->method('setMayEditShippingMethod')
             ->with(true);
 
-        $quoteMock->expects(self::exactly(2))
-            ->method('getShippingAddress')
-            ->willReturn($this->shippingAddressMock);
-        $quoteMock->expects(self::exactly(2))
+        $quote->method('getShippingAddress')
+            ->willReturn($this->shippingAddress);
+        $quote->expects(self::exactly(2))
             ->method('getBillingAddress')
-            ->willReturn($this->billingAddressMock);
+            ->willReturn($this->billingAddress);
 
-        $this->updateQuoteAddressStep($quoteMock, $details);
+        $this->updateQuoteAddressStep($quote, $details);
         $this->disabledQuoteAddressValidationStep();
 
-        $quoteMock->expects(self::once())
-            ->method('collectTotals');
+        $quote->method('collectTotals');
 
-        $this->quoteRepositoryMock->expects(self::once())
-            ->method('save')
-            ->with($quoteMock);
+        $this->quoteRepository->method('save')
+            ->with($quote);
     }
 
     /**
-     * @return Quote|\PHPUnit_Framework_MockObject_MockObject
+     * Creates a mock for Quote object.
+     *
+     * @return Quote|MockObject
      */
-    private function getQuoteMock()
+    private function getQuoteMock(): MockObject
     {
-        return $this->getMockBuilder(Quote::class)
+        $quote = $this->getMockBuilder(Quote::class)
             ->setMethods(
                 [
                     'getIsVirtual',
                     'getPayment',
+                    'getExtensionAttributes',
                     'setMayEditShippingAddress',
                     'setMayEditShippingMethod',
                     'collectTotals',
                     'getShippingAddress',
                     'getBillingAddress',
                 ]
-            )->disableOriginalConstructor()
+            )
+            ->disableOriginalConstructor()
             ->getMock();
+
+        $cartExtension = $this->getMockBuilder(CartExtensionInterface::class)
+            ->setMethods(['setShippingAssignments'])
+            ->disableOriginalConstructor()
+            ->getMockForAbstractClass();
+
+        $quote->method('getExtensionAttributes')
+            ->willReturn($cartExtension);
+        return $quote;
     }
 
     /**
-     * @return Payment|\PHPUnit_Framework_MockObject_MockObject
+     * Creates a mock for Payment object.
+     *
+     * @return Payment|MockObject
      */
-    private function getPaymentMock()
+    private function getPaymentMock(): MockObject
     {
         return $this->getMockBuilder(Payment::class)
             ->disableOriginalConstructor()