Skip to content

Commit 3d8075d

Browse files
committed
AC-10821: Static Test fixes
1 parent 99ec7d7 commit 3d8075d

File tree

5 files changed

+12
-30
lines changed

5 files changed

+12
-30
lines changed

app/code/Magento/Backend/Test/Unit/Block/MenuTest.php

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,14 @@
1616
use Magento\Backend\Model\Menu\Filter\IteratorFactory;
1717
use Magento\Backend\Model\Menu\Item;
1818
use Magento\Backend\Model\UrlInterface;
19-
use Magento\Directory\Helper\Data as DirectoryHelper;
20-
use Magento\Framework\Json\Helper\Data as JsonHelper;
2119
use Magento\Framework\Locale\ResolverInterface;
2220
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager as ObjectManagerHelper;
2321
use PHPUnit\Framework\MockObject\MockObject;
2422
use PHPUnit\Framework\TestCase;
2523

24+
/**
25+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
26+
*/
2627
class MenuTest extends TestCase
2728
{
2829
/**
@@ -75,17 +76,8 @@ protected function setUp(): void
7576
->getMock();
7677

7778
$objectManagerHelper = new ObjectManagerHelper($this);
78-
$objects = [
79-
[
80-
JsonHelper::class,
81-
$this->createMock(JsonHelper::class)
82-
],
83-
[
84-
DirectoryHelper::class,
85-
$this->createMock(DirectoryHelper::class)
86-
]
87-
];
88-
$objectManagerHelper->prepareObjectManager($objects);
79+
$objectManagerHelper->prepareObjectManager();
80+
8981
$this->menu = $objectManagerHelper->getObject(
9082
Menu::class,
9183
[

app/code/Magento/CatalogUrlRewrite/Test/Unit/Model/Storage/DynamicStorageTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@
2525
use ReflectionMethod;
2626
use Psr\Log\LoggerInterface;
2727

28+
/**
29+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
30+
*/
2831
class DynamicStorageTest extends TestCase
2932
{
3033
/**

app/code/Magento/Checkout/Test/Unit/Model/DefaultConfigProviderTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ protected function setUp(): void
128128
$this->addressMetadata = $this->createMock(AddressMetadataInterface::class);
129129
$attributeOptionManager = $this->createMock(AttributeOptionManagementInterface::class);
130130
$customerAddressData = $this->createMock(CustomerAddressDataProvider::class);
131-
$escaper = $this->createMock(Escaper::class);
131+
$this->escaper = $this->createMock(Escaper::class);
132132
$this->model = new DefaultConfigProvider(
133133
$checkoutHelper,
134134
$this->checkoutSession,
@@ -160,7 +160,7 @@ protected function setUp(): void
160160
$this->addressMetadata,
161161
$attributeOptionManager,
162162
$customerAddressData,
163-
$escaper
163+
$this->escaper
164164
);
165165
}
166166

app/code/Magento/Cms/Test/Unit/Block/Adminhtml/Wysiwyg/Images/TreeTest.php

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,10 @@
1111
use Magento\Cms\Block\Adminhtml\Wysiwyg\Images\Tree;
1212
use Magento\Cms\Helper\Wysiwyg\Images;
1313
use Magento\Cms\Model\Wysiwyg\Images\Storage;
14-
use Magento\Directory\Helper\Data as DirectoryHelper;
1514
use Magento\Framework\App\Filesystem\DirectoryList;
1615
use Magento\Framework\DataObject;
1716
use Magento\Framework\Filesystem;
1817
use Magento\Framework\Filesystem\Directory\Read;
19-
use Magento\Framework\Json\Helper\Data as JsonHelper;
2018
use Magento\Framework\Registry;
2119
use Magento\Framework\Serialize\Serializer\Json;
2220
use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
@@ -64,18 +62,7 @@ class TreeTest extends TestCase
6462
protected function setUp(): void
6563
{
6664
$objectManager = new ObjectManager($this);
67-
68-
$objects = [
69-
[
70-
JsonHelper::class,
71-
$this->createMock(JsonHelper::class)
72-
],
73-
[
74-
DirectoryHelper::class,
75-
$this->createMock(DirectoryHelper::class)
76-
]
77-
];
78-
$objectManager->prepareObjectManager($objects);
65+
$objectManager->prepareObjectManager();
7966

8067
$contextMock = $this->createMock(Context::class);
8168
$this->cmsWysiwygImagesMock = $this->createMock(Images::class);

lib/internal/Magento/Framework/TestFramework/Unit/Helper/ObjectManager.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ public function setBackwardCompatibleProperty($object, $propertyName, $propertyV
368368
*
369369
* @param array $map
370370
*/
371-
public function prepareObjectManager($map)
371+
public function prepareObjectManager(array $map = [])
372372
{
373373
$objectManagerMock = $this->_testObject->getMockBuilder(ObjectManagerInterface::class)
374374
->addMethods(['getInstance'])

0 commit comments

Comments
 (0)