Sattic-test fix

Author: Nazar65

dev/tests/static/framework/Magento/CodeMessDetector/Test/Unit/Rule/Design/AllPurposeActionTest.php

@@ -40,7 +40,8 @@ public function getCases(): array
     {
         return [
             [
-                new class implements ActionInterface, HttpGetActionInterface {
+                new class implements ActionInterface, HttpGetActionInterface
+                {
                     /**
                      * @inheritDoc
                      */
@@ -52,7 +53,8 @@ public function execute()
                 false
             ],
             [
-                new class implements ActionInterface {
+                new class implements ActionInterface
+                {
                     /**
                      * @inheritDoc
                      */
@@ -64,7 +66,8 @@ public function execute()
                 true
             ],
             [
-                new class implements HttpGetActionInterface {
+                new class implements HttpGetActionInterface
+                {
                     /**
                      * @inheritDoc
                      */
@@ -76,7 +79,8 @@ public function execute()
                 false
             ],
             [
-                new class {
+                new class
+                {
 
                 },
                 false
@@ -94,13 +98,15 @@ private function createNodeMock($dynamicObject): MockObject
         $node = $this->getMockBuilder(ClassNode::class)
             ->disableOriginalConstructor()
             ->disableProxyingToOriginalMethods()
-            ->setMethods([
-                // disable name lookup from AST artifact
-                'getNamespaceName',
-                'getParentName',
-                'getName',
-                'getFullQualifiedName',
-            ])
+            ->setMethods(
+                [
+                    // disable name lookup from AST artifact
+                    'getNamespaceName',
+                    'getParentName',
+                    'getName',
+                    'getFullQualifiedName',
+                ]
+            )
             ->getMock();
         $node->expects($this->any())
             ->method('getFullQualifiedName')
@@ -114,10 +120,8 @@ private function createNodeMock($dynamicObject): MockObject
      * @param bool $expects
      * @return InvocationMocker
      */
-    private function expectsRuleViolation(
-        AllPurposeAction $rule,
-        bool $expects
-    ): InvocationMocker {
+    private function expectsRuleViolation(AllPurposeAction $rule, bool $expects): InvocationMocker
+    {
         /** @var Report|MockObject $report */
         $report = $this->getMockBuilder(Report::class)->getMock();
         if ($expects) {

lib/internal/Magento/Framework/Api/ExtensionAttribute/JoinProcessor.php

@@ -61,7 +61,7 @@ public function __construct(
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function process(DbCollection $collection, $extensibleEntityClass = null)
     {
@@ -95,7 +95,7 @@ private function getReferenceTableAlias($attributeCode)
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function extractExtensionAttributes($extensibleEntityClass, array $data)
     {
@@ -127,10 +127,9 @@ public function extractExtensionAttributes($extensibleEntityClass, array $data)
      *
      * @param string $attributeCode
      * @param array $directive
-     * @param array &$data
-     * @param array &$extensionData
+     * @param array $data
+     * @param array $extensionData
      * @param string $extensibleEntityClass
-     * @return void
      */
     private function populateAttributeCodeWithDirective(
         $attributeCode,

lib/internal/Magento/Framework/ObjectManager/Profiler/Log.php

@@ -3,6 +3,7 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+
 namespace Magento\Framework\ObjectManager\Profiler;
 
 use Magento\Framework\ObjectManager\Profiler\Tree\Item;
@@ -133,17 +134,19 @@ public function display()
     {
         $this->stats['used'] = count($this->used);
         $this->stats['unused'] = $this->stats['total'] - $this->stats['used'];
+        //phpcs:disable
         echo '<table border="1" cellspacing="0" cellpadding="2">',
-            '<thead><tr><th>',
-            "Creation chain (Red items are never used) Total: {$this->stats['total']}\n",
-            "Used: {$this->stats['used']} Not used: {$this->stats['unused']}",
-            '</th></tr></thead>',
-            '<tbody>',
-            '<tr><th>Instance class</th></tr>';
+        '<thead><tr><th>',
+        "Creation chain (Red items are never used) Total: {$this->stats['total']}\n",
+        "Used: {$this->stats['used']} Not used: {$this->stats['unused']}",
+        '</th></tr></thead>',
+        '<tbody>',
+        '<tr><th>Instance class</th></tr>';
         foreach ($this->roots as $root) {
             $this->displayItem($root);
         }
         echo '</tbody></table>';
+        //phpcs:enable
     }
 
     /**
@@ -156,9 +159,10 @@ public function display()
     protected function displayItem(Item $item, $level = 0)
     {
         $colorStyle = isset($this->used[$item->getHash()]) ? '' : ' style="color:red" ';
-
+        //phpcs:disable
         echo "<tr><td $colorStyle>" . str_repeat('·&nbsp;', $level) . $item->getClass()
             . ' - ' . $item->getHash() . '</td></tr>';
+        //phpcs:enable
 
         foreach ($item->getChildren() as $child) {
             $this->displayItem($child, $level + 1);