[PSR-2 Compliance] The Great @codingStandardsIgnoreFile Massacre

- revert files with nontrivial issues to mainline state for further investigation
- Step 0. Run command: git checkout bbe3ee0 -- lib/internal/Magento/Framework/Code/Test/Unit/_files/app/code/Magento/SomeModule/Model/SevenInterface.php lib/internal/Magento/Framework/ObjectManager/Test/Unit/Config/Reader/_files/ConfigDomMock.php lib/internal/Magento/Framework/Interception lib/internal/Magento/Framework/Model/Test/Unit/ResourceModel/Db/Collection/AbstractCollectionTest.php lib/internal/Magento/Framework/Code/ dev/tests/integration/testsuite/Magento/ImportExport/Model/Export/EntityAbstractTest.php dev/tests/integration/testsuite/Magento/Cms/Controller/RouterTest.php dev/tests/integration/testsuite/Magento/Framework/Interception/ dev/tests/static/get_github_changes.php setup/src/Magento/Setup/Test/Unit/Module/I18n/Parser/ParserTest.php  app/code/Magento/Vault/Api/Data/PaymentTokenInterfaceFactory.php app/code/Magento/Customer/Test/Unit/Model/ResourceModel/AddressTest.php dev/tests/integration/testsuite/Magento/Setup/Console/Command/_files/phrases/TestPhrases.php

Author: orlangur

app/code/Magento/Customer/Test/Unit/Model/ResourceModel/AddressTest.php

@@ -4,6 +4,7 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
 namespace Magento\Customer\Test\Unit\Model\ResourceModel;
 
 use Magento\Framework\Model\ResourceModel\Db\VersionControl\RelationComposite;
@@ -188,11 +189,7 @@ protected function prepareEavConfig()
             ->method('getBackend')
             ->willReturn(
                 $this->getMock(
-                    \Magento\Eav\Model\Entity\Attribute\Backend\AbstractBackend::class,
-                    [],
-                    [],
-                    '',
-                    false
+                    \Magento\Eav\Model\Entity\Attribute\Backend\AbstractBackend::class, [], [], '', false
                 )
             );
 

app/code/Magento/Vault/Api/Data/PaymentTokenInterfaceFactory.php

