Codeception / Codeception

src/Codeception/Lib/Driver/AmazonSQS.php

Spacing +5 added lines, -5 removed lines

@@ -21,19 +21,19 @@ 
             'region' => $config['region'],
         ];
 
-        if (! empty($config['key']) && ! empty($config['secret'])) {
+        if (!empty($config['key']) && !empty($config['secret'])) {
             $params['credentials'] = new Credentials($config['key'], $config['secret']);
         }
 
-        if (! empty($config['profile'])) {
+        if (!empty($config['profile'])) {
             $params['profile'] = $config['profile'];
         }
 
-        if (! empty($config['version'])) {
+        if (!empty($config['version'])) {
             $params['version'] = $config['version'];
         }
 
-        if (! empty($config['endpoint'])) {
+        if (!empty($config['endpoint'])) {
             $params['endpoint'] = $config['endpoint'];
         }
 
@@ -137,7 +137,7 @@ 
                 return $queueURL;
             }
         }
-        throw new TestRuntimeException('queue [' . $queue . '] not found');
+        throw new TestRuntimeException('queue ['.$queue.'] not found');
     }
 
     public function getRequiredConfig()

src/Codeception/Application.php

Spacing +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
         $interfaces = class_implements($commandClass);
 
         if (!in_array('Codeception\CustomCommandInterface', $interfaces)) {
-            throw new ConfigurationException("Extension: Command {$commandClass} must implement " .
+            throw new ConfigurationException("Extension: Command {$commandClass} must implement ".
                                              "the interface `Codeception\\CustomCommandInterface`");
         }
 
@@ -150,7 +150,7 @@ 
                         $this->preloadConfiguration($argv[++$i]);
                     }
                     if (!empty($match[1])) {
-                        $argvWithoutConfig[] = "-" . $match[1]; //rest commands
+                        $argvWithoutConfig[] = "-".$match[1]; //rest commands
                     }
                     continue;
                 }

src/Codeception/Subscriber/Console.php

Spacing +11 added lines, -11 removed lines

@@ -137,7 +137,7 @@ 
                 implode(
                     ', ',
                     array_map(
-                        function ($module) {
+                        function($module) {
                             return $module->_getName();
                         },
                         $e->getSuite()->getModules()
@@ -217,7 +217,7 @@ 
             return $path;
         }
 
-        return codecept_output_dir() . $path;
+        return codecept_output_dir().$path;
     }
 
     public function testSuccess(TestEvent $e)
@@ -294,7 +294,7 @@ 
         }
         $metaStep = $e->getStep()->getMetaStep();
         if ($metaStep and $this->metaStep != $metaStep) {
-            $this->message(' ' . $metaStep->getPrefix())
+            $this->message(' '.$metaStep->getPrefix())
                 ->style('bold')
                 ->append($metaStep->__toString())
                 ->writeln();
@@ -332,7 +332,7 @@ 
         $messages = Notification::all();
         foreach (array_count_values($messages) as $message => $count) {
             if ($count > 1) {
-                $message = $count . 'x ' . $message;
+                $message = $count.'x '.$message;
             }
             $this->output->notification($message);
         }
@@ -343,7 +343,7 @@ 
         $failedTest = $e->getTest();
         $fail = $e->getFail();
 
-        $this->output->write($e->getCount() . ") ");
+        $this->output->write($e->getCount().") ");
         $this->writeCurrentTest($failedTest, false);
         $this->output->writeln('');
         $this->message("<error> Test </error> ")
@@ -407,11 +407,11 @@ 
     public function printScenarioFail(ScenarioDriven $failedTest, $fail)
     {
         if ($this->conditionalFails) {
-            $failedStep = (string) array_shift($this->conditionalFails);
+            $failedStep = (string)array_shift($this->conditionalFails);
         } else {
-            $failedStep = (string) $failedTest->getScenario()->getMetaStep();
+            $failedStep = (string)$failedTest->getScenario()->getMetaStep();
             if ($failedStep === '') {
-                $failedStep = (string) array_shift($this->failedStep);
+                $failedStep = (string)array_shift($this->failedStep);
             }
         }
 
@@ -465,7 +465,7 @@ 
                 $message->writeln();
                 continue;
             }
-            $message->append($step['file'] . ':' . $step['line']);
+            $message->append($step['file'].':'.$step['line']);
             $message->writeln();
         }
 
