Skip to content

Commit 075641d

Browse files
author
Mariana Lashch
committed
Merge branch 'MAGETWO-91867' into mpi-PR-1607
2 parents 2f27a36 + d627253 commit 075641d

File tree

3 files changed

+133
-43
lines changed

3 files changed

+133
-43
lines changed

‎app/code/Magento/GiftMessage/Observer/SalesEventQuoteMerge.php

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
declare(strict_types=1);
7+
8+
namespace Magento\GiftMessage\Observer;
9+
10+
use Magento\Framework\Event\ObserverInterface;
11+
use Magento\Quote\Model\Quote;
12+
13+
/**
14+
* Gift Message Observer Model
15+
*/
16+
class SalesEventQuoteMerge implements ObserverInterface
17+
{
18+
/**
19+
* Sets gift message to customer quote from guest quote.
20+
*
21+
* @param \Magento\Framework\Event\Observer $observer
22+
* @return $this
23+
*/
24+
public function execute(\Magento\Framework\Event\Observer $observer)
25+
{
26+
/** @var Quote $targetQuote */
27+
$targetQuote = $observer->getData('quote');
28+
/** @var Quote $sourceQuote */
29+
$sourceQuote = $observer->getData('source');
30+
31+
$targetQuote->setGiftMessageId($sourceQuote->getGiftMessageId());
32+
33+
return $this;
34+
}
35+
}

‎app/code/Magento/GiftMessage/etc/frontend/events.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@
1212
<event name="sales_convert_order_to_quote">
1313
<observer name="giftmessage" instance="Magento\GiftMessage\Observer\SalesEventOrderToQuoteObserver" shared="false" />
1414
</event>
15+
<event name="sales_quote_merge_after">
16+
<observer name="giftmessage" instance="Magento\GiftMessage\Observer\SalesEventQuoteMerge" shared="false" />
17+
</event>
1518
<event name="checkout_type_multishipping_create_orders_single">
1619
<observer name="giftmessage" instance="Magento\GiftMessage\Observer\MultishippingEventCreateOrdersObserver" shared="false" />
1720
</event>

‎dev/tests/integration/testsuite/Magento/Quote/Model/QuoteTest.php

Lines changed: 95 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -8,16 +8,33 @@
88
use Magento\Catalog\Model\ProductRepository;
99
use Magento\Customer\Api\Data\CustomerInterfaceFactory;
1010
use Magento\Framework\Exception\LocalizedException;
11+
use Magento\Quote\Api\Data\CartInterface;
1112
use Magento\TestFramework\Helper\Bootstrap;
13+
use Magento\TestFramework\ObjectManager;
14+
use Magento\Framework\Api\SearchCriteriaBuilder;
15+
use Magento\Quote\Api\CartRepositoryInterface;
1216

