added extend Action back, updated tests

Author: Rudolf Vince

app/code/Magento/Wishlist/Controller/Shared/Allcart.php

@@ -8,14 +8,16 @@
 
 namespace Magento\Wishlist\Controller\Shared;
 
+use Magento\Framework\App\Action\Action;
+use Magento\Framework\App\Action\Context;
 use Magento\Framework\App\Action\HttpGetActionInterface;
 use Magento\Framework\App\RequestInterface;
-use Magento\Framework\Controller\ResultFactory;
 use Magento\Framework\Controller\Result\Forward;
 use Magento\Framework\Controller\Result\Redirect;
+use Magento\Framework\Controller\ResultFactory;
 use Magento\Wishlist\Model\ItemCarrier;
 
-class Allcart implements HttpGetActionInterface
+class Allcart extends Action implements HttpGetActionInterface
 {
     /**
      * @var ItemCarrier
@@ -38,12 +40,14 @@ class Allcart implements HttpGetActionInterface
     private $resultFactory;
 
     /**
+     * @param Context $context
      * @param ItemCarrier $itemCarrier
      * @param RequestInterface $request
      * @param ResultFactory $resultFactory
      * @param WishlistProvider $wishlistProvider
      */
     public function __construct(
+        Context $context,
         ItemCarrier $itemCarrier,
         RequestInterface $request,
         ResultFactory $resultFactory,
@@ -53,6 +57,7 @@ public function __construct(
         $this->request = $request;
         $this->resultFactory = $resultFactory;
         $this->wishlistProvider = $wishlistProvider;
+        parent::__construct($context);
     }
 
     /**

app/code/Magento/Wishlist/Controller/Shared/Cart.php

@@ -11,14 +11,16 @@
 use Magento\Catalog\Model\Product\Exception as ProductException;
 use Magento\Checkout\Helper\Cart as CartHelper;
 use Magento\Checkout\Model\Cart as CustomerCart;
+use Magento\Framework\App\Action\Action;
+use Magento\Framework\App\Action\Context as ActionContext;
 use Magento\Framework\App\Action\HttpGetActionInterface;
 use Magento\Framework\App\RequestInterface;
-use Magento\Framework\Controller\ResultFactory;
+use Magento\Framework\App\Response\RedirectInterface;
 use Magento\Framework\Controller\Result\Redirect as ResultRedirect;
+use Magento\Framework\Controller\ResultFactory;
 use Magento\Framework\Escaper;
 use Magento\Framework\Exception\LocalizedException;
 use Magento\Framework\Message\ManagerInterface as MessageManagerInterface;
-use Magento\Framework\App\Response\RedirectInterface;
 use Magento\Wishlist\Model\Item;
 use Magento\Wishlist\Model\Item\OptionFactory;
 use Magento\Wishlist\Model\ItemFactory;
@@ -29,7 +31,7 @@
  *
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
-class Cart implements HttpGetActionInterface
+class Cart extends Action implements HttpGetActionInterface
 {
     /**
      * @var CustomerCart
@@ -77,6 +79,7 @@ class Cart implements HttpGetActionInterface
     private $resultFactory;
 
     /**
+     * @param ActionContext $context
      * @param CustomerCart $cart
      * @param OptionFactory $optionFactory
      * @param ItemFactory $itemFactory
@@ -88,6 +91,7 @@ class Cart implements HttpGetActionInterface
      * @param ResultFactory $resultFactory
      */
     public function __construct(
+        ActionContext $context,
         CustomerCart $cart,
         OptionFactory $optionFactory,
         ItemFactory $itemFactory,
@@ -107,6 +111,7 @@ public function __construct(
         $this->redirect = $redirect;
         $this->messageManager = $messageManager;
         $this->resultFactory = $resultFactory;
+        parent::__construct($context);
     }
 
     /**

app/code/Magento/Wishlist/Test/Unit/Controller/Shared/AllcartTest.php

@@ -27,14 +27,14 @@ class AllcartTest extends TestCase
     protected $allcartController;
 
     /**
-     * @var Context
+     * @var WishlistProvider|MockObject
      */
-    protected $context;
+    protected $wishlistProviderMock;
 
     /**
-     * @var WishlistProvider|MockObject
+     * @var Context|MockObject
      */
-    protected $wishlistProviderMock;
+    protected $contextMock;
 
     /**
      * @var ItemCarrier|MockObject
@@ -74,6 +74,9 @@ protected function setUp(): void
         $this->itemCarrierMock = $this->getMockBuilder(ItemCarrier::class)
             ->disableOriginalConstructor()
             ->getMock();
+        $this->contextMock = $this->getMockBuilder(Context::class)
+            ->disableOriginalConstructor()
+            ->getMock();
         $this->wishlistMock = $this->getMockBuilder(Wishlist::class)
             ->disableOriginalConstructor()
             ->getMock();
@@ -100,6 +103,7 @@ protected function setUp(): void
             );
 
         $this->allcartController = new Allcart(
+            $this->contextMock,
             $this->itemCarrierMock,
             $this->requestMock,
             $this->resultFactoryMock,

app/code/Magento/Wishlist/Test/Unit/Controller/Shared/CartTest.php

@@ -178,6 +178,7 @@ protected function setUp(): void
             ->getMock();
 
         $this->model = new SharedCart(
+            $this->context,
             $this->cart,
             $this->optionFactory,
             $this->itemFactory,