consolidation / robo

tests/src/Traits/TestTasksTrait.php

Doc Comments +16 added lines

@@ -17,6 +17,9 @@ 
     protected $capturedOutput;
     protected $logger;
 
+    /**
+     * @return \League\Container\ContainerInterface
+     */
     public function initTestTasksTrait($commandClass = null, $container = null, $input = null)
     {
         if (!$container) {
@@ -47,6 +50,9 @@ 
         return $container;
     }
 
+    /**
+     * @return null|OutputInterface
+     */
     public function capturedOutputStream()
     {
         if (!$this->testPrinter) {
@@ -66,6 +72,9 @@ 
         return $this->capturedOutput;
     }
 
+    /**
+     * @param string $value
+     */
     public function assertOutputContains($value)
     {
         $output = $this->accumulate();
@@ -73,6 +82,9 @@ 
         $this->assertContains($value, $output);
     }
 
+    /**
+     * @param string $value
+     */
     public function assertOutputNotContains($value)
     {
         $output = $this->accumulate();
@@ -80,6 +92,9 @@ 
         $this->assertNotContains($value, $output);
     }
 
+    /**
+     * @param string $value
+     */
     public function assertOutputEquals($value)
     {
         $output = $this->accumulate();
@@ -90,6 +105,7 @@ 
     /**
      * Make our output comparisons more platform-agnostic by converting
      * CRLF (Windows) or raw CR (confused output) to a LF (unix/Mac).
+     * @param string $output
      */
     protected function simplify($output)
     {

tests/src/Fixtures.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      */
     public function mktmpdir($basedir = false)
     {
-        $tempfile = tempnam($basedir ?: $this->testDir ?: sys_get_temp_dir(),'robo-tests');
+        $tempfile = tempnam($basedir ?: $this->testDir ?: sys_get_temp_dir(), 'robo-tests');
         unlink($tempfile);
         mkdir($tempfile);
         $this->tmpDirs[] = $tempfile;

Doc Comments +4 added lines, -1 removed lines

@@ -39,7 +39,7 @@ 
      * Create a new temporary directory.
      *
      * @param string|bool $basedir Where to store the temporary directory
-     * @return type
+     * @return string
      */
     public function mktmpdir($basedir = false)
     {
@@ -61,6 +61,9 @@ 
         return $targetSandbox;
     }
 
+    /**
+     * @param string $filename
+     */
     public function dataFile($filename)
     {
         return $this->fixturesDir() . '/' . $filename;

src/Task/File/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 trait loadTasks
 {
     /**
-     * @param array|\Iterator $files
+     * @param string[] $files
      *
      * @return \Robo\Task\File\Concat|\Robo\Collection\CollectionBuilder
      */

src/Task/Filesystem/FlattenDir.php

Doc Comments +2 added lines, -2 removed lines

@@ -136,7 +136,7 @@ 
      * Sets the value from which direction and how much parent dirs should be included.
      * Accepts a positive or negative integer or an array with two integer values.
      *
-     * @param int|int[] $parents
+     * @param integer[] $parents
      *
      * @return $this
      *
@@ -198,7 +198,7 @@ 
     }
 
     /**
-     * @param array $dirs
+     * @param string[] $dirs
      *
      * @return array|\Robo\Result
      *

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
      */
     public function dirPermissions($permission)
     {
-        $this->chmod = (int) $permission;
+        $this->chmod = (int)$permission;
 
         return $this;
     }

src/Task/Filesystem/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 trait loadTasks
 {
     /**
-     * @param string|string[] $dirs
+     * @param string[] $dirs
      *
      * @return \Robo\Task\Filesystem\CleanDir|\Robo\Collection\CollectionBuilder
      */

tests/functional/FlattenDirTest.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         $this->fixtures->createAndCdToSandbox();
 
         $result = $this->taskFlattenDir('some/deeply/nested/*.re')
-            ->includeParents([1,1])
+            ->includeParents([1, 1])
             ->parentDir('some')
             ->to('flattened')
             ->run();

tests/functional/ExecTest.php

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $result = $task->run();
         $this->assertTrue($result->wasSuccessful());
-        $start_count = (int) $result->getMessage();
+        $start_count = (int)$result->getMessage();
         $this->assertGreaterThan(0, $start_count);
 
         // Verify that we get the same amount of environment variables with
@@ -78,7 +78,7 @@ 
         $this->assertTrue($result->wasSuccessful());
         $this->assertEquals(
             $start_count,
-            (int) $result->getMessage());
+            (int)$result->getMessage());
 
         // Now run the same command, but this time add another environment
         // variable, and see if our count increases by one.
@@ -86,6 +86,6 @@ 
         $task->env('FOO', 'BAR');
         $result = $task->run();
         $this->assertTrue($result->wasSuccessful());
-        $this->assertEquals($start_count + 1, (int) $result->getMessage());
+        $this->assertEquals($start_count + 1, (int)$result->getMessage());
     }
 }

src/Common/ProcessUtils.php

Doc Comments +3 added lines

@@ -74,6 +74,9 @@
         return "'" . str_replace("'", "'\\''", $argument) . "'";
     }
 
+    /**
+     * @param string $char
+     */
     private static function isSurroundedBy($arg, $char)
     {
         return 2 < strlen($arg) && $char === $arg[0] && $char === $arg[strlen($arg) - 1];

src/Runner.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     public function __construct($roboClass = null, $roboFile = null)
     {
         // set the const as class properties to allow overwriting in child classes
-        $this->roboClass = $roboClass ? $roboClass : self::ROBOCLASS ;
+        $this->roboClass = $roboClass ? $roboClass : self::ROBOCLASS;
         $this->roboFile  = $roboFile ? $roboFile : self::ROBOFILE;
         $this->dir = getcwd();
     }