Skip to content

Commit a34c5d6

Browse files
committed
MAGETWO-66793: [Backport] Merchant can't unsubscribe Customer from Newsletter in Admin
- Update unit and integration tests;
1 parent 8f6458f commit a34c5d6

File tree

3 files changed

+7
-1
lines changed

3 files changed

+7
-1
lines changed

app/code/Magento/Customer/Controller/Adminhtml/Index/Save.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@
1313
use Magento\Customer\Model\Metadata\Form;
1414
use Magento\Framework\Exception\LocalizedException;
1515

16+
/**
17+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
18+
*/
1619
class Save extends \Magento\Customer\Controller\Adminhtml\Index
1720
{
1821
/**

app/code/Magento/Newsletter/Test/Unit/Model/SubscriberTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55
*/
66
namespace Magento\Newsletter\Test\Unit\Model;
77

8+
/**
9+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
10+
*/
811
class SubscriberTest extends \PHPUnit_Framework_TestCase
912
{
1013
/**

dev/tests/integration/testsuite/Magento/Customer/Controller/Adminhtml/IndexTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ public function testSaveActionExistingCustomerUnsubscribeNewsletter()
347347
'firstname' => 'test firstname',
348348
'lastname' => 'test lastname',
349349
],
350-
'subscription' => 'false'
350+
'subscription' => '0'
351351
];
352352
$this->getRequest()->setPostValue($post);
353353
$this->getRequest()->setParam('id', 1);

0 commit comments

Comments
 (0)