Skip to content

Commit a77faf3

Browse files
committed
Refactor code to pass review
1 parent dd7479d commit a77faf3

File tree

1 file changed

+39
-12
lines changed

1 file changed

+39
-12
lines changed

app/code/Magento/Search/Test/Unit/Ui/Component/Listing/Column/SynonymActionsTest.php

Lines changed: 39 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,21 @@
1616

1717
class SynonymActionsTest extends TestCase
1818
{
19+
/**
20+
* Stub synonym group id
21+
*/
22+
private const STUB_SYNONYM_GROUP_ID = 1;
23+
24+
/**
25+
* Synonym group delete url
26+
*/
27+
private const SYNONYM_GROUP_DELETE_URL = 'http://localhost/magento2/admin/search/synonyms/delete/group_id/%d';
28+
29+
/**
30+
* Synonym group edit url
31+
*/
32+
private const SYNONYM_GROUP_EDIT_URL = 'http://localhost/magento2/admin/search/synonyms/edit/group_id/%d';
33+
1934
/**
2035
* @var SynonymActions
2136
*/
@@ -72,31 +87,38 @@ public function testPrepareDataSourceWithItems()
7287
'data' => [
7388
'items' => [
7489
[
75-
'group_id' => 1
90+
'group_id' => self::STUB_SYNONYM_GROUP_ID
7691
]
7792
]
7893
]
7994
];
95+
8096
$expected = [
8197
'data' => [
8298
'items' => [
8399
[
84-
'group_id' => 1,
100+
'group_id' => self::STUB_SYNONYM_GROUP_ID,
85101
'actions' => [
86102
'delete' => [
87-
'href' => 'http://localhost/magento2/admin/search/synonyms/delete/group_id/1',
103+
'href' => sprintf(
104+
self::SYNONYM_GROUP_DELETE_URL,
105+
self::STUB_SYNONYM_GROUP_ID
106+
),
88107
'label' => (string)__('Delete'),
89108
'confirm' => [
90109
'title' => (string)__('Delete'),
91110
'message' => (string)__(
92111
'Are you sure you want to delete synonym group with id: %1?',
93-
1
112+
self::STUB_SYNONYM_GROUP_ID
94113
)
95114
],
96115
'__disableTmpl' => true
97116
],
98117
'edit' => [
99-
'href' => 'http://localhost/magento2/admin/search/synonyms/edit/group_id/1',
118+
'href' => sprintf(
119+
self::SYNONYM_GROUP_EDIT_URL,
120+
self::STUB_SYNONYM_GROUP_ID
121+
),
100122
'label' => (string)__('View/Edit'),
101123
'__disableTmpl' => true
102124
]
@@ -106,13 +128,18 @@ public function testPrepareDataSourceWithItems()
106128
]
107129
];
108130

109-
$this->urlBuilderMock->expects($this->at(0))->method('getUrl')
110-
->with(SynonymActions::SYNONYM_URL_PATH_DELETE, ['group_id' => 1])
111-
->willReturn('http://localhost/magento2/admin/search/synonyms/delete/group_id/1');
112-
113-
$this->urlBuilderMock->expects($this->at(1))->method('getUrl')
114-
->with(SynonymActions::SYNONYM_URL_PATH_EDIT, ['group_id' => 1])
115-
->willReturn('http://localhost/magento2/admin/search/synonyms/edit/group_id/1');
131+
$this->urlBuilderMock->method('getUrl')->will(
132+
$this->returnValueMap([
133+
[
134+
SynonymActions::SYNONYM_URL_PATH_DELETE, ['group_id' => self::STUB_SYNONYM_GROUP_ID],
135+
sprintf(self::SYNONYM_GROUP_DELETE_URL, self::STUB_SYNONYM_GROUP_ID)
136+
],
137+
[
138+
SynonymActions::SYNONYM_URL_PATH_EDIT, ['group_id' => self::STUB_SYNONYM_GROUP_ID],
139+
sprintf(self::SYNONYM_GROUP_EDIT_URL, self::STUB_SYNONYM_GROUP_ID)
140+
]
141+
])
142+
);
116143

117144
/**
118145
* Assert Result

0 commit comments

Comments
 (0)