1317
/**
1418
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
1519
*/
1620
class QuoteTest extends \PHPUnit\Framework\TestCase
1721
{
22+
/**
23+
* @var ObjectManager
24+
*/
25+
private $objectManager;
26+
27+
/**
28+
* @inheritdoc
29+
*/
30+
protected function setUp()
31+
{
32+
$this->objectManager = Bootstrap::getObjectManager();
33+
}
34+
1835
private function convertToArray($entity)
1936
{
20-
return Bootstrap::getObjectManager()
37+
return $this->objectManager
2138
->create(\Magento\Framework\Api\ExtensibleDataObjectConverter::class)
2239
->toFlatArray($entity);
2340
}
@@ -28,10 +45,10 @@ private function convertToArray($entity)
2845
*/
2946
public function testCollectTotalsWithVirtual()
3047
{
31-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
48+
$quote = $this->objectManager->create(Quote::class);
3249
$quote->load('test01', 'reserved_order_id');
3350

34-
$productRepository = Bootstrap::getObjectManager()->create(
51+
$productRepository = $this->objectManager->create(
3552
\Magento\Catalog\Api\ProductRepositoryInterface::class
3653
);
3754
$product = $productRepository->get('virtual-product', false, null, true);
@@ -46,14 +63,14 @@ public function testCollectTotalsWithVirtual()
4663

4764
public function testSetCustomerData()
4865
{
49-
/** @var \Magento\Quote\Model\Quote $quote */
50-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
66+
/** @var Quote $quote */
67+
$quote = $this->objectManager->create(Quote::class);
5168
/** @var CustomerInterfaceFactory $customerFactory */
52-
$customerFactory = Bootstrap::getObjectManager()->create(
69+
$customerFactory = $this->objectManager->create(
5370
CustomerInterfaceFactory::class
5471
);
5572
/** @var \Magento\Framework\Api\DataObjectHelper $dataObjectHelper */
56-
$dataObjectHelper = Bootstrap::getObjectManager()->create(\Magento\Framework\Api\DataObjectHelper::class);
73+
$dataObjectHelper = $this->objectManager->create(\Magento\Framework\Api\DataObjectHelper::class);
5774
$expected = $this->_getCustomerDataArray();
5875
$customer = $customerFactory->create();
5976
$dataObjectHelper->populateWithArray(
@@ -72,13 +89,13 @@ public function testSetCustomerData()
7289

7390
public function testUpdateCustomerData()
7491
{
75-
/** @var \Magento\Quote\Model\Quote $quote */
76-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
77-
$customerFactory = Bootstrap::getObjectManager()->create(
92+
/** @var Quote $quote */
93+
$quote = $this->objectManager->create(Quote::class);
94+
$customerFactory = $this->objectManager->create(
7895
CustomerInterfaceFactory::class
7996
);
8097
/** @var \Magento\Framework\Api\DataObjectHelper $dataObjectHelper */
81-
$dataObjectHelper = Bootstrap::getObjectManager()->create(\Magento\Framework\Api\DataObjectHelper::class);
98+
$dataObjectHelper = $this->objectManager->create(\Magento\Framework\Api\DataObjectHelper::class);
8299
$expected = $this->_getCustomerDataArray();
83100
//For save in repository
84101
$expected = $this->removeIdFromCustomerData($expected);
@@ -92,7 +109,7 @@ public function testUpdateCustomerData()
92109
/**
93110
* @var \Magento\Customer\Api\CustomerRepositoryInterface $customerRepository
94111
*/
95-
$customerRepository = Bootstrap::getObjectManager()
112+
$customerRepository = $this->objectManager
96113
->create(\Magento\Customer\Api\CustomerRepositoryInterface::class);
97114
$customerRepository->save($customerDataSet);
98115
$quote->setCustomer($customerDataSet);
@@ -121,13 +138,13 @@ public function testGetCustomerGroupFromCustomer()
121138
{
122139
/** Preconditions */
123140
/** @var CustomerInterfaceFactory $customerFactory */
124-
$customerFactory = Bootstrap::getObjectManager()->create(
141+
$customerFactory = $this->objectManager->create(
125142
CustomerInterfaceFactory::class
126143
);
127144
$customerGroupId = 3;
128145
$customerData = $customerFactory->create()->setId(1)->setGroupId($customerGroupId);
129-
/** @var \Magento\Quote\Model\Quote $quote */
130-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
146+
/** @var Quote $quote */
147+
$quote = $this->objectManager->create(Quote::class);
131148
$quote->setCustomer($customerData);
132149
$quote->unsetData('customer_group_id');
133150

@@ -146,10 +163,10 @@ public function testGetCustomerTaxClassId()
146163
$fixtureGroupCode = 'custom_group';
147164
$fixtureTaxClassId = 3;
148165
/** @var \Magento\Customer\Model\Group $group */
149-
$group = Bootstrap::getObjectManager()->create(\Magento\Customer\Model\Group::class);
166+
$group = $this->objectManager->create(\Magento\Customer\Model\Group::class);
150167
$fixtureGroupId = $group->load($fixtureGroupCode, 'customer_group_code')->getId();
151-
/** @var \Magento\Quote\Model\Quote $quote */
152-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
168+
/** @var Quote $quote */
169+
$quote = $this->objectManager->create(Quote::class);
153170
$quote->setCustomerGroupId($fixtureGroupId);
154171

155172
/** Execute SUT */
@@ -169,8 +186,8 @@ public function testAssignCustomerWithAddressChangeAddressesNotSpecified()
169186
* Customer with two addresses created
170187
* First address is default billing, second is default shipping.
171188
*/
172-
/** @var \Magento\Quote\Model\Quote $quote */
173-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
189+
/** @var Quote $quote */
190+
$quote = $this->objectManager->create(Quote::class);
174191
$customerData = $this->_prepareQuoteForTestAssignCustomerWithAddressChange($quote);
175192

176193
/** Execute SUT */
@@ -234,9 +251,8 @@ public function testAssignCustomerWithAddressChange()
234251
* Customer with two addresses created
235252
* First address is default billing, second is default shipping.
236253
*/
237-
/** @var \Magento\Quote\Model\Quote $quote */
238-
$objectManager = Bootstrap::getObjectManager();
239-
$quote = $objectManager->create(\Magento\Quote\Model\Quote::class);
254+
/** @var Quote $quote */
255+
$quote = $this->objectManager->create(Quote::class);
240256
$customerData = $this->_prepareQuoteForTestAssignCustomerWithAddressChange($quote);
241257
/** @var \Magento\Quote\Model\Quote\Address $quoteBillingAddress */
242258
$expectedBillingAddressData = [
@@ -249,7 +265,7 @@ public function testAssignCustomerWithAddressChange()
249265
'firstname' => 'FirstBilling',
250266
'region_id' => 1
251267
];
252-
$quoteBillingAddress = $objectManager->create(\Magento\Quote\Model\Quote\Address::class);
268+
$quoteBillingAddress = $this->objectManager->create(\Magento\Quote\Model\Quote\Address::class);
253269
$quoteBillingAddress->setData($expectedBillingAddressData);
254270

255271
$expectedShippingAddressData = [
@@ -262,7 +278,7 @@ public function testAssignCustomerWithAddressChange()
262278
'firstname' => 'FirstShipping',
263279
'region_id' => 1
264280
];
265-
$quoteShippingAddress = $objectManager->create(\Magento\Quote\Model\Quote\Address::class);
281+
$quoteShippingAddress = $this->objectManager->create(\Magento\Quote\Model\Quote\Address::class);
266282
$quoteShippingAddress->setData($expectedShippingAddressData);
267283

268284
/** Execute SUT */
@@ -295,13 +311,13 @@ public function testAssignCustomerWithAddressChange()
295311
*/
296312
public function testAddProductUpdateItem()
297313
{
298-
/** @var \Magento\Quote\Model\Quote $quote */
299-
$quote = Bootstrap::getObjectManager()->create(\Magento\Quote\Model\Quote::class);
314+
/** @var Quote $quote */
315+
$quote = $this->objectManager->create(Quote::class);
300316
$quote->load('test01', 'reserved_order_id');
301317

302318
$productStockQty = 100;
303319

304-
$productRepository = Bootstrap::getObjectManager()->create(
320+
$productRepository = $this->objectManager->create(
305321
\Magento\Catalog\Api\ProductRepositoryInterface::class
306322
);
307323
$product = $productRepository->get('simple-1', false, null, true);
@@ -336,17 +352,16 @@ public function testAddProductUpdateItem()
336352
*
337353
* Customer with two addresses created. First address is default billing, second is default shipping.
338354
*
339-
* @param \Magento\Quote\Model\Quote $quote
355+
* @param Quote $quote
340356
* @return \Magento\Customer\Api\Data\CustomerInterface
341357
*/
342358
protected function _prepareQuoteForTestAssignCustomerWithAddressChange($quote)
343359
{
344-
$objectManager = Bootstrap::getObjectManager();
345360
/** @var \Magento\Customer\Api\CustomerRepositoryInterface $customerRepository */
346-
$customerRepository = $objectManager->create(\Magento\Customer\Api\CustomerRepositoryInterface::class);
361+
$customerRepository = $this->objectManager->create(\Magento\Customer\Api\CustomerRepositoryInterface::class);
347362
$fixtureCustomerId = 1;
348363
/** @var \Magento\Customer\Model\Customer $customer */
349-
$customer = $objectManager->create(\Magento\Customer\Model\Customer::class);
364+
$customer = $this->objectManager->create(\Magento\Customer\Model\Customer::class);
350365
$fixtureSecondAddressId = 2;
351366
$customer->load($fixtureCustomerId)->setDefaultShipping($fixtureSecondAddressId)->save();
352367
$customerData = $customerRepository->getById($fixtureCustomerId);
@@ -415,9 +430,8 @@ protected function _getCustomerDataArray()
415430
*/
416431
public function testReserveOrderId()
417432
{
418-
$objectManager = Bootstrap::getObjectManager();
419-
/** @var \Magento\Quote\Model\Quote $quote */
420-
$quote = $objectManager->create(\Magento\Quote\Model\Quote::class);
433+
/** @var Quote $quote */
434+
$quote = $this->objectManager->create(Quote::class);
421435
$quote->load('reserved_order_id', 'reserved_order_id');
422436
$quote->reserveOrderId();
423437
$this->assertEquals('reserved_order_id', $quote->getReservedOrderId());
@@ -433,20 +447,19 @@ public function testReserveOrderId()
433447
public function testAddedProductToQuoteIsSalable()
434448
{
435449
$productId = 99;
436-
$objectManager = Bootstrap::getObjectManager();
437450

438451
/** @var ProductRepository $productRepository */
439-
$productRepository = $objectManager->get(ProductRepository::class);
452+
$productRepository = $this->objectManager->get(ProductRepository::class);
440453

441-
/** @var \Magento\Quote\Model\Quote $quote */
454+
/** @var Quote $quote */
442455
$product = $productRepository->getById($productId, false, null, true);
443456

444457
$this->expectException(
445458
LocalizedException::class,
446459
'Product that you are trying to add is not available.'
447460
);
448461

449-
$quote = $objectManager->create(\Magento\Quote\Model\Quote::class);
462+
$quote = $this->objectManager->create(Quote::class);
450463
$quote->addProduct($product);
451464
}
452465

@@ -456,13 +469,12 @@ public function testAddedProductToQuoteIsSalable()
456469
*/
457470
public function testGetItemById()
458471
{
459-
$objectManager = Bootstrap::getObjectManager();
460-
$quote = $objectManager->create(\Magento\Quote\Model\Quote::class);
472+
$quote = $this->objectManager->create(Quote::class);
461473
$quote->load('test01', 'reserved_order_id');
462474

463-
$quoteItem = $objectManager->create(\Magento\Quote\Model\Quote\Item::class);
475+
$quoteItem = $this->objectManager->create(\Magento\Quote\Model\Quote\Item::class);
464476

465-
$productRepository = $objectManager->create(\Magento\Catalog\Api\ProductRepositoryInterface::class);
477+
$productRepository = $this->objectManager->create(\Magento\Catalog\Api\ProductRepositoryInterface::class);
466478
$product = $productRepository->get('simple');
467479

468480
$quoteItem->setProduct($product);
@@ -472,4 +484,44 @@ public function testGetItemById()
472484
$this->assertInstanceOf(\Magento\Quote\Model\Quote\Item::class, $quote->getItemById($quoteItem->getId()));
473485
$this->assertEquals($quoteItem->getId(), $quote->getItemById($quoteItem->getId())->getId());
474486
}
487+
488+
/**
489+
* Tests of quotes merging.
490+
*
491+
* @magentoDataFixture Magento/Sales/_files/quote.php
492+
*/
493+
public function testMerge()
494+
{
495+
$giftMessageId = 1;
496+
497+
/** @var Quote $quote */
498+
$guestQuote = $this->getQuote('test01');
499+
$guestQuote->setGiftMessageId($giftMessageId);
500+
501+
/** @var Quote $customerQuote */
502+
$customerQuote = $this->objectManager->create(Quote::class);
503+
$customerQuote->merge($guestQuote);
504+
505+
self::assertEquals($giftMessageId, $customerQuote->getGiftMessageId());
506+
}
507+
508+
/**
509+
* Gets quote by reserved order id.
510+
*
511+
* @param string $reservedOrderId
512+
* @return Quote
513+
*/
514+
private function getQuote($reservedOrderId)
515+
{
516+
/** @var SearchCriteriaBuilder $searchCriteriaBuilder */
517+
$searchCriteriaBuilder = $this->objectManager->get(SearchCriteriaBuilder::class);
518+
$searchCriteria = $searchCriteriaBuilder->addFilter('reserved_order_id', $reservedOrderId)
519+
->create();
520+
521+
/** @var CartRepositoryInterface $quoteRepository */
522+
$quoteRepository = $this->objectManager->get(CartRepositoryInterface::class);
523+
$items = $quoteRepository->getList($searchCriteria)->getItems();
524+
525+
return array_pop($items);
526+
}
475527
}

0 commit comments

Comments
 (0)