Skip to content

Commit c4a799a

Browse files
committed
Merge branch 'SPRINT-40' into 'SPRINT-41'
Sprint 40 See merge request !273
2 parents fcd06a3 + 48503c0 commit c4a799a

File tree

8 files changed

+224
-15
lines changed

8 files changed

+224
-15
lines changed

app/code/Magento/Catalog/Pricing/Price/TierPrice.php

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
use Magento\Framework\Pricing\Price\AbstractPrice;
1717
use Magento\Framework\Pricing\Price\BasePriceProviderInterface;
1818
use Magento\Framework\Pricing\PriceInfoInterface;
19+
use Magento\Customer\Model\Group\RetrieverInterface as CustomerGroupRetrieverInterface;
1920

2021
/**
2122
* @api
@@ -30,6 +31,7 @@ class TierPrice extends AbstractPrice implements TierPriceInterface, BasePricePr
3031

3132
/**
3233
* @var Session
34+
* @deprecated
3335
*/
3436
protected $customerSession;
3537

@@ -57,30 +59,39 @@ class TierPrice extends AbstractPrice implements TierPriceInterface, BasePricePr
5759
*/
5860
protected $groupManagement;
5961

62+
/**
63+
* @var CustomerGroupRetrieverInterface
64+
*/
65+
private $customerGroupRetriever;
66+
6067
/**
6168
* @param Product $saleableItem
6269
* @param float $quantity
6370
* @param CalculatorInterface $calculator
6471
* @param \Magento\Framework\Pricing\PriceCurrencyInterface $priceCurrency
6572
* @param Session $customerSession
6673
* @param GroupManagementInterface $groupManagement
74+
* @param CustomerGroupRetrieverInterface|null $customerGroupRetriever
6775
*/
6876
public function __construct(
6977
Product $saleableItem,
7078
$quantity,
7179
CalculatorInterface $calculator,
7280
\Magento\Framework\Pricing\PriceCurrencyInterface $priceCurrency,
7381
Session $customerSession,
74-
GroupManagementInterface $groupManagement
82+
GroupManagementInterface $groupManagement,
83+
CustomerGroupRetrieverInterface $customerGroupRetriever = null
7584
) {
7685
$quantity = $quantity ?: 1;
7786
parent::__construct($saleableItem, $quantity, $calculator, $priceCurrency);
7887
$this->customerSession = $customerSession;
7988
$this->groupManagement = $groupManagement;
89+
$this->customerGroupRetriever = $customerGroupRetriever
90+
?? \Magento\Framework\App\ObjectManager::getInstance()->get(CustomerGroupRetrieverInterface::class);
8091
if ($saleableItem->hasCustomerGroupId()) {
8192
$this->customerGroup = (int) $saleableItem->getCustomerGroupId();
8293
} else {
83-
$this->customerGroup = (int) $this->customerSession->getCustomerGroupId();
94+
$this->customerGroup = (int) $this->customerGroupRetriever->getCustomerGroupId();
8495
}
8596
}
8697

app/code/Magento/Catalog/Test/Unit/Pricing/Price/TierPriceTest.php

Lines changed: 21 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -25,47 +25,52 @@ class TierPriceTest extends \PHPUnit\Framework\TestCase
2525
*
2626
* @var int
2727
*/
28-
protected $customerGroup = Group::NOT_LOGGED_IN_ID;
28+
private $customerGroup = Group::NOT_LOGGED_IN_ID;
2929

3030
/**
3131
* @var \PHPUnit_Framework_MockObject_MockObject
3232
*/
33-
protected $priceInfo;
33+
private $priceInfo;
3434

3535
/**
3636
* @var \PHPUnit_Framework_MockObject_MockObject
3737
*/
38-
protected $product;
38+
private $product;
3939

4040
/**
4141
* @var float
4242
*/
43-
protected $quantity = 3.;
43+
private $quantity = 3.;
4444

4545
/**
4646
* @var \PHPUnit_Framework_MockObject_MockObject
4747
*/
48-
protected $calculator;
48+
private $calculator;
4949

5050
/**
5151
* @var \PHPUnit_Framework_MockObject_MockObject
5252
*/
53-
protected $session;
53+
private $session;
5454

5555
/**
5656
* @var TierPrice
5757
*/
58-
protected $model;
58+
private $model;
5959

6060
/**
6161
* @var \Magento\Framework\Pricing\PriceCurrencyInterface|\PHPUnit_Framework_MockObject_MockObject
6262
*/
63-
protected $priceCurrencyMock;
63+
private $priceCurrencyMock;
6464