@@ -3,6 +3,7 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+// @codingStandardsIgnoreFile
 namespace Magento\Vault\Api\Data;
 
 /**

dev/tests/integration/testsuite/Magento/Cms/Controller/RouterTest.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Cms\Controller;
 
 /**
@@ -21,8 +23,7 @@ protected function setUp()
         $this->markTestIncomplete('MAGETWO-3393');
         $this->_model = new \Magento\Cms\Controller\Router(
             \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->get(
-                \Magento\Framework\App\ActionFactory::class
-            ),
+                \Magento\Framework\App\ActionFactory::class),
             new \Magento\Framework\Event\ManagerInterfaceStub(
                 $this->getMockForAbstractClass(\Magento\Framework\Event\InvokerInterface::class),
                 $this->getMock(\Magento\Framework\Event\Config::class, [], [], '', false),

dev/tests/integration/testsuite/Magento/Framework/Interception/AbstractPlugin.php

@@ -94,8 +94,10 @@ public function setUpInterceptionConfig($pluginConfig)
         $config->extend(
             [
                 'preferences' => [
-                    \Magento\Framework\Interception\PluginListInterface::class => \Magento\Framework\Interception\PluginList\PluginList::class,
-                    \Magento\Framework\Interception\ChainInterface::class => \Magento\Framework\Interception\Chain\Chain::class,
+                    \Magento\Framework\Interception\PluginListInterface::class =>
+                        \Magento\Framework\Interception\PluginList\PluginList::class,
+                    \Magento\Framework\Interception\ChainInterface::class =>
+                        \Magento\Framework\Interception\Chain\Chain::class,
                 ],
             ]
         );

dev/tests/integration/testsuite/Magento/Framework/Interception/Fixture/Intercepted.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Fixture;
 
 class Intercepted extends InterceptedParent implements InterceptedInterface

dev/tests/integration/testsuite/Magento/Framework/Interception/Fixture/InterceptedInterface.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Fixture;
 
 interface InterceptedInterface

dev/tests/integration/testsuite/Magento/Framework/Interception/Fixture/InterceptedParent.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Fixture;
 
 class InterceptedParent implements InterceptedParentInterface

dev/tests/integration/testsuite/Magento/Framework/Interception/Fixture/InterceptedParentInterface.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Fixture;
 
 interface InterceptedParentInterface

dev/tests/integration/testsuite/Magento/Framework/Interception/GeneralTest.php

@@ -14,14 +14,17 @@ class GeneralTest extends AbstractPlugin
     public function setUp()
     {
         $this->setUpInterceptionConfig(
-            [\Magento\Framework\Interception\Fixture\InterceptedInterface::class => [
+            [\Magento\Framework\Interception\Fixture\InterceptedInterface::class =>
+                    [
                         'plugins' => [
                             'first' => [
-                                'instance'  => \Magento\Framework\Interception\Fixture\Intercepted\InterfacePlugin::class,
+                                'instance'  =>
+                                    \Magento\Framework\Interception\Fixture\Intercepted\InterfacePlugin::class,
                                 'sortOrder' => 10,
                             ],
                         ],
-                    ], \Magento\Framework\Interception\Fixture\Intercepted::class  => [
+                    ], \Magento\Framework\Interception\Fixture\Intercepted::class  =>
+                    [
                         'plugins' => [
                             'second' => [
                                 'instance'  => \Magento\Framework\Interception\Fixture\Intercepted\Plugin::class,

dev/tests/integration/testsuite/Magento/ImportExport/Model/Export/EntityAbstractTest.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 /**
  * Test for abstract export model
  */
@@ -102,7 +104,8 @@ public function testFilterAttributeCollection()
 /**
  * Stub abstract class which provide to change protected property "$_disabledAttrs" and test methods depended on it
  */
