MC-39900: Stabilize unit tests

- CS fixes

Author: svera

app/code/Magento/Review/Test/Unit/Block/Adminhtml/RssTest.php

@@ -123,11 +123,26 @@ public function testGetRssData()
         $this->assertEquals($rssData['charset'], $data['charset']);
         $this->assertEquals($rssData['entries']['title'], $data['entries'][0]['title']);
         $this->assertEquals($rssData['entries']['link'], $data['entries'][0]['link']);
-        $this->assertStringContainsString($rssData['entries']['description']['rss_url'], $data['entries'][0]['description']);
-        $this->assertStringContainsString($rssData['entries']['description']['name'], $data['entries'][0]['description']);
-        $this->assertStringContainsString($rssData['entries']['description']['summary'], $data['entries'][0]['description']);
-        $this->assertStringContainsString($rssData['entries']['description']['review'], $data['entries'][0]['description']);
-        $this->assertStringContainsString($rssData['entries']['description']['store'], $data['entries'][0]['description']);
+        $this->assertStringContainsString(
+            $rssData['entries']['description']['rss_url'],
+            $data['entries'][0]['description']
+        );
+        $this->assertStringContainsString(
+            $rssData['entries']['description']['name'],
+            $data['entries'][0]['description']
+        );
+        $this->assertStringContainsString(
+            $rssData['entries']['description']['summary'],
+            $data['entries'][0]['description']
+        );
+        $this->assertStringContainsString(
+            $rssData['entries']['description']['review'],
+            $data['entries'][0]['description']
+        );
+        $this->assertStringContainsString(
+            $rssData['entries']['description']['store'],
+            $data['entries'][0]['description']
+        );
     }
 
     /**

app/code/Magento/SalesRule/Test/Unit/Block/Rss/DiscountsTest.php

@@ -175,16 +175,20 @@ public function testGetRssData()
         $this->assertEquals($rssData['entries']['title'], $data['entries'][0]['title']);
         $this->assertEquals($rssData['entries']['link'], $data['entries'][0]['link']);
         $this->assertStringContainsString(
-            $rssData['entries']['description']['description'], $data['entries'][0]['description']
+            $rssData['entries']['description']['description'],
+            $data['entries'][0]['description']
         );
         $this->assertStringContainsString(
-            $rssData['entries']['description']['start_date'], $data['entries'][0]['description']
+            $rssData['entries']['description']['start_date'],
+            $data['entries'][0]['description']
         );
         $this->assertStringContainsString(
-            $rssData['entries']['description']['end_date'], $data['entries'][0]['description']
+            $rssData['entries']['description']['end_date'],
+            $data['entries'][0]['description']
         );
         $this->assertStringContainsString(
-            $rssData['entries']['description']['coupon_code'], $data['entries'][0]['description']
+            $rssData['entries']['description']['coupon_code'],
+            $data['entries'][0]['description']
         );
     }
 

app/code/Magento/Theme/Test/Unit/Console/Command/ThemeUninstallCommandTest.php

@@ -282,15 +282,13 @@ public function setUpExecute()
         $this->themeUninstaller->expects($this->once())
             ->method('uninstallRegistry')
             ->with(
-                $this->isInstanceOf(
-                    \Symfony\Component\Console\Output\OutputInterface::class),
+                $this->isInstanceOf(\Symfony\Component\Console\Output\OutputInterface::class),
                 $this->anything()
             );
         $this->themeUninstaller->expects($this->once())
             ->method('uninstallCode')
             ->with(
-                $this->isInstanceOf(
-                    \Symfony\Component\Console\Output\OutputInterface::class),
+                $this->isInstanceOf(\Symfony\Component\Console\Output\OutputInterface::class),
                 $this->anything()
             );
     }

app/code/Magento/Vault/Test/Unit/Model/PaymentTokenManagementTest.php

@@ -24,7 +24,6 @@
 /**
  * Class PaymentTokenManagementTest
  * Test for PaymentTokenManagement
- * 
  * @see \Magento\Vault\Model\PaymentTokenManagement
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */

composer.lock

@@ -102,7 +102,9 @@ public function testCreateUndefinedClass()
     public function testCreateInvalidClass()
     {
         $this->expectException(\InvalidArgumentException::class);
-        $this->expectExceptionMessage('Driver class "stdClass" must implement \\Magento\\Framework\\Profiler\\DriverInterface.');
+        $this->expectExceptionMessage(
+            'Driver class "stdClass" must implement \\Magento\\Framework\\Profiler\\DriverInterface.'
+        );
 
         $this->_factory->create(['type' => 'stdClass']);
     }

lib/internal/Magento/Framework/Profiler/Test/Unit/Driver/FactoryTest.php

@@ -13,13 +13,13 @@
 
 /**
  * Class ProxyTesting
+ *
  * Test for Proxy
  */
 class ProxyTesting
 {
     /**
-     * Invoke the proxy's method, imposing expectations on proxied object,
-     * that it must be invoked as well with appropriate parameters.
+     * Invoke the proxy's method, imposing expectations on proxied object, that it must be invoked as well.
      *
      * @param mixed $object Proxy
      * @param \PHPUnit\Framework\MockObject\MockObject $proxiedObject

lib/internal/Magento/Framework/TestFramework/Unit/Helper/ProxyTesting.php

@@ -7,7 +7,7 @@
 
 /**
  * Class BuilderTest
- *
+ * Test for Builder
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  */
 class BuilderTest extends \PHPUnit\Framework\TestCase

lib/internal/Magento/Framework/Validator/Test/Unit/BuilderTest.php

@@ -79,7 +79,8 @@ public function testRenderJson()
         )->method(
             'encode'
         )->willReturnCallback(
-            [$this, 'callbackJsonEncode'], $this->returnArgument(0)
+            [$this, 'callbackJsonEncode'],
+            $this->returnArgument(0)
         );
         /** Init output buffering to catch output via echo function. */
         ob_start();
@@ -119,7 +120,8 @@ public function testRenderJsonInDeveloperMode()
         )->method(
             'encode'
         )->willReturnCallback(
-            [$this, 'callbackJsonEncode'], $this->returnArgument(0)
+            [$this, 'callbackJsonEncode'],
+            $this->returnArgument(0)
         );
         ob_start();
         $this->_errorProcessor->renderErrorMessage('Message', 'Message trace.', 401);
@@ -235,11 +237,9 @@ public function testCriticalExceptionStackTrace()
         $this->_loggerMock->expects($this->once())
             ->method('critical')
             ->willReturnCallback(
-
-                    function (\Exception $loggedException) use ($thrownException) {
-                        $this->assertSame($thrownException, $loggedException->getPrevious());
-                    }
-
+                function (\Exception $loggedException) use ($thrownException) {
+                    $this->assertSame($thrownException, $loggedException->getPrevious());
+                }
             );
         $this->_errorProcessor->maskException($thrownException);
     }