Skip to content

Commit 2d00990

Browse files
author
Joan He
committed
MAGETWO-32371: Move jsonEncode and jsonDecode from core helper to framework
- Addressed CR commits
1 parent 012015d commit 2d00990

File tree

2 files changed

+5
-92
lines changed
  • dev/tests

2 files changed

+5
-92
lines changed

dev/tests/integration/testsuite/Magento/ImportExport/Block/Adminhtml/Import/Edit/BeforeTest.php

Lines changed: 4 additions & 91 deletions
Original file line numberDiff line numberDiff line change
@@ -18,98 +18,10 @@ class BeforeTest extends \PHPUnit_Framework_TestCase
1818
*/
1919
protected $_model;
2020

21-
/**
22-
* Source entity behaviors
23-
*
24-
* @var array
25-
*/
26-
protected $_sourceEntities = [
27-
'entity_1' => ['code' => 'behavior_1', 'token' => 'Some_Random_First_Class'],
28-
'entity_2' => ['code' => 'behavior_2', 'token' => 'Some_Random_Second_Class'],
29-
];
30-
31-
/**
32-
* Expected entity behaviors
33-
*
34-
* @var array
35-
*/
36-
protected $_expectedEntities = ['entity_1' => 'behavior_1', 'entity_2' => 'behavior_2'];
37-
38-
/**
39-
* Source unique behaviors
40-
*
41-
* @var array
42-
*/
43-
protected $_sourceBehaviors = [
44-
'behavior_1' => 'Some_Random_First_Class',
45-
'behavior_2' => 'Some_Random_Second_Class',
46-
];
47-
48-
/**
49-
* Expected unique behaviors
50-
*
51-
* @var array
52-
*/
53-
protected $_expectedBehaviors = ['behavior_1', 'behavior_2'];
54-
5521
protected function setUp()
5622
{
57-
$jsonEncoderMock = $this->getMockBuilder('Magento\Framework\Json\EncoderInterface')
58-
->disableOriginalConstructor()
59-
->getMock();
60-
$jsonEncoderMock->expects(
61-
$this->any()
62-
)->method(
63-
'encode'
64-
)->will(
65-
$this->returnCallback([$this, 'jsonEncodeCallback'])
66-
);
67-
68-
$importModel = $this->getMock(
69-
'Magento\ImportExport\Model\Import',
70-
['getEntityBehaviors', 'getUniqueEntityBehaviors'],
71-
[],
72-
'',
73-
false
74-
);
75-
$importModel->expects(
76-
$this->any()
77-
)->method(
78-
'getEntityBehaviors'
79-
)->will(
80-
$this->returnValue($this->_sourceEntities)
81-
);
82-
$importModel->expects(
83-
$this->any()
84-
)->method(
85-
'getUniqueEntityBehaviors'
86-
)->will(
87-
$this->returnValue($this->_sourceBehaviors)
88-
);
89-
90-
$arguments = [
91-
'jsonEncoder' => $jsonEncoderMock,
92-
'importModel' => $importModel,
93-
'urlBuilder' => $this->getMock('Magento\Backend\Model\Url', [], [], '', false),
94-
];
9523
$objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
96-
$this->_model = $objectManager->create('Magento\ImportExport\Block\Adminhtml\Import\Edit\Before', $arguments);
97-
}
98-
99-
protected function tearDown()
100-
{
101-
unset($this->_model);
102-
}
103-
104-
/**
105-
* Callback method for \Magento\Framework\Json\EncoderInterface::jsonEncode
106-
*
107-
* @param mixed $data
108-
* @return string
109-
*/
110-
public function jsonEncodeCallback($data)
111-
{
112-
return \Zend_Json::encode($data);
24+
$this->_model = $objectManager->create('Magento\ImportExport\Block\Adminhtml\Import\Edit\Before');
11325
}
11426

11527
/**
@@ -120,7 +32,8 @@ public function jsonEncodeCallback($data)
12032
public function testGetEntityBehaviors()
12133
{
12234
$actualEntities = $this->_model->getEntityBehaviors();
123-
$expectedEntities = \Zend_Json::encode($this->_expectedEntities);
35+
$expectedEntities = '{"catalog_product":"basic_behavior","customer_finance":"custom_behavior",' .
36+
'"customer_composite":"basic_behavior","customer":"custom_behavior","customer_address":"custom_behavior"}';
12437
$this->assertEquals($expectedEntities, $actualEntities);
12538
}
12639

@@ -132,7 +45,7 @@ public function testGetEntityBehaviors()
13245
public function testGetUniqueBehaviors()
13346
{
13447
$actualBehaviors = $this->_model->getUniqueBehaviors();
135-
$expectedBehaviors = \Zend_Json::encode($this->_expectedBehaviors);
48+
$expectedBehaviors = '["basic_behavior","custom_behavior"]';
13649
$this->assertEquals($expectedBehaviors, $actualBehaviors);
13750
}
13851
}

dev/tests/unit/testsuite/Magento/ConfigurableProduct/Model/Product/Type/ConfigurableTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ class ConfigurableTest extends \PHPUnit_Framework_TestCase
6767
protected $_productFactoryMock;
6868

6969
/**
70-
* @var \PHPUnit_Framework_MockObject_MockObject|\Magento\Framework\Json\Helper\
70+
* @var \PHPUnit_Framework_MockObject_MockObject|\Magento\Framework\Json\Helper\Data
7171
*/
7272
protected $jsonHelperMock;
7373

0 commit comments

Comments
 (0)