@@ -535,14 +535,14 @@ 
             if (getenv('COLUMNS')) {
                 $this->width = getenv('COLUMNS');
             } else {
-                $this->width = (int) (`command -v tput >> /dev/null 2>&1 && tput cols`) - 2;
+                $this->width = (int)(`command -v tput >> /dev/null 2>&1 && tput cols`) - 2;
             }
         } elseif ($this->isWin() && (php_sapi_name() === "cli")) {
             exec('mode con', $output);
             if (isset($output[4])) {
                 preg_match('/^ +.* +(\d+)$/', $output[4], $matches);
                 if (!empty($matches[1])) {
-                    $this->width = (int) $matches[1];
+                    $this->width = (int)$matches[1];
                 }
             }
         }

Doc Comments +6 added lines

@@ -360,6 +360,9 @@ 
         $this->printExceptionTrace($fail);
     }
 
+    /**
+     * @param string $cause
+     */
     public function printException($e, $cause = null)
     {
         if ($e instanceof \PHPUnit\Framework\SkippedTestError or $e instanceof \PHPUnit\Framework_IncompleteTestError) {
@@ -404,6 +407,9 @@ 
         $message->writeln();
     }
 
+    /**
+     * @param \Exception $fail
+     */
     public function printScenarioFail(ScenarioDriven $failedTest, $fail)
     {
         if ($this->conditionalFails) {

src/Codeception/Lib/InnerBrowser.php

Spacing +19 added lines, -19 removed lines

@@ -62,8 +62,8 @@ 
             return;
         }
         $filename = preg_replace('~\W~', '.', Descriptor::getTestSignatureUnique($test));
-        $filename = mb_strcut($filename, 0, 244, 'utf-8') . '.fail.html';
-        $this->_savePageSource($report = codecept_output_dir() . $filename);
+        $filename = mb_strcut($filename, 0, 244, 'utf-8').'.fail.html';
+        $this->_savePageSource($report = codecept_output_dir().$filename);
         $test->getMetadata()->addReport('html', $report);
     }
 
@@ -173,7 +173,7 @@ 
             if (preg_match('#^(//|https?://(?!localhost))#', $uri)) {
                 $hostname = parse_url($uri, PHP_URL_HOST);
                 if (!$this->isInternalDomain($hostname)) {
-                    throw new ExternalUrlException(get_class($this) . " can't open external URL: " . $uri);
+                    throw new ExternalUrlException(get_class($this)." can't open external URL: ".$uri);
                 }
             }
 
@@ -501,12 +501,12 @@ 
     {
         $crawler = $this->getCrawler()->selectLink($text);
         if ($crawler->count() === 0) {
-            $this->fail("No links containing text '$text' were found in page " . $this->_getCurrentUri());
+            $this->fail("No links containing text '$text' were found in page ".$this->_getCurrentUri());
         }
         if ($url) {
             $crawler = $crawler->filterXPath(sprintf('.//a[contains(@href, %s)]', Crawler::xpathLiteral($url)));
             if ($crawler->count() === 0) {
-                $this->fail("No links containing text '$text' and URL '$url' were found in page " . $this->_getCurrentUri());
+                $this->fail("No links containing text '$text' and URL '$url' were found in page ".$this->_getCurrentUri());
             }
         }
         $this->assertTrue(true);
@@ -517,12 +517,12 @@ 
         $crawler = $this->getCrawler()->selectLink($text);
         if (!$url) {
             if ($crawler->count() > 0) {
-                $this->fail("Link containing text '$text' was found in page " . $this->_getCurrentUri());
+                $this->fail("Link containing text '$text' was found in page ".$this->_getCurrentUri());
             }
         }
         $crawler = $crawler->filterXPath(sprintf('.//a[contains(@href, %s)]', Crawler::xpathLiteral($url)));
         if ($crawler->count() > 0) {
-            $this->fail("Link containing text '$text' and URL '$url' was found in page " . $this->_getCurrentUri());
+            $this->fail("Link containing text '$text' and URL '$url' was found in page ".$this->_getCurrentUri());
         }
     }
 
@@ -573,7 +573,7 @@ 
         $matches = [];
         $res     = preg_match($uri, $this->_getCurrentUri(), $matches);
         if (!$res) {
-            $this->fail("Couldn't match $uri in " . $this->_getCurrentUri());
+            $this->fail("Couldn't match $uri in ".$this->_getCurrentUri());
         }
         if (!isset($matches[1])) {
             $this->fail("Nothing to grab. A regex parameter required. Ex: '/user/(\\d+)'");
@@ -839,7 +839,7 @@ 
 
         $url = $this->getFormUrl($frmCrawl);
         if (strcasecmp($form->getMethod(), 'GET') === 0) {
-            $url = Uri::mergeUrls($url, '?' . http_build_query($requestParams));
+            $url = Uri::mergeUrls($url, '?'.http_build_query($requestParams));
         }
 
         $url = preg_replace('/#.*/', '', $url);
@@ -848,7 +848,7 @@ 
         $this->debugSection('Method', $form->getMethod());
         $this->debugSection('Parameters', $requestParams);
 
-        $requestParams= $this->getFormPhpValues($requestParams);
+        $requestParams = $this->getFormPhpValues($requestParams);
 
         $this->crawler = $this->clientRequest(
             $form->getMethod(),
@@ -1154,7 +1154,7 @@ 
     public function attachFile($field, $filename)
     {
         $form = $this->getFormFor($field = $this->getFieldByLabelOrCss($field));
-        $filePath = codecept_data_dir() . $filename;
+        $filePath = codecept_data_dir().$filename;
         if (!file_exists($filePath)) {
             throw new \InvalidArgumentException("File does not exist: $filePath");
         }
@@ -1310,7 +1310,7 @@ 
     {
         foreach ($attributes as $attr => $val) {
             $nodes = $nodes->reduce(
-                function (Crawler $node) use ($attr, $val) {
+                function(Crawler $node) use ($attr, $val) {
                     return $node->attr($attr) == $val;
                 }
             );
@@ -1401,11 +1401,11 @@ 
 
         $expires      = isset($params['expiry']) ? $params['expiry'] : null; // WebDriver compatibility
         $expires      = isset($params['expires']) && !$expires ? $params['expires'] : null;
-        $path         = isset($params['path'])    ? $params['path'] : null;
-        $domain       = isset($params['domain'])  ? $params['domain'] : '';
-        $secure       = isset($params['secure'])  ? $params['secure'] : false;
-        $httpOnly     = isset($params['httpOnly'])  ? $params['httpOnly'] : true;
-        $encodedValue = isset($params['encodedValue'])  ? $params['encodedValue'] : false;
+        $path         = isset($params['path']) ? $params['path'] : null;
+        $domain       = isset($params['domain']) ? $params['domain'] : '';
+        $secure       = isset($params['secure']) ? $params['secure'] : false;
+        $httpOnly     = isset($params['httpOnly']) ? $params['httpOnly'] : true;
+        $encodedValue = isset($params['encodedValue']) ? $params['encodedValue'] : false;
 
 
 
@@ -1471,7 +1471,7 @@ 
         $selector = $this->stringifySelector($selector);
         if (!empty($attributes)) {
             $nodes = $this->filterByAttributes($nodes, $attributes);
-            $selector .= "' with attribute(s) '" . trim(json_encode($attributes), '{}');
+            $selector .= "' with attribute(s) '".trim(json_encode($attributes), '{}');
         }
         $this->assertDomContains($nodes, $selector);
     }
@@ -1482,7 +1482,7 @@ 
         $selector = $this->stringifySelector($selector);
         if (!empty($attributes)) {
             $nodes = $this->filterByAttributes($nodes, $attributes);
-            $selector .= "' with attribute(s) '" . trim(json_encode($attributes), '{}');
+            $selector .= "' with attribute(s) '".trim(json_encode($attributes), '{}');
         }
         $this->assertDomNotContains($nodes, $selector);
     }

Doc Comments +34 added lines, -6 removed lines

@@ -110,7 +110,7 @@ 
      * @param array $files
      * @param array $server
      * @param null $content
-     * @return mixed|Crawler
+     * @return string
      * @throws ExternalUrlException
      * @see `_loadPage`
      */
@@ -195,6 +195,9 @@ 
         return $this->redirectIfNecessary($result, $maxRedirects, 0);
     }
 
+    /**
+     * @param string|false $domain
+     */
     protected function isInternalDomain($domain)
     {
         if ($this->internalDomains === null) {
@@ -266,6 +269,9 @@ 
         return $this->client;
     }
 
+    /**
+     * @param string $filename
+     */
     public function _savePageSource($filename)
     {
         file_put_contents($filename, $this->_getResponseContent());
@@ -381,7 +387,7 @@ 
 
     /**
      * @param $link
-     * @return bool
+     * @return boolean|null
      */
     protected function clickByLocator($link)
     {
@@ -615,6 +621,9 @@ 
         $this->proceedSeeInFormFields($formSelector, $params, true);
     }
 
+    /**
+     * @param boolean $assertNot
+     */
     protected function proceedSeeInFormFields($formSelector, array $params, $assertNot)
     {
         $form = $this->match($formSelector)->first();
@@ -859,6 +868,9 @@ 
         $this->forms = [];
     }
 
+    /**
+     * @param string $selector
+     */
     public function submitForm($selector, array $params, $button = null)
     {
         $form = $this->match($selector)->first();
@@ -1224,7 +1236,7 @@ 
      *
      * ```
      *
-     * @param $method
+     * @param string $method
      * @param $uri
      * @param $params
      */
@@ -1394,6 +1406,9 @@ 
         $this->fail("Element $nodes is not a form field or does not contain a form field");
     }
 
+    /**
+     * @param string $name
+     */
     public function setCookie($name, $val, array $params = [])
     {
         $cookies = $this->client->getCookieJar();
@@ -1611,18 +1626,27 @@ 
         $this->assertNotContains($title, $nodes->first()->text(), "page title contains $title");
     }
 
+    /**
+     * @param Crawler $nodes
+     */
     protected function assertDomContains($nodes, $message, $text = '')
     {
         $constraint = new CrawlerConstraint($text, $this->_getCurrentUri());
         $this->assertThat($nodes, $constraint, $message);
     }
 
+    /**
+     * @param Crawler $nodes
+     */
     protected function assertDomNotContains($nodes, $message, $text = '')
     {
         $constraint = new CrawlerNotConstraint($text, $this->_getCurrentUri());
         $this->assertThat($nodes, $constraint, $message);
     }
 
+    /**
+     * @param string $needle
+     */
     protected function assertPageContains($needle, $message = '')
     {
         $constraint = new PageConstraint($needle, $this->_getCurrentUri());
@@ -1633,6 +1657,9 @@ 
         );
     }
 
+    /**
+     * @param string $needle
+     */
     protected function assertPageNotContains($needle, $message = '')
     {
         $constraint = new PageConstraint($needle, $this->_getCurrentUri());
@@ -1664,8 +1691,8 @@ 
     }
 
     /**
-     * @param $name
-     * @param $form
+     * @param string|null $name
+     * @param Form $form
      * @param $dynamicField
      * @return FormField
      */
@@ -1733,7 +1760,8 @@ 
     }
 
     /**
-     * @param $result
+     * @param Crawler|null $result
+     * @param integer $redirectCount
      * @return mixed
      */
     protected function redirectIfNecessary($result, $maxRedirects, $redirectCount)

src/Codeception/Module/Yii2.php

Doc Comments +3 added lines, -3 removed lines

@@ -535,7 +535,7 @@ 
      * $mailer = $I->grabComponent('mailer');
      * ```
      *
-     * @param $component
+     * @param string $component
      * @return mixed
      * @throws ModuleException
      */
@@ -656,7 +656,7 @@ 
     /**
      * Returns a list of regex patterns for recognized domain names
      *
-     * @return array
+     * @return string[]
      */
     public function getInternalDomains()
     {
@@ -683,7 +683,7 @@ 
     /**
      * Sets a cookie and, if validation is enabled, signs it.
      * @param string $name The name of the cookie
-     * @param string $value The value of the cookie
+     * @param string $val The value of the cookie
      * @param array $params Additional cookie params like `domain`, `path`, `expires` and `secure`.
      */
     public function setCookie($name, $val, array $params = [])

src/Codeception/Module/MongoDb.php

Spacing +7 added lines, -7 removed lines

@@ -98,7 +98,7 @@ 
                 $this->config['password']
             );
         } catch (\MongoConnectionException $e) {
-            throw new ModuleException(__CLASS__, $e->getMessage() . ' while creating Mongo connection');
+            throw new ModuleException(__CLASS__, $e->getMessage().' while creating Mongo connection');
         }
 
         // starting with loading dump
@@ -112,14 +112,14 @@ 
     private function validateDump()
     {
         if ($this->config['dump'] && ($this->config['cleanup'] or ($this->config['populate']))) {
-            if (!file_exists(Configuration::projectDir() . $this->config['dump'])) {
+            if (!file_exists(Configuration::projectDir().$this->config['dump'])) {
                 throw new ModuleConfigException(
                     __CLASS__,
                     "File with dump doesn't exist.\n
                     Please, check path for dump file: " . $this->config['dump']
                 );
             }
-            $this->dumpFile = Configuration::projectDir() . $this->config['dump'];
+            $this->dumpFile = Configuration::projectDir().$this->config['dump'];
             $this->isDumpFileEmpty = false;
 
             if ($this->config['dump_type'] === self::DUMP_TYPE_JS) {
@@ -171,9 +171,9 @@ 
             throw new ModuleConfigException(
                 __CLASS__,
                 '\"dump_type\" must be one of ["'
-                . self::DUMP_TYPE_JS . '", "'
-                . self::DUMP_TYPE_MONGODUMP . '", "'
-                . self::DUMP_TYPE_MONGODUMP_TAR_GZ . '"].'
+                . self::DUMP_TYPE_JS.'", "'
+                . self::DUMP_TYPE_MONGODUMP.'", "'
+                . self::DUMP_TYPE_MONGODUMP_TAR_GZ.'"].'
             );
         }
     }
@@ -268,7 +268,7 @@ 
         }
 
         $response = $collection->insertOne($data);
-        return (string) $response->getInsertedId();
+        return (string)$response->getInsertedId();
     }
 
     /**

src/Codeception/Module/Laravel5.php

Spacing +30 added lines, -30 removed lines

@@ -128,7 +128,7 @@ 
                 'run_database_seeder' => false,
                 'database_seeder_class' => '',
                 'environment_file' => '.env',
-                'bootstrap' => 'bootstrap' . DIRECTORY_SEPARATOR . 'app.php',
+                'bootstrap' => 'bootstrap'.DIRECTORY_SEPARATOR.'app.php',
                 'root' => '',
                 'packages' => 'workbench',
                 'vendor_dir' => 'vendor',
@@ -144,7 +144,7 @@ 
         $projectDir .= $this->config['root'];
 
         $this->config['project_dir'] = $projectDir;
-        $this->config['bootstrap_file'] = $projectDir . $this->config['bootstrap'];
+        $this->config['bootstrap_file'] = $projectDir.$this->config['bootstrap'];
 
         parent::__construct($container);
     }
@@ -226,7 +226,7 @@ 
      */
     private function applicationUsesDatabase()
     {
-        return ! empty($this->app['config']['database.default']);
+        return !empty($this->app['config']['database.default']);
     }
 
     /**
@@ -252,7 +252,7 @@ 
      */
     protected function registerAutoloaders()
     {
-        require $this->config['project_dir'] . $this->config['vendor_dir'] . DIRECTORY_SEPARATOR . 'autoload.php';
+        require $this->config['project_dir'].$this->config['vendor_dir'].DIRECTORY_SEPARATOR.'autoload.php';
     }
 
     /**
@@ -561,7 +561,7 @@ 
         $rootNamespace = $this->getRootControllerNamespace();
 
         if ($rootNamespace && !(strpos($action, '\\') === 0)) {
-            return $rootNamespace . '\\' . $action;
+            return $rootNamespace.'\\'.$action;
         }
 
         return trim($action, '\\');
@@ -604,11 +604,11 @@ 
             return;
         }
 
-        if (! $this->app['session']->has($key)) {
+        if (!$this->app['session']->has($key)) {
             $this->fail("No session variable with key '$key'");
         }
 
-        if (! is_null($value)) {
+        if (!is_null($value)) {
             $this->assertEquals($value, $this->app['session']->get($key));
         }
     }
@@ -671,7 +671,7 @@ 
     {
         $viewErrorBag = $this->app->make('view')->shared('errors');
         if (count($viewErrorBag) > 0) {
-            $this->fail("Found the following form errors: \n\n" . $viewErrorBag->toJson(JSON_PRETTY_PRINT));
+            $this->fail("Found the following form errors: \n\n".$viewErrorBag->toJson(JSON_PRETTY_PRINT));
         }
     }
 
@@ -723,7 +723,7 @@ 
             $this->fail("No form error message for key '$key'\n");
         }
 
-        if (! is_null($expectedErrorMessage)) {
+        if (!is_null($expectedErrorMessage)) {
             $this->assertContains($expectedErrorMessage, $viewErrorBag->first($key));
         }
     }
@@ -765,8 +765,8 @@ 
             return;
         }
 
-        if (! $guard->attempt($user)) {
-            $this->fail("Failed to login with credentials " . json_encode($user));
+        if (!$guard->attempt($user)) {
+            $this->fail("Failed to login with credentials ".json_encode($user));
         }
     }
 
@@ -791,7 +791,7 @@ 
             $auth = $auth->guard($guard);
         }
 
-        if (! $auth->check()) {
+        if (!$auth->check()) {
             $this->fail("There is no authenticated user");
         }
     }
@@ -864,7 +864,7 @@ 
         if (class_exists($table)) {
             $model = new $table;
 
-            if (! $model instanceof EloquentModel) {
+            if (!$model instanceof EloquentModel) {
                 throw new \RuntimeException("Class $table is not an Eloquent model");
             }
 
@@ -876,7 +876,7 @@ 
         try {
             return $this->app['db']->table($table)->insertGetId($attributes);
         } catch (\Exception $e) {
-            $this->fail("Could not insert record into table '$table':\n\n" . $e->getMessage());
+            $this->fail("Could not insert record into table '$table':\n\n".$e->getMessage());
         }
     }
 
@@ -898,10 +898,10 @@ 
     public function seeRecord($table, $attributes = [])
     {
         if (class_exists($table)) {
-            if (! $this->findModel($table, $attributes)) {
-                $this->fail("Could not find $table with " . json_encode($attributes));
+            if (!$this->findModel($table, $attributes)) {
+                $this->fail("Could not find $table with ".json_encode($attributes));
             }
-        } elseif (! $this->findRecord($table, $attributes)) {
+        } elseif (!$this->findRecord($table, $attributes)) {
             $this->fail("Could not find matching record in table '$table'");
         }
     }
@@ -925,7 +925,7 @@ 
     {
         if (class_exists($table)) {
             if ($this->findModel($table, $attributes)) {
-                $this->fail("Unexpectedly found matching $table with " . json_encode($attributes));
+                $this->fail("Unexpectedly found matching $table with ".json_encode($attributes));
             }
         } elseif ($this->findRecord($table, $attributes)) {
             $this->fail("Unexpectedly found matching record in table '$table'");
@@ -952,14 +952,14 @@ 
     public function grabRecord($table, $attributes = [])
     {
         if (class_exists($table)) {
-            if (! $model = $this->findModel($table, $attributes)) {
-                $this->fail("Could not find $table with " . json_encode($attributes));
+            if (!$model = $this->findModel($table, $attributes)) {
+                $this->fail("Could not find $table with ".json_encode($attributes));
             }
 
             return $model;
         }
 
-        if (! $record = $this->findRecord($table, $attributes)) {
+        if (!$record = $this->findRecord($table, $attributes)) {
             $this->fail("Could not find matching record in table '$table'");
         }
 
@@ -987,12 +987,12 @@ 
         if (class_exists($table)) {
             $currentNum = $this->countModels($table, $attributes);
             if ($currentNum != $expectedNum) {
-                $this->fail("The number of found $table ($currentNum) does not match expected number $expectedNum with " . json_encode($attributes));
+                $this->fail("The number of found $table ($currentNum) does not match expected number $expectedNum with ".json_encode($attributes));
             }
         } else {
             $currentNum = $this->countRecords($table, $attributes);
             if ($currentNum != $expectedNum) {
-                $this->fail("The number of found records ($currentNum) does not match expected number $expectedNum in table $table with " . json_encode($attributes));
+                $this->fail("The number of found records ($currentNum) does not match expected number $expectedNum in table $table with ".json_encode($attributes));
             }
         }
     }
@@ -1015,7 +1015,7 @@ 
      */
     public function grabNumRecords($table, $attributes = [])
     {
-        return class_exists($table)? $this->countModels($table, $attributes) : $this->countRecords($table, $attributes);
+        return class_exists($table) ? $this->countModels($table, $attributes) : $this->countRecords($table, $attributes);
     }
 
     /**
@@ -1046,7 +1046,7 @@ 
             $query->where($key, $value);
         }
 
-        return (array) $query->first();
+        return (array)$query->first();
     }
 
     /**
@@ -1136,7 +1136,7 @@ 
 
             return $result;
         } catch (\Exception $e) {
-            $this->fail("Could not create model: \n\n" . get_class($e) . "\n\n" . $e->getMessage());
+            $this->fail("Could not create model: \n\n".get_class($e)."\n\n".$e->getMessage());
         }
     }
 
@@ -1165,7 +1165,7 @@ 
         try {
             return $this->modelFactory($model, $name, $times)->create($attributes);
         } catch (\Exception $e) {
-            $this->fail("Could not create model: \n\n" . get_class($e) . "\n\n" . $e->getMessage());
+            $this->fail("Could not create model: \n\n".get_class($e)."\n\n".$e->getMessage());
         }
     }
 
@@ -1178,8 +1178,8 @@ 
      */
     protected function modelFactory($model, $name, $times = 1)
     {
-        if (! function_exists('factory')) {
-            throw new ModuleException($this, 'The factory() method does not exist. ' .
+        if (!function_exists('factory')) {
+            throw new ModuleException($this, 'The factory() method does not exist. '.
                 'This functionality relies on Laravel model factories, which were introduced in Laravel 5.1.');
         }
 
@@ -1213,7 +1213,7 @@ 
         $server = ReflectionHelper::readPrivateProperty($this->client, 'server');
         $domain = $server['HTTP_HOST'];
 
-        return '/^' . str_replace('.', '\.', $domain) . '$/';
+        return '/^'.str_replace('.', '\.', $domain).'$/';
     }
 
     /**

src/Codeception/Lib/ParamsLoader.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
                 return $this->loadDotEnvFile();
             }
         } catch (\Exception $e) {
-            throw new ConfigurationException("Failed loading params from $paramStorage\n" . $e->getMessage());
+            throw new ConfigurationException("Failed loading params from $paramStorage\n".$e->getMessage());
         }
 
         throw new ConfigurationException("Params can't be loaded from `$paramStorage`.");
@@ -87,7 +87,7 @@ 
         }
 
         throw new ConfigurationException(
-            "`vlucas/phpdotenv` library is required to parse .env files.\n" .
+            "`vlucas/phpdotenv` library is required to parse .env files.\n".
             "Please install it via composer: composer require vlucas/phpdotenv"
         );
     }

src/Codeception/Command/Run.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Codeception\Codecept;
 use Codeception\Configuration;
-use Codeception\Util\PathResolver;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;

Spacing +8 added lines, -8 removed lines

@@ -121,8 +121,8 @@ 
         $this->setDefinition([
             new InputArgument('suite', InputArgument::OPTIONAL, 'suite to be tested'),
             new InputArgument('test', InputArgument::OPTIONAL, 'test to be run'),
-            new InputOption('override', 'o', InputOption::VALUE_IS_ARRAY  | InputOption::VALUE_REQUIRED, 'Override config values'),
-            new InputOption('ext', 'e', InputOption::VALUE_IS_ARRAY  | InputOption::VALUE_REQUIRED, 'Run with extension enabled'),
+            new InputOption('override', 'o', InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED, 'Override config values'),
+            new InputOption('ext', 'e', InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED, 'Run with extension enabled'),
             new InputOption('report', '', InputOption::VALUE_NONE, 'Show output in compact style'),
             new InputOption('html', '', InputOption::VALUE_OPTIONAL, 'Generate html with results', 'report.html'),
             new InputOption('xml', '', InputOption::VALUE_OPTIONAL, 'Generate JUnit XML Log', 'report.xml'),
@@ -241,7 +241,7 @@ 
         }
         if (!$this->options['silent']) {
             $this->output->writeln(
-                Codecept::versionString() . "\nPowered by " . \PHPUnit\Runner\Version::getVersionString()
+                Codecept::versionString()."\nPowered by ".\PHPUnit\Runner\Version::getVersionString()
             );
         }
         if ($this->options['debug']) {
@@ -315,7 +315,7 @@ 
                             );
                         }
 
-                        $testsPath = $include . DIRECTORY_SEPARATOR.  $config['paths']['tests'];
+                        $testsPath = $include.DIRECTORY_SEPARATOR.$config['paths']['tests'];
 
                         try {
                             list(, $suite, $test) = $this->matchTestFromFilename($suite, $testsPath);
@@ -387,7 +387,7 @@ 
         // Workaround when codeception.yml is inside tests directory and tests path is set to "."
         // @see https://github.com/Codeception/Codeception/issues/4432
         if ($config['paths']['tests'] === '.' && !preg_match('~^\.[/\\\]~', $suite)) {
-            $suite = './' . $suite;
+            $suite = './'.$suite;
         }
 
         // running a single test when suite has a configured path
@@ -396,7 +396,7 @@ 
                 if (!isset($suiteConfig['path'])) {
                     continue;
                 }
-                $testsPath = $config['paths']['tests'] . DIRECTORY_SEPARATOR . $suiteConfig['path'];
+                $testsPath = $config['paths']['tests'].DIRECTORY_SEPARATOR.$suiteConfig['path'];
                 if ($suiteConfig['path'] === '.') {
                     $testsPath = $config['paths']['tests'];
                 }
@@ -409,7 +409,7 @@ 
         }
 
         // Run single test without included tests
-        if (! Configuration::isEmpty() && strpos($suite, $config['paths']['tests']) === 0) {
+        if (!Configuration::isEmpty() && strpos($suite, $config['paths']['tests']) === 0) {
             return $this->matchTestFromFilename($suite, $config['paths']['tests']);
         }
     }
@@ -423,7 +423,7 @@ 
     protected function runIncludedSuites($suites, $parent_dir)
     {
         foreach ($suites as $relativePath) {
-            $current_dir = rtrim($parent_dir, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . $relativePath;
+            $current_dir = rtrim($parent_dir, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR.$relativePath;
             $config = Configuration::config($current_dir);
             $suites = Configuration::suites();