Skip to content

Commit 1b8c55e

Browse files
committed
sonar 4
1 parent 1607352 commit 1b8c55e

File tree

1 file changed

+34
-23
lines changed
  • tests/unit/Traits/DataProvider/Mage/Cms/Block/Widget/Page

1 file changed

+34
-23
lines changed

tests/unit/Traits/DataProvider/Mage/Cms/Block/Widget/Page/LinkTrait.php

Lines changed: 34 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,20 @@
2222

2323
trait LinkTrait
2424
{
25+
public static $defaults = [
26+
'custom_title' => 'Custom Title',
27+
'custom_text' => 'Custom Text',
28+
];
29+
30+
public static $tests = [
31+
'empty' => 'empty',
32+
'href' => 'href is set',
33+
'no_data' => 'no data is set',
34+
'page_id' => 'page_id is set',
35+
'text' => 'text is set',
36+
'title' => 'title is set',
37+
];
38+
2539
public function provideGetHrefData(): Generator
2640
{
2741
$emptyData = [
@@ -38,17 +52,17 @@ public function provideGetHrefData(): Generator
3852

3953
$data = $emptyData;
4054
$data['href'] = 'home';
41-
yield 'href is set' => [
55+
yield self::$tests['href'] => [
4256
'home',
4357
$data,
4458
];
4559

46-
yield 'empty' => [
60+
yield self::$tests['empty'] => [
4761
'',
4862
$emptyData,
4963
];
5064

51-
yield 'no data set' => [
65+
yield self::$tests['no_data'] => [
5266
'',
5367
[],
5468
];
@@ -63,32 +77,32 @@ public function provideGetTitleData(): Generator
6377
];
6478

6579
$data = $emptyData;
66-
$data['title'] = 'Custom Title';
67-
yield 'title is set' => [
68-
'Custom Title',
80+
$data['title'] = self::$defaults['custom_title'];
81+
yield self::$tests['title'] => [
82+
self::$defaults['custom_title'],
6983
$data,
7084
];
7185

7286
$data = $emptyData;
7387
$data['page_id'] = 1;
74-
yield 'page_id is set' => [
88+
yield self::$tests['page_id'] => [
7589
'404 Not Found 1',
7690
$data,
7791
];
7892

7993
$data = $emptyData;
8094
$data['href'] = 'home';
81-
yield 'href is set' => [
95+
yield self::$tests['href'] => [
8296
'Home page',
8397
$data,
8498
];
8599

86-
yield 'empty' => [
100+
yield self::$tests['empty'] => [
87101
'',
88102
$emptyData,
89103
];
90104

91-
yield 'no data set' => [
105+
yield self::$tests['no_data'] => [
92106
'',
93107
[],
94108
];
@@ -103,43 +117,40 @@ public function provideGetAnchorTextData(): Generator
103117
'test' => 'Test',
104118
];
105119

106-
$customText = 'Custom Text';
107-
$customTitle = 'Custom Title';
108-
109120
$data = $emptyData;
110-
$data['anchor_text'] = $customText;
111-
yield 'text is set' => [
112-
$customText,
121+
$data['anchor_text'] = self::$defaults['custom_text'];
122+
yield self::$tests['text'] => [
123+
self::$defaults['custom_text'],
113124
$data,
114125
];
115126

116127
$data = $emptyData;
117-
$data['title'] = $customTitle;
118-
yield 'title is set' => [
119-
$customTitle,
128+
$data['title'] = self::$defaults['custom_title'];
129+
yield self::$tests['title'] => [
130+
self::$defaults['custom_title'],
120131
$data,
121132
];
122133

123134
$data = $emptyData;
124135
$data['page_id'] = 1;
125-
yield 'page_id is set' => [
136+
yield self::$tests['page_id'] => [
126137
'404 Not Found 1',
127138
$data,
128139
];
129140

130141
$data = $emptyData;
131142
$data['href'] = 'home';
132-
yield 'href is set' => [
143+
yield self::$tests['href'] => [
133144
'Home page',
134145
$data,
135146
];
136147

137-
yield 'empty' => [
148+
yield self::$tests['empty'] => [
138149
null,
139150
$emptyData,
140151
];
141152

142-
yield 'no data set' => [
153+
yield self::$tests['no_data'] => [
143154
null,
144155
[],
145156
];

0 commit comments

Comments
 (0)