magento · magento-team · Jul 17, 2015 · Jun 26, 2015
diff --git a/app/code/Magento/AdminNotification/Controller/Adminhtml/Notification.php b/app/code/Magento/AdminNotification/Controller/Adminhtml/Notification.php
@@ -7,7 +7,7 @@
  */
 namespace Magento\AdminNotification\Controller\Adminhtml;
 
-class Notification extends \Magento\Backend\App\AbstractAction
+abstract class Notification extends \Magento\Backend\App\AbstractAction
 {
     /**
      * @return bool

diff --git a/app/code/Magento/Backend/App/Action.php b/app/code/Magento/Backend/App/Action.php
@@ -12,6 +12,6 @@
 /**
  * @SuppressWarnings(PHPMD.NumberOfChildren)
  */
-class Action extends \Magento\Backend\App\AbstractAction
+abstract class Action extends \Magento\Backend\App\AbstractAction
 {
 }
diff --git a/app/code/Magento/Backend/Controller/Adminhtml/Auth.php b/app/code/Magento/Backend/Controller/Adminhtml/Auth.php
@@ -10,7 +10,7 @@
 /**
  * Auth backend controller
  */
-class Auth extends AbstractAction
+abstract class Auth extends AbstractAction
 {
     /**
      * Check if user has permissions to access this controller

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/Cache.php b/app/code/Magento/Backend/Controller/Adminhtml/Cache.php
@@ -8,7 +8,7 @@
 use Magento\Backend\App\Action;
 use Magento\Framework\Exception\LocalizedException;
 
-class Cache extends Action
+abstract class Cache extends Action
 {
     /**
      * @var \Magento\Framework\App\Cache\TypeListInterface

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/Dashboard.php b/app/code/Magento/Backend/Controller/Adminhtml/Dashboard.php
@@ -11,7 +11,7 @@
  */
 namespace Magento\Backend\Controller\Adminhtml;
 
-class Dashboard extends \Magento\Backend\App\Action
+abstract class Dashboard extends \Magento\Backend\App\Action
 {
     /**
      * @return bool

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/Index.php b/app/code/Magento/Backend/Controller/Adminhtml/Index.php
@@ -10,7 +10,7 @@
 /**
  * Index backend controller
  */
-class Index extends AbstractAction
+abstract class Index extends AbstractAction
 {
     /**
      * Check if user has permissions to access this controller

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/System.php b/app/code/Magento/Backend/Controller/Adminhtml/System.php
@@ -12,7 +12,7 @@
  *
  * @author     Magento Core Team <core@magentocommerce.com>
  */
-class System extends AbstractAction
+abstract class System extends AbstractAction
 {
     /**
      * @return bool

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/System/Account.php b/app/code/Magento/Backend/Controller/Adminhtml/System/Account.php
@@ -12,7 +12,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Account extends Action
+abstract class Account extends Action
 {
     /**
      * @return bool

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/System/Design.php b/app/code/Magento/Backend/Controller/Adminhtml/System/Design.php
@@ -7,7 +7,7 @@
 
 use Magento\Backend\App\Action;
 
-class Design extends Action
+abstract class Design extends Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Backend/Controller/Adminhtml/System/Store.php b/app/code/Magento/Backend/Controller/Adminhtml/System/Store.php
@@ -17,7 +17,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Store extends Action
+abstract class Store extends Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Backup/Controller/Adminhtml/Index.php b/app/code/Magento/Backup/Controller/Adminhtml/Index.php
@@ -10,7 +10,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Index extends \Magento\Backend\App\Action
+abstract class Index extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Category.php b/app/code/Magento/Catalog/Controller/Adminhtml/Category.php
@@ -8,7 +8,7 @@
 /**
  * Catalog category controller
  */
-class Category extends \Magento\Backend\App\Action
+abstract class Category extends \Magento\Backend\App\Action
 {
     /**
      * Initialize requested category and put it into registry.

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Category/Widget.php b/app/code/Magento/Catalog/Controller/Adminhtml/Category/Widget.php
@@ -12,7 +12,7 @@
  *
  * @author     Magento Core Team <core@magentocommerce.com>
  */
-class Widget extends \Magento\Backend\App\Action
+abstract class Widget extends \Magento\Backend\App\Action
 {
     /**
      * @var \Magento\Framework\View\LayoutFactory

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product.php
@@ -11,7 +11,7 @@
  * Catalog product controller
  * @SuppressWarnings(PHPMD.NumberOfChildren)
  */
-class Product extends \Magento\Backend\App\Action
+abstract class Product extends \Magento\Backend\App\Action
 {
     /**
      * @var Product\Builder

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/AbstractProductGrid.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/AbstractProductGrid.php
@@ -10,7 +10,7 @@
 use Magento\Catalog\Controller\Adminhtml\Product;
 use Magento\Framework\Controller\Result;
 
-class AbstractProductGrid extends Product
+abstract class AbstractProductGrid extends Product
 {
     /**
      * @var \Magento\Framework\View\Result\LayoutFactory

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/Action/Attribute.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/Action/Attribute.php
@@ -13,7 +13,7 @@
 /**
  * Adminhtml catalog product action attribute update controller
  */
-class Attribute extends Action
+abstract class Attribute extends Action
 {
     /**
      *  @var \Magento\Catalog\Helper\Product\Edit\Action\Attribute

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/Attribute.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/Attribute.php
@@ -12,7 +12,7 @@
 use Magento\Framework\Controller\Result;
 use Magento\Framework\View\Result\PageFactory;
 
-class Attribute extends \Magento\Backend\App\Action
+abstract class Attribute extends \Magento\Backend\App\Action
 {
     /**
      * @var \Magento\Framework\Cache\FrontendInterface

diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/Set.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/Set.php
@@ -10,7 +10,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Set extends \Magento\Backend\App\Action
+abstract class Set extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Catalog/Controller/Product.php b/app/code/Magento/Catalog/Controller/Product.php
@@ -10,7 +10,7 @@
 use Magento\Catalog\Controller\Product\View\ViewInterface;
 use Magento\Catalog\Model\Product as ModelProduct;
 
-class Product extends \Magento\Framework\App\Action\Action implements ViewInterface
+abstract class Product extends \Magento\Framework\App\Action\Action implements ViewInterface
 {
     /**
      * Initialize requested product object

diff --git a/app/code/Magento/Catalog/Controller/Product/Compare.php b/app/code/Magento/Catalog/Controller/Product/Compare.php
@@ -15,7 +15,7 @@
  * @SuppressWarnings(PHPMD.LongVariable)
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
-class Compare extends \Magento\Framework\App\Action\Action
+abstract class Compare extends \Magento\Framework\App\Action\Action
 {
     /**
      * Customer id

diff --git a/app/code/Magento/CatalogRule/Controller/Adminhtml/Promo/Catalog.php b/app/code/Magento/CatalogRule/Controller/Adminhtml/Promo/Catalog.php
@@ -18,7 +18,7 @@
 use Magento\Framework\Registry;
 use Magento\Framework\Stdlib\DateTime\Filter\Date;
 
-class Catalog extends Action
+abstract class Catalog extends Action
 {
     /**
      * Dirty rules notice message

diff --git a/app/code/Magento/CatalogRule/Controller/Adminhtml/Promo/Widget.php b/app/code/Magento/CatalogRule/Controller/Adminhtml/Promo/Widget.php
@@ -7,7 +7,7 @@
 
 use Magento\Backend\App\Action;
 
-class Widget extends Action
+abstract class Widget extends Action
 {
     /**
      * @return bool

diff --git a/app/code/Magento/CatalogWidget/Controller/Adminhtml/Product/Widget.php b/app/code/Magento/CatalogWidget/Controller/Adminhtml/Product/Widget.php
@@ -10,7 +10,7 @@
 /**
  * Class Widget
  */
-class Widget extends Action
+abstract class Widget extends Action
 {
     /**
      * @return bool

diff --git a/app/code/Magento/Checkout/Controller/Cart.php b/app/code/Magento/Checkout/Controller/Cart.php
@@ -12,7 +12,7 @@
 /**
  * Shopping cart controller
  */
-class Cart extends \Magento\Framework\App\Action\Action implements ViewInterface
+abstract class Cart extends \Magento\Framework\App\Action\Action implements ViewInterface
 {
     /**
      * @var \Magento\Framework\App\Config\ScopeConfigInterface

diff --git a/app/code/Magento/Checkout/Controller/Onepage.php b/app/code/Magento/Checkout/Controller/Onepage.php
@@ -13,7 +13,7 @@
 /**
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
-class Onepage extends Action
+abstract class Onepage extends Action
 {
     /**
      * @var array

diff --git a/app/code/Magento/Checkout/Test/Unit/Controller/OnepageTest.php b/app/code/Magento/Checkout/Test/Unit/Controller/OnepageTest.php
@@ -85,9 +85,9 @@ protected function setUp()
         $context->expects($this->once())
             ->method('getEventManager')
             ->willReturn($this->eventManager);
-
+        
         $this->controller = $objectManager->getObject(
-            'Magento\Checkout\Controller\Onepage',
+            'Magento\Checkout\Test\Unit\Controller\Stub\OnepageStub',
             [
                 'context' => $context
             ]

diff --git a/app/code/Magento/Checkout/Test/Unit/Controller/Stub/OnepageStub.php b/app/code/Magento/Checkout/Test/Unit/Controller/Stub/OnepageStub.php
@@ -0,0 +1,12 @@
+<?php
+
+
+namespace Magento\Checkout\Test\Unit\Controller\Stub;
+
+class OnepageStub extends \Magento\Checkout\Controller\Onepage
+{
+    protected function execute()
+    {
+        // Empty method stub for test
+    }
+}
diff --git a/app/code/Magento/CheckoutAgreements/Controller/Adminhtml/Agreement.php b/app/code/Magento/CheckoutAgreements/Controller/Adminhtml/Agreement.php
@@ -5,7 +5,7 @@
  */
 namespace Magento\CheckoutAgreements\Controller\Adminhtml;
 
-class Agreement extends \Magento\Backend\App\Action
+abstract class Agreement extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Cms/Controller/Adminhtml/Block.php b/app/code/Magento/Cms/Controller/Adminhtml/Block.php
@@ -10,7 +10,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Block extends \Magento\Backend\App\Action
+abstract class Block extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Cms/Controller/Adminhtml/Wysiwyg/Images.php b/app/code/Magento/Cms/Controller/Adminhtml/Wysiwyg/Images.php
@@ -10,7 +10,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Images extends \Magento\Backend\App\Action
+abstract class Images extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Contact/Controller/Index.php b/app/code/Magento/Contact/Controller/Index.php
@@ -12,7 +12,7 @@
 /**
  * Contact index controller
  */
-class Index extends \Magento\Framework\App\Action\Action
+abstract class Index extends \Magento\Framework\App\Action\Action
 {
     /**
      * Recipient email config path

diff --git a/app/code/Magento/Contact/Test/Unit/Controller/IndexTest.php b/app/code/Magento/Contact/Test/Unit/Controller/IndexTest.php
@@ -51,7 +51,7 @@ public function setUp()
             )
         );
 
-        $this->_controller = new \Magento\Contact\Controller\Index(
+        $this->_controller = new \Magento\Contact\Test\Unit\Controller\Stub\IndexStub(
             $context,
             $this->getMock('\Magento\Framework\Mail\Template\TransportBuilder', [], [], '', false),
             $this->getMockForAbstractClass('\Magento\Framework\Translate\Inline\StateInterface', [], '', false),

diff --git a/app/code/Magento/Contact/Test/Unit/Controller/Stub/IndexStub.php b/app/code/Magento/Contact/Test/Unit/Controller/Stub/IndexStub.php
@@ -0,0 +1,12 @@
+<?php
+
+
+namespace Magento\Contact\Test\Unit\Controller\Stub;
+
+class IndexStub extends \Magento\Contact\Controller\Index
+{
+    protected function execute()
+    {
+        // Empty method stub for test
+    }
+}
diff --git a/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currency.php b/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currency.php
@@ -11,7 +11,7 @@
  */
 namespace Magento\CurrencySymbol\Controller\Adminhtml\System;
 
-class Currency extends \Magento\Backend\App\Action
+abstract class Currency extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currencysymbol.php b/app/code/Magento/CurrencySymbol/Controller/Adminhtml/System/Currencysymbol.php
@@ -11,7 +11,7 @@
  */
 namespace Magento\CurrencySymbol\Controller\Adminhtml\System;
 
-class Currencysymbol extends \Magento\Backend\App\Action
+abstract class Currencysymbol extends \Magento\Backend\App\Action
 {
     /**
      * Check the permission to run it

diff --git a/app/code/Magento/Customer/Controller/Account.php b/app/code/Magento/Customer/Controller/Account.php
@@ -16,7 +16,7 @@
  * @SuppressWarnings(PHPMD.NumberOfChildren)
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
-class Account extends \Magento\Framework\App\Action\Action
+abstract class Account extends \Magento\Framework\App\Action\Action
 {
     /**
      * List of actions that are allowed for not authorized users

diff --git a/app/code/Magento/Customer/Controller/Address.php b/app/code/Magento/Customer/Controller/Address.php
@@ -12,7 +12,7 @@
  *
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
-class Address extends \Magento\Framework\App\Action\Action
+abstract class Address extends \Magento\Framework\App\Action\Action
 {
     /**
      * @var \Magento\Customer\Model\Session

diff --git a/app/code/Magento/Customer/Controller/Adminhtml/Cart/Product/Composite/Cart.php b/app/code/Magento/Customer/Controller/Adminhtml/Cart/Product/Composite/Cart.php
@@ -13,7 +13,7 @@
  *
  * @author      Magento Core Team <core@magentocommerce.com>
  */
-class Cart extends \Magento\Backend\App\Action
+abstract class Cart extends \Magento\Backend\App\Action
 {
     /**
      * Customer we're working with

diff --git a/app/code/Magento/Customer/Controller/Adminhtml/Group.php b/app/code/Magento/Customer/Controller/Adminhtml/Group.php
@@ -11,7 +11,7 @@
 /**
  * Customer groups controller
  */
-class Group extends \Magento\Backend\App\Action
+abstract class Group extends \Magento\Backend\App\Action
 {
     /**
      * Core registry

diff --git a/app/code/Magento/Customer/Controller/Adminhtml/Index.php b/app/code/Magento/Customer/Controller/Adminhtml/Index.php
@@ -24,7 +24,7 @@
  * @SuppressWarnings(PHPMD.TooManyFields)
  * @SuppressWarnings(PHPMD.NumberOfChildren)
  */
-class Index extends \Magento\Backend\App\Action
+abstract class Index extends \Magento\Backend\App\Action
 {
     /**
      * @var \Magento\Framework\Validator

diff --git a/app/code/Magento/Customer/Controller/Adminhtml/System/Config/Validatevat.php b/app/code/Magento/Customer/Controller/Adminhtml/System/Config/Validatevat.php
@@ -10,7 +10,7 @@
  *
  * @author     Magento Core Team <core@magentocommerce.com>
  */
-class Validatevat extends \Magento\Backend\App\Action
+abstract class Validatevat extends \Magento\Backend\App\Action
 {
     /**
      * Perform customer VAT ID validation

diff --git a/app/code/Magento/Customer/Controller/Adminhtml/Wishlist/Product/Composite/Wishlist.php b/app/code/Magento/Customer/Controller/Adminhtml/Wishlist/Product/Composite/Wishlist.php
@@ -10,7 +10,7 @@
 /**
  * Catalog composite product configuration controller
  */
-class Wishlist extends \Magento\Backend\App\Action
+abstract class Wishlist extends \Magento\Backend\App\Action
 {
     /**
      * Wishlist we're working with.

diff --git a/app/code/Magento/DesignEditor/Controller/Adminhtml/System/Design/Editor.php b/app/code/Magento/DesignEditor/Controller/Adminhtml/System/Design/Editor.php
@@ -14,7 +14,7 @@
  *
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
-class Editor extends \Magento\Backend\App\Action
+abstract class Editor extends \Magento\Backend\App\Action
 {
     /**
      * @var \Magento\Theme\Model\Config