Skip to content

Commit 8c0dee3

Browse files
author
roman
committed
MAGETWO-96757: Fixed incorrect displaying of the sales rule conditions
1 parent 405a7d4 commit 8c0dee3

File tree

4 files changed

+15
-7
lines changed

4 files changed

+15
-7
lines changed

lib/internal/Magento/Framework/Data/Test/Unit/Form/Element/AbstractElementTest.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,12 @@ class AbstractElementTest extends \PHPUnit\Framework\TestCase
3333

3434
protected function setUp()
3535
{
36+
$objectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
3637
$this->_factoryMock =
3738
$this->createMock(\Magento\Framework\Data\Form\Element\Factory::class);
3839
$this->_collectionFactoryMock =
3940
$this->createMock(\Magento\Framework\Data\Form\Element\CollectionFactory::class);
40-
$this->_escaperMock = $this->createMock(\Magento\Framework\Escaper::class);
41+
$this->_escaperMock = $objectManager->getObject(\Magento\Framework\Escaper::class);
4142

4243
$this->_model = $this->getMockForAbstractClass(
4344
\Magento\Framework\Data\Form\Element\AbstractElement::class,
@@ -423,9 +424,6 @@ public function testGetHtmlContainerIdWithFieldContainerIdPrefix()
423424
*/
424425
public function testAddElementValues(array $initialData, $expectedValue)
425426
{
426-
$this->_escaperMock->expects($this->any())
427-
->method('escapeHtml')
428-
->will($this->returnArgument(0));
429427
$this->_model->setValues($initialData['initial_values']);
430428
$this->_model->addElementValues($initialData['add_values'], $initialData['overwrite']);
431429

lib/internal/Magento/Framework/Data/Test/Unit/Form/Element/LinkTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@ class LinkTest extends \PHPUnit\Framework\TestCase
2323

2424
protected function setUp()
2525
{
26+
$objectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
2627
$factoryMock = $this->createMock(\Magento\Framework\Data\Form\Element\Factory::class);
2728
$collectionFactoryMock = $this->createMock(\Magento\Framework\Data\Form\Element\CollectionFactory::class);
28-
$escaperMock = $this->createMock(\Magento\Framework\Escaper::class);
29+
$escaperMock = $objectManager->getObject(\Magento\Framework\Escaper::class);
2930
$this->_link = new \Magento\Framework\Data\Form\Element\Link(
3031
$factoryMock,
3132
$collectionFactoryMock,

lib/internal/Magento/Framework/Data/Test/Unit/Form/Element/MultiselectTest.php

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
*/
66
namespace Magento\Framework\Data\Test\Unit\Form\Element;
77

8+
use Magento\Framework\Escaper;
9+
810
class MultiselectTest extends \PHPUnit\Framework\TestCase
911
{
1012
/**
@@ -15,7 +17,13 @@ class MultiselectTest extends \PHPUnit\Framework\TestCase
1517
protected function setUp()
1618
{
1719
$testHelper = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
18-
$this->_model = $testHelper->getObject(\Magento\Framework\Data\Form\Element\Editablemultiselect::class);
20+
$escaper = new Escaper();
21+
$this->_model = $testHelper->getObject(
22+
\Magento\Framework\Data\Form\Element\Editablemultiselect::class,
23+
[
24+
'_escaper' => $escaper
25+
]
26+
);
1927
$this->_model->setForm(new \Magento\Framework\DataObject());
2028
}
2129

lib/internal/Magento/Framework/Data/Test/Unit/Form/Element/NoteTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@ class NoteTest extends \PHPUnit\Framework\TestCase
2323

2424
protected function setUp()
2525
{
26+
$objectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
2627
$factoryMock = $this->createMock(\Magento\Framework\Data\Form\Element\Factory::class);
2728
$collectionFactoryMock = $this->createMock(\Magento\Framework\Data\Form\Element\CollectionFactory::class);
28-
$escaperMock = $this->createMock(\Magento\Framework\Escaper::class);
29+
$escaperMock = $objectManager->getObject(\Magento\Framework\Escaper::class);
2930
$this->_model = new \Magento\Framework\Data\Form\Element\Note(
3031
$factoryMock,
3132
$collectionFactoryMock,

0 commit comments

Comments
 (0)