Address codesniffer issues

Author: sivaram7

composer.json

@@ -5,10 +5,6 @@
   "version": "2002.1.19",
   "license": "OSL-3.0",
   "repositories": {
-    "mcd": {
-      "type": "vcs",
-      "url": "https://github.com/magento-commerce/magento-cloud-docker.git"
-    },
     "repo.magento.com": {
       "type": "composer",
       "url": "https://repo.magento.com/"
@@ -26,7 +22,7 @@
     "guzzlehttp/guzzle": "^7.3 || ^7.5",
     "illuminate/config": "^10.0 || ^11.0",
     "magento/magento-cloud-components": "^1.0.8",
-    "magento/magento-cloud-docker": "dev-MCLOUD-13147 as 1.3.99",
+    "magento/magento-cloud-docker": "^1.0.0",
     "magento/magento-cloud-patches": "^1.0.20",
     "magento/quality-patches": "^1.1.0",
     "monolog/monolog": "^2.3 || ^2.7 || ^3.6",

src/Test/Unit/App/Logger/Gelf/HandlerFactoryTest.php

@@ -72,7 +72,7 @@ public function testCreate(): void
                 ['additional'] => [
                     'project' => 'some_project'
                 ]
-            });            
+            });
         $series = [
             [['http', ['host' => 'localhost']], $httpTransportMock],
             [['tcp', ['host' => '127.0.0.1']], $tcpTransportMock],

src/Test/Unit/Command/CronUnlockTest.php

@@ -113,7 +113,6 @@ public function testExecuteWithJobCode(): void
                 }),
                 CronUnlock::UNLOCK_MESSAGE
             );
-            
 
         $tester = new CommandTester(
             $this->cronUnlockCommand

src/Test/Unit/Command/Dev/UpdateComposerTest.php

@@ -140,7 +140,7 @@ public function testExecute(): void
             // withConsecutive() alternative.
             ->with(
                 $this->callback(function (string $script) use (&$scripts) {
-                  return array_shift($scripts) === $script;
+                    return array_shift($scripts) === $script;
                 })
             );
 

src/Test/Unit/Command/Wizard/IdealStateTest.php

@@ -92,13 +92,14 @@ public function testExecuteWithErrors()
             ->method('writeItem')
             // withConsecutive() alternative.
             ->willReturnCallback(
-                function ($arg1, $arg2) use ($outputMock, $error1, $error2){
+                function ($arg1, $arg2) use ($outputMock, $error1, $error2) {
                     if ($arg1 == $outputMock && $arg2 == $error1) {
                         return $this->outputFormatterMock;
                     } elseif ($arg1 == $outputMock && $arg2 == $error2) {
                         return $this->outputFormatterMock;
                     }
-                });
+                }
+            );
         $this->outputFormatterMock->expects($this->once())
             ->method('writeResult')
             ->with($outputMock, false, 'State is not ideal');

src/Test/Unit/Config/Validator/Deploy/ServiceVersionTest.php

@@ -191,7 +191,7 @@ public function testValidateWithErrors(): void
             ->method('validateService')
             // withConsecutive() alternative.
             ->willReturnCallback(
-                function ($arg1, $arg2) use ($errorMessages){
+                function ($arg1, $arg2) use ($errorMessages) {
                     if ($arg1 == ServiceInterface::NAME_RABBITMQ && $arg2 == '1.5') {
                         return $errorMessages[0];
                     } elseif ($arg1 == ServiceInterface::NAME_REDIS && $arg2 == '2.2') {
@@ -205,7 +205,8 @@ function ($arg1, $arg2) use ($errorMessages){
                     } elseif ($arg1 == ServiceInterface::NAME_DB_MYSQL && $arg2 == '5.7') {
                         return $errorMessages[5];
                     }
-                });
+                }
+            );
         $this->resultFactoryMock->expects($this->once())
             ->method('error')
             ->with($this->anything(), implode(PHP_EOL, $errorMessages));

src/Test/Unit/Config/Validator/GlobalStage/ScdOnDeployTest.php

@@ -108,7 +108,7 @@ public function testGetErrors()
             // withConsecutive() alternative.
             ->with(
                 $this->callback(function (string $message) use (&$messages) {
-                  return array_shift($messages) === $message;
+                    return array_shift($messages) === $message;
                 })
             );
 

src/Test/Unit/DB/DumpProcessorTest.php

@@ -146,7 +146,7 @@ public function testExecute(array $dbConfig, $expects, bool $removeDefiners)
             // withConsecutive() alternative.
             ->with(
                 $this->callback(function (string $mock) use (&$mocks) {
-                  return array_shift($mocks) === $mock;
+                    return array_shift($mocks) === $mock;
                 }),
                 $this->connectionDataMock,
                 $removeDefiners
@@ -217,7 +217,7 @@ public function testExecuteWithDatabases(
         array $databases,
         $expects
     ) {
-       $series = [
+        $series = [
             ['main', $this->connectionDataMock, true],
             ['quote', $this->connectionDataMock, true],
             ['sales', $this->connectionDataMock, true]
@@ -248,7 +248,7 @@ public function testExecuteWithDatabases(
             // withConsecutive() alternative.
             ->with(
                 $this->callback(function (string $mock) use (&$mocks) {
-                  return array_shift($mocks) === $mock;
+                    return array_shift($mocks) === $mock;
                 }),
                 $this->connectionDataMock,
                 true

src/Test/Unit/Step/Build/DeployStaticContent/GenerateTest.php

@@ -98,7 +98,7 @@ public function testExecute()
                     1 => $message === "Generating static content for locales: ua_UA fr_FR es_ES en_US\nUsing 3 Threads"
                 };
             }));
-$this->commandFactoryMock->expects($this->once())
+        $this->commandFactoryMock->expects($this->once())
             ->method('matrix')
             ->with($this->optionMock, ['some_matrix'])
             ->willReturn($commands);

src/Test/Unit/Step/Build/EnableEventingTest.php

@@ -137,7 +137,7 @@ public function testExecuteEnableModuleCommandFailed()
             if ($arg1 == 'events:generate:module') {
                 return $this->processMock;
             } elseif ($arg1 == 'module:enable Magento_AdobeCommerceEvents') {
-                throw new ShellException('error during module enablement'); 
+                throw new ShellException('error during module enablement');
             }
         });
         $this->loggerMock->expects(self::exactly(2))