Added lines *Deleted unnecessary uses

Author: sedonik

app/code/Magento/Customer/Test/Unit/Block/Form/RegisterTest.php

@@ -64,6 +64,7 @@ protected function setUp()
         $this->newsletterConfig = $this->createMock(\Magento\Newsletter\Model\Config::class);
         $context = $this->createMock(\Magento\Framework\View\Element\Template\Context::class);
         $context->expects($this->any())->method('getScopeConfig')->will($this->returnValue($this->_scopeConfig));
+
         $this->_block = new \Magento\Customer\Block\Form\Register(
             $context,
             $this->directoryHelperMock,
@@ -296,13 +297,15 @@ public function testIsNewsletterEnabled($isNewsletterEnabled, $isNewsletterActiv
         )->will(
             $this->returnValue($isNewsletterEnabled)
         );
+
         $this->newsletterConfig->expects(
             $this->any()
         )->method(
             'isActive'
         )->will(
             $this->returnValue($isNewsletterActive)
         );
+
         $this->assertEquals($expectedValue, $this->_block->isNewsletterEnabled());
     }
 

app/code/Magento/Newsletter/Test/Unit/Observer/PredispatchNewsletterObserverTest.php

@@ -88,17 +88,21 @@ public function testNewsletterEnabled() : void
             ->disableOriginalConstructor()
             ->setMethods(['getResponse', 'getData', 'setRedirect'])
             ->getMockForAbstractClass();
+
         $this->newsletterConfig->expects($this->once())
             ->method('isActive')
             ->with(ScopeInterface::SCOPE_STORE)
             ->willReturn(true);
+
         $observerMock->expects($this->never())
             ->method('getData')
             ->with('controller_action')
             ->willReturnSelf();
+
         $observerMock->expects($this->never())
             ->method('getResponse')
             ->willReturnSelf();
+
         $this->assertNull($this->mockObject->execute($observerMock));
     }
 
@@ -111,27 +115,34 @@ public function testNewsletterDisabled() : void
             ->disableOriginalConstructor()
             ->setMethods(['getControllerAction', 'getResponse'])
             ->getMockForAbstractClass();
+
         $this->newsletterConfig->expects($this->once())
             ->method('isActive')
             ->with(ScopeInterface::SCOPE_STORE)
             ->willReturn(false);
+
         $expectedRedirectUrl = 'https://test.com/index';
         $this->configMock->expects($this->once())
             ->method('getValue')
             ->with('web/default/no_route', ScopeInterface::SCOPE_STORE)
             ->willReturn($expectedRedirectUrl);
+
         $this->urlMock->expects($this->once())
             ->method('getUrl')
             ->willReturn($expectedRedirectUrl);
+
         $observerMock->expects($this->once())
             ->method('getControllerAction')
             ->willReturnSelf();
+
         $observerMock->expects($this->once())
             ->method('getResponse')
             ->willReturn($this->responseMock);
+
         $this->responseMock->expects($this->once())
             ->method('setRedirect')
             ->with($expectedRedirectUrl);
+
         $this->assertNull($this->mockObject->execute($observerMock));
     }
 }

dev/tests/integration/testsuite/Magento/Customer/_files/customer_subscribe.php

@@ -3,8 +3,6 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
-use Magento\Customer\Model\CustomerRegistry;
-
 $resourceConfig = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()
                     ->create(\Magento\Config\Model\ResourceModel\Config::class);