Codeception / Codeception

src/Codeception/Lib/Generator/Test.php

Spacing +2 added lines, -2 removed lines

@@ -54,10 +54,10 @@
     {
         $actor = $this->settings['actor'];
         if ($this->settings['namespace']) {
-            $actor = $this->settings['namespace'] . '\\' . $actor;
+            $actor = $this->settings['namespace'].'\\'.$actor;
         }
 
-        $ns = $this->getNamespaceHeader($this->settings['namespace'] . '\\' . $this->name);
+        $ns = $this->getNamespaceHeader($this->settings['namespace'].'\\'.$this->name);
 
         $tester = '';
         if ($this->settings['actor']) {

src/Codeception/Module/REST.php

Spacing +13 added lines, -13 removed lines

@@ -302,7 +302,7 @@ 
      */
     public function amBearerAuthenticated($accessToken)
     {
-        $this->haveHttpHeader('Authorization', 'Bearer ' . $accessToken);
+        $this->haveHttpHeader('Authorization', 'Bearer '.$accessToken);
     }
 
     /**
@@ -524,7 +524,7 @@ 
                 $linkEntry,
                 'linkEntry should contain property "link-param"'
             );
-            $values[] = $linkEntry['uri'] . '; ' . $linkEntry['link-param'];
+            $values[] = $linkEntry['uri'].'; '.$linkEntry['link-param'];
         }
 
         $this->haveHttpHeader('Link', implode(', ', $values));
@@ -568,9 +568,9 @@ 
     {
         // allow full url to be requested
         if (strpos($url, '://') === false) {
-            $url = $this->config['url'] . $url;
+            $url = $this->config['url'].$url;
             if ($this->config['url'] && strpos($url, '://') === false && $this->config['url'][0] !== '/') {
-                $url = '/' . $url;
+                $url = '/'.$url;
             }
         }
 
@@ -591,7 +591,7 @@ 
                 $this->debugSection("Request", "$method $url");
                 $files = [];
             } else {
-                $this->debugSection("Request", "$method $url " . json_encode($parameters));
+                $this->debugSection("Request", "$method $url ".json_encode($parameters));
                 $files = $this->formatFilesArray($files);
             }
             $this->response = (string)$this->connectionModule->_request($method, $url, $parameters, $files);
@@ -600,7 +600,7 @@ 
             if ($this->isBinaryData($requestData)) {
                 $requestData = $this->binaryToDebugString($requestData);
             }
-            $this->debugSection("Request", "$method $url " . $requestData);
+            $this->debugSection("Request", "$method $url ".$requestData);
             $this->response = (string)$this->connectionModule->_request($method, $url, [], $files, [], $parameters);
         }
         $printedResponse = $this->response;
@@ -629,7 +629,7 @@ 
      */
     protected function binaryToDebugString($data)
     {
-        return '[binary-data length:' . strlen($data) . ' md5:' . md5($data) . ']';
+        return '[binary-data length:'.strlen($data).' md5:'.md5($data).']';
     }
 
     protected function encodeApplicationJson($method, $parameters)
@@ -894,7 +894,7 @@ 
         $this->assertGreaterThan(
             0,
             (new JsonArray($response))->filterByXPath($xpath)->length,
-            "Received JSON did not match the XPath `$xpath`.\nJson Response: \n" . $response
+            "Received JSON did not match the XPath `$xpath`.\nJson Response: \n".$response
         );
     }
 
@@ -910,7 +910,7 @@ 
         $this->assertEquals(
             0,
             (new JsonArray($response))->filterByXPath($xpath)->length,
-            "Received JSON matched the XPath `$xpath`.\nJson Response: \n" . $response
+            "Received JSON matched the XPath `$xpath`.\nJson Response: \n".$response
         );
     }
 
@@ -964,7 +964,7 @@ 
         $response = $this->connectionModule->_getResponseContent();
         $this->assertNotEmpty(
             (new JsonArray($response))->filterByJsonPath($jsonPath),
-            "Received JSON did not match the JsonPath `$jsonPath`.\nJson Response: \n" . $response
+            "Received JSON did not match the JsonPath `$jsonPath`.\nJson Response: \n".$response
         );
     }
 
@@ -979,7 +979,7 @@ 
         $response = $this->connectionModule->_getResponseContent();
         $this->assertEmpty(
             (new JsonArray($response))->filterByJsonPath($jsonPath),
-            "Received JSON matched the JsonPath `$jsonPath`.\nJson Response: \n" . $response
+            "Received JSON matched the JsonPath `$jsonPath`.\nJson Response: \n".$response
         );
     }
 
@@ -995,8 +995,8 @@ 
         $this->assertFalse(
             $jsonResponseArray->containsArray($json),
             "Response JSON contains provided JSON\n"
-            . "- <info>" . var_export($json, true) . "</info>\n"
-            . "+ " . var_export($jsonResponseArray->toArray(), true)
+            . "- <info>".var_export($json, true)."</info>\n"
+            . "+ ".var_export($jsonResponseArray->toArray(), true)
         );
     }
 

