codestyle fixes (Codeception#3994)

Author: DavertMik

RoboFile.php

@@ -869,17 +869,17 @@ protected function documentApiClass($file, $className, $all = false)
 
         $this->taskGenDoc($file)
             ->docClass($className)
-            ->filterMethods(function(ReflectionMethod $r) use ($all, $className) {
+            ->filterMethods(function (ReflectionMethod $r) use ($all, $className) {
                 return $all || $r->isPublic();
             })
             ->append(
                 '<p>&nbsp;</p><div class="alert alert-warning">Reference is taken from the source code. '
                 . '<a href="' . $source . '">Help us to improve documentation. Edit module reference</a></div>'
             )
-            ->processPropertySignature(function($r) {
+            ->processPropertySignature(function ($r) {
                 return "\n#### $" . $r->name. "\n\n";
             })
-            ->processPropertyDocBlock(function($r, $text) {
+            ->processPropertyDocBlock(function ($r, $text) {
                 $modifiers = implode(' ', \Reflection::getModifierNames($r->getModifiers()));
                 $text = ' *' . $modifiers . '* **$' . $r->name . "**\n" . $text;
                 $text = preg_replace("~@(.*?)\s(.*)~", 'type `$2`', $text);
@@ -890,7 +890,7 @@ function (ReflectionClass $r, $text) {
                     return $text . "\n";
                 }
             )
-            ->processMethodSignature(function($r, $text) {
+            ->processMethodSignature(function ($r, $text) {
                 return "#### {$r->name}()\n\n" . ltrim($text, '#');
             })
             ->processMethodDocBlock(

src/Codeception/Command/ConfigValidate.php

@@ -57,7 +57,6 @@ public function execute(InputInterface $input, OutputInterface $output)
         $this->addStyles($output);
 
         if ($suite = $input->getArgument('suite')) {
-
             $output->write("Validating <bold>$suite</bold> config... ");
             $config = $this->getSuiteConfig($suite, $input->getOption('config'));
             $output->writeln("Ok");

src/Codeception/Lib/Di.php

@@ -113,7 +113,7 @@ public function injectDependencies($object, $injectMethodName = self::DEFAULT_IN
             $args = $this->prepareArgs($reflectedMethod, $defaults);
         } catch (\Exception $e) {
             $msg = $e->getMessage();
-            if($e->getPrevious()){ // injection failed because PHP code is invalid. See #3869
+            if ($e->getPrevious()) { // injection failed because PHP code is invalid. See #3869
                 $msg .= '; '. $e->getPrevious();
             }
             throw new InjectionException(

src/Codeception/Lib/Generator/GherkinSnippets.php

@@ -6,7 +6,6 @@
 use Codeception\Util\Template;
 use Symfony\Component\Finder\Finder;
 
-
 class GherkinSnippets
 {
     protected $template = <<<EOF

src/Codeception/Lib/ParamsLoader.php

@@ -91,4 +91,4 @@ protected function loadEnvironmentVars()
     {
         return $_SERVER;
     }
-}
+}

src/Codeception/Module/REST.php

@@ -505,7 +505,7 @@ protected function execute($method, $url, $parameters = [], $files = [])
 
     /**
      * Check if data has non-printable bytes and it is not a valid unicode string
-     * 
+     *
      * @param string $data the text or binary data string
      * @return boolean
      */
@@ -516,7 +516,7 @@ protected function isBinaryData($data)
 
     /**
      * Format a binary string for debug printing
-     * 
+     *
      * @param string $data the binary data string
      * @return string the debug string
      */

src/Codeception/Module/WebDriver.php

@@ -2948,7 +2948,6 @@ public function performOn($element, $actions, $timeout = 10)
                 $class = get_class($e); // rethrow exception for a specific action
                 throw new $class($e->getMessage() . "\nat $step", $e->getResults());
             }
-
         }
         $this->setBaseElement();
     }
@@ -2961,5 +2960,4 @@ protected function setBaseElement($element = null)
         }
         $this->baseElement = $this->matchFirstOrFail($this->webDriver, $element);
     }
-
 }

src/Codeception/PHPUnit/ResultPrinter/HTML.php

@@ -107,14 +107,12 @@ public function endTest(\PHPUnit_Framework_Test $test, $time)
 
         foreach ($steps as $step) {
             if ($step->getMetaStep()) {
-
                 if (! empty($subStepsRendered[$step->getMetaStep()->getAction()])) {
                     $subStepsBuffer = implode('', $subStepsRendered[$step->getMetaStep()->getAction()]);
                     unset($subStepsRendered[$step->getMetaStep()->getAction()]);
 
                     $stepsBuffer .= $this->renderSubsteps($step->getMetaStep(), $subStepsBuffer);
                 }
-
             } else {
                 $stepsBuffer .= $this->renderStep($step);
             }
@@ -148,7 +146,6 @@ public function endTest(\PHPUnit_Framework_Test $test, $time)
                 $localPath = PathResolver::getRelativeDir($reports['html'], codecept_output_dir());
                 $html = "<tr><td class='error'>See <a href='$localPath' target='_blank'>HTML snapshot</a> of a failed page</td></tr>";
             }
-
         }
 
         $toggle = $stepsBuffer ? '<span class="toggle">+</span>' : '';

src/Codeception/Subscriber/GracefulTermination.php

@@ -6,7 +6,6 @@
 use Codeception\Events;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
-
 class GracefulTermination implements EventSubscriberInterface
 {
     const SIGNAL_FUNC = 'pcntl_signal';

src/Codeception/Test/Interfaces/ScenarioDriven.php

@@ -1,7 +1,6 @@
 <?php
 namespace Codeception\Test\Interfaces;
 
-
 interface ScenarioDriven
 {
     public function getFeature();