6565
/**
6666
* @var \PHPUnit_Framework_MockObject_MockObject
6767
*/
68-
protected $groupManagement;
68+
private $groupManagement;
69+
70+
/**
71+
* @var \Magento\Customer\Model\Group\RetrieverInterface|\PHPUnit_Framework_MockObject_MockObject
72+
*/
73+
private $customerGroupRetriever;
6974

7075
/**
7176
* Initialize base dependencies
@@ -76,11 +81,12 @@ protected function setUp()
7681

7782
$this->product = $this->createPartialMock(\Magento\Catalog\Model\Product::class, ['getPriceInfo', 'hasCustomerGroupId', 'getCustomerGroupId', 'getResource', '__wakeup']);
7883
$this->product->expects($this->any())->method('getPriceInfo')->will($this->returnValue($this->priceInfo));
79-
84+
$this->customerGroupRetriever = $this->getMockBuilder(\Magento\Customer\Model\Group\RetrieverInterface::class)
85+
->disableOriginalConstructor()->getMock();
8086
$this->session = $this->createMock(\Magento\Customer\Model\Session::class);
8187
$this->session->expects($this->any())->method('getCustomerGroupId')
8288
->will($this->returnValue($this->customerGroup));
83-
89+
$this->customerGroupRetriever = $this->createMock(\Magento\Customer\Model\Group\RetrieverInterface::class);
8490
$this->calculator = $this->createMock(\Magento\Framework\Pricing\Adjustment\Calculator::class);
8591
$this->groupManagement = $this->createMock(\Magento\Customer\Api\GroupManagementInterface::class);
8692

@@ -92,7 +98,8 @@ protected function setUp()
9298
$this->calculator,
9399
$this->priceCurrencyMock,
94100
$this->session,
95-
$this->groupManagement
101+
$this->groupManagement,
102+
$this->customerGroupRetriever
96103
);
97104
}
98105

@@ -218,7 +225,8 @@ public function testGetterStoredTierPrices()
218225
$this->calculator,
219226
$this->priceCurrencyMock,
220227
$this->session,
221-
$this->groupManagement
228+
$this->groupManagement,
229+
$this->customerGroupRetriever
222230
);
223231
$group = $this->createMock(\Magento\Customer\Model\Data\Group::class);
224232
$group->expects($this->once())->method('getId')->willReturn(GroupManagement::CUST_GROUP_ALL);
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+
namespace Magento\Customer\Model\Group;
7+
8+
use Magento\Customer\Model\Session;
9+
10+
/**
11+
* Class for getting current customer group from customer session.
12+
*/
13+
class Retriever implements RetrieverInterface
14+
{
15+
/**
16+
* @var Session
17+
*/
18+
private $customerSession;
19+
20+
/**
21+
* @param Session $customerSession
22+
*/
23+
public function __construct(Session $customerSession)
24+
{
25+
$this->customerSession = $customerSession;
26+
}
27+
28+
/**
29+
* @inheritdoc
30+
*/
31+
public function getCustomerGroupId()
32+
{
33+
return $this->customerSession->getCustomerGroupId();
34+
}
35+
}
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
namespace Magento\Customer\Model\Group;
7+
8+
/**
9+
* Interface for getting current customer group from session.
10+
*
11+
* @api
12+
*/
13+
interface RetrieverInterface
14+
{
15+
/**
16+
* Retrieve customer group id.
17+
*
18+
* @return int
19+
*/
20+
public function getCustomerGroupId();
21+
}