src/Codeception/Suite.php

Doc Comments +3 added lines

@@ -28,6 +28,9 @@
         $this->tests = $queue;
     }
 
+    /**
+     * @param \PHPUnit\Framework\TestCase $test
+     */
     protected function getDependencies($test)
     {
         if (!$test instanceof Dependent) {

src/Codeception/Test/Descriptor.php

Doc Comments +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
      * Provides a test file name relative to Codeception root
      *
      * @param \PHPUnit\Framework\SelfDescribing $testCase
-     * @return mixed
+     * @return string
      */
     public static function getTestFileName(\PHPUnit\Framework\SelfDescribing $testCase)
     {
@@ -74,7 +74,7 @@ 
 
     /**
      * @param \PHPUnit\Framework\SelfDescribing $testCase
-     * @return mixed|string
+     * @return string
      */
     public static function getTestFullName(\PHPUnit\Framework\SelfDescribing $testCase)
     {

Spacing +7 added lines, -7 removed lines

@@ -19,7 +19,7 @@ 
             return $testCase->getSignature();
         }
         if ($testCase instanceof \PHPUnit\Framework\TestCase) {
-            return get_class($testCase) . ':' . $testCase->getName(false);
+            return get_class($testCase).':'.$testCase->getName(false);
         }
         return $testCase->toString();
     }
@@ -37,10 +37,10 @@ 
         if (is_callable([$testCase, 'getMetadata'])
             && $example = $testCase->getMetadata()->getCurrent('example')
         ) {
-            $example = ':' . substr(sha1(json_encode($example)), 0, 7);
+            $example = ':'.substr(sha1(json_encode($example)), 0, 7);
         }
 
-        return self::getTestSignature($testCase) . $example;
+        return self::getTestSignature($testCase).$example;
     }
 
     public static function getTestAsString(\PHPUnit\Framework\SelfDescribing $testCase)
@@ -52,7 +52,7 @@ 
             $text = preg_replace('/^test /', '', $text);
             $text = ucfirst(strtolower($text));
             $text = str_replace(['::', 'with data set'], [':', '|'], $text);
-            return ReflectionHelper::getClassShortName($testCase) . ': ' . $text;
+            return ReflectionHelper::getClassShortName($testCase).': '.$text;
         }
 
         return $testCase->toString();
@@ -83,12 +83,12 @@ 
         }
         if ($testCase instanceof Descriptive) {
             $signature = $testCase->getSignature(); // cut everything before ":" from signature
-            return self::getTestFileName($testCase) . ':' . preg_replace('~^(.*?):~', '', $signature);
+            return self::getTestFileName($testCase).':'.preg_replace('~^(.*?):~', '', $signature);
         }
         if ($testCase instanceof \PHPUnit\Framework\TestCase) {
-            return self::getTestFileName($testCase) . ':' . $testCase->getName(false);
+            return self::getTestFileName($testCase).':'.$testCase->getName(false);
         }
