Replace Preference with Plugin

Author: fredden

…ramework/ControllerActionPredispatch.php …/BypassTwoFactorAuthForTestFramework.phpTwoFactorAuth/TestFramework/ControllerActionPredispatch.php renamed to TwoFactorAuth/Plugin/BypassTwoFactorAuthForTestFramework.php TwoFactorAuth/TestFramework/ControllerActionPredispatch.php renamed to TwoFactorAuth/Plugin/BypassTwoFactorAuthForTestFramework.php

@@ -6,32 +6,33 @@
 
 declare(strict_types=1);
 
-namespace Magento\TwoFactorAuth\TestFramework;
+namespace Magento\TwoFactorAuth\Plugin;
 
 use Magento\Backend\App\AbstractAction;
 use Magento\Framework\Event\Observer;
-use Magento\TwoFactorAuth\Observer\ControllerActionPredispatch as ParentObserver;
+use Magento\TestFramework\Request;
+use Magento\TwoFactorAuth\Observer\ControllerActionPredispatch;
 
 /**
- * Observer that allows integration controller tests that are not aware of 2FA to run.
+ * Plugin that allows integration controller tests that are not aware of 2FA to run.
  */
-class ControllerActionPredispatch extends ParentObserver
+class BypassTwoFactorAuthForTestFramework
 {
-    /**
-     * @inheritDoc
-     */
-    public function execute(Observer $observer)
-    {
+    public function aroundExecute(
+        ControllerActionPredispatch $subject,
+        Observer $observer,
+        callable $proceed
+    ) : void {
         /** @var $controllerAction AbstractAction */
         $controllerAction = $observer->getEvent()->getData('controller_action');
         if (method_exists($controllerAction, 'getRequest')
-            && $controllerAction->getRequest() instanceof \Magento\TestFramework\Request
+            && $controllerAction->getRequest() instanceof Request
             && !$controllerAction->getRequest()->getParam('tfa_enabled')
         ) {
             //Hack that allows integration controller tests that are not aware of 2FA to run
             return;
         }
 
-        parent::execute($observer);
+        $proceed($observer);
     }
 }

TwoFactorAuth/etc/di.xml

@@ -87,6 +87,10 @@
         <plugin name="fixDefaultUrl" type="Magento\TwoFactorAuth\Plugin\FirstAvailableMenu"/>
     </type>
 
+    <type name="Magento\TwoFactorAuth\Observer\ControllerActionPredispatch">
+        <plugin name="bypass_twofactorauth_testframework" type="Magento\TwoFactorAuth\Plugin\BypassTwoFactorAuthForTestFramework" />
+    </type>
+
     <virtualType name="Magento\TwoFactorAuth\Model\Provider\Google" type="Magento\TwoFactorAuth\Model\Provider">
         <arguments>
             <argument name="engine" xsi:type="object">Magento\TwoFactorAuth\Model\Provider\Engine\Google</argument>
@@ -159,5 +163,4 @@
     <preference for="Magento\TwoFactorAuth\Api\UserConfigRequestManagerInterface" type="Magento\TwoFactorAuth\Model\UserConfig\UserConfigRequestManager" />
     <preference for="Magento\TwoFactorAuth\Api\UserConfigTokenManagerInterface" type="Magento\TwoFactorAuth\Model\UserConfig\SignedTokenManager" />
     <preference for="Magento\TwoFactorAuth\Api\UserNotifierInterface" type="Magento\TwoFactorAuth\Model\EmailUserNotifier" />
-    <preference for="Magento\TwoFactorAuth\Observer\ControllerActionPredispatch" type="Magento\TwoFactorAuth\TestFramework\ControllerActionPredispatch" />
 </config>