app/code/Magento/Customer/etc/di.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,8 @@
5959
type="Magento\Customer\Model\Customer\Attribute\Source\Group"/>
6060
<preference for="Magento\Customer\Block\Account\SortLinkInterface"
6161
type="Magento\Customer\Block\Account\SortLink"/>
62+
<preference for="Magento\Customer\Model\Group\RetrieverInterface"
63+
type="Magento\Customer\Model\Group\Retriever"/>
6264
<type name="Magento\Customer\Model\Session">
6365
<arguments>
6466
<argument name="configShare" xsi:type="object">Magento\Customer\Model\Config\Share\Proxy</argument>
Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
namespace Magento\Sales\Model;
7+
8+
use Magento\Backend\Model\Session\Quote;
9+
use Magento\Customer\Api\GroupManagementInterface;
10+
11+
/**
12+
* Class for getting customer group from quote session for adminhtml area.
13+
*/
14+
class CustomerGroupRetriever implements \Magento\Customer\Model\Group\RetrieverInterface
15+
{
16+
/**
17+
* @var Quote
18+
*/
19+
private $quoteSession;
20+
21+
/**
22+
* @var GroupManagementInterface
23+
*/
24+
private $groupManagement;
25+
26+
/**
27+
* @param Quote $quoteSession
28+
* @param GroupManagementInterface $groupManagement
29+
*/
30+
public function __construct(Quote $quoteSession, GroupManagementInterface $groupManagement)
31+
{
32+
$this->quoteSession = $quoteSession;
33+
$this->groupManagement = $groupManagement;
34+
}
35+
36+
/**
37+
* @inheritdoc
38+
*/
39+
public function getCustomerGroupId()
40+
{
41+
if ($this->quoteSession->getQuoteId() && $this->quoteSession->getQuote()) {
42+
return $this->quoteSession->getQuote()->getCustomerGroupId();
43+
}
44+
return $this->groupManagement->getNotLoggedInGroup()->getId();
45+
}
46+
}
Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
<?php
2+
/**
3+
* Copyright © Magento, Inc. All rights reserved.
4+
* See COPYING.txt for license details.
5+
*/
6+
namespace Magento\Sales\Test\Unit\Model;
7+
8+
use Magento\Backend\Model\Session\Quote;
9+
use Magento\Customer\Api\GroupManagementInterface;
10+
11+
/**
12+
* Test for class CustomerGroupRetriever.
13+
*/
14+
class CustomerGroupRetrieverTest extends \PHPUnit\Framework\TestCase
15+
{
16+
/**
17+
* @var \Magento\Sales\Model\CustomerGroupRetriever
18+
*/
19+
private $retriever;
20+
21+
/**
22+
* @var Quote|\PHPUnit_Framework_MockObject_MockObject
23+
*/
24+
private $quoteSession;
25+
26+
/**
27+
* @var GroupManagementInterface|\PHPUnit_Framework_MockObject_MockObject
28+
*/
29+
private $groupManagement;
30+
31+
/**
32+
* @inheritdoc
33+
*/
34+
protected function setUp()
35+
{
36+
$this->quoteSession = $this->getMockBuilder(Quote::class)
37+
->disableOriginalConstructor()
38+
->setMethods(['getQuoteId', 'getQuote'])
39+
->getMock();
40+
$this->groupManagement = $this->getMockBuilder(GroupManagementInterface::class)
41+
->disableOriginalConstructor()
42+
->getMockForAbstractClass();
43+
44+
$helper = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
45+
$this->retriever = $helper->getObject(
46+
\Magento\Sales\Model\CustomerGroupRetriever::class,
47+
[
48+
'quoteSession' => $this->quoteSession,
49+
'groupManagement' => $this->groupManagement
50+
]
51+
);
52+
}
53+
54+
/**
55+
* Test method getCustomerGroupId with quote session.
56+
*/
57+
public function testGetCustomerGroupIdQuote()
58+
{
59+
$this->quoteSession->expects($this->atLeastOnce())->method('getQuoteId')->willReturn(1);
60+
$quote = $this->getMockBuilder(\Magento\Quote\Model\Quote::class)
61+
->disableOriginalConstructor()
62+
->getMock();
63+
$this->quoteSession->expects($this->atLeastOnce())->method('getQuote')->willReturn($quote);
64+
$quote->expects($this->once())->method('getCustomerGroupId')->willReturn(2);
65+
66+
$this->assertEquals(2, $this->retriever->getCustomerGroupId());
67+
}
68+
69+
/**
70+
* Test method getCustomerGroupId without quote session.
71+
*/
72+
public function testGetCustomerGroupIdDefault()
73+
{
74+
$this->quoteSession->expects($this->atLeastOnce())->method('getQuoteId')->willReturn(0);
75+
$this->quoteSession->expects($this->never())->method('getQuote');
76+
$group = $this->getMockBuilder(\Magento\Customer\Api\Data\GroupInterface::class)
77+
->disableOriginalConstructor()
78+
->getMockForAbstractClass();
79+
$this->groupManagement->expects($this->once())->method('getNotLoggedInGroup')->willReturn($group);
80+
$group->expects($this->once())->method('getId')->willReturn(2);
81+
82+
$this->assertEquals(2, $this->retriever->getCustomerGroupId());
83+
}
84+
}

app/code/Magento/Sales/etc/adminhtml/di.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
*/
77
-->
88
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
9+
<preference for="Magento\Customer\Model\Group\RetrieverInterface"
10+
type="Magento\Sales\Model\CustomerGroupRetriever"/>
911
<type name="Magento\Framework\App\Rss\RssManagerInterface">
1012
<arguments>
1113
<argument name="dataProviders" xsi:type="array">

0 commit comments

Comments
 (0)