-abstract class Stub_Magento_ImportExport_Model_Export_AbstractEntity extends \Magento\ImportExport\Model\Export\AbstractEntity
+abstract class Stub_Magento_ImportExport_Model_Export_AbstractEntity
+    extends \Magento\ImportExport\Model\Export\AbstractEntity
 {
     public function __construct(
         \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig,

dev/tests/integration/testsuite/Magento/Setup/Console/Command/_files/phrases/TestPhrases.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 use Magento\Framework\Phrase;
 
 /**

dev/tests/static/get_github_changes.php

@@ -9,6 +9,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 define(
     'USAGE',
     <<<USAGE
@@ -283,8 +285,7 @@ public function compareChanges($remoteAlias, $remoteBranch)
         $result = $this->call(sprintf('log %s/%s..HEAD  --name-status --oneline', $remoteAlias, $remoteBranch));
 
         return is_array($result)
-            ? $this->filterChangedFiles(
-                $result,
+            ? $this->filterChangedFiles($result,
                 $remoteAlias,
                 $remoteBranch
             )
@@ -313,11 +314,8 @@ protected function filterChangedFiles(array $changes, $remoteAlias, $remoteBranc
                     $fileName = trim($fileName);
                     if (!in_array($fileName, $filteredChanges) && is_file($this->workTree . '/' . $fileName)) {
                         $result = $this->call(sprintf(
-                            'diff HEAD %s/%s -- %s',
-                            $remoteAlias,
-                            $remoteBranch,
-                            $this->workTree . '/' . $fileName
-                        ));
+                                'diff HEAD %s/%s -- %s', $remoteAlias, $remoteBranch, $this->workTree . '/' . $fileName)
+                        );
                         if ($result) {
                             if (!(isset($this->changedContentFiles[$fileName]))) {
                                 $this->setChangedContentFile($result, $fileName);

lib/internal/Magento/Framework/Code/Minifier/Adapter/Css/CSSmin.php

@@ -5,8 +5,8 @@
  */
 namespace Magento\Framework\Code\Minifier\Adapter\Css;
 
-use Magento\Framework\Code\Minifier\AdapterInterface;
 use tubalmartin\CssMin\Minifier as CssMinLibrary;
+use Magento\Framework\Code\Minifier\AdapterInterface;
 
 /**
  * Adapter for CSSmin library

lib/internal/Magento/Framework/Code/Test/Unit/GeneratedFilesTest.php

@@ -50,6 +50,7 @@ protected function setUp()
      */
     public function testCleanGeneratedFiles($getPathMap, $isDirectoryMap, $deleteMap)
     {
+
         $this->writeInterface
             ->expects($this->any())
             ->method('isExist')

lib/internal/Magento/Framework/Code/Test/Unit/Generator/InterfaceGeneratorTest.php

@@ -129,12 +129,14 @@ public function generateDataProvider()
             'Non public interface method  exception' => [
                 'additionalMethodsData' => ['testMethod2' => ['visibility' => 'protected']],
                 'expectedException' => '\LogicException',
-                'expectedExceptionMessage' => "Interface method visibility can only be 'public'. Method name: 'testMethod2'"
+                'expectedExceptionMessage' =>
+                    "Interface method visibility can only be 'public'. Method name: 'testMethod2'"
             ],
             '"abstract" usage exception' => [
                 'additionalMethodsData' => ['testMethod1' => ['abstract' => true]],
                 'expectedException' => '\LogicException',
-                'expectedExceptionMessage' => "'abstract' modifier cannot be used for interface method. Method name: 'testMethod1'"
+                'expectedExceptionMessage' =>
+                    "'abstract' modifier cannot be used for interface method. Method name: 'testMethod1'"
             ],
         ];
     }

lib/internal/Magento/Framework/Code/Test/Unit/GeneratorTest.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Code\Test\Unit;
 
 use Magento\Framework\Code\Generator;

lib/internal/Magento/Framework/Code/Test/Unit/Reader/_files/ClassesForArgumentsReader.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 class ClassWithAllArgumentTypes
 {
     const DEFAULT_VALUE = 'Const Value';

lib/internal/Magento/Framework/Code/Test/Unit/Validator/ConstructorArgumentTypesTest.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Code\Test\Unit\Validator;
 
 class ConstructorArgumentTypesTest extends \PHPUnit_Framework_TestCase

lib/internal/Magento/Framework/Code/Test/Unit/Validator/_files/ClassesForArgumentSequence.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace ArgumentSequence;
 
 class ContextObject implements \Magento\Framework\ObjectManager\ContextInterface

lib/internal/Magento/Framework/Code/Test/Unit/Validator/_files/ClassesForConstructorIntegrity.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 class ClassA
 {
 }

lib/internal/Magento/Framework/Code/Test/Unit/Validator/_files/ClassesForContextAggregation.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 class ClassFirst
 {
 }

lib/internal/Magento/Framework/Code/Test/Unit/Validator/_files/ClassesForTypeDuplication.php

@@ -5,6 +5,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace TypeDuplication;
 
 interface ArgumentInterface

lib/internal/Magento/Framework/Code/Test/Unit/ValidatorTest.php

@@ -5,7 +5,7 @@
  */
 namespace Magento\Framework\Code\Test\Unit;
 
-use Magento\Framework\Code\Validator;
+use \Magento\Framework\Code\Validator;
 
 class ValidatorTest extends \PHPUnit_Framework_TestCase
 {

lib/internal/Magento/Framework/Code/Test/Unit/_files/app/code/Magento/SomeModule/Model/SevenInterface.php

@@ -4,8 +4,11 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\SomeModule\Model;
 
+use Magento\SomeModule\Model\Two\Test as TestTwo;
 use Magento\SomeModule\Model\Three\Test as TestThree;
 
 /**
@@ -19,13 +22,14 @@
  */
 interface SevenInterface extends \Magento\Framework\Code\Generator\CodeGeneratorInterface
 {
+
     /**
      * Method short description
      *
      * @param array $data
      * @return TestThree
      */
-    public static function testMethod1(array &$data = []);
+    public static function testMethod1(array &$data = array());
 
     /**
      * Method short description
@@ -39,4 +43,6 @@ public static function testMethod1(array &$data = []);
     public function testMethod2($data = 'test_default', $flag = true);
 
     public function testMethod3();
+
+
 }

lib/internal/Magento/Framework/Interception/Code/Generator/Interceptor.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Code\Generator;
 
 class Interceptor extends \Magento\Framework\Code\Generator\EntityAbstract

lib/internal/Magento/Framework/Interception/Config/Config.php

@@ -7,8 +7,8 @@
  */
 namespace Magento\Framework\Interception\Config;
 
-use Magento\Framework\Serialize\Serializer\Serialize;
 use Magento\Framework\Serialize\SerializerInterface;
+use Magento\Framework\Serialize\Serializer\Serialize;
 
 class Config implements \Magento\Framework\Interception\ConfigInterface
 {

lib/internal/Magento/Framework/Interception/ObjectManager/Config/Developer.php

@@ -9,8 +9,8 @@
 
 use Magento\Framework\Interception\ObjectManager\ConfigInterface;
 use Magento\Framework\ObjectManager\DefinitionInterface;
-use Magento\Framework\ObjectManager\InterceptableValidator;
 use Magento\Framework\ObjectManager\RelationsInterface;
+use Magento\Framework\ObjectManager\InterceptableValidator;
 
 class Developer extends \Magento\Framework\ObjectManager\Config\Config implements ConfigInterface
 {

lib/internal/Magento/Framework/Interception/PluginList/PluginList.php

@@ -10,13 +10,13 @@
 use Magento\Framework\Config\ReaderInterface;
 use Magento\Framework\Config\ScopeInterface;
 use Magento\Framework\Interception\DefinitionInterface;
-use Magento\Framework\Interception\ObjectManager\ConfigInterface;
 use Magento\Framework\Interception\PluginListInterface as InterceptionPluginList;
-use Magento\Framework\ObjectManager\DefinitionInterface as ClassDefinitions;
+use Magento\Framework\Interception\ObjectManager\ConfigInterface;
 use Magento\Framework\ObjectManager\RelationsInterface;
+use Magento\Framework\ObjectManager\DefinitionInterface as ClassDefinitions;
 use Magento\Framework\ObjectManagerInterface;
-use Magento\Framework\Serialize\Serializer\Serialize;
 use Magento\Framework\Serialize\SerializerInterface;
+use Magento\Framework\Serialize\Serializer\Serialize;
 
 /**
  * Plugin config, provides list of plugins for a type

lib/internal/Magento/Framework/Interception/Test/Unit/Code/Generator/InterceptorTest.php

@@ -4,6 +4,8 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Test\Unit\Code\Generator;
 
 class InterceptorTest extends \PHPUnit_Framework_TestCase

lib/internal/Magento/Framework/Interception/Test/Unit/Code/InterfaceValidatorTest.php

@@ -4,9 +4,11 @@
  * See COPYING.txt for license details.
  */
 
+// @codingStandardsIgnoreFile
+
 namespace Magento\Framework\Interception\Test\Unit\Code;
 
-use Magento\Framework\Interception\Code\InterfaceValidator;
+use \Magento\Framework\Interception\Code\InterfaceValidator;
 
 class InterfaceValidatorTest extends \PHPUnit_Framework_TestCase
 {

lib/internal/Magento/Framework/Interception/Test/Unit/Config/ConfigTest.php

@@ -3,6 +3,7 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+// @codingStandardsIgnoreFile
 namespace Magento\Framework\Interception\Test\Unit\Config;
 
 use Magento\Framework\Serialize\SerializerInterface;