Skip to content

Commit c3b4b3e

Browse files
committed
ACP2E-1700: Updating URL Suffix configurations too many times at once will result in mismatch URL suffix in url_rewrite table and the configurations
1 parent 151ec11 commit c3b4b3e

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

app/code/Magento/Config/Test/Unit/Model/ConfigTest.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,8 @@ protected function setUp(): void
175175
*/
176176
public function testSaveDoesNotDoAnythingIfGroupsAreNotPassed(): void
177177
{
178+
$this->appConfigMock->expects($this->never())
179+
->method('reinit');
178180
$this->configLoaderMock->expects($this->never())->method('getConfigByPath');
179181
$this->model->save();
180182
}
@@ -184,6 +186,8 @@ public function testSaveDoesNotDoAnythingIfGroupsAreNotPassed(): void
184186
*/
185187
public function testSaveEmptiesNonSetArguments(): void
186188
{
189+
$this->appConfigMock->expects($this->never())
190+
->method('reinit');
187191
$this->structureReaderMock->expects($this->never())->method('getConfiguration');
188192
$this->assertNull($this->model->getSection());
189193
$this->assertNull($this->model->getWebsite());
@@ -199,6 +203,8 @@ public function testSaveEmptiesNonSetArguments(): void
199203
*/
200204
public function testSaveToCheckAdminSystemConfigChangedSectionEvent(): void
201205
{
206+
$this->appConfigMock->expects($this->exactly(2))
207+
->method('reinit');
202208
$transactionMock = $this->createMock(Transaction::class);
203209

204210
$this->transFactoryMock->expects($this->any())->method('create')->willReturn($transactionMock);
@@ -227,6 +233,8 @@ public function testSaveToCheckAdminSystemConfigChangedSectionEvent(): void
227233
*/
228234
public function testDoNotSaveReadOnlyFields(): void
229235
{
236+
$this->appConfigMock->expects($this->exactly(2))
237+
->method('reinit');
230238
$transactionMock = $this->createMock(Transaction::class);
231239
$this->transFactoryMock->expects($this->any())->method('create')->willReturn($transactionMock);
232240

@@ -265,6 +273,8 @@ public function testDoNotSaveReadOnlyFields(): void
265273
*/
266274
public function testSaveToCheckScopeDataSet(): void
267275
{
276+
$this->appConfigMock->expects($this->exactly(2))
277+
->method('reinit');
268278
$transactionMock = $this->createMock(Transaction::class);
269279
$this->transFactoryMock->expects($this->any())->method('create')->willReturn($transactionMock);
270280

0 commit comments

Comments
 (0)