VisualPHPUnit / VisualPHPUnit

src/Console/Application/Vpu.php

Doc Comments +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      * Overridden so that the application doesn't expect the command
      * name to be the first argument.
      *
-     * @return InputDefinition
+     * @return \Symfony\Component\Console\Input\InputDefinition
      */
     public function getDefinition()
     {

src/Core/Parser.php

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
     /**
      * Require bootstrap if vpu can find it
      *
-     * @param array $tests
+     * @param string[] $tests
      *
      * @return void
      */
@@ -189,7 +189,7 @@ 
     /**
      * Convert camelCase to friendly name
      *
-     * @param sreing $camelCaseString
+     * @param string $camelCaseString
      *
      * @return string
      */

Spacing +15 added lines, -15 removed lines

@@ -56,13 +56,13 @@ 
                 $case2 = strpos($filename, 'Tests');
                 
                 if (is_numeric($case1)) {
-                    $path = substr($filename, 0, $case1 + 6) . 'bootstrap.php';
+                    $path = substr($filename, 0, $case1 + 6).'bootstrap.php';
                     if (file_exists($path)) {
                         require_once $path;
                     }
                 }
                 if (is_numeric($case2)) {
-                    $path = substr($filename, 0, $case2 + 6) . 'bootstrap.php';
+                    $path = substr($filename, 0, $case2 + 6).'bootstrap.php';
                     if (file_exists($path)) {
                         require_once $path;
                     }
@@ -88,26 +88,26 @@ 
         $tests = [];
         foreach ($result->passed() as $key => $value) {
             $tests[] = $this->parseTest('passed', $key);
-            $passed ++;
+            $passed++;
         }
         foreach ($result->failures() as $obj) {
             $tests[] = $this->parseTest('failed', $obj);
-            $failed ++;
+            $failed++;
         }
         foreach ($result->skipped() as $obj) {
             $tests[] = $this->parseTest('skipped', $obj);
-            $skipped ++;
+            $skipped++;
         }
         foreach ($result->notImplemented() as $obj) {
             $tests[] = $this->parseTest('notImplemented', $obj);
-            $notImplemented ++;
+            $notImplemented++;
         }
         foreach ($result->errors() as $obj) {
             $tests[] = $this->parseTest('error', $obj);
-            $error ++;
+            $error++;
         }
         
-        usort($tests, function ($a, $b) {
+        usort($tests, function($a, $b) {
             return strnatcmp($a['class'], $b['class']);
         });
         
@@ -132,17 +132,17 @@ 
      */
     private function filterTrace($trace)
     {
-        $vpuPath = realpath(__DIR__ . '/../');
-        $vendorPath = realpath(__DIR__ . '/../../vendor');
-        $backendPath = realpath(__DIR__ . '/../../backend');
+        $vpuPath = realpath(__DIR__.'/../');
+        $vendorPath = realpath(__DIR__.'/../../vendor');
+        $backendPath = realpath(__DIR__.'/../../backend');
         
         $newTrace = [];
-        if (! empty($trace)) {
+        if (!empty($trace)) {
             foreach ($trace as $entity) {
                 if (isset($entity['file'])
-                    && ! strstr($entity['file'], $vendorPath)
-                    && ! strstr($entity['file'], $vpuPath)
-                    && ! strstr($entity['file'], $backendPath)) {
+                    && !strstr($entity['file'], $vendorPath)
+                    && !strstr($entity['file'], $vpuPath)
+                    && !strstr($entity['file'], $backendPath)) {
                     $newTrace[] = $entity;
                 }
             }

src/Core/Test.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
      * @param \Doctrine\DBAL\Connection $connection
      * @param mixed[] $result
      *
-     * @return boolean
+     * @return boolean|null
      */
     public static function store(Connection $connection, $result)
     {

Spacing +10 added lines, -10 removed lines

@@ -137,53 +137,53 @@
         $sql = "
             SELECT
             strftime('%Y-%m-%d %H:%M:%S', executed) as datetime,
-            ltrim(strftime('" . $unit . "', executed), '0') as unit,
+            ltrim(strftime('" . $unit."', executed), '0') as unit,
             COUNT() as number,
             status
             FROM tests
             WHERE executed BETWEEN datetime(?) AND datetime(?)
             AND status = 'passed'
-            GROUP BY strftime('" . $unit . "', executed)
+            GROUP BY strftime('" . $unit."', executed)
             UNION
             SELECT
             strftime('%Y-%m-%d %H:%M:%S', executed) as datetime,
-            ltrim(strftime('" . $unit . "', executed), '0') as unit,
+            ltrim(strftime('" . $unit."', executed), '0') as unit,
             COUNT() as number,
             status
             FROM tests
             WHERE executed BETWEEN datetime(?) AND datetime(?)
             AND status = 'failed'
-            GROUP BY strftime('" . $unit . "', executed)
+            GROUP BY strftime('" . $unit."', executed)
             UNION
             SELECT
             strftime('%Y-%m-%d %H:%M:%S', executed) as datetime,
-            ltrim(strftime('" . $unit . "', executed), '0') as unit,
+            ltrim(strftime('" . $unit."', executed), '0') as unit,
             COUNT() as number,
             status
             FROM tests
             WHERE executed BETWEEN datetime(?) AND datetime(?)
             AND status = 'notImplemented'
-            GROUP BY strftime('" . $unit . "', executed)
+            GROUP BY strftime('" . $unit."', executed)
             UNION
             SELECT
             strftime('%Y-%m-%d %H:%M:%S', executed) as datetime,
-            ltrim(strftime('" . $unit . "', executed), '0') as unit,
+            ltrim(strftime('" . $unit."', executed), '0') as unit,
             COUNT() as number,
             status
             FROM tests
             WHERE executed BETWEEN datetime(?) AND datetime(?)
             AND status = 'skipped'
-            GROUP BY strftime('" . $unit . "', executed)
+            GROUP BY strftime('" . $unit."', executed)
             UNION
             SELECT
             strftime('%Y-%m-%d %H:%M:%S', executed) as datetime,
-            ltrim(strftime('" . $unit . "', executed), '0') as unit,
+            ltrim(strftime('" . $unit."', executed), '0') as unit,
             COUNT() as number,
             status
             FROM tests
             WHERE executed BETWEEN datetime(?) AND datetime(?)
             AND status = 'error'
-            GROUP BY strftime('" . $unit . "', executed)
+            GROUP BY strftime('" . $unit."', executed)
             ORDER BY status";
         
         $stmt = $connection->prepare($sql);

tests/ExpectedFailureTest.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      */
     public function skipThisTest()
     {
-        if (! extension_loaded('something_bogus')) {
+        if (!extension_loaded('something_bogus')) {
             $this->markTestSkipped('The something_bogus extension is not available.');
         }
     }

tests/bootstrap.php

Spacing +2 added lines, -2 removed lines

@@ -13,6 +13,6 @@
  */
 
 //You need to update the function name if you want to run vpu's own tests via vpu.phar
-if (! function_exists('composerRequire6f35294d06ff43eeb876443f450df6c4')) {
-    require_once realpath(__DIR__ . '/../vendor/autoload.php');
+if (!function_exists('composerRequire6f35294d06ff43eeb876443f450df6c4')) {
+    require_once realpath(__DIR__.'/../vendor/autoload.php');
 }

src/Api/Application/Vpu.php

Spacing +2 added lines, -2 removed lines

@@ -36,13 +36,13 @@
         $app = $this;
         $app['debug'] = true;
         
-        $appRoot = realpath(__DIR__ . '/../../..');
+        $appRoot = realpath(__DIR__.'/../../..');
         $app->register(new ConfigServiceProvider("../vpu.json"));
         
         $app->register(new DoctrineServiceProvider(), array(
             'db.options' => array(
                 'driver' => $app['config']['database']['driver'],
-                'path' => $appRoot . '/vpu.db',
+                'path' => $appRoot.'/vpu.db',
             )
         ));
         

src/Api/Action/Test.php

Spacing +3 added lines, -3 removed lines

@@ -130,7 +130,7 @@ 
     private function isPhpUnitTestCase($path)
     {
         $result1 = preg_grep('/PHPUnit_Framework_TestCase$/', file($path));
-        if (! empty($result1)) {
+        if (!empty($result1)) {
             return true;
         }
         return false;
@@ -159,10 +159,10 @@ 
             preg_match('/^class\s([A-Za-z0-9]+).+$/', array_pop($result2), $matches2);
             $class = $matches2[1];
             require_once $path;
-            $obj = new ReflectionClass($namespace . '\\' . $class);
+            $obj = new ReflectionClass($namespace.'\\'.$class);
             $methods = [];
             foreach ($obj->getMethods() as $method) {
-                if ($method->class == $namespace . '\\' . $class) {
+                if ($method->class == $namespace.'\\'.$class) {
                     if ($method->isPublic()) {
                         $methods[] = $method->name;
                     }

src/Console/Command/Run.php

Spacing +6 added lines, -6 removed lines

@@ -101,7 +101,7 @@ 
             $this->stop($output);
             $output->writeln('<comment>VPU stopped</comment>');
         } else {
-            if (! empty($input->getArgument('files'))) {
+            if (!empty($input->getArgument('files'))) {
                 $parser = new Parser();
                 $result = $parser->run($input->getArgument('files'));
                 Test::createTable($this->getDbConnection());
@@ -131,12 +131,12 @@ 
                 'php -S %s:%d -t %s >/dev/null 2>&1 & echo $!',
                 $config['host'],
                 $config['port'],
-                $this->appRoot . '/'.$config['docroot']
+                $this->appRoot.'/'.$config['docroot']
             );
             $output = [];
             exec($cmd, $output);
             $pid = (int) $output[0];
-            file_put_contents($this->appRoot . '/' . $server . '.pid', $pid);
+            file_put_contents($this->appRoot.'/'.$server.'.pid', $pid);
         }
     }
 
@@ -148,9 +148,9 @@ 
     private function stop()
     {
         foreach ($this->serverConfig as $server => $config) {
-            $path = $this->appRoot . '/' . $server . '.pid';
+            $path = $this->appRoot.'/'.$server.'.pid';
             if (file_exists($path)) {
-                exec('kill ' . file_get_contents($path));
+                exec('kill '.file_get_contents($path));
                 unlink($path);
             }
         }
@@ -167,7 +167,7 @@ 
     {
         $config = $this->config;
         $connectionParams = array(
-            'path' => $this->appRoot . '/vpu.db',
+            'path' => $this->appRoot.'/vpu.db',
             'driver' => $config['config']['database']['driver']
         );
         return DriverManager::getConnection($connectionParams, new Configuration());

backend/index.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  * @license http://opensource.org/licenses/BSD-3-Clause The BSD License
  * @link https://github.com/VisualPHPUnit/VisualPHPUnit VisualPHPUnit
  */
-require __DIR__ . '/../vendor/autoload.php';
+require __DIR__.'/../vendor/autoload.php';
 use Visualphpunit\Api\Application\Vpu;
 
 $app = new Vpu();