Skip to content

Commit 6e10c91

Browse files
committed
minor
1 parent 8582457 commit 6e10c91

File tree

2 files changed

+42
-4
lines changed

2 files changed

+42
-4
lines changed

tests/unit/Mage/Adminhtml/Helper/DataTest.php

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,33 @@ public static function setUpBeforeClass(): void
2525
self::$subject = Mage::helper('adminhtml/data');
2626
}
2727

28+
/**
29+
* @covers Mage_Adminhtml_Helper_Data::getPageHelpUrl()
30+
* @group Helper
31+
*/
32+
public function testGetPageHelpUrl(): void
33+
{
34+
self::assertNull(self::$subject->getPageHelpUrl());
35+
}
36+
37+
/**
38+
* @covers Mage_Adminhtml_Helper_Data::setPageHelpUrl()
39+
* @group Helper
40+
*/
41+
public function testSetPageHelpUrl(): void
42+
{
43+
self::assertInstanceOf(self::$subject::class, self::$subject->setPageHelpUrl());
44+
}
45+
46+
/**
47+
* @covers Mage_Adminhtml_Helper_Data::addPageHelpUrl()
48+
* @group Helper
49+
*/
50+
public function testAddPageHelpUrl(): void
51+
{
52+
self::assertInstanceOf(self::$subject::class, self::$subject->addPageHelpUrl(null));
53+
}
54+
2855
/**
2956
* @covers Mage_Adminhtml_Helper_Data::getUrl()
3057
* @group Helper
@@ -36,6 +63,15 @@ public function testGetUrl(): void
3663
self::assertIsString(self::$subject->getUrl());
3764
}
3865

66+
/**
67+
* @covers Mage_Adminhtml_Helper_Data::getCustomAdminUrl()
68+
* @group Helper
69+
*/
70+
public function testGetCustomAdminUrl(): void
71+
{
72+
self::assertIsString(self::$subject->getCustomAdminUrl());
73+
}
74+
3975
/**
4076
* @covers Mage_Adminhtml_Helper_Data::getCurrentUserId()
4177
* @group Helper

tests/unit/Mage/Core/Helper/EnvironmentConfigLoaderTest.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,9 @@ public static function tearDownAfterClass(): void
6969
}
7070
}
7171

72+
/**
73+
* @group Helper
74+
*/
7275
public function testStoresAreCreated(): void
7376
{
7477
foreach (self::$storeData as $stores) {
@@ -118,8 +121,7 @@ public function testEnvFilter(): void
118121
}
119122

120123
/**
121-
* @group Mage_Core
122-
* @group Mage_Core_Helper
124+
* @group Helper
123125
*/
124126
public function testBuildNodePath(): void
125127
{
@@ -234,7 +236,7 @@ public function envOverridesCorrectConfigKeysDataProvider(): Generator
234236
/**
235237
* @runInSeparateProcess
236238
* @dataProvider envAsArrayDataProvider
237-
* @group Mage_Core
239+
* @group Helper
238240
*
239241
* @param array<string, string> $config
240242
*/
@@ -285,7 +287,7 @@ public function envAsArrayDataProvider(): Generator
285287
/**
286288
* @runInSeparateProcess
287289
* @dataProvider envHasPathDataProvider
288-
* @group Mage_Core
290+
* @group Helper
289291
*
290292
* @param array<string, string> $config
291293
*/

0 commit comments

Comments
 (0)