12178 Fix code style violations

Author: avstudnitz

app/code/Magento/Config/Console/Command/ConfigSet/ProcessorFacade.php

@@ -108,7 +108,6 @@ public function process($path, $value, $scope, $scopeCode, $lock)
         return $this->processWithLockTarget($path, $value, $scope, $scopeCode, $lock);
     }
 
-
     /**
      * Processes config:set command with the option to set a target file.
      *
@@ -121,8 +120,14 @@ public function process($path, $value, $scope, $scopeCode, $lock)
      * @return string Processor response message
      * @throws ValidatorException If some validation is wrong
      */
-    public function processWithLockTarget($path, $value, $scope, $scopeCode, $lock, $lockTarget = ConfigFilePool::APP_ENV)
-    {
+    public function processWithLockTarget(
+        $path,
+        $value,
+        $scope,
+        $scopeCode,
+        $lock,
+        $lockTarget = ConfigFilePool::APP_ENV
+    ) {
         try {
             $this->scopeValidator->isValid($scope, $scopeCode);
             $this->pathValidator->validate($path);
@@ -158,4 +163,4 @@ public function processWithLockTarget($path, $value, $scope, $scopeCode, $lock,
 
         return $message;
     }
-}
+}

app/code/Magento/Config/Test/Unit/Console/Command/ConfigSet/ProcessorFacadeTest.php

@@ -123,7 +123,13 @@ public function testProcess()
 
         $this->assertSame(
             'Value was saved.',
-            $this->model->processWithLockTarget('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, false)
+            $this->model->processWithLockTarget(
+                'test/test/test',
+                'test',
+                ScopeConfigInterface::SCOPE_TYPE_DEFAULT,
+                null,
+                false
+            )
         );
     }
 
@@ -139,7 +145,13 @@ public function testProcessWithValidatorException(LocalizedException $exception)
             ->method('isValid')
             ->willThrowException($exception);
 
-        $this->model->processWithLockTarget('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, false);
+        $this->model->processWithLockTarget(
+            'test/test/test',
+            'test',
+            ScopeConfigInterface::SCOPE_TYPE_DEFAULT,
+            null,
+            false
+        );
     }
 
     /**
@@ -174,7 +186,13 @@ public function testProcessWithConfigurationMismatchException()
         $this->configMock->expects($this->never())
             ->method('clean');
 
-        $this->model->processWithLockTarget('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, false);
+        $this->model->processWithLockTarget(
+            'test/test/test',
+            'test',
+            ScopeConfigInterface::SCOPE_TYPE_DEFAULT,
+            null,
+            false
+        );
     }
 
     /**
@@ -200,7 +218,13 @@ public function testProcessWithCouldNotSaveException()
         $this->configMock->expects($this->never())
             ->method('clean');
 
-        $this->model->processWithLockTarget('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, false);
+        $this->model->processWithLockTarget(
+            'test/test/test',
+            'test',
+            ScopeConfigInterface::SCOPE_TYPE_DEFAULT,
+            null,
+            false
+        );
     }
 
     public function testExecuteLockEnv()
@@ -220,7 +244,13 @@ public function testExecuteLockEnv()
 
         $this->assertSame(
             'Value was saved in app/etc/env.php and locked.',
-            $this->model->processWithLockTarget('test/test/test', 'test', ScopeConfigInterface::SCOPE_TYPE_DEFAULT, null, true)
+            $this->model->processWithLockTarget(
+                'test/test/test',
+                'test',
+                ScopeConfigInterface::SCOPE_TYPE_DEFAULT,
+                null,
+                true
+            )
         );
     }