ACP2E-3657: Weak Password Management

Author: soklymeach

app/code/Magento/User/Controller/Adminhtml/User/Save.php

@@ -140,6 +140,8 @@ public function execute()
             }
             $this->redirectToEdit($model, $data);
         }
+
+        return $this->getResponse();
     }
 
     /**

app/code/Magento/User/Test/Unit/Controller/Adminhtml/User/SaveTest.php

@@ -14,7 +14,6 @@
 use Magento\Framework\App\ResponseInterface;
 use Magento\Framework\Message\ManagerInterface;
 use Magento\Framework\ObjectManagerInterface;
-use Magento\Framework\Registry;
 use Magento\User\Controller\Adminhtml\User\Save;
 use Magento\User\Model\User;
 use Magento\User\Model\UserFactory;
@@ -63,11 +62,6 @@ class SaveTest extends TestCase
      */
     private $contextMock;
 
-    /**
-     * @var Registry|MockObject
-     */
-    private $registryMock;
-
     protected function setUp(): void
     {
         $this->requestMock = $this->createMock(Http::class);
@@ -83,8 +77,9 @@ protected function setUp(): void
             ->disableOriginalConstructor()
             ->addMethods(['setUserData'])
             ->getMock();
+        // phpcs:ignore Magento2.Deprecation.Registry -- Required for testing core dependency
+        $registryMock = $this->createMock(\Magento\Framework\Registry::class);
         $this->contextMock = $this->createMock(Context::class);
-        $this->registryMock = $this->createMock(Registry::class);
         $this->userFactoryMock->expects($this->once())
             ->method('create')
             ->willReturn($this->userModelMock);
@@ -112,7 +107,7 @@ protected function setUp(): void
             ->setConstructorArgs([
                 'context' => $this->contextMock,
                 'userFactory' => $this->userFactoryMock,
-                'coreRegistry' => $this->registryMock
+                'coreRegistry' => $registryMock
             ])
             ->onlyMethods(['redirectToEdit'])
             ->getMock();