-        return self::getTestFileName($testCase) . ':' . $testCase->toString();
+        return self::getTestFileName($testCase).':'.$testCase->toString();
     }
 
     /**

autoload.php

Spacing +9 added lines, -9 removed lines

@@ -11,18 +11,18 @@ 
     //require package/bin instead of codecept to avoid printing hashbang line
     require './vendor/codeception/codeception/package/bin';
     die;
-} elseif (file_exists(__DIR__ . '/vendor/autoload.php')) {
+} elseif (file_exists(__DIR__.'/vendor/autoload.php')) {
     // for phar
-    require_once(__DIR__ . '/vendor/autoload.php');
-} elseif (file_exists(__DIR__ . '/../../autoload.php')) {
+    require_once(__DIR__.'/vendor/autoload.php');
+} elseif (file_exists(__DIR__.'/../../autoload.php')) {
     //for composer
-    require_once __DIR__ . '/../../autoload.php';
+    require_once __DIR__.'/../../autoload.php';
 }
 unset($autoloadFile);
 
 // @codingStandardsIgnoreStart
 
-include_once __DIR__ . DIRECTORY_SEPARATOR . 'shim.php';
+include_once __DIR__.DIRECTORY_SEPARATOR.'shim.php';
 // compat
 if (PHP_MAJOR_VERSION < 7) {
     if (false === interface_exists('Throwable', false)) {
@@ -66,28 +66,28 @@ 
 if (!function_exists('codecept_root_dir')) {
     function codecept_root_dir($appendPath = '')
     {
-        return \Codeception\Configuration::projectDir() . $appendPath;
+        return \Codeception\Configuration::projectDir().$appendPath;
     }
 }
 
 if (!function_exists('codecept_output_dir')) {
     function codecept_output_dir($appendPath = '')
     {
-        return \Codeception\Configuration::outputDir() . $appendPath;
+        return \Codeception\Configuration::outputDir().$appendPath;
     }
 }
 
 if (!function_exists('codecept_log_dir')) {
     function codecept_log_dir($appendPath = '')
     {
-        return \Codeception\Configuration::outputDir() . $appendPath;
+        return \Codeception\Configuration::outputDir().$appendPath;
     }
 }
 
 if (!function_exists('codecept_data_dir')) {
     function codecept_data_dir($appendPath = '')
     {
-        return \Codeception\Configuration::dataDir() . $appendPath;
+        return \Codeception\Configuration::dataDir().$appendPath;
     }
 }
 

src/Codeception/Util/Shared/Asserts.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
             $not = '';
         }
 
-        call_user_func_array(['\PHPUnit\Framework\Assert', 'assert' . $not . $method], $arguments);
+        call_user_func_array(['\PHPUnit\Framework\Assert', 'assert'.$not.$method], $arguments);
     }
 
     protected function assertNot($arguments)

src/Codeception/Module/Cli.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
             \PHPUnit\Framework\Assert::fail("$command can't be executed");
         }
         if ($resultCode !== 0 && $failNonZero) {
-            \PHPUnit\Framework\Assert::fail("Result code was $resultCode.\n\n" . $this->output);
+            \PHPUnit\Framework\Assert::fail("Result code was $resultCode.\n\n".$this->output);
         }
         $this->debug(preg_replace('~s/\e\[\d+(?>(;\d+)*)m//g~', '', $this->output));
     }

src/Codeception/Command/Console.php

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
             throw new ConfigurationException("Interactive shell can't be started without an actor");
         }
         if (isset($config["namespace"])) {
-            $settings['actor'] = $config["namespace"] .'\\' . $settings['actor'];
+            $settings['actor'] = $config["namespace"].'\\'.$settings['actor'];
         }
         $actor = $settings['actor'];
         $I = new $actor($scenario);
@@ -126,7 +126,7 @@ 
 
             $command = $dialog->ask($input, $output, $question);
             if ($command == 'actions') {
-                $output->writeln("<info>" . implode(' ', $this->actions));
+                $output->writeln("<info>".implode(' ', $this->actions));
                 continue;
             }
             if ($command == 'exit') {
@@ -141,9 +141,9 @@ 
                     codecept_debug($value);
                 }
             } catch (\PHPUnit\Framework\AssertionFailedError $fail) {
-                $output->writeln("<error>fail</error> " . $fail->getMessage());
+                $output->writeln("<error>fail</error> ".$fail->getMessage());
             } catch (\Exception $e) {
-                $output->writeln("<error>error</error> " . $e->getMessage());
+                $output->writeln("<error>error</error> ".$e->getMessage());
             }
         } while (true);
     }

src/Codeception/Lib/GroupManager.php

Spacing +6 added lines, -6 removed lines

@@ -62,10 +62,10 @@ 
             if (is_array($tests)) {
                 foreach ($tests as $test) {
                     $file = str_replace(['/', '\\'], [DIRECTORY_SEPARATOR, DIRECTORY_SEPARATOR], $test);
-                    $this->testsInGroups[$group][] = Configuration::projectDir() . $file;
+                    $this->testsInGroups[$group][] = Configuration::projectDir().$file;
                 }
-            } elseif (is_file(Configuration::projectDir() . $tests)) {
-                $handle = @fopen(Configuration::projectDir() . $tests, "r");
+            } elseif (is_file(Configuration::projectDir().$tests)) {
+                $handle = @fopen(Configuration::projectDir().$tests, "r");
                 if ($handle) {
                     while (($test = fgets($handle, 4096)) !== false) {
                         // if the current line is blank then we need to move to the next line
@@ -75,7 +75,7 @@ 
                             continue;
                         }
 
-                        $file = trim(Configuration::projectDir() . $test);
+                        $file = trim(Configuration::projectDir().$test);
                         $file = str_replace(['/', '\\'], [DIRECTORY_SEPARATOR, DIRECTORY_SEPARATOR], $file);
                         $this->testsInGroups[$group][] = $file;
                     }
@@ -115,13 +115,13 @@ 
                 if ($filename == $testPattern) {
                     $groups[] = $group;
                 }
-                if (strpos($filename . ':' . $test->getName(false), $testPattern) === 0) {
+                if (strpos($filename.':'.$test->getName(false), $testPattern) === 0) {
                     $groups[] = $group;
                 }
                 if ($test instanceof \PHPUnit\Framework\TestSuite\DataProvider) {
                     $firstTest = $test->testAt(0);
                     if ($firstTest != false && $firstTest instanceof TestInterface) {
-                        if (strpos($filename . ':' . $firstTest->getName(false), $testPattern) === 0) {
+                        if (strpos($filename.':'.$firstTest->getName(false), $testPattern) === 0) {
                             $groups[] = $group;
                